Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0031385Openbravo ERPZ. Otherspublic2015-11-06 09:572015-11-23 21:17
Reportermaite 
Assigned Toaferraz 
PriorityimmediateSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target Version3.0PR16Q1Fixed in Version 
Merge Request Status
Review Assigned Tovmromanos
OBNetwork customer
Web browser
ModulesAdvanced Payables and Receivables Mngmt
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2014-09-23
Regression introduced in release3.0PR14Q4
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/179d97ee602d [^]
Triggers an Emergency PackNo
Summary0031385: Business Partner name defined in bank statement line is not loaded in Description field when Add Transaction from Match Statemen
DescriptionBefore Match Statement refactor, when running Add TRansaction process from Match Statement button, both Business Partner Name and Description values where loaded in "Descrition" field when selecting "GL Item" type

Business Partner Name information is necessary in order to perform a proper matching
Steps To Reproduce1. Access Cuenta de Banco financial account, create new Bank Statement record
2. Add Bank Statement line seeting Business Partner Name as "name of my bp" and Description as "bsl description". Process bank statement
3. Run "Match Statement" process and press Add Transaction button (+) from your line. Realize that Description field only contains "bsl description"

If you follow similar process in version prior to refctor (for example MP30.3) you can verify that description field when trying to match against GL Item type is

"name of my bp
 bsl description"
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
depends on backport 00314203.0PR15Q4 closed aferraz Business Partner name defined in bank statement line is not loaded in Description field when Add Transaction from Match Statemen 
depends on backport 00314213.0PR15Q3.4 closed aferraz Business Partner name defined in bank statement line is not loaded in Description field when Add Transaction from Match Statemen 
caused by feature request 00293903.0PR14Q4 closed Sandrahuguet Reconciliation refactor project 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-11-06 09:57maiteNew Issue
2015-11-06 09:57maiteAssigned To => Triage Finance
2015-11-06 09:57maiteModules => Advanced Payables and Receivables Mngmt
2015-11-06 09:57maiteResolution time => 1449702000
2015-11-06 09:57maiteTriggers an Emergency Pack => No
2015-11-06 09:58maiteIssue Monitored: networkb
2015-11-06 09:58maiteSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=10088#r10088
2015-11-10 13:47aferrazAssigned ToTriage Finance => aferraz
2015-11-10 13:54aferrazRelationship addedcaused by 0029390
2015-11-10 13:58aferrazRegression level => Production - Confirmed Stable
2015-11-10 13:58aferrazRegression date => 2014-09-23
2015-11-10 13:58aferrazRegression introduced in release => 3.0PR14Q4
2015-11-10 13:58aferrazRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/179d97ee602d [^]
2015-11-10 19:07aferrazStatusnew => scheduled
2015-11-10 19:13aferrazNote Added: 0081673
2015-11-12 19:20hgbotCheckin
2015-11-12 19:20hgbotNote Added: 0081780
2015-11-12 19:20hgbotStatusscheduled => resolved
2015-11-12 19:20hgbotResolutionopen => fixed
2015-11-12 19:20hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/c5898a27e5dea48311c292253a46d84223f4145e [^]
2015-11-12 19:25vmromanosReview Assigned To => vmromanos
2015-11-12 19:25vmromanosNote Added: 0081790
2015-11-12 19:25vmromanosStatusresolved => closed
2015-11-20 15:12aferrazNote Added: 0082029
2015-11-23 21:17hudsonbotCheckin
2015-11-23 21:17hudsonbotNote Added: 0082216
2016-02-26 13:59ngarciaRelationship addedrelated to 0032364
2016-02-26 14:00ngarciaRelationship deletedrelated to 0032364

Notes
(0081673)
aferraz   
2015-11-10 19:13   
Test plan:

As F&B Admin:

- Access Cuenta de Banco financial account, create new Bank Statement record
  Add Bank Statement line seting Business Partner Name as "name of my bp" and Description as "bsl description".
  Add Bank Statement line seting Business Partner Name as "name of my bp" and Description empty.
  Add Bank Statement line seting Business Partner Name empty and Description as "bsl description".
  Add Bank Statement line seting Business Partner Name empty and Description empty.
  Process bank statement

- Run "Match Statement" process and press Add Transaction button (+) from first line. Realize that Description field contains:
"name of my bp
 bsl description"

- Run "Match Statement" process and press Add Transaction button (+) from second line. Realize that Description field contains:
"name of my bp"

- Run "Match Statement" process and press Add Transaction button (+) from third line. Realize that Description field contains:
"bsl description"

- Run "Match Statement" process and press Add Transaction button (+) from fourth line. Realize that Description field is empty.
(0081780)
hgbot   
2015-11-12 19:20   
Repository: erp/devel/pi
Changeset: c5898a27e5dea48311c292253a46d84223f4145e
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Tue Nov 10 19:05:45 2015 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/c5898a27e5dea48311c292253a46d84223f4145e [^]

Fixes issue 31385: BusinessPartnerName not loaded in AddTransaction description

Add Transaction description will be loaded with Business Partner name + Bank Statement Line description, in case Business Partner name is not empty.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/filterexpression/AddTransactionFilterExpression.java
---
(0081790)
vmromanos   
2015-11-12 19:25   
Code review + testing OK
(0082029)
aferraz   
2015-11-20 15:12   
https://testlink.openbravo.com/testlink/linkto.php?tprojectPrefix=Communit&item=testcase&id=Communit-8627 [^]
(0082216)
hudsonbot   
2015-11-23 21:17   
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/7b56bebaaa88 [^]
Maturity status: Test