Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0030820Openbravo ERP09. Financial managementpublic2015-09-14 17:442015-09-16 02:20
Reportermalsasua 
Assigned ToAtulOpenbravo 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS20OS VersionCommunity Appliance
Product Version3.0PR15Q3.1 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Tovmromanos
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - QA Approved
Regression date2015-05-14
Regression introduced in release3.0PR15Q3
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/a810d05065cfe7eb3199928ed9104c7b54989ecb [^]
Triggers an Emergency PackNo
Summary0030820: accounting process is not working with payment out and negative amount
Descriptionwhen you run the accounting process to negative payment amount, error is returned:
Error
Process failed during execution
Steps To Reproduce. in F&B
 . go to financial account "Cuenta de Banco"
  -> tab: "payment method" -> transferencia
   . upon payment use: in transit payment account

 . go to purchase invoice:
  . new header:
   bp: Generación Electrica
  . new lines:
   product: Sitemas de información
   qty: -20
   amt: 5
 . process invoice

 . add payment-> process and deposit
 . go to payment out -> the payment created above
  . post it
error is returned
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
depends on backport 00308213.0PR15Q3.2 closed AtulOpenbravo accounting process is not working with payment out and negative amount 
caused by defect 0029618 closed aferraz G/L Journal Accounting amounts are not setted correctly 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-09-14 17:44malsasuaNew Issue
2015-09-14 17:44malsasuaAssigned To => Triage Finance
2015-09-14 17:44malsasuaModules => Core
2015-09-14 17:44malsasuaRegression level => Production - QA Approved
2015-09-14 17:44malsasuaTriggers an Emergency Pack => No
2015-09-14 18:18vmromanosRegression date => 2015-05-14
2015-09-14 18:18vmromanosRegression introduced in release => 3.0PR15Q3
2015-09-14 18:18vmromanosRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/a810d05065cfe7eb3199928ed9104c7b54989ecb [^]
2015-09-14 18:18vmromanosNote Added: 0080389
2015-09-14 18:18vmromanosStatusnew => scheduled
2015-09-14 18:18vmromanosAssigned ToTriage Finance => AtulOpenbravo
2015-09-14 18:19vmromanosRelationship addedcaused by 0029618
2015-09-15 11:47AtulOpenbravoNote Added: 0080399
2015-09-15 11:48AtulOpenbravoNote Edited: 0080399bug_revision_view_page.php?bugnote_id=0080399#r9434
2015-09-15 16:50vmromanosNote Added: 0080412
2015-09-15 16:50hgbotCheckin
2015-09-15 16:50hgbotNote Added: 0080413
2015-09-15 16:50hgbotStatusscheduled => resolved
2015-09-15 16:50hgbotResolutionopen => fixed
2015-09-15 16:50hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/db293dedb2ec916914f093105d9d8abb2a61192f [^]
2015-09-15 16:51vmromanosReview Assigned To => vmromanos
2015-09-15 16:51vmromanosNote Added: 0080416
2015-09-15 16:51vmromanosStatusresolved => closed
2015-09-16 02:20hudsonbotCheckin
2015-09-16 02:20hudsonbotNote Added: 0080426

Notes
(0080389)
vmromanos   
2015-09-14 18:18   
Detected NPE when trying to access attributes from docLine object https://code.openbravo.com/erp/devel/pi/rev/a810d05065cfe7eb3199928ed9104c7b54989ecb#l1.24 [^]
(0080399)
AtulOpenbravo   
2015-09-15 11:47   
(edited on: 2015-09-15 11:48)
Test Plan
- Login as F&B International Group Admin
- Go to Financial Account "Cuenta de Banco".
- Go to Payment Method "Transferencia"
- Set upon payment use as in transit payment account.

- Go to purchase invoice:
- Create new record for business partner "Generación Electrica".
- In lines set following as:
   product: Sitemas de información
   qty: -20
   amt: 5
- Process invoice.
- Add payment and process it.
- Go to payment out and select above created payment record and post it.
- Check that posting is successful.
(0080412)
vmromanos   
2015-09-15 16:50   
Verified also scenario for 0029618
(0080413)
hgbot   
2015-09-15 16:50   
Repository: erp/devel/pi
Changeset: db293dedb2ec916914f093105d9d8abb2a61192f
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Tue Sep 15 15:07:15 2015 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/db293dedb2ec916914f093105d9d8abb2a61192f [^]

Fixes Issue 30820:Accounting process fails for payment out with negative amt

If the docbasetype is not GLJournal is null then docLine Object is null. A
docbasetype check is added before accessing docLine Object.

---
M src/org/openbravo/erpCommon/ad_forms/Fact.java
---
(0080416)
vmromanos   
2015-09-15 16:51   
Code review + testing OK
(0080426)
hudsonbot   
2015-09-16 02: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/ab68e2868765 [^]
Maturity status: Test