r23255 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r23254‎ | r23255 | r23256 >
Date:21:23, 22 June 2007
Author:robchurch
Status:old
Tags:
Comment:
* Don't abuse globals to store messages
* Don't mess about with backwards-compatible fiddling for new special page extensions when using functions not found in those old versions
Modified paths:
  • /trunk/extensions/Resign/SpecialResign.i18n.php (modified) (history)
  • /trunk/extensions/Resign/SpecialResign.php (modified) (history)

Diff [purge]

Index: trunk/extensions/Resign/SpecialResign.i18n.php
@@ -1,36 +1,42 @@
22 <?php
 3+
34 /**
4 - * Internationalisation file for Resign extension.
 5+ * Internationalisation file for Resign extension
56 *
67 * @addtogroup Extensions
78 */
89
9 -$wgResignMessages = array();
10 -
11 -$wgResignMessages['en'] = array(
12 - 'resign' => 'Resign',
13 - 'resign-text' => "You can '''remove your permissions''' (e.g. sysop) in this page. Please check the permissions you would like to remove.
 10+function efResignMessages() {
 11+ $messages = array(
1412
15 - Remember: '''You cannot get your permissions back this way!''' You will have to convince a bureaucrat or steward to grant you back the permissions. Please use this page very carefully.",
16 - 'resign-reason' => 'Reason:',
17 - 'resign-confirm' => 'I confirm that I would like to remove all the permissions I checked.',
18 - 'resign-noconfirm' => 'Please check the confirm checkbox if you would like to remove your permissions.',
19 - 'resign-nogroups' => 'Please check the permissions you would like to remove.',
20 - 'resign-success' => "'''The permissions were successfully removed.'''",
21 - 'resign-logentry' => 'removed permissions $2 from himself',
22 -);
 13+'en' => array(
 14+'resign' => 'Resign',
 15+'resign-text' => "You can '''remove your permissions''' (e.g. sysop) in this page. Please check the permissions you would like to remove.
2316
24 -$wgResignMessages['he'] = array(
25 - 'resign' => 'התפטרות',
26 - 'resign-text' => "בדף זה תוכלו '''להסיר את ההרשאות שלכם''' (למשל, הרשאות מפעיל מערכת). אנא סמנו את ההרשאות שברצונכם להסיר.
27 -
28 - להזכירכם: '''אינכם יכולים לקבל בחזרה את ההרשאות בדרך זו!''' יהיה עליכם לשכנע ביורוקרט או דייל להעניק לכם את הרשאותיכם בחזרה. אנא השתמשו בדף זה בזהירות רבה.",
29 - 'resign-reason' => 'סיבה:',
30 - 'resign-confirm' => 'אני מאשר שברצוני להסיר את כל ההרשאות שלי שסימנתי, וידוע לי שלא אוכל לקבל אותן בחזרה בדרך זו.',
31 - 'resign-noconfirm' => 'אנא סמנו את תיבת הסימון כדי לאשר את ההסרה.',
32 - 'resign-nogroups' => 'אנא בחרו את ההרשאות שברצונכם להסיר.',
33 - 'resign-success' => "'''ההרשאות הוסרו בהצלחה.'''",
34 - 'resign-logentry' => 'הסיר את ההרשאות $2 שלו',
35 -);
 17+Remember: '''You cannot get your permissions back this way!''' You will have to convince a bureaucrat or steward to grant you back the permissions. Please use this page very carefully.",
 18+'resign-reason' => 'Reason:',
 19+'resign-confirm' => 'I confirm that I would like to remove all the permissions I checked.',
 20+'resign-noconfirm' => 'Please check the confirm checkbox if you would like to remove your permissions.',
 21+'resign-nogroups' => 'Please check the permissions you would like to remove.',
 22+'resign-success' => "'''The permissions were successfully removed.'''",
 23+'resign-logentry' => 'removed permissions $2 from himself',
 24+),
3625
 26+'he' => array(
 27+'resign' => 'התפטרות',
 28+'resign-text' => "בדף זה תוכלו '''להסיר את ההרשאות שלכם''' (למשל, הרשאות מפעיל מערכת). אנא סמנו את ההרשאות שברצונכם להסיר.
 29+
 30+להזכירכם: '''אינכם יכולים לקבל בחזרה את ההרשאות בדרך זו!''' יהיה עליכם לשכנע ביורוקרט או דייל להעניק לכם את הרשאותיכם בחזרה. אנא השתמשו בדף זה בזהירות רבה.",
 31+'resign-reason' => 'סיבה:',
 32+'resign-confirm' => 'אני מאשר שברצוני להסיר את כל ההרשאות שלי שסימנתי, וידוע לי שלא אוכל לקבל אותן בחזרה בדרך זו.',
 33+'resign-noconfirm' => 'אנא סמנו את תיבת הסימון כדי לאשר את ההסרה.',
 34+'resign-nogroups' => 'אנא בחרו את ההרשאות שברצונכם להסיר.',
 35+'resign-success' => "'''ההרשאות הוסרו בהצלחה.'''",
 36+'resign-logentry' => 'הסיר את ההרשאות $2 שלו',
 37+),
 38+
 39+ );
 40+ return $messages;
 41+}
 42+
3743 ?>
Index: trunk/extensions/Resign/SpecialResign.php
@@ -16,9 +16,6 @@
1717 'description' => 'Gives users the ability to remove their permissions'
1818 );
1919
20 -# Internationalisation file
21 -require_once( dirname(__FILE__) . '/SpecialResign.i18n.php' );
22 -
2320 # Add resign permission for every group set in the database
2421 foreach( $wgGroupPermissions as $key => $value ) {
2522 if ( $key != '*' && $key != 'user' && $key != 'autoconfirmed' && $key != 'emailconfirmed' ) {
@@ -29,16 +26,13 @@
3027 # Add log action
3128 $wgLogActions['rights/resign'] = 'resign-logentry';
3229
33 -# Register special page
34 -if ( !function_exists( 'extAddSpecialPage' ) ) {
35 - require( dirname(__FILE__) . '/../ExtensionFunctions.php' );
36 -}
37 -extAddSpecialPage( dirname(__FILE__) . '/SpecialResign_body.php', 'Resign', 'ResignPage' );
 30+$wgAutoloadClasses['ResignPage'] = dirname( __FILE__ ) . '/SpecialResign_body.php';
 31+$wgSpecialPages['Resign'] = 'ResignPage';
3832
3933 function wfSpecialResign() {
40 - # Add messages
41 - global $wgMessageCache, $wgResignMessages;
42 - $wgMessageCache->addMessagesByLang( $wgResignMessages );
 34+ global $wgMessageCache;
 35+ require_once( dirname(__FILE__) . '/SpecialResign.i18n.php' );
 36+ $wgMessageCache->addMessagesByLang( efResignMessages() );
4337 }
4438
4539 ?>

Status & tagging log