r44118 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r44117‎ | r44118 | r44119 >
Date:16:51, 1 December 2008
Author:aaron
Status:ok
Tags:
Comment:
Add comments
Modified paths:
  • /trunk/extensions/Configure/Configure.func.php (modified) (history)
  • /trunk/extensions/Configure/Configure.handler-files.php (modified) (history)
  • /trunk/extensions/Configure/Configure.obj.php (modified) (history)
  • /trunk/extensions/Configure/SpecialExtensions.php (modified) (history)

Diff [purge]

Index: trunk/extensions/Configure/Configure.func.php
@@ -29,8 +29,7 @@
3030 foreach ( $all as $right ) {
3131 $id = Sanitizer::escapeId( 'wpwgGroupPermissions-' . $group . '-' . $right );
3232 $desc = ( is_callable( array( 'User', 'getRightDescription' ) ) ) ?
33 - User::getRightDescription( $right ) :
34 - $right;
 33+ User::getRightDescription( $right ) : $right;
3534 $row .= '<li>' . Xml::checkLabel( $desc, $id, $id ) . "</li>\n";
3635 }
3736 $row .= '</ul></div>';
Index: trunk/extensions/Configure/SpecialExtensions.php
@@ -84,6 +84,7 @@
8585
8686 /**
8787 * Build the content of the form
 88+ * FIXME: register_globals
8889 *
8990 * @return xhtml
9091 */
Index: trunk/extensions/Configure/Configure.obj.php
@@ -81,12 +81,12 @@
8282 if ( defined( 'EXT_CONFIGURE_NO_EXTRACT' ) )
8383 return;
8484
85 - // Include files before so that customized settings won't be overriden
 85+ // Include files before so that customized settings won't be overridden
8686 // by the default ones
8787 $this->includeFiles();
8888
89 - ## Snapshot current settings before overriding.
90 - ## ialex tells me this weird contraption will work
 89+ // Snapshot current settings before overriding.
 90+ // -ialex tells me this weird contraption will work
9191 $allSettings = array_keys( ConfigurationSettings::singleton( CONF_SETTINGS_BOTH )->getAllSettings() );
9292
9393 foreach( $allSettings as $setting ) {
Index: trunk/extensions/Configure/Configure.handler-files.php
@@ -43,6 +43,7 @@
4444 /**
4545 * Return the configuration from the conf-{$ts}.ser file in the $this->mDir
4646 * Does *not* return site specific settings but *all* settings
 47+ * FIXME: timestamp is not unique
4748 *
4849 * @param $ts timestamp
4950 * @return array
@@ -62,6 +63,7 @@
6364
6465 /**
6566 * Returns the wikis in $ts version
 67+ * FIXME: timestamp is not unique
6668 *
6769 * @param $ts timestamp
6870 * @return array
@@ -121,7 +123,8 @@
122124 if (isset( $settings['__metadata'] )) {
123125 $metadata = $settings['__metadata'];
124126
125 - $files[$m[1]] = array( 'username' => $metadata['user_name'], 'userwiki' => $metadata['user_wiki'], 'reason' => $metadata['reason'] );
 127+ $files[$m[1]] = array( 'username' => $metadata['user_name'],
 128+ 'userwiki' => $metadata['user_wiki'], 'reason' => $metadata['reason'] );
126129 } else {
127130 $files[$m[1]] = array( 'username' => false, 'userwiki' => false, 'reason' => false );
128131 }

Status & tagging log