openerp-dev-web team mailing list archive

["Jiten \(OpenERP\)" <jra@xxxxxxxxxxx>]

Jiten (OpenERP) has proposed merging lp:~openerp-dev/openobject-client-web/trunk-bug-723006-jra into lp:openobject-client-web.

Requested reviews:
  OpenERP SA's Web Client R&D (openerp-dev-web)
Related bugs:
  Bug #723006 in OpenERP Web Client: "can't do purchase order in simplified interface"
  https://bugs.launchpad.net/openobject-client-web/+bug/723006

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-client-web/trunk-bug-723006-jra/+merge/54192

Hidden required field should not be disabled.
-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/trunk-bug-723006-jra/+merge/54192
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-client-web/trunk-bug-723006-jra.

=== modified file 'addons/openerp/static/javascript/form.js'
--- addons/openerp/static/javascript/form.js	2011-03-18 10:58:29 +0000
+++ addons/openerp/static/javascript/form.js	2011-03-21 11:54:58 +0000
@@ -135,21 +135,6 @@
     }
 
 
-	var inactive_notebook_length = jQuery('.notebook-tabs-strip li:hidden', '.notebook:visible').length;
-	if(inactive_notebook_length) {
-		var inactive_notebooks = [];
-		jQuery('.notebook-tabs-strip li:hidden', '.notebook:visible').each(function() {
-			var inactive_notebook_index = jQuery('.notebook-tabs-strip li', '.notebook:visible').index(jQuery(this));
-			inactive_notebooks.push(jQuery('div.notebook-pages div.notebook-page').get(inactive_notebook_index))
-		});
-
-		jQuery(inactive_notebooks).each(function(){
-			jQuery('.requiredfield', this).each(function() {
-				jQuery(this).attr('disabled', 'disabled')
-			})
-		});
-	}
-
     var elements = MochiKit.Base.filter(function(el){
         return !el.disabled && el.id && el.name && el.id.indexOf('_terp_listfields/') == -1 && hasElementClass(el, 'requiredfield');
     }, form.elements);