IDempiere/FullMeeting20130911 - Revision history

CarlosRuiz: drop JIRA notifications from log

2013-09-11T16:10:46Z

drop JIRA notifications from log

CarlosRuiz: full meeting

2013-09-11T16:10:12Z

full meeting

New page

⇐
[[IDempiere|Table of Contents]] |
[[IDempiere/Full Meeting Minutes|Full Meeting Minutes]] |
Full Meeting 2013-09-11


'''''CarlosRuiz''''': Good Morning 
'''''nmicoud''''': Bonjour 
'''''mhernandezve''''': Good Morning 
'''''CarlosRuiz''''': long list to peer review today :-) 
'''''CarlosRuiz''''': https://bitbucket.org/trekglobal/idempiere/compare/release-1.0c..idempiere/idempiere:release-1.0c 
'''''nmicoud''''': I would like to implement a new stuff for trees : a list on tabs to allow showing tree when tab is in master, in detail or both. Do you think this can be included in trunk? 
'''''CarlosRuiz''''': it sounds useful 
'''''nmicoud''''': ok, it's not 100% operational, 
'''''nmicoud''''': will upload a patch when it will be ok 
'''''JanThielemann''''': CarlosRuiz and nmicoud, is the following a security issue? 
'''''JanThielemann''''': We have a limited tree assigned to a role. A user does for example has no possibility to open the price list window. However, if there is a place to zoom into a pricelist (which is at product price level) it is possible to open that window 
'''''JanThielemann''''': i know there is also a role configuration to allow/prohibit a role from opening a window 
'''''JanThielemann''''': i'm just curious about this 
'''''CarlosRuiz''''': yes - restricting the menu doesn't restrict the access 
'''''nmicoud''''': i guess you have to use the window access tab of role 
'''''CarlosRuiz''''': you need to restrict the window access too 
'''''JanThielemann''''': so this is the original intention and no error 
'''''CarlosRuiz''''': I think so 
'''''nmicoud''''': that could allow to have a 'lighter' menu ; but allowing to go everywhere if access are authorized 
'''''JanThielemann''''': i think it might confuse a user if he zoomed into a window he doesn't have in his tree and then later tries to find that window again :> 
'''''JanThielemann''''': "damnit, where is this f***n price list window?!" 
'''''JanThielemann''''': :) 
'''''nmicoud''''': the implementor has to explain why ;) 
'''''nmicoud''''': Carlos, if you have some time (or want to do something else), could you have a look at ticket 1338 ? And espacially at comment #2 (bug with city list). I think the solution must be simple, but actually, i can't find it 
'''''JanThielemann''''': Carlos, if i read that right, the idempiere main repository moved to hengsins bitbucket but you just postet a idempiere repo from trekglobal... sooo, which one is now the real trunk? 
'''''CarlosRuiz''''': is that a different bug? I think is better to open a new jira 
'''''nmicoud''''': in fact, i don't think it's a bug, but i guess i heven't implement correctly the panel 
'''''CarlosRuiz''''': ah, so you're using the quick entry for that button 
'''''nmicoud''''': actually not, i'm using the WBPartner 
'''''nmicoud''''': but in the future, i think it should open the QuickEntry 
'''''nmicoud''''': so, the button could be implemented on every info window 
'''''CarlosRuiz''''': I though idea was to zoom to window in New mode 
'''''nmicoud''''': if you have opened the info window from menu, yes 
'''''nmicoud''''': but if you open from a field (C_BPartner), i think it's better to create it rapidly and fill the field when validating the form 
'''''nmicoud''''': you want to create a sales order, you search the BP, don't find him and then click on Create => panel open, you write important fields and validate. BP is created and field is filled. You can proceed on the creation 
'''''nmicoud''''': without changing the active window 
'''''CarlosRuiz''''': so - we would assume that the user wanted a quick entry instead of a zoom 
'''''nmicoud''''': otherwise, he would do a right click on the field and open the window (and no need to use info window) 
'''''CarlosRuiz''''': ah, I see, you're using a deprecated window 
'''''nmicoud''''': yes 
'''''nmicoud''''': temporarely, the goal is to use WQuickEntry 
'''''nmicoud''''': but i think the problem related to the city list bug will be the same, as we open a lot of forms 
'''''CarlosRuiz''''': I think better try to use WQuickEntry instead of trying to fix a deprecated class 
'''''CarlosRuiz''''': and yes, I think this idea can be general 
'''''CarlosRuiz''''': having a new button on all info windows 
'''''CarlosRuiz''''': and maybe we can have a "quick entry" button when the related window has quick entry 
'''''nmicoud''''': means a new button => open the window in creation AND a quickentry button => quick entry if available ? 
'''''CarlosRuiz''''': maybe new and quick entry buttons can be enabled just when the query returns zero records 
'''''nmicoud''''': not necesseraly , 
'''''nmicoud''''': if you are searching a Mr Smith 
'''''nmicoud''''': you can find 10 
'''''nmicoud''''': but you need the 11th 
'''''CarlosRuiz''''': ah, yes 
'''''nmicoud''''': but i think the city list bug will still be there as i've modified the way if calls WLocationDialog 
'''''CarlosRuiz''''': I noticed also you added a parameter to a public method - in such case is better to preserve the old method calling the new method with new parameter as null 
'''''nmicoud''''': right, i will update the patch 
'''''egil0902_ghintec''''': good morning to everyone 
'''''CarlosRuiz''''': Hi Eduardo 
'''''egil0902_ghintec''''': i carlos 
'''''egil0902_ghintec''''': i have a problem with de 2pack. imade a packout and put it into a 2pack on the municipality and parish pluging that i made. but when i install the plugin it does not work 
'''''egil0902_ghintec''''': but 
'''''egil0902_ghintec''''': when i make packing it works perfectly 
'''''CarlosRuiz''''': is maybe related to this fix? 
'''''CarlosRuiz''''': https://bitbucket.org/trekglobal/idempiere/commits/33fc9a0 
'''''CarlosRuiz''''': I'm just about to commit it 
'''''egil0902_ghintec''''': let me see.. 
'''''Not-002''''': [iDempiere] CarlosRuiz_globalqss pushed 19 commits to development [+4/-0/±18] https://bitbucket.org/idempiere/idempiere/commits/ 
'''''Not-002''''': [iDempiere] globalqss fd1988e - IDEMPIERE-1323 Zoom Condition bug 
'''''Not-002''''': [iDempiere] globalqss 2a91f28 - IDEMPIERE-1025 Improve translation for custom tables / do not insert translations if the language is not installed 
'''''Not-002''''': [iDempiere] globalqss a034371 - IDEMPIERE-1270 PaymentFormOnCredit is not translating payment terms / based on patch from Naim Berisha 
'''''Not-002''''': [iDempiere] hengsin 0c95923 - IDEMPIERE-1297 2Pack: Remove code for the lookup of record and reference using business key. Fixed handling of column that reference zero id table. 
'''''Not-002''''': [iDempiere] hengsin 4f4b835 - Merge a03437121c3f 
'''''Not-002''''': [iDempiere] hengsin 5d57d85 - IDEMPIERE-1297 2Pack: Remove code for the lookup of record and reference using business key. Fixed NPE in previous revision. 
'''''Not-002''''': [iDempiere] hengsin 33fc9a0 - Fixed auto packin doesn't work with attachment due to invalid package directory setting. 
'''''Not-002''''': [iDempiere] hengsin 2da3fe2 - IDEMPIERE-1343 GL Journal Generator Show Ok When no journal have been created. 
'''''Not-002''''': [iDempiere] hengsin 78718d2 - Minor: added null check to prevent runtime NPE. 
'''''Not-002''''': [iDempiere] hengsin a61e05f - IDEMPIERE-1347 2Pack: ColumnElementHandler should ensure column exists in database when IsSyncDatabase=Y. 
'''''Not-002''''': [iDempiere] hengsin 50aa749 - IDEMPIERE-1305 Payment allocation form- Need alignment of field to be corrected. 
'''''Not-002''''': [iDempiere] hengsin 942fb09 - IDEMPIERE-1304 Document allow to select price list without version. Commit patch from Richard Morales Herrera. 
'''''Not-002''''': [iDempiere] hengsin 1f85bff - IDEMPIERE-1313 Web Client: Reduce space used by collapsed context help pane. Better implementation that doesn't take up any space on the right when the help pane is collapsed. 
'''''Not-002''''': [iDempiere] globalqss 666e806 - IDEMPIERE-1345 Allow Translation Export from webui 
'''''Not-002''''': [iDempiere] hengsin 8368695 - IDEMPIERE-1350 Deleting a line from the "Payments into Batch" window. 
'''''Not-002''''': [iDempiere] hengsin bffe980 - IDEMPIERE-1351 Quick Entry when Field can not zoom to 2 different windows. 
'''''Not-002''''': [iDempiere] hengsin 404decc - Merged release-1.0c into development 
'''''Not-002''''': [iDempiere] hengsin 63fe9e9 - IDEMPIERE-1331 Save and new Button. 
'''''Not-002''''': [iDempiere] hengsin a5eb498 - Merged release-1.0c into development 
'''''egil0902_ghintec''''': Yes 
'''''egil0902_ghintec''''': that it whats hapening 
'''''egil0902_ghintec''''': *what it was hapening 
'''''CarlosRuiz''''': jenkins is now compiling 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1270 status set to "Resolved" -resolution set to "Fixed" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1270 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1343 status set to "Resolved" -resolution set to "Fixed" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1343 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1347 
'''''Not-002''''': [IDEMPIERE] Pending to check comments at https://bitbucket.org/trekglobal/idempiere/commits/a61e05f 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1347 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1305 status set to "Resolved" -resolution set to "Fixed" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1305 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1304 status set to "Resolved" -assignee set to "Richard Morales Herrera" -resolution set to "Fixed" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1304 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1345 status set to "Resolved" -resolution set to "Fixed" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1345 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1350 status set to "Resolved" -assignee set to "Juliana Corredor T." -resolution set to "Fixed" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1350 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1351 
'''''Not-002''''': [IDEMPIERE] Pending to check comments at https://bitbucket.org/trekglobal/idempiere/commits/bffe980 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1351 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1331 status set to "Resolved" -resolution set to "Fixed" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1331 
'''''Not-002''''': [IDEMPIERE] nmicoud created IDEMPIERE-1353 Choose to display trees on master, detail or both tabs 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1353 
'''''Not-002''''': [IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1348 assignee set to "Carlos Antonio Ruiz Gomez" 
'''''Not-002''''': [IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-1348

@@ Line 73: / Line 73: @@
 '''''CarlosRuiz''''': I'm just about to commit it<br>
 '''''egil0902_ghintec''''': let me see..<br>
 '''''egil0902_ghintec''''': Yes<br>
 '''''egil0902_ghintec''''': that it whats hapening<br>
 '''''egil0902_ghintec''''': *what it was hapening<br>
 '''''CarlosRuiz''''': jenkins is now compiling<br>