svn commit: r1506274 - /ofbiz/trunk/framework/images/webapp/images/selectall.js

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r1506274 - /ofbiz/trunk/framework/images/webapp/images/selectall.js

jleroux@apache.org
Author: jleroux
Date: Tue Jul 23 20:50:04 2013
New Revision: 1506274

URL: http://svn.apache.org/r1506274
Log:
Fixes "Wrong to use !jQuery('#content-messages') in \framework\images\webapp\images\selectall.js line #354" https://issues.apache.org/jira/browse/OFBIZ-5279

Replaces
if(!jQuery('#content-messages'))
by
if(jQuery('#content-messages').length==0)
in framework\images\webapp\images\selectall.js line #354.

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=1506274&r1=1506273&r2=1506274&view=diff
==============================================================================
--- ofbiz/trunk/framework/images/webapp/images/selectall.js (original)
+++ ofbiz/trunk/framework/images/webapp/images/selectall.js Tue Jul 23 20:50:04 2013
@@ -351,7 +351,7 @@ function ajaxSubmitFormUpdateAreas(form,
    }
    updateFunction = function(data) {
        if (data._ERROR_MESSAGE_LIST_ != undefined || data._ERROR_MESSAGE_ != undefined) {
-           if(!jQuery('#content-messages')) {
+           if(jQuery('#content-messages').length == 0)
               //add this div just after app-navigation
               if(jQuery('#content-main-section')){
                   jQuery('#content-main-section' ).before('<div id="content-messages" onclick="hideErrorContainer()"></div>');