Merge lp:~openerp-dev/openobject-client-web/trunk-bug-733089-jra into lp:openobject-client-web/trunk

Proposed by Jiten (OpenERP)
Status: Merged
Approved by: Xavier (Open ERP)
Approved revision: 4635
Merged at revision: 4635
Proposed branch: lp:~openerp-dev/openobject-client-web/trunk-bug-733089-jra
Merge into: lp:openobject-client-web/trunk
Diff against target: 35 lines (+9/-6)
2 files modified
addons/openerp/controllers/openo2m.py (+4/-0)
addons/openerp/static/javascript/o2m.js (+5/-6)
To merge this branch: bzr merge lp:~openerp-dev/openobject-client-web/trunk-bug-733089-jra
Reviewer Review Type Date Requested Status
OpenERP R&D Web Team Pending
Review via email: mp+55726@code.launchpad.net

Description of the change

In the case of nested o2m, Button "New" for next o2m does not open by first click.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'addons/openerp/controllers/openo2m.py'
--- addons/openerp/controllers/openo2m.py 2011-02-10 16:09:23 +0000
+++ addons/openerp/controllers/openo2m.py 2011-03-31 10:48:32 +0000
@@ -49,6 +49,10 @@
4949
50 form = tw.form_view.ViewForm(vp, name="view_form", action="/openerp/openo2m/save")50 form = tw.form_view.ViewForm(vp, name="view_form", action="/openerp/openo2m/save")
51 cherrypy.request.terp_validators = {}51 cherrypy.request.terp_validators = {}
52
53 if '/' in params.o2m:
54 params.o2m = '.'.join(params.o2m.split('/')[-1:])
55
52 wid = form.screen.widget.get_widgets_by_name(params.o2m)[0]56 wid = form.screen.widget.get_widgets_by_name(params.o2m)[0]
5357
54 # save view_params for later phazes58 # save view_params for later phazes
5559
=== modified file 'addons/openerp/static/javascript/o2m.js'
--- addons/openerp/static/javascript/o2m.js 2011-01-18 20:50:22 +0000
+++ addons/openerp/static/javascript/o2m.js 2011-03-31 10:48:32 +0000
@@ -125,12 +125,11 @@
125 var parents = [];125 var parents = [];
126 // get the required view params to get proper view126 // get the required view params to get proper view
127 var params = {127 var params = {
128 '_terp_view_params/_terp_model': openobject.dom.get('_terp_model').value,128 '_terp_view_params/_terp_model': this.parent_model,
129 '_terp_view_params/_terp_id': openobject.dom.get('_terp_id').value,129 '_terp_view_params/_terp_id': this.parent_id,
130 '_terp_view_params/_terp_ids': openobject.dom.get('_terp_ids').value,130 '_terp_view_params/_terp_view_id':this.parent_view_id,
131 '_terp_view_params/_terp_view_ids': openobject.dom.get('_terp_view_ids').value,131 '_terp_view_params/_terp_view_mode': this.mode,
132 '_terp_view_params/_terp_view_mode': openobject.dom.get('_terp_view_mode').value,132 '_terp_view_params/_terp_context': this.parent_context || {},
133 '_terp_view_params/_terp_context': openobject.dom.get('_terp_context').value || {},
134 '_terp_view_params/_terp_view_type': 'form'133 '_terp_view_params/_terp_view_type': 'form'
135 };134 };
136135