#idempiere IRC log for Friday, 2017-10-27

Not-520a[iDempiereDaily] jenkins built #540 completed (failure) http://ci.idempiere.org/job/iDempiereDaily/540/02:35
*** aoslee has joined #idempiere02:53
*** nmicoud has joined #idempiere06:03
*** a42niem has joined #idempiere06:09
Not-520a[IDEMPIERE] mpowacht updated IDEMPIERE-154607:45
Not-520a[IDEMPIERE] Hi Carlos, I debugged the issue deeper and here is what a found: It seems my fragment class "com.xxxxxxxxxx.idempiere.plugin.HSBCiFilePmtExport" (with host plug-in=org.adempiere.ui.zk) is found but can't be loaded: m_PaymentExport = NULL after calling PayPrint.java=> "Class<?> clazz = Class.forName (m_PaymentExportClass); m_PaymentExport = (PaymentExport)clazz.newInstance();" If I remove the class from07:45
Not-520abank account, the system loads the generic payment export class and the button is not greyed out. I implement the PaymentExport interface with method signature "public int exportToFile(MPaySelectionCheck[] checks, File file, StringBuffer err)" and I return "return noLines" I also updated my class to include the new methods according to your new interface PaymentExport and rebuilt the fragment. Any ideas? Michael07:45
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-154607:45
Not-520a[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-154608:58
Not-520a[IDEMPIERE] Hi [~mpowacht], as a fragment - the test I did I set the host plugin to org.adempiere.base - you say yours is in org.adempiere.ui.zk, that can be the cause about not being able to load it. This option is intended for backward compatibility, so the old classes must keep working, you don't need to change them, or add new methods. If you change your class to use/add the new way, then do I think is better you08:58
Not-520amigrate to the plugin approach (not a fragment). You can see the attachment de.bozem.sepa_ModifiedNewApproach.tbz for an example about how to implement it.08:58
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-154608:58
Not-520a[IDEMPIERE] mpowacht updated IDEMPIERE-154609:14
Not-520a[IDEMPIERE] Hi Carlos, adding to above, I just realized I wouldn't have had to implement these methods as they were defined as Java8-"default" methods in interface PaymentExport. So the problem must lie somewhere else.09:14
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-154609:14
Not-520a[IDEMPIERE] mpowacht updated IDEMPIERE-154609:15
Not-520a[IDEMPIERE] Thanks Carlos, I'll check that out if it's possible via host plug-in org.adempiere.base.09:15
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-154609:15
Not-520a[IDEMPIERE] tbayen updated IDEMPIERE-352610:07
Not-520a[IDEMPIERE] I found this bug while we used the bank statement import table to create/modifiy entries. This bug is very annoying for us. Our first impression was that the bug did not activate the "Save" button. But testing showed that the combobox fields do not set any value. That means when I save the record the written values are different from what I see on the screen. That is not acceptable. The bug arises at10:07
Not-520aediting old records and also at new created records.10:07
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-352610:07
Not-520a[IDEMPIERE] tbayen updated IDEMPIERE-352610:09
Not-520a[IDEMPIERE] I found this bug while we used the bank statement import table to create/modifiy entries. This bug is very annoying for us. Our first impression was that the bug did not activate the "Save" button. But testing showed that the combobox fields do not set any value. That means when I save the record the written values are different from what I see on the screen. That is not acceptable. The bug arises at10:09
Not-520aediting old records and also at new created records. The bug was not in our old version before the update.10:09
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-352610:09
Not-520a[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3526 priority set to "Critical"10:28
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-352610:28
*** CarlosRuiz has joined #idempiere14:18
*** norbertbede has joined #idempiere14:21
norbertbedehi all14:21
norbertbede@CarlosRuiz, what you mean about REST implementation14:21
norbertbedei know some changes on vaadin online, solution, however most of today js works in REST json world14:22
norbertbedewould like to hear your opintion14:23
CarlosRuizHi norbertbede14:46
CarlosRuizyou mean to write a client using JSON/REST?14:46
CarlosRuizI think what Pedro Rozo wrote in forums is right14:47
CarlosRuizhttps://groups.google.com/d/msg/idempiere/_7Sljegs_qc/v633olRJAgAJ14:47
CarlosRuizSimilar topic I also found this reading very interesting:14:51
CarlosRuizhttp://www.tonymarston.net/php-mysql/3-tier-architecture.html#different-presentation-layers14:51
Not-520a[IDEMPIERE] hieplq created IDEMPIERE-3531 move idempiere build to maven tycho15:00
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-353115:00
norbertbedei mean improve or create new endpoint for REST data operation15:05
norbertbedebased on true REST approach15:06
Not-520a[IDEMPIERE] hieplq updated IDEMPIERE-353015:06
Not-520a[IDEMPIERE] my opinion is get rib of "is active" on inventory line make it consistent with other *line. in case you have to use it, change code on MInventory.getLines to .setOrderBy(MInventoryLine.COLUMNNAME_Line).setOnlyActiveRecords(true).15:06
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-353015:06
Not-520a[IDEMPIERE] hieplq updated IDEMPIERE-353015:07
Not-520a[IDEMPIERE] hi [~carlosruiz_globalqss], do you know any reason to have isActive on Inventory line?15:07
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-353015:07
norbertbedewe are slowly working on IONIC mobile client but we have issues with communication, while mobile app frameworks are supports REST and JSON15:07
norbertbedeprobably you heard something from others trek etc. else we start to work on it15:07
Not-520a[IDEMPIERE] nmicoud created IDEMPIERE-3532 Remove hardcoded values for InfoInvoiceWindow (BPartnerID / IsSOTrx) and InfoPaymentWindow (IsSOTrx)15:10
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-353215:10
CarlosRuizno, norbertbede, haven't seen that - what I've seen is work on specific apps - not a generic mobile client15:22
Not-520a[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-353015:24
Not-520a[IDEMPIERE] [~hieplq], is *mpiere model - all tables must have IsActive flag and an inactive record must be considered as deleted. So, I think we have a lot of bugs like what [~aconn] is reporting here - most queries on functional stuff not filtering IsActive can be considered an issue. Regards, Carlos Ruiz15:24
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-353015:24
Not-520a[IDEMPIERE] nmicoud updated IDEMPIERE-3532 Attachment set to "IDEMPIERE-3532.patch"15:26
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-353215:26
Not-520a[IDEMPIERE] nmicoud updated IDEMPIERE-3532 assignee set to "Carlos Ruiz"15:26
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-353215:26
Not-520a[IDEMPIERE] nmicoud updated IDEMPIERE-3532 status set to "Peer Review Queue"15:26
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-353215:26
Not-520a[IDEMPIERE] nmicoud updated IDEMPIERE-353215:27
Not-520a[IDEMPIERE] Hi [~carlosruiz_globalqss], The patch removes hardcoded values and replace them with default values on the InfoColumn Regards, Nicolas15:27
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-353215:27
*** CarlosRuiz has quit IRC15:28
*** CarlosRuiz has joined #idempiere15:28
Not-520a[IDEMPIERE] nmicoud updated IDEMPIERE-3532 labels set to "+Patch"15:28
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-353215:28
*** norbertbede has quit IRC15:42
*** CarlosRuiz has quit IRC16:39
Not-520a[IDEMPIERE] hieplq updated IDEMPIERE-353016:42
Not-520a[IDEMPIERE] document table have status column for delete case (reverse status) and almost *line don't show isActive column. so i think for this case just hidden isActive column16:43
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-353016:43
*** nmicoud has quit IRC16:46
*** norbertbede has joined #idempiere17:15
*** ecelis has left #idempiere17:26
*** norbertbede has quit IRC18:07
Not-520a[IDEMPIERE] hieplq created IDEMPIERE-3533 reuse browse tab when open idempiere from a destop app (or ever from other web app)18:35
Not-520a[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-353318:35
*** a42niem has quit IRC23:09

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