From 2fb5c57fba7cb1faa282c1e40b11420b96e7a226 Mon Sep 17 00:00:00 2001
From: Umherirrender <umherirrender_de.wp@web.de>
Date: Wed, 6 Jan 2021 00:21:50 +0100
Subject: [PATCH] Remove unneeded break after throw

Change-Id: Ief0dd9ecbf62ab84a3e504634eff4a236c115691
---
 .phpcs.xml                 | 2 --
 maintenance/replaceAll.php | 1 -
 2 files changed, 3 deletions(-)

diff --git a/.phpcs.xml b/.phpcs.xml
index e47d8387..cd875029 100644
--- a/.phpcs.xml
+++ b/.phpcs.xml
@@ -6,11 +6,9 @@
 		<exclude name="MediaWiki.Commenting.PropertyDocumentation.MissingDocumentationPrivate" />
 		<exclude name="MediaWiki.Usage.ExtendClassUsage.FunctionConfigUsage" />
 		<exclude name="MediaWiki.Usage.ExtendClassUsage.FunctionVarUsage" />
-		<exclude name="Squiz.PHP.NonExecutableCode.Unreachable" />
 		<exclude name="Squiz.Scope.MethodScope.Missing" />
 	</rule>
 	<file>.</file>
 	<arg name="extensions" value="php" />
 	<arg name="encoding" value="UTF-8" />
-	<exclude-pattern>vendor</exclude-pattern>
 </ruleset>
diff --git a/maintenance/replaceAll.php b/maintenance/replaceAll.php
index bb2794ac..494209c3 100755
--- a/maintenance/replaceAll.php
+++ b/maintenance/replaceAll.php
@@ -144,7 +144,6 @@ class ReplaceAll extends Maintenance {
 		$handle = fopen( $file, "r" );
 		if ( $handle === false ) {
 			throw new MWException( "Trouble opening file: $file\n" );
-			return false;
 		}
 
 		$this->defaultContinue = true;