Feature Request - Quarantine warehouse #26384
Comments
|
Hello, maybe the constant ENTREPOT_EXTRA_STATUS already do the job ? |
|
@altairis-tof Well spotted :) It's close but not quite. The "internal" warehouse status would also prohibit reception (i.e. if you want to process QA steps on purchased items such as contract-manufactured goods and only release after QC pass on goods prior to sale), and won't allow you to sell the quarantine goods if you explicitly choose that warehouse (i.e. you are performing a liquidation on damaged items). Now of course some of this could be handled by instead having inventory lotting (as would be needed for doing FIFO/LIFO costing too) with quantity parameters linked to lots, ABC analysis, etc. but this is all too complex for Dolibarr at this stage IMHO. Simpler to just implement a quarantine warehouse type to support some basic GxP processes. |
|
Hi there |
|
@josett225 This is a good idea functionally to use virtual stock in this way if all you need is to block the inventory from sale, but it breaks when you need to separate your tangible stock from virtual stock for valuation. Virtual stock is future/maybe stock and doesn't count when you look at inventory valuation. When you prepare an inventory value report, you should be looking at quantities and values such as landed cost of your physical inventory. By having a dedicated quarantine warehouse, in the manner described, you don't need to jump through hoops to calculate your inventory value as the physical quantities are correctly registered. |
|
Hi @Ashley-Butcher |
…ting warehouses e.g. quarantine This feature complements the need explained in Dolibarr#26384 and the implementation of the STATUS_OPENEXT_ALL warehouse type. With the changes a warehouse can be used e.g. as a quarantine warehouse or a temporary pre-QC warehouse. The status "Open (omitted from stock calculation)" keeps the definition vague enough as not to constrain the use of this status too much in the sense of "either quarantine or pre-QC". For code-readability and convenience the change is using the load_stock and loadWarehouses implementation. As the option of warehouseclosed, warehouseopen and warehouseinternal is already introduced in the code we can piggyback ride with the introduction of the STATUS_OPENEXT_ALL onto the product list view, stock transfer, delivery selection etc.
|
Hello together, We were having a similar need (quarantine warehouse "clogging up" the stock calculation in the product list. Therefore I brewed up this PR. Happy to hear feedback. |
…non-counting warehouses e.g. quarantine" This reverts commit 0d069b4.
…on-counting warehouses e.g. quarantine This feature implements the global variable of STOCK_USE_WAREHOUSE_USAGE. This variable allows warehouses to exclude their products from stock calculations. A button was added to the stock module in the admin panel. A simple "AND" check for the warehouse_usage column in the entrepot SQL table is added to the load_stock function to facilitate this change.
Feature Request
It would be useful to be able to mark a warehouse as a quarantine warehouse. This would allow stock to be recorded in inventory but be unavailable for order, without having to implement stock lotting and inventory quality recording.
A quarantine warehouse should function exactly the same as a normal warehouse, only with a boolean flag "Quarantine" set to "true" which indicates that this stock is not to be proposed on a sales order when preparing a shipment.
Use case
Dolibarr doesn't have any quality control processes, which makes rigidly implementing GxP difficult, but not impossible.
Helpful would be the ability to control the quality of the stock. At the very least, damaged stock or returned stock should be in a quarantined status pending a quality control. To do this simply, stock can enter a quarantine warehouse where it is isolated from the main stock and cannot be sold by accident.
Implementing a quarantine warehouse is a simple but very effective way of adding a small level quality control.
Scenario 1:
Scenario 2:
Scenario 3:
Suggested implementation
The simplest implemention is as follows:
Suggested steps
llx_entrepot) of integer type with values0= "Normal" and1= "Quarantine" (maybe this can be extended in future with 'transit', but that's a user story for another day)1)The text was updated successfully, but these errors were encountered: