Resolve required parameter after optional in specials
authorErik Bernhardson <ebernhardson@wikimedia.org>
Fri, 8 Jun 2018 21:42:27 +0000 (14:42 -0700)
committerErik Bernhardson <ebernhardson@wikimedia.org>
Fri, 8 Jun 2018 22:26:03 +0000 (15:26 -0700)
Change-Id: If2c85d97d68b24fc5cb2bec19804034b994b348d

includes/specials/SpecialPrefixindex.php
includes/specials/SpecialProtectedpages.php
includes/specials/SpecialProtectedtitles.php
includes/specials/pagers/ProtectedPagesPager.php
includes/specials/pagers/ProtectedTitlesPager.php

index 3ca3a85..6848d2c 100644 (file)
@@ -138,11 +138,11 @@ class SpecialPrefixindex extends SpecialAllPages {
        }
 
        /**
-        * @param int $namespace Default NS_MAIN
+        * @param int $namespace
         * @param string $prefix
         * @param string $from List all pages from this name (default false)
         */
-       protected function showPrefixChunk( $namespace = NS_MAIN, $prefix, $from = null ) {
+       protected function showPrefixChunk( $namespace, $prefix, $from = null ) {
                global $wgContLang;
 
                if ( $from === null ) {
index 26f4da5..4b1b344 100644 (file)
@@ -92,7 +92,7 @@ class SpecialProtectedpages extends SpecialPage {
         *   cascadeOnly, noRedirect
         * @return string Input form
         */
-       protected function showOptions( $namespace, $type = 'edit', $level, $sizetype,
+       protected function showOptions( $namespace, $type, $level, $sizetype,
                $size, $filters
        ) {
                $formDescriptor = [
index 2770bc5..00bfba9 100644 (file)
@@ -112,7 +112,7 @@ class SpecialProtectedtitles extends SpecialPage {
         * @return string
         * @private
         */
-       function showOptions( $namespace, $type = 'edit', $level ) {
+       function showOptions( $namespace, $type, $level ) {
                $formDescriptor = [
                        'namespace' => [
                                'class' => 'HTMLSelectNamespace',
index 3b69698..0d4b5ab 100644 (file)
@@ -44,8 +44,8 @@ class ProtectedPagesPager extends TablePager {
         * @param bool $noredirect
         * @param LinkRenderer $linkRenderer
         */
-       function __construct( $form, $conds = [], $type, $level, $namespace,
-               $sizetype = '', $size = 0, $indefonly = false, $cascadeonly = false, $noredirect = false,
+       function __construct( $form, $conds, $type, $level, $namespace,
+               $sizetype, $size, $indefonly, $cascadeonly, $noredirect,
                LinkRenderer $linkRenderer
        ) {
                $this->mForm = $form;
index 8f172f8..ed437be 100644 (file)
@@ -26,7 +26,7 @@ class ProtectedTitlesPager extends AlphabeticPager {
 
        public $mForm, $mConds;
 
-       function __construct( $form, $conds = [], $type, $level, $namespace,
+       function __construct( $form, $conds, $type, $level, $namespace,
                $sizetype = '', $size = 0
        ) {
                $this->mForm = $form;