Merge "Declare dynamic properties"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Fri, 13 Sep 2019 21:49:14 +0000 (21:49 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Fri, 13 Sep 2019 21:49:14 +0000 (21:49 +0000)
1  2 
includes/logging/LogPager.php

@@@ -53,6 -53,12 +53,12 @@@ class LogPager extends ReverseChronolog
        /** @var bool */
        private $actionRestrictionsEnforced = false;
  
+       /** @var array */
+       private $mConds;
+       /** @var string */
+       private $mTagFilter;
        /** @var LogEventsList */
        public $mLogEventsList;
  
                $needReindex = false;
                foreach ( $types as $type ) {
                        if ( isset( $wgLogRestrictions[$type] )
 -                              && !$user->isAllowed( $wgLogRestrictions[$type] )
 +                              && !MediaWikiServices::getInstance()
 +                                      ->getPermissionManager()
 +                                      ->userHasRight( $user, $wgLogRestrictions[$type] )
                        ) {
                                $needReindex = true;
                                $types = array_diff( $types, [ $type ] );
                }
                $this->actionRestrictionsEnforced = true;
                $user = $this->getUser();
 -              if ( !$user->isAllowed( 'deletedhistory' ) ) {
 +              $permissionManager = MediaWikiServices::getInstance()->getPermissionManager();
 +              if ( !$permissionManager->userHasRight( $user, 'deletedhistory' ) ) {
                        $this->mConds[] = $this->mDb->bitAnd( 'log_deleted', LogPage::DELETED_ACTION ) . ' = 0';
 -              } elseif ( !MediaWikiServices::getInstance()
 -                      ->getPermissionManager()
 -                      ->userHasAnyRight( $user, 'suppressrevision', 'viewsuppressed' )
 -              ) {
 +              } elseif ( !$permissionManager->userHasAnyRight( $user, 'suppressrevision', 'viewsuppressed' ) ) {
                        $this->mConds[] = $this->mDb->bitAnd( 'log_deleted', LogPage::SUPPRESSED_ACTION ) .
                                ' != ' . LogPage::SUPPRESSED_USER;
                }
                }
                $this->performerRestrictionsEnforced = true;
                $user = $this->getUser();
 -              if ( !$user->isAllowed( 'deletedhistory' ) ) {
 +              $permissionManager = MediaWikiServices::getInstance()->getPermissionManager();
 +              if ( !$permissionManager->userHasRight( $user, 'deletedhistory' ) ) {
                        $this->mConds[] = $this->mDb->bitAnd( 'log_deleted', LogPage::DELETED_USER ) . ' = 0';
 -              } elseif ( !MediaWikiServices::getInstance()
 -                      ->getPermissionManager()
 -                      ->userHasAnyRight( $user, 'suppressrevision', 'viewsuppressed' )
 -              ) {
 +              } elseif ( !$permissionManager->userHasAnyRight( $user, 'suppressrevision', 'viewsuppressed' ) ) {
                        $this->mConds[] = $this->mDb->bitAnd( 'log_deleted', LogPage::SUPPRESSED_USER ) .
                                ' != ' . LogPage::SUPPRESSED_ACTION;
                }