Merge lp:~camptocamp/stock-logistic-barcode/7.0-fix_1330459-sge+afe into lp:stock-logistic-barcode
Proposed by
Alexandre Fayolle - camptocamp
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 66 | ||||
Proposed branch: | lp:~camptocamp/stock-logistic-barcode/7.0-fix_1330459-sge+afe | ||||
Merge into: | lp:stock-logistic-barcode | ||||
Diff against target: |
141 lines (+39/-12) 6 files modified
tr_barcode/__openerp__.py (+1/-1) tr_barcode/migrations/1.1.4/pre-migrate.py (+24/-0) tr_barcode/tr_barcode.py (+9/-6) tr_barcode/tr_barcode_view.xml (+1/-1) tr_barcode/wizard/tr_barcode_wizard.py (+2/-2) tr_barcode_config/barcode/barcode_osv.py (+2/-2) |
||||
To merge this branch: | bzr merge lp:~camptocamp/stock-logistic-barcode/7.0-fix_1330459-sge+afe | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Yannick Vaucher @ Camptocamp | code review no tests | Approve | |
Pedro Manuel Baeza | code review | Approve | |
Guewen Baconnier @ Camptocamp | code review | Approve | |
Sébastien Gendre - Open-Net | Pending | ||
Review via email: mp+223866@code.launchpad.net |
Description of the change
Rename hight to height
Integrate and enhance Sébastien Gendre's fix, add a migration script.
To post a comment you must log in.
When you want to rename a field, you can also use the 'oldname' argument, example in crm:
'opt_out': fields. boolean( 'Opt-Out' , oldname='optout',
help= "If opt-out is checked, this contact has refused to receive emails for mass mailing and marketing campaign. "
" Filter 'Available for Mass Mailing' allows users to filter the leads when performing mass mailing."),
The ORM would automatically rename the field.
Your version is also a valid way to rename the field so LGTM.