Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0031517Openbravo ERP09. Financial managementpublic2015-11-17 13:322015-11-18 14:03
Reporteraferraz 
Assigned Toaferraz 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS30OS VersionProfessional Appliance
Product Version 
Target Version3.0PR15Q3.4Fixed in Version 
Merge Request Status
Review Assigned Tovmromanos
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelPackaging and release
Regression date2015-11-11
Regression introduced in releasepi
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/fe7e7c17712b [^]
Triggers an Emergency PackNo
Summary0031517: Empty matched document when running Match Statement pocess
DescriptionMatched Document keeps empty in Match Statement grid although record is matched.
Steps To ReproduceAs FB Admin:

- Go to Financial Account windonw and select Cuenta de Banco
  Move to Imported Bank Statements tab and create a record.
  Move to Bank Statement Lines tab and create a record with payment in = 10.
  Click on Process button.

- Click on Match Statement button.
  Click Cancel in Confirm popup.
  Select the record shown in the grid and click + button (Add new transaction).
  Select any gl item in Add Transaction popup and click Done.
  Clear filter is necessary to show matched bank statements.
  Realize record appears as matched but Matched Document column is empty.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks defect 00315153.0PR16Q1 closed aferraz Empty matched document when running Match Statement pocess 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-11-17 15:29aferrazTypedefect => backport
2015-11-17 15:29aferrazTarget Version3.0PR16Q1 => 3.0PR15Q3.4
2015-11-18 14:01hgbotCheckin
2015-11-18 14:01hgbotNote Added: 0081950
2015-11-18 14:01hgbotStatusscheduled => resolved
2015-11-18 14:01hgbotResolutionopen => fixed
2015-11-18 14:01hgbotFixed in SCM revision => http://code.openbravo.com/erp/backports/3.0PR15Q3.4/rev/d816f0fd4dd9296a4c2f280b0c652df5e67fa28c [^]
2015-11-18 14:03vmromanosReview Assigned To => vmromanos
2015-11-18 14:03vmromanosNote Added: 0081952
2015-11-18 14:03vmromanosStatusresolved => closed

Notes
(0081950)
hgbot   
2015-11-18 14:01   
Repository: erp/backports/3.0PR15Q3.4
Changeset: d816f0fd4dd9296a4c2f280b0c652df5e67fa28c
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Tue Nov 17 18:13:59 2015 +0100
URL: http://code.openbravo.com/erp/backports/3.0PR15Q3.4/rev/d816f0fd4dd9296a4c2f280b0c652df5e67fa28c [^]

Fixes issue 31517: Empty matched document in Match Statement

The problem was that two new columns were added in aprm_matchstatement hql query in the middle of the select clause, while they were added as two new columns at the end of ad_column. As the columns order was not the same between the hql query and the ad_column and some columns did not have an alias in hql query and some others did not have an entity alias in ad_column, some column values were lost.
To fix it, we added an alias to all columns in aprm_matchstatement hql query (matchedDocument had no alias) and an entity alias to all ad_columns in aprm_matchstatement table (Transaction, bslUpdated and matchedDocument columns had no entity alias and Affinity column had a wrong entity alias).
With this, all columns of aprm_matchstatement will be correctly filled with their values in Match Statement grid.
If new columns are added to the hql query, they can go in any order in the select clause and in the ad_column, but they will need to have both an alias in the hql query and an entity alias in the ad_column, following this documentation: http://wiki.openbravo.com/wiki/How_to_create_a_HQL_Based_Table#Column_Definition [^]

---
M modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_COLUMN.xml
M modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_TABLE.xml
---
(0081952)
vmromanos   
2015-11-18 14:03   
Code review + testing OK