Chirag Patel (OpenERP) has proposed merging 
lp:~openerp-dev/openobject-client-web/6.0-opw-57667-cpa into 
lp:openobject-client-web.

Requested reviews:
  OpenERP Core Team (openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-57667-cpa/+merge/88029

Hello,

On view mode, the fields.boolean with an attrs for make the field readonly, if 
the domain on the attrs doesn't match with the object attributes, the checkbox 
is not readonly.

The given video link is for reproduce error.
http://blip.tv/file/get/Openerpsupport-attrs_view_mode_boolean489.ogv
or
http://blip.tv/openerp-support/attrs_view_mode_boolean-5875557

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-57667-cpa/+merge/88029
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-client-web/6.0-opw-57667-cpa.
=== modified file 'addons/openerp/static/javascript/form_state.js'
--- addons/openerp/static/javascript/form_state.js	2011-10-11 12:52:43 +0000
+++ addons/openerp/static/javascript/form_state.js	2012-01-10 10:41:48 +0000
@@ -164,7 +164,9 @@
     var result = form_evalExpr(prefix, expr, elem);
 
     switch (attr) {
-        case 'readonly': form_setReadonly(container, widget, result);
+        case 'readonly': 
+            if (!jQuery(".non-editable").length)
+                form_setReadonly(container, widget, result);
             break;
         case 'required': form_setRequired(container, widget, result);
             break;

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to