IE compatibility fix (thanks, howcheng): don't pass a null second param to insertBefore()

diff --git a/includes/DefaultSettings.php b/includes/DefaultSettings.php
index 20c3915..2f473ee 100644 (file)
--- a/includes/DefaultSettings.php
+++ b/includes/DefaultSettings.php
@@ -1105,7 +1105,7 @@ $wgCacheEpoch = '20030516000000';
  * to ensure that client-side caches don't keep obsolete copies of global
  * styles.
  */
-$wgStyleVersion = '49';
+$wgStyleVersion = '50';
 
 
 # Server-side caching:

diff --git a/skins/common/wikibits.js b/skins/common/wikibits.js
index c6c8fc7..6af4514 100644 (file)
--- a/skins/common/wikibits.js
+++ b/skins/common/wikibits.js
@@ -576,8 +576,10 @@ function addPortletLink(portlet, href, text, id, tooltip, accesskey, nextnode) {
                updateTooltipAccessKeys( new Array( link ) );
        }
 
-       if ( nextnode && nextnode.parentNode != node ) nextnode = null;
-       node.insertBefore( item, nextnode );
+       if ( nextnode && nextnode.parentNode == node )
+               node.insertBefore( item, nextnode );
+       else
+               node.appendChild( item );  // IE compatibility (?)
 
        return item;
 }