Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0028663Openbravo ERP09. Financial managementpublic2015-01-18 21:172015-05-07 22:16
Reporterrjapoova 
Assigned Toaferraz 
PriorityimmediateSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version3.0PR14Q4 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Tovmromanos
OBNetwork customer
Web browser
ModulesCore
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
Summary0028663: G/L Item default is lost in Match Statement -> Add New Transaction when transaction type is set to Bank Fee
DescriptionIn the Match Statement flow, when you need to add a transaction to match a record coming from an imported statement, the G/L Item - if present on the statement - is defaulted on the transaction.

However, if you accidentally change the Transaction Type to Bank Fee and then back to Withdrawal or Deposit, the G/L Item attribution is lost.
Steps To ReproduceSee this video from 3':45" to 4':30":
https://drive.google.com/file/d/0B2uCUQlrOPvIUXZ5cDdmd3IydDg/view?usp=sharing [^]
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
depends on backport 00296593.0PR15Q2 closed aferraz G/L Item default is lost in Match Statement -> Add New Transaction when transaction type is set to Bank Fee 
depends on backport 00296603.0PR15Q1.4 closed aferraz G/L Item default is lost in Match Statement -> Add New Transaction when transaction type is set to Bank Fee 
caused by feature request 00293903.0PR14Q4 closed Sandrahuguet Reconciliation refactor project 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-01-18 21:17rjapoovaNew Issue
2015-01-18 21:17rjapoovaAssigned To => dmiguelez
2015-01-18 21:17rjapoovaModules => Core
2015-01-18 21:17rjapoovaTriggers an Emergency Pack => No
2015-04-17 20:35dmitry_mezentsevAssigned Todmiguelez => Triage Finance
2015-04-17 20:35dmitry_mezentsevStatusnew => acknowledged
2015-04-17 20:35dmitry_mezentsevRelationship addedcaused by 0029390
2015-04-17 20:36dmitry_mezentsevPrioritynormal => immediate
2015-04-17 20:41dmitry_mezentsevRegression level => Production - QA Approved
2015-04-21 18:01umartirenaRegression date => 2014-09-23
2015-04-21 18:01umartirenaRegression introduced in release => 3.0PR14Q4
2015-04-21 18:01umartirenaRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/179d97ee602d [^]
2015-04-21 18:01umartirenaStatusacknowledged => scheduled
2015-04-21 18:01umartirenaAssigned ToTriage Finance => aferraz
2015-04-21 18:27aferrazRegression levelProduction - QA Approved => Production - Confirmed Stable
2015-04-23 12:27hgbotCheckin
2015-04-23 12:27hgbotNote Added: 0076711
2015-04-23 12:27hgbotStatusscheduled => resolved
2015-04-23 12:27hgbotResolutionopen => fixed
2015-04-23 12:27hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/de5d8592f23c308674f5dd34eebaf8a267d3a8e9 [^]
2015-04-23 12:31vmromanosReview Assigned To => vmromanos
2015-04-23 12:31vmromanosNote Added: 0076715
2015-04-23 12:31vmromanosStatusresolved => closed
2015-05-07 22:16hudsonbotCheckin
2015-05-07 22:16hudsonbotNote Added: 0077147

Notes
(0076711)
hgbot   
2015-04-23 12:27   
Repository: erp/devel/pi
Changeset: de5d8592f23c308674f5dd34eebaf8a267d3a8e9
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Wed Apr 22 14:10:57 2015 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/de5d8592f23c308674f5dd34eebaf8a267d3a8e9 [^]

Fixes issue 28663: GLItem lost in Add new Transaction window

Neither payment nor glitem will be deleted when changing to Bank fee in Add new Transaction window. Instead, when clicking on Done and creating the transaction, AddTransactionActionHandler will check if the transaction is of type Bank fee and in this case it will set as null both fields.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/AddTransactionActionHandler.java
M modules/org.openbravo.advpaymentmngt/web/org.openbravo.advpaymentmngt/js/ob-aprm-addTransaction.js
---
(0076715)
vmromanos   
2015-04-23 12:31   
Code review + testing OK
(0077147)
hudsonbot   
2015-05-07 22:16   
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/a1817f84bd8b [^]
Maturity status: Test