#idempiere IRC log for Wednesday, 2017-11-15

*** RubenCR has quit IRC04:23
*** a42niem has joined #idempiere05:01
*** CarlosRuiz has joined #idempiere06:00
*** ItSolution has joined #idempiere06:15
ItSolutionGood morning06:16
*** nmicoud has joined #idempiere06:34
CarlosRuizGood morning ItSolution06:44
*** CarlosRuiz has left #idempiere06:44
*** CarlosRuiz has joined #idempiere06:44
*** silphium has quit IRC06:46
ItSolutionCongrats  to all community the new release06:47
nmicoudHi all06:51
CarlosRuizHi nmicoud06:51
ItSolutionComparing the code, i saw a lot of changes and improvement, i would be glade to know if there if a new business functionalities that was added06:51
ItSolutionEx: price and cost management06:51
ItSolutionA new business flow06:51
ItSolutionThanks06:52
CarlosRuizItSolution, you usually see bug fixes on those areas, and some improvements06:53
CarlosRuizfor big things usually they are done in plugins06:53
CarlosRuiziDempiere tends to be more decentralized - we encourage an ecosystem of plugins - instead of trying to push everything into core06:54
nmicoudCarlosRuiz, could you have a look at #3554 (i don't know how it can be fixed) and #3515 (there's a patch attached, but which can be improved) ?07:01
ItSolutionYes better, in the last iDempiere conference also, you mentioned some features that should iDempiere handle, like price based on ASI, how also iDempiere could be handled by a small businesses, is this points style in agenda ?07:01
CarlosRuizwill take a look nmicoud07:02
nmicoudthanks07:03
CarlosRuizItSolution, haven't seen any movement on those areas07:03
ItSolutionOkey07:12
*** ItSolution has quit IRC07:33
*** mderout has joined #idempiere08:11
Not-ba81[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3515 assignee set to "Nicolas Micoud"08:13
Not-ba81[IDEMPIERE] Hi [~nmicoud], Idea is really good, I think that's a big add-on to Info Windows, my suggestions: - the export button must be shown just if the role has permission to export - it must not export the ID column - I think we must follow the same approach that we have in windows, implementing some interface called IInfoWindowExporter - similar to IGridTabExporter, but instead of receiving a GridTab as08:13
Not-ba81parameter I guess we would need to pass maybe the InfoWindow object (to have accesss to the query, columns, etc) - about your patch when I tried product info window with warehouse and without price list it threw an error: java.lang.IndexOutOfBoundsException: Index: 12, Size: 12 Regards, Carlos Ruiz08:13
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-351508:13
Not-ba81[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3554 assignee set to "Nicolas Micoud"08:20
Not-ba81[IDEMPIERE] Hi [~nmicoud], any test case failing? I tried setting a different zoom in table and it worked. Regards, Carlos Ruiz08:20
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-355408:20
Not-ba81[IDEMPIERE] nmicoud updated IDEMPIERE-355408:25
Not-ba81[IDEMPIERE] Duplicate Business Partner Window Remove access to Business Partner (original one) Give access to Business Partner (new one) Login with the role, open InfoWindow BP and try to zoom. It fails (just done in test.idempiere.org through GardenUser) Nicolas08:25
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-355408:25
Not-ba81[IDEMPIERE] nmicoud updated IDEMPIERE-355408:26
Not-ba81[IDEMPIERE] Duplicate Business Partner Window Remove access to Business Partner (original one) Give access to Business Partner (new one) Login with the role, open InfoWindow BP and try to zoom. It fails (just done in test.idempiere.org through GardenUser) Log says : 09:23:53.442===========> GridWindowVO.create: No Window - AD_Window_ID=123, AD_Role_ID=MRole[103,GardenWorld User,UserLevel= O,AD_Client_ID08:26
Not-ba81IN(0,11),AD_Org_ID IN(0,11,50000,50001,50007)] - SELECT Name,Description,Help,WindowType, AD_Color_ID,AD_Image_ID,WinHeight,WinWidth, IsSOTrx, AD_Window_UU FROM AD_Window w WHERE w.AD_Window_ID=? AND w.IsActive='Y' [637] 09:23:53.635===========> CLogger.saveError: AccessTableNoView - (Not found) [637] Nicolas08:26
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-355408:26
Not-ba81[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-355408:59
Not-ba81[IDEMPIERE] It's working fine in test.idempiere.org08:59
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-355408:59
nmicoudCarlosRuiz : about #3554, did you do something special ?  or just login and zoom directly ?09:02
CarlosRuizcache reset09:02
CarlosRuizrole access update09:03
CarlosRuizthen edit the garden user role and inactivate the bp access09:03
CarlosRuizmaybe you forgot cache reset?  role definition is cached09:03
nmicoudlook at the log ; at 09:58 :    09:58:22.952===========> GridWindowVO.create: No Window - AD_Window_ID=123, AD_Role_ID=MRole[103,GardenWorld User,UserLevel=  O,AD_Client_ID IN(0,11),AD_Org_ID IN(0,11,50000,50001,50007)] - SELECT Name,Description,Help,WindowType, AD_Color_ID,AD_Image_ID,WinHeight,WinWidth, IsSOTrx, AD_Window_UU FROM AD_Window w WHERE w.AD_Window_ID=? AND w.IsActive='Y' [26] 09:58:23.122===========> CLogger.saveErro09:04
nmicoudthis error is generated by InfoWindow (when trying to determine if NewRecord button should be added)09:04
nmicoudMaybe the cache reset is the key09:05
CarlosRuizso - we can close the ticket better?09:07
nmicoudyep09:09
nmicoudwait09:09
nmicoudmaybe we should handle the error ?09:10
CarlosRuizwhich error?09:10
nmicoudI mean, why InfoWindow tries to open WindowID = 123 instead of the copy ?09:10
nmicoud09:5809:10
CarlosRuizyou say that's for the New button09:10
nmicoudyes09:11
CarlosRuizand must be looking for the zoom window from ad_table - which sounds correct - no?09:11
nmicoudno, the new button should open the window using zoom condition09:11
CarlosRuizno - because is New - there is no condition yet to evaluate09:11
CarlosRuizsame behavior on BP field on sales order09:12
CarlosRuizcannot add a new BP on sales order - because there is no access to the zoom window09:12
nmicoudthere can be some "special" condition like @AD_Client_ID@ = 45609:12
nmicoudi see09:12
nmicoudin those case, must be handled via plugin (which extends InfoBPartnerWindow) ?09:13
CarlosRuizwhat I understand is that the "quick edit" window is not looking for any zoom condition09:14
CarlosRuizdifferent discovery09:14
CarlosRuizthat would be an improvement maybe - to look for context based zoom09:14
nmicoudI think the QuickEntry panel should use same logic as Zoom09:15
nmicoudI'm not familiar with QuickEntry, looking at code09:16
nmicoudThe idea would be to use Env.getZoomWindowID instead of "direct" zoom (from AD_Table), right ?09:18
Not-ba81[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3554 status set to "Resolved" -resolution set to "Cannot Reproduce"09:18
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-355409:18
CarlosRuizyes - I think that's right09:21
nmicoudok, i'll create the ticket then09:21
CarlosRuizI think that must be on InfoWindow.getADWindowID - for the quick entry in info window09:23
CarlosRuizand MLookup.getZoom - for the quick entry in windows09:23
nmicoudok, i'll write those hint in the ticket09:24
Not-ba81[IDEMPIERE] nmicoud created IDEMPIERE-3561 QuickEntry panel doesn't use ZoomCondition09:28
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-356109:28
Not-ba81[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-5.1 [+0/-0/±2] https://bitbucket.org/idempiere/idempiere/commits/09:46
Not-ba81[iDempiere] globalqss 17aec2a - IDEMPIERE-3559 DrillDown report didn't work because of two bug / integrate suggestions from Denis Kuznetsov09:46
Not-ba81[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3559 status set to "Resolved" -assignee set to "Carlos Ruiz" -resolution set to "Fixed"09:46
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-355909:46
Not-ba81[iDempiere5.1] jenkins built #22 completed (success) http://ci.idempiere.org/job/iDempiere5.1/22/10:05
*** CarlosRuiz has quit IRC11:16
*** mderout has quit IRC12:28
*** CarlosRuiz has joined #idempiere15:25
*** nmicoud has quit IRC16:05
*** CarlosRuiz has quit IRC16:39
Not-ba81[IDEMPIERE] hieplq updated IDEMPIERE-3551 status set to "Reopened" -assignee set to "hiep lq" -resolution set to "None"17:50
Not-ba81[IDEMPIERE] i see this ticket remove "Bundle-ActivationPolicy: lazy", i guess to resolve "cyclic dependencies" but it let service IDictionaryService is not yet registry, so stop 2pack until start bundle "org.adempiere.pipo" "cyclic dependencies" reason Bundle-Activator of org.adempiere.pipo get reference to IDictionaryService but IDictionaryService also lie on "org.adempiere.pipo" refer explain here:17:50
Not-ba81https://wiki.eclipse.org/Lazy_Start_Bundles https://www.osgi.org/developer/design/lazy-start/ i think solution for "cyclic dependencies" is IDEMPIERE-3542 and maybe IDEMPIERE-3463 for now work-around is keep current status and let org.adempiere.pipo auto start.17:50
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-355117:50
Not-ba81[IDEMPIERE] hieplq updated IDEMPIERE-3551 Attachment set to "IDEMPIERE-3551-work-around.patch"17:51
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-355117:51
Not-ba81[IDEMPIERE] hieplq updated IDEMPIERE-354217:51
Not-ba81[IDEMPIERE] it's a good practise https://www.ibm.com/developerworks/websphere/techjournal/1007_charters/1007_charters.html#sec517:51
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-354217:51
Not-ba81[IDEMPIERE] hieplq updated IDEMPIERE-3542 status set to "Peer Review Queue"18:11
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-354218:11
Not-ba81[IDEMPIERE] hieplq updated IDEMPIERE-3542 labels set to "+Patch"18:11
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-354218:11
Not-ba81[IDEMPIERE] hieplq updated IDEMPIERE-3542 status set to "Open"18:11
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-354218:11
Not-ba81[IDEMPIERE] hieplq updated IDEMPIERE-3542 labels set to ""18:11
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-354218:11
Not-ba81[IDEMPIERE] hieplq updated IDEMPIERE-3551 status set to "Peer Review Queue"18:12
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-355118:12
Not-ba81[IDEMPIERE] hieplq updated IDEMPIERE-3551 labels set to "+Patch"18:12
Not-ba81[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-355118:12
*** sudeep has joined #idempiere19:31
*** a42niem has quit IRC21:30

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