Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0027944Openbravo ERP07. Sales managementpublic2014-10-22 10:412014-12-30 23:24
ngarcia 
reinaldoguerra 
immediatemajoralways
closedfixed 
5
 
3.0PR15Q1 
pramakrishnan
Core
No
0027944: Not Stocked BOM lines are printed without price in orders
Not Stocked BOM lines are printed without price in orders
As group admin role:
   Create a new product set as Bill Of Materials = 'Y' and Stocked = 'N'
   Add a sales price of 0 to it
   Add a product in the Bill Of Materials tab and Verify BOM
   Create a Sales Order with the BOM product and book it
   Print it and check the total doesn't match the total gross amount of the order as the price and total of the exploded BOM lines is not included
No tags attached.
related to defect 0034498 closed alekosmp86 Duplicity in lines printing a sales order 
diff 27944.diff (3,212) 2014-10-28 17:59
https://issues.openbravo.com/file_download.php?file_id=7441&type=bug
Issue History
2014-10-22 10:41ngarciaNew Issue
2014-10-22 10:41ngarciaAssigned To => AtulOpenbravo
2014-10-22 10:41ngarciaModules => Core
2014-10-22 10:41ngarciaResolution time => 1416697200
2014-10-22 10:41ngarciaTriggers an Emergency Pack => No
2014-10-22 10:45ngarciaTarget Version => 3.0PR15Q1
2014-10-22 10:49ngarciaIssue Monitored: networkb
2014-10-28 16:55maiteResolution time1416697200 => 1414101600
2014-10-28 16:55maitePriorityurgent => immediate
2014-10-28 16:56jonalegriaesarteAssigned ToAtulOpenbravo => jonalegriaesarte
2014-10-28 17:59jonalegriaesarteFile Added: 27944.diff
2014-10-28 18:00jonalegriaesarteNote Added: 0071234
2014-10-28 23:52reinaldoguerraAssigned Tojonalegriaesarte => reinaldoguerra
2014-10-28 23:52reinaldoguerraStatusnew => scheduled
2014-10-29 05:34reinaldoguerraNote Added: 0071236
2014-11-11 09:51hgbotCheckin
2014-11-11 09:51hgbotNote Added: 0071542
2014-11-11 09:51hgbotStatusscheduled => resolved
2014-11-11 09:51hgbotResolutionopen => fixed
2014-11-11 09:51hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/3c4f28793d604240eeb3ef20feded24e7a812089 [^]
2014-11-11 09:53pramakrishnanReview Assigned To => pramakrishnan
2014-11-11 09:53pramakrishnanNote Added: 0071545
2014-11-11 09:53pramakrishnanStatusresolved => closed
2014-12-30 23:24hudsonbotCheckin
2014-12-30 23:24hudsonbotNote Added: 0073037
2016-11-25 10:47aferrazRelationship addedrelated to 0034498

Notes
(0071234)
jonalegriaesarte   
2014-10-28 18:00   
Attached possible patch
(0071236)
reinaldoguerra   
2014-10-29 05:34   
Test plan used:
1- Login into Openbravo ERP
2- Go to product window and create a new one.
3- Set Bill Of Materials as 'Y' and Stocked as 'N'
4- Go to price tab and select a sales price list version, set unit and list price as zero.
5- Go to Bill of Materials tab and add a new product for example: Cerveza Lager 0,5L.
6- Press Verify BOM button.
7- Go to sales order window and create a new one.
8- Add a new line selecting created product and book it.
9- Verify that lines are populated with defined BOM product.
10- Print the order and check the total in the report now match the total gross amount of the order. Also the price and total of the exploded BOM lines are included in the report.
(0071542)
hgbot   
2014-11-11 09:51   
Repository: erp/devel/pi
Changeset: 3c4f28793d604240eeb3ef20feded24e7a812089
Author: Reinaldo Guerra <reinaldo.guerra <at> peoplewalking.com>
Date: Tue Oct 28 22:55:46 2014 -0500
URL: http://code.openbravo.com/erp/devel/pi/rev/3c4f28793d604240eeb3ef20feded24e7a812089 [^]

Fixed bug 27944: Not Stocked BOM lines are printed without price in orders.

Print sales order report was modified to take into account price and total amount for order lines reffering products with some bill of material defined. Now it is possible to delete invoice header.
This change affects also the purchase flow so now when printing an order, price and total of the exploded BOM lines are included, and the total match the total gross amount of the order.

---
M src/org/openbravo/erpReports/C_OrderLinesJR_new.jrxml
M src/org/openbravo/erpReports/C_OrderLinesTaxIncludedJR_new.jrxml
---
(0071545)
pramakrishnan   
2014-11-11 09:53   
Code Review + Testing - OK.
(0073037)
hudsonbot   
2014-12-30 23:24   
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/6525fe229e06 [^]
Maturity status: Test