Preferences: Don't load all default options for every default preference
authorAlex Monk <krenair@gmail.com>
Wed, 12 Aug 2015 22:13:13 +0000 (23:13 +0100)
committerAlex Monk <krenair@gmail.com>
Wed, 12 Aug 2015 22:29:21 +0000 (23:29 +0100)
Change-Id: Iaaebea56674dfaa48dbfedf6c7e5ad9eaa05759e

includes/Preferences.php

index 9497ee7..cd675c6 100644 (file)
@@ -124,6 +124,7 @@ class Preferences {
 
                $disable = !$user->isAllowed( 'editmyoptions' );
 
+               $defaultOptions = User::getDefaultOptions();
                ## Prod in defaults from the user
                foreach ( $defaultPreferences as $name => &$info ) {
                        $prefFromUser = self::getOptionFromUser( $name, $info, $user );
@@ -131,7 +132,6 @@ class Preferences {
                                $info['disabled'] = 'disabled';
                        }
                        $field = HTMLForm::loadInputFromParameters( $name, $info, $dummyForm ); // For validation
-                       $defaultOptions = User::getDefaultOptions();
                        $globalDefault = isset( $defaultOptions[$name] )
                                ? $defaultOptions[$name]
                                : null;