(bug 24073) Recognize MS Office 2003 style files that have been saved by MS 2007.
[lhc/web/wiklou.git] / includes / StubObject.php
index 0cd453b..c93e3fc 100644 (file)
@@ -2,17 +2,17 @@
 
 /**
  * Class to implement stub globals, which are globals that delay loading the
- * their associated module code by deferring initialisation until the first 
- * method call. 
+ * their associated module code by deferring initialisation until the first
+ * method call.
  *
- * Note on unstub loops: 
+ * Note on unstub loops:
  *
- * Unstub loops (infinite recursion) sometimes occur when a constructor calls 
- * another function, and the other function calls some method of the stub. The 
+ * Unstub loops (infinite recursion) sometimes occur when a constructor calls
+ * another function, and the other function calls some method of the stub. The
  * best way to avoid this is to make constructors as lightweight as possible,
- * deferring any initialisation which depends on other modules. As a last 
- * resort, you can use StubObject::isRealObject() to break the loop, but as a 
- * general rule, the stub object mechanism should be transparent, and code 
+ * deferring any initialisation which depends on other modules. As a last
+ * resort, you can use StubObject::isRealObject() to break the loop, but as a
+ * general rule, the stub object mechanism should be transparent, and code
  * which refers to it should be kept to a minimum.
  */
 class StubObject {
@@ -21,10 +21,10 @@ class StubObject {
        /**
         * Constructor.
         *
-        * @param String $global name of the global variable.
-        * @param String $class name of the class of the real object.
-        * @param Array $param array of parameters to pass to contructor of the real
-        *                     object.
+        * @param $global String: name of the global variable.
+        * @param $class String: name of the class of the real object.
+        * @param $params Array: parameters to pass to contructor of the real
+        *               object.
         */
        function __construct( $global = null, $class = null, $params = array() ) {
                $this->mGlobal = $global;
@@ -36,8 +36,8 @@ class StubObject {
         * Returns a bool value whetever $obj is a stub object. Can be used to break
         * a infinite loop when unstubbing an object.
         *
-        * @param Object $obj object to check.
-        * @return bool true if $obj is not an instance of StubObject class.
+        * @param $obj Object to check.
+        * @return Boolean: true if $obj is not an instance of StubObject class.
         */
        static function isRealObject( $obj ) {
                return is_object( $obj ) && !($obj instanceof StubObject);
@@ -50,8 +50,8 @@ class StubObject {
         * This function will also call the function with the same name in the real
         * object.
         *
-        * @param String $name name of the function called.
-        * @param Array $args array of arguments.
+        * @param $name String: name of the function called
+        * @param $args Array: arguments
         */
        function _call( $name, $args ) {
                $this->_unstub( $name, 5 );
@@ -69,8 +69,8 @@ class StubObject {
         * Function called by PHP if no function with that name exists in this
         * object.
         *
-        * @param String $name name of the function called
-        * @param Array $args array of arguments
+        * @param $name String: name of the function called
+        * @param $args Array: arguments
         */
        function __call( $name, $args ) {
                return $this->_call( $name, $args );
@@ -79,15 +79,19 @@ class StubObject {
        /**
         * This function creates a new object of the real class and replace it in
         * the global variable.
-        * This is public, for the convenience of external callers wishing to access 
+        * This is public, for the convenience of external callers wishing to access
         * properties, e.g. eval.php
         *
-        * @param String $name name of the method called in this object.
-        * @param Integer $level level to go in the stact trace to get the function
-        *                       who called this function.
+        * @param $name String: name of the method called in this object.
+        * @param $level Integer: level to go in the stact trace to get the function
+        *               who called this function.
         */
        function _unstub( $name = '_unstub', $level = 2 ) {
                static $recursionLevel = 0;
+
+               if ( !($GLOBALS[$this->mGlobal] instanceof StubObject) )
+                       return $GLOBALS[$this->mGlobal]; // already unstubbed.
+               
                if ( get_class( $GLOBALS[$this->mGlobal] ) != $this->mClass ) {
                        $fname = __METHOD__.'-'.$this->mGlobal;
                        wfProfileIn( $fname );
@@ -95,8 +99,8 @@ class StubObject {
                        if ( ++$recursionLevel > 2 ) {
                                throw new MWException( "Unstub loop detected on call of \${$this->mGlobal}->$name from $caller\n" );
                        }
-                       wfDebug( "Unstubbing \${$this->mGlobal} on call of \${$this->mGlobal}->$name from $caller\n" );
-                       $GLOBALS[$this->mGlobal] = $this->_newObject();
+                       wfDebug( "Unstubbing \${$this->mGlobal} on call of \${$this->mGlobal}::$name from $caller\n" );
+                       $obj = $GLOBALS[$this->mGlobal] = $this->_newObject();
                        --$recursionLevel;
                        wfProfileOut( $fname );
                }
@@ -144,17 +148,11 @@ class StubUserLang extends StubObject {
        function _newObject() {
                global $wgContLanguageCode, $wgRequest, $wgUser, $wgContLang;
                $code = $wgRequest->getVal( 'uselang', $wgUser->getOption( 'language' ) );
-
-               // if variant is explicitely selected, use it instead the one from wgUser
-               // see bug #7605
-               if( $wgContLang->hasVariants() ){
-                       $variant = $wgContLang->getPreferredVariant();
-                       if( $variant != $wgContLanguageCode )
-                               $code = $variant;
-               }        
+               // BCP 47 - letter case MUST NOT carry meaning
+               $code = strtolower( $code );
 
                # Validate $code
-               if( empty( $code ) || !preg_match( '/^[a-z-]+$/', $code ) ) {
+               if( empty( $code ) || !preg_match( '/^[a-z-]+$/', $code ) || ( $code === 'qqq' ) ) {
                        wfDebug( "Invalid user language code\n" );
                        $code = $wgContLanguageCode;
                }
@@ -183,14 +181,13 @@ class StubUser extends StubObject {
        function __call( $name, $args ) {
                return $this->_call( $name, $args );
        }
-       
+
        function _newObject() {
                global $wgCommandLineMode;
                if( $wgCommandLineMode ) {
                        $user = new User;
                } else {
                        $user = User::newFromSession();
-                       wfRunHooks( 'AutoAuthenticate', array( &$user ) );
                }
                return $user;
        }