From 86a7cd223bc47c173a5569d3600fce9356d7cb24 Mon Sep 17 00:00:00 2001 From: Aaron Schulz Date: Wed, 19 Sep 2007 14:10:38 +0000 Subject: [PATCH] *Revert bogus code from r24419. indexOf() is meant for Strings, not arrays. If used on such, it dies miserably in IE. --- skins/common/protect.js | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/skins/common/protect.js b/skins/common/protect.js index b3eec3bd59..90ff35636d 100644 --- a/skins/common/protect.js +++ b/skins/common/protect.js @@ -47,7 +47,7 @@ function allowCascade() { if( lists[i].selectedIndex > -1 ) { var items = lists[i].getElementsByTagName( 'option' ); var selected = items[ lists[i].selectedIndex ].value; - if( wgCascadeableLevels.indexOf( selected ) == -1 ) { + if( !isCascadeableLevel(selected) ) { document.getElementById( 'mwProtect-cascade' ).checked = false; document.getElementById( 'mwProtect-cascade' ).disabled = true; return false; @@ -58,6 +58,15 @@ function allowCascade() { return true; } +function isCascadeableLevel( level ) { + for (var k = 0; k < wgCascadeableLevels.length; k++) { + if ( wgCascadeableLevels[k] == level ) { + return true; + } + } + return false; +} + /** * When protection levels are locked together, update the rest * when one action's level changes -- 2.20.1