IDempiere/FullMeeting20150422 - Revision history

CarlosRuiz: drop JIRA notifications from log

2015-04-22T20:51:24Z

drop JIRA notifications from log

CarlosRuiz: full meeting

2015-04-22T20:50:53Z

full meeting

New page

⇐
[[IDempiere|Table of Contents]] |
[[IDempiere/Full Meeting Minutes|Full Meeting Minutes]] |
Full Meeting 2015-04-22


'''''CarlosRuiz''''': Good Morning 
'''''Deepak''''': Good Morning 
'''''CarlosRuiz''''': I'll be checking peer review queue as usual 
'''''Not-3656''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-847 Attachment set to "msg-english.png" 
'''''Not-3656''''': [IDEMPIERE] [~hieplq] - I'm lost here - just tested without applying the patch and the message is correctly translated (see screenshots msg-spanish and msg-english) - so, seems like the patch is not needed, as Flemming said, maybe is just that message is not translated and solution could be to add it as a fragment plugin? 
'''''Not-3656''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-847 
'''''Vyrlokar''''': Heya everybody 
'''''CarlosRuiz''''': Hi Antoni 
'''''CarlosRuiz''''': I'm reviewing IDEMPIERE-2591 
'''''Vyrlokar''''': CarlosRuiz, I've attached a patch for it, it's a really simple patch 
'''''CarlosRuiz''''': do you have a bitbucket user? 
'''''Vyrlokar''''': nope 
'''''CarlosRuiz''''': to use as developer for the commit 
'''''Vyrlokar''''': CarlosRuiz, Should I create one? 
'''''CarlosRuiz''''': if you want I can name you as the committer - you're the contributor so I'm glad to credit that way 
'''''ngordon7000''''': We were wondering about IDEMPIERE-2248 
'''''CarlosRuiz''''': ngordon7000, maybe adding user_id to the filename can solve it? 
'''''CarlosRuiz''''': ah - we already commented that on ticket :-) 
'''''CarlosRuiz''''': Vyrlokar, test passed 
'''''CarlosRuiz''''': the patch was in a weird format - next time you can generate it with "hg diff" 
'''''Vyrlokar''''': Oh, I generated it with the diff command file, sorry 
'''''Vyrlokar''''': command line utility I mean 
'''''CarlosRuiz''''': or if you want the patch comes with developer-id and commit message you can do: "hg commit" - "hg export [changeset]" "hg rollback" 
'''''Vyrlokar''''': ok CarlosRuiz 
'''''CarlosRuiz''''': do you want me to commit with your developer name? I'm ready to commit :-9 
'''''CarlosRuiz''''': :-) 
'''''Not-3656''''': [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/ 
'''''Not-3656''''': [iDempiere] Antoni Ten Monrós 81898d6 - IDEMPIERE-2591 OrderPoCreate does not check that Product_PO entries active 
'''''ngordon7000''''': Thanks Carlos - it is quite a difficult issue to troubleshoot , and only becomes evident in production under load - so that's why I mentioned it. 
'''''Not-3656''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2585 Attachment set to "timeout-spa.png" 
'''''Not-3656''''': [IDEMPIERE] [~sjeffen] I tested logged in spanish and the timeout message + button is in spanish actually (see attached screenshot) - what's the test case? 
'''''Not-3656''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-2585 
'''''Not-3656''''': [IDEMPIERE] Kuroshima created IDEMPIERE-2592 Move schedule date calculation out of the nonstandard constructor and into a separate function to allow overridding it 
'''''Deepak''''': Carlos, I show your comments on Posting issue with use of LineMA on Movement and production 
'''''Not-3656''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-2592 
'''''Deepak''''': But I think we need to do more for movement 
'''''Deepak''''': I am thinking if movement from one asi to another 
'''''ocurieles''''': hi everybody 
'''''CarlosRuiz''''': Hi Orlando 
'''''Deepak''''': User can create lineMA totaling to 0 but having lineMA with positive and negative qty 
'''''Kuroshima''''': Deepak, Using the Manufacturing module to change ASI, if I understood you correctly? 
'''''Deepak''''': No 
'''''Deepak''''': That is not correct 
'''''Deepak''''': Movement is not manufacturing 
'''''Not-3656''''': [IDEMPIERE] Kuroshima updated IDEMPIERE-2592 
'''''Not-3656''''': [IDEMPIERE] The way to do it without this change requires using an eventHandler and intercepting the PO_AFTER_SAVE event. It's problematic because if the user manually creates the pay schedule for the order/invoice, his input gets overridden by the system on the first save. 
'''''Not-3656''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-2592 
'''''Kuroshima''''': Deepak, Manufacturing creates movements, though. You mean creating the movements manually? 
'''''Kuroshima''''': Deepak, Seems dangerous, I would not allow users to create movements manually, too much room for fat finger errors 
'''''Not-3656''''': [IDEMPIERE] hieplq updated IDEMPIERE-847 Attachment set to "msgzk.properties" 
'''''Not-3656''''': [IDEMPIERE] hi [~carlosruiz_globalqss], agree, patch is not need. at first i test and confirm in developer tool, i see zk.lang isn't reloaded. because i must issue patch. for now i retest and yes, message is ok without patch (even zk.lang isn't reloaded. maybe a anonymous function is call for update message) for Danish ([~sjeffen] maybe you help translate and issue a patch) and other language. place file as below: file msgzk_xx.js into folder org.ade 
'''''Not-3656''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-847 
'''''Not-3656''''': [IDEMPIERE] hieplq updated IDEMPIERE-847 status set to "Open" 
'''''Not-3656''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-847 
'''''Not-3656''''': [IDEMPIERE] sjeffen updated IDEMPIERE-2586 
'''''Not-3656''''': [IDEMPIERE] OK - then the description should have been: Label "Detail record" is not translated in the window. :-) I would say that the doc is wrong, at least when looking at the genereated html! The generated html states: class="z-south-collapsed" /Flemming 
'''''Not-3656''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-2586 
'''''Not-3656''''': [IDEMPIERE] sjeffen updated IDEMPIERE-2585 
'''''Not-3656''''': [IDEMPIERE] The test case is running in danish and the button is not translated! Then again come to think about it - this could be the case of missing danish translation of ZK. The above patch gets the translated text for the button from iDempiere, just as the message does! 
'''''Not-3656''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-2585 
'''''Not-3656''''': [IDEMPIERE] sjeffen updated IDEMPIERE-847 
'''''Not-3656''''': [IDEMPIERE] Hi I'm following your advice hieplq and I'm in the progres of translating the ZK files 
'''''Not-3656''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-847 
'''''CarlosRuiz''''': Deepak, also a user can register ASI_From and ASI_To on the movement line 
'''''CarlosRuiz''''': must be the same effect 
'''''mhernandezve''''': Hi comunity! 
'''''Not-3656''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2585 status set to "Resolved" -resolution set to "Cannot Reproduce" 
'''''Not-3656''''': [IDEMPIERE] so, is ok to close this as "cannot reproduce"? Please reopen if you think there is still an issue to solve. 
'''''Not-3656''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-2585 
'''''Not-3656''''': [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.1 [+2/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/ 
'''''Not-3656''''': [iDempiere] globalqss aa7bd3f - IDEMPIERE-581 Store divider location for window per user / DetailHeight must be user preferences - found DetailCollapsed preference was also implemented as general client preference in IDEMPIERE-369 
'''''CarlosRuiz''''': Hi mhernandezve 
'''''Not-3656''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2585 status set to "Reopened" -resolution set to "None" 
'''''Not-3656''''': [IDEMPIERE] Hi [~sjeffen], just noticed that normal timeout works - but the issue can be seen when the server is restarted. 
'''''Not-3656''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-2585 
'''''Not-3656''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2586 
'''''Not-3656''''': [IDEMPIERE] [~sjeffen], but in my tests I see the "Detail record" message translated - where did you take the screenshot? Also the patch is not translating the message, just changing some css, I don't see any change before/after the patch 
'''''Not-3656''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-2586 
'''''Not-3656''''': [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.1 [+0/-0/±2] https://bitbucket.org/idempiere/idempiere/commits/ 
'''''Not-3656''''': [iDempiere] globalqss 6042c1c - IDEMPIERE-2585 Sessiontimeout button text is not translated users language / integrate patch from Flemming Birch (sjeffen) 
'''''Not-3656''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2585 status set to "Resolved" -assignee set to "Flemming Birch" -resolution set to "Fixed" 
'''''Not-3656''''': [IDEMPIERE] Thanks [~sjeffen], I tested your patch and it solves the issue when restarting the server. For next commit please let me know your bitbucket user and if you want me to commit your patches with your developer name 
'''''Not-3656''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-2585 
'''''Not-3656''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2573 status set to "Resolved" -resolution set to "Won't Fix" 
'''''Not-3656''''': [IDEMPIERE] Thanks [~tsvikruha], I think this would hide a misconfiguration in dictionary. Maybe would be better to implement a check when saving AD_Column to see if the foreign element exists. 
'''''Not-3656''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-2573 
'''''Kuroshima''''': cya everybody 
'''''CarlosRuiz''''': brb 
'''''red1_''''': Hi hieplq 
'''''red1_''''': hi mhernandezve 
'''''red1_''''': hi norbertbede 
'''''red1_''''': and ocuriele_ 
'''''ocuriele_''''': Hi Mr. @Red1 
'''''red1_''''': ah you are alive! :) 
'''''ocuriele_''''': always :D 
'''''red1_''''': Hows Samuel? 
'''''norbertbede''''': hi red1 
'''''ocuriele_''''': in the school 
'''''ocuriele_''''': is Ok... 
'''''ocuriele_''''': adapting to Ecuador 
'''''ocuriele_''''': and learning new words 
'''''ocuriele_''''': how are your kid ? 
'''''red1_''''': he is shy 
'''''red1_''''': but he looks happy 
'''''red1_''''': Hi norbertbede when can we drop by your place? 
'''''red1_''''': I want my son to meet your beautiful family 
'''''red1_''''': June 1st be in Russia for three months 
'''''norbertbede''''': we are in progress rollout a new customer 1st may 
'''''norbertbede''''': so we are busy. but i estimate around 15 april is affordable 
'''''norbertbede''''': is it ok for you ? 
'''''norbertbede''''': i mean that time we can invite you for some days sure 
'''''red1_''''': You mean 15 May 
'''''red1_''''': Now is April 22 
'''''norbertbede''''': ah sure 
'''''norbertbede''''': yes 
'''''red1_''''': OK i will plan for it 
'''''red1_''''': it is nearby for us to come by train 
'''''red1_''''': Anything you want me to get your and your family from Vienna? 
'''''norbertbede''''': no thanks. just yourself ;) 
'''''norbertbede''''': and your son 
'''''red1_''''': ask your daughters what they like. I can afford something 
'''''red1_''''': haha 
'''''aguerra''''': hi everybody 
'''''red1_''''': Hol aguerra 
'''''aguerra''''': hola red1 
'''''Not-3656''''': [iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/ 
'''''Not-3656''''': [iDempiere] globalqss 35d3e4a - IDEMPIERE-2248 ZK Jasper report viewer prints incorrect file 
'''''Not-3656''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2248 status set to "Resolved" -assignee set to "Carlos Antonio Ruiz Gomez" -resolution set to "Fixed" 
'''''Not-3656''''': [IDEMPIERE] Thanks [~ngordon7000], followed better your advice, just added #AD_Session_ID 
'''''Not-3656''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-2248 
'''''Not-3656''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2097 
'''''Not-3656''''': [IDEMPIERE] Hi [~alan.lesc1], I was not able to reproduce the issue, maybe you can create the test case in one of the test servers? 
'''''Not-3656''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-2097 
'''''Not-3656''''': [iDempiere] CarlosRuiz_globalqss pushed 1 commit to development [+8/-0/±13] https://bitbucket.org/idempiere/idempiere/commits/ 
'''''Not-3656''''': [iDempiere] globalqss 0fc582f - hg merge release-2.1 (merge release2.1 into development)

@@ Line 10: / Line 10: @@
 '''''Deepak''''': Good Morning<br>
 '''''CarlosRuiz''''': I'll be checking peer review queue as usual<br>
 '''''Vyrlokar''''': Heya everybody<br>
 '''''CarlosRuiz''''': Hi Antoni<br>
@@ Line 33: / Line 30: @@
 '''''CarlosRuiz''''': do you want me to commit with your developer name? I'm ready to commit  :-9<br>
 '''''CarlosRuiz''''': :-)<br>
 '''''ngordon7000''''': Thanks Carlos - it is quite a difficult issue to troubleshoot , and only becomes evident in production under load - so that's why I mentioned it.<br>
 '''''Deepak''''': Carlos, I show your comments on Posting issue with use of LineMA on Movement and production<br>
 '''''Deepak''''': But I think we need to do more for movement<br>
 '''''Deepak''''': I am thinking if movement from one asi to another<br>
@@ Line 51: / Line 41: @@
 '''''Deepak''''': That is not correct<br>
 '''''Deepak''''': Movement is not manufacturing<br>
 '''''Kuroshima''''': Deepak, Manufacturing creates movements, though. You mean creating the movements manually?<br>
 '''''Kuroshima''''': Deepak, Seems dangerous, I would not allow users to create movements manually, too much room for fat finger errors<br>
 '''''CarlosRuiz''''': Deepak, also a user can register ASI_From and ASI_To on the movement line<br>
 '''''CarlosRuiz''''': must be the same effect<br>
 '''''mhernandezve''''': Hi comunity!<br>
 '''''CarlosRuiz''''': Hi mhernandezve <br>
 '''''Kuroshima''''': cya everybody<br>
 '''''CarlosRuiz''''': brb<br>
@@ Line 132: / Line 86: @@
 '''''red1_''''': Hol aguerra<br>
 '''''aguerra''''': hola red1<br>