#idempiere IRC log for Wednesday, 2015-04-08

Not-311d[IDEMPIERE] carlosruiz_globalqss created IDEMPIERE-2566 Print Format labels wrong after opening Report Wizard01:23
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-256601:23
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2566 Attachment set to "jira-capture-screenshot-20150407-202136-607.png"01:23
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-256601:23
Not-311d[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/01:25
Not-311d[iDempiere] globalqss b3a7808 - IDEMPIERE-2566 Print Format labels wrong after opening Report Wizard01:25
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2566 status set to "Resolved" -assignee set to "Carlos Antonio Ruiz Gomez" -resolution set to "Fixed"01:25
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-256601:25
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-256101:35
Not-311d[IDEMPIERE] [~hieplq], line 66 is supposed to revert the sysconfig when an exception is found - below comment "reset to N on exception" - perhaps is because the cache is not reset after that?01:35
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-256101:35
Not-311d[iDempiere] CarlosRuiz_globalqss pushed 2 commits to release-2.1 [+0/-0/±8] https://bitbucket.org/idempiere/idempiere/commits/02:07
Not-311d[iDempiere] hieplq 9cd67c5 - IDEMPIERE-2544:to more warning compiler hidden good warning leak resource02:07
Not-311d[iDempiere] globalqss 44e38bb - IDEMPIERE-2447 Migration script from packin adding unnecessary updates / fix issue -> Unknown column reference type=null02:07
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-255502:08
Not-311d[IDEMPIERE] thanks [~hieplq], applied the patch and commented on ticket IDEMPIERE-254402:08
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-255502:08
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2544 status set to "Reopened" -assignee set to "hieplq" -resolution set to "None"02:09
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-254402:09
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2544 status set to "Resolved" -resolution set to "Fixed"02:10
Not-311d[IDEMPIERE] Thank you very much [~hieplq], I applied the patch you uploaded on IDEMPIERE-2555 related to this ticket. Good catch!!02:10
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-254402:10
Not-311d[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.1 [+0/-0/±2] https://bitbucket.org/idempiere/idempiere/commits/02:20
Not-311d[iDempiere] tsvikruha 4199d3a - IDEMPIERE-2238 Printformat header not translated (report)02:20
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2238 status set to "Resolved" -resolution set to "Fixed"02:20
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-223802:20
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2544 status set to "Reopened" -resolution set to "None"02:21
Not-311d[IDEMPIERE] Reopening this - there are still some patches worthy to review (like Dead Code and generic)02:21
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-254402:21
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2544 labels set to "+Patch"02:21
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-254402:21
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2536 status set to "Resolved" -assignee set to "Carlos Antonio Ruiz Gomez" -resolution set to "Won't Fix"02:24
Not-311d[IDEMPIERE] [~steveh], I think this behavior is correct as changing quantity can potentially change the price - because of price list breaks or discount breaks.02:24
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-253602:24
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-252902:36
Not-311d[IDEMPIERE] Hi [~muriloht], I see the issue but don't understand why is it needed - the m_userpreference is just used locally to preselect the records on the lists.02:36
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-252902:36
*** CarlosRuiz has quit IRC02:45
Not-311d[IDEMPIERE] muriloht updated IDEMPIERE-252902:47
Not-311d[IDEMPIERE] Hi Carlos, I don't know if it's a problem or a expected behavior, maybe you can reproduce: - login do demo.idempiere.com as "superuser @ idempiere.com" (check select role) - the default selected Client is "client", then change to "System" and login - Logout - then try to login again with "superuser @ idempiere.com" (again, check select role) - and again the "client" Client is the default selected. The preferences from the last session was02:47
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-252902:47
*** norbertbede has joined #idempiere04:49
*** hieplq has joined #idempiere04:53
*** HansR42 has joined #idempiere06:10
Not-311d[IDEMPIERE] Yu created IDEMPIERE-2567 RfQResponseRank's message is hard coding,cannot translate06:24
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-256706:24
*** hieplq has quit IRC06:31
*** KermitTheFragger has joined #idempiere06:33
*** nmicoud has joined #idempiere06:37
*** a42niem has joined #idempiere06:38
*** KermitTheFragger has quit IRC07:08
*** KermitTheFragger has joined #idempiere07:09
*** Vyrlokar has joined #idempiere07:17
VyrlokarHello everyone07:17
HansR42Hi07:48
*** is-mw has joined #idempiere08:04
Not-311d[IDEMPIERE] hieplq updated IDEMPIERE-2561 status set to "Open"08:11
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-256108:11
Not-311d[IDEMPIERE] hieplq updated IDEMPIERE-2561 status set to "Closed" -resolution set to "Cannot Reproduce"08:13
Not-311d[IDEMPIERE] thanks [~carlosruiz_globalqss], just review code. you correct and hash password do in a transaction. will investigate my issue late.08:13
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-256108:13
*** hieplq has joined #idempiere08:15
*** HansR42 has quit IRC08:50
*** hieplq has quit IRC10:02
*** adnan_T has joined #idempiere10:09
*** is-mw has quit IRC11:05
*** is-mw has joined #idempiere11:37
*** hieplq has joined #idempiere11:49
*** CarlosRuiz has joined #idempiere12:57
*** ChanServ sets mode: +o CarlosRuiz12:57
*** CarlosRuiz has quit IRC12:58
*** CarlosRuiz has joined #idempiere12:58
*** ChanServ sets mode: +o CarlosRuiz12:58
*** ocurieles has joined #idempiere13:00
CarlosRuizGood Morning13:00
nmicoudBonjour13:01
a42niemGuten Tag13:01
*** HideakiHagiwara has joined #idempiere13:01
nmicoudCould you have a look at https://groups.google.com/d/msg/idempiere/ykc9dB-FdHs/y4qIfBo4-XoJ please ? I'd like to know what is the best practice13:01
CarlosRuizsure - brb - give me 5 min13:14
nmicoudthanks13:14
CarlosRuiznmicoud, I get it - but it sounds like a bad practice to use X instead of M13:25
nmicoudhow should i do if i just want to update a record ?13:25
CarlosRuizwhy is that?  I didn't understand -> "as i didn't want to control update"13:25
nmicoudi don't want to get into the beforeSave method13:26
nmicoudas it could failed13:26
nmicoudin fact,i just need to do a "update c_invoice set... = abc where c_invoice_id = 123;"13:26
*** smartjsp has joined #idempiere13:26
CarlosRuizok, I'll change LCO according to that13:28
nmicoudbut means that it should be done in all plugins, right ?13:29
CarlosRuizyes - I have seen the same bug if somebody tries to use GenericPO13:30
nmicoudin my plugin, i just check for the table name, never on the object itself.  So, maybe instead of making a tableName.equals("C_Invoice"), we should do a   po instanceof MInvoice13:30
nmicoudi also see it :D13:30
nmicoudBesides, i haven't had the time to review the ticket for updating the print format name.  Just do the https://idempiere.atlassian.net/browse/IDEMPIERE-2554 (add acknowledgement of receipt on email). If you could see if is ok to integrate.  Also, you send me your POC on Special Editor (ability to update fields for processed records). I think i will try to implement it for our instance.  Would you be ok to publish it on bitbucket in13:33
Vyrlokarnmicoud, heya13:35
nmicoudhi13:36
VyrlokarYou're interested in having payments work on a fixed day, with X days delay, right?13:37
nmicoudyes, date invoiced + x days and then take the last day of month13:37
VyrlokarI'm the guy who replied to your post on the google group13:37
nmicoudah ok :)13:37
VyrlokarOur use cases are slightly different, as you're interested in payments without schedule, while I'm interested in payments with schedule.13:39
Vyrlokarbut we could definitively try and solve both use cases13:39
nmicoudsure13:39
Vyrlokarpayments without schedule are handled exclusively by database functions AFAIK13:39
CarlosRuizsure nmicoud - please publish that13:40
nmicoudyes, there is no field "due date" on c_invoice (could be interesting, but that's another debate :))13:40
nmicoudok, Carlos, will do that13:40
Vyrlokarnmicoud, well, there is, you only need to define a payment term schedule with a single payment term13:41
Vyrlokarthe database function for your use case is paymenttermduedate(paymentterm_id numeric, docdate timestamp with time zone)13:41
Vyrlokarnmicoud, it's weird (but not that unusual, it's IMHO similar to how there's a swing client and a ZK client) that there are two ways to do the same thing, with different features13:43
Vyrlokarschedules don't handle fixed days, at all13:44
CarlosRuiznmicoud, committed and compiled13:45
CarlosRuizhttp://ci.idempiere.org/job/globalqss-iDempiere-LCO/13:45
nmicoudthanks carlos, need to update mine now :)13:45
Vyrlokarnmicoud, On my side, things would be much simpler if the part where due date is calculated was in a function outside of the constructor, as that way, it could be overridden in a module, via a model factory13:49
a42niemwe once created a paymenttermbasedate for a customer so he got some flexibility for strange paymentterm constructs13:51
nmicoudnot simple to implement as it would change the behaviour which was created since compiere times13:51
*** smartjsp has quit IRC13:51
a42niemit needed a callout to set it from invoicedate and change some views13:52
nmicoudbut perhaps we could add a DueDate on C_Invoice (in trunk, using a jira ticket). This would be filled with the sql function. And then you will be able to update it in plugin13:52
Vyrlokarnmicoud, if you're replying to me, then yes, it would be a simple matter of making the constructor call something like "set due date" for the payment term, as that could be then overridden13:53
nmicoudyes Vyrlokar13:53
Vyrlokarnmicoud, invoices don't necessarily have a single due date, that's why making it a field of c_invoice would not work for me13:53
nmicouda42niem : as it need to update views, it's hard to maintain in plugin13:54
nmicoudVyrlokar : due dates  could be found on C_Invoice or C_InvoicePaySchedule13:54
nmicoudaccording to the payment term13:55
Vyrlokarnmicoud, my usual use cases are X% immediate, Y% 30 days, and X% 30 days, Y% 60 days13:55
Vyrlokarall dates with possible fixed day13:55
nmicoudwhen you say 'fixed day', is it equal to 'last day of month' ?13:56
Vyrlokaragain, I solve this with an event handler (not the most elegant solution, but I believe it is the only way it can work without hacking core)13:56
VyrlokarLast day of month is one possibility13:56
*** aguerra has joined #idempiere13:57
Vyrlokarcommon fixed days are 25th, last day of month, and 1st13:57
aguerrabuenos dias a todos!!!13:57
Vyrlokarbuenos dias aguerra13:57
nmicoudhi aguerra13:57
CarlosRuizHi Alejandro13:58
aguerraSalu2 a @Vyrlokar @nmicoud @CarlosRuiz13:58
CarlosRuiznmicoud, I think maybe taking FixMonthDay=31 as last day of month13:58
nmicoudseems it could have problem with february13:59
nmicoudsee https://groups.google.com/d/msg/idempiere/VA3707UDwN8/rQxjrgbehJ4J13:59
CarlosRuizyes - I mean - changing the DB function to use 31 as LDOM14:00
nmicoudi don't understand :(14:01
nmicoudyou want to change the db function ? or just put a differnt value in c_paymentterm ?14:01
CarlosRuizI think actual algorithm doesn't make sense - to say duedate=05/02/15 when you said FixMonthDay = 30 & FixMonthCutOff = 3014:01
VyrlokarCarlosRuiz, what would be the right course of action for requesting a small refactoring of MInvoicePaySchedule, so the dates are handled in a separate function that is then called by the constructor that takes a MInvoice and a MPaySchedule? That way, that function would be overriddable in a plugin, to incorporate advanced functionality14:02
CarlosRuizAntoni, where is it being calculated actually?14:04
Vyrlokarright now, lines 164 to 167 of org.compiere.model.MInvoicePaySchedule in the org.adempiere.base project14:06
Vyrlokarthose 4 lines right now handle the due date calculation for schedules, in a simple way (just add days, no fixed date, no next business day)14:07
VyrlokarCarlosRuiz, as the calculation it done in a nonstandard constructor, I don't see a way to override it in a plugin14:09
*** CarlosRuiz has quit IRC14:10
*** CarlosRuiz has joined #idempiere14:10
*** ChanServ sets mode: +o CarlosRuiz14:10
VyrlokarWB CarlosRuiz14:11
Vyrlokarnmicoud, at least we can curse weird business requirements together ;)14:12
nmicoud:D14:12
CarlosRuizand c_paymentterm.afterdelivery is not used anywhere - another candidate for inactivate14:12
CarlosRuizAntoni, so you can create an EventHandler (ModelValidator) to change the date on beforeSave as I see14:15
VyrlokarCarlosRuiz, That's how I've done it, but it's hackish14:15
CarlosRuizpaymenttermduedate is not taking into account the p.IsNextBusinessDay flag14:16
VyrlokarCarlosRuiz, as it will overwrite user input on the first save, when created manually, IIRC14:16
VyrlokarCarlosRuiz, I created a plugin that extends MInvoicePaySchedule, adds some extra fields and an updateDueDate function, and then use an EventHandler to call that function on PO_AFTER_NEW14:18
VyrlokarI'm testing it right now14:19
* Vyrlokar wonders if his opened a can of worms by asking on payment schedules, as it seems like the code is old...14:20
* Vyrlokar also wonders on the purpose of MOrderPaySchedule - Is it for proforma invoices, or something like that? Speaking of which, how are proforma invoices actually handled in iDempiere?14:21
VyrlokarCarlosRuiz, MOrderPaySchedule looks like a copy of MInvoicePaySchedule with some minor changes, with the original copyright left intact, even if that file never existed in Compiere14:23
CarlosRuizcheck http://www.adempiere.com/FS01_Commitment_AR_AP14:23
VyrlokarCarlosRuiz, thanks, interesting. However, since dates are calculated from order date for orders, and from invoice date for invoices, unless invoice date==order date, the schedules will not match...14:27
* Vyrlokar had to argue with his boss that it's not possible to have the system calculate due dates based on weird things like shipment dates for shipments that don't exist yet, so as they say in his country "Esta curado de espanto"14:29
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2567 labels set to "complex-easy language"14:37
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-256714:37
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-252914:41
Not-311d[IDEMPIERE] [~muriloht] I was not able to reproduce the case you described. Maybe is caused because another person opened a session with superuser in Client while you were there as System I checked the context in both cases and it's not affecting any context variable - that's why I ask if there is a need to use the m_userpreference out of RolePanel.14:41
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-252914:41
*** KermitTheFragger has quit IRC14:59
Not-311d[iDempiere] CarlosRuiz_globalqss pushed 2 commits to release-2.1 [+0/-0/±2] https://bitbucket.org/idempiere/idempiere/commits/15:16
Not-311d[iDempiere] hieplq fadbf6b - IDEMPIERE-2565:NPE when run init client without input city of country has "Allow Cities out of List" = false15:16
Not-311d[iDempiere] globalqss 2a9e282 - IDEMPIERE-2565 - found transaction not rolled back when initial client setup was failing with an exception15:16
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2565 status set to "Resolved" -assignee set to "hieplq" -resolution set to "Fixed"15:16
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-256515:16
*** norbertbede has joined #idempiere15:17
nmicoudgtg, (if you have the time CarlosRuiz, could you look at https://idempiere.atlassian.net/browse/IDEMPIERE-2554) please ? thanks and cu later15:18
*** nmicoud has quit IRC15:19
*** norbertbede has quit IRC15:26
Not-311d[iDempiere] CarlosRuiz_globalqss pushed 1 commit to development [+0/-0/±2] https://bitbucket.org/idempiere/idempiere/commits/15:28
Not-311d[iDempiere] nmicoud 3b5a8fa - IDEMPIERE-2554 Mail : add acknowledgement of receipt15:28
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2554 status set to "Resolved" -assignee set to "Nicolas Micoud" -resolution set to "Fixed"15:28
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-255415:28
Not-311d[IDEMPIERE] hieplq created IDEMPIERE-2568 Can't user build in function of jasper report by miss lib15:31
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-256815:31
Not-311d[IDEMPIERE] hieplq updated IDEMPIERE-2568 status set to "Peer Review Queue"15:32
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-256815:32
Not-311d[iDempiere] CarlosRuiz_globalqss pushed 2 commits to development [+0/-0/±3] https://bitbucket.org/idempiere/idempiere/commits/15:41
Not-311d[iDempiere] globalqss 3b73421 - IDEMPIERE-2554 Mail : add acknowledgement of receipt - minor peer review serialVersionUID15:41
Not-311d[iDempiere] hieplq 2d379de - IDEMPIERE-2563:packout with datatype without sql make NPE15:41
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2563 status set to "Resolved" -assignee set to "hieplq" -resolution set to "Fixed"15:41
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-256315:41
*** HideakiHagiwara has quit IRC15:43
Not-311d[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/15:46
Not-311d[iDempiere] brerp 2c3c76a - ModelValidationEngine - old style validator needed to osgi events work and FireFacts OSGI event is not sent15:46
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2530 status set to "Resolved" -assignee set to "Murilo Habermann Torquato" -resolution set to "Fixed"15:47
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-253015:47
*** adnan_T has quit IRC15:48
*** norbertbede has joined #idempiere15:57
Vyrlokargtg, cya everybody15:57
norbertbedehi alls15:57
CarlosRuizbye Antoni - hi norbertbede15:57
Vyrlokarhi norbertbede  and bye15:57
*** Vyrlokar has quit IRC15:58
norbertbedelooking for experience zk7/idempiere3/nginx(gzip). im getting extremly frequently message Unexpected token < (Syntax Error)15:59
norbertbedereported> https://idempiere.atlassian.net/browse/IDEMPIERE-249615:59
norbertbedeim totally confused :( how to catch15:59
norbertbedei found lot topics about potential issues with javascripts16:00
norbertbedewhen a googled16:00
norbertbedewhen i googled16:01
*** ocurieles has quit IRC16:16
*** allgood has joined #idempiere16:16
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2558 Attachment set to "IDEMPIERE-2558_PR.patch"16:16
Not-311d[IDEMPIERE] Thanks [~hieplq], tests on this went fine but I have some concerns to solve: 1 - attaching a patch with minor fixes IDEMPIERE-2558_PR.patch 2 - the classes X_ must not be changed manually - on next release all the X_ classes are regenerated and your comment about deprecation is lost 3 - In MPasswordHistory.getPasswordHistoryForCheck - the usage of sysconfig USER_LOCKING_MAX_PASSWORD_AGE_DAY is not related - locking is a different feature t16:16
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-255816:16
*** allgood has left #idempiere16:17
*** ocurieles has joined #idempiere16:18
*** norbertbede has quit IRC16:40
*** ocurieles has quit IRC16:43
*** norbertbede has joined #idempiere16:43
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2564 Attachment set to "IDEMPIERE-2564_CR.patch"17:01
Not-311d[IDEMPIERE] Thanks [~tsvikruha] - can you please test the attached alternate patch IDEMPIERE-2564_CR.patch17:01
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-256417:01
Not-311d[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/17:05
Not-311d[iDempiere] hieplq 64c918b - IDEMPIERE-2552:sometime has NPE in CLogErrorBuffer17:05
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2552 status set to "Resolved" -assignee set to "hieplq" -resolution set to "Fixed"17:05
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-255217:05
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2551 Tested By set to "Carlos Antonio Ruiz Gomez"17:20
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-255117:20
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2551 labels set to "+Patch"17:20
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-255117:20
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-256217:27
Not-311d[IDEMPIERE] Hi [~HansR42a], the patch seems outdated with the current code. I don't understand what is the issue - can you please provide steps to reproduce and what is the actual result vs the expected result?17:27
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-256217:27
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-252517:30
Not-311d[IDEMPIERE] it sounds like a massive change to core classes - I think in such case is better to use EventHandler to modify the behavior17:30
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-252517:31
hieplqhi @CarlosRuiz. i has a issue relate "allow null for city when "Allow Cities out of List = false". not sure it's issue or feature,17:32
hieplqmaybe you can confirm https://groups.google.com/forum/#!topic/idempiere/7IDj_fnrQEE17:32
CarlosRuizso, your question is if city must be mandatory?17:33
CarlosRuizI think it must - a location without a city is untraceable17:33
hieplq@CarlosRuiz it's ok if always mandatory but when i "Allow Cities out of List = true" it's don't request input city17:46
CarlosRuizand the city field is filled?17:47
hieplqno.17:48
hieplqi fill same case "Allow Cities out of List = false"17:48
CarlosRuizallow cities out of list = false -> must force to choose a city from the list - is an autocomplete list so it shows the cities when you type the first letters17:50
CarlosRuizallow cities out of list = true -> it means it allows you to type a city in that field that is not on the list17:51
*** nmicoud has joined #idempiere17:52
hieplqthanks, just confirm to know it's issue or feature.17:52
hieplqwith me it still is a issue :D17:53
CarlosRuizis it working that way?17:53
hieplqi know what you mean. but my case is "Allow Cities out of List = false" => user can don't input city and location save success17:55
hieplq"Allow Cities out of List = true" => user don't input city (null) and location save unseccess17:55
hieplqin "Allow Cities out of List = true", user even input nothing in city text.17:57
hieplqi test in location window (exact from init client) not from location dialog17:58
CarlosRuizah - that's intended for location dialog17:58
CarlosRuizyou would need to add validations on initial client setup to control that17:58
hieplqok. i want ask other issue relate config hazelcast.18:00
hieplqi explain at https://groups.google.com/forum/#!topic/idempiere/aWgia827mZ018:00
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-175218:03
Not-311d[IDEMPIERE] [~hieplq], I reproduce the issue about the NPE - but your patch didn't solve it - still throwing the same exception.18:03
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-175218:03
CarlosRuizhieplq, haven't tested using two different ports in adempiere simultaneously18:04
CarlosRuizbut it doesn't sound as hazelcast - maybe zk session?18:05
hieplqmabey hazelcast effect session18:05
CarlosRuizyou can try disabling the hazelcast plugin to check if that's the cause18:05
hieplqit's ok when i access my local instance and demo instance in same browse18:05
hieplqbut it's not importace, will investigate late.18:07
hieplqabout IDEMPIERE-1752 i will re-test it.18:07
hieplqhi @CarlosRuiz, what NPE message you see when test IDEMPIERE-1752.18:15
hieplqif at org.adempiere.base.callout.CostAdjustmentCalloutFactory$CostAdjustmentLineProduct.start(CostAdjustmentCalloutFactory.java:69)18:15
nmicoudCarlosRuiz, i will publish Special Editor sources soon. But i can't find the email you sent me - neither the google forum thread i create for explaining the purpose - could you send me it again please ? as everything was well explain in it, i will copy/paste the content18:16
hieplqyou must resolve IDEMPIERE-1744 to see cast issue of IDEMPIERE-175218:20
hieplqand patch in IDEMPIERE-1752 is for cast issue not NPE issue18:21
*** aguerra has quit IRC18:22
CarlosRuizIDEMPIERE-1744 doesn't have a patch18:25
CarlosRuiznmicoud, is an email from 28/02/201418:26
CarlosRuiztwo emails indeed18:26
nmicoudi think i've deleted it :(18:26
nmicoudor i move it somewhere, but seems a very dark place18:27
CarlosRuizresent both18:27
nmicoudthanks18:27
CarlosRuizbrb - @ lunch18:28
nmicoudbon ap' :)18:28
*** hieplq has quit IRC18:35
norbertbedehi nmicoud are you running already version 3 or still 2,118:36
nmicoudhi norbert  2.118:37
norbertbedeare you planning to upgrade ?18:40
nmicoudnot now18:41
nmicoudmaybe near the end of year, but nothing is planned18:43
norbertbedeok.18:44
norbertbedethanks18:44
Not-311d[IDEMPIERE] norbert.bede created IDEMPIERE-2569 ZK7 - Unexpected token in DSP/CSS19:14
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-256919:14
*** herdem has joined #idempiere19:46
Not-311d[iDempiere] CarlosRuiz_globalqss pushed 3 commits to release-2.1 [+0/-0/±5] https://bitbucket.org/idempiere/idempiere/commits/20:32
Not-311d[iDempiere] nmicoud 70ba3c3 - IDEMPIERE-2235 Report wizard : ability to change the print format name (integrated by Carlos Ruiz, PO.java didn't pass)20:32
Not-311d[iDempiere] globalqss ab74bc8 - IDEMPIERE-2235 Report wizard : ability to change the print format name - peer review20:32
Not-311d[iDempiere] globalqss 86139d7 - IDEMPIERE-1135 Report wizard not clearing order items20:32
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2235 status set to "Resolved" -resolution set to "Fixed"20:34
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-223520:34
nmicoudoh! thanks for integrating #2235 Carlos :)20:34
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1135 status set to "Resolved" -assignee set to "Carlos Antonio Ruiz Gomez" -resolution set to "Fixed"20:34
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-113520:34
CarlosRuizthanks to you for developing that20:35
nmicoud:)20:35
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-253020:37
Not-311d[IDEMPIERE] https://bitbucket.org/idempiere/idempiere/commits/2c3c76a20:37
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-253020:37
*** a42niem has quit IRC20:41
Not-311d[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.1 [+0/-0/±2] https://bitbucket.org/idempiere/idempiere/commits/21:02
Not-311d[iDempiere] globalqss 507e137 - IDEMPIERE-2531 Archive Viewer open EnterQuery tab instead of ViewResult / IDEMPIERE-556 / IDEMPIERE-72421:02
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-253121:02
Not-311d[IDEMPIERE] Found this was caused by a commit on IDEMPIERE-724 that reverted the behavior implemented with IDEMPIERE-55621:02
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-253121:03
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2531 status set to "Resolved" -assignee set to "Carlos Antonio Ruiz Gomez" -resolution set to "Fixed"21:03
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-253121:03
Not-311d[IDEMPIERE] carlosruiz_globalqss created IDEMPIERE-2570 Form reports stacking on same page when issued for several records21:18
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-257021:18
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2570 status set to "Resolved" -resolution set to "Fixed"21:18
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-257021:18
Not-311d[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/21:18
Not-311d[iDempiere] globalqss ab1da70 - IDEMPIERE-2570 Form reports stacking on same page when issued for several records21:18
Not-311d[iDempiere] CarlosRuiz_globalqss pushed 1 commit to development [+0/-0/±20] https://bitbucket.org/idempiere/idempiere/commits/21:22
Not-311d[iDempiere] globalqss 50617e9 - hg merge release-2.1 (merge release2.1 into development)21:22
*** nmicoud has quit IRC21:30
*** norbertbede has quit IRC21:45
Not-311d[iDempiere] CarlosRuiz_globalqss pushed 16 commits to development [+12/-0/±61] https://bitbucket.org/idempiere/idempiere/commits/22:04
Not-311d[iDempiere] globalqss 865ea2d - creating branch experimental22:04
Not-311d[iDempiere] dpansheriya 5af9b8c - IDEMPIERE-2389 Print format and View Report selection on process dialog.22:04
Not-311d[iDempiere] hieplq 0ba4b6e - IDEMPIERE-2389:Print format and Report type selection on process dialog add summary checkbox22:04
Not-311d[iDempiere] hieplq 64e5917 - IDEMPIERE-2389:Print format and Report type selection on process dialog remove duplicate code (IDEMPIERE-1882) remain some issue relate UI22:04
Not-311d[iDempiere] hieplq 003da45 - IDEMPIERE-2389:Print format and Report type selection on process dialog add language combobox22:04
Not-311d[iDempiere] hieplq 46ad165 - Merge with development22:04
Not-311d[iDempiere] hieplq 1697e4f - IDEMPIERE-2532:issue relate show dialog when run process fix to show parameter dialog at center of info window22:04
Not-311d[iDempiere] hieplq be8284b - IDEMPIERE-1657: common show window with mask cover below case + refactor * 1. show a dialog => show mask * 2. process a event * 2.1. show other dialog => show mask * 2.1. hidden dialog in step 1 => hidden mask * 3. result dialog show without mask22:04
Not-311d[iDempiere] hieplq 20eacd2 - IDEMPIERE-2532:issue relate show dialog when run process show message success event hasn't any m_logs show message error same success (use ProcessInfoDialog not use FDialog)22:04
Not-311d[iDempiere] hieplq 737d701 - IDEMPIERE-2389:Print format and Report type selection on process dialog recreate serialVersionUID fix show dialog when auto run process22:04
Not-311d[iDempiere] hieplq 3ddb7cb - IDEMPIERE-2389:Print format and Report type selection on process dialog update Mandatory flag when load parameter value from save22:04
Not-311d[iDempiere] globalqss abc0c8b - IDEMPIERE-2389 Print format and View Report selection on process dialog. / fix oracle migration script22:04
Not-311d[iDempiere] hieplq d289afe - Merge with development22:04
Not-311d[iDempiere] hieplq a8aef8f - IDEMPIERE-2389 Print format and View Report selection on process dialog NPE: when click save at process (not report) parameter22:04
Not-311d[iDempiere] globalqss 32b868d - merge development into experimental22:04
Not-311d[iDempiere] globalqss 89c06bb - merge experimental into development22:04
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-1657 status set to "Resolved" -resolution set to "Fixed"22:06
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-165722:06
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2532 status set to "Resolved" -assignee set to "hieplq" -resolution set to "Fixed"22:07
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-253222:07
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-238922:07
Not-311d[IDEMPIERE] Ready [~hieplq], I integrated the experimental branch into development - please finish the layout of fields directly in development and we can use now the experimental repository for another ticket - maybe all the pending info window changes?22:08
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-238922:08
Not-311d[iDempiere] CarlosRuiz_globalqss pushed 3 commits to development [+0/-0/±71] https://bitbucket.org/idempiere/idempiere/commits/22:52
Not-311d[iDempiere] hieplq a6181c5 - IDEMPIERE-2544:to more warning compiler hidden good warning warning from build.xml22:52
Not-311d[iDempiere] hieplq 62732d5 - IDEMPIERE-2544:to more warning compiler hidden good warning dead code22:52
Not-311d[iDempiere] hieplq 1bab594 - IDEMPIERE-2544:to more warning compiler hidden good warning generic type warning22:52
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2544 status set to "Resolved" -resolution set to "Fixed"22:58
Not-311d[IDEMPIERE] Thanks [~hieplq] for solving all those warnings - I integrated buildWarning, DeadCode and genericWarning. The leakResourceWarning maybe deserves more research to avoid the big impact. I think is worthy to check also all the deprecation from jasper (maybe upgrade jasper library and then solve the deprecations). Also research if is possible to let eclipse know that DB.close is a valid statement to close rs and pstmt.22:58
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-254422:58
Not-311d[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-2.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/23:08
Not-311d[iDempiere] globalqss 20e82fc - IDEMPIERE-2555 Another leak found in CommissionCalc23:08
*** jmpiloq has quit IRC23:14
*** CarlosRuiz has quit IRC23:20
*** jmpiloq has joined #idempiere23:27
Not-311d[IDEMPIERE] hieplq updated IDEMPIERE-2544 status set to "Reopened" -resolution set to "None"23:29
Not-311d[IDEMPIERE] thanks [~carlosruiz_globalqss], will kill deprecation warning. we will update jasper lib to 6.x?23:29
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-254423:29
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2420 status set to "Reopened" -resolution set to "None"23:45
Not-311d[IDEMPIERE] is this issue solved? the is no commit related23:45
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-242023:45
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-2420 status set to "Resolved" -assignee set to "Heng Sin Low" -resolution set to "Fixed"23:46
Not-311d[IDEMPIERE] Ah, sorry - 2 commits linked23:46
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-242023:46
Not-311d[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-249023:47
Not-311d[IDEMPIERE] clicking on the Feedback when Home is shown (with dashboards) kill the server with this error23:47
Not-311d[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-249023:47

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