#idempiere IRC log for Tuesday, 2017-07-04

Not-6102[IDEMPIERE] soo.fang@gmail.com updated IDEMPIERE-3419 Attachment set to "3419.patch"02:51
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-341902:51
Not-6102[IDEMPIERE] soo.fang@gmail.com updated IDEMPIERE-3419 assignee set to "hengsin"02:51
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-341902:51
Not-6102[IDEMPIERE] hieplq created IDEMPIERE-3421 wtable dir editor load wrong list03:39
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-342103:39
Not-6102[IDEMPIERE] hieplq updated IDEMPIERE-3421 description set to "1. open "Table and Column", navigate to "C_Order" record 2. go to column AD_Org_ID of tab column 3. un-check "Updatable" 4. login to GardenWorld by role "GardenWorld Admin" 5. open Role window and navigate to "GardenWorld User" record 6. change "User Level" to "Client+Organization" 7. go to "Org Access" tab, create new record, set Organization = * 8. de-05:05
Not-6102active other record just keep HQ and * active so this role only can access to Organization * and HQ 9. login to GardenWorld by role "GardenWorld User" and Organization = * 10. click to new sale order now Organization field just have "*" value on drop down list key to redo is: 1. un-check "Updatable" 2. role only can access to Organization * and HQ 3. login to GardenWorld by role "GardenWorld User" and Organization =05:05
Not-6102* 4. default logic = @#AD_Org_ID@ (already set) if you do refresh on field Organization, list re-load correct value, have only HQ on drop down list "05:05
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-342105:05
Not-6102[IDEMPIERE] hieplq updated IDEMPIERE-342105:16
Not-6102[IDEMPIERE] this issue happen by combine of three step: 1. when new sale order, by default logic Organization = * (login org) ever * is out of correct value of this field (already filter out by Dynamic Validation) when init wtabledireditor it see this field isn't readwrite (updateable = false) so it do direct query to load value 2. go far it will reload filed by dynamic display but logic on05:16
Not-6102WTableDirEditor.dynamicDisplay avoid it reload {code:java} if ((lookup != null) && (!lookup.isValidated() || !lookup.isLoaded() || (isReadWrite() && lookup.getSize() != getComponent().getItemCount()))) {code} lookup.getSize() = 1 because it have HQ value getComponent().getItemCount() also = 1 because it have * value to resolve this issue i think we have validate default value before set to field05:16
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-342105:16
Not-6102[IDEMPIERE] hieplq updated IDEMPIERE-303605:26
Not-6102[IDEMPIERE] when investigate IDEMPIERE-3421, i guess this issue take effect from IDEMPIERE-340505:26
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-303605:26
*** mbozem has joined #idempiere07:34
*** CarlosRuiz has joined #idempiere07:46
Not-6102[iDempiere] hengsin pushed 1 commit to release-4.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/08:19
Not-6102[iDempiere] hengsin 41d351d - IDEMPIERE-3350 Accounting Processor GL postings to "secondary" schema unreliable. Posted not updated after posting completed with error ( it stay as N ).08:19
Not-6102[IDEMPIERE] hengsin updated IDEMPIERE-335008:32
Not-6102[IDEMPIERE] https://bitbucket.org/idempiere/idempiere/commits/41d351d - Posted not updated after posting completed with error ( it stay as N ).08:32
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-335008:32
*** nmicoud has joined #idempiere08:40
Not-6102[iDempiere4.1] jenkins built #144 completed (success) http://ci.idempiere.org/job/iDempiere4.1/144/08:46
Not-6102[IDEMPIERE] nmicoud created IDEMPIERE-3422 More info for error message "Failed to update user record with date last login"09:22
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-342209:22
Not-6102[IDEMPIERE] nmicoud updated IDEMPIERE-3422 Attachment set to "IDEMPIERE-3422.patch"09:23
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-342209:23
Not-6102[IDEMPIERE] nmicoud updated IDEMPIERE-3422 assignee set to "carlosruiz_globalqss"09:23
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-342209:23
Not-6102[IDEMPIERE] nmicoud updated IDEMPIERE-3422 status set to "Peer Review Queue"09:23
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-342209:23
Not-6102[IDEMPIERE] nmicoud updated IDEMPIERE-342209:24
Not-6102[IDEMPIERE] Hi [~carlosruiz_globalqss], Please review this patch which indicates which on which use the save fails Regards, Nicolas09:24
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-342209:24
Not-6102[iDempiere] hengsin pushed 1 commit to release-4.1 [+0/-0/±2] https://bitbucket.org/idempiere/idempiere/commits/09:26
Not-6102[iDempiere] hengsin a8388a8 - IDEMPIERE-3352 Enhancement of Batch Level Costing. Fixed checking of Reversal_ID not working when productionplan is use.09:26
Not-6102[IDEMPIERE] hengsin updated IDEMPIERE-3352 status set to "Resolved" -resolution set to "Fixed"09:28
Not-6102[IDEMPIERE] https://bitbucket.org/idempiere/idempiere/commits/a8388a8 - Fixed checking of Reversal_ID not working when productionplan is use.09:28
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-335209:28
Not-6102[iDempiere4.1] jenkins built #145 completed (success) http://ci.idempiere.org/job/iDempiere4.1/145/09:46
Not-6102[iDempiere] hengsin pushed 1 commit to release-4.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/09:48
Not-6102[iDempiere] hengsin 13ef720 - IDEMPIERE-3419 Wrong Account date in Matched PO if invoice and MR in different periods09:48
Not-6102[IDEMPIERE] hengsin updated IDEMPIERE-3419 status set to "Resolved" -Fix Version set to "4.1" -resolution set to "Fixed"09:48
Not-6102[IDEMPIERE] https://bitbucket.org/idempiere/idempiere/commits/13ef72009:48
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-341909:49
*** maudud has joined #idempiere09:50
Not-6102[iDempiere4.1] jenkins built #146 completed (success) http://ci.idempiere.org/job/iDempiere4.1/146/10:17
*** nmicoud has quit IRC10:25
Not-6102[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-4.1 [+0/-0/±3] https://bitbucket.org/idempiere/idempiere/commits/10:43
Not-6102[iDempiere] globalqss 023496f - IDEMPIERE-3421 wtable dir editor load wrong list10:43
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3421 status set to "Resolved" -assignee set to "carlosruiz_globalqss" -resolution set to "Fixed"10:43
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-342110:43
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-303610:44
Not-6102[IDEMPIERE] Commit [023496f|https://bitbucket.org/idempiere/idempiere/commits/023496f] from IDEMPIERE-3421 reopened this issue, now is again refreshing correctly the list.10:44
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-303610:44
Not-6102[IDEMPIERE] hieplq updated IDEMPIERE-335210:47
Not-6102[IDEMPIERE] Hi [~hengsin] below code will rise cast exception when have plugin implement IModelFactory for table M_ProductionLine. {code:java} MProductionLine productionLine = (MProductionLine) getM_ProductionLine(); {code} i'm don't correct remember test case, but productionLine can be null [^IDEMPIERE-3352_1750.patch] do same idea but don't meet above issue.10:47
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-335210:47
*** aguerra has joined #idempiere10:58
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-342211:10
Not-6102[IDEMPIERE] [~nmicoud], any clue in the console why the update of AD_User is failing?11:10
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-342211:10
Not-6102[iDempiere4.1] jenkins built #147 completed (success) http://ci.idempiere.org/job/iDempiere4.1/147/11:15
Not-6102[IDEMPIERE] nmicoud updated IDEMPIERE-342211:16
Not-6102[IDEMPIERE] I only got this in log : 10:54:02.824-----------> MUser_Tgi.save: Validation failed - java.lang.NullPointerException [88] 10:54:02.825===========> MUser_Tgi.saveError: Error - java.lang.NullPointerException [88] 10:54:02.825===========> Login.getClients: Failed to update user record with date last login [88] So i guess this is because of my plugin but i haven't debugged it - as i don't know which user got11:16
Not-6102issue.11:16
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-342211:16
*** nmicoud has joined #idempiere11:17
Not-6102[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-4.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/11:24
Not-6102[iDempiere] nmicoud 3900c43 - IDEMPIERE-3422 More info for error message "Failed to update user record with date last login"11:24
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3422 status set to "Resolved" -assignee set to "Nicolas Micoud" -resolution set to "Fixed"11:24
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-342211:24
nmicoudHi CarlosRuiz, and thanks ; I don't know if you have time ATM, but can you also have a look at tickets 3411 (fix list of print format according to report view) and 3360 (ability to get PO/BP/Use of the mail text) ?11:27
CarlosRuizHi nmicoud - will try11:33
nmicoudthanks Carlos11:34
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3418 status set to "Resolved" -resolution set to "Incomplete"11:35
Not-6102[IDEMPIERE] Hi [~mbozem], I think this cannot be integrated. There can be integration with other systems that trust in such code to assign the price, also I think is correct if assigning the price then the UOM of the same price must be assigned. I've seen this problem happening with some integrations and the workaround is usually to set a price list if I recall correctly. Regards, Carlos Ruiz11:35
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-341811:35
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-341011:40
Not-6102[IDEMPIERE] [~hieplq], I think your test case is not reproducible - if there is a reproducible test case where disallow negative inventory is set and a storage is saved negative then is a bug (we recently fixed one in production doc). About the sync between the logic in validation and the logic in storage - if the data can be fixed after set (maybe with a Storage Cleanup, or a movement, or a physical inventory) that11:40
Not-6102would be OK, if the data cannot be fixed then we have a problem.11:40
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-341011:40
Not-6102[iDempiere] CarlosRuiz_globalqss pushed 1 commit to development [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/11:44
Not-6102[iDempiere] nmicoud 18a9aab - IDEMPIERE-3360 Methods to retrieve values of mail template11:44
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3360 status set to "Resolved" -assignee set to "Nicolas Micoud" -resolution set to "Fixed"11:44
Not-6102[IDEMPIERE] committed in dev11:44
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-336011:44
Not-6102[iDempiere4.1] jenkins built #148 completed (success) http://ci.idempiere.org/job/iDempiere4.1/148/11:46
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-341111:50
Not-6102[IDEMPIERE] Hi [~nmicoud], I'm not sure this is a problem or a nice feature to have - why do you want to hide it? Is there value added? Actually other people could think that is valuable to be able to run reports based on the table.11:50
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-341111:50
Not-6102[iDempiere] jenkins built #1843 completed (success) http://ci.idempiere.org/job/iDempiere/1843/11:56
Not-6102[IDEMPIERE] nmicoud updated IDEMPIERE-341111:58
Not-6102[IDEMPIERE] In my use case, i have several reports which execute java class in order to populate a temporary table and then iDempiere display the records using AD_PInstance. ATM, I have about 350 columns. So I use ReportView (and news table AD_ReportView_Column) to display only column that are treated by the process. Means one report view per report (Report1 -> RV 1 ; Report2 -> RV2, ...). So when launching a report11:58
Not-6102from the menu, i can select any print format based on the table. I would expect to select only print format according to the report view. Columns displayed in RV1 will not be consistant with columns populated by Report2. The patch only filter if a reportView is set on the report. Or did i miss something ?11:58
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-341111:58
Not-6102[iDempiere] hengsin pushed 1 commit to release-4.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/12:04
Not-6102[iDempiere] hengsin 25ed4b5 - IDEMPIERE-3352 Enhancement of Batch Level Costing. Use safer approach for production reversal checking.12:04
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-341112:06
Not-6102[IDEMPIERE] I see your problem, but still it sounds like a problem created by your approach about managing the report view, and could break a feature that others are using.12:06
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-341112:06
Not-6102[IDEMPIERE] hengsin updated IDEMPIERE-335212:07
Not-6102[IDEMPIERE] https://bitbucket.org/idempiere/idempiere/commits/25ed4b5 - Use safer approach for production reversal checking.12:07
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-335212:07
Not-6102[IDEMPIERE] nmicoud updated IDEMPIERE-341112:09
Not-6102[IDEMPIERE] Ok, then I will patch my local version - not a problem - but maybe there is a kind of "best practice" i am not aware of ?12:09
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-341112:09
Not-6102[iDempiere4.1] jenkins built #149 completed (success) http://ci.idempiere.org/job/iDempiere4.1/149/12:15
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-341112:30
Not-6102[IDEMPIERE] [~nmicoud], not trying to be a problem for you - just concerned about breaking something that is useful for others.12:30
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-341112:30
Not-6102[IDEMPIERE] nmicoud updated IDEMPIERE-341112:36
Not-6102[IDEMPIERE] Yes, i know Carlos, no problem :) A solution would be to add another Sysconfig key... or a new checkbox on ReportView or AD_Process ("Display only printformats based on this ReportView") in order to keep all behaviours wdyt ?12:36
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-341112:36
Not-6102[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-341112:48
Not-6102[IDEMPIERE] Yes, a tenant SysConfig could do it better12:48
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-341112:48
Not-6102[IDEMPIERE] nmicoud updated IDEMPIERE-341112:49
Not-6102[IDEMPIERE] Ok, will update the patch with the new key thanks, Nicolas12:49
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-341112:49
Not-6102[IDEMPIERE] nmicoud updated IDEMPIERE-3411 Attachment set to "IDEMPIERE-3411_v2.patch"13:51
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-341113:51
Not-6102[IDEMPIERE] nmicoud updated IDEMPIERE-341113:51
Not-6102[IDEMPIERE] [~carlosruiz_globalqss], Here the updated patch with the SysConfig key ! Thanks & Regards, Nicolas13:51
Not-6102[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-341113:51
*** mbozem has quit IRC14:19
*** nmicoud has quit IRC15:01
*** possy has quit IRC15:28
*** possy has joined #idempiere15:28
*** CarlosRuiz has quit IRC17:51
*** ecelis has joined #idempiere20:15
*** aguerra has quit IRC22:35

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