From 1a4fe10fe0677712459e1c6c8aee0e71b087d886 Mon Sep 17 00:00:00 2001
From: umherirrender <umherirrender_de.wp@web.de>
Date: Thu, 5 Jun 2014 20:23:09 +0200
Subject: [PATCH] Some phpcs-strict changes on includes/revisiondelete/

Apply the non visibility changes from
I0622c0156925c938b10cc1eccb4b26988452f57d (see bug 65733)

Change-Id: I982562df8208797c6fe15ba90ee84f860a58fb24
---
 includes/revisiondelete/RevisionDelete.php    | 34 +++++++++++++------
 .../RevisionDeleteAbstracts.php               |  2 ++
 2 files changed, 26 insertions(+), 10 deletions(-)

diff --git a/includes/revisiondelete/RevisionDelete.php b/includes/revisiondelete/RevisionDelete.php
index fbfe325a7f..e7a09d7271 100644
--- a/includes/revisiondelete/RevisionDelete.php
+++ b/includes/revisiondelete/RevisionDelete.php
@@ -31,6 +31,7 @@
  * See RevDelRevisionItem and RevDelArchivedRevisionItem for items.
  */
 class RevDelRevisionList extends RevDelList {
+	/** @var int */
 	var $currentRevId;
 
 	public function getType() {
@@ -147,6 +148,7 @@ class RevDelRevisionList extends RevDelList {
  * Item class for a live revision table row
  */
 class RevDelRevisionItem extends RevDelItem {
+	/** @var Revision */
 	var $revision;
 
 	public function __construct( $list, $row ) {
@@ -470,6 +472,15 @@ class RevDelArchivedRevisionItem extends RevDelArchiveItem {
  * List for oldimage table items
  */
 class RevDelFileList extends RevDelList {
+	/** @var array */
+	var $storeBatch;
+
+	/** @var array */
+	var $deleteBatch;
+
+	/** @var array */
+	var $cleanupBatch;
+
 	public function getType() {
 		return 'oldimage';
 	}
@@ -486,8 +497,6 @@ class RevDelFileList extends RevDelList {
 		return File::DELETED_FILE;
 	}
 
-	var $storeBatch, $deleteBatch, $cleanupBatch;
-
 	/**
 	 * @param DatabaseBase $db
 	 * @return mixed
@@ -569,10 +578,7 @@ class RevDelFileList extends RevDelList {
  * Item class for an oldimage table row
  */
 class RevDelFileItem extends RevDelItem {
-
-	/**
-	 * @var File
-	 */
+	/** @var File */
 	var $file;
 
 	public function __construct( $list, $row ) {
@@ -1037,7 +1043,9 @@ class RevDelLogItem extends RevDelItem {
 		// User links and action text
 		$action = $formatter->getActionText();
 		// Comment
-		$comment = $this->list->getLanguage()->getDirMark() . Linker::commentBlock( $this->row->log_comment );
+		$comment = $this->list->getLanguage()->getDirMark()
+			. Linker::commentBlock( $this->row->log_comment );
+
 		if ( LogEventsList::isDeleted( $this->row, LogPage::DELETED_COMMENT ) ) {
 			$comment = '<span class="history-deleted">' . $comment . '</span>';
 		}
@@ -1053,9 +1061,15 @@ class RevDelLogItem extends RevDelItem {
 			'type' => $logEntry->getType(),
 			'action' => $logEntry->getSubtype(),
 		);
-		$ret += $logEntry->isDeleted( LogPage::DELETED_USER ) ? array( 'userhidden' => '' ) : array();
-		$ret += $logEntry->isDeleted( LogPage::DELETED_COMMENT ) ? array( 'commenthidden' => '' ) : array();
-		$ret += $logEntry->isDeleted( LogPage::DELETED_ACTION ) ? array( 'actionhidden' => '' ) : array();
+		$ret += $logEntry->isDeleted( LogPage::DELETED_USER )
+			? array( 'userhidden' => '' )
+			: array();
+		$ret += $logEntry->isDeleted( LogPage::DELETED_COMMENT )
+			? array( 'commenthidden' => '' )
+			: array();
+		$ret += $logEntry->isDeleted( LogPage::DELETED_ACTION )
+			? array( 'actionhidden' => '' )
+			: array();
 
 		if ( LogEventsList::userCan( $this->row, LogPage::DELETED_ACTION, $user ) ) {
 			ApiQueryLogEvents::addLogParams(
diff --git a/includes/revisiondelete/RevisionDeleteAbstracts.php b/includes/revisiondelete/RevisionDeleteAbstracts.php
index 4dc93883b3..086db3a721 100644
--- a/includes/revisiondelete/RevisionDeleteAbstracts.php
+++ b/includes/revisiondelete/RevisionDeleteAbstracts.php
@@ -105,7 +105,9 @@ abstract class RevDelList extends RevisionListBase {
 			$status->itemStatuses = array();
 		}
 
+		// @codingStandardsIgnoreStart Generic.CodeAnalysis.ForLoopWithTestFunctionCall.NotAllowed
 		for ( $this->reset(); $this->current(); $this->next() ) {
+			// @codingStandardsIgnoreEnd
 			$item = $this->current();
 			unset( $missing[$item->getId()] );
 
-- 
2.20.1