#idempiere IRC log for Tuesday, 2015-04-21

*** allgood has joined #idempiere00:40
*** allgood has left #idempiere00:41
*** egil0902 has quit IRC01:17
*** Devastator has joined #idempiere02:21
*** hieplq has quit IRC02:23
*** Devastator_ has quit IRC02:23
*** CarlosRuiz has quit IRC03:00
*** Devastator has quit IRC03:27
*** Devastator has joined #idempiere03:27
*** jmpiloq has quit IRC04:57
*** tbayen has quit IRC05:54
*** a42niem has joined #idempiere05:56
*** norbertbede has joined #idempiere06:24
*** KermitTheFragger has joined #idempiere07:06
*** red1 has joined #idempiere08:02
red1Hi a42niem08:08
*** cltech has joined #idempiere08:36
*** cltech has left #idempiere08:38
*** Huma has joined #idempiere09:12
*** hieplq has joined #idempiere09:17
red1xin chao hieplq09:18
red1no news yet about visa?09:18
*** a42niem_ has joined #idempiere11:53
*** CarlosRuiz has joined #idempiere12:57
*** ChanServ sets mode: +o CarlosRuiz12:57
*** red1 has quit IRC13:14
*** carlos_ has joined #idempiere13:19
*** CarlosRuiz has quit IRC13:23
*** carlos_ has quit IRC13:23
*** CarlosRuiz has joined #idempiere13:24
*** ChanServ sets mode: +o CarlosRuiz13:24
*** banym has quit IRC13:55
*** banym has joined #idempiere13:56
*** red1 has joined #idempiere14:28
*** ChanServ sets mode: +o red114:28
*** KermitTheFragger has quit IRC15:09
*** Vyrlokar has joined #idempiere15:23
Vyrlokarheya15:27
CarlosRuizHi Antoni15:28
VyrlokarHello CarlosRuiz15:28
VyrlokarI believe I've found a bug, should I directly create a JIRA ticket and (eventually) a patch? or could someone verify that it's indeed undesired behavior?15:29
CarlosRuizif you're pretty sure is a bug - then you can go to JIRA directly - otherwise maybe better to confirm in forums15:32
Vyrlokarorg.compiere.process.OrderPoCreate, in the org.adempiere.base.process does not verify that the product is sold, just that it has a Product PO with a current vendor. The solution is to change the SQL by adding a join to the product table and checking that the product is sold15:32
Vyrlokaris bought, not sold, sorry15:32
CarlosRuizah - those two flags in product are just informative - sold and bought are not used anywhere AFAIR15:33
red1sounds good Vyrlokar15:33
CarlosRuizissue I see is that adding that will break searches and lists on many people that have not configured those flags because they are useless15:33
red1i think i found the same area few days ago while answering in forum15:34
red1and put forward two JIRAs15:34
VyrlokarCarlosRuiz, It will only break for those using the OrderPoCreate process for products that have vendors but are not marked as purchased15:34
Vyrlokarred1, Got links?15:35
CarlosRuizyes - if you configure a product PO and assign a vendor - is not that enough to know that you're buying it?15:35
VyrlokarCarlosRuiz, Also, it doesn't check that the PorductPo is active15:36
*** red1_ has joined #idempiere15:36
*** ChanServ sets mode: +o red1_15:36
VyrlokarWB Red115:36
CarlosRuizthat one is important - IsActive=Y is mandatory to check15:37
red1_Is IDEMPIERE-258415:37
red1_and IDEMPIERE-258915:38
red1_sorry disconnected and using mobile phone hard to get links15:38
*** red1 has quit IRC15:38
Vyrlokarred1_, no problem15:39
red1_It gives an idea that such area is untested15:39
red1_Where you located Vyrlokar ?15:41
Vyrlokarred1_, It seems that the functionality that I need is always untested ;)15:41
Vyrlokarred1_, Spain15:41
red1_I see. We have some members there before15:41
red1_But haven't heard since15:41
red1_I am now in Austria15:41
VyrlokarCarlosRuiz, So, the IsActive='Y' part merits a jira ticket right?15:45
CarlosRuizyes15:47
CarlosRuizred1 - I think that area is well tested15:49
CarlosRuizand reading 2589 - I don't think a PO inherits necessarily the same delivery rule than a SO15:49
red1_So my patches not needed?15:50
CarlosRuizyou deliver via Shipper - but you can buy picking those items15:50
VyrlokarCya everybody15:59
*** Vyrlokar has quit IRC15:59
*** red1_ has quit IRC16:20
*** jmpiloq has joined #idempiere17:17
*** red1 has joined #idempiere17:20
*** ChanServ sets mode: +o red117:20
*** red1 has quit IRC17:20
*** red1 has joined #idempiere17:21
*** ChanServ sets mode: +o red117:21
*** Not-3656 has joined #idempiere17:23
Not-3656[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.1 [+2/-0/±0] https://bitbucket.org/idempiere/idempiere/commits/17:23
Not-3656[iDempiere] globalqss 95aef58 - IDEMPIERE-2587 Can't login by SuperUser to client with client roles has Use User Org Access / reversing IDEMPIERE-140817:23
Not-3656[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2587 status set to "Resolved" -assignee set to "Carlos Antonio Ruiz Gomez" -resolution set to "Fixed"17:23
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-258717:23
Not-3656[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-140817:24
Not-3656[IDEMPIERE] This ticket was reversed because it stopped a valid configuration to be applied as explained in ticket IDEMPIERE-258717:24
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-140817:24
*** red1_ has joined #idempiere17:25
*** ChanServ sets mode: +o red1_17:25
red1_CarlosRuiz: i get your point about PO from SO can be different delivery17:42
red1_i think his case is DropShipment17:43
red1_i will write him to clarify17:43
red1_But if it is a drop shipment with a shipper defined, then i feel its intuitive to copy those values17:46
Not-3656[IDEMPIERE] red1 updated IDEMPIERE-258918:01
Not-3656[IDEMPIERE] Carlin, I chatted with Carlos just now and he gave some good point that SO and PO maybe different DeliveryVia methods even though it is a Drop shipment. Perhaps you need a Preference or default setting and thus this fix is not really needed? Please argue the case if you think the case has merit.18:01
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-258918:01
CarlosRuizyep - probably if is drop ship some rule can apply - in such case just half of the patch would be needed18:08
Not-3656[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-258918:11
Not-3656[IDEMPIERE] xref https://groups.google.com/d/msg/idempiere/-oQiB-xbYdw/JNBsW4BoE1YJ18:11
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-258918:11
*** red1_ has quit IRC18:18
Not-3656[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.1 [+2/-0/±0] https://bitbucket.org/idempiere/idempiere/commits/18:26
Not-3656[iDempiere] globalqss 700d7fb - IDEMPIERE-2584 Field Sequence items not placed together in Purchase Order18:26
Not-3656[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2584 status set to "Resolved" -assignee set to "Carlos Antonio Ruiz Gomez" -resolution set to "Fixed"18:26
Not-3656[IDEMPIERE] Thanks [~red1], the order was correct - what was wrong is the field group assigned to those fields - I committed a different patch changing the field group18:26
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-258418:27
Not-3656[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2586 status set to "Resolved" -resolution set to "Duplicate"18:34
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-258618:34
Not-3656[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2590 status set to "Resolved" -resolution set to "Won't Fix"18:37
Not-3656[IDEMPIERE] I think this is not a bug but a misconfiguration of the system. When changing the attachment store you need to do a migration process for the actual attachments - otherwise you're messing up the system. There is no direct migration path for the attachments from db to filesystem - but it can be done using 2Pack.18:37
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-259018:37
CarlosRuizAYT hieplq ?18:38
Not-3656[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-84718:41
Not-3656[IDEMPIERE] so [~hieplq], all those messages are not integrated into zk by default? how can they be integrated, is it possible to do it in a fragment plugin?18:41
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-84718:41
Not-3656[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2574 status set to "Resolved" -assignee set to "Carlos Antonio Ruiz Gomez" -resolution set to "Cannot Reproduce"18:42
Not-3656[IDEMPIERE] please reopen if the issue still exists18:42
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-257418:42
*** red1 has quit IRC18:52
*** red1 has joined #idempiere18:53
*** ChanServ sets mode: +o red118:53
*** norbertbede has quit IRC18:54
*** norbertbede has joined #idempiere20:14
Not-3656[IDEMPIERE] jmartin updated IDEMPIERE-258920:22
Not-3656[IDEMPIERE] Redhuan or Carlos, could you elaborate on your thoughts as to why SO and PO DeliveryVia could be different on a Drop Shipment? Here is my thought on the matter, if a SO is created as a Drop Ship then the customer may have dictated a specific shipper that they want used, or a specific shipping account would need to be used based on special rates that have been negotiated, just some possible scenarios. If that is not carried through to the P20:22
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-258920:22
Not-3656[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-258920:28
Not-3656[IDEMPIERE] Thanks [~jmartin], the description didn't mention it's for drop shipment and the patch is also beyond dropship as I see. I think for dropship can make sense what you suggest.20:28
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-258920:28
Not-3656[IDEMPIERE] jmartin updated IDEMPIERE-258920:35
Not-3656[IDEMPIERE] Thank you Carlos. Question then, would this fix for Drop Ship SO DeliveryVia to carry over to PO from "Generate PO from SO" be logical to add to core? Just for Drop Ship only.20:35
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-258920:35
Not-3656[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-258920:38
Not-3656[IDEMPIERE] Ah sorry - my fault - I think [~red1] patch is ok - let me test it20:38
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-258920:38
Not-3656[IDEMPIERE] sjeffen updated IDEMPIERE-258620:41
Not-3656[IDEMPIERE] this patch actually solve an issue with an error in the generated stylesheet: - cssContent.append(".adtab-form-borderlayout .z-south-colpsd:before { "); + cssContent.append(".adtab-form-borderlayout .z-south-collapsed:before { "); reopening this on again.20:41
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-258620:41
Not-3656[IDEMPIERE] sjeffen updated IDEMPIERE-2586 status set to "Reopened" -resolution set to "None"20:41
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-258620:41
Not-3656[IDEMPIERE] sjeffen updated IDEMPIERE-2586 status set to "Peer Review Queue"20:42
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-258620:42
Not-3656[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/20:58
Not-3656[iDempiere] globalqss 60757dd - IDEMPIERE-2589 Generate PO From Sales Order function and deliveryviarule field inconsistencies / based on patch provided by Redhuan (red1)20:58
Not-3656[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2589 status set to "Resolved" -resolution set to "Fixed"20:58
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-258920:58
Not-3656[IDEMPIERE] creed.gds updated IDEMPIERE-258921:03
Not-3656[IDEMPIERE] Thank you, gentlemen. I am sorry I didn't have my own testing feedback yet. I just still have to learn some of the finer points on tasks such as how to apply a patch to mercurial for testing, etc. so I am kind of slow-going at this time.21:03
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-258921:04
*** banym has quit IRC21:06
*** banym has joined #idempiere21:06
Not-3656[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-258622:04
Not-3656[IDEMPIERE] Thanks [~sjeffen], then the description of the ticket and the attached image doesn't correspond with the patch. Checking this page http://books.zkoss.org/wiki/ZK%20Style%20Guide/XUL%20Component%20Specification/Borderlayout/Default%20(South) it's mentioned the css .z-south-colpsd but not .z-south-collapsed - is the doc wrong?22:04
Not-3656[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-258622:04
*** a42niem has quit IRC22:07
*** RecordingBot has joined #idempiere22:48
Not-3656[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/23:20
Not-3656[iDempiere] globalqss 16f9287 - IDEMPIERE-379 - minor usability on labels behaving as buttons23:20
*** carlos_ has joined #idempiere23:32
*** CarlosRuiz has quit IRC23:33
*** norbertbede has quit IRC23:40
*** carlos_ has quit IRC23:52
*** CarlosRuiz has joined #idempiere23:53
*** ChanServ sets mode: +o CarlosRuiz23:53

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!