You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2013/07/24 13:20:46 UTC
svn commit: r1506499 -
/ofbiz/trunk/framework/images/webapp/images/selectall.js
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);