r107563 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r107562‎ | r107563 | r107564 >
Date:12:12, 29 December 2011
Author:santhosh
Status:ok
Tags:
Comment:
Change the 'Disable input method' preference to 'Enable input method'.
Simplify some interface messages.

i18n card #362
Modified paths:
  • /trunk/extensions/Narayam/Narayam.hooks.php (modified) (history)
  • /trunk/extensions/Narayam/Narayam.i18n.php (modified) (history)
  • /trunk/extensions/Narayam/Narayam.php (modified) (history)

Diff [purge]

Index: trunk/extensions/Narayam/Narayam.hooks.php
@@ -9,17 +9,13 @@
1010
1111 /// Hook: BeforePageDisplay
1212 public static function addModules( $out, $skin ) {
13 - if ( $out->getUser()->getOption( 'narayamDisable' ) ) {
14 - return true;
 13+ if ( $out->getUser()->getOption( 'narayamEnable' ) ) {
 14+ $schemes = array_values( self::getSchemes () );
 15+ if ( count( $schemes ) ) {
 16+ $out->addModules( $schemes );
 17+ $out->addModules( 'ext.narayam' );
 18+ }
1519 }
16 -
17 - $schemes = array_values( self::getSchemes () );
18 -
19 - if ( count( $schemes ) ) {
20 - $out->addModules( $schemes );
21 - $out->addModules( 'ext.narayam' );
22 - }
23 -
2420 return true;
2521 }
2622
@@ -36,10 +32,6 @@
3733 public static function addVariables( &$vars ) {
3834 global $wgUser, $wgNarayamSchemes, $wgNarayamUseBetaMapping;
3935
40 - if ( $wgUser->getOption( 'narayamDisable' ) ) {
41 - return true;
42 - }
43 -
4436 $vars['wgNarayamAvailableSchemes'] = self::getSchemes(); // Note: scheme names must be keys, not values
4537 $allSchemes = $wgNarayamSchemes;
4638 foreach ( $allSchemes as $lang => $schemes ) {
@@ -99,13 +91,24 @@
10092 /// Hook: GetPreferences
10193 public static function addPreference( $user, &$preferences ) {
10294 // A checkbox in preferences to disable Narayam
103 - $preferences['narayamDisable'] = array(
 95+ $preferences['narayamEnable'] = array(
10496 'type' => 'toggle',
105 - 'label-message' => 'narayam-disable-preference',
 97+ 'label-message' => 'narayam-enable-preference',
10698 'section' => 'editing/advancedediting', // under 'Advanced options' section of 'Editing' tab
 99+ 'default' => $user->getOption( 'narayamEnable' )
107100 );
108101
109102 return true;
110103 }
111 -
 104+ /**
 105+ * UserGetDefaultOptions hook handler.
 106+ * @param $defaultOptions array
 107+ * @return bool
 108+ */
 109+ public static function addDefaultOptions( &$defaultOptions ) {
 110+ global $wgNarayamEnabledByDefault;
 111+ // By default, the preference page option to enable Narayam is set to wgNarayamEnabledByDefault value.
 112+ $defaultOptions['narayamEnable'] = $wgNarayamEnabledByDefault;
 113+ return true;
 114+ }
112115 }
Index: trunk/extensions/Narayam/Narayam.i18n.php
@@ -17,11 +17,11 @@
1818 'narayam-desc' => 'Allows to add custom input methods for input fields',
1919 'narayam-toggle-ime' => 'Enable ($1)',
2020 'narayam-menu' => 'Input Method',
21 - 'narayam-menu-tooltip' => 'Control Narayam Input Method Editor (IME)',
 21+ 'narayam-menu-tooltip' => 'Select an input method',
2222 'narayam-help' => 'Help',
2323 'narayam-help-page' => 'Help:Typing',
2424 'narayam-checkbox-tooltip' => 'To toggle input method on and off', // FIXME: better message
25 - 'narayam-disable-preference' => 'Disable Narayam Input Method Editor (IME)',
 25+ 'narayam-enable-preference' => 'Enable input method',
2626 'narayam-more-imes' => 'More input methods',
2727 'narayam-am' => 'Amharic Transliteration',
2828 'narayam-as' => 'Assamese Transliteration',
@@ -1606,12 +1606,12 @@
16071607 'narayam-desc' => 'ഇൻപുട്ട് ഫീൽഡുകളിൽ ഇച്ഛാനുസരണമുള്ള ലിപികളിൽ എഴുതാനുള്ള സൗകര്യം കൂട്ടിച്ചേർക്കുന്നു',
16081608 'narayam-toggle-ime' => 'സജീവമാക്കുക ($1)',
16091609 'narayam-menu' => 'എഴുത്തുപകരണം',
1610 - 'narayam-menu-tooltip' => 'നാരായം എഴുത്തുപകരണം (ഐ.എം.ഇ.) നിയന്ത്രിക്കുക',
 1610+ 'narayam-menu-tooltip' => 'ഒരു എഴുത്തുരീതി തിരഞ്ഞെടുക്കുക',
16111611 'narayam-help' => 'സഹായം',
16121612 'narayam-help-page' => 'Help:എഴുത്ത്',
16131613 'narayam-checkbox-tooltip' => 'എഴുത്തുപകരണം ഉപയോഗിക്കാനും ഒഴിവാക്കാനും ഇവിടെ ഞെക്കുക',
1614 - 'narayam-disable-preference' => 'നാരായം എഴുത്തുപകരണം (IME) പ്രവർത്തനരഹിതമാക്കുക',
1615 - 'narayam-more-imes' => 'കൂടുതൽ ലേഖന രീതികൾ',
 1614+ 'narayam-enable-preference' => 'എഴുത്തുപകരണം ഉപയോഗിക്കുക',
 1615+ 'narayam-more-imes' => 'കൂടുതൽ എഴുത്തുരീതികൾ',
16161616 'narayam-am' => 'അംഹാറിക് ലിപ്യന്തരണം',
16171617 'narayam-as' => 'ആസാമീസ് ലിപ്യന്തരണം',
16181618 'narayam-as-avro' => 'ആസാമീസ് ആവ്റോ',
Index: trunk/extensions/Narayam/Narayam.php
@@ -168,10 +168,13 @@
169169 $wgHooks['ResourceLoaderGetConfigVars'][] = 'NarayamHooks::addConfig';
170170 $wgHooks['MakeGlobalVariablesScript'][] = 'NarayamHooks::addVariables';
171171 $wgHooks['GetPreferences'][] = 'NarayamHooks::addPreference';
 172+$wgHooks['UserGetDefaultOptions'][] = 'NarayamHooks::addDefaultOptions';
172173
173174 // Autoloader
174175 $wgAutoloadClasses['NarayamHooks'] = $dir . '/Narayam.hooks.php';
175176
 177+$wgNarayamEnabledByDefault = true;
 178+
176179 // ResourceLoader module registration
177180 $narayamTpl = array(
178181 'localBasePath' => $dir,

Status & tagging log