rewrote js for revision checkboxes to work in xhtml

1 files changed, 18 insertions, 16 deletions

diff --git a/lib/scripts/behaviour.js b/lib/scripts/behaviour.js
index b05949a90..18308d68a 100644
--- a/lib/scripts/behaviour.js
+++ b/lib/scripts/behaviour.js
@@ -168,26 +168,28 @@ var dw_behaviour = {
      * disable multiple revisions checkboxes if two are checked
      *
      * @author Andreas Gohr <andi@splitbrain.org>
+     * @author Anika Henke <anika@selfthinker.org>
      */
-    revisionBoxHandler: function(){
-        var $checked = jQuery('#page__revisions input[type=checkbox]:checked');
-        var $all     = jQuery('#page__revisions input[type=checkbox]');
-
-        if($checked.length < 2){
-            $all.attr('disabled',false);
-            jQuery('#page__revisions button').attr('disabled',true);
-        }else{
-            $all.attr('disabled',true);
-            jQuery('#page__revisions button').attr('disabled',false);
-            for(var i=0; i<$checked.length; i++){
-                $checked[i].disabled = false;
-                if(i>1){
-                    $checked[i].checked = false;
+    revisionBoxHandler: function() {
+        var $revisions = jQuery('#page__revisions');
+        var $all       = jQuery('input[type=checkbox]', $revisions);
+        var $checked   = $all.filter(':checked');
+        var $button    = jQuery('button', $revisions);
+
+        if($checked.length < 2) {
+            $all.removeAttr('disabled');
+            $button.attr('disabled', true);
+        } else {
+            $all.attr('disabled', true);
+            $button.removeAttr('disabled');
+            $checked.each(function(i) {
+                jQuery(this).removeAttr('disabled');
+                if(i>1) {
+                    jQuery(this).attr('checked', false);
                 }
-            }
+            });
         }
     }
-
 };
 
 jQuery(dw_behaviour.init);