Reverted r48983: obsolete since r48989, replaced by less intrusive optimisations.

diff --git a/includes/Title.php b/includes/Title.php
index 3da5888..f4c4641 100644 (file)
--- a/includes/Title.php
+++ b/includes/Title.php
@@ -1018,8 +1018,7 @@ class Title {
 
        /**
         * Can $wgUser perform $action on this page?
-        * This skips potentially expensive cascading permission checks
-        * as well as avoids expensive error formatting
+        * This skips potentially expensive cascading permission checks.
         *
         * Suitable for use for nonessential UI controls in common cases, but
         * _not_ for functional access control.
@@ -1204,14 +1203,8 @@ class Title {
                        }
                } elseif( !$user->isAllowed( $action ) ) {
                        $return = null;
-                       
-                       // We avoid expensive display logic for quickUserCan's and such
-                       $groups = false; 
-                       if (!$short) {
-                               $groups = array_map( array( 'User', 'makeGroupLinkWiki' ),
-                                       User::getGroupsWithPermission( $action ) );
-                       } 
-                       
+                       $groups = array_map( array( 'User', 'makeGroupLinkWiki' ),
+                               User::getGroupsWithPermission( $action ) );
                        if( $groups ) {
                                $return = array( 'badaccess-groups',
                                        array( implode( ', ', $groups ), count( $groups ) ) );