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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0019112
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Modules] Cuaderno 43majoralways2011-11-18 13:372011-12-01 10:01
ReportermaiteView Statuspublic 
Assigned Tomirurita 
PriorityurgentResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision01090a96c0db
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Regression date
Regression introduced by commit
Regression level
Review Assigned To
Regression introduced in release
Summary

0019112: Improve algorithm to match business partner when importing cuaderno 43 bank file

DescriptionImprove algorithm to match business partner when importing cuaderno 43 bank file. As bank statement files received are quite different, actual method is too generic
Steps To Reproduce0. install cuaderno 43 module and apply dataset
1. import Bank statement in Financial Account and realize that business partner does not match
Proposed SolutionImprove the algorithm for matching business partner.
Use both concepts fields (concept1 and concept2) of the Cuaderno 43 to do the matching.
Probably the string should be clean, removing for example multiple spaces, numbers and other special characters.
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to defect 0019149 closedmirurita Improve algorithm to match business partner when importing cuaderno 43 bank file 

-  Notes
(0043104)
mirurita (developer)
2011-11-23 13:12

 SOLUTION

Cuaderno 43
-----------
  + concept1 and concept2 are used to generate business partner name. (before only concept2 was used). Resultant string is as well filtered to avoid multiple spaces, numbers and special characters (any character not included in abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ)
  + description field in the bank statement line is populated with concept1 and concept2 (not filtered, as they appear in the file).

APRM
----
  + Match Statement popup: matched business partners appear in bold. When the bank statement line has a business partner the business partner will be presented in bold.
  + Matching algorithm for business partners has been improved:
    1) Find a match in previous bank statement for the same financial account.
    2) Try a full match by name
    3) New function to obtain closest match: first filter business partners which name contains one of the tokens and second filter this list with Levenshtein algorithm.
  + Some more constraints added to hql queries (organization validation and financial account validation).
(0043118)
hgbot (developer)
2011-11-23 16:39

 Repository: erp/pmods/org.openbravo.module.cuaderno43.es
Changeset: 4969ef4a81b245b7971ed95ead04e0adfe021568
Author: Mikel Irurita <mikel.irurita <at> openbravo.com>
Date: Wed Nov 23 16:38:33 2011 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.module.cuaderno43.es/rev/4969ef4a81b245b7971ed95ead04e0adfe021568 [^]

Related to issue 19112, issue 19149: Improve algorithm to match business partner when importing cuaderno43

---
M src/org/openbravo/module/cuaderno43/es/utility/Cuaderno43.java
---
(0043121)
hgbot (developer)
2011-11-23 17:33

 Repository: erp/devel/pi
Changeset: 01090a96c0dbefc157987318e750986313eebc4d
Author: Mikel Irurita <mikel.irurita <at> openbravo.com>
Date: Wed Nov 23 17:32:01 2011 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/01090a96c0dbefc157987318e750986313eebc4d [^]

Fixes issue 19112: Improve algorithm to match business partner when importing cuaderno43

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/MatchTransaction.java
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/MatchTransactionGrid.srpt
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/MatchTransactionGrid.xml
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/utility/FIN_BankStatementImport.java
---
(0043132)
hgbot (developer)
2011-11-23 18:52

 Repository: erp/pmods/org.openbravo.module.cuaderno43.es
Changeset: fe48cf7d8a24f667d157462c89597774bdf81257
Author: Mikel Irurita <mikel.irurita <at> openbravo.com>
Date: Wed Nov 23 18:52:34 2011 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.module.cuaderno43.es/rev/fe48cf7d8a24f667d157462c89597774bdf81257 [^]

Related to issue 19112: if any exception occur trying to clean return input string

---
M src/org/openbravo/module/cuaderno43/es/utility/Cuaderno43.java
---
(0043133)
mirurita (developer)
2011-11-23 19:10

 TESTING PLAN
Expected result in the following cases is to have as business partner name concept1 and concept2 fields concatenated, filtering out multiple spaces, numbers and special characters (any character not included in abcdefghijklmnnopqrstuvwxyzABCDEFGHIJKLMNÑOPQRSTUVWXYZ)

Import a c43 file:
- concept1 and concept2 field not empty.
- concept1 field empty and concept2 field not empty.
- concept1 field not empty and concept2 field empty.
- concept1 field not empty and concept2 field empty.
- concept1 or concept2 having numbers.
- concept1 or concept2 having any special character.
- concept1 or concept2 having multiple spaces.
(0043342)
hudsonbot (developer)
2011-11-25 16:20

 A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/98907f0c5af2 [^]

Maturity status: Test
(0043623)
eduardo_Argal (developer)
2011-12-01 10:01

 Tested and working. A fix was required and applied:
19192: Bank Statement Import does not find Business partner information from previous bank statement files.

Mikel please close it
;)

- Issue History
Date Modified Username Field Change
2011-11-18 13:37 maite New Issue
2011-11-18 13:37 maite Assigned To => eduardo_Argal
2011-11-18 13:37 maite Issue Monitored: networkb
2011-11-23 13:12 mirurita Note Added: 0043104
2011-11-23 13:12 mirurita Proposed Solution updated
2011-11-23 13:14 mirurita Issue cloned 0019149
2011-11-23 13:15 mirurita Relationship added related to 0019149
2011-11-23 16:39 hgbot Checkin
2011-11-23 16:39 hgbot Note Added: 0043118
2011-11-23 16:40 mirurita Assigned To eduardo_Argal => mirurita
2011-11-23 17:33 hgbot Checkin
2011-11-23 17:33 hgbot Note Added: 0043121
2011-11-23 17:33 hgbot Status new => resolved
2011-11-23 17:33 hgbot Resolution open => fixed
2011-11-23 17:33 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/01090a96c0dbefc157987318e750986313eebc4d [^]
2011-11-23 18:52 hgbot Checkin
2011-11-23 18:52 hgbot Note Added: 0043132
2011-11-23 19:10 mirurita Note Added: 0043133
2011-11-25 16:20 hudsonbot Checkin
2011-11-25 16:20 hudsonbot Note Added: 0043342
2011-12-01 10:01 eduardo_Argal Note Added: 0043623
2011-12-01 10:01 eduardo_Argal Status resolved => closed

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker