IDempiere/FullMeeting20130911

⇐ Table of Contents | 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