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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0023061
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 03. Procurement managementmajoralways2013-02-13 09:472013-02-20 04:50
ReportermiruritaView Statuspublic 
Assigned Toeduardo_Argal 
PrioritynormalResolutionfixedFixed in Version3.0MP20
StatusclosedFix in branchpiFixed in SCM revisionb32799632a82
ProjectionnoneETAnoneTarget Version
OSLinux 32 bitDatabasePostgreSQLJava version1.6.0_18
OS VersionCommunity ApplianceDatabase version8.3.9Ant version1.7.1
Product VersionpiSCM revision 
Review Assigned Tomirurita
Web browser
ModulesCore
Regression levelProduction - QA Approved
Regression date2012-12-13
Regression introduced in release
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/dc211cf0ad600ca95beca6deab04dea7967e929b [^]
Triggers an Emergency PackNo
Summary

0023061: Goods Receipt accounting is wrong when using breakdown records and product configured as "Book Using Purchase Order Price"

DescriptionGoods Receipt accounting is wrong when using breakdown records and product configured as "Book Using Purchase Order Price".

It is multiplying breakdown lines with the quantity of the line instead of the quantity of the breakdown.
Steps To Reproduce1) Login as System Administrator
2) Go to Purchase Order || Accounting dimension tab and change:
  - UI Pattern: Standard
  - Active: true
3) Login as F&B International Group Admin
4) Create 2 projects
  - MyProject1
  - MyProject2
5) Enable project dimension to be displayed at purchase order line breakdown
6) Go to Product window and update
     - Cerveza Ale 0,5L
          - Product Type: expense type
          - Sale: false
          - Book Using Purchase Order Price: true
     - Vino Blanco 0,75L
          - Product Type: expense type
          - Sale: false
          - Book Using Purchase Order Price: true
7) Create new Purchase Order
  Header
   - Organization: F&B España - Región Norte
   - Business Partner: Bebidas Alegres, S.L.

   * Line
     - Line No: 10
     - Product: Cerveza Ale 0,5L
     - Ordered Quantity: 100
     - Net Unit Price: 1.36
     - Tax: Compras exentas

      * Accounting dimension 1
        - Line: 10
        - Amount: 100
        - Project: MyProject1
      * Accounting dimension
        - Line: 20
        - Amount: 36
        - Project: MyProject2

   * Line
     - Line No: 20
     - Product: Vino Blanco 0,75L
     - Ordered Quantity: 1
     - Net Unit Price: 3.55
     - Tax: Compras exentas
8) Go to Pending Goods receipts and receive the full order
9) Enable goods receipt accounting (General Ledger Configuration || Active Tables)
10) Go to created Goods Receipt document and post
11) Realize that the generated accounting is wrong. See Accounting tab.

(see attached screenshot)
Proposed Solutionsee attached the proposed solution
TagsNo tags attached.
Attached Filespng file icon DocInOut_Accounting_Breakdown.png [^] (140,047 bytes) 2013-02-13 09:47


diff file icon DocInOut_Breakdown.diff [^] (733 bytes) 2013-02-13 09:48 [Show Content]
diff file icon other_proposed_change.diff [^] (2,723 bytes) 2013-02-14 15:47 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]
related to defect 0035499 closedmcobas Currency conversion problem when product is configured as "Book Using Purchase Order Price" 

-  Notes
(0056438)
hgbot (developer)
2013-02-14 12:32

Repository: erp/devel/pi
Changeset: 3c767e36278c9df506e4ebe109f78737d57da964
Author: Eduardo Argal Guibert <eduardo.argal <at> openbravo.com>
Date: Thu Feb 14 12:31:52 2013 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/3c767e36278c9df506e4ebe109f78737d57da964 [^]

Fixes bug 23061

---
M src/org/openbravo/erpCommon/ad_forms/DocInOut.java
---
(0056447)
mirurita (developer)
2013-02-14 15:47

Not working if the product is not marked as "Book Using Purchase Order Price". The cost is calculated based only in the line quantity, not breakdown quantity.

see attached diff as starting point.
(0056487)
hgbot (developer)
2013-02-15 11:31

Repository: erp/devel/pi
Changeset: b32799632a82fed7c36f356e6751e2861b35b416
Author: Eduardo Argal Guibert <eduardo.argal <at> openbravo.com>
Date: Fri Feb 15 11:31:06 2013 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/b32799632a82fed7c36f356e6751e2861b35b416 [^]

Fixes bug 23061

---
M src/org/openbravo/erpCommon/ad_forms/DocInOut.java
M src/org/openbravo/erpCommon/ad_forms/DocLineInOut_data.xsql
M src/org/openbravo/erpCommon/ad_forms/DocLine_Material.java
---
(0056495)
mirurita (developer)
2013-02-15 17:53

code review + testing OK

Testing Plans:
1) Scenario described in the issue.
2) Same as 1) but both products not flagged as "Book Using Purchase Order Price"
3) Same as 2) but without breakdown records
4) Sames as 1) but both products not flagged as "Book Using Purchase Order Price"
(0056600)
hudsonbot (developer)
2013-02-20 04:50

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/d828167a9e83 [^]

Maturity status: Test
(0056607)
hudsonbot (developer)
2013-02-20 04:50

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/d828167a9e83 [^]

Maturity status: Test

- Issue History
Date Modified Username Field Change
2013-02-13 09:47 mirurita New Issue
2013-02-13 09:47 mirurita Assigned To => dmiguelez
2013-02-13 09:47 mirurita File Added: DocInOut_Accounting_Breakdown.png
2013-02-13 09:47 mirurita Modules => Core
2013-02-13 09:47 mirurita Triggers an Emergency Pack => No
2013-02-13 09:48 mirurita File Added: DocInOut_Breakdown.diff
2013-02-13 09:49 mirurita Regression level => Production - QA Approved
2013-02-13 09:49 mirurita Regression date => 2012-12-13
2013-02-13 09:49 mirurita Regression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/dc211cf0ad600ca95beca6deab04dea7967e929b [^]
2013-02-13 13:06 eduardo_Argal Assigned To dmiguelez => eduardo_Argal
2013-02-13 17:26 eduardo_Argal Status new => scheduled
2013-02-13 17:26 eduardo_Argal fix_in_branch => pi
2013-02-14 12:32 hgbot Checkin
2013-02-14 12:32 hgbot Note Added: 0056438
2013-02-14 12:32 hgbot Status scheduled => resolved
2013-02-14 12:32 hgbot Resolution open => fixed
2013-02-14 12:32 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/3c767e36278c9df506e4ebe109f78737d57da964 [^]
2013-02-14 15:47 mirurita Note Added: 0056447
2013-02-14 15:47 mirurita Status resolved => new
2013-02-14 15:47 mirurita Resolution fixed => open
2013-02-14 15:47 mirurita File Added: other_proposed_change.diff
2013-02-15 11:29 eduardo_Argal Status new => scheduled
2013-02-15 11:31 hgbot Checkin
2013-02-15 11:31 hgbot Note Added: 0056487
2013-02-15 11:31 hgbot Status scheduled => resolved
2013-02-15 11:31 hgbot Resolution open => fixed
2013-02-15 11:31 hgbot Fixed in SCM revision http://code.openbravo.com/erp/devel/pi/rev/3c767e36278c9df506e4ebe109f78737d57da964 [^] => http://code.openbravo.com/erp/devel/pi/rev/b32799632a82fed7c36f356e6751e2861b35b416 [^]
2013-02-15 17:53 mirurita Review Assigned To => mirurita
2013-02-15 17:53 mirurita Note Added: 0056495
2013-02-15 17:53 mirurita Status resolved => closed
2013-02-15 17:53 mirurita Fixed in Version => 3.0MP20
2013-02-20 04:50 hudsonbot Checkin
2013-02-20 04:50 hudsonbot Note Added: 0056600
2013-02-20 04:50 hudsonbot Checkin
2013-02-20 04:50 hudsonbot Note Added: 0056607
2017-03-23 13:16 aferraz Relationship added related to 0035499


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker