*** RubenCR has quit IRC | 04:23 | |
*** a42niem has joined #idempiere | 05:01 | |
*** CarlosRuiz has joined #idempiere | 06:00 | |
*** ItSolution has joined #idempiere | 06:15 | |
ItSolution | Good morning | 06:16 |
---|---|---|
*** nmicoud has joined #idempiere | 06:34 | |
CarlosRuiz | Good morning ItSolution | 06:44 |
*** CarlosRuiz has left #idempiere | 06:44 | |
*** CarlosRuiz has joined #idempiere | 06:44 | |
*** silphium has quit IRC | 06:46 | |
ItSolution | Congrats to all community the new release | 06:47 |
nmicoud | Hi all | 06:51 |
CarlosRuiz | Hi nmicoud | 06:51 |
ItSolution | Comparing 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 added | 06:51 |
ItSolution | Ex: price and cost management | 06:51 |
ItSolution | A new business flow | 06:51 |
ItSolution | Thanks | 06:52 |
CarlosRuiz | ItSolution, you usually see bug fixes on those areas, and some improvements | 06:53 |
CarlosRuiz | for big things usually they are done in plugins | 06:53 |
CarlosRuiz | iDempiere tends to be more decentralized - we encourage an ecosystem of plugins - instead of trying to push everything into core | 06:54 |
nmicoud | CarlosRuiz, 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 |
ItSolution | Yes 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 |
CarlosRuiz | will take a look nmicoud | 07:02 |
nmicoud | thanks | 07:03 |
CarlosRuiz | ItSolution, haven't seen any movement on those areas | 07:03 |
ItSolution | Okey | 07:12 |
*** ItSolution has quit IRC | 07:33 | |
*** mderout has joined #idempiere | 08: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 as | 08:13 |
Not-ba81 | parameter 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 Ruiz | 08:13 |
Not-ba81 | [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-3515 | 08: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 Ruiz | 08:20 |
Not-ba81 | [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-3554 | 08:20 |
Not-ba81 | [IDEMPIERE] nmicoud updated IDEMPIERE-3554 | 08: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) Nicolas | 08:25 |
Not-ba81 | [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-3554 | 08:25 |
Not-ba81 | [IDEMPIERE] nmicoud updated IDEMPIERE-3554 | 08: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_ID | 08:26 |
Not-ba81 | 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' [637] 09:23:53.635===========> CLogger.saveError: AccessTableNoView - (Not found) [637] Nicolas | 08:26 |
Not-ba81 | [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-3554 | 08:26 |
Not-ba81 | [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3554 | 08:59 |
Not-ba81 | [IDEMPIERE] It's working fine in test.idempiere.org | 08:59 |
Not-ba81 | [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-3554 | 08:59 |
nmicoud | CarlosRuiz : about #3554, did you do something special ? or just login and zoom directly ? | 09:02 |
CarlosRuiz | cache reset | 09:02 |
CarlosRuiz | role access update | 09:03 |
CarlosRuiz | then edit the garden user role and inactivate the bp access | 09:03 |
CarlosRuiz | maybe you forgot cache reset? role definition is cached | 09:03 |
nmicoud | look 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.saveErro | 09:04 |
nmicoud | this error is generated by InfoWindow (when trying to determine if NewRecord button should be added) | 09:04 |
nmicoud | Maybe the cache reset is the key | 09:05 |
CarlosRuiz | so - we can close the ticket better? | 09:07 |
nmicoud | yep | 09:09 |
nmicoud | wait | 09:09 |
nmicoud | maybe we should handle the error ? | 09:10 |
CarlosRuiz | which error? | 09:10 |
nmicoud | I mean, why InfoWindow tries to open WindowID = 123 instead of the copy ? | 09:10 |
nmicoud | 09:58 | 09:10 |
CarlosRuiz | you say that's for the New button | 09:10 |
nmicoud | yes | 09:11 |
CarlosRuiz | and must be looking for the zoom window from ad_table - which sounds correct - no? | 09:11 |
nmicoud | no, the new button should open the window using zoom condition | 09:11 |
CarlosRuiz | no - because is New - there is no condition yet to evaluate | 09:11 |
CarlosRuiz | same behavior on BP field on sales order | 09:12 |
CarlosRuiz | cannot add a new BP on sales order - because there is no access to the zoom window | 09:12 |
nmicoud | there can be some "special" condition like @AD_Client_ID@ = 456 | 09:12 |
nmicoud | i see | 09:12 |
nmicoud | in those case, must be handled via plugin (which extends InfoBPartnerWindow) ? | 09:13 |
CarlosRuiz | what I understand is that the "quick edit" window is not looking for any zoom condition | 09:14 |
CarlosRuiz | different discovery | 09:14 |
CarlosRuiz | that would be an improvement maybe - to look for context based zoom | 09:14 |
nmicoud | I think the QuickEntry panel should use same logic as Zoom | 09:15 |
nmicoud | I'm not familiar with QuickEntry, looking at code | 09:16 |
nmicoud | The 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-3554 | 09:18 |
CarlosRuiz | yes - I think that's right | 09:21 |
nmicoud | ok, i'll create the ticket then | 09:21 |
CarlosRuiz | I think that must be on InfoWindow.getADWindowID - for the quick entry in info window | 09:23 |
CarlosRuiz | and MLookup.getZoom - for the quick entry in windows | 09:23 |
nmicoud | ok, i'll write those hint in the ticket | 09:24 |
Not-ba81 | [IDEMPIERE] nmicoud created IDEMPIERE-3561 QuickEntry panel doesn't use ZoomCondition | 09:28 |
Not-ba81 | [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-3561 | 09: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 Kuznetsov | 09: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-3559 | 09:46 |
Not-ba81 | [iDempiere5.1] jenkins built #22 completed (success) http://ci.idempiere.org/job/iDempiere5.1/22/ | 10:05 |
*** CarlosRuiz has quit IRC | 11:16 | |
*** mderout has quit IRC | 12:28 | |
*** CarlosRuiz has joined #idempiere | 15:25 | |
*** nmicoud has quit IRC | 16:05 | |
*** CarlosRuiz has quit IRC | 16: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-ba81 | https://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-3551 | 17: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-3551 | 17:51 |
Not-ba81 | [IDEMPIERE] hieplq updated IDEMPIERE-3542 | 17:51 |
Not-ba81 | [IDEMPIERE] it's a good practise https://www.ibm.com/developerworks/websphere/techjournal/1007_charters/1007_charters.html#sec5 | 17:51 |
Not-ba81 | [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-3542 | 17: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-3542 | 18:11 |
Not-ba81 | [IDEMPIERE] hieplq updated IDEMPIERE-3542 labels set to "+Patch" | 18:11 |
Not-ba81 | [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-3542 | 18:11 |
Not-ba81 | [IDEMPIERE] hieplq updated IDEMPIERE-3542 status set to "Open" | 18:11 |
Not-ba81 | [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-3542 | 18:11 |
Not-ba81 | [IDEMPIERE] hieplq updated IDEMPIERE-3542 labels set to "" | 18:11 |
Not-ba81 | [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-3542 | 18: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-3551 | 18:12 |
Not-ba81 | [IDEMPIERE] hieplq updated IDEMPIERE-3551 labels set to "+Patch" | 18:12 |
Not-ba81 | [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-3551 | 18:12 |
*** sudeep has joined #idempiere | 19:31 | |
*** a42niem has quit IRC | 21:30 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!