Commit Graph

13 Commits

Author SHA1 Message Date
Lucas Perais (lpe) 46263eb398 [FIX] stock: backport fix negative quants
Original commit (in 10.0) be9dce625c55e1b2d6039573c7035d61f762edc8

From original commit:
It is still possible to have negative and positive quants in the same
location because of returns: if you send something to the customer that
is not there and you return it, you will still be able to reserve the
returned goods to send to another client.

Before, if you would do an inventory adjustment, it would not take into
account these returned quants and their negative counterpart, which made
them difficult to get out of the system.

This fix takes them into account by creating two movements for one
inventory line: move the positive counterpart to the inventory location
before getting back from this location the same quantity.

This way, even if you have 0 as quantity on hand but you have those 2
quants, it will eliminate them. (if you are increasing the stock, part
of the process might have done it automatically already).

Also, a key of context has been added which authorizes the process described above in the case of both a tracked product and no lot_id on the stock inventory

OPW 743107
Closes #17167
2017-06-15 08:42:57 +02:00
Josse Colpaert 1de37862d6 [FIX] stock: when matching moves with pack operations through quants, it did not match the packages correctly. Courtesy of Pedro Baeza
Before, it did a search to see if the package existed,
but the only thing it needs to do is see if the package
on the pack operation corresponds to that of the quant.  (no need to check children also)

There is a test added:
A picking with 120 pieces incoming, 120 in pack 1, 80 in pack2
When we deliver those in a picking out, with product pack operations: (by taking them out of the pack)

120 from pack 1 and 80 from pack2,
we should only have 2 quants and links between moves in the end

And before, it generated 3 because it matched the wrong quants and made the wrong links.

opw 693760 closes #13836
2016-11-16 11:26:20 +01:00
Alexandre Fayolle 74b7b97209 [FIX] stock: let resupply from warehouse work
If you set WH B to be resupplied from WH A, then the scheduler will
generate a procurement with warehouse_id = B and location_id = B.stock.
Running the procurement will find the resupply rule, and this will
create another procurement with warehouse_id = A and location_id =
transit location.

However, without this patch, the resupply route is not part of the
route_ids of warehouse A, and so the 2nd procurement goes in exception
because if cannot find a rule (the search will force a rule linked to a
route which is part of A.route_ids).

Closes #7956
2015-08-12 08:14:48 +02:00
Leonardo Pistone 01f1d91006 [FIX] stock: use owner for virtual stock and in/out
Fixes odoo/odoo#5814.

It will also calculate virtual stock when lot/owner/pack is specified, but will
not filter the future stock on lot/pack.
2015-04-10 11:14:30 +02:00
Josse Colpaert 09abac7608 [FIX] Stock: Make sure inventory works with packs and lot = False
The inventory should work with packs.  If a pack is not indicated
in the inventory line, it means we correct the quantity for the product
that is not in a pack.

Also, when the lot is False, it should not behave as normal stock moves.
It should correct for the quantity that has no lot.

We try to reconcile the negative quants in the pack also.
2015-03-13 12:11:17 +01:00
Xavier Morel 65cd4a2a33 [FIX] remove deprecated checks/fast_suite test attributes from standard modules 2015-01-15 14:31:40 +01:00
Josse Colpaert 88169ec625 [FIX] Dozen / Ton spelling
Closes #4554
2015-01-06 17:21:41 +01:00
jir 6c23a9ca19 [ADD] Test cases of stock in new api. 2015-01-06 17:05:11 +01:00
jke-openerp 5bce036a6a Test on multiCompany updated + some other tiny bug
bzr revid: jke@openerp.com-20131015122216-m5itjbwouetc3loj
2013-10-15 14:22:16 +02:00
Quentin (OpenERP) 3494e46002 [FIX] temporary removed test suite of stock that will need to be set back later on. The problem is that the defauly_picking_type_id is not given in the context, leading to no default values for locations which raise a false positive in the tests
bzr revid: qdp-launchpad@openerp.com-20131011114514-y2vcnkgae70r5h4f
2013-10-11 13:45:14 +02:00
Quentin (OpenERP) c1a49a0544 [FIX] stock: removed test on stock.fill.inventory as the model doesn't exist anymore
bzr revid: qdp-launchpad@openerp.com-20131011065957-mgealr16658k5t24
2013-10-11 08:59:57 +02:00
Martin Trigaux 1d431fe4f8 [FIX] stock,mrp: move multicomp user to test instead of demo data
bzr revid: mat@openerp.com-20130828085019-cgbj6zrug73it3nw
2013-08-28 10:50:19 +02:00
Martin Trigaux 22a3ba7d66 [IMP] stock: convert yml test to python
bzr revid: mat@openerp.com-20130823115851-36tcrj17n5bme3uu
2013-08-23 13:58:51 +02:00