Merge lp:~numerigraphe-team/ocb-addons/7.0-revert-sale-properties-1291903-ls into lp:ocb-addons
Proposed by
Lionel Sausin - Initiatives/Numérigraphe
Status: | Merged |
---|---|
Approved by: | Holger Brunn (Therp) |
Approved revision: | no longer in the source branch. |
Merged at revision: | 10084 |
Proposed branch: | lp:~numerigraphe-team/ocb-addons/7.0-revert-sale-properties-1291903-ls |
Merge into: | lp:ocb-addons |
Diff against target: |
11 lines (+0/-1) 1 file modified
sale_stock/sale_stock.py (+0/-1) |
To merge this branch: | bzr merge lp:~numerigraphe-team/ocb-addons/7.0-revert-sale-properties-1291903-ls |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Holger Brunn (Therp) | code review | Approve | |
Pedro Manuel Baeza | code review | Approve | |
Stefan Rijnhart (Opener) | test, code review | Approve | |
Review via email: mp+214435@code.launchpad.net |
Description of the change
This reverts the fix for bug #1083077. It was fixed in OCB in the wrong module, and eventually fixed the right way in 7.0.
Incidentally this is the last change needed to make OCB run green on runbot : http://
(Resubmitting because LP seems to have stalled the first time)
To post a comment you must log in.
Great, thanks for picking this up!
Quite a history this one has in upstream OpenERP: bazaar. launchpad. net/~openerp/ openobject- addons/ 7.0/revision/ 9172, bazaar. launchpad. net/~openerp/ openobject- addons/ 7.0/revision/ 9175, presumably because it breaks tests like it backport did in OCB. bazaar. launchpad. net/~openerp/ openobject- addons/ 7.0/revision/ 9176 bazaar. launchpad. net/~openerp/ openobject- addons/ 7.0/revision/ 9378 reintroduces the original fix bazaar. launchpad. net/~openerp/ openobject- addons/ 7.0/revision/ 9622 removes the original fix again because it breaks tests bazaar. launchpad. net/~openerp/ openobject- addons/ 7.0/revision/ 9741 moves the fix to the correct method override on sale.order model but in a module that does not depend on sale bazaar. launchpad. net/~openerp/ openobject- addons/ 7.0/revision/ 9751 correctly moves the method override to sale_mrp
- Initial fix committed as http://
- reverted in http://
- The next commit moves the fix to a method override on the wrong model: http://
- http://
- http://
- http://
- http://
For my review, I verified that OCB still propagates the properties with this change.