#idempiere IRC log for Tuesday, 2019-03-12

*** a42niem <a42niem!~niem@2003:a:57d:3200:3885:8824:a8d3:906c> has joined #idempiere06:19
*** nmicoud <nmicoud!~Nicolas@riv38-2-78-235-108-177.fbx.proxad.net> has joined #idempiere06:57
*** brent- <brent-!~brent@207.89.23.133> has joined #idempiere07:12
*** brent_ <brent_!~brent@c-73-96-165-11.hsd1.wa.comcast.net> has quit IRC07:12
*** a42niem <a42niem!~niem@2003:a:57d:3200:3885:8824:a8d3:906c> has quit IRC08:14
*** CarlosRuiz <CarlosRuiz!~CarlosRui@aftr-37-201-193-12.unity-media.net> has joined #idempiere09:34
*** xapiens <xapiens!~xapiens@190.236.204.78> has joined #idempiere10:32
*** tgi_nmicoud <tgi_nmicoud!~Nicolas@riv38-2-78-235-108-177.fbx.proxad.net> has joined #idempiere12:57
*** nmicoud <nmicoud!~Nicolas@riv38-2-78-235-108-177.fbx.proxad.net> has quit IRC13:00
tgi_nmicoudHello CarlosRuiz, did you get my email with the pictures ?13:04
*** tgi_nmicoud is now known as nmicoud13:07
CarlosRuizHi nmicoud - yes :-)  very funny pictures13:12
nmicoud:D13:12
nmicoudCarlosRuiz: did you see https://groups.google.com/forum/#!topic/idempiere/XZdZq4MQ2UM ? I will have to manage it, so... as always... Jira ticket ?13:36
CarlosRuizyes, it sounds like a improvement13:42
nmicoudOk, i will do that13:42
CarlosRuizwhat you mention about callout doesn't sound too generic13:42
CarlosRuizI think it would be better something more generic - if the window has a field with any record filled must be taken into account13:43
CarlosRuizfor New - and also for the Update case13:43
nmicoudAh... you mean the change must be done in the QuickEntry panel directly ?13:43
nmicoudand if called from C_BPartner_ID field, then it should fill BPLocation_ID and AD_User_ID fields ?13:44
nmicoudWith I guess a hardcoded rule for C_Order.InvoiceBPartnerID13:44
CarlosRuizI haven't seen how it works for the invoiceBP case13:47
CarlosRuizyes - sounds like a hardcoded rule13:47
CarlosRuizand yes - I mean in general - not just BP - the case can be extended for other windows too13:47
nmicoudSo idea is to add automatisms when the QuickEntry panel closes, right ?13:51
nmicoudNot creating other callouts13:51
CarlosRuizI don't remember how is the mechanism how QuickEntry is filling the field value13:55
CarlosRuizbut yes - wherever that mechanism happens - try to extend it13:56
nmicoudthere is a actionSave method, but I think callouts of tables are also called. Will have a  look13:57
CarlosRuizno - no callouts13:58
CarlosRuizbecause that's working in all windows13:59
*** RecordingBot` <RecordingBot`!~supybot@www.globalqss.com> has joined #idempiere14:01
tgi_nmicoudAnd yes, callouts are called ; see in actionSave (around line 419) -> // Call all callouts to allow quick entry special behavior14:04
CarlosRuizI think the thing is in WTableDirEditor and WSearchEditor14:05
CarlosRuizwhere you see this line14:05
CarlosRuizint result = vqe.getRecord_ID();14:05
*** nmicoud <nmicoud!~Nicolas@riv38-2-78-235-108-177.fbx.proxad.net> has quit IRC14:09
*** RecordingBot <RecordingBot!~supybot@www.globalqss.com> has quit IRC14:09
tgi_nmicoudWhat do you mean by "the thing is" ?14:10
tgi_nmicoudah ok, I get it14:15
Not[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3914 status set to "Resolved" -resolution set to "Cannot Reproduce"14:43
Not[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-391414:43
tgi_nmicoudCarlosRuiz : debugging here. If i open the QE panel from Invoice window, the CalloutInvoice.bpartner is called when the QE closes. But AFAIU, callouts are called because a change is done on the field (they are not called 'directly' by the QE panel)14:46
CarlosRuizcorrect14:49
CarlosRuizthat's what I understand too14:49
tgi_nmicoudThat's why my 1st idea was to add a callout for opportunity. Less generic, but easier to handle specific rules (such as InvoiceBPartner_ID)14:51
CarlosRuizyep - that sounds like a customization easier to write - but for core would be better a generic solution14:56
tgi_nmicoudWhat I propose : First, write the OpportunityCallout and then create another ticket to have a generic way.15:03
tgi_nmicoudBut the migration will not be easy as I think all implementations have specific callouts. So they should be migrated into the "new generic"15:03
CarlosRuizdidn't understand - the callout will be your customization15:04
tgi_nmicoudAh, ok15:04
CarlosRuizand new generic is not about callouts - is about quick entry behavior  :)15:04
tgi_nmicoudyes, new generic will replace what callouts are doing now (fill location from bpartner)15:05
CarlosRuizbut I think the callout on BPartner is not filling the user from the quick entry - must be filling is something like "the first user found"15:09
tgi_nmicoudThat's right. But it's ok when you open the QE panel in "New" mode15:09
tgi_nmicoudas there is only one contact15:10
*** brent- <brent-!~brent@207.89.23.133> has quit IRC16:27
*** brent_ <brent_!~brent@207.89.23.133> has joined #idempiere16:31
Not[IDEMPIERE] hieplq created IDEMPIERE-3915 support customize workflow node on workflow editor16:34
Not[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-391516:34
Not[IDEMPIERE] hieplq updated IDEMPIERE-3915 Sprint set to "" -labels set to "+Patch" -Attachment set to "IDEMPIERE-3915.patch"16:37
Not[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-391516:37
Not[IDEMPIERE] hieplq updated IDEMPIERE-3915 status set to "Peer Review Queue"16:37
Not[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-391516:37
*** tgi_nmicoud <tgi_nmicoud!~Nicolas@riv38-2-78-235-108-177.fbx.proxad.net> has quit IRC16:43
*** CarlosRuiz <CarlosRuiz!~CarlosRui@aftr-37-201-193-12.unity-media.net> has quit IRC17:37
Not[IDEMPIERE] hieplq updated IDEMPIERE-3915 description set to "[~jray] sponsor to make a graph to show workflow status of document. almost done on plugin (ssh://hg@bitbucket.org/idplugin/tw.idempiere.document.wf.graph). i reuse WFEditor for draw graph so need to improve core for easy to extend this issue to manage that change"17:39
Not[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-391517:39
*** a42niem <a42niem!~niem@2003:a:57d:3200:3885:8824:a8d3:906c> has joined #idempiere17:55
*** a42niem <a42niem!~niem@2003:a:57d:3200:3885:8824:a8d3:906c> has quit IRC21:07
*** CarlosRuiz <CarlosRuiz!~CarlosRui@aftr-37-201-193-12.unity-media.net> has joined #idempiere21:44
*** CarlosRuiz <CarlosRuiz!~CarlosRui@aftr-37-201-193-12.unity-media.net> has quit IRC22:38

Generated by irclog2html.py 2.17.1 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!