Merge "Upgrade moment from 2.19.3 to 2.24.0"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Sun, 24 Feb 2019 00:38:34 +0000 (00:38 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Sun, 24 Feb 2019 00:38:34 +0000 (00:38 +0000)
21 files changed:
RELEASE-NOTES-1.33
docs/extension.schema.v1.json
docs/extension.schema.v2.json
includes/parser/Sanitizer.php
includes/resourceloader/ResourceLoaderFileModule.php
tests/phpunit/includes/DiffHistoryBlobTest.php
tests/phpunit/includes/FauxResponseTest.php
tests/phpunit/includes/HtmlTest.php
tests/phpunit/includes/MultiHttpClientTest.php
tests/phpunit/includes/htmlform/HTMLAutoCompleteSelectFieldTest.php
tests/phpunit/includes/htmlform/HTMLCheckMatrixTest.php
tests/phpunit/includes/installer/OracleInstallerTest.php
tests/phpunit/includes/libs/IEUrlExtensionTest.php
tests/phpunit/includes/libs/ProcessCacheLRUTest.php
tests/phpunit/includes/site/DBSiteStoreTest.php
tests/phpunit/includes/site/MediaWikiSiteTest.php
tests/phpunit/includes/site/SiteExporterTest.php
tests/phpunit/includes/site/SiteImporterTest.php
tests/phpunit/includes/site/SiteListTest.php
tests/phpunit/includes/site/SiteTest.php
tests/phpunit/maintenance/backupPrefetchTest.php

index e1f17c0..4a923c5 100644 (file)
@@ -263,6 +263,8 @@ because of Phabricator reports.
 * The ChangeList::insertArticleLink() method, that was deprecated in 1.27, has
   been removed.
 * MessageBlobStore::__construct() now requires its $rl parameter.
+* Second parameter to Sanitizer::escapeIdReferenceList() (deprecated in 1.31)
+  has been removed.
 
 === Deprecations in 1.33 ===
 * The configuration option $wgUseESI has been deprecated, and is expected
index f9b71fb..8cd4e71 100644 (file)
                                                        "properties": {
                                                                "class": {
                                                                        "type": "string",
-                                                                       "pattern": "^((?!ResourceLoader(File|Image)Module).)*$"
+                                                                       "pattern": "^(?!ResourceLoader(File|Image|Wiki)Module).*$"
                                                                }
                                                        },
                                                        "required": ["class"]
index 22edac0..1d64095 100644 (file)
                                                        "properties": {
                                                                "class": {
                                                                        "type": "string",
-                                                                       "pattern": "^((?!ResourceLoader(File|Image)Module).)*$"
+                                                                       "pattern": "^(?!ResourceLoader(File|Image|Wiki)Module).*$"
                                                                }
                                                        },
                                                        "required": ["class"]
index f8c3bc2..0aa58c1 100644 (file)
@@ -1375,20 +1375,14 @@ class Sanitizer {
 
        /**
         * Given a string containing a space delimited list of ids, escape each id
-        * to match ids escaped by the escapeId() function.
-        *
-        * @todo remove $options completely in 1.32
+        * to match ids escaped by the escapeIdForAttribute() function.
         *
         * @since 1.27
         *
         * @param string $referenceString Space delimited list of ids
-        * @param string|array $options Deprecated and does nothing.
         * @return string
         */
-       static function escapeIdReferenceList( $referenceString, $options = [] ) {
-               if ( $options ) {
-                       wfDeprecated( __METHOD__ . ' with $options', '1.31' );
-               }
+       public static function escapeIdReferenceList( $referenceString ) {
                # Explode the space delimited list string into an array of tokens
                $references = preg_split( '/\s+/', "{$referenceString}", -1, PREG_SPLIT_NO_EMPTY );
 
index 6680777..2848871 100644 (file)
@@ -1093,15 +1093,10 @@ class ResourceLoaderFileModule extends ResourceLoaderModule {
                        if ( is_string( $fileInfo ) ) {
                                $fileInfo = [ 'name' => $fileInfo, 'file' => $fileInfo ];
                        } elseif ( !isset( $fileInfo['name'] ) ) {
-                               // Backwards compatibility
-                               if ( !is_numeric( $alias ) ) {
-                                       $fileInfo['name'] = $alias;
-                               } else {
-                                       $msg = __METHOD__ . ": invalid package file definition for module " .
-                                               "\"{$this->getName()}\": 'name' key is required when value is not a string";
-                                       wfDebugLog( 'resourceloader', $msg );
-                                       throw new MWException( $msg );
-                               }
+                               $msg = __METHOD__ . ": invalid package file definition for module " .
+                                       "\"{$this->getName()}\": 'name' key is required when value is not a string";
+                               wfDebugLog( 'resourceloader', $msg );
+                               throw new MWException( $msg );
                        }
 
                        // Infer type from alias if needed
index c267a30..b75862e 100644 (file)
@@ -15,7 +15,6 @@ class DiffHistoryBlobTest extends MediaWikiTestCase {
        }
 
        /**
-        * Test for DiffHistoryBlob::xdiffAdler32()
         * @dataProvider provideXdiffAdler32
         * @covers DiffHistoryBlob::xdiffAdler32
         */
index eac56fb..8085bc7 100644 (file)
@@ -1,7 +1,5 @@
 <?php
 /**
- * Tests for the FauxResponse class
- *
  * Copyright @ 2011 Alexandre Emsenhuber
  *
  * This program is free software; you can redistribute it and/or modify
index 1d687e5..999e0bb 100644 (file)
@@ -212,7 +212,6 @@ class HtmlTest extends MediaWikiTestCase {
        }
 
        /**
-        * Test for Html::expandAttributes()
         * Please note it output a string prefixed with a space!
         * @covers Html::expandAttributes
         */
index 7073b71..1c7e62d 100644 (file)
@@ -1,8 +1,6 @@
 <?php
 
 /**
- * Tests for MultiHttpClient
- *
  * The urls herein are not actually called, because we mock the return results.
  *
  * @covers MultiHttpClient
index 99bea68..eaba22d 100644 (file)
@@ -1,7 +1,5 @@
 <?php
 /**
- * Unit tests for HTMLAutoCompleteSelectField
- *
  * @covers HTMLAutoCompleteSelectField
  */
 class HTMLAutoCompleteSelectFieldTest extends MediaWikiTestCase {
index 4089470..05c567d 100644 (file)
@@ -1,7 +1,6 @@
 <?php
 
 /**
- * Unit tests for the HTMLCheckMatrix
  * @covers HTMLCheckMatrix
  */
 class HTMLCheckMatrixTest extends MediaWikiTestCase {
index 2811a9c..e255089 100644 (file)
@@ -1,8 +1,6 @@
 <?php
 
 /**
- * Tests for OracleInstaller
- *
  * @group Database
  * @group Installer
  */
index 9ec660e..e04b2e2 100644 (file)
@@ -1,8 +1,5 @@
 <?php
 
-/**
- * Tests for IEUrlExtension::findIE6Extension
- */
 class IEUrlExtensionTest extends PHPUnit\Framework\TestCase {
 
        use MediaWikiCoversValidator;
index c9fa320..8e91e70 100644 (file)
@@ -1,8 +1,6 @@
 <?php
 
 /**
- * Test for ProcessCacheLRU class.
- *
  * Note that it uses the ProcessCacheLRUTestable class which extends some
  * properties and methods visibility. That class is defined at the end of the
  * file containing this class.
index da6e9f9..14ee15b 100644 (file)
@@ -3,8 +3,6 @@
 use MediaWiki\MediaWikiServices;
 
 /**
- * Tests for the DBSiteStore class.
- *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; either version 2 of the License, or
index d198462..a9732d1 100644 (file)
@@ -1,8 +1,6 @@
 <?php
 
 /**
- * Tests for the MediaWikiSite class.
- *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; either version 2 of the License, or
index db900da..97a43f8 100644 (file)
@@ -1,8 +1,6 @@
 <?php
 
 /**
- * Tests for the SiteExporter class.
- *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; either version 2 of the License, or
index bd95a50..dbdbd6f 100644 (file)
@@ -1,8 +1,6 @@
 <?php
 
 /**
- * Tests for the SiteImporter class.
- *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; either version 2 of the License, or
index a4a171c..3c4a4bf 100644 (file)
@@ -1,8 +1,6 @@
 <?php
 
 /**
- * Tests for the SiteList class.
- *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; either version 2 of the License, or
index ac5f956..9ad61b2 100644 (file)
@@ -1,8 +1,6 @@
 <?php
 
 /**
- * Tests for the Site class.
- *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; either version 2 of the License, or
index 8824c7a..747d2bf 100644 (file)
@@ -6,8 +6,6 @@ use BaseDump;
 use MediaWikiTestCase;
 
 /**
- * Tests for BaseDump
- *
  * @group Dump
  * @covers BaseDump
  */