IDempiere/FullMeeting20131211

⇐ Table of Contents | Full Meeting Minutes | Full Meeting 2013-12-11

CarlosRuiz: Good Morning
nmicoud: Bonjour
tbayen: Hi
tbayen: Daarestiet
red1: Hola
CarlosRuiz: I'll be integrating things from the peer review queue
a42niem: Guten Tag
nmicoud: tbayen: do you think you'll have time to review https://idempiere.atlassian.net/browse/IDEMPIERE-1395 ? otherwise, i could put it directly in the peer review queue
CarlosRuiz: I tested yesterday the attachment scanner plugin - it's working fine - I sent the author patch to work with java6
aguerra: Good Morning everybody!!!
CarlosRuiz: Hi Alejandro
Not-002: [IDEMPIERE] nmicoud created IDEMPIERE-1623 MAIL_SEND_BCC_TO_FROM is not used
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1623
Not-002: [IDEMPIERE] nmicoud updated IDEMPIERE-1623 Attachment set to "IDEMPIERE-1623.patch"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1623
Not-002: [IDEMPIERE] nmicoud updated IDEMPIERE-1623 assignee set to "Carlos Antonio Ruiz Gomez"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1623
Not-002: [IDEMPIERE] nmicoud updated IDEMPIERE-1623 status set to "Peer Review Queue"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1623
Not-002: [IDEMPIERE] nmicoud updated IDEMPIERE-1623 labels set to "+Patch"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1623
tbayen: nmicoud, I will find time to review it. I prefer to first review it inside the "Swing User Community" before giving it to Carlos.
nmicoud: cool, that's the goal :) ; thanks
CarlosRuiz: nmicoud, why is that needed? by default all tabs are now in single row - having a tab in multi-row is because of a user pref - or a window customization
nmicoud: you can be in multi-row view to search a record ; you don't find it and want to create. So click on New and you switch to single mode automatically
nmicoud: it's useful for swing, less for zk
CarlosRuiz: ah - nice catch on IDEMPIERE-1623 (a ; at the end of an if)
nmicoud: yep :D
CarlosRuiz: supposedly findbugs catch that - don't know if we missed it - or findbugs didn't
nmicoud: i've try lots of time before going in the code ;)
nmicoud: and it fits my needs now !
CarlosRuiz: yes - findbugs report that line with pattern UCF_USELESS_CONTROL_FLOW_NEXT_LINE
nmicoud: i have findbugs in my eclipse and it does not report anything about this
CarlosRuiz: it reported to me this one http://findbugs.sourceforge.net/bugDescriptions.html#UCF_USELESS_CONTROL_FLOW_NEXT_LINE
CarlosRuiz: checked all code and seems like is the only one
CarlosRuiz: there is a strange construction on line 84 of ExpenseTypesFromAccounts
CarlosRuiz: but is more an indentation and readability issue
nmicoud: ctrl+i ;)
Not-002: [iDempiere] CarlosRuiz_globalqss pushed 4 commits to release-1.0c [+2/-0/±4] https://bitbucket.org/idempiere/idempiere/commits/
Not-002: [iDempiere] elainetan 90590db - IDEMPIERE-1621 Mandatory field in process parameter panel not in red label
Not-002: [iDempiere] globalqss ad59443 - IDEMPIERE-1587 Error ORA-12704 for some columns
Not-002: [iDempiere] globalqss b4967e0 - IDEMPIERE-1575 Increase some columns length / based on patch from Nicolas Micoud (nmicoud)
Not-002: [iDempiere] nmicoud d90291f - IDEMPIERE-1623 MAIL_SEND_BCC_TO_FROM is not used
Not-002: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1621 status set to "Resolved" -assignee set to "Elaine Tan" -resolution set to "Fixed"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1621
Not-002: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1587 status set to "Resolved" -assignee set to "Carlos Antonio Ruiz Gomez" -resolution set to "Fixed"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1587
Not-002: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1575 status set to "Resolved" -assignee set to "Nicolas Micoud" -resolution set to "Fixed"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1575
Not-002: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1623 status set to "Resolved" -assignee set to "Nicolas Micoud" -resolution set to "Fixed"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1623
nmicoud: CarlosRuiz: About https://groups.google.com/forum/?fromgroups#!topic/idempiere/nMMyoZS_td4 (Restrict access to dashboard by role) ; do you think it is feasable in sql ?
Not-002: [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-1.0c [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/
Not-002: [iDempiere] globalqss 3605d0e - minor - improve readability of ExpenseTypesFromAccounts
CarlosRuiz: nmicoud, maybe an "AND NOT IN"
CarlosRuiz: on the second part of the query
nmicoud: will try this
CarlosRuiz: about the question raised from Kevin at https://groups.google.com/d/msg/idempiere/Alryi_jLclA/rnBlrfl2uUcJ
CarlosRuiz: do you think we must allow non-FOSS plugins to be published in our wiki?
CarlosRuiz: tbayen, nmicoud - have you tested IDEMPIERE-1605 ?
nmicoud: not yet, but i can
nmicoud: seems ok
CarlosRuiz: if the SMT wants it (and it doesn't affect zk or core like this case) just let me know and I'll push it
nmicoud: it seems useful, so +1 for me ; tbayen WDYT ?
tbayen: It looks useful but I did not test it yet. If you think it is ok I agree to integrate it.
Not-002: [IDEMPIERE] nmicoud created IDEMPIERE-1624 Restrict access to System dashboards
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1624
Not-002: [IDEMPIERE] nmicoud updated IDEMPIERE-1624 Attachment set to "IDEMPIERE-1624.patch"
Not-002: [IDEMPIERE] Here's a modified sql with a new 'AND NOT IN' clause to remove the inactive records. Wondering if a new checkbox (something like 'IsAuthorized') wouldn't be better than the IsActive flag. WDYT ? Regards, Nicolas
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1624
Not-002: [IDEMPIERE] nmicoud updated IDEMPIERE-1624 status set to "Peer Review Queue"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1624
Not-002: [IDEMPIERE] nmicoud updated IDEMPIERE-1624 assignee set to "Carlos Antonio Ruiz Gomez"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1624
Not-002: [IDEMPIERE] nmicoud updated IDEMPIERE-1624 labels set to "+Patch"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1624
Not-002: [IDEMPIERE] carlosruiz_globalqss created IDEMPIERE-1625 Remittance is printing inactive address
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1625
Not-002: [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-1.0c [+2/-0/±2] https://bitbucket.org/idempiere/idempiere/commits/
Not-002: [iDempiere] globalqss 7ce954a - IDEMPIERE-1625 Remittance is printing inactive address
Not-002: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1625 status set to "Resolved" -assignee set to "Carlos Antonio Ruiz Gomez" -resolution set to "Fixed"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1625
CarlosRuiz: nmicoud,
CarlosRuiz: nevermind - testing your patch for 1624
Not-002: [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.0 [+0/-0/±2] https://bitbucket.org/idempiere/idempiere/commits/
egil_ghintech: Good morning to every one.
Not-002: [iDempiere] globalqss c844743 - IDEMPIERE-1624 Restrict access to System dashboards / based on patch from Nicolas Micoud (nmicoud)
Not-002: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1624 status set to "Resolved" -resolution set to "Fixed"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1624
CarlosRuiz: Hi Eduardo
egil_ghintech: I carlos i need to ask you something
egil_ghintech: i have a customer who want in the subject of the request mail the name of the bussiness partner
egil_ghintech: can i do this whitout a plugin?
CarlosRuiz: have you tried @Name@ ?
egil_ghintech: no.
egil_ghintech: but in the subjet i can add this. I could doit on the body adding by default @Name@ in the sumary
egil_ghintech: Efectively it works but in the body. if i want to add this in the subject where sould i put this?
CarlosRuiz: doesn't work in the subject?
CarlosRuiz: I mean subject on mail template
egil_ghintech: ok i am going to test it
Not-002: [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-1.0c [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/
Not-002: [iDempiere] dantam 6b9e5f0 - IDEMPIERE-1605 - Save path when working with attachments.
Not-002: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1605 status set to "Resolved" -assignee set to "Daniel Tamm" -resolution set to "Fixed"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1605
Not-002: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1604 labels set to ""
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1604
bhoot: Any iDempiere/Adempiere trainer out there?
CarlosRuiz: I can go to India if you want :-)
bhoot: you are welcome.
tbayen: I had a training with Carlos this spring and can recommend him. You will not find anyone better.
tbayen: :-)
CarlosRuiz: :-) thanks for the feedback - maybe we can do similar - next conference in India
bhoot: That is good idea.
bhoot: I am looking for basic training over teamviewer and willing to pay. I don't want to spend time asking silly questions on forum and irc.
tbayen: If there is a conference the trainers will like more to come. I would come too for this. Hehe :-)
tbayen: But I can only give a training together with Carlos if there is Karaoke in the city. Haha!
bhoot: I don't know the number of iD/AD users in india. So don't know if a conference can be organized.
tbayen: I thought the same last year. In the Krefeld conference mostly people from all over the world came.
CarlosRuiz: iDempiere Karaoke is an institution
tbayen: The first step was to commit the main developers. The others came from alone... And if they would not have come it would have been a success either because the main developers were in one room and could meet.
tbayen: Are walking tree working with iDempiere or are still stucked on ADempiere? They are from india too.
CarlosRuiz: I think really there are LOTS of *mpiere users and developers in India - just they are not so active in forums
CarlosRuiz: I could visit Deepak
CarlosRuiz: yes - walking tree is hired as release manager for Adempiere - so, they can join
aguerra: Hola Eduardo!!!!
egil_ghintech: hola alejandro
egil_ghintech: resolviste lo del csv?
aguerra: yeah!!!!
aguerra: listo!!! subi 250 registros.
aguerra: nmicoud, "NF1.0 Document Sequence Improved", nice contributions!!! THK
nmicoud: yeah! it rocks ! ;)
nmicoud: carlos did a good part of the job
aguerra: .... ups .. CarlosRuiz, excelente contribucion.... Gracias!!!!
nmicoud: it was a Colombian/French enhancement :)
nmicoud: gtg ; see you
aguerra: I am new but I hope to make contributions soon !
Not-002: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1588 Attachment set to "IDEMPIERE-1588_v2.patch"
Not-002: [IDEMPIERE] Nicolas, peer reviewed your patch, I think it's breaking paymentterm function and also any view/report using the original nextBusinessDay. I would suggest better IDEMPIERE-1588_v2.patch - less disruptive. Could you please test it and provide feedback? Regards, Carlos Ruiz
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1588
Not-002: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1588 assignee set to "Nicolas Micoud"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1588
Not-002: [IDEMPIERE] juan.arboleda updated IDEMPIERE-1597 assignee set to "Juan David Arboleda Ruiz"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1597
Not-002: [IDEMPIERE] carlos.augusto updated IDEMPIERE-1574 Attachment set to "20131211_IDEMPIERE-1574_NULL_CONTROL.patch"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1574
Not-002: [IDEMPIERE] carlos.augusto updated IDEMPIERE-1574 assignee set to "Carlos Antonio Ruiz Gomez"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1574
Not-002: [iDempiere] CarlosRuiz_globalqss pushed 2 commits to release-1.0c [+0/-0/±4] https://bitbucket.org/idempiere/idempiere/commits/
Not-002: [iDempiere] j2garcia b271610 - IDEMPIERE-1516 Performance indicators display fails when role user assigment inactive
Not-002: [iDempiere] caaugustoss 46ac4a4 - IDEMPIERE-1574 Sales Order Line lets Price under the Price Limit when updating / fix NPE
Not-002: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1516 status set to "Resolved" -assignee set to "Jesus Garcia" -resolution set to "Fixed"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1516
Not-002: [IDEMPIERE] tbayen updated IDEMPIERE-1618
Not-002: [IDEMPIERE] To part 1 (caused probably by new Eclipse version): For completelyness (and googling people) I give the whole error I got in the log (or on console with my patch) here. My workaround is to delete $HOME/buckminster.output/* and then retry. This seems to solve this issue (but after two weeks of trying I do not believe anything about this). {noformat} org.eclipse.m2e.logback.configuration: The org.eclipse.m2e.logback.configuration bundle was
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1618
tbayen: CarlosRuiz, what do you think about my patch in IDEMPIERE-1618? Is is worth to do it also for the server script and submit as a patch? Or shall we just close the issue and let people find out themself to look into the log?
tbayen: (I speak of the thing I call "Part 2" in the issue comments)
CarlosRuiz: yes - I think is worthy
tbayen: ok, I will do it.
Not-002: [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.0 [+0/-0/±2] https://bitbucket.org/idempiere/idempiere/commits/
Not-002: [iDempiere] globalqss df90d4b - IDEMPIERE-1558 Performance indicators display fails using context variables in measure calculation / based on patch from Jesus Garcia (j2garcia)
Not-002: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1558 status set to "Resolved" -assignee set to "Jesus Garcia" -resolution set to "Fixed"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1558
Not-002: [IDEMPIERE] tbayen updated IDEMPIERE-1490 labels set to "+Patch report"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1490
Not-002: [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.0 [+3/-0/±0] https://bitbucket.org/idempiere/idempiere/commits/
Not-002: [iDempiere] globalqss 8519175 - IDEMPIERE-1521 Ability to delete a view within AD / based on patch from Nicolas Micoud (nmicoud)
Not-002: [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1521 status set to "Resolved" -assignee set to "Nicolas Micoud" -resolution set to "Fixed"
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1521
Not-002: [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.0 [+2/-0/±0] https://bitbucket.org/idempiere/idempiere/commits/
Not-002: [iDempiere] globalqss 801c4d1 - IDEMPIERE-1132 Generate official dictionary indexes
bhoot: is it possible to create table in database using Application Dictionary or table needs to be created first?
bhoot: I know how to create Application Dictionary from table existing in db. But is otherway around possible?
CarlosRuiz: yes, you can also create the table in DB and with a button read it and the process creates the entries in dictionary
bhoot: My question is if I make entry in to dictionary first , can application create table in db?
CarlosRuiz: yes - tbayen answered that yesterday - the synchronize column button
CarlosRuiz: both ways are supported
bhoot: ok thanks I have to find that button...
CarlosRuiz: http://wiki.idempiere.org/en/Table_and_Column_(Window_ID-100)#TAB:_Column
CarlosRuiz: button at bottom
banym: CarlosRuiz, do you prefer the sun/oracle jdk or does idempiere run full functional with openjdk aswell?
norbertbede: hi community
norbertbede: looking stakeholders to issue https://idempiere.atlassian.net/browse/IDEMPIERE-1606?filter=-2
norbertbede: hops
norbertbede: https://idempiere.atlassian.net/browse/IDEMPIERE-1606
norbertbede: looks me serious issue. i tested it deeply, well
CarlosRuiz: banym, I haven't tested with sun/oracle - just openjdk
CarlosRuiz: Hi norbertbede
CarlosRuiz: I have noticed that after you run the hash password process - you need to restart the server
CarlosRuiz: did you?
norbertbede: yes sure.
norbertbede: i consult it also with my java guy try to fix issues inside my ticket, but too deep level for us...
norbertbede: i have reported 3 issues becase relate each other
CarlosRuiz: when using email for login - I think first step after initial client setup is that SuperUser must assign an email to the two new users
norbertbede: if you deploy client manually then yes its working
norbertbede: we want save time and deploy client by external web form trough API
norbertbede: in that case that is not works
norbertbede: so proces is stalled
norbertbede: that is the reason why i classify it as improvement not bug
CarlosRuiz: calling web services I guess - in such case you can update the emails also via webservices
norbertbede: so you suggest to add client email as superuser email ?
norbertbede: let say that is acceptable
norbertbede: in issue 2 BUG: Initial Client Setup Process - doesn’t hash passwords for Client Admin and Client User
norbertbede: password field encryption setting up but value after client setup password same as name - but no hashed
norbertbede: so cant login
CarlosRuiz: ah
CarlosRuiz: that's a bug - the value is being inserted directly
CarlosRuiz: MSetup.java - lines 285, 308
CarlosRuiz: I would think the Initial Setup must be improved to ask for emails instead of names when the system is using emails to login - and fix that bug on the password if the system is using hashed
norbertbede: can you fix this bug ?pls
norbertbede: issue 3 in ticket bug too, altough rare
norbertbede: ly happend
norbertbede: but idea to improve initial client is good point, because then follow system wide rule
CarlosRuiz: I think that bug will be in queue - is not blocker as it has a workaround to finish the setup with SuperUser
CarlosRuiz: or if you want we can guide your java guy about how to solve it
norbertbede: we invest 1 day, want to contribute fix, but was too deep.
norbertbede: maybe he should try it again…and ask
CarlosRuiz: we have solved more complex problems on some IRC meetings :-)
CarlosRuiz: the good on IRC meetings is that you have cooperative work with others pointing to the exact line and suggesting ways to solve it
norbertbede: its no question. i was at university in chech today dealing about get new contributors from Masaryk university
norbertbede: hehe ;)
norbertbede: anyway i sent you a private email today. maybe explain our feature steps
norbertbede: so in some weeks we will have - more complex - issues too ;)
Not-002: [IDEMPIERE] norbert.bede updated IDEMPIERE-1606
Not-002: [IDEMPIERE] hi community looking stakeholders to issue https://idempiere.atlassian.net/browse/IDEMPIERE-1606?filter=-2 hops https://idempiere.atlassian.net/browse/IDEMPIERE-1606 looks me serious issue. i tested it deeply, well 22:29 CarlosRuiz banym, I haven't tested with sun/oracle - just openjdk 22:29 CarlosRuiz Hi norbertbede 22:30 CarlosRuiz I have noticed that after you run the hash password process - you need to restart the server did you? 22:31
Not-002: [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1606
norbertbede: i have updated issue by chat...
Not-002: [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.0 [+2/-0/±0] https://bitbucket.org/idempiere/idempiere/commits/
Not-002: [iDempiere] globalqss d951d89 - IDEMPIERE-1132 Generate official foreign keys
Not-002: [iDempiere] CarlosRuiz_globalqss pushed 2 commits to development [+11/-0/±16] https://bitbucket.org/idempiere/idempiere/commits/
Not-002: [iDempiere] globalqss b43fe00 - hg merge release-1.0c (merge release1 into release2)
Not-002: [iDempiere] globalqss 52646e1 - hg merge release-2.0 (merge release2 into development)