r29583 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:< r29582‎ | r29583 | r29584 >
Date:10:35, 11 January 2008
Author:siebrand
Status:old
Tags:
Comment:
* use wfLoadExtensionMessages for Interwiki
* Delay message loading
* add version in extension credits
* update Translate extension
Modified paths:
  • /trunk/extensions/Interwiki/SpecialInterwiki.i18n.php (modified) (history)
  • /trunk/extensions/Interwiki/SpecialInterwiki.php (modified) (history)
  • /trunk/extensions/Translate/MessageGroups.php (modified) (history)

Diff [purge]

Index: trunk/extensions/Interwiki/SpecialInterwiki.i18n.php
——@@ -1,5 +1,9 @@
22 <?php
3 -/*
 3+/**
 4+ * Internationalisation file for extension Interwiki.
 5+ *
 6+ * @addtogroup Extensions
 7+ *
48 * This program is free software; you can redistribute it and/or modify
59 * it under the terms of the GNU General Public License as published by
610 * the Free Software Foundation; either version 2 of the License, or
——@@ -12,9 +16,9 @@
1317 * @license http://www.gnu.org/copyleft/gpl.html GNU General Public License 2.0 or later
1418 */
1519
16 -$wgSpecialInterwikiMessages = array();
 20+$messages = array();
1721
18 -$wgSpecialInterwikiMessages['en'] = array(
 22+$messages['en'] = array(
1923 # general messages
2024 'interwiki' => 'View and edit interwiki data',
2125 'interwiki_prefix' => 'Prefix',
——@@ -48,7 +52,7 @@
4953 /** Arabic (العربية)
5054 * @author Meno25
5155 */
52 -$wgSpecialInterwikiMessages['ar'] = array(
 56+$messages['ar'] = array(
5357 'interwiki' => 'عرض وتعديل بيانات الإنترويكي',
5458 'interwiki_prefix' => 'بادئة',
5559 'interwiki_reasonfield' => 'سبب',
——@@ -75,7 +79,7 @@
7680 /** Bulgarian (Български)
7781 * @author DCLXVI
7882 */
79 -$wgSpecialInterwikiMessages['bg'] = array(
 83+$messages['bg'] = array(
8084 'interwiki' => 'Преглед и управление на междууикитата',
8185 'interwiki_prefix' => 'Представка:',
8286 'interwiki_reasonfield' => 'Причина',
——@@ -93,7 +97,7 @@
9498 * @author MF-Warburg
9599 * @author Raimond Spekking
96100 */
97 -$wgSpecialInterwikiMessages['de'] = array(
 101+$messages['de'] = array(
98102 'interwiki' => 'Interwiki-Daten betrachten und bearbeiten',
99103 'interwiki_prefix' => 'Präfix',
100104 'interwiki_reasonfield' => 'Grund',
——@@ -118,7 +122,7 @@
119123 /** Greek (Ελληνικά)
120124 * @author Consta
121125 */
122 -$wgSpecialInterwikiMessages['el'] = array(
 126+$messages['el'] = array(
123127 'interwiki_prefix' => 'Πρόθεμα',
124128 'interwiki_reasonfield' => 'Λόγος',
125129 'interwiki_defaultreason' => 'Δεν δίνετε λόγος',
——@@ -128,7 +132,7 @@
129133 * @author Grondin
130134 * @author Sherbrooke
131135 */
132 -$wgSpecialInterwikiMessages['fr'] = array(
 136+$messages['fr'] = array(
133137 'interwiki' => 'Voir et manipuler les données interwiki',
134138 'interwiki_prefix' => 'Préfixe',
135139 'interwiki_reasonfield' => 'Motif',
——@@ -154,7 +158,7 @@
155159 * @author Alma
156160 * @author Xosé
157161 */
158 -$wgSpecialInterwikiMessages['gl'] = array(
 162+$messages['gl'] = array(
159163 'interwiki' => 'Ver e manipular datos interwiki',
160164 'interwiki_prefix' => 'Prefixo',
161165 'interwiki_reasonfield' => 'Razón',
——@@ -177,19 +181,19 @@
178182 );
179183
180184 /** Gujarati (ગુજરાતી) */
181 -$wgSpecialInterwikiMessages['gu'] = array(
 185+$messages['gu'] = array(
182186 'interwiki_reasonfield' => 'કારણ',
183187 );
184188
185189 /** Hawaiian (Hawai`i)
186190 * @author SPQRobin
187191 */
188 -$wgSpecialInterwikiMessages['haw'] = array(
 192+$messages['haw'] = array(
189193 'interwiki_reasonfield' => 'Ke kumu',
190194 'interwiki_defaultreason' => '‘a‘ohe kumu',
191195 );
192196
193 -$wgSpecialInterwikiMessages['hr'] = array(
 197+$messages['hr'] = array(
194198 'interwiki' => 'Vidi i uredi međuwiki podatke',
195199 'interwiki_prefix' => 'Prefiks',
196200 'interwiki_local' => 'Lokalno',
——@@ -204,7 +208,7 @@
205209 /** Upper Sorbian (Hornjoserbsce)
206210 * @author Michawiki
207211 */
208 -$wgSpecialInterwikiMessages['hsb'] = array(
 212+$messages['hsb'] = array(
209213 'interwiki' => 'Interwiki-daty wobhladać a změnić',
210214 'interwiki_prefix' => 'Prefiks',
211215 'interwiki_reasonfield' => 'Přičina',
——@@ -231,7 +235,7 @@
232236 * @author UV
233237 * @author SPQRobin
234238 */
235 -$wgSpecialInterwikiMessages['la'] = array(
 239+$messages['la'] = array(
236240 'interwiki' => 'Videre et recensere data intervica',
237241 'interwiki_prefix' => 'Praefixum',
238242 'interwiki_reasonfield' => 'Causa',
——@@ -255,7 +259,7 @@
256260 /** Luxembourgish (Lëtzebuergesch)
257261 * @author Robby
258262 */
259 -$wgSpecialInterwikiMessages['lb'] = array(
 263+$messages['lb'] = array(
260264 'interwiki' => 'Interwiki-Date kucken a veränneren',
261265 'interwiki_prefix' => 'Prefix',
262266 'interwiki_reasonfield' => 'Grond',
——@@ -266,7 +270,7 @@
267271 /** Dutch (Nederlands)
268272 * @author SPQRobin
269273 */
270 -$wgSpecialInterwikiMessages['nl'] = array(
 274+$messages['nl'] = array(
271275 'interwiki' => 'Interwikigegevens bekijken en wijzigen',
272276 'interwiki_prefix' => 'Voorvoegsel',
273277 'interwiki_reasonfield' => 'Reden',
——@@ -288,7 +292,7 @@
289293 'interwiki_defaultreason' => 'geen reden gegeven',
290294 );
291295
292 -$wgSpecialInterwikiMessages['no'] = array(
 296+$messages['no'] = array(
293297 'interwiki' => 'Vis og manipuler interwikidata',
294298 'interwiki_prefix' => 'Prefiks',
295299 'interwiki_local' => 'Lokal',
——@@ -312,7 +316,7 @@
313317 /** Occitan (Occitan)
314318 * @author Cedric31
315319 */
316 -$wgSpecialInterwikiMessages['oc'] = array(
 320+$messages['oc'] = array(
317321 'interwiki' => 'Veire e editar las donadas interwiki',
318322 'interwiki_reasonfield' => 'Motiu',
319323 'interwiki_intro' => "Vejatz $1 per obténer mai d'informacions per çò que concernís la taula interwiki. Aquò es lo [[Special:Log/interwiki|jornal de las modificacions]] de la taula interwiki.",
——@@ -333,7 +337,7 @@
334338 'interwiki_defaultreason' => 'Cap de motiu balhat',
335339 );
336340
337 -$wgSpecialInterwikiMessages['pt'] = array(
 341+$messages['pt'] = array(
338342 'interwiki' => 'Ver e manipular dados de interwikis',
339343 'interwiki_prefix' => 'Prefixo',
340344 'interwiki_local' => 'Local',#identical but defined
——@@ -358,7 +362,7 @@
359363 * @author Alexander Chemeris
360364 * @author Illusion
361365 */
362 -$wgSpecialInterwikiMessages['ru'] = array(
 366+$messages['ru'] = array(
363367 'interwiki' => 'Просмотр и изменение префиксов интервики',
364368 'interwiki_prefix' => 'Префикс',
365369 'interwiki_reasonfield' => 'Причина',
——@@ -380,7 +384,7 @@
381385 /** Sassaresu (Sassaresu)
382386 * @author Felis
383387 */
384 -$wgSpecialInterwikiMessages['sdc'] = array(
 388+$messages['sdc'] = array(
385389 'interwiki' => 'Vidè e mudìfiggà li dati interwiki',
386390 'interwiki_prefix' => 'Prefissu',
387391 'interwiki_reasonfield' => 'Rasgioni',
——@@ -395,7 +399,7 @@
396400 /** Slovak (Slovenčina)
397401 * @author Helix84
398402 */
399 -$wgSpecialInterwikiMessages['sk'] = array(
 403+$messages['sk'] = array(
400404 'interwiki' => 'Zobraziť a upravovať údaje interwiki',
401405 'interwiki_prefix' => 'Predpona',
402406 'interwiki_reasonfield' => 'Dôvod',
——@@ -420,7 +424,7 @@
421425 /** Seeltersk (Seeltersk)
422426 * @author Pyt
423427 */
424 -$wgSpecialInterwikiMessages['stq'] = array(
 428+$messages['stq'] = array(
425429 'interwiki' => 'Interwiki-Doaten bekiekje un beoarbaidje',
426430 'interwiki_prefix' => 'Präfix',
427431 'interwiki_reasonfield' => 'Gruund',
Index: trunk/extensions/Interwiki/SpecialInterwiki.php
——@@ -14,7 +14,6 @@
1515 */
1616
1717 if (!defined('MEDIAWIKI')) die();
18 -$wgExtensionFunctions[] = "Interwiki";
1918
2019 $wgExtensionCredits['specialpage'][] = array(
2120 'name' => 'SpecialInterwiki',
——@@ -24,12 +23,11 @@
2524 'author' => array( 'Stephanie Amanda Stevens', 'SPQRobin', 'others' ),
2625 );
2726
 27+$wgExtensionMessagesFiles['Interwiki'] = dirname(__FILE__) . '/SpecialInterwiki.i18n.php';
 28+$wgExtensionFunctions[] = "Interwiki";
 29+
2830 function Interwiki() {
29 - global $IP, $wgMessageCache, $wgHooks, $wgSpecialInterwikiMessages;
30 - require_once( "$IP/includes/SpecialPage.php" );
31 - require_once( 'SpecialInterwiki.i18n.php');
32 - foreach( $wgSpecialInterwikiMessages as $key => $value ) {
33 - $wgMessageCache->addMessages( $wgSpecialInterwikiMessages[$key], $key );}
 31+ global $IP, $wgHooks;
3432
3533 # Add a new log type
3634 $wgHooks['LogPageValidTypes'][] = 'wfInterwikiAddLogType';
——@@ -40,13 +38,13 @@
4139 $wgHooks['LogPageLogHeader'][] = 'wfInterwikiAddLogHeader';
4240 $wgHooks['LogPageActionText'][] = 'wfInterwikiAddActionText';
4341
44 - class Interwiki extends SpecialPage {
45 - function Interwiki() {
46 - SpecialPage::SpecialPage( 'Interwiki' );
47 - $this->includable( true );
 42+ class Interwiki extends IncludableSpecialPage {
 43+ public function __construct() {
 44+ parent::__construct( 'Interwiki' );
4845 }
4946
50 - function execute( $par = null ) {
 47+ function execute( $para ) {
 48+ wfLoadExtensionMessages( 'Interwiki' );
5149 $fname = 'Interwiki::execute';
5250 global $wgOut, $wgRequest, $wgUser;
5351
——@@ -62,6 +60,8 @@
6361 $safePost = $wgRequest->wasPosted() &&
6462 $wgUser->matchEditToken( $wgRequest->getVal( 'wpEditToken' ) );
6563
 64+ $out = "";
 65+
6666 if ($do == "delete") {
6767 if (!$admin) { $wgOut->permissionRequired('interwiki'); return; }
6868 $prefix = $wgRequest->getVal( 'prefix' );
——@@ -227,7 +227,7 @@
228228 }
229229 }
230230
231 - SpecialPage::addPage( new Interwiki );
 231+ IncludableSpecialPage::addPage( new Interwiki );
232232 }
233233
234234 function wfInterwikiAddLogType( &$types ) {
Index: trunk/extensions/Translate/MessageGroups.php
——@@ -1209,11 +1209,10 @@
12101210 }
12111211
12121212 class InterwikiMessageGroup extends ExtensionMessageGroup {
1213 - protected $fileExporter = null;
12141213 protected $label = 'Interwiki Edit Page';
12151214 protected $id = 'ext-interwiki';
12161215
1217 - protected $arrName = 'wgSpecialInterwikiMessages';
 1216+ protected $arrName = 'messages';
12181217 protected $messageFile = 'Interwiki/SpecialInterwiki.i18n.php';
12191218
12201219 protected $optional = array(

Status & tagging log