Merge "Convert last usages of wfMsg*() to wfMessage()."
authorSiebrand <siebrand@wikimedia.org>
Thu, 13 Sep 2012 01:49:41 +0000 (01:49 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Thu, 13 Sep 2012 01:49:41 +0000 (01:49 +0000)
1  2 
tests/jasmine/spec_makers/makeJqueryMsgSpec.php

@@@ -8,13 -8,13 +8,13 @@@
   * which can be used with the JasmineBDD framework. This specification can then be used by simply including it into
   * the SpecRunner.html file.
   *
 - * This is similar to Michael Dale (mdale@mediawiki.org)'s parser tests, except that it doesn't look up the 
 + * This is similar to Michael Dale (mdale@mediawiki.org)'s parser tests, except that it doesn't look up the
   * API results while doing the test, so the Jasmine run is much faster(at the cost of being out of date in rare
   * circumstances. But mostly the parsing that we are doing in Javascript doesn't change much.)
   *
 - */ 
 + */
  
 -$maintenanceDir = dirname( dirname( dirname( dirname( __DIR__ ) ) ) ) . '/maintenance';
 +$maintenanceDir = dirname( dirname( dirname( __DIR__ ) ) ) . '/maintenance';
  
  require( "$maintenanceDir/Maintenance.php" );
  
@@@ -51,27 -51,22 +51,22 @@@ class MakeLanguageSpec extends Maintena
        private function getMessagesAndTests() {
                $messages = array();
                $tests = array();
-               $wfMsgExtOptions = array( 'parsemag' );
                foreach ( array( 'en', 'fr', 'ar', 'jp', 'zh' ) as $languageCode ) {
-                       $wfMsgExtOptions['language'] = $languageCode;
                        foreach ( self::$keyToTestArgs as $key => $testArgs ) {
                                foreach ($testArgs as $args) {
                                        // get the raw template, without any transformations
                                        $template = wfMessage( $key )->inLanguage( $languageCode )->plain();
  
-                                       // get the magic-parsed version with args
-                                       $wfMsgExtArgs = array_merge( array( $key, $wfMsgExtOptions ), $args );
-                                       // @todo FIXME: Use Message class.
-                                       $result = call_user_func_array( 'wfMsgExt', $wfMsgExtArgs ); 
+                                       $result = wfMessage( $key, $args )->inLanguage( $languageCode )->text();
  
                                        // record the template, args, language, and expected result
-                                       // fake multiple languages by flattening them together  
+                                       // fake multiple languages by flattening them together
                                        $langKey = $languageCode . '_' . $key;
                                        $messages[ $langKey ] = $template;
-                                       $tests[] = array( 
+                                       $tests[] = array(
                                                'name' => $languageCode . " " . $key . " " . join( ",", $args ),
                                                'key' => $langKey,
-                                               'args' => $args, 
+                                               'args' => $args,
                                                'result' => $result,
                                                'lang' => $languageCode
                                        );