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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0032469
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Modules] Warehouse Pickingmajoralways2016-03-14 14:232018-04-04 15:18
ReporterpsanjuanView Statuspublic 
Assigned Tocollazoandy4 
PriorityurgentResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revisiond3a9f9fa95ab
ProjectionnoneETAnoneTarget Version
OSLinux 32 bitDatabasePostgreSQLJava version1.6.0_18
OS VersionCommunity ApplianceDatabase version8.3.9Ant version1.7.1
Product VersionSCM revision 
Merge Request Status
Regression date
Regression introduced by commit
Regression level
Review Assigned Todmiguelez
Support ticket740
OBNetwork customerOBPS
Regression introduced in release
Summary

0032469: Not possible to create a partial "Outbound" Picking List if there is not a line that can be completely shipped

DescriptionNot possible to create a partial "Outbound" Picking List if there is not a line that can be completely shipped.

In case there exists a sales order line with all the quantity available in the stock, the rest of lines will be created with the available stock although it is not enough
Steps To ReproduceAs F&B International Group Admin
  
  Go to Product window
    Create a new product (ie. Product 32281 Test 3)
    Add a sales price list (Tarifa Ventas)
    Add a purchase price list (Tarifa Bebidas Alegres)
  
  Go to Good Receipts window
    Receipt 2 units of this product, from vendor Bebidas Alegres
    WH = España Region Norte, RN-1-0-0
    Complete it.

  Go to Sales Order window
    Create a new Sales Order for BP = Alimentos y Supermercados
    Add line of this new product. Qty = 10
    Book it.

  Go to Sales Order window main tab
    Generate Picking List
    Choose "Outbound Picking List"
    España Region Norte: RN-DEV (Outbound bin)
    Press OK

  Check that a message is shown: There is not enough available stock.

  Reactivate the Sales Order
  Add a line with 1 unit of a product with stock
  Book it
  
  Generate Picking List
  Choose "Outbound Picking List"
  España Region Norte: RN-DEV (Outbound bin)
  Press OK
  Check 2 units of previously created product have been included in the Warehouse Picking List
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to defect 0032281 closedrqueralta wrong movement quantity of the picking list without enough stock available 

-  Notes
(0103392)
collazoandy4 (viewer)
2018-03-21 22:04

 Test Plan
  Enable Stock Reservation Preference
  Edit RN-Dev bin and set Type: Outbound from Region Norte Warehouse
  Create a new Product and add a Sales Price List
  Create a new Sales Order for that product qty = 10 and book it
  In Sales Order window main tab
    Generate Picking List
    Choose "Outbound Picking List"
    España Region Norte: RN-DEV
    Press OK and check that an error message is shown: There is not enough available stock.
  Receipt 2 units of this product
  In Sales Order window main tab
    Generate Picking List
    Choose "Outbound Picking List"
    España Region Norte: RN-DEV
    Press OK and check that an information message is shown indicating a partial Picking was created


Test Plan II
  Create a new Sales Order for the previous product qty = 10 and book it
  Create a new Picking List outbound document
  Clic on Sales Orders
    Select the previous order created
    Clic on Done and check that an error message is shown: There is not enough available stock.
  Reactivate and edit the previous order, set a new line with an stocked product qty = 6
  Go to the previous Picking List created and Clic on Sales Orders
    Select the previous order created
    Clic on Done and check the process ends without errors and the stocked line was added to the Picking
(0103673)
hgbot (developer)
2018-04-04 15:16

 Repository: erp/pmods/org.openbravo.warehouse.pickinglist
Changeset: d3a9f9fa95abb1333e2b0e33690e4595ea618c4d
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Tue Mar 27 11:42:32 2018 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.warehouse.pickinglist/rev/d3a9f9fa95abb1333e2b0e33690e4595ea618c4d [^]

Fixes Issue 32469. Properly manage the zeroStockLines variable.
If it is not incremented for partial pickings, it might happen that
the "No Stock available" error is raised instead of creating a partial
Picking.

Changed StringBuilder with StringBuffer
Changed HashSet with more generic Set type

---
M src/org/openbravo/warehouse/pickinglist/Utilities.java
---
(0103674)
hgbot (developer)
2018-04-04 15:16

 Repository: erp/pmods/org.openbravo.warehouse.pickinglist
Changeset: aa00aab8050e7fd8a3a67d72f37f6dfa59a914df
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Tue Mar 27 12:14:59 2018 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.warehouse.pickinglist/rev/aa00aab8050e7fd8a3a67d72f37f6dfa59a914df [^]

Related to Issue 32469. Refactor code:

* Extracted validations code into it's own method
* Extracted loop logic for orderLines into it's own method
* Remove commented code
* Simplify message variable logic. It was initiated but never used until the end
  So it is not necessary to check it's lenght and append any message, just
  return it
* Extracted query to retrieve lines into it's own method

---
M src/org/openbravo/warehouse/pickinglist/Utilities.java
---
(0103675)
hgbot (developer)
2018-04-04 15:16

 Repository: erp/pmods/org.openbravo.warehouse.pickinglist
Changeset: 348d436ddce4778895b4247f15981bf1dfcc5a19
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Wed Apr 04 15:09:51 2018 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.warehouse.pickinglist/rev/348d436ddce4778895b4247f15981bf1dfcc5a19 [^]

Related to Issue 32469: Updated Module version

---
M src-db/database/sourcedata/AD_MODULE.xml
---
(0103676)
dmiguelez (viewer)
2018-04-04 15:18

 Code Review + Testing Ok

- Issue History
Date Modified Username Field Change
2016-03-14 14:23 psanjuan New Issue
2016-03-14 14:23 psanjuan Assigned To => Triage Finance
2016-03-14 14:23 psanjuan OBNetwork customer => No
2016-03-14 14:24 psanjuan Relationship added related to 0032281
2016-03-14 14:25 psanjuan Steps to Reproduce Updated View Revisions
2018-03-14 16:49 ngarcia Support ticket => 740
2018-03-14 16:49 ngarcia OBNetwork customer No => Yes
2018-03-14 16:49 ngarcia Summary Not possible to create a partial "Outbound" Picking List. => Not possible to create a partial "Outbound" Picking List if there is not a line that can be completely shipped
2018-03-14 16:49 ngarcia Description Updated View Revisions
2018-03-14 16:49 ngarcia Steps to Reproduce Updated View Revisions
2018-03-14 16:49 ngarcia Issue Monitored: ngarcia
2018-03-14 16:49 ngarcia Resolution time => 1522792800
2018-03-16 08:57 aferraz Assigned To Triage Finance => collazoandy4
2018-03-20 23:14 collazoandy4 Status new => scheduled
2018-03-21 22:04 collazoandy4 Note Added: 0103392
2018-04-04 15:16 hgbot Checkin
2018-04-04 15:16 hgbot Note Added: 0103673
2018-04-04 15:16 hgbot Status scheduled => resolved
2018-04-04 15:16 hgbot Resolution open => fixed
2018-04-04 15:16 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.warehouse.pickinglist/rev/d3a9f9fa95abb1333e2b0e33690e4595ea618c4d [^]
2018-04-04 15:16 hgbot Checkin
2018-04-04 15:16 hgbot Note Added: 0103674
2018-04-04 15:16 hgbot Checkin
2018-04-04 15:16 hgbot Note Added: 0103675
2018-04-04 15:18 dmiguelez Review Assigned To => dmiguelez
2018-04-04 15:18 dmiguelez Note Added: 0103676
2018-04-04 15:18 dmiguelez Status resolved => closed

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker