Add missing @since tags for getLanguage and deal with this sanitizeLangCode fatal...
authorDaniel Friesen <dantman@users.mediawiki.org>
Mon, 5 Dec 2011 18:56:09 +0000 (18:56 +0000)
committerDaniel Friesen <dantman@users.mediawiki.org>
Mon, 5 Dec 2011 18:56:09 +0000 (18:56 +0000)
includes/context/ContextSource.php
includes/context/DerivativeContext.php
includes/context/IContextSource.php
includes/context/RequestContext.php

index 0eb5d3e..5fc8eba 100644 (file)
@@ -107,6 +107,7 @@ abstract class ContextSource implements IContextSource {
         * Get the Language object
         *
         * @return Language
+        * @since 1.19
         */
        public function getLanguage() {
                return $this->getContext()->getLanguage();
index e440575..cff827c 100644 (file)
@@ -169,12 +169,13 @@ class DerivativeContext extends ContextSource {
         * Set the Language object
         *
         * @param $l Mixed Language instance or language code
+        * @since 1.19
         */
        public function setLanguage( $l ) {
                if ( $l instanceof Language ) {
                        $this->lang = $l;
                } elseif ( is_string( $l ) ) {
-                       $l = self::sanitizeLangCode( $l ); // FIXME: Undefined method, is at RequestContext::sanitizeLangCode()
+                       $l = RequestContext::sanitizeLangCode( $l );
                        $obj = Language::factory( $l );
                        $this->lang = $obj;
                } else {
@@ -194,6 +195,7 @@ class DerivativeContext extends ContextSource {
         * Get the Language object
         *
         * @return Language
+        * @since 1.19
         */
        public function getLanguage() {
                if ( !is_null( $this->lang ) ) {
index 4a8e93b..446cb3f 100644 (file)
@@ -68,6 +68,7 @@ interface IContextSource {
         * Get the Language object
         *
         * @return Language
+        * @since 1.19
         */
        public function getLanguage();
 
index 7ca184a..b2568e4 100644 (file)
@@ -146,8 +146,9 @@ class RequestContext implements IContextSource {
        /**
         * Accepts a language code and ensures it's sane. Outputs a cleaned up language
         * code and replaces with $wgLanguageCode if not sane.
+        * @private
         */
-       private static function sanitizeLangCode( $code ) {
+       static function sanitizeLangCode( $code ) {
                global $wgLanguageCode;
 
                // BCP 47 - letter case MUST NOT carry meaning
@@ -176,6 +177,7 @@ class RequestContext implements IContextSource {
         * Set the Language object
         *
         * @param $l Mixed Language instance or language code
+        * @since 1.19
         */
        public function setLanguage( $l ) {
                if ( $l instanceof Language ) {
@@ -201,6 +203,7 @@ class RequestContext implements IContextSource {
         * Get the Language object
         *
         * @return Language
+        * @since 1.19
         */
        public function getLanguage() {
                if ( $this->lang === null ) {