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..
egil0902_ghintec: Yes
egil0902_ghintec: that it whats hapening
egil0902_ghintec: *what it was hapening
CarlosRuiz: jenkins is now compiling