Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0021017Openbravo ERP07. Sales managementpublic2012-07-11 17:132012-07-27 13:25
ReporterVictorVillar 
Assigned Tojecharri 
PriorityhighSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version3.0MP12.1 
Target Version3.0MP14Fixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0021017: Closing an order with negative lines change the qty to zero even when there is a goods shipment/receipt associated
DescriptionClosing an order with negative qty in their lines, change the qty to zero even when there is a goods shipment/receipt associated
Steps To Reproduceas group Admin role,
Go to Sales order,
  Create a new register
 Go to Lines tab,
   Create a new Line
        Product: Cola
        Quantity: -1
   2nd line:
        Product: Ale beer
        Qty: 1
Complete it
Go to Create shipment from Order
   select the order created before
Check that the Goods Shipment created has the same lines
Go to Sales order again
   Close it
Realise that the line with negative quantity, now has zero
This is not correct. It should have the same value as the Delivered quantiy

The same is happening in Purchase flow.
  
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2012-07-11 17:13VictorVillarNew Issue
2012-07-11 17:13VictorVillarAssigned To => mirurita
2012-07-11 17:13VictorVillarModules => Core
2012-07-11 17:18miruritaAssigned Tomirurita => jonalegriaesarte
2012-07-11 18:06VictorVillarResolution time => 1344981600
2012-07-11 18:06VictorVillarIssue Monitored: networkb
2012-07-11 18:07VictorVillarversion => 3.0MP12.1
2012-07-11 18:07VictorVillarTarget Version => 3.0MP14
2012-07-24 08:51jonalegriaesarteAssigned Tojonalegriaesarte => jecharri
2012-07-26 11:37hgbotCheckin
2012-07-26 11:37hgbotNote Added: 0050896
2012-07-26 11:37hgbotStatusnew => resolved
2012-07-26 11:37hgbotResolutionopen => fixed
2012-07-26 11:37hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/e31d81e9efa7fcf9d9b25d16968637d1781f52f6 [^]
2012-07-26 11:42jecharriNote Added: 0050897
2012-07-27 02:18hudsonbotCheckin
2012-07-27 02:18hudsonbotNote Added: 0050917
2012-07-27 13:25VictorVillarNote Added: 0050939
2012-07-27 13:25VictorVillarStatusresolved => closed

Notes
(0050896)
hgbot   
2012-07-26 11:37   
Repository: erp/devel/pi
Changeset: e31d81e9efa7fcf9d9b25d16968637d1781f52f6
Author: Javier Etxarri <javier.echarri <at> openbravo.com>
Date: Thu Jul 26 11:36:17 2012 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/e31d81e9efa7fcf9d9b25d16968637d1781f52f6 [^]

Fixes issue 21017: Closing an order with negative
lines change the qty to zero even when there is a goods shipment/receipt associated

---
M src-db/database/model/functions/C_ORDER_POST1.xml
---
(0050897)
jecharri   
2012-07-26 11:42   
Test plan:

1.Create a sales order Qty Ordered: 5 units, Qty delivered: 0, Qty invoiced: 3, close it. Qty ordered: 3
2.Create a sales order Qty Ordered: 5 units, Qty delivered: 2, Qty invoiced: 3, close it. Qty ordered: 3
3.Create a sales order Qty Ordered: 5 units, Qty delivered: 3, Qty invoiced: 0, close it. Qty ordered: 3
4.Create a sales order Qty Ordered: 5 units, Qty delivered: 4, Qty invoiced: 2, close it. Qty ordered: 4
5.Create a sales order Qty Ordered: 5 units, Qty delivered: 3, Qty invoiced: 1. close it. Qty ordered: 3
6.Create a sales order Qty Ordered: -5 units, Qty delivered: -3, Qty invoiced: 0, close it. Qty ordered: -3
7.Create a sales order Qty Ordered: -5 units, Qty delivered: 0, Qty invoiced: -2, close it. Qty ordered: -2
8.Create a sales order Qty Ordered: -5 units, Qty delivered: -3, Qty invoiced: -1. close it. Qty ordered: -3
9.Create a sales order Qty Ordered: 5 units, Qty delivered: -3, Qty invoiced: 1. close it. Qty ordered: 1
(0050917)
hudsonbot   
2012-07-27 02:18   
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/4cfe6f4cd84a [^]

Maturity status: Test
(0050939)
VictorVillar   
2012-07-27 13:25   
verified for Sales and Puchase flow with the steps defined in the testing plan