r50164 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r50163‎ | r50164 | r50165 >
Date:08:57, 4 May 2009
Author:tstarling
Status:deferred
Tags:
Comment:
Reverted r48983: obsolete since r48989, replaced by less intrusive optimisations.
Modified paths:
  • /trunk/phase3/includes/Title.php (modified) (history)
  • /trunk/phase3/includes/specials/SpecialCreatePage.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/Title.php
@@ -1018,8 +1018,7 @@
10191019
10201020 /**
10211021 * Can $wgUser perform $action on this page?
1022 - * This skips potentially expensive cascading permission checks
1023 - * as well as avoids expensive error formatting
 1022+ * This skips potentially expensive cascading permission checks.
10241023 *
10251024 * Suitable for use for nonessential UI controls in common cases, but
10261025 * _not_ for functional access control.
@@ -1204,14 +1203,8 @@
12051204 }
12061205 } elseif( !$user->isAllowed( $action ) ) {
12071206 $return = null;
1208 -
1209 - // We avoid expensive display logic for quickUserCan's and such
1210 - $groups = false;
1211 - if (!$short) {
1212 - $groups = array_map( array( 'User', 'makeGroupLinkWiki' ),
1213 - User::getGroupsWithPermission( $action ) );
1214 - }
1215 -
 1207+ $groups = array_map( array( 'User', 'makeGroupLinkWiki' ),
 1208+ User::getGroupsWithPermission( $action ) );
12161209 if( $groups ) {
12171210 $return = array( 'badaccess-groups',
12181211 array( implode( ', ', $groups ), count( $groups ) ) );
Property changes on: trunk/phase3/includes/specials/SpecialCreatePage.php
___________________________________________________________________
Name: svn:mergeinfo
12191212 -

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r48983I'd love not to do such fixes:...midom08:28, 29 March 2009
r48989fix some of RC rollback madness:...midom11:02, 29 March 2009

Status & tagging log