Merge lp:~camptocamp/sale-wkfl/product_procurement_type into lp:~sale-core-editors/sale-wkfl/7.0
Status: | Rejected |
---|---|
Rejected by: | Leonardo Pistone |
Proposed branch: | lp:~camptocamp/sale-wkfl/product_procurement_type |
Merge into: | lp:~sale-core-editors/sale-wkfl/7.0 |
Diff against target: |
617 lines (+563/-0) 10 files modified
product_procurement_type/__init__.py (+22/-0) product_procurement_type/__openerp__.py (+52/-0) product_procurement_type/product.py (+139/-0) product_procurement_type/product_view.xml (+42/-0) product_procurement_type/test/test_onchange_procurement_type.yml (+56/-0) product_procurement_type_dropshipping/__init__.py (+22/-0) product_procurement_type_dropshipping/__openerp__.py (+46/-0) product_procurement_type_dropshipping/product.py (+115/-0) product_procurement_type_dropshipping/product_view.xml (+17/-0) product_procurement_type_dropshipping/test/test_onchange_procurement_type.yml (+52/-0) |
To merge this branch: | bzr merge lp:~camptocamp/sale-wkfl/product_procurement_type |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Pedro Manuel Baeza | Needs Resubmitting | ||
Lorenzo Battistini (community) | code review | Needs Fixing | |
Yannick Vaucher @ Camptocamp | code review, no tests | Pending | |
Leonardo Pistone | code review | Pending | |
Stefan Rijnhart (Opener) | Pending | ||
Alexandre Fayolle - camptocamp | code review, test | Pending | |
Joël Grand-Guillaume @ camptocamp | code review + no tests | Pending | |
Review via email: mp+222111@code.launchpad.net |
This proposal supersedes a proposal from 2014-03-03.
Description of the change
Here are two new addons who brings a simple field to manage several combinations
for two procurements fields : procure_method & supply_method :
First addon adds:
- Normal: MTS + buy
- Bill of Materials: MTO + produce
Second addon depends of sale_dropshipping addon and adds:
- Dropshipping: MTO + buy
I'm not sure if it's the right community branch to propose it, if you think about a better
place, let me know.
Unmerged revisions
- 48. By Romain Deheele - Camptocamp
-
[UPD] update manifest description
- 47. By Romain Deheele - Camptocamp
-
replace ValueError exception with False value that will be refused because a value is required : avoid ugly traceback
- 46. By Romain Deheele - Camptocamp
-
[UPD] update manifest according to selection values
- 45. By Alexandre Fayolle - camptocamp
-
the migration scripts are not longer called on installation -> move to init method
- 44. By Alexandre Fayolle - camptocamp
-
[IMP] remove unnecessary code pointed out by lpistone
- 43. By Alexandre Fayolle - camptocamp
-
[FIX] issue when importing products from CSV with procurement_type attribute
There is an issue in the framework (I think) which will break the previous
version of the code in CSV imports: the get_product_procurement_ type method (in
product_template) is called with an instance of product_product for self, and
the call to super() crashed. This redesign works around the issue by extracting
get_product_procurement_ type to a function and thus avoiding self altogether. - 42. By Alexandre Fayolle - camptocamp
-
[FIX] tests
- 41. By Alexandre Fayolle - camptocamp
-
[ADD] migration script to ensure that the new attribute is populated
- 40. By Alexandre Fayolle - camptocamp
-
[FIX] values of procurement_type in dispatch method + help string + module version number
- 39. By Alexandre Fayolle - camptocamp
-
[IMP] follow changes in product_
procurement_ type_dropshippi ng
LGTM for the code
However, I don't think we have a proper branch to extend procurement.
Shall we create one?