You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@myfaces.apache.org by ma...@apache.org on 2011/01/26 10:55:01 UTC

svn commit: r1063662 - /myfaces/trinidad/branches/trinidad-1.2.x/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/TableProxy.js

Author: matzew
Date: Wed Jan 26 09:55:00 2011
New Revision: 1063662

URL: http://svn.apache.org/viewvc?rev=1063662&view=rev
Log:
TRINIDAD-2009 - tr:table selectAll also selects disabled chekcboxes

Thanks to Markus Dreher for the patch

Modified:
    myfaces/trinidad/branches/trinidad-1.2.x/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/TableProxy.js

Modified: myfaces/trinidad/branches/trinidad-1.2.x/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/TableProxy.js
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/trinidad-1.2.x/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/TableProxy.js?rev=1063662&r1=1063661&r2=1063662&view=diff
==============================================================================
--- myfaces/trinidad/branches/trinidad-1.2.x/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/TableProxy.js (original)
+++ myfaces/trinidad/branches/trinidad-1.2.x/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/TableProxy.js Wed Jan 26 09:55:00 2011
@@ -153,7 +153,10 @@ CollectionComponent.defineMultiSelect = 
         for(var i=0; i<boxes.length; i++)
         {
             var box = boxes[i];
-            box.checked = selectAll;
+            if(!box.disabled)
+            {
+                box.checked = selectAll;
+            }
         }
         var selectedMode = this.getFormElement(this._selectedModeKey);
         if (selectAll)