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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0038685
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 04. Warehouse managementmajoralways2018-06-01 13:422018-06-28 08:21
ReportermaiteView Statuspublic 
Assigned Tocollazoandy4 
PriorityurgentResolutionfixedFixed in Version3.0PR18Q3
StatusclosedFix in branchFixed in SCM revision20cd44382a65
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned ToSandrahuguet
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0038685: It is not possible to process a Physical Inventory for a product with attributes that has negative stock without attributes

DescriptionWhen there is no stock for a product defined with attributes but is sold via POS, a goods shipment document will be created for this product without attributes. Then the user needs to regularize this stock but is not able to process a Physical Inventory document.
Steps To Reproduce0. Access POS Terminal (https://livebuilds.openbravo.com/retail_mp_3.0RR18Q2/web/org.openbravo.retail.posterminal/?terminal=VBS-1#retail.pointofsale [^]) and create ticket using "Hiking Boots" product. Pay ticket
1. Access to backoffice. Create new Physical Inventory for "Vall Blanca Store" organization. Add line and open product selector to set "Hiking Boots" products with qty=-1. Run "Process Inventory Count" and error "In line 10 the product has no attribute set" will be received
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
related to defect 0038687 closedrqueralta Retail Modules Autobom not working wth products that have attributes and if there is not stock. 

-  Notes
(0105288)
collazoandy4 (developer)
2018-06-21 00:15

Test Plan I
  Access POS Terminal and create ticket using "Hiking Boots" product. Pay ticket
  Access to backoffice. Create new Physical Inventory for Vall Blanca Store organization.
  Add line and open product selector to set Hiking Boots products with qty=0.
  Run Process Inventory Count and check the process ends without errors
  Check the product stock was updated

Test Plan II
  Create another Physical Inventory for Vall Blanca Store organization
  Add line and open product selector to set Hiking Boots products with qty=5.
  Run Process Inventory Count and check the error message 'In line 10 the product has no attribute set' is shown
(0105461)
hgbot (developer)
2018-06-27 08:57

Repository: erp/devel/pi
Changeset: 20cd44382a6502e47c1aee6a5e6d9eabd996d6fa
Author: Armaignac <collazoandy4 <at> gmail.com>
Date: Wed Jun 20 18:17:10 2018 -0400
URL: http://code.openbravo.com/erp/devel/pi/rev/20cd44382a6502e47c1aee6a5e6d9eabd996d6fa [^]

Fixes issue 38685:It is not possible to process a Physical Inventory for a
product with attributes that has negative stock without attributes

When a new physical inventory was process with a product without an attribute set
defined an exception was throw aborting the process.

Now a physical inventory can be process if the product have not attribute set defined
and no available stock in the bin.

---
M src-db/database/model/triggers/M_TRANSACTION_TRG.xml
M src/org/openbravo/materialmgmt/InventoryCountProcess.java
---
(0105462)
hgbot (developer)
2018-06-27 08:57

Repository: erp/devel/pi
Changeset: 8da978492eeb1bb897ba220dfee905030d17cd12
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Mon Jun 25 16:11:54 2018 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/8da978492eeb1bb897ba220dfee905030d17cd12 [^]

Related to Issue 38685. Code Review changes

---
M src-db/database/model/triggers/M_TRANSACTION_TRG.xml
M src/org/openbravo/materialmgmt/InventoryCountProcess.java
---
(0105463)
hgbot (developer)
2018-06-27 08:57

Repository: erp/devel/pi
Changeset: fcd2b0c4708176dd341897f220e4d96a06bc01b0
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Mon Jun 25 17:15:16 2018 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/fcd2b0c4708176dd341897f220e4d96a06bc01b0 [^]

Related to Issue 38685. Code Review changes:

* When querying for exising stock, take into account that the uom
  is the same one as the uom of the inventory line
* When querying for existing stock, check that there is no stock
  assign to draft transactions
* When checking for existing stock, do it only when the quantitycount
  is 0
* Changed the query to be more readable
* Added explanatory commentary

---
M src/org/openbravo/materialmgmt/InventoryCountProcess.java
---
(0105464)
hgbot (developer)
2018-06-27 08:57

Repository: erp/devel/pi
Changeset: 91b8d9df5a3bbe33e18b2669db5320452d4bcb19
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Mon Jun 25 17:57:48 2018 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/91b8d9df5a3bbe33e18b2669db5320452d4bcb19 [^]

Related to Issue 38685. Refactor of runChecks method

* Split method into smaller ones
* Renamed variables with obtuse names
* Delete unnecessary exceptions thrown
* Changed queries to make them more readable

---
M src/org/openbravo/materialmgmt/InventoryCountProcess.java
---
(0105465)
hgbot (developer)
2018-06-27 08:57

Repository: erp/devel/pi
Changeset: 79f006ced359347f2a6cbf384ff95586eb243b9f
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Tue Jun 26 11:42:34 2018 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/79f006ced359347f2a6cbf384ff95586eb243b9f [^]

Related to Issue 38685. Code Review changes:

* Formatted code to make it more readable
* Added check of quantity in draft transactions to query that retrieves stock
* Changed logic to check if the attribute can be null
  * I the QuantityCount of the line is <> 0, attribute is mandatory
  * If there is no stock with emtpy attribute, attribute is mandatory
  * If there is previous stock with empty attribute and the quantity count
    is 0 then the attribute can be null

---
M src-db/database/model/triggers/M_TRANSACTION_TRG.xml
---
(0105466)
Sandrahuguet (developer)
2018-06-27 09:40

Code review + testing OK
(0105470)
hudsonbot (developer)
2018-06-27 13:04

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/7721d0db9ee1 [^]
Maturity status: Test
(0105471)
hudsonbot (developer)
2018-06-27 13:04

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/7721d0db9ee1 [^]
Maturity status: Test
(0105472)
hudsonbot (developer)
2018-06-27 13:04

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/7721d0db9ee1 [^]
Maturity status: Test
(0105473)
hudsonbot (developer)
2018-06-27 13:04

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/7721d0db9ee1 [^]
Maturity status: Test
(0105474)
hudsonbot (developer)
2018-06-27 13:04

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/7721d0db9ee1 [^]
Maturity status: Test
(0105479)
hgbot (developer)
2018-06-28 08:21

Repository: erp/devel/pi
Changeset: 58eb040e2d99f678863d84205db30f1337a9245d
Author: Sandra Huguet <sandra.huguet <at> openbravo.com>
Date: Thu Jun 28 08:21:40 2018 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/58eb040e2d99f678863d84205db30f1337a9245d [^]

related to issue 38685 delete unused variable

---
M src-db/database/model/triggers/M_TRANSACTION_TRG.xml
---

- Issue History
Date Modified Username Field Change
2018-06-01 13:42 maite New Issue
2018-06-01 13:42 maite Assigned To => Triage Finance
2018-06-01 13:42 maite Modules => Core
2018-06-01 13:42 maite Resolution time => 1529618400
2018-06-01 13:42 maite Triggers an Emergency Pack => No
2018-06-01 13:43 maite Issue Monitored: networkb
2018-06-01 14:16 VictorVillar Relationship added related to 0038687
2018-06-13 11:48 dmiguelez Assigned To Triage Finance => AtulOpenbravo
2018-06-19 12:42 dmiguelez Assigned To AtulOpenbravo => collazoandy4
2018-06-21 00:15 collazoandy4 Status new => scheduled
2018-06-21 00:15 collazoandy4 Note Added: 0105288
2018-06-27 08:57 hgbot Checkin
2018-06-27 08:57 hgbot Note Added: 0105461
2018-06-27 08:57 hgbot Status scheduled => resolved
2018-06-27 08:57 hgbot Resolution open => fixed
2018-06-27 08:57 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/20cd44382a6502e47c1aee6a5e6d9eabd996d6fa [^]
2018-06-27 08:57 hgbot Checkin
2018-06-27 08:57 hgbot Note Added: 0105462
2018-06-27 08:57 hgbot Checkin
2018-06-27 08:57 hgbot Note Added: 0105463
2018-06-27 08:57 hgbot Checkin
2018-06-27 08:57 hgbot Note Added: 0105464
2018-06-27 08:57 hgbot Checkin
2018-06-27 08:57 hgbot Note Added: 0105465
2018-06-27 09:40 Sandrahuguet Review Assigned To => Sandrahuguet
2018-06-27 09:40 Sandrahuguet Note Added: 0105466
2018-06-27 09:40 Sandrahuguet Status resolved => closed
2018-06-27 09:40 Sandrahuguet Fixed in Version => 3.0PR18Q3
2018-06-27 13:04 hudsonbot Checkin
2018-06-27 13:04 hudsonbot Note Added: 0105470
2018-06-27 13:04 hudsonbot Checkin
2018-06-27 13:04 hudsonbot Note Added: 0105471
2018-06-27 13:04 hudsonbot Checkin
2018-06-27 13:04 hudsonbot Note Added: 0105472
2018-06-27 13:04 hudsonbot Checkin
2018-06-27 13:04 hudsonbot Note Added: 0105473
2018-06-27 13:04 hudsonbot Checkin
2018-06-27 13:04 hudsonbot Note Added: 0105474
2018-06-28 08:21 hgbot Checkin
2018-06-28 08:21 hgbot Note Added: 0105479


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker