From: jenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Date: Fri, 21 Sep 2018 02:20:23 +0000 (+0000)
Subject: Merge "Collapse some unnecessary else conditions"
X-Git-Tag: 1.34.0-rc.0~4034
X-Git-Url: https://git.heureux-cyclage.org/?p=lhc%2Fweb%2Fwiklou.git;a=commitdiff_plain;h=0f65db0f2a5e40e17474b9dfea1d0516e697dd5b;hp=4e8eed74233db1db69f178fb0f4f1f8d6416bd52

Merge "Collapse some unnecessary else conditions"
---

diff --git a/RELEASE-NOTES-1.32 b/RELEASE-NOTES-1.32
index 9f0bc1b777..51a149814c 100644
--- a/RELEASE-NOTES-1.32
+++ b/RELEASE-NOTES-1.32
@@ -311,6 +311,7 @@ because of Phabricator reports.
   * ChangesListSpecialPage::customFilters
 * The global function wfUseMW, deprecated since 1.26, has now been removed. Use
   the "requires" property of static extension registration instead.
+* $wgSpecialPages no longer accepts array syntax, deprecated since 1.18.
 
 === Deprecations in 1.32 ===
 * HTMLForm::setSubmitProgressive() is deprecated. No need to call it. Submit
diff --git a/includes/specialpage/SpecialPageFactory.php b/includes/specialpage/SpecialPageFactory.php
index b9c2204e1b..013ceb24e5 100644
--- a/includes/specialpage/SpecialPageFactory.php
+++ b/includes/specialpage/SpecialPageFactory.php
@@ -34,7 +34,6 @@ use RequestContext;
 use SpecialPage;
 use Title;
 use User;
-use Wikimedia\ObjectFactory;
 
 /**
  * Factory for handling the special page list and generating SpecialPage objects.
@@ -401,16 +400,6 @@ class SpecialPageFactory {
 			} elseif ( is_string( $rec ) ) {
 				$className = $rec;
 				$page = new $className;
-			} elseif ( is_array( $rec ) ) {
-				$className = array_shift( $rec );
-				// @deprecated, officially since 1.18, unofficially since forever
-				wfDeprecated( "Array syntax for \$wgSpecialPages is deprecated ($className), " .
-					"define a subclass of SpecialPage instead.", '1.18' );
-				$page = ObjectFactory::getObjectFromSpec( [
-					'class' => $className,
-					'args' => $rec,
-					'closure_expansion' => false,
-				] );
 			} elseif ( $rec instanceof SpecialPage ) {
 				$page = $rec; // XXX: we should deep clone here
 			} else {
diff --git a/tests/parser/parserTests.txt b/tests/parser/parserTests.txt
index b1510029a1..88797d7510 100644
--- a/tests/parser/parserTests.txt
+++ b/tests/parser/parserTests.txt
@@ -12670,10 +12670,13 @@ parsoid=wt2html
 <p about="#mwt1" typeof="mw:Transclusion" data-mw='{"parts":[{"template":{"target":{"wt":"echo","href":"./Template:Echo"},"params":{"1":{"wt":"hi {{}}"}},"i":0}}]}'>hi {{}}</p>
 !! end
 
+# Ensure we're using a language without variants for this test; even
+# if $wgUsePigLatinVariant is true.  We've picked `de` arbitrarily.
 !! test
 Preprocessor precedence 18: another rightmost wins scenario
 !! options
 parsoid=wt2html
+language=de
 !! wikitext
 {{ -{{{{1|tplarg}}} }} }-
 !! html/php