Issue #3219513 by quietone, smustgrave, Vidushi Mehta, sourabhjain, rpayanm, longwave, xjm, dww, catch: Fix spelling for words used once, beginning with 'n' -> 'p', inclusive

core/lib/Drupal/Component/EventDispatcher/ContainerAwareEventDispatcher.php

@@ -17,7 +17,7 @@ use Symfony\Component\EventDispatcher\EventSubscriberInterface;
  *   <dt>Faster instantiation of the event dispatcher service</dt>
  *   <dd>
  *     Instead of calling <code>addSubscriberService</code> once for each
- *     subscriber, a precompiled array of listener definitions is passed
+ *     subscriber, a pre-compiled array of listener definitions is passed
  *     directly to the constructor. This is faster by roughly an order of
  *     magnitude. The listeners are collected and prepared using a compiler
  *     pass.

core/lib/Drupal/Core/Database/StatementPrefetch.php

@@ -8,7 +8,7 @@ use Drupal\Core\Database\Event\StatementExecutionEndEvent;
 use Drupal\Core\Database\Event\StatementExecutionStartEvent;
 
 /**
- * An implementation of StatementInterface that prefetches all data.
+ * An implementation of StatementInterface that pre-fetches all data.
  *
  * This class behaves very similar to a StatementWrapper of a \PDOStatement
  * but as it always fetches every row it is possible to manipulate those

core/lib/Drupal/Core/EventSubscriber/ActiveLinkResponseFilter.php

@@ -12,6 +12,7 @@ use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 use Symfony\Component\HttpKernel\Event\ResponseEvent;
 use Symfony\Component\HttpKernel\KernelEvents;
 
+// cspell:ignore noemptytag
 /**
  * Subscribes to filter HTML responses, to set the 'is-active' class on links.
  *

core/lib/Drupal/Core/Installer/Form/SiteConfigureForm.php

@@ -293,7 +293,7 @@ class SiteConfigureForm extends ConfigFormBase {
       }
     }
 
-    // We precreated user 1 with placeholder values. Let's save the real values.
+    // We created user 1 with placeholder values. Let's save the real values.
     $account = $this->userStorage->load(1);
     $account->init = $account->mail = $account_values['mail'];
     $account->roles = $account->getRoles();

core/lib/Drupal/Core/Routing/RoutePreloader.php

@@ -11,7 +11,7 @@ use Symfony\Component\HttpKernel\KernelEvents;
 use Symfony\Component\Routing\Route;
 
 /**
- * Defines a class which preloads non-admin routes.
+ * Defines a class that can pre-load non-admin routes.
  *
  * On an actual site we want to avoid too many database queries so we build a
  * list of all routes which most likely appear on the actual site, which are all

core/lib/Drupal/Core/Session/SessionManager.php

@@ -105,7 +105,7 @@ class SessionManager extends NativeSessionStorage implements SessionManagerInter
       // If a session cookie exists, initialize the session. Otherwise the
       // session is only started on demand in save(), making
       // anonymous users not use a session cookie unless something is stored in
-      // $_SESSION. This allows HTTP proxies to cache anonymous pageviews.
+      // $_SESSION. This allows HTTP proxies to cache anonymous page views.
       $result = $this->startNow();
     }
 

core/misc/cspell/dictionary.txt

@@ -730,7 +730,6 @@ narf
 navs
 nbaz
 nbchoices
-ndelay
 ndocs
 nearbykeyword
 necolas
@@ -753,20 +752,15 @@ noadd
 noblanks
 nocache
 nocase
-nocdata
 nocookie
 nodelist
 nodir
 nodo
-noemptytag
-noevent
 nofields
 nojs
 nolink
 nomask
-nonamespace
 nonconfigurable
-nonexistingfilename
 nonoptional
 noquotes
 noschema
@@ -774,7 +768,6 @@ nosniff
 nostart
 notag
 notexisting
-nothere
 notnull
 notsimpletest
 nourriture
@@ -789,7 +782,6 @@ oembed
 officedocument
 olivero
 olivero's
-omitscript
 omittable
 onecol
 oneplusfourgrid
@@ -811,13 +803,10 @@ overridable
 overrider
 overrider's
 overriders
-overridetest
 owasp
 oypa
-pageable
 pagecache
 pagetop
-pageviews
 pagina
 paramconverter
 parens
@@ -844,7 +833,6 @@ phpserialize
 phpspec
 phpstan
 phpunit
-phpunit's
 pianura
 pickable
 picturefill
@@ -871,12 +859,9 @@ postorder
 postprocess
 postupdate
 poweredby
-powriter
 prebar
 precache
-preconfigure
 preconfigured
-precreated
 predelete
 prefixsqlite
 prefoo

core/modules/language/tests/language_config_override_test/language_config_override_test.info.yml

@@ -1,4 +1,4 @@
-name: 'Language config overridetest'
+name: 'Language config override test'
 type: module
 description: 'Support module for the language config override test.'
 package: Testing

core/modules/media/src/OEmbed/ResourceFetcher.php

@@ -8,6 +8,8 @@ use GuzzleHttp\ClientInterface;
 use GuzzleHttp\Exception\TransferException;
 use GuzzleHttp\RequestOptions;
 
+// cspell:ignore nocdata
+
 /**
  * Fetches and caches oEmbed resources.
  */

core/modules/media/src/OEmbed/UrlResolver.php

@@ -9,6 +9,8 @@ use Drupal\Core\Extension\ModuleHandlerInterface;
 use GuzzleHttp\ClientInterface;
 use GuzzleHttp\Exception\TransferException;
 
+// cspell:ignore omitscript
+
 /**
  * Converts oEmbed media URLs into endpoint-specific resource URLs.
  */

core/modules/migrate_drupal/src/Plugin/migrate/EntityReferenceTranslationDeriver.php

@@ -189,7 +189,7 @@ class EntityReferenceTranslationDeriver extends DeriverBase implements Container
             // If this is an existing derivative, it means this bundle has more
             // than one entity reference field. In that case, we only want to
             // add the field to the process pipeline and add it to
-            // overwrite_properties so it can be over written.
+            // overwrite_properties so it can be overwritten.
             $this->derivatives[$derivative_key]['process'] += $derivative['process'];
             $this->derivatives[$derivative_key]['destination']['overwrite_properties'] += $derivative['destination']['overwrite_properties'];
           }

core/modules/syslog/src/Logger/SysLog.php

@@ -7,6 +7,8 @@ use Drupal\Core\Logger\LogMessageParserInterface;
 use Drupal\Core\Logger\RfcLoggerTrait;
 use Psr\Log\LoggerInterface;
 
+// cspell:ignore ndelay
+
 /**
  * Redirects logging messages to syslog.
  */

core/modules/system/tests/modules/pager_test/src/Controller/PagerTestController.php

@@ -39,7 +39,7 @@ class PagerTestController extends ControllerBase implements TrustedCallbackInter
   }
 
   /**
-   * Builds a render array for a pageable test table.
+   * Builds a render array for a multi-page test table.
    *
    * @param int $element
    *   The pager element to be used for paging.

core/modules/update/update.compare.inc

@@ -462,7 +462,7 @@ function update_calculate_project_update_status(&$project_data, $available) {
     }
 
     // Look for the 'recommended' version if we haven't found it yet (see
-    // phpdoc at the top of this function for the definition).
+    // PHPDoc at the top of this function for the definition).
     if (!isset($project_data['recommended'])
         && $release_major_version == $target_major) {
       if ($recommended_version_without_extra !== $release_version_without_extra) {

core/modules/views/src/Plugin/views/style/Table.php

@@ -170,7 +170,7 @@ class Table extends StylePluginBase implements CacheableDependencyInterface {
     if ($fields === NULL) {
       $fields = $this->displayHandler->getOption('fields');
     }
-    // Preconfigure the sanitized array so that the order is retained.
+    // Pre-configure the sanitized array so that the order is retained.
     foreach ($fields as $field => $info) {
       // Set to itself so that if it isn't touched, it gets column
       // status automatically.

core/tests/Drupal/KernelTests/Core/Config/ConfigCRUDTest.php

@@ -190,7 +190,7 @@ class ConfigCRUDTest extends KernelTestBase {
    */
   public function testNameValidation() {
     // Verify that an object name without namespace causes an exception.
-    $name = 'nonamespace';
+    $name = 'no_namespace';
     try {
       $this->config($name)->save();
       $this->fail('Expected ConfigNameException was thrown for a name without a namespace.');

core/tests/Drupal/KernelTests/Core/Database/TaggingTest.php

@@ -123,7 +123,7 @@ class TaggingTest extends DatabaseTestBase {
     $return = $query->getMetaData('test');
     $this->assertEquals($data, $return, 'Correct metadata returned.');
 
-    $return = $query->getMetaData('nothere');
+    $return = $query->getMetaData('not_here');
     $this->assertNull($return, 'Non-existent key returned NULL.');
   }
 

core/tests/Drupal/KernelTests/Core/File/ReadOnlyStreamWrapperTest.php

@@ -78,7 +78,7 @@ class ReadOnlyStreamWrapperTest extends FileTestBase {
     // Attempt to close the stream.  (Suppress errors, as fclose triggers fflush.)
     $this->assertTrue(fclose($handle), 'Able to close file using the read_only stream wrapper.');
     // Test the rename() function
-    $this->assertFalse(@rename($uri, $this->scheme . '://newname.txt'), 'Unable to rename files using the read-only stream wrapper.');
+    $this->assertFalse(@rename($uri, $this->scheme . '://new_name.txt'), 'Unable to rename files using the read-only stream wrapper.');
     // Test the unlink() function
     $this->assertTrue(@$file_system->unlink($uri), 'Able to unlink file using read-only stream wrapper.');
     $this->assertFileExists($filepath);

core/tests/Drupal/KernelTests/Core/File/ScanDirectoryTest.php

@@ -78,7 +78,7 @@ class ScanDirectoryTest extends FileTestBase {
   public function testOptionCallback() {
 
     // When nothing is matched nothing should be passed to the callback.
-    $all_files = $this->fileSystem->scanDirectory($this->path, '/^NONEXISTINGFILENAME/', ['callback' => 'file_test_file_scan_callback']);
+    $all_files = $this->fileSystem->scanDirectory($this->path, '/^NON-EXISTING-FILENAME/', ['callback' => 'file_test_file_scan_callback']);
     $this->assertCount(0, $all_files, 'No files were found.');
     $results = file_test_file_scan_callback();
     file_test_file_scan_callback_reset();

core/tests/Drupal/Tests/Component/EventDispatcher/ContainerAwareEventDispatcherTest.php

@@ -310,7 +310,7 @@ class ContainerAwareEventDispatcherTest extends TestCase {
     $this->dispatcher->dispatch(new Event(), self::PREFOO);
     $this->assertTrue($this->listener->preFooInvoked);
     $this->assertFalse($this->listener->postFooInvoked);
-    $this->assertInstanceOf(Event::class, $this->dispatcher->dispatch(new Event(), 'noevent'));
+    $this->assertInstanceOf(Event::class, $this->dispatcher->dispatch(new Event(), 'no_event'));
     $this->assertInstanceOf(Event::class, $this->dispatcher->dispatch(new Event(), self::PREFOO));
     // Any kind of object can be dispatched, not only instances of Event.
     $this->assertInstanceOf(\stdClass::class, $this->dispatcher->dispatch(new \stdClass(), self::PREFOO));

core/tests/Drupal/Tests/Component/Gettext/PoStreamWriterTest.php

@@ -44,7 +44,7 @@ class PoStreamWriterTest extends TestCase {
     $this->poWriter->setHeader($poHeader->reveal());
 
     $root = vfsStream::setup();
-    $this->poFile = new vfsStreamFile('powriter.po');
+    $this->poFile = new vfsStreamFile('poWriter.po');
     $root->addChild($this->poFile);
   }
 

core/tests/bootstrap.php

@@ -118,8 +118,8 @@ if (!defined('PHPUNIT_COMPOSER_INSTALL')) {
  * Populate class loader with additional namespaces for tests.
  *
  * We run this in a function to avoid setting the class loader to a global
- * that can change. This change can cause unpredictable false positives for
+ * that can change. This change can cause unpredictable false positives for the
- * phpunit's global state change watcher. The class loader can be retrieved from
+ * PHPUnit global state change watcher. The class loader can be retrieved from
  * composer at any time by requiring autoload.php.
  */
 function drupal_phpunit_populate_class_loader() {