Issue #2572619 by attiks, andypost, pfrenssen, vprocessor, tatarbj, andriyun,...

Issue #2572619 by attiks, andypost, pfrenssen, vprocessor, tatarbj, andriyun, alexpott: Fix 'Drupal.Classes.ClassDeclaration' coding standard

core/lib/Drupal/Core/Cache/Context/LanguagesCacheContext.php

@@ -8,7 +8,7 @@
 /**
  * Defines the LanguagesCacheContext service, for "per language" caching.
  */
-class LanguagesCacheContext implements CalculatedCacheContextInterface  {
+class LanguagesCacheContext implements CalculatedCacheContextInterface {
 
   /**
    * The language manager.

core/lib/Drupal/Core/CoreServiceProvider.php

@@ -40,7 +40,7 @@
  *
  * @ingroup container
  */
-class CoreServiceProvider implements ServiceProviderInterface, ServiceModifierInterface  {
+class CoreServiceProvider implements ServiceProviderInterface, ServiceModifierInterface {
 
   /**
    * {@inheritdoc}

core/lib/Drupal/Core/Entity/Sql/SqlContentEntityStorage.php

@@ -36,7 +36,7 @@
  *
  * @ingroup entity_api
  */
-class SqlContentEntityStorage extends ContentEntityStorageBase implements SqlEntityStorageInterface, DynamicallyFieldableEntityStorageSchemaInterface, EntityBundleListenerInterface  {
+class SqlContentEntityStorage extends ContentEntityStorageBase implements SqlEntityStorageInterface, DynamicallyFieldableEntityStorageSchemaInterface, EntityBundleListenerInterface {
 
   /**
    * The mapping of field columns to SQL tables.

core/lib/Drupal/Core/Extension/InfoParserInterface.php

@@ -5,7 +5,7 @@
 /**
  * Interface for classes that parses Drupal's info.yml files.
  */
-interface InfoParserInterface  {
+interface InfoParserInterface {
 
   /**
    * Parses Drupal module, theme and profile .info.yml files.

core/lib/Drupal/Core/Render/Element/Pager.php

@@ -30,7 +30,7 @@
  *
  * @RenderElement("pager")
  */
-class Pager extends RenderElement{
+class Pager extends RenderElement {
 
   /**
    * {@inheritdoc}

core/lib/Drupal/Core/Render/MainContent/MainContentRendererInterface.php

@@ -13,7 +13,7 @@
  * (HTML, JSON …) and/or in a certain decorated manner (e.g. in the case of the
  * default HTML main content renderer: with a page display variant applied).
  */
-interface MainContentRendererInterface  {
+interface MainContentRendererInterface {
 
   /**
    * Renders the main content render array into a response.

core/lib/Drupal/Core/TypedData/ComplexDataDefinitionInterface.php

@@ -9,7 +9,7 @@
  *
  * @ingroup typed_data
  */
-interface ComplexDataDefinitionInterface extends DataDefinitionInterface  {
+interface ComplexDataDefinitionInterface extends DataDefinitionInterface {
 
   /**
    * Gets the definition of a contained property.

core/lib/Drupal/Core/TypedData/ComplexDataInterface.php

@@ -18,7 +18,7 @@
  *
  * @ingroup typed_data
  */
-interface ComplexDataInterface extends TraversableTypedDataInterface  {
+interface ComplexDataInterface extends TraversableTypedDataInterface {
 
   /**
    * Gets the data definition.

core/lib/Drupal/Core/TypedData/DataReferenceBase.php

@@ -14,7 +14,7 @@
  *
  * @see \Drupal\Core\TypedData\DataReferenceDefinition
  */
-abstract class DataReferenceBase extends TypedData implements DataReferenceInterface  {
+abstract class DataReferenceBase extends TypedData implements DataReferenceInterface {
 
   /**
    * The referenced data.

core/lib/Drupal/Core/TypedData/DataReferenceDefinitionInterface.php

@@ -10,7 +10,7 @@
  *
  * @ingroup typed_data
  */
-interface DataReferenceDefinitionInterface extends DataDefinitionInterface  {
+interface DataReferenceDefinitionInterface extends DataDefinitionInterface {
 
   /**
    * Gets the data definition of the referenced data.

core/lib/Drupal/Core/TypedData/DataReferenceInterface.php

@@ -7,7 +7,7 @@
  *
  * @see \Drupal\Core\TypedData\DataReferenceDefinitionInterface
  */
-interface DataReferenceInterface  {
+interface DataReferenceInterface {
 
   /**
    * Gets the referenced data.

core/modules/block/tests/modules/block_test/src/Controller/TestMultipleFormController.php

@@ -8,7 +8,7 @@
 /**
  * Controller for block_test module
  */
-class TestMultipleFormController  extends ControllerBase {
+class TestMultipleFormController extends ControllerBase {
 
   public function testMultipleForms() {
     $form_state = new FormState();

core/modules/book/src/Access/BookNodeIsRemovableAccessCheck.php

@@ -10,7 +10,7 @@
 /**
  * Determines whether the requested node can be removed from its book.
  */
-class BookNodeIsRemovableAccessCheck implements AccessInterface{
+class BookNodeIsRemovableAccessCheck implements AccessInterface {
 
   /**
    * Book Manager Service.

core/modules/forum/src/Breadcrumb/ForumBreadcrumbBuilderBase.php

@@ -17,7 +17,7 @@
  * This just holds the dependency-injected config, entity manager, and forum
  * manager objects.
  */
-abstract class ForumBreadcrumbBuilderBase  implements BreadcrumbBuilderInterface {
+abstract class ForumBreadcrumbBuilderBase implements BreadcrumbBuilderInterface {
   use StringTranslationTrait;
 
   /**

core/modules/migrate_drupal/src/Plugin/migrate/cckfield/CckFieldPluginBase.php

@@ -17,7 +17,7 @@
  *
  * @ingroup migration
  */
-abstract class CckFieldPluginBase extends PluginBase implements MigrateCckFieldInterface  {
+abstract class CckFieldPluginBase extends PluginBase implements MigrateCckFieldInterface {
 
   /**
    * {@inheritdoc}

core/modules/taxonomy/src/Plugin/views/relationship/NodeTermData.php

@@ -16,7 +16,7 @@
  *
  * @ViewsRelationship("node_term_data")
  */
-class NodeTermData extends RelationshipPluginBase  {
+class NodeTermData extends RelationshipPluginBase {
 
   /**
    * The vocabulary storage.

core/modules/views/src/Plugin/views/relationship/EntityReverse.php

@@ -13,7 +13,7 @@
  *
  * @ViewsRelationship("entity_reverse")
  */
-class EntityReverse extends RelationshipPluginBase  {
+class EntityReverse extends RelationshipPluginBase {
 
   /**
    * Constructs an EntityReverse object.

core/phpcs.xml.dist

@@ -13,13 +13,7 @@
   <!-- Only include specific sniffs that pass. This ensures that, if new sniffs are added, HEAD does not fail.-->
   <!-- Drupal sniffs -->
   <rule ref="Drupal.Classes.ClassCreateInstance"/>
-  <rule ref="Drupal.Classes.ClassDeclaration">
-    <exclude name="Drupal.Classes.ClassDeclaration.BraceOnNewLine"/>
-    <exclude name="Drupal.Classes.ClassDeclaration.SpaceAfterName"/>
-    <exclude name="Drupal.Classes.ClassDeclaration.SpaceBeforeBrace"/>
-    <exclude name="Drupal.Classes.ClassDeclaration.SpaceBeforeExtends"/>
-    <exclude name="Drupal.Classes.ClassDeclaration.SpaceBeforeImplements"/>
-  </rule>
+  <rule ref="Drupal.Classes.ClassDeclaration"/>
   <rule ref="Drupal.Classes.FullyQualifiedNamespace"/>
   <rule ref="Drupal.Classes.UnusedUseStatement"/>
   <rule ref="Drupal.CSS.ClassDefinitionNameSpacing"/>

core/tests/Drupal/KernelTests/Core/Entity/EntityAccessControlHandlerTest.php

@@ -18,7 +18,7 @@
  *
  * @group Entity
  */
-class EntityAccessControlHandlerTest extends EntityLanguageTestBase  {
+class EntityAccessControlHandlerTest extends EntityLanguageTestBase {
 
   /**
    * Asserts entity access correctly grants or denies access.

core/tests/Drupal/KernelTests/Core/Entity/EntityBundleFieldTest.php

@@ -7,7 +7,7 @@
  *
  * @group Entity
  */
-class EntityBundleFieldTest extends EntityKernelTestBase  {
+class EntityBundleFieldTest extends EntityKernelTestBase {
 
   /**
    * Modules to enable.