r100597 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r100596‎ | r100597 | r100598 >
Date:10:01, 24 October 2011
Author:cervidae
Status:deferred (Comments)
Tags:
Comment:
->getText() → ->getBaseText()
Modified paths:
  • /trunk/extensions/SharedCssJs/SharedCssJs.i18n.php (modified) (history)
  • /trunk/extensions/SharedCssJs/SharedCssJsProtector.php (modified) (history)

Diff [purge]

Index: trunk/extensions/SharedCssJs/SharedCssJs.i18n.php
@@ -1,6 +1,6 @@
22 <?php
33 /**
4 - * Localisation file for the SharedCssJs extension
 4+ * Internationalization file for the SharedCssJs extension
55 *
66 * @since 1.0
77 *
@@ -41,8 +41,8 @@
4242 * @author SVG
4343 */
4444 $messages['de'] = array(
45 - 'sharedcssjs-desc' => 'Ermöglicht das Nutzen globaler, in einem zentralen Wiki hinterlegter, CSS- und JS-Dateien für Wiki sowie Benutzer',
46 - 'sharedcssjs-error' => 'Diese Seite wird aus einem zentralen Wiki eingebunden und kann auch nur dort bearbeitet werden.',
 45+ 'sharedcssjs-desc' => 'Ermöglicht das Nutzen globaler, in einem zentralen Wiki hinterlegter, CSS- und JS-Dateien für das ganze Wiki sowie einzelner Benutzer',
 46+ 'sharedcssjs-error' => 'Diese Seite wird aus einem zentralen Wiki eingebunden und kann ausschließlich dort bearbeitet werden.',
4747 'global.css' => '/* Das folgende CSS wird für alle Benutzeroberflächen auf allen Wikis der Wiki-Farm geladen */',
4848 'global.js' => '/* Das folgende JavaScript wird für alle Benutzeroberflächen auf allen Wikis der Wiki-Farm geladen */',
4949 );
@@ -83,7 +83,7 @@
8484 * @author Purodha
8585 */
8686 $messages['ksh'] = array(
87 - 'sharedcssjs-desc' => 'Määd et müjjelesch, CSSe un JavaSkreppte us enem jemeinsame Wiki ze holle.',
 87+ 'sharedcssjs-desc' => 'Määd et müjjelesch, CSSe un JavaSkreppte us enem jemeinsame Wiki ze holle',
8888 'sharedcssjs-error' => 'Heh di Sigg küdd uss_enem jemeinsame Wiki un blohß doh kam_mer se ändere.',
8989 'global.css' => '/* Heh dat CSS weed vun alle Bovverfläshe vun alle Wikis heh em Schtall vun Wikis jebruch */',
9090 'global.js' => '/* Heh dat JavaSkrepp weed vun alle Bovverfläshe vun alle Wikis heh em Schtall vun Wikis jebruch */',
Index: trunk/extensions/SharedCssJs/SharedCssJsProtector.php
@@ -18,29 +18,29 @@
1919
2020 $wgHooks['getUserPermissionsErrors'][] = 'fnProtectSharedCssJs';
2121
22 -function fnProtectSharedCssJs( &$title, &$user, $action, &$result) {
 22+function fnProtectSharedCssJs( &$title, &$user, $action, &$result ) {
2323 global $wgSharedCssJsDB, $wgDBname;
2424
2525 // only protect MediaWiki:Global.css and MediaWiki:Global.js on non-central wikis
2626 if( $wgSharedCssJsDB != $wgDBname ) {
2727
2828 // block actions 'edit' and 'create'
29 - if( $action != 'edit' && $action != 'create') {
 29+ if( $action != 'edit' && $action != 'create' ) {
3030 return true;
3131 }
3232
33 - // check pagenames
34 - if( $title->getText() != 'Global.css' && $title->getText() != 'Global.js' ) {
 33+ // check pagenames (includes subpages)
 34+ if( $title->getBaseText() != 'Global.css' && $title->getBaseText() != 'Global.js' ) {
3535 return true;
3636 }
3737
3838 $ns = $title->getNamespace();
3939
4040 // check namespaces
41 - if($ns == 8 || $ns == 9 ) {
 41+ if( $ns == 8 || $ns == 9 ) {
4242
4343 // error message if action is blocked
44 - $result = array('sharedcssjs-error');
 44+ $result = array( 'sharedcssjs-error' );
4545
4646 // bail, and stop the request
4747 return false;

Comments

#Comment by Raymond (talk | contribs)   10:14, 24 October 2011

Please change translation at translatewiki.net (twn) only. This avoids backwards synching for twn staff. Thanks.

#Comment by SVG (talk | contribs)   10:55, 24 October 2011

Okay, sorry.

Status & tagging log