IDempiere/FullMeeting20131204 - Revision history

CarlosRuiz: drop JIRA notifications from log

2013-12-04T19:57:54Z

drop JIRA notifications from log

CarlosRuiz: full meeting

2013-12-04T19:57:15Z

full meeting

New page

⇐
[[IDempiere|Table of Contents]] |
[[IDempiere/Full Meeting Minutes|Full Meeting Minutes]] |
Full Meeting 2013-12-04


'''''CarlosRuiz''''': Good Morning 
'''''nmicoud''''': Bonjour 
'''''nmicoud''''': Found and reported some bugs this week :) ; can we deal with 1587 ? 
'''''ocurieles''''': Good Morning Srs 
'''''CarlosRuiz''''': Hi Orlando 
'''''nmicoud''''': Hi 
'''''CarlosRuiz''''': sure nmicoud, please let me integrate some commits from hengsin first ... 
'''''nmicoud''''': sure 
'''''ocurieles_''''': @JanThielemann Nice Asterisk release... 
'''''JanThielemann''''': please test and report bugs :) 
'''''JanThielemann''''': i know there are some of them xD 
'''''JanThielemann''''': or at least i cannot imagine that there are none *g* 
'''''CarlosRuiz''''': yes, JanThielemann and jmpiloq - thanks for the contribution! it looks great 
'''''ocurieles_''''': Sure.. i have a asterisk in my office... 
'''''JanThielemann''''': improvement suggestions are welcome :) 
'''''ocurieles_''''': please check this https://bitbucket.org/dcs_bitbucket/attachmentscanner 
'''''ocurieles_''''': for scanning directly from idempiere as attachment 
'''''ocurieles_''''': work with sane 
'''''aguerra''''': Hi everybody!!!! 
'''''ocurieles_''''': Hi @aguerra 
'''''nmicoud''''': it's xmas ! congrats ! 
'''''CarlosRuiz''''': Hi Alejandro 
'''''CarlosRuiz''''': cybermonday contribs :-) 
'''''tbayen''''': Hi 
'''''ocurieles_''''': :D 
'''''CarlosRuiz''''': hi Thomas 
'''''nmicoud''''': hi thomas 
'''''ocurieles_''''': Hi Thomas 
'''''aguerra''''': Hi Thomas 
'''''aguerra''''': Hi @ocurieles_ 
'''''aguerra''''': hi @CarlosRuiz 
'''''ocurieles_''''': i put the scanning images on Facebook https://www.facebook.com/groups/idempiere/ 
'''''ocurieles_''''': somebody know a good irc client for Mac ? 
'''''ocurieles_''''': @JanThielemann maybe you can put some pictures of Asterisk integration on Facebook :D 
'''''JanThielemann''''': limechat 
'''''JanThielemann''''': sure 
'''''tbayen''''': Better make a wikipage with some pictures. It will last longer than facebook. 
'''''ocurieles_''''': sure i promess work the next days in the wiki plugins pages :( 
'''''hengsin''''': I think we should make a page in wiki to list extensions available 
'''''ocurieles_''''': i will work on that i pro mess :D 
'''''CarlosRuiz''''': advance on that task was this: 
'''''tbayen''''': Someone wanted to create a "plugin shop" page in the wiki. But until someone does that a Category "Plugins" could be sufficient. 
'''''CarlosRuiz''''': http://wiki.idempiere.org/en/Category:AvailablePlugins 
'''''CarlosRuiz''''': and installation of the rating plugin as shown here 
'''''CarlosRuiz''''': http://wiki.idempiere.org/en/User:TBayen 
'''''CarlosRuiz''''': so, I think is just to start filling pages and assigning that category 
'''''ocurieles_''''': sure 
'''''CarlosRuiz''''': nmicoud, finished the integration 
'''''nmicoud''''': ok 
'''''nmicoud''''': so, i think there is a bug as is wrote in my 2nd comment and 'old' columns should be updated 
'''''CarlosRuiz''''': yep - so, to generate the migration script we need to fix first the code 
'''''CarlosRuiz''''': right? 
'''''nmicoud''''': yep 
'''''ocurieles_DCS''''': testing limechat 
'''''nmicoud''''': about the fix, what is the interest to have CHAR with length > 1 ? 
'''''CarlosRuiz''''': AFAIR - char columns are just problematic in lists 
'''''nmicoud''''': that would explain why docaction is set as Char 
'''''hengsin''''': updating the old columns from char to nvarchat will break many views though 
'''''ocurieles_DCS''''': Hi Mr @hengsin some news about https://idempiere.atlassian.net/browse/IDEMPIERE-1531 ? 
'''''ocurieles_DCS''''': the field New Accumulated Amt is for accumulated inventory cost or accumulated inventory value 
'''''ocurieles_DCS''''': ? 
'''''CarlosRuiz''''': nmicoud, this query is showing 55 different columns 
'''''CarlosRuiz''''': select column_name, data_length, count(*) from user_tab_columns 
'''''CarlosRuiz''''': where data_type='CHAR' and data_length>1 
'''''CarlosRuiz''''': group by column_name, data_length 
'''''CarlosRuiz''''': order by 3 desc, 1 
'''''CarlosRuiz''''': -> oracle query 
'''''nmicoud''''': yep ; that's a lot :-/ 
'''''nmicoud''''': should be update all of them ? 
'''''CarlosRuiz''''': or maybe we add an exception for DocAction and DocStatus 
'''''nmicoud''''': only for docaction 
'''''CarlosRuiz''''': ah - because is a button 
'''''nmicoud''''': new column for docstatus are ok 
'''''nmicoud''''': yes 
'''''nmicoud''''': only 'old' docstatus need to be updated 
'''''CarlosRuiz''''': no, the opposite 
'''''nmicoud''''': yes :D 
'''''hengsin''''': dcs, that's to accumulate inventory cost 
'''''CarlosRuiz''''': nmicoud, AD_LANGUAGE - we have 17 CHAR and 93 VARCHAR2 
'''''CarlosRuiz''''': :-) worst -> 17 char(5), 93 varchar2(6) 
'''''nmicoud''''': should we laugh or cry ? 
'''''CarlosRuiz''''': so, I think the problem you found is not because char<>varchar - but because of the N on nvarchar 
'''''CarlosRuiz''''': and would be just oracle related 
'''''nmicoud''''': those type are not available on postgre ? 
'''''CarlosRuiz''''': nope - nvarchar is oracle specific - to use national character set instead of utf8 
'''''nmicoud''''': ok 
'''''CarlosRuiz''''': so - a possible fix is to change line 611 
'''''CarlosRuiz''''': return "NVARCHAR2(" + fieldLength + ")"; 
'''''CarlosRuiz''''': must be 
'''''CarlosRuiz''''': return "VARCHAR2(" + fieldLength + ")"; 
'''''nmicoud''''': that will not effect DocAction columns 
'''''CarlosRuiz''''': your problem is with DocStatus 
'''''nmicoud''''': yes 
'''''nmicoud''''': means that the problem is that new DocStatus are created with the wrong type ? 
'''''nmicoud''''': right ? 
'''''CarlosRuiz''''': actually we have 9 docstatus nvarchar2 and 68 char 
'''''nmicoud''''': i'm testing the patch locally 
'''''nmicoud''''': that would mean that the new DocStatus columns (created as NVARCHAR2) should be modified to VARCHAR2 ? 
'''''CarlosRuiz''''': I think that would make it compatible to use it in a view 
'''''nmicoud''''': let me check : C_Invoice.DocStatus => become VARCHAR2 and myNewTable.DocStatus => stay NVARCHAR2 ? 
'''''CarlosRuiz''''': reading the documentation about nvarchar2 vs varchar2 - seems like nvarchar2 doesn't represent really an advantage - just adding problems :-) 
'''''CarlosRuiz''''': your new table must be varchar2 to if you plan to join those in a view 
'''''nmicoud''''': so i will have to update its type because it has already been created as NVARCHAR2 
'''''nmicoud''''': but i could deal with that 
'''''CarlosRuiz''''': yep - if you sync the column after the patch - must conver it 
'''''nmicoud''''': it seems to work 
'''''nmicoud''''': i mean, patch of line 611 
'''''CarlosRuiz''''': TkTech service is not notifying today JIRA or commits 
'''''TkTech''''': CarlosRuiz: Should respond on your next commit. Thanks for the heads up. 
'''''CarlosRuiz''''': excellent - thanks for your quick answer 
'''''CarlosRuiz''''': nmicoud, reviewing IDEMPIERE-1591 
'''''nmicoud''''': thanks 
'''''CarlosRuiz''''': now I see who deleted the role for SuperUser :D 
'''''nmicoud''''': :D 
'''''CarlosRuiz''''': yesterday red1 could not login as SuperUser and we found the role deleted :-DDD 
'''''nmicoud''''': i'm guilty ;) 
'''''red1''''': Aha.... alors 
'''''CarlosRuiz''''': I see why you cannot save 
'''''red1''''': It was good i tested the night before 
'''''red1''''': Its my favorite to talk about the Felix and iUIMobile.. they are easy and quick to demonstrate 
'''''CarlosRuiz''''': the reason why you cannot save is because the ui is not aware that detail record has changed 
'''''tbayen''''': How can you see who deletes something? 
'''''CarlosRuiz''''': if you touch any of the fields (i.e. inactivate it and activate it again) then you can save it 
'''''nmicoud''''': i see 
'''''CarlosRuiz''''': tbayen, usually the change log saves the deletions - but in this case for user assignment it's not saved - is just for records with ID column 
'''''nmicoud''''': not necesseraly a bug ? 
'''''CarlosRuiz''''': yes - is a bug 
'''''CarlosRuiz''''': will change the summary and annotate the finding 
'''''nmicoud''''': but only related to detail tab with all mandatory field filled with default values 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1591 summary set to "Can't save a new record on detail tab unless record is touched" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1591 
'''''nmicoud''''': but a bug anyway 
'''''CarlosRuiz''''': exactly 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1591 
'''''Not-002''''': [IDEMPIERE] Found the issue is because the UI is not aware that detail record is ready to be saved when you are just inserting and it has all default values, if you touch any field then the save works. 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1591 
'''''CarlosRuiz''''': the same happens if the user set the preference for the tab to be opened as grid 
'''''Not-002''''': [IDEMPIERE] hagi created IDEMPIERE-1603 Copy Info Window Columns from other Info Window's 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1603 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1603 labels set to "+Patch" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1603 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1574 labels set to "+Patch" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1574 
'''''Not-002''''': [IDEMPIERE] nmicoud updated IDEMPIERE-1600 
'''''Not-002''''': [IDEMPIERE] It also happens on calendar window, but you have to reduce the height of your browser. Issues created : http://tracker.zkoss.org/browse/ZKCAL-44 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1600 
'''''CarlosRuiz''''': https://twitter.com/iDempiere/status/408250552795938816 
'''''ocurieles_DCS''''': I can make a user just into the system from the local network? 
'''''ocurieles_DCS''''': can i make a user only login from local network ? 
'''''CarlosRuiz''''': ocurieles_DCS, maybe a login validator 
'''''aguerra''''': You can allow connections only if the server certificate in the browser idempiere have installed? 
'''''CarlosRuiz''''': aguerra, yes - you can configure the web server (i.e. apache) like that - and give certificates using openssh 
'''''CarlosRuiz''''': sorry - openssl 
'''''aguerra''''': And only generate certificates that can connect to the server? 
'''''aguerra''''': Have some documentation?? 
'''''CarlosRuiz''''': http://www.sslshopper.com/article-how-to-create-and-install-an-apache-self-signed-certificate.html 
'''''aguerra''''': thk a lot!!! 
'''''CarlosRuiz''''': I did that once with apache - it works fine - I guess is similar with nginx 
'''''aguerra''''': Ok. I'll try, because I'm going to have a server with 2 remote branches and I just want to connect who have the certificate installed. 
'''''aguerra''''': otherwise I will use openvpn 
'''''ocurieles_DCS''''': 
'''''Not-002''''': [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-1.0c [+0/-0/±4] https://bitbucket.org/idempiere/idempiere/commits/ 
'''''Not-002''''': [iDempiere] j2garcia 040b78a - IDEMPIERE-1574 Sales Order Line lets Price under the Price Limit when updating 
'''''nmicoud''''': @CarlosRuiz: could you integrate your patch for 1461 ? 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1574 status set to "Resolved" -assignee set to "Jesus Garcia" -resolution set to "Fixed" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1574 
'''''CarlosRuiz''''': nmicoud, on 1461 I would like a peer review from hengsin_ first 
'''''nmicoud''''': ok 
'''''CarlosRuiz''''': it's also breaking a functionality that open the zoom window in New mode when the field is empty 
'''''nmicoud''''': ok 
'''''nmicoud''''': thus, any chance with 1575, 1571 and 1589 ? 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1461 status set to "Peer Review Queue" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1461 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1461 assignee set to "Heng Sin Low" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1461 
'''''CarlosRuiz''''': let me test 1571 and 1589 
'''''CarlosRuiz''''': nmicoud, which is the test case for 1571 - inactivate which column on which table? 
'''''nmicoud''''': columns acct type like Ch_Revenue_Acct,... all *_Acct that have been disabled beacuse useless 
'''''CarlosRuiz''''': ah - so, you mean we disabled the field, but not the column - if we inactivate the column then the import account throw errors - right? 
'''''nmicoud''''': no, the column is disabled 
'''''CarlosRuiz''''': ah - I see - your csv has a row with a default account deprecated 
'''''nmicoud''''': that could be also that :) 
'''''nmicoud''''': i still use the same after all years and i only add accounts, never remove some 
'''''nmicoud''''': but i think that lots of csv are not up to date, so this little modification can still be used 
'''''nmicoud''''': but message could be more like "this account is deprecated, you should remove it from your import file" 
'''''Mark___''''': how i can download the manual from the Adempiere? 
'''''CarlosRuiz''''': Mark___, welcome to iDempiere - which is your interest on the system? Functional or technical? 
'''''Mark___''''': yes 
'''''Mark___''''': Functional and technical 
'''''CarlosRuiz''''': then I think you must consider reading the books published 
'''''Not-002''''': [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.0 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/ 
'''''Not-002''''': [iDempiere] globalqss 50bb861 - IDEMPIERE-1571 Import Account generates error because of not active columns / improve warning message for user - thanks to Nicolas Micoud (nmicoud) 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1571 status set to "Resolved" -assignee set to "Nicolas Micoud" -resolution set to "Fixed" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1571 
'''''Mark___''''': where do I get the books? 
'''''CarlosRuiz''''': http://www.google.com/search?q=adempiere%20books 
'''''Not-002''''': [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.0 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/ 
'''''Not-002''''': [iDempiere] nmicoud 7f7bae1 - IDEMPIERE-1589 MailText template don't use language 
'''''CarlosRuiz''''': thanks nmicoud - I committed your 1589 
'''''CarlosRuiz''''': I found in tests there are many other places to improve with similar approaches - it would be good to have deeper analysis on that 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1589 status set to "Resolved" -assignee set to "Nicolas Micoud" -resolution set to "Fixed" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1589 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1587 
'''''Not-002''''': [IDEMPIERE] Discussed on IRC meeting http://www.globalqss.com/wiki/index.php/IDempiere/FullMeeting20131204 {quote} so - a possible fix is to change line 611 return "NVARCHAR2(" + fieldLength + ")"; must be return "VARCHAR2(" + fieldLength + ")"; {quote} pending confirmation 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1587 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1587 labels set to "+Patch Oracle," 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1587 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1587 labels set to "+Patch oracle" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1587 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1249 labels set to "oracle" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1249 
'''''nmicoud''''': thanks Carlos ! 
'''''Not-002''''': [IDEMPIERE] ocurieles created IDEMPIERE-1604 Extend features for Window Customization 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1604 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1579 Attachment set to "IDEMPIERE-1579_v1.patch" 
'''''Not-002''''': [IDEMPIERE] Attaching here two range implementations for dates and numeric fields. Both implementations add a second upper limit field for dates and numbers. First implementation uses the Equal operand if the upper limit field is empty. Second implementation adds a flag to define operand to use if the upper limit field is empty, it uses Equal when flag is unchecked and Greater Or Equal if the flag is checked. WDYT? Regards, Carlos Ruiz 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1579 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1579 labels set to "+Patch" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1579

@@ Line 136: / Line 136: @@
 '''''CarlosRuiz''''': will change the summary and annotate the finding<br>
 '''''nmicoud''''': but only related to detail tab with all mandatory field filled with default values<br>
 '''''nmicoud''''': but a bug anyway<br>
 '''''CarlosRuiz''''': exactly<br>
 '''''CarlosRuiz''''': the same happens if the user set the preference for the tab to be opened as grid<br>
 '''''CarlosRuiz''''': https://twitter.com/iDempiere/status/408250552795938816<br>
 '''''ocurieles_DCS''''': I can make a user just into the system from the local network?<br>
@@ Line 168: / Line 154: @@
 '''''aguerra''''': otherwise I will use openvpn<br>
 '''''ocurieles_DCS''''':   <br>
 '''''nmicoud''''': @CarlosRuiz: could you integrate your patch for 1461 ?<br>
 '''''CarlosRuiz''''': nmicoud, on 1461 I would like a peer review from hengsin_ first<br>
 '''''nmicoud''''': ok<br>
@@ Line 178: / Line 160: @@
 '''''nmicoud''''': ok<br>
 '''''nmicoud''''': thus, any chance with 1575, 1571 and 1589 ?<br>
 '''''CarlosRuiz''''': let me test 1571 and 1589<br>
 '''''CarlosRuiz''''': nmicoud, which is the test case for 1571 - inactivate which column on which table?<br>
@@ Line 197: / Line 175: @@
 '''''Mark___''''': Functional and  technical<br>
 '''''CarlosRuiz''''': then I think you must consider reading the books published<br>
 '''''Mark___''''': where do I get the books?<br>
 '''''CarlosRuiz''''': http://www.google.com/search?q=adempiere%20books<br>
 '''''CarlosRuiz''''': thanks nmicoud - I committed your 1589<br>
 '''''CarlosRuiz''''': I found in tests there are many other places to improve with similar approaches - it would be good to have deeper analysis on that<br>
 '''''nmicoud''''': thanks Carlos !<br>