Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0029983
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Modules] SEPA Direct Debit Customer-to-Bankmajoralways2015-05-22 14:232015-05-27 18:43
ReporterngarciaView Statuspublic 
Assigned Tojorge-garcia 
PriorityurgentResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision61c90b1fd05d
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Merge Request Status
Regression date
Regression introduced by commit
Regression level
Review Assigned Tovmromanos
Support ticket36154
OBNetwork customerOBPS
Regression introduced in release
Summary

0029983: Bank Instruction description's last character is removed when adding it to the ustrd field

DescriptionBank Instruction description's last character is removed when adding it to the ustrd field
Steps To ReproduceAs group admin role:
   Create a Remittance with Remittance type "SEPA Core Direct Debit Customer-to-Bank (pain.008.001.02)" and process it
   Check a Bank Instructions record has a description and if not add it
   Create Remittance File and check the last character is removed from that description
   Repeat the process with "SEPA Core Direct Debit Customer-to-Bank (pain.008.001.03)" and check the same problem happens
Proposed SolutionThe problem is in the following code of the CoreDirectDebit.java file:

          remInfo7.getUstrd()
              .add(
                  parseText(line.getDescription().substring(
                      0,
                      line.getDescription().length() > 140 ? 139
                          : line.getDescription().length() - 1)));
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to defect 0029984 closedjorge-garcia Bank Instruction description's last character is removed when adding it to the ustrd field 

-  Notes
(0077826)
jorge-garcia (viewer)
2015-05-27 16:45

 Test Plan

As group admin role:
   Create a Remittance with Remittance type "SEPA Core Direct Debit Customer-to-Bank (pain.008.001.02)" and process it.
   Check a Bank Instructions record has a description and if not add it (try to insert a description with more than/less than/equal to 140 characters).
   Create Remittance File and check that the description is filled correctly
   Repeat the process with "SEPA Core Direct Debit Customer-to-Bank (pain.008.001.03)" and check the description is filled correctly.
(0077834)
hgbot (developer)
2015-05-27 18:42

 Repository: erp/pmods/org.openbravo.sepa.iso20022.coredirectdebit.customertobank
Changeset: 61c90b1fd05de1b5ec866a0ad35f5a72ae31aebe
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed May 27 16:19:49 2015 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.sepa.iso20022.coredirectdebit.customertobank/rev/61c90b1fd05de1b5ec866a0ad35f5a72ae31aebe [^]

Fixed issue 29983: Bank Instruction description's last character is removed

Bank Instruction description's last character is removed when adding it to the
USTRD field.

The problem was that the description only takes the first 139 characters instead
of the 140 allowed. Also, if you have blank spaces before and after the
description, these blank spaces counts for the count of the string and the
description is filled wrong. If the string length was smaller than 140
characters, the last character of the string is also deleted.

The solution is to check if the string has more than 140 characters. If so, we
retrieve the first 140 characters excluding blank spaces. If is lesser than 140
characters, the function don't do anything.

---
M src/org/openbravo/sepa/iso20022/coredirectdebit/customertobank/pain/x008/x001/x02/report/CoreDirectDebit.java
M src/org/openbravo/sepa/iso20022/coredirectdebit/customertobank/pain/x008/x001/x03/report/CoreDirectDebit.java
---
(0077839)
vmromanos (viewer)
2015-05-27 18:43

 Code review + testing OK

Available in SEPA Direct Debit Customer-to-Bank (CORE and B2B) 1.0.7 or higher

- Issue History
Date Modified Username Field Change
2015-05-22 14:23 ngarcia New Issue
2015-05-22 14:23 ngarcia Assigned To => Triage Finance
2015-05-22 14:23 ngarcia Support ticket => 36154
2015-05-22 14:23 ngarcia OBNetwork customer => Yes
2015-05-22 14:24 ngarcia Issue cloned 0029984
2015-05-22 14:24 ngarcia Category SEPA Credit Transfer Customer-to-Bank => SEPA Direct Debit Customer-to-Bank
2015-05-22 14:25 ngarcia Steps to Reproduce Updated View Revisions
2015-05-22 14:25 ngarcia Issue Monitored: networkb
2015-05-22 14:28 ngarcia Resolution time => 1435269600
2015-05-22 14:28 ngarcia Proposed Solution updated
2015-05-22 14:31 ngarcia Relationship added related to 0029984
2015-05-27 16:45 jorge-garcia Note Added: 0077826
2015-05-27 16:46 jorge-garcia Status new => scheduled
2015-05-27 16:46 jorge-garcia Assigned To Triage Finance => jorge-garcia
2015-05-27 18:42 hgbot Checkin
2015-05-27 18:42 hgbot Note Added: 0077834
2015-05-27 18:42 hgbot Status scheduled => resolved
2015-05-27 18:42 hgbot Resolution open => fixed
2015-05-27 18:42 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.sepa.iso20022.coredirectdebit.customertobank/rev/61c90b1fd05de1b5ec866a0ad35f5a72ae31aebe [^]
2015-05-27 18:43 vmromanos Review Assigned To => vmromanos
2015-05-27 18:43 vmromanos Note Added: 0077839
2015-05-27 18:43 vmromanos Status resolved => closed

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker