Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0029685
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 08. Project and service managementminoralways2015-04-23 18:272016-12-16 18:37
ReporterngarciaView Statuspublic 
Assigned ToAtulOpenbravo 
PriorityhighResolutionfixedFixed in Version3.0PR17Q1
StatusclosedFix in branchFixed in SCM revision227ef1f50901
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Toaferraz
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0029685: Create AP Expense Invoices process creates wrong line numbers

DescriptionCreate AP Expense Invoices process creates wrong line numbers: 10, 20, 40 80...
Steps To ReproduceAs group admin role:
   Set an Employee also as Vendor and configure defaults values for Payment Terms, Financial Account, Price List...
   Create 4 products with Product Type = Expense Type
   Add price to them in the price list defined for the Business Partner
   Create a Expense Sheet for that Business Partner
   Add the four products in lines
   Process it
   Launch the Create AP Expense Invoices process
   Check the created Purchase Invoice's line numbers are 10, 20, 40 and 80 instead of 10, 20, 30 and 40
Proposed SolutionAttached a possible fix
TagsNo tags attached.
Attached Filesdiff file icon 29685.diff [^] (736 bytes) 2015-04-23 18:38 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0079138)
AtulOpenbravo (developer)
2015-07-31 10:14

Test Plan
- Login as group admin role.
- Set an Employee also as Vendor and configure defaults values for Payment Terms, Financial Account, Price List, etc
- Create 4 products with Product Type = Expense Type.
- Add price to them in the price list defined for the Business Partner.
- Create a Expense Sheet for that Business Partner.
   Add the four products in lines.
   Process it.
-Launch the Create AP Expense Invoices process.
- Check the created Purchase Invoice's line numbers are 10, 20, 30 and 40.
(0084723)
psanjuan (manager)
2016-03-04 11:54

Repro - steps required to reproduce this issue:

- Set BP "Javier Martín" (Employee) as Vendor by setting up a Purchase Price List, a payment method, payment terms and a financial account.
- Create 4 products named "Expense 1", "Expense 2", "Expense 3" and "Expense 4". All of them set up as Product Type = Expense type. In lines tab select a purchase price list (i.e. Tarifa Bebidas Alegres) and enter a unit/list price for all of them.
- Go to Expense sheet and create a new one for Javier Martin, dated on 04-03-2016.
- Go to lines tab and enter below info:
line 1 = Product "Expense 1" Qty = 1
line 2 = Product "Expense 2" Qty = 2
line 3 = Product "Expense 3" Qty = 3
line 4 = Product "Expense 4" Qty = 4
- Process Expenses
- Launch the "Create AP Expense Invoices" process by choosing Javier Martin employee. From Date 01-03-2016 To date 05-03-2016 Dated Invoiced 04-03-2016
Press Process.
- Navigate to the purchase invoice created and check that purchase order line numbers are not correct, as it is shown as: 10,20,40,80
(0084724)
psanjuan (manager)
2016-03-04 11:54

Issue Verified
(0084785)
psanjuan (manager)
2016-03-07 17:06

Test Plan above verified.

Issue can be code reviewed and closed.
(0090634)
hgbot (developer)
2016-10-17 12:58

Repository: erp/devel/pi
Changeset: 227ef1f5090155944f9c89a3f5158e43f8c030b1
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Fri Jul 31 13:38:35 2015 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/227ef1f5090155944f9c89a3f5158e43f8c030b1 [^]

Fixes issue 29685: Create AP Expense Invoices process creates wrong line number

Getting line no is used similar that is used to set default value for line
column in Tables and Columns. For eg:- select coalesce(max(line), 0) + 10
Unnecessary variable int line is no more needed so removed.
While executing create AP Expense invoice process there was null pointer
exception raised from AccDefUtility class which is fixed.

---
M src/org/openbravo/erpCommon/ad_process/ExpenseAPInvoice.java
M src/org/openbravo/erpCommon/ad_process/ExpenseAPInvoice_data.xsql
M src/org/openbravo/erpCommon/utility/AccDefUtility.java
---
(0090636)
aferraz (manager)
2016-10-17 12:59

Code review OK
(0092522)
hudsonbot (developer)
2016-12-16 18:37

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/dc8bf00badd0 [^]
Maturity status: Test

- Issue History
Date Modified Username Field Change
2015-04-23 18:27 ngarcia New Issue
2015-04-23 18:27 ngarcia Assigned To => Triage Finance
2015-04-23 18:27 ngarcia Modules => Core
2015-04-23 18:27 ngarcia Triggers an Emergency Pack => No
2015-04-23 18:38 ngarcia File Added: 29685.diff
2015-04-23 18:38 ngarcia Proposed Solution updated
2015-04-23 18:46 ngarcia Issue Monitored: networkb
2015-07-31 10:12 AtulOpenbravo Assigned To Triage Finance => AtulOpenbravo
2015-07-31 10:13 AtulOpenbravo Status new => scheduled
2015-07-31 10:14 AtulOpenbravo Note Added: 0079138
2016-03-04 11:54 psanjuan Note Added: 0084723
2016-03-04 11:54 psanjuan Note Added: 0084724
2016-03-07 17:06 psanjuan Note Added: 0084785
2016-10-17 12:58 hgbot Checkin
2016-10-17 12:58 hgbot Note Added: 0090634
2016-10-17 12:58 hgbot Status scheduled => resolved
2016-10-17 12:58 hgbot Resolution open => fixed
2016-10-17 12:58 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/227ef1f5090155944f9c89a3f5158e43f8c030b1 [^]
2016-10-17 12:59 aferraz Review Assigned To => aferraz
2016-10-17 12:59 aferraz Note Added: 0090636
2016-10-17 12:59 aferraz Status resolved => closed
2016-10-17 12:59 aferraz Fixed in Version => 3.0PR17Q1
2016-12-16 18:37 hudsonbot Checkin
2016-12-16 18:37 hudsonbot Note Added: 0092522


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker