Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0027562Openbravo ERP09. Financial managementpublic2014-09-09 10:552014-12-30 23:28
Reporterngarcia 
Assigned Toreinaldoguerra 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target Version3.0PR15Q1Fixed in Version3.0PR15Q1 
Merge Request Status
Review Assigned ToSandrahuguet
OBNetwork customerOBPS
Web browser
ModulesCore
Support ticket30929
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0027562: Taxable amount of 0 for not stocked bom lines and a tax rate with Alternate Tax Base Amount
DescriptionTaxable amount of 0 for not stocked bom lines and a tax rate with Alternate Tax Base Amount
Steps To ReproduceAs group admin role:
   Create a new product, set it as Stocked = 'N' and Bill of materials = 'Y'
   Add at least one product (with a price different from 0) in the Bill Of Materials tab and Verify BOM
   Change the base amount of 'Entregas IVA 21%' tax rate to Alternate Tax Base Amount
   Create a Sales Order and add the previously created product to it. Check that both when you explode the lines through the process button and when you book the sales order without exploding them, the taxable and tax amount in the Line Tax tab is 0 for exploded lines
   The same happens in Sales Invoice window
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
causes defect 0028560 closed Sandrahuguet Cannot modify the alternate taxable amount of invoice and order lines 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2014-09-09 10:55ngarciaNew Issue
2014-09-09 10:55ngarciaAssigned To => jonalegriaesarte
2014-09-09 10:55ngarciaOBNetwork customer => Yes
2014-09-09 10:55ngarciaModules => Core
2014-09-09 10:55ngarciaSupport ticket => 30929
2014-09-09 10:55ngarciaResolution time => 1413151200
2014-09-09 10:55ngarciaTriggers an Emergency Pack => No
2014-09-09 10:56ngarciaIssue Monitored: networkb
2014-09-10 17:09heccamIssue Monitored: heccam
2014-09-22 16:36jonalegriaesarteTarget Version => 3.0PR15Q1
2014-10-06 16:32jonalegriaesarteAssigned Tojonalegriaesarte => AtulOpenbravo
2014-11-11 16:50reinaldoguerraAssigned ToAtulOpenbravo => reinaldoguerra
2014-11-11 16:50reinaldoguerraStatusnew => scheduled
2014-11-13 01:08reinaldoguerraNote Added: 0071621
2014-12-17 16:54hgbotCheckin
2014-12-17 16:54hgbotNote Added: 0072651
2014-12-17 16:54hgbotStatusscheduled => resolved
2014-12-17 16:54hgbotResolutionopen => fixed
2014-12-17 16:54hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/3dc87d15fe4707ce523e1dd95d909c6cb2d320f7 [^]
2014-12-17 16:55SandrahuguetReview Assigned To => Sandrahuguet
2014-12-17 16:55SandrahuguetNote Added: 0072652
2014-12-17 16:55SandrahuguetStatusresolved => closed
2014-12-17 16:55SandrahuguetFixed in Version => 3.0PR15Q1
2014-12-30 23:28hudsonbotCheckin
2014-12-30 23:28hudsonbotNote Added: 0073236
2015-01-12 11:00ngarciaRelationship addedcauses 0028560

Notes
(0071621)
reinaldoguerra   
2014-11-13 01:08   
Test plan used:
1- Login into Openbravo ERP
2- Go to product window and create a new one
3- Set it as Stocked = 'N' and Bill of materials = 'Y'
4- Go to price tab and add a sales price list version for example: tarifa de ventas. Define a product price.
5- Add at least one product (with a price different from 0 for example Cola 0,5L) in the Bill Of Materials tab
6- Click on Verify BOM button
7- Select "Entregas IVA 21%" record in tax rate window
8- Change the base amount value to Alternate Tax Base Amount
9- Create a new Sales Order, go to lines tab and add the previously created product. Also select Entregas IVA 21% as tax.
10- Execute Explode process and note that a new line is added for BOM product
11- Select it, go to Line tax tab and realize that taxable and tax amounts are properly calculated.
12- Create a new sales order under the same circunstances but now do not execute explode process
13- Execute book process and realize that for the BOM produect line, taxable and tax amounts fields are properly calculated.

14- Now create a new sales invoice, go to lines tab and add the previously created product. Also select Entregas IVA 21% as tax.
15- Execute Explode process and note that a new line is added for BOM product
16- Select it, go to Line tax tab and realize that taxable and tax amounts are properly calculated.
17- Create a new sales invoice under the same circunstances but now do not execute explode process
18- Complete it and realize that for the BOM produect line, taxable and tax amounts fields are properly calculated.
(0072651)
hgbot   
2014-12-17 16:54   
Repository: erp/devel/pi
Changeset: 3dc87d15fe4707ce523e1dd95d909c6cb2d320f7
Author: Reinaldo Guerra <reinaldo.guerra <at> peoplewalking.com>
Date: Tue Nov 11 19:11:47 2014 -0500
URL: http://code.openbravo.com/erp/devel/pi/rev/3dc87d15fe4707ce523e1dd95d909c6cb2d320f7 [^]

Fixed bug 27562: Taxable amount for not stocked bom lines is not calculated.

Sales order and invoice saving trigger's logic was changed to set Alternate Taxable Amount value, in tax rates defined as Alternate Tax Base Amount.
Now when explode or book processes are executed for BOM line products, the new lines are inserted with the Alternate Taxable Amount value well defined, so taxable and tax amounts are properly calculated.

---
M src-db/database/model/triggers/C_INVLINE_CHK_RESTRICTIONS_TRG.xml
M src-db/database/model/triggers/C_ORDERLINE_TRG.xml
---
(0072652)
Sandrahuguet   
2014-12-17 16:55   
Code review + testing OK
(0073236)
hudsonbot   
2014-12-30 23:28   
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