Issue #3099583 by alexpott, klausi, neclimdul, heddn: Update coder to 8.3.7

composer.json

@@ -20,7 +20,7 @@
         "behat/mink-goutte-driver": "^1.2",
         "behat/mink-selenium2-driver": "1.3.x-dev",
         "composer/composer": "^1.9.1",
-        "drupal/coder": "^8.3.2",
+        "drupal/coder": "^8.3.7",
         "jcalderonzumba/gastonjs": "^1.0.2",
         "jcalderonzumba/mink-phantomjs-driver": "^0.3.1",
         "mikey179/vfsstream": "^1.6.8",

composer.lock

@@ -4,7 +4,7 @@
         "Read more about it at https://getcomposer.org/doc/01-basic-usage.md#installing-dependencies",
         "This file is @generated automatically"
     ],
-    "content-hash": "c6c500e6567b37ef3e100b285ab9910c",
+    "content-hash": "086a959b9c67661541137d16cbcce8e2",
     "packages": [
         {
             "name": "asm89/stack-cors",
@@ -4056,11 +4056,11 @@
         },
         {
             "name": "drupal/coder",
-            "version": "8.3.6",
+            "version": "8.3.7",
             "source": {
                 "type": "git",
                 "url": "https://git.drupalcode.org/project/coder.git",
-                "reference": "4337ddf58d28dbdee4e1367bf71ee13393ab9820"
+                "reference": "c11c2957653bdbfd68adc851692d094b43d39221"
             },
             "require": {
                 "ext-mbstring": "*",
@@ -4069,7 +4069,7 @@
                 "symfony/yaml": ">=2.0.5"
             },
             "require-dev": {
-                "phpunit/phpunit": ">=3.7 <6"
+                "phpunit/phpunit": "^4.0 || ^5.0 || ^6.0 || ^7.0"
             },
             "type": "phpcodesniffer-standard",
             "autoload": {
@@ -4089,7 +4089,7 @@
                 "phpcs",
                 "standards"
             ],
-            "time": "2019-08-09T09:27:26+00:00"
+            "time": "2019-12-07T16:00:28+00:00"
         },
         {
             "name": "fabpot/goutte",

composer/Metapackage/DevDependencies/composer.json

@@ -11,7 +11,7 @@
         "behat/mink-goutte-driver": "^1.2",
         "behat/mink-selenium2-driver": "1.4.0 | 1.3.1.1 | 1.3.x-dev",
         "composer/composer": "^1.9.1",
-        "drupal/coder": "^8.3.2",
+        "drupal/coder": "^8.3.7",
         "jcalderonzumba/gastonjs": "^1.0.2",
         "jcalderonzumba/mink-phantomjs-driver": "^0.3.1",
         "justinrainbow/json-schema": "^5.2",

composer/Metapackage/PinnedDevDependencies/composer.json

@@ -17,7 +17,7 @@
         "composer/spdx-licenses": "1.5.2",
         "composer/xdebug-handler": "1.3.3",
         "doctrine/instantiator": "1.0.5",
-        "drupal/coder": "8.3.6",
+        "drupal/coder": "8.3.7",
         "fabpot/goutte": "v3.2.3",
         "instaclick/php-webdriver": "1.4.6",
         "jcalderonzumba/gastonjs": "v1.0.2",

core/lib/Drupal/Component/Datetime/DateTimePlus.php

@@ -129,7 +129,8 @@ class DateTimePlus {
    * @param \DateTime $datetime
    *   A DateTime object.
    * @param array $settings
-   *   @see __construct()
+   *   (optional) A keyed array for settings, suitable for passing on to
+   *   __construct().
    *
    * @return static
    *   A new DateTimePlus object.
@@ -183,9 +184,11 @@ public static function createFromArray(array $date_parts, $timezone = NULL, $set
    * @param int $timestamp
    *   A UNIX timestamp.
    * @param mixed $timezone
-   *   @see __construct()
+   *   (optional) \DateTimeZone object, time zone string or NULL. See
+   *   __construct() for more details.
    * @param array $settings
-   *   @see __construct()
+   *   (optional) A keyed array for settings, suitable for passing on to
+   *   __construct().
    *
    * @return static
    *   A new DateTimePlus object.
@@ -211,11 +214,14 @@ public static function createFromTimestamp($timestamp, $timezone = NULL, $settin
    *   any other specialized input with a known format. If provided the
    *   date will be created using the createFromFormat() method.
    *   @see http://php.net/manual/datetime.createfromformat.php
-   * @param mixed $time
-   *   @see __construct()
+   * @param string $time
+   *   String representing the time.
    * @param mixed $timezone
-   *   @see __construct()
+   *   (optional) \DateTimeZone object, time zone string or NULL. See
+   *   __construct() for more details.
    * @param array $settings
+   *   (optional) A keyed array for settings, suitable for passing on to
+   *   __construct(). Supports an additional key:
    *   - validate_format: (optional) Boolean choice to validate the
    *     created date using the input format. The format used in
    *     createFromFormat() allows slightly different values than format().
@@ -223,7 +229,6 @@ public static function createFromTimestamp($timestamp, $timezone = NULL, $settin
    *     possible to a validation step to confirm that the date created
    *     from a format string exactly matches the input. This option
    *     indicates the format can be used for validation. Defaults to TRUE.
-   *   @see __construct()
    *
    * @return static
    *   A new DateTimePlus object.

core/lib/Drupal/Core/Config/Entity/Query/QueryFactory.php

@@ -136,7 +136,7 @@ protected function deleteConfigKeyStore(ConfigEntityTypeInterface $entity_type,
    *
    * @param \Drupal\Core\Config\Config $config
    *   The configuration object.
-   *  @param string $key
+   * @param string $key
    *   The configuration key to look for.
    * @param string $get_method
    *   Which method on the config object to call to get the value. Either 'get'

core/modules/content_moderation/src/ModerationInformation.php

@@ -181,7 +181,7 @@ public function isDefaultRevisionPublished(ContentEntityInterface $entity) {
     // If no default revision could be loaded, the entity has not yet been
     // saved. In this case the moderation_state of the unsaved entity can be
     // used, since once saved it will become the default.
-    $default_revision = $default_revision  ?: $entity;
+    $default_revision = $default_revision ?: $entity;
 
     // Ensure we are checking all translations of the default revision.
     if ($default_revision instanceof TranslatableInterface && $default_revision->isTranslatable()) {

core/modules/update/tests/src/Functional/UpdateCoreTest.php

@@ -572,7 +572,6 @@ public function testLocalActions() {
     $this->assertUrl('admin/reports/updates/install');
   }
 
-
   /**
    * {@inheritdoc}
    */

core/modules/update/tests/src/Unit/ProjectCoreCompatibilityTest.php

@@ -132,4 +132,3 @@ public function providerSetProjectCoreCompatibilityRanges() {
   }
 
 }
-

core/phpcs.xml.dist

@@ -141,7 +141,6 @@
   <rule ref="Drupal.WhiteSpace.ObjectOperatorSpacing"/>
   <rule ref="Drupal.WhiteSpace.OpenBracketSpacing"/>
   <rule ref="Drupal.WhiteSpace.OpenTagNewline"/>
-  <rule ref="Drupal.WhiteSpace.OperatorSpacing"/>
   <rule ref="Drupal.WhiteSpace.ScopeClosingBrace"/>
   <rule ref="Drupal.WhiteSpace.ScopeIndent"/>
 
@@ -329,6 +328,11 @@
     </properties>
   </rule>
   <rule ref="Squiz.WhiteSpace.LanguageConstructSpacing" />
+  <rule ref="Squiz.WhiteSpace.OperatorSpacing">
+    <properties>
+      <property name="ignoreNewlines" value="true"/>
+    </properties>
+  </rule>
   <rule ref="Squiz.WhiteSpace.SemicolonSpacing"/>
   <rule ref="Squiz.WhiteSpace.SuperfluousWhitespace"/>