registration: Ignore short variables in convertExtensionToRegistration
authorGergő Tisza <gtisza@wikimedia.org>
Fri, 20 May 2016 14:27:56 +0000 (14:27 +0000)
committerUmherirrender <umherirrender_de.wp@web.de>
Fri, 20 May 2016 18:56:40 +0000 (18:56 +0000)
substr returns false when the string is shorter than the initial
position, which makes array_key_exists freak out.

Change-Id: I38c2dd3a1ae5d700d66ff590dda4be67e4dc5514

maintenance/convertExtensionToRegistration.php

index f9dd58c..aab2087 100644 (file)
@@ -94,6 +94,9 @@ class ConvertExtensionToRegistration extends Maintenance {
                $globalSettings = $this->getAllGlobals();
                foreach ( $vars as $name => $value ) {
                        $realName = substr( $name, 2 ); // Strip 'wg'
+                       if ( $realName === false ) {
+                               continue;
+                       }
 
                        // If it's an empty array that we likely set, skip it
                        if ( is_array( $value ) && count( $value ) === 0 && in_array( $realName, $__settings ) ) {