#idempiere IRC log for Wednesday, 2017-01-18

*** giv has joined #idempiere00:25
*** silphium has joined #idempiere02:48
Not-4af[iDempiere4.1Daily] jenkins built #33 completed (failure) http://ci.idempiere.org/job/iDempiere4.1Daily/33/05:22
*** nmicoud has joined #idempiere07:24
*** CarlosRuiz has joined #idempiere07:43
*** a42niem has joined #idempiere07:45
*** CarlosRuiz_ has joined #idempiere08:38
*** a42niem has quit IRC08:39
*** CarlosRuiz has quit IRC08:39
Not-4af[iDempiere4.1] jenkins built #51 completed (success) http://ci.idempiere.org/job/iDempiere4.1/51/11:48
*** aguerra has joined #idempiere11:59
*** red1 has joined #idempiere12:15
*** ChanServ sets mode: +o red112:15
*** SmartJSP has joined #idempiere12:52
*** red1 has quit IRC13:04
*** jdpaniagua has quit IRC13:18
*** SmartJSP has quit IRC13:29
*** mbozem18 has joined #idempiere13:29
aguerraGood morning !!!!13:42
mbozem18Hello13:54
*** CarlosRuiz_ has quit IRC13:54
*** CarlosRuiz has joined #idempiere13:54
mbozem18I have made a patch for including iban columns and fields into idempiere (IDEMPIERE-3039). And i would like to ask about your opinion.14:03
CarlosRuizHi mbozem1814:06
mbozem18Hi CarlosRuiz14:07
CarlosRuizI think swift code is related to bank - so - doesn't sound necessary in all those tables if we already have it in bank14:07
mbozem18But i think its for documentation. For example im c_payment, you can always see what swift code was used.14:08
CarlosRuizit sounds redundant14:10
CarlosRuizadding C_BP_BankAccount.IBAN sounds right14:15
mbozem18i did the implementation like the existing one. I put the swift code to payment, so that the export process is very similar to the existing one (which uses RoutingNo).14:16
mbozem18In Germany you even don´t need the SWIFT-Code anymore, when you transfer money to european countries.14:17
CarlosRuizand to make it European maybe we would need to add C_Bank.BIC14:17
CarlosRuizyep, you need BIC & IBAN for SEPA transfers14:18
mbozem18I think BIC and SWIFT are the same14:18
mbozem18Your don´t need a BIC anymore14:18
mbozem18only outside europe i think14:19
mbozem18SWIFT is the company and BIC is the code14:19
CarlosRuizseems like BIC = SWIFT14:20
CarlosRuizI would suggest to: C_BP_BankAccount.IBAN C_Payment.IBAN I_Payment.IBAN (if somebody still uses old importers) C_PaymentTransaction.IBAN -> C_POSPayment is not required as that's for sales make them optional and in payment window use it with Direct Deposit tender type (same as Account No) And add nmicoud validation on IDEMPIERE-1200 to those IBAN fields14:24
CarlosRuizmulti-line paste  here :-)14:24
CarlosRuizlet me copy each line separate14:24
CarlosRuizI would suggest to:14:24
CarlosRuizC_BP_BankAccount.IBAN14:24
CarlosRuizC_Payment.IBAN14:24
CarlosRuizI_Payment.IBAN (if somebody still uses old importers)14:24
CarlosRuizC_PaymentTransaction.IBAN14:24
CarlosRuiz-> C_POSPayment is not required as that's for sales14:24
CarlosRuizmake them optional and in payment window use it with Direct Deposit tender type (same as Account No)14:24
CarlosRuizAnd add nmicoud validation on IDEMPIERE-1200 to those IBAN fields14:24
mbozem18Ok. I can change this in the patch.14:25
mbozem18For me it is ok. When somebody else needs SWIFTCode it can be added later.14:26
CarlosRuizah I see14:27
CarlosRuizcan be IBAN+SwiftCode - that's ok14:27
CarlosRuizseems is just not necessary in c_pospayment14:27
CarlosRuizso - sounds ok - reviewing your patch ....14:29
CarlosRuizfor your patch I would suggest:14:29
CarlosRuiz- exclude C_POSPayment from the patch14:29
mbozem18ok14:29
CarlosRuizMBPBankAccount.getSwiftCode sounds wrong - not necessary - must return the added column14:30
mbozem18I didn´t add a field SWIFTCode to C_BP_BankAccount because, it is handled like RoutingNo.14:32
CarlosRuizI see - agree14:34
mbozem18There is a field RoutingNo in C_BP_BankAccount which overrides the RoutingNo in Bank14:34
CarlosRuizyep14:35
CarlosRuizI think also as you already added those to C_POSPayment - is fine to leave them there14:35
CarlosRuizso your patch seems ok at first sight - would need to test14:35
CarlosRuizdoes the new fields have same displaylogic as the routingno+accountno ?14:36
mbozem18Yes, i copied the display logic14:36
CarlosRuizgood14:36
mbozem18And i tested the swing client14:36
CarlosRuizthen - I'll test zkwebui and integrate if everything goes fine14:36
mbozem18thanks.... so what about the SWIFTCode fields in C_Payment, I_Payment, C_PaymentTransaction... Should i remove them or should they stay like RoutingNo?14:37
*** jdpaniagua has joined #idempiere14:38
CarlosRuizis ok - let's keep them14:39
mbozem18ok. i think it´s better so.14:39
CarlosRuizwould be just advisable to add the nmicoud validation to all those new IBAN fields - and the currently existing in C_Bank14:40
CarlosRuizbut can be done after14:40
mbozem18I can do this after... With that config key.14:40
mbozem18At the moment the code is without checking the configuration key i think14:41
mbozem18I´m so happy: My first patch :-)14:43
CarlosRuizwell - your first looks good :-)  thanks for contributing - testing14:44
CarlosRuizmbozem18: which is your developer code - in bitbucket - or your email?  to use it as the commit developer14:51
mbozem18mbozem@bozem.de14:51
mbozem18I think bitbucket is markus_bozem, but mbozem@bozem.de is better14:52
Not-4af[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-4.1 [+2/-0/±15] https://bitbucket.org/idempiere/idempiere/commits/14:56
Not-4af[iDempiere] mark...@bozem.de 8e89c5b - IDEMPIERE-3039 Add IBAN and Swift fields to Business Partner/Bank Account14:56
CarlosRuizok14:56
CarlosRuiztests went fine - I integrated it into 4.114:56
Not-4af[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3039 status set to "Resolved" -assignee set to "Markus Bozem" -resolution set to "Fixed"14:57
Not-4af[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-303914:57
mbozem18Thank you very much. The next days i will do IDEMPIERE-1200.14:58
mbozem18I put the migration scripts into i4.1z... did you see it?14:58
CarlosRuizyes - I moved them to 4.114:59
mbozem18ok14:59
mbozem18see you15:02
CarlosRuizbye15:05
Not-4af[iDempiere4.1] jenkins built #52 completed (success) http://ci.idempiere.org/job/iDempiere4.1/52/15:18
Not-4af[iDempiere] CarlosRuiz_globalqss pushed 1 commit to release-4.1 [+0/-0/±1] https://bitbucket.org/idempiere/idempiere/commits/16:10
Not-4af[iDempiere] globalqss 25d8011 - IDEMPIERE-3147 Financial Report translation of the data not only headers (based on patch from Gaurav Sontake)16:10
Not-4af[IDEMPIERE] carlosruiz_globalqss updated IDEMPIERE-3147 status set to "Resolved" -assignee set to "Carlos Antonio Ruiz Gomez" -resolution set to "Fixed"16:10
Not-4af[IDEMPIERE] http://idempiere.atlassian.net/browse/IDEMPIERE-314716:10
*** CarlosRuiz has quit IRC16:11
*** mbozem18 has quit IRC16:20
Not-4af[iDempiere4.1] jenkins built #53 completed (success) http://ci.idempiere.org/job/iDempiere4.1/53/16:42
*** nmicoud has quit IRC17:22
*** a42niem has joined #idempiere17:22
*** jdpaniagua has quit IRC18:26
*** jdpaniagua has joined #idempiere18:44
*** a42niem has quit IRC19:47
*** mbozem has joined #idempiere21:11
*** jdpaniagua has quit IRC23:24
*** jdpaniagua has joined #idempiere23:24

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