Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033368Openbravo ERP09. Financial managementpublic2016-06-28 11:262016-08-11 18:40
Reporteramayavicente 
Assigned Toumartirena 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR16Q4 
Merge Request Status
Review Assigned Toaferraz
OBNetwork customerOBPS
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0033368: [SER QA 1367] Wrong Line no in Financial Account || Transactions
DescriptionWrong Line no in Financial Account || Transactions when a payment in deposit in a fiancial account different to the selected financial account in the payment header.
Steps To Reproduce1. For the payment method 'Wire Transfer' set Automatic Deposit = No in the financial accounts 'Bank-Account 1' and 'Bank-Account 2'.
2. Create a payment in for the organization 'F & B US, Inc.', Payment Method = 'Wire Transfer' and Deposit to = 'Bank-Account 1'.
3. Add details, for instanca a gl item, and process the payment with the action 'Process Received Payment(s)'.
4. Deposit the payment created previously in the financial account 'Bank-Account 2' using the button 'Add Multiple Payments'.

ERROR: check that in the tab 'Transaction' the payment has been deposit with the wrong 'Line No' = 6630 instead with the number 20. This is due to the fact that:
-Line No = Max (Line No of the FA linked to the payment) + 10
and it should be:
-Line No = Max (Line No of the FA where the payment is deposit) + 10
Proposed SolutionSee attached diff made by Eduardo Argal.
Additional Information
TagsSER-QA
Relationships
Attached Filesdiff AddMultiplePayments.diff (815) 2016-06-28 11:26
https://issues.openbravo.com/file_download.php?file_id=9591&type=bug
Issue History
Date ModifiedUsernameFieldChange
2016-06-28 11:26amayavicenteNew Issue
2016-06-28 11:26amayavicenteAssigned To => Triage Finance
2016-06-28 11:26amayavicenteFile Added: AddMultiplePayments.diff
2016-06-28 11:26amayavicenteOBNetwork customer => Yes
2016-06-28 11:26amayavicenteModules => Core
2016-06-28 11:26amayavicenteResolution time => 1468274400
2016-06-28 11:26amayavicenteTriggers an Emergency Pack => No
2016-06-28 11:26amayavicenteTag Attached: SER-QA
2016-07-01 09:56amayavicenteSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=12567#r12567
2016-07-01 10:07hgbotCheckin
2016-07-01 10:07hgbotNote Added: 0088136
2016-07-01 10:07hgbotStatusnew => resolved
2016-07-01 10:07hgbotResolutionopen => fixed
2016-07-01 10:07hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/4044ca6f9d568076ea64d4b47375881f2be5c2b3 [^]
2016-07-01 10:07hgbotCheckin
2016-07-01 10:07hgbotNote Added: 0088137
2016-07-01 10:08aferrazAssigned ToTriage Finance => umartirena
2016-07-01 10:08aferrazReview Assigned To => aferraz
2016-07-01 10:08aferrazNote Added: 0088138
2016-07-01 10:08aferrazStatusresolved => closed
2016-07-01 10:08aferrazFixed in Version => 3.0PR16Q4
2016-08-11 18:40hudsonbotCheckin
2016-08-11 18:40hudsonbotNote Added: 0088979
2016-08-11 18:40hudsonbotCheckin
2016-08-11 18:40hudsonbotNote Added: 0088980

Notes
(0088136)
hgbot   
2016-07-01 10:07   
Repository: erp/devel/pi
Changeset: 4044ca6f9d568076ea64d4b47375881f2be5c2b3
Author: Unai Martirena <unai.martirena <at> openbravo.com>
Date: Fri Jun 17 12:53:30 2016 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/4044ca6f9d568076ea64d4b47375881f2be5c2b3 [^]

Fixes issue 33368: Set line no to transaction in AddMultiplePaymentsHandler

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/AddMultiplePaymentsHandler.java
---
(0088137)
hgbot   
2016-07-01 10:07   
Repository: erp/devel/pi
Changeset: 59f5c92e9921d28424c96dd9e4859797443d8944
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Fri Jul 01 10:00:46 2016 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/59f5c92e9921d28424c96dd9e4859797443d8944 [^]

Related to issue 33368: Update copyright

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/actionHandler/AddMultiplePaymentsHandler.java
---
(0088138)
aferraz   
2016-07-01 10:08   
Code review + Testing OK
(0088979)
hudsonbot   
2016-08-11 18:40   
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/edaa05b1fb18 [^]
Maturity status: Test
(0088980)
hudsonbot   
2016-08-11 18:40   
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/edaa05b1fb18 [^]
Maturity status: Test