#idempiere IRC log for Saturday, 2017-06-24

Not-6102[IDEMPIERE] hieplq updated IDEMPIERE-340700:25
Not-6102[IDEMPIERE] at this level, get lineno is complicate, asi can be get.00:25
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-340700:25
Not-6102[iDempiere4.1Daily] jenkins built #82 completed (success) http://ci.idempiere.org/job/iDempiere4.1Daily/82/04:38
Not-6102[IDEMPIERE] hieplq created IDEMPIERE-3408 Encrypted Field can explore by normal user08:23
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-340808:23
*** CarlosRuiz has joined #idempiere09:14
*** a42niem has joined #idempiere09:32
*** nmicoud has joined #idempiere09:46
*** nmicoud has quit IRC10:03
*** a42niem has quit IRC10:56
Not-6102[iDempiere] CarlosRuiz_globalqss pushed 2 commits to release-4.1 [+0/-0/±4] https://bitbucket.org/idempiere/idempiere/commits/11:34
Not-6102[iDempiere] globalqss 284a29f - IDEMPIERE-1937 NPE when click cancel from screen "change role" when login and "forget password"11:34
Not-6102[iDempiere] globalqss d24d8d2 - Peer review from a suggested patch from Hiep Lq -> IDEMPIERE-3408 Encrypted Field can explore by normal user11:34
Not-6102[iDempiere4.1] jenkins built #131 completed (success) http://ci.idempiere.org/job/iDempiere4.1/131/12:03
Not-6102[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-4.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/12:44
Not-6102[iDempiere] globalqss 1632794 - Peer review from a suggested patch from Hiep Lq -> IDEMPIERE-3408 Encrypted Field can explore by normal user12:44
Not-6102[IDEMPIERE] hieplq updated IDEMPIERE-3043 status set to "Resolved" -resolution set to "Won't Fix"12:59
Not-6102[IDEMPIERE] set Bundle-ActivationPolicy: lazy on MANIFEST.MF can be helpful12:59
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-304312:59
Not-6102[iDempiere4.1] jenkins built #132 completed (failure) http://ci.idempiere.org/job/iDempiere4.1/132/13:01
Not-6102[IDEMPIERE] hieplq updated IDEMPIERE-3408 status set to "Resolved" -resolution set to "Fixed"13:20
Not-6102[IDEMPIERE] waiting people update server before detail about redo step.13:20
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-340813:20
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-340713:39
Not-6102[IDEMPIERE] @Hiep, there must be a patch from TrekGlobal that shows the line numbers ... let me check if I can find it.13:39
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-340713:39
Not-6102[iDempiere4.1] jenkins built #133 completed (failure) http://ci.idempiere.org/job/iDempiere4.1/133/13:40
Not-6102[iDempiere-experimental] jenkins built #162 completed (success) http://ci.idempiere.org/job/iDempiere-experimental/162/13:49
Not-6102[IDEMPIERE] carlosruiz_globalqss created IDEMPIERE-3409 Make separator used between identifier fields configurable15:32
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-340915:32
Not-6102[iDempiere] CarlosRuiz_globalqss pushed 2 commits to release-4.1 [+2/-0/±3] https://bitbucket.org/idempiere/idempiere/commits/16:08
Not-6102[iDempiere] hengsin f6d21d5 - IDEMPIERE-3409 Make separator used between identifier fields configurable (1004703)16:08
Not-6102[iDempiere] globalqss cc37302 - IDEMPIERE-3409 Make separator used between identifier fields configurable16:08
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3409 status set to "Resolved" -assignee set to "carlosruiz_globalqss" -resolution set to "Fixed"16:09
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-340916:09
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-340716:10
Not-6102[IDEMPIERE] Sorry, I failed to integrate it as there are many changes involved. In the meantime I think your patch is safe.16:10
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-340716:10
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3401 description set to "steps to reproduce 1. create new order 1. 2. select partner Joe Block, with disc scheme 10% break 100 3. create new line select product mulch, change qty>100 system properly update discount 4. on same tab create new order 2 for BP without disc. schema 5. create lines, change Qty 6. on the same tab return back to order 1. line 1. change qty>>16:23
Not-6102BUG. system doesnt change Discount. issue: in step 5. we set contextual variable disc. schema on for tab/window to 'N', from this moment never will be discount recalculated on orders with same tab. *Solution* we will submit patch with workaround - then we need cooperate, implement good concept well. "16:23
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-340116:23
Not-6102[iDempiere4.1] jenkins built #134 completed (failure) http://ci.idempiere.org/job/iDempiere4.1/134/16:31
Not-6102[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-4.1 [+2/-0/±2] https://bitbucket.org/idempiere/idempiere/commits/17:01
Not-6102[iDempiere] globalqss d4f4d0e - IDEMPIERE-3401 Orderline Discount Context issue17:01
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3401 status set to "Resolved" -assignee set to "carlosruiz_globalqss" -resolution set to "Fixed"17:02
Not-6102[IDEMPIERE] Hi Norbert, the best way to solve this is with a navigate callout. I just committed https://bitbucket.org/idempiere/idempiere/commits/d4f4d0e following that approach. Can you please test and give us feedback. Regards, Carlos Ruiz17:02
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-340117:02
Not-6102[iDempiere4.1] jenkins built #135 completed (failure) http://ci.idempiere.org/job/iDempiere4.1/135/17:31
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-303622:40
Not-6102[IDEMPIERE] Hi [~hieplq], at some point of time seems this was "solved", could not identify which commit did the trick. Now I'm wondering if it is a bug to allow a user to view records with values that he's not allowed, or was better the previous behavior of clearing the field. I feel the actual way is better, but a bit afraid it could lead to some data errors. Let's see if somebody reports a misbehavior later - I22:40
Not-6102think we could close this ticket as solved - don't know how :-)22:40
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-303622:40
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3405 summary set to "When Window Preference doesn't exists then Default Doctype Taken - also from another client"22:40
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-340522:40
Not-6102[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-4.1 [+2/-0/±3] https://bitbucket.org/idempiere/idempiere/commits/22:41
Not-6102[iDempiere] globalqss 66f9559 - IDEMPIERE-3405 When Window Preference doesn't exists then Default Doctype Taken - also from another client / IDEMPIERE-267222:41
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3405 status set to "Resolved" -assignee set to "carlosruiz_globalqss" -resolution set to "Fixed"22:45
Not-6102[IDEMPIERE] Hi [~norbert.bede], I committed https://bitbucket.org/idempiere/idempiere/commits/66f9559 So, now when creating a new record, default values (or copied values) are validated against the valid list. It is possible that this open other issues because of dictionary misconfiguration. For example, after I applied this I caught that the document type field on invoices and orders were not including the document22:45
Not-6102with ID=0 and that was the default. So, I needed to modify that in dictionary to include that record in the valid list. Please test and give feedback about. Regards, Carlos Ruiz22:45
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-340522:45
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-303722:57
Not-6102[IDEMPIERE] Thanks [~hieplq] for checking that code. I noticed that duplicated if/else was introduced here into MMovement: https://bitbucket.org/idempiere/idempiere/commits/e757cb9 and I didn't see it. Now I think [~gabriel.schneider@devcoffee.com.br] copied the code from MMovement into MInOut and is spreading the dup code :-) I'll fix the reversed code, but wondering if there is still and issue here as you reopened22:57
Not-6102this ticket. Can you please describe the issue to review it?22:57
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-303722:57
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-303722:59
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-303722:59
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-303723:00
Not-6102[IDEMPIERE] Thanks [~hieplq] for checking that code. Seems like duplicated code was introduced by a copy/paste from MMovement into MInout I'll fix the reversed code, but wondering if there is still and issue here as you reopened this ticket. Can you please describe the issue to review it?23:00
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-303723:00
Not-6102[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-4.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/23:02
Not-6102[iDempiere] globalqss d3e278e - IDEMPIERE-3037 Date Material Policy not respected in shipment / remove duplicated code (thanks to Hiep)23:03
Not-6102[iDempiere4.1] jenkins built #136 completed (success) http://ci.idempiere.org/job/iDempiere4.1/136/23:04
*** CarlosRuiz has quit IRC23:16
Not-6102[iDempiere4.1] jenkins built #137 completed (success) http://ci.idempiere.org/job/iDempiere4.1/137/23:33

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!