Followup r95753 per CR: prevent extensions from making isMovable() return true for...

[lhc/web/wiklou.git] / includes / resourceloader / ResourceLoaderUserOptionsModule.php

diff --git a/includes/resourceloader/ResourceLoaderUserOptionsModule.php b/includes/resourceloader/ResourceLoaderUserOptionsModule.php
index b1e80d5..1989353 100644 (file)
--- a/includes/resourceloader/ResourceLoaderUserOptionsModule.php
+++ b/includes/resourceloader/ResourceLoaderUserOptionsModule.php
@@ -33,6 +33,10 @@ class ResourceLoaderUserOptionsModule extends ResourceLoaderModule {
 
        /* Methods */
 
+       /**
+        * @param $context ResourceLoaderContext
+        * @return array|int|Mixed
+        */
        public function getModifiedTime( ResourceLoaderContext $context ) {
                $hash = $context->getHash();
                if ( isset( $this->modifiedTime[$hash] ) ) {
@@ -66,12 +70,22 @@ class ResourceLoaderUserOptionsModule extends ResourceLoaderModule {
                }
        }
 
+       /**
+        * @param $context ResourceLoaderContext
+        * @return string
+        */
        public function getScript( ResourceLoaderContext $context ) {
                return Xml::encodeJsCall( 'mw.user.options.set', 
                        array( $this->contextUserOptions( $context ) ) );
        }
 
+       /**
+        * @param $context ResourceLoaderContext
+        * @return array
+        */
        public function getStyles( ResourceLoaderContext $context ) {
+               // FIXME: This stuff should really be in its own module, because it gets double-loaded otherwise
+               // (once through a <link>, once when embedded as JS)
                global $wgAllowUserCssPrefs;
 
                if ( $wgAllowUserCssPrefs ) {
@@ -79,6 +93,8 @@ class ResourceLoaderUserOptionsModule extends ResourceLoaderModule {
 
                        // Build CSS rules
                        $rules = array();
+
+                       // Underline: 2 = browser default, 1 = always, 0 = never
                        if ( $options['underline'] < 2 ) {
                                $rules[] = "a { text-decoration: " . 
                                        ( $options['underline'] ? 'underline' : 'none' ) . "; }";
@@ -111,7 +127,14 @@ class ResourceLoaderUserOptionsModule extends ResourceLoaderModule {
                return array();
        }
 
+       /**
+        * @return string
+        */
        public function getGroup() {
                return 'private';
        }
+       
+       public function getDependencies() {
+               return array( 'mediawiki.user' );
+       }
 }