Author: jleroux Date: Wed Jul 24 11:20:45 2013 New Revision: 1506499 URL: http://svn.apache.org/r1506499 Log: Completes "Wrong to use !jQuery('#content-messages') in \framework\images\webapp\images\selectall.js line #354" https://issues.apache.org/jira/browse/OFBIZ-5279
Using submitted patch as a basis. Modified: ofbiz/trunk/framework/images/webapp/images/selectall.js Modified: ofbiz/trunk/framework/images/webapp/images/selectall.js URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/images/webapp/images/selectall.js?rev=1506499&r1=1506498&r2=1506499&view=diff ============================================================================== --- ofbiz/trunk/framework/images/webapp/images/selectall.js (original) +++ ofbiz/trunk/framework/images/webapp/images/selectall.js Wed Jul 24 11:20:45 2013 @@ -347,6 +347,7 @@ function submitFormInBackground(form, ar function ajaxSubmitFormUpdateAreas(form, areaCsvString) { waitSpinnerShow(); hideErrorContainer = function() { + Query('#content-messages').html(''); jQuery('#content-messages').removeClass('errorMessage').fadeIn('fast'); } updateFunction = function(data) { @@ -367,7 +368,8 @@ function ajaxSubmitFormUpdateAreas(form, } jQuery('#content-messages').fadeIn('fast'); }else { - if(jQuery('#content-messages')) { + if (jQuery('#content-messages').length == 0) { + jQuery('#content-messages').html(''); jQuery('#content-messages').removeClass('errorMessage').fadeIn("fast"); } ajaxUpdateAreas(areaCsvString);