Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0029685 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] 08. Project and service management | minor | always | 2015-04-23 18:27 | 2016-12-16 18:37 | |||
Reporter | ngarcia | View Status | public | |||||
Assigned To | AtulOpenbravo | |||||||
Priority | high | Resolution | fixed | Fixed in Version | 3.0PR17Q1 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 227ef1f50901 | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | aferraz | |||||||
OBNetwork customer | OBPS | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | 35881 | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0029685: Create AP Expense Invoices process creates wrong line numbers | |||||||
Description | Create AP Expense Invoices process creates wrong line numbers: 10, 20, 40 80... | |||||||
Steps To Reproduce | As 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 Solution | Attached a possible fix | |||||||
Tags | No tags attached. | |||||||
Attached Files | ![]() | |||||||
![]() |
|
![]() |
|
(0079138) AtulOpenbravo (viewer) 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 (viewer) 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 (viewer) 2016-03-04 11:54 |
Issue Verified |
(0084785) psanjuan (viewer) 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 (viewer) 2016-10-17 12:59 |
Code review OK |
(0092522) hudsonbot (viewer) 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 |
![]() |
|||
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 | OBNetwork customer | => Yes |
2015-04-23 18:27 | ngarcia | Modules | => Core |
2015-04-23 18:27 | ngarcia | Support ticket | => 35881 |
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 |