Fix some typos

Bug: T201491
Change-Id: I5c9408c262f09c936525f35abfacfa92a193b791

docs/extension.schema.v1.json

@@ -1178,7 +1178,7 @@
 						"properties": {
 							"class": {
 								"type": "string",
-								"description": "The fully-qualified class name of a class implemeting Wikimedia\\Parsoid\\Ext\\ExtensionModule. This should be omitted if a factory is specified."
+								"description": "The fully-qualified class name of a class implementing Wikimedia\\Parsoid\\Ext\\ExtensionModule. This should be omitted if a factory is specified."
 							}
 						},
 						"required": [ "class" ]

docs/extension.schema.v2.json

@@ -1362,7 +1362,7 @@
 						"properties": {
 							"class": {
 								"type": "string",
-								"description": "The fully-qualified class name of a class implemeting Wikimedia\\Parsoid\\Ext\\ExtensionModule. This should be omitted if a factory is specified."
+								"description": "The fully-qualified class name of a class implementing Wikimedia\\Parsoid\\Ext\\ExtensionModule. This should be omitted if a factory is specified."
 							}
 						},
 						"required": [ "class" ]

includes/Hook/EditPageNoSuchSectionHook.php

@@ -13,7 +13,7 @@ use MediaWiki\EditPage\EditPage;
  */
 interface EditPageNoSuchSectionHook {
 	/**
-	 * This hook is called when a section edit request is given for an non-existent section.
+	 * This hook is called when a section edit request is given for a non-existent section.
 	 *
 	 * @since 1.35
 	 *

includes/OutputPage.php

@@ -2924,7 +2924,7 @@ class OutputPage extends ContextSource {
 	/**
 	 * Prepare this object to display an error page; disable caching and
 	 * indexing, clear the current text and redirect, set the page's title
-	 * and optionally an custom HTML title (content of the "<title>" tag).
+	 * and optionally a custom HTML title (content of the "<title>" tag).
 	 *
 	 * @param string|Message $pageTitle Will be passed directly to setPageTitle()
 	 * @param string|Message|false $htmlTitle Will be passed directly to setHTMLTitle();

includes/ServiceWiring.php

@@ -1,6 +1,6 @@
 <?php
 /**
- * Service implemenations for %MediaWiki core.
+ * Service implementations for %MediaWiki core.
  *
  * This file returns the array loaded by the MediaWikiServices class
  * for use through `MediaWiki\MediaWikiServices::getInstance()`

includes/auth/AuthManager.php

@@ -508,7 +508,7 @@ class AuthManager implements LoggerAwareInterface {
 				$req->returnToUrl = $state['returnToUrl'];
 			}
 
-			// Step 1: Choose an primary authentication provider, and call it until it succeeds.
+			// Step 1: Choose a primary authentication provider, and call it until it succeeds.
 
 			if ( $state['primary'] === null ) {
 				// We haven't picked a PrimaryAuthenticationProvider yet

includes/context/RequestContext.php

@@ -596,7 +596,7 @@ class RequestContext implements IContextSource, MutableContext {
 	}
 
 	/**
-	 * Import an client IP address, HTTP headers, user ID, and session ID
+	 * Import a client IP address, HTTP headers, user ID, and session ID
 	 *
 	 * This sets the current session, $wgUser, and $wgRequest from $params.
 	 * Once the return value falls out of scope, the old context is restored.

includes/deferred/DeferredUpdates.php

@@ -157,7 +157,7 @@ class DeferredUpdates {
 	 * Note that it is rarely the case that this method should be called outside of a few
 	 * select entry points. For simplicity, that kind of recursion is discouraged. Recursion
 	 * cannot happen if an explicit transaction round is active, which limits usage to updates
-	 * with TRX_ROUND_ABSENT that do not leave open an transactions round of their own during
+	 * with TRX_ROUND_ABSENT that do not leave open any transactions round of their own during
 	 * the call to this method.
 	 *
 	 * In the less-common case of this being called within an in-progress DeferrableUpdate,

includes/htmlform/HTMLForm.php

@@ -1904,7 +1904,7 @@ class HTMLForm extends ContextSource {
 	}
 
 	/**
-	 * Wraps the given $section into an user-visible fieldset.
+	 * Wraps the given $section into a user-visible fieldset.
 	 * @stable to override
 	 *
 	 * @param string $legend Legend text for the fieldset
@@ -2181,7 +2181,7 @@ class HTMLForm extends ContextSource {
 		// meaning that getLocalURL() would return something like "index.php?title=...".
 		// As browser remove the query string before submitting GET forms,
 		// it means that the title would be lost. In such case use script path instead
-		// and put title in an hidden field (see getHiddenFields()).
+		// and put title in a hidden field (see getHiddenFields()).
 		if ( str_contains( $articlePath, '?' ) && $this->getMethod() === 'get' ) {
 			return $this->getConfig()->get( MainConfigNames::Script );
 		}

includes/libs/MappedIterator.php

@@ -36,7 +36,7 @@ class MappedIterator extends FilterIterator {
 	protected $rewound = false; // boolean; whether rewind() has been called
 
 	/**
-	 * Build an new iterator from a base iterator by having the former wrap the
+	 * Build a new iterator from a base iterator by having the former wrap the
 	 * later, returning the result of "value" callback for each current() invocation.
 	 * The callback takes the result of current() on the base iterator as an argument.
 	 * The keys of the base iterator are reused verbatim.

includes/libs/objectcache/wancache/WANObjectCache.php

@@ -1663,7 +1663,7 @@ class WANObjectCache implements
 		}
 
 		$isKeyTombstoned = ( $curState[self::RES_TOMB_AS_OF] !== null );
-		// Use the interim key as an temporary alternative if the key is tombstoned
+		// Use the interim key as a temporary alternative if the key is tombstoned
 		if ( $isKeyTombstoned ) {
 			$volState = $this->getInterimValue( $key, $minAsOf, $startTime, $touchedCb );
 			$volValue = $volState[self::RES_VALUE];

includes/libs/rdbms/database/DatabaseFactory.php

@@ -160,7 +160,7 @@ class DatabaseFactory {
 		// an array can be used, keyed by extension name. In case of an array, the
 		// optional 'driver' parameter can be used to force a specific driver. Otherwise,
 		// we auto-detect the first available driver. For types without built-in support,
-		// an class named "Database<Type>" us used, eg. DatabaseFoo for type 'foo'.
+		// a class named "Database<Type>" is used, eg. DatabaseFoo for type 'foo'.
 		static $builtinTypes = [
 			'mysql' => [ 'mysqli' => DatabaseMysqli::class ],
 			'sqlite' => DatabaseSqlite::class,

includes/libs/rdbms/database/IReadableDatabase.php

@@ -320,7 +320,7 @@ interface IReadableDatabase extends ISQLPlatform, DbQuoter, IDatabaseFlags {
 	 *   - GROUP BY: May be either an SQL fragment string naming a field or
 	 *     expression to group by, or an array of such SQL fragments.
 	 *
-	 *   - HAVING: May be either an string containing a HAVING clause or an array of
+	 *   - HAVING: May be either a string containing a HAVING clause or an array of
 	 *     conditions building the HAVING clause. If an array is given, the conditions
 	 *     constructed from each element are combined with AND.
 	 *

includes/libs/rdbms/querybuilder/SelectQueryBuilder.php

@@ -425,7 +425,7 @@ class SelectQueryBuilder extends JoinGroupBase {
 	}
 
 	/**
-	 * Add a HAVING clause. May be either an string containing a HAVING clause
+	 * Add a HAVING clause. May be either a string containing a HAVING clause
 	 * or an array of conditions building the HAVING clause. If an array is
 	 * given, the conditions constructed from each element are combined with
 	 * AND.

includes/linker/LinkTargetLookup.php

@@ -43,7 +43,7 @@ interface LinkTargetLookup {
 	public function newLinkTargetFromRow( stdClass $row ): LinkTarget;
 
 	/**
-	 * Find an link target by $id.
+	 * Find a link target by $id.
 	 *
 	 * @param int $linkTargetId
 	 * @return LinkTarget|null Returns null if no link target with this $linkTargetId exists in the database.

includes/page/ImagePage.php

@@ -722,7 +722,7 @@ EOT
 	}
 
 	/**
-	 * Creates an thumbnail of specified size and returns an HTML link to it
+	 * Creates a thumbnail of specified size and returns an HTML link to it
 	 * @param array $params Scaler parameters
 	 * @param int $width
 	 * @param int $height

includes/pager/ReverseChronologicalPager.php

@@ -230,7 +230,7 @@ abstract class ReverseChronologicalPager extends IndexPager {
 	 * Core logic of determining the offset timestamp such that we can get all items with
 	 * a timestamp up to the specified parameters. Given parameters for a day up to which to get
 	 * items, this function finds the timestamp of the day just after the end of the range for use
-	 * in an database strict inequality filter.
+	 * in a database strict inequality filter.
 	 *
 	 * This is separate from getDateCond so we can use this logic in other places, such as in
 	 * RangeChronologicalPager, where this function is used to convert year/month/day filter options

includes/parser/Parsoid/Config/DataAccess.php

@@ -392,7 +392,7 @@ class DataAccess extends IDataAccess {
 		$wikitext = $parser->getStripState()->unstripBoth( $wikitext );
 
 		// XXX: Ideally we will eventually have the legacy parser use our
-		// ContentMetadataCollector instead of having an new ParserOutput
+		// ContentMetadataCollector instead of having a new ParserOutput
 		// created (implicitly in ::prepareParser()/Parser::resetOutput() )
 		// which we then have to manually merge.
 		$out = $parser->getOutput();

includes/specials/SpecialRenameuser.php

@@ -230,7 +230,7 @@ class SpecialRenameuser extends SpecialPage {
 			return;
 		}
 
-		// If this user is renaming his/herself, make sure that MovePage::move()
+		// If this user is renaming themself, make sure that MovePage::move()
 		// doesn't make a bunch of null move edits under the old name!
 		if ( $performer->getId() === $uid ) {
 			$performer->setName( $newTitle->getText() );

includes/title/Title.php

@@ -3626,7 +3626,7 @@ class Title implements LinkTarget, PageIdentity, IDBAccessObject {
 	}
 
 	/**
-	 * Get the default (plain) message contents for an page that overrides an
+	 * Get the default (plain) message contents for a page that overrides an
 	 * interface message key.
 	 *
 	 * Primary use cases:

maintenance/includes/BackupDumper.php

@@ -377,7 +377,7 @@ abstract class BackupDumper extends Maintenance {
 
 	/**
 	 * Initialise starting time and maximum revision count.
-	 * We'll make ETA calculations based an progress, assuming relatively
+	 * We'll make ETA calculations based on progress, assuming relatively
 	 * constant per-revision rate.
 	 * @param int $history WikiExporter::CURRENT or WikiExporter::FULL
 	 */

maintenance/mcc.php

@@ -83,7 +83,7 @@ class Mcc extends Maintenance {
 			// process command
 			switch ( $command ) {
 				case 'help':
-					// show an help message
+					// show a help message
 					print $this->mccGetHelp( array_shift( $args ) );
 					break;
 

resources/lib/intersection-observer/README.md

@@ -171,7 +171,7 @@ Legacy support is also possible in very old browsers by including a shim for ES5
 <script src="https://polyfill.io/v3/polyfill.min.js?features=IntersectionObserver"></script>
 ```
 
-With these polyfills, `IntersectionObserver` has been tested an known to work in the following browsers:
+With these polyfills, `IntersectionObserver` has been tested and known to work in the following browsers:
 
 <table>
   <tr>

resources/lib/jquery.i18n/src/jquery.i18n.messagestore.js

@@ -74,7 +74,7 @@
 			}
 
 			if ( locale ) {
-				// source is an key-value pair of messages for given locale
+				// source is a key-value pair of messages for given locale
 				messageStore.set( locale, source );
 
 				return $.Deferred().resolve();

resources/lib/jquery.ui/jquery.ui.effect.js

@@ -173,7 +173,7 @@ function clamp( value, prop, allowEmpty ) {
 		return (allowEmpty || !prop.def) ? null : prop.def;
 	}
 
-	// ~~ is an short way of doing floor for positive numbers
+	// ~~ is a short way of doing floor for positive numbers
 	value = type.floor ? ~~value : parseFloat( value );
 
 	// IE will pass in empty strings as value for alpha,

resources/src/mediawiki.rcfilters/Controller.js

@@ -957,7 +957,7 @@ Controller.prototype.updateNumericPreference = function ( prefName, newValue ) {
 
 /**
  * Synchronize the URL with the current state of the filters
- * without adding an history entry.
+ * without adding a history entry.
  */
 Controller.prototype.replaceUrl = function () {
 	this.uriProcessor.updateURL();

resources/src/startup/mediawiki.loader.js

@@ -64,7 +64,7 @@
 	 *
 	 * @private
 	 * @param {string} str String to hash
-	 * @return {string} hash as an five-character base 36 string
+	 * @return {string} hash as a five-character base 36 string
 	 */
 	function fnv132( str ) {
 		var hash = 0x811C9DC5;

tests/parser/TestRecorder.php

@@ -25,7 +25,7 @@ use Wikimedia\Parsoid\ParserTests\TestMode as ParserTestMode;
 /**
  * Interface to record parser test results.
  *
- * The TestRecorder is an class hierarchy to record the result of
+ * The TestRecorder is a class hierarchy to record the result of
  * MediaWiki parser tests. One should call start() before running the
  * full parser tests and end() once all the tests have been finished.
  * After each test, you should use record() to keep track of your tests

tests/phpunit/includes/language/LanguageClassesTestCase.php

@@ -8,7 +8,7 @@ use MediaWiki\MediaWikiServices;
  * This is intended for the MediaWiki language class tests under
  * tests/phpunit/includes/languages.
  *
- * Before each tests, a new language object is build which you
+ * Before each tests, a new language object is built which you
  * can retrieve in your test using the $this->getLang() method:
  *
  * @par Using the crafted language object:
@@ -62,7 +62,7 @@ abstract class LanguageClassesTestCase extends MediaWikiIntegrationTestCase {
 			$lang = 'en';
 			wfDebug(
 				__METHOD__ . ' could not extract a language name '
-					. 'out of ' . static::class . " failling back to 'en'"
+					. 'out of ' . static::class . ", falling back to 'en'"
 			);
 		}
 		$this->languageObject = MediaWikiServices::getInstance()->getLanguageFactory()
@@ -70,7 +70,7 @@ abstract class LanguageClassesTestCase extends MediaWikiIntegrationTestCase {
 	}
 
 	/**
-	 * Delete the internal language object so each test start
+	 * Delete the internal language object so each test starts
 	 * out with a fresh language instance.
 	 */
 	protected function tearDown(): void {

tests/phpunit/includes/libs/serialization/SerializationTestUtils.php

@@ -125,7 +125,7 @@ class SerializationTestUtils {
 	}
 
 	/**
-	 * Get an test instance of $class for test case named $testCaseName.
+	 * Get a test instance of $class for test case named $testCaseName.
 	 * @param string $testCaseName
 	 * @return object
 	 */

tests/phpunit/maintenance/DumpAsserter.php

@@ -314,7 +314,7 @@ class DumpAsserter {
 	}
 
 	/**
-	 * Asserts that the xml reader is at an closing element of given name, and optionally
+	 * Asserts that the xml reader is at a closing element of given name, and optionally
 	 * skips past it. If the reader is at a whitespace element, the whitespace is
 	 * skipped first.
 	 *

tests/phpunit/mocks/MockServiceDependenciesTrait.php

@@ -14,7 +14,7 @@ use ReflectionParameter;
 trait MockServiceDependenciesTrait {
 
 	/**
-	 * Construct an new instance of $serviceClass with all constructor arguments
+	 * Construct a new instance of $serviceClass with all constructor arguments
 	 * mocked. $parameterOverrides allows to provide some constructor argument.
 	 *
 	 * @param string $serviceClass

tests/phpunit/unit/includes/SiteConfigurationTest.php

@@ -179,7 +179,7 @@ class SiteConfigurationTest extends \MediaWikiUnitTestCase {
 		$this->assertEquals(
 			'wiki',
 			$this->mConf->get( 'SimpleKey', 'wiki', 'wiki' ),
-			'get(): simple setting on an suffix'
+			'get(): simple setting on a suffix'
 		);
 		$this->assertEquals(
 			'wiki',
@@ -212,12 +212,12 @@ class SiteConfigurationTest extends \MediaWikiUnitTestCase {
 		$this->assertEquals(
 			'wiki',
 			$this->mConf->get( 'Fallback', 'wiki', 'wiki' ),
-			'get(): fallback setting on an suffix'
+			'get(): fallback setting on a suffix'
 		);
 		$this->assertEquals(
 			'wiki',
 			$this->mConf->get( 'Fallback', 'wiki', 'wiki', [], [ 'tag' ] ),
-			'get(): fallback setting on an suffix (with wiki tag)'
+			'get(): fallback setting on a suffix (with wiki tag)'
 		);
 		$this->assertEquals(
 			'wiki',
@@ -266,12 +266,12 @@ class SiteConfigurationTest extends \MediaWikiUnitTestCase {
 		$this->assertEquals(
 			[ 'wiki' => 'wiki' ] + $common,
 			$this->mConf->get( 'MergeIt', 'wiki', 'wiki' ),
-			'get(): merging setting on an suffix'
+			'get(): merging setting on a suffix'
 		);
 		$this->assertEquals(
 			[ 'wiki' => 'wiki' ] + $commonTag,
 			$this->mConf->get( 'MergeIt', 'wiki', 'wiki', [], [ 'tag' ] ),
-			'get(): merging setting on an suffix (with tag)'
+			'get(): merging setting on a suffix (with tag)'
 		);
 		$this->assertEquals(
 			$common,
@@ -332,7 +332,7 @@ class SiteConfigurationTest extends \MediaWikiUnitTestCase {
 		$this->assertEquals(
 			' wiki wiki',
 			$this->mConf->get( 'WithParams', 'wiki', 'wiki' ),
-			'get(): parameter replacement on an suffix'
+			'get(): parameter replacement on a suffix'
 		);
 		$this->assertEquals(
 			'es wiki eswiki',

tests/phpunit/unit/includes/json/FormatJsonTest.php

@@ -109,7 +109,7 @@ class FormatJsonTest extends MediaWikiUnitTestCase {
 		$this->assertNotEquals(
 			'\ud840\udc00',
 			strtolower( FormatJson::encode( "\u{20000}" ) ),
-			'Test encoding an broken json_encode character (U+20000)'
+			'Test encoding of broken json_encode character (U+20000)'
 		);
 	}
 

thumb.php

@@ -564,7 +564,7 @@ function wfExtractThumbRequestInfo( $thumbRel ) {
 	// Check if this is a thumbnail of an original in the local file repo
 	if ( preg_match( "!^((archive/)?$hashDirReg([^/]*)/([^/]*))$!", $thumbRel, $m ) ) {
 		[ /*all*/, $rel, $archOrTemp, $filename, $thumbname ] = $m;
-	// Check if this is a thumbnail of an temp file in the local file repo
+	// Check if this is a thumbnail of a temp file in the local file repo
 	} elseif ( preg_match( "!^(temp/)($hashDirReg([^/]*)/([^/]*))$!", $thumbRel, $m ) ) {
 		[ /*all*/, $archOrTemp, $rel, $filename, $thumbname ] = $m;
 	} else {