Merge lp:~openerp-dev/openobject-client-web/6.0-opw-5796-sma into lp:openobject-client-web
Proposed by
Sananaz (Open ERP)
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 4630 | ||||
Proposed branch: | lp:~openerp-dev/openobject-client-web/6.0-opw-5796-sma | ||||
Merge into: | lp:openobject-client-web | ||||
Diff against target: |
13 lines (+2/-2) 1 file modified
addons/openerp/static/javascript/form.js (+2/-2) |
||||
To merge this branch: | bzr merge lp:~openerp-dev/openobject-client-web/6.0-opw-5796-sma | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Navrang Oza (community) | Needs Resubmitting | ||
Xavier (Open ERP) (community) | Needs Fixing | ||
Review via email: mp+61388@code.launchpad.net |
Description of the change
Hello,
To generate problem:
- open a sale order line in a sale order
- you can't open the m2o fields (product_id or product_uom)
If can't generate problem please clear your browser cache.
Thank you.
To post a comment you must log in.
The '/' prefix was added at the same time the conditional above was modified to add a 'split' in it.
I'm pretty sure if we go through conditional, things are going to break because there will not be a final '/' and so we'll get `somefield_ terp_model` instead of `somefield/ _terp_model` .
I think we need to leave it before `_terp_model` but remove adding of trailing '/' in original creation of `prefix`. What do you think?