0037193: Negative Stock can be obtained although the storage bin is defined as OverIssue = N (not allowing negative stock)

Description

Negative Stock can be obtained although the storage bin is defined as OverIssue = N (not allowing negative stock)

Steps To Reproduce

As group admin role:
   Check all the storage bins for Region Norte and Region Sur warehouses have a status that do not allow overissue
   Create a new product for F&B España, S.A
   Set it as Purchase, Sale and Stocked
   Add a purchase price list to it
   Create a purchase order for F&B España - Región Norte and add two units of the product
   Create the related goods receipt
   Create a goods movement for F&B España, S.A and select the stock of Region Norte warehouse
   Increase the Movement Quantity from 2 to 3
   Move it to a storage bin in Region Sur warehouse
   Check you can process the document and you should not
   Create a Goods Shipment F&B España - Región Sur
   Select the stock of Region Sur warehouse
   Increase the Movement Quantity from 3 to 4
   Check you can process the document and you should not
   Launch the stock report and check the stock of the product is -1 in each warehouse

Tags

No tags attached.

Attached Files

37193.diff [^] (615 bytes) 2018-01-25 18:03 [Show Content] [Hide Content]

diff -r 688697b19d36 src-db/database/model/functions/M_CHECK_STOCK.xml
--- a/src-db/database/model/functions/M_CHECK_STOCK.xml	Fri Nov 03 04:09:50 2017 +0000
+++ b/src-db/database/model/functions/M_CHECK_STOCK.xml	Thu Jan 25 18:02:48 2018 +0100
@@ -51,7 +51,7 @@
       FROM M_STORAGE_DETAIL
       WHERE M_PRODUCT_ID=p_M_Product_ID
         AND AD_CLIENT_ID=p_AD_Client_ID
-        AND AD_ORG_ID=p_AD_Org_ID
+        AND AD_ORG_ISINNATURALTREE(AD_ORG_ID, p_AD_Org_ID, p_AD_Client_ID) = 'Y'
       AND (QTYONHAND<0 OR COALESCE(QTYORDERONHAND,0)<0));    
     IF(v_ExistsNegativeStock <> 0) THEN
       p_Result:=0;

Relationships [ Relation Graph ] [ Dependency Graph ]

blocks

defect

0037168

closed

dmiguelez

Negative Stock can be obtained although the storage bin is defined as OverIssue = N (not allowing negative stock)

Notes
(0100219) dmiguelez (developer) 2017-11-02 16:00	This regression has been introduced with the Inventory Status functionality. This functionality has been published in 17Q3. The problem does not happen before. http://wiki.openbravo.com/wiki/Modules:Inventory_Status [^]

(0101976) ngarcia (developer) 2018-01-25 18:03 edited on: 2018-01-25 18:07	Attached a possible patch for 3.0PR17Q2.4

Issue History
Date Modified	Username	Field	Change
2017-10-30 21:20	collazoandy4	Type	defect => backport
2017-10-30 21:20	collazoandy4	Target Version	=> 3.0PR17Q2.4
2017-11-02 16:00	dmiguelez	Review Assigned To	=> dmiguelez
2017-11-02 16:00	dmiguelez	Note Added: 0100219
2017-11-02 16:00	dmiguelez	Status	scheduled => closed
2017-11-02 16:00	dmiguelez	Resolution	open => unable to reproduce
2018-01-25 18:03	ngarcia	File Added: 37193.diff
2018-01-25 18:03	ngarcia	Note Added: 0101976
2018-01-25 18:07	ngarcia	Note Edited: 0101976	View Revisions