← Back to team overview

openerp-dev-web team mailing list archive

[Merge] lp:~openerp-dev/openobject-client-web/6.0-bug-744311-kch into lp:openobject-client-web/6.0

 

Kunal Chavda (OpenERP) has proposed merging lp:~openerp-dev/openobject-client-web/6.0-bug-744311-kch into lp:openobject-client-web/6.0.

Requested reviews:
  OpenERP Core Team (openerp)
Related bugs:
  Bug #744311 in OpenERP Web Client: "Default value doesn't work in Cash Registers form"
  https://bugs.launchpad.net/openobject-client-web/+bug/744311

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-bug-744311-kch/+merge/55700

Hello,

When we set default value from context menu in editable list view it create problem.
Can't set the default value.
The problem is model was going wrong due to prefix setting.
So I have fixed the prefix, so now set the proper model name.

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-bug-744311-kch/+merge/55700
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-client-web/6.0-bug-744311-kch.
=== modified file 'addons/openerp/static/javascript/form.js'
--- addons/openerp/static/javascript/form.js	2011-03-30 09:50:53 +0000
+++ addons/openerp/static/javascript/form.js	2011-03-31 07:42:34 +0000
@@ -803,10 +803,10 @@
 
     var prefix = id.indexOf('/') > -1 ? id.slice(0, id.lastIndexOf('/')) + '/' : '';
     if ((prefix.split('/')[0])== '_terp_listfields') {
-        prefix = ''
+        prefix = (prefix.split('/')[1]);
     }
 
-    var model = prefix ? openobject.dom.get(prefix + '_terp_model').value : openobject.dom.get('_terp_model').value;
+    var model = prefix ? openobject.dom.get(prefix + '/_terp_model').value : openobject.dom.get('_terp_model').value;
 
     openobject.http.postJSON(act, {
         'model': model,


Follow ups