From 46ff5726c6e5be1ee73fd5a988f92c1e61b1bfc3 Mon Sep 17 00:00:00 2001
From: umherirrender <umherirrender_de.wp@web.de>
Date: Mon, 26 May 2014 20:21:10 +0200
Subject: [PATCH] SpecialRevisiondelete: Simplify if/elseif logic

This avoids checking $comment twice
by Krinkle on I3b21f66f84d68777a18e66e4e5f59886e55d7936

Change-Id: I39df27d3edb097024d136455145505429ccdbe39
---
 includes/specials/SpecialRevisiondelete.php | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/includes/specials/SpecialRevisiondelete.php b/includes/specials/SpecialRevisiondelete.php
index 797f588a0d..d3b168b35b 100644
--- a/includes/specials/SpecialRevisiondelete.php
+++ b/includes/specials/SpecialRevisiondelete.php
@@ -572,12 +572,12 @@ class SpecialRevisionDelete extends UnlistedSpecialPage {
 		// from dropdown
 		$listReason = $this->getRequest()->getText( 'wpRevDeleteReasonList', 'other' );
 		$comment = $listReason;
-		if ( $comment != 'other' && $this->otherReason != '' ) {
+		if ( $comment === 'other' ) {
+			$comment = $this->otherReason;
+		} elseif ( $this->otherReason !== '' ) {
 			// Entry from drop down menu + additional comment
 			$comment .= $this->msg( 'colon-separator' )->inContentLanguage()->text()
 				. $this->otherReason;
-		} elseif ( $comment == 'other' ) {
-			$comment = $this->otherReason;
 		}
 		# Can the user set this field?
 		if ( $bitParams[Revision::DELETED_RESTRICTED] == 1
-- 
2.20.1