Merge lp:~openerp-dev/openobject-addons/7.0-opw-606423-han into lp:openobject-addons/7.0
Proposed by
Hardik Ansodariya (OpenERP)
Status: | Rejected |
---|---|
Rejected by: | Martin Trigaux (OpenERP) |
Proposed branch: | lp:~openerp-dev/openobject-addons/7.0-opw-606423-han |
Merge into: | lp:openobject-addons/7.0 |
Diff against target: |
22 lines (+3/-2) 1 file modified
stock/product.py (+3/-2) |
To merge this branch: | bzr merge lp:~openerp-dev/openobject-addons/7.0-opw-606423-han |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Naresh(OpenERP) | Pending | ||
Review via email: mp+215086@code.launchpad.net |
Description of the change
The loop in do_change_
Create a product, average stock valuation and real time accounting.
Let's assume 2 internal locations - Output and Stock
Output has quantity of zero
Stock has quantity of 10.
Go to product maintenance and click "- update" to update the cost.
It creates journal entries with # of internal location even if quantity is 0.
To post a comment you must log in.
Unmerged revisions
- 9912. By Hardik Ansodariya (OpenERP)
-
[FIXED] stock/product.py: Fixed the issue of duplicate journal entries with # of location when updating cost price of product(Maintenace: 606423)
Hello,
Thanks for your fix but actually I prefer to avoid recreating a full browse record (and refetching unchanged values in the database).
Instead I have fixed it using a read to the qty_available field only. This allows to keep a global browse in the for loop.
It was merged in 7.0 at revision 10023.