Issue #2572707 by attiks, alexpott: Fix 'Drupal.Files.EndFileNewline' coding standard

core/lib/Drupal/Component/Diff/Diff.php

@@ -169,4 +169,3 @@ class Diff {
     return $this->edits;
   }
 }
-

core/lib/Drupal/Component/Diff/MappedDiff.php

@@ -56,4 +56,3 @@ class MappedDiff extends Diff {
     }
   }
 }
-

core/lib/Drupal/Component/Discovery/YamlDiscovery.php

@@ -86,4 +86,3 @@ class YamlDiscovery implements DiscoverableInterface {
   }
 
 }
-

core/lib/Drupal/Core/Ajax/DataCommand.php

@@ -71,4 +71,3 @@ class DataCommand implements CommandInterface {
   }
 
 }
-

core/lib/Drupal/Core/AppRootFactory.php

@@ -34,4 +34,3 @@ class AppRootFactory {
   }
 
 }
-

core/lib/Drupal/Core/Archiver/ArchiverException.php

@@ -6,4 +6,3 @@ namespace Drupal\Core\Archiver;
  * Defines an exception class for Drupal\Core\Archiver\ArchiverInterface.
  */
 class ArchiverException extends \Exception {}
-

core/lib/Drupal/Core/Config/ConfigCrudEvent.php

@@ -49,4 +49,3 @@ class ConfigCrudEvent extends Event {
   }
 
 }
-

core/lib/Drupal/Core/Config/ConfigModuleOverridesEvent.php

@@ -100,4 +100,3 @@ class ConfigModuleOverridesEvent extends Event {
     return $this;
   }
 }
-

core/lib/Drupal/Core/Config/ExtensionInstallStorage.php

@@ -132,4 +132,3 @@ class ExtensionInstallStorage extends InstallStorage {
     return $this->folders;
   }
 }
-

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

@@ -11,4 +11,3 @@ namespace Drupal\Core\Database;
  * developers should account for it separately.
  */
 interface DatabaseException { }
-

core/lib/Drupal/Core/Database/Driver/sqlite/Select.php

@@ -12,4 +12,4 @@ class Select extends QuerySelect {
     // SQLite does not support FOR UPDATE so nothing to do.
     return $this;
   }
-}
+}

core/lib/Drupal/Core/Datetime/DateHelper.php

@@ -531,4 +531,3 @@ class DateHelper {
   }
 
 }
-

core/lib/Drupal/Core/Datetime/Element/DateElementBase.php

@@ -70,4 +70,3 @@ abstract class DateElementBase extends FormElement {
   }
 
 }
-

core/lib/Drupal/Core/Entity/Controller/EntityListController.php

@@ -23,4 +23,3 @@ class EntityListController extends ControllerBase {
   }
 
 }
-

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

@@ -68,4 +68,3 @@ class EntityRouteProviderSubscriber implements EventSubscriberInterface {
   }
 
 }
-

core/lib/Drupal/Core/Executable/ExecutablePluginBase.php

@@ -84,4 +84,3 @@ abstract class ExecutablePluginBase extends ContextAwarePluginBase implements Ex
     return $this;
   }
 }
-

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

@@ -60,4 +60,3 @@ interface InfoParserInterface  {
   public function parse($filename);
 
 }
-

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

@@ -82,4 +82,3 @@ interface ModuleInstallerInterface {
   public function validateUninstall(array $module_list);
 
 }
-

core/lib/Drupal/Core/Field/Plugin/Field/FieldType/BooleanItem.php

@@ -120,4 +120,3 @@ class BooleanItem extends FieldItemBase implements OptionsProviderInterface {
     return $values;
   }
 }
-

core/lib/Drupal/Core/KeyValueStore/KeyValueExpirableFactory.php

@@ -14,4 +14,3 @@ class KeyValueExpirableFactory extends KeyValueFactory implements KeyValueExpira
   const DEFAULT_SETTING = 'keyvalue_expirable_default';
 
 }
-

core/lib/Drupal/Core/KeyValueStore/KeyValueFactory.php

@@ -75,4 +75,3 @@ class KeyValueFactory implements KeyValueFactoryInterface {
   }
 
 }
-

core/lib/Drupal/Core/KeyValueStore/KeyValueFactoryInterface.php

@@ -19,4 +19,3 @@ interface KeyValueFactoryInterface {
   public function get($collection);
 
 }
-

core/lib/Drupal/Core/PageCache/ResponsePolicy/DenyNoCacheRoutes.php

@@ -42,4 +42,3 @@ class DenyNoCacheRoutes implements ResponsePolicyInterface {
   }
 
 }
-

core/lib/Drupal/Core/Queue/QueueFactory.php

@@ -64,4 +64,3 @@ class QueueFactory implements ContainerAwareInterface {
     return $this->queues[$name];
   }
 }
-

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

@@ -134,4 +134,3 @@ class AccessAwareRouter implements AccessAwareRouterInterface {
   }
 
 }
-

core/lib/Drupal/Core/SitePathFactory.php

@@ -35,4 +35,3 @@ class SitePathFactory {
   }
 
 }
-

core/lib/Drupal/Core/StackMiddleware/KernelPreHandle.php

@@ -49,4 +49,3 @@ class KernelPreHandle implements HttpKernelInterface {
   }
 
 }
-

core/lib/Drupal/Core/TypedData/Plugin/DataType/DateTimeIso8601.php

@@ -43,4 +43,3 @@ class DateTimeIso8601 extends StringData implements DateTimeInterface {
     }
   }
 }
-

core/modules/block/tests/modules/block_test/src/Plugin/Block/TestAccessBlock.php

@@ -76,4 +76,3 @@ class TestAccessBlock extends BlockBase implements ContainerFactoryPluginInterfa
   }
 
 }
-

core/modules/book/src/Form/BookSettingsForm.php

@@ -81,4 +81,3 @@ class BookSettingsForm extends ConfigFormBase {
   }
 
 }
-

core/modules/config/src/Form/ConfigImportForm.php

@@ -100,4 +100,3 @@ class ConfigImportForm extends FormBase {
   }
 
 }
-

core/modules/config_translation/config_translation.module

@@ -196,4 +196,3 @@ function config_translation_config_schema_info_alter(&$definitions) {
     }
   }
 }
-

core/modules/field/tests/modules/field_third_party_test/field_third_party_test.module

@@ -46,4 +46,3 @@ function field_third_party_test_field_formatter_settings_summary_alter(&$summary
   $summary[] = 'field_test_field_formatter_settings_summary_alter';
   return $summary;
 }
-

core/modules/field/tests/src/Kernel/FieldImportChangeTest.php

@@ -49,4 +49,3 @@ class FieldImportChangeTest extends FieldKernelTestBase {
     $this->assertEqual($field->getLabel(), $new_label, 'field label updated');
   }
 }
-

core/modules/field/tests/src/Kernel/FieldImportCreateTest.php

@@ -116,4 +116,3 @@ class FieldImportCreateTest extends FieldKernelTestBase {
     $this->assertTrue($field, 'Test import field 2b from sync exists');
   }
 }
-

core/modules/field/tests/src/Kernel/FieldImportDeleteTest.php

@@ -109,4 +109,3 @@ class FieldImportDeleteTest extends FieldKernelTestBase {
     $this->assertTrue(empty($deleted_storages), 'Fields are deleted');
   }
 }
-

core/modules/filter/src/Plugin/migrate/source/d6/FilterFormat.php

@@ -90,5 +90,3 @@ class FilterFormat extends DrupalSqlBase {
   }
 
 }
-
-

core/modules/hal/src/HalServiceProvider.php

@@ -20,4 +20,3 @@ class HalServiceProvider implements ServiceModifierInterface {
   }
 
 }
-

core/modules/image/src/Tests/ImageFieldWidgetTest.php

@@ -29,4 +29,3 @@ class ImageFieldWidgetTest extends ImageFieldTestBase {
   }
 
 }
-

core/modules/language/src/ContentLanguageSettingsInterface.php

@@ -76,4 +76,3 @@ interface ContentLanguageSettingsInterface extends ConfigEntityInterface {
   public function isDefaultConfiguration();
 
 }
-

core/modules/language/src/Form/NegotiationBrowserForm.php

@@ -212,4 +212,3 @@ class NegotiationBrowserForm extends ConfigFormBase {
     return $config->get('map');
   }
 }
-

core/modules/link/src/Plugin/Field/FieldFormatter/LinkSeparateFormatter.php

@@ -86,4 +86,3 @@ class LinkSeparateFormatter extends LinkFormatter {
     return $element;
   }
 }
-

core/modules/link/src/Plugin/Validation/Constraint/LinkTypeConstraint.php

@@ -75,4 +75,3 @@ class LinkTypeConstraint extends Constraint implements ConstraintValidatorInterf
     }
   }
 }
-

core/modules/locale/src/LocaleDefaultConfigStorage.php

@@ -157,4 +157,3 @@ class LocaleDefaultConfigStorage {
   }
 
 }
-

core/modules/node/src/Tests/Views/NodeRevisionWizardTest.php

@@ -59,4 +59,3 @@ class NodeRevisionWizardTest extends WizardTestBase {
   }
 
 }
-

core/modules/quickedit/tests/modules/quickedit_test.module

@@ -18,4 +18,3 @@ function quickedit_test_quickedit_render_field(EntityInterface $entity, $field_n
     '#suffix' => '</div>',
   );
 }
-

core/modules/rdf/tests/src/Kernel/Field/FieldRdfaDatatypeCallbackTest.php

@@ -55,4 +55,3 @@ class FieldRdfaDatatypeCallbackTest extends FieldRdfaTestBase {
   }
 
 }
-

core/modules/search/search.api.php

@@ -82,4 +82,3 @@ function hook_search_plugin_alter(array &$definitions) {
 /**
  * @} End of "addtogroup hooks".
  */
-

core/modules/search/search.pages.inc

@@ -65,4 +65,3 @@ function template_preprocess_search_result(&$variables) {
     '#context' => array('info' => $info),
   );
 }
-

core/modules/serialization/tests/src/Unit/Normalizer/ConfigEntityNormalizerTest.php

@@ -31,4 +31,3 @@ class ConfigEntityNormalizerTest extends UnitTestCase {
   }
 
 }
-

core/modules/simpletest/tests/fixtures/simpletest_phpunit_run_command_test.php

@@ -20,4 +20,3 @@ class SimpletestPhpunitRunCommandTestWillDie extends UnitTestCase {
     $this->assertTrue(TRUE, 'Assertion to ensure test pass');
   }
 }
-

core/modules/statistics/src/StatisticsSettingsForm.php

@@ -97,4 +97,3 @@ class StatisticsSettingsForm extends ConfigFormBase {
     parent::submitForm($form, $form_state);
   }
 }
-

core/modules/statistics/statistics.php

@@ -35,4 +35,3 @@ if ($views) {
       ->execute();
   }
 }
-

core/modules/system/src/Tests/Plugin/Discovery/DiscoveryTestBase.php

@@ -82,4 +82,3 @@ abstract class DiscoveryTestBase extends KernelTestBase {
   }
 
 }
-

core/modules/system/src/Tests/Plugin/Discovery/StaticDiscoveryTest.php

@@ -43,4 +43,3 @@ class StaticDiscoveryTest extends DiscoveryTestBase {
     $this->emptyDiscovery = $discovery;
   }
 }
-

core/modules/system/src/Tests/Theme/TwigEnvironmentTest.php

@@ -128,4 +128,3 @@ class TwigEnvironmentTest extends KernelTestBase {
   }
 
 }
-

core/modules/system/system.admin.inc

@@ -385,4 +385,3 @@ function template_preprocess_system_themes_page(&$variables) {
   }
   $variables['theme_groups'] = $groups;
 }
-

core/modules/system/tests/fixtures/update/drupal-8.block-content-uninstall.php

@@ -147,4 +147,3 @@ $connection->delete('router')
 $connection->delete('router')
   ->condition('name', 'entity.block_content_type.%', 'LIKE')
   ->execute();
-

core/modules/system/tests/fixtures/update/drupal-8.broken_routing.php

@@ -20,4 +20,3 @@ $connection->update('config')
 $connection->insert('key_value')
   ->fields(['collection' => 'system.schema', 'name' => 'update_script_test', 'value' => serialize(8000)])
   ->execute();
-

core/modules/system/tests/fixtures/update/drupal-8.update-test-postupdate-enabled.php

@@ -37,4 +37,3 @@ $connection->update('config')
   ->condition('collection', '')
   ->condition('name', 'core.extension')
   ->execute();
-

core/modules/system/tests/fixtures/update/drupal-8.update-test-postupdate-failing-enabled.php

@@ -36,4 +36,3 @@ $connection->update('config')
   ->condition('collection', '')
   ->condition('name', 'core.extension')
   ->execute();
-

core/modules/system/tests/modules/database_test/database_test.module

@@ -47,4 +47,3 @@ function database_test_query_alter(AlterableInterface $query) {
 function database_test_query_database_test_alter_remove_range_alter(AlterableInterface $query) {
   $query->range();
 }
-

core/modules/system/tests/modules/plugin_test/src/Plugin/plugin_test/fruit/FruitInterface.php

@@ -6,4 +6,3 @@ namespace Drupal\plugin_test\Plugin\plugin_test\fruit;
  * Provides an interface for test plugins.
  */
 interface FruitInterface { }
-

core/modules/system/tests/modules/system_test/src/Controller/PageCacheAcceptHeaderController.php

@@ -28,4 +28,3 @@ class PageCacheAcceptHeaderController {
     }
   }
 }
-

core/modules/system/tests/modules/trusted_hosts_test/src/Controller/TrustedHostsTestController.php

@@ -17,4 +17,3 @@ class TrustedHostsTestController {
   }
 
 }
-

core/modules/system/tests/modules/twig_extension_test/twig_extension_test.module

@@ -20,4 +20,3 @@ function twig_extension_test_theme($existing, $type, $theme, $path) {
     ),
   );
 }
-

core/modules/system/tests/themes/test_subsubtheme/test_subsubtheme.theme

@@ -10,4 +10,3 @@
  */
 function test_subsubtheme_preprocess_theme_test_template_test(&$variables) {
 }
-

core/modules/taxonomy/src/Tests/TokenReplaceTest.php

@@ -144,4 +144,3 @@ class TokenReplaceTest extends TaxonomyTestBase {
     }
   }
 }
-

core/modules/tracker/tests/src/Kernel/Migrate/d7/MigrateTrackerNodeTest.php

@@ -65,4 +65,3 @@ class MigrateTrackerNodeTest extends MigrateDrupal7TestBase {
   }
 
 }
-

core/modules/update/update.fetch.inc

@@ -48,4 +48,3 @@ function _update_cron_notify() {
     }
   }
 }
-

core/modules/user/src/Form/UserPasswordResetForm.php

@@ -115,4 +115,3 @@ class UserPasswordResetForm extends FormBase {
   }
 
 }
-

core/modules/user/src/PermissionHandlerInterface.php

@@ -50,4 +50,3 @@ interface PermissionHandlerInterface {
   public function moduleProvidesPermissions($module_name);
 
 }
-

core/modules/user/src/Plugin/views/access/Role.php

@@ -163,4 +163,3 @@ class Role extends AccessPluginBase implements CacheableDependencyInterface {
   }
 
 }
-

core/modules/user/tests/src/Unit/PrivateTempStoreTest.php

@@ -278,4 +278,3 @@ class PrivateTempStoreTest extends UnitTestCase {
   }
 
 }
-

core/modules/user/tests/src/Unit/SharedTempStoreTest.php

@@ -350,4 +350,3 @@ class SharedTempStoreTest extends UnitTestCase {
   }
 
 }
-

core/modules/views/src/Plugin/views/field/MultiItemsFieldHandlerInterface.php

@@ -45,4 +45,3 @@ interface MultiItemsFieldHandlerInterface extends FieldHandlerInterface {
   public function renderItems($items);
 
 }
-

core/modules/views/src/ViewsDataHelper.php

@@ -190,4 +190,3 @@ class ViewsDataHelper {
   }
 
 }
-

core/modules/views/tests/src/Kernel/Handler/FilterBooleanOperatorTest.php

@@ -215,4 +215,3 @@ class FilterBooleanOperatorTest extends ViewsKernelTestBase {
   }
 
 }
-

core/modules/views/tests/src/Kernel/Wizard/WizardPluginBaseKernelTest.php

@@ -71,4 +71,3 @@ class WizardPluginBaseKernelTest extends ViewsKernelTestBase {
     $this->assertEqual($view->get('langcode'), 'it');
   }
 }
-

core/modules/views/tests/src/Unit/Controller/ViewAjaxControllerTest.php

@@ -360,4 +360,3 @@ class ViewAjaxControllerTest extends UnitTestCase {
 }
 
 }
-

core/modules/views_ui/src/Form/Ajax/RearrangeFilter.php

@@ -352,4 +352,3 @@ class RearrangeFilter extends ViewsFormBase {
   }
 
 }
-

core/phpcs.xml.dist

@@ -18,6 +18,7 @@
   <rule ref="Drupal.Commenting.DocCommentStar"/>
   <rule ref="Drupal.Commenting.FileComment"/>
   <rule ref="Drupal.ControlStructures.ElseIf"/>
+  <rule ref="Drupal.Files.EndFileNewline"/>
   <rule ref="Drupal.Files.TxtFileLineLength"/>
   <rule ref="Drupal.Functions.DiscouragedFunctions"/>
   <rule ref="Drupal.Functions.FunctionDeclaration.SpaceAfter"/>

core/tests/Drupal/KernelTests/Config/DefaultConfigTest.php

@@ -112,4 +112,3 @@ class DefaultConfigTest extends KernelTestBase {
   }
 
 }
-

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

@@ -117,4 +117,3 @@ class ConfigLanguageOverrideTest extends KernelTestBase {
     $this->assertEqual($override->get('value'), NULL);
   }
 }
-

core/tests/Drupal/Tests/Component/Plugin/DefaultFactoryTest.php

@@ -154,4 +154,3 @@ class DefaultFactoryTest extends UnitTestCase {
   }
 
 }
-

core/tests/Drupal/Tests/Component/Utility/HtmlTest.php

@@ -320,4 +320,4 @@ class HtmlTest extends UnitTestCase {
     $result = Html::serialize($document);
     $this->assertSame('', $result);
   }
-}
+}

core/tests/Drupal/Tests/Core/DependencyInjection/Compiler/TaggedHandlersPassTest.php

@@ -378,4 +378,3 @@ class ValidHandler implements HandlerInterface {
 }
 class InvalidHandler {
 }
-

core/tests/Drupal/Tests/Core/DrupalKernel/DrupalKernelTest.php

@@ -144,4 +144,4 @@ namespace {
       return FALSE;
     }
   }
-}
+}

core/tests/Drupal/Tests/Core/EventSubscriber/CustomPageExceptionHtmlSubscriberTest.php

@@ -162,4 +162,3 @@ class CustomPageExceptionHtmlSubscriberTest extends UnitTestCase {
   }
 
 }
-

core/tests/Drupal/Tests/Core/Logger/LoggerChannelTest.php

@@ -161,4 +161,3 @@ class NaughtyRecursiveLogger implements LoggerInterface  {
     $this->channel->log(rand(0, 7), $message, $context);
   }
 }
-

core/tests/Drupal/Tests/Core/Menu/LocalTaskManagerTest.php

@@ -491,4 +491,3 @@ class LocalTaskManagerTest extends UnitTestCase {
   }
 
 }
-

core/tests/Drupal/Tests/Core/Path/PathValidatorTest.php

@@ -444,4 +444,3 @@ class PathValidatorTest extends UnitTestCase {
   }
 
 }
-

core/tests/bootstrap.php

@@ -142,4 +142,3 @@ date_default_timezone_set('Australia/Sydney');
 // make PHP 5 and 7 handle assertion failures the same way, but this call does
 // not turn runtime assertions on if they weren't on already.
 \Drupal\Component\Assertion\Handle::register();
-