diff --git a/core/includes/batch.inc b/core/includes/batch.inc
index 402f195..5b67f4d 100644
--- a/core/includes/batch.inc
+++ b/core/includes/batch.inc
@@ -160,7 +160,6 @@ function _batch_progress_page() {
}
else {
// This is one of the later requests; do some processing first.
-
// Error handling: if PHP dies due to a fatal error (e.g. a nonexistent
// function), it will output whatever is in the output buffer, followed by
// the error message.
@@ -331,7 +330,6 @@ function _batch_process() {
if ($batch['progressive']) {
// Gather progress information.
-
// Reporting 100% progress will cause the whole batch to be considered
// processed. If processing was paused right after moving to a new set,
// we have to use the info from the new (unprocessed) set.
diff --git a/core/includes/errors.inc b/core/includes/errors.inc
index 284d183..b51d2f6 100644
--- a/core/includes/errors.inc
+++ b/core/includes/errors.inc
@@ -230,7 +230,6 @@ function _drupal_log_error($error, $fatal = FALSE) {
if ($error_level != ERROR_REPORTING_DISPLAY_VERBOSE) {
// Without verbose logging, use a simple message.
-
// We call SafeMarkup::format() directly here, rather than use t() since
// we are in the middle of error handling, and we don't want t() to
// cause further errors.
@@ -238,7 +237,6 @@ function _drupal_log_error($error, $fatal = FALSE) {
}
else {
// With verbose logging, we will also include a backtrace.
-
// First trace is the error itself, already contained in the message.
// While the second trace is the error source and also contained in the
// message, the message doesn't contain argument values, so we output it
diff --git a/core/includes/pager.inc b/core/includes/pager.inc
index 62e8278..9d59ea8 100644
--- a/core/includes/pager.inc
+++ b/core/includes/pager.inc
@@ -199,7 +199,6 @@ function template_preprocess_pager(&$variables) {
// max is the maximum page number.
$pager_max = $pager_total[$element];
// End of marker calculations.
-
// Prepare for generation loop.
$i = $pager_first;
if ($pager_last > $pager_max) {
@@ -213,7 +212,6 @@ function template_preprocess_pager(&$variables) {
$i = 1;
}
// End of generation loop preparation.
-
// Create the "first" and "previous" links if we are not on the first page.
if ($pager_page_array[$element] > 0) {
$items['first'] = [];
diff --git a/core/includes/theme.inc b/core/includes/theme.inc
index 4cc6424..c4e2b64 100644
--- a/core/includes/theme.inc
+++ b/core/includes/theme.inc
@@ -445,7 +445,6 @@ function theme_render_and_autoescape($arg) {
// This is a normal render array, which is safe by definition, with special
// simple cases already handled.
-
// Early return if this element was pre-rendered (no need to re-render).
if (isset($arg['#printed']) && $arg['#printed'] == TRUE && isset($arg['#markup']) && strlen($arg['#markup']) > 0) {
return (string) $arg['#markup'];
@@ -1412,7 +1411,6 @@ function theme_get_suggestions($args, $base, $delimiter = '__') {
// page__node__%
// page__node__1
// page__node__edit
-
$suggestions = [];
$prefix = $base;
foreach ($args as $arg) {
diff --git a/core/lib/Drupal/Component/Gettext/PoStreamReader.php b/core/lib/Drupal/Component/Gettext/PoStreamReader.php
index f84d251..b9f80db 100644
--- a/core/lib/Drupal/Component/Gettext/PoStreamReader.php
+++ b/core/lib/Drupal/Component/Gettext/PoStreamReader.php
@@ -272,7 +272,6 @@ private function readLine() {
if (!strncmp('#', $line, 1)) {
// Lines starting with '#' are comments.
-
if ($this->_context == 'COMMENT') {
// Already in comment context, add to current comment.
$this->_current_item['#'][] = substr($line, 1);
@@ -297,7 +296,6 @@ private function readLine() {
}
elseif (!strncmp('msgid_plural', $line, 12)) {
// A plural form for the current source string.
-
if ($this->_context != 'MSGID') {
// A plural form can only be added to an msgid directly.
$this->_errors[] = SafeMarkup::format('The translation stream %uri contains an error: "msgid_plural" was expected but not found on line %line.', $log_vars);
@@ -328,7 +326,6 @@ private function readLine() {
}
elseif (!strncmp('msgid', $line, 5)) {
// Starting a new message.
-
if (($this->_context == 'MSGSTR') || ($this->_context == 'MSGSTR_ARR')) {
// We are currently in string context, save current item.
$this->setItemFromArray($this->_current_item);
@@ -359,7 +356,6 @@ private function readLine() {
}
elseif (!strncmp('msgctxt', $line, 7)) {
// Starting a new context.
-
if (($this->_context == 'MSGSTR') || ($this->_context == 'MSGSTR_ARR')) {
// We are currently in string context, save current item.
$this->setItemFromArray($this->_current_item);
@@ -389,7 +385,6 @@ private function readLine() {
}
elseif (!strncmp('msgstr[', $line, 7)) {
// A message string for a specific plurality.
-
if (($this->_context != 'MSGID') &&
($this->_context != 'MSGCTXT') &&
($this->_context != 'MSGID_PLURAL') &&
@@ -431,7 +426,6 @@ private function readLine() {
}
elseif (!strncmp("msgstr", $line, 6)) {
// A string pair for an msgid (with optional context).
-
if (($this->_context != 'MSGID') && ($this->_context != 'MSGCTXT')) {
// Strings are only valid within an id or context scope.
$this->_errors[] = SafeMarkup::format('The translation stream %uri contains an error: "msgstr" is unexpected on line %line.', $log_vars);
@@ -456,7 +450,6 @@ private function readLine() {
}
elseif ($line != '') {
// Anything that is not a token may be a continuation of a previous token.
-
$quoted = $this->parseQuoted($line);
if ($quoted === FALSE) {
// This string must be quoted.
diff --git a/core/lib/Drupal/Component/Plugin/ContextAwarePluginBase.php b/core/lib/Drupal/Component/Plugin/ContextAwarePluginBase.php
index b9f6e02..7a485a0 100644
--- a/core/lib/Drupal/Component/Plugin/ContextAwarePluginBase.php
+++ b/core/lib/Drupal/Component/Plugin/ContextAwarePluginBase.php
@@ -143,7 +143,6 @@ public function validateContexts() {
$violations = new ConstraintViolationList();
// @todo: Implement symfony validator API to let the validator traverse
// and set property paths accordingly.
-
foreach ($this->getContexts() as $context) {
$violations->addAll($context->validate());
}
diff --git a/core/lib/Drupal/Core/Archiver/Tar.php b/core/lib/Drupal/Core/Archiver/Tar.php
index 3b33ddd..99055a2 100644
--- a/core/lib/Drupal/Core/Archiver/Tar.php
+++ b/core/lib/Drupal/Core/Archiver/Tar.php
@@ -45,7 +45,6 @@ public function remove($file_path) {
// so we'll have to simulate it somehow, probably by
// creating a new archive with everything but the removed
// file.
-
return $this;
}
diff --git a/core/lib/Drupal/Core/Cache/MemoryBackend.php b/core/lib/Drupal/Core/Cache/MemoryBackend.php
index edda4d3..5ba7355 100644
--- a/core/lib/Drupal/Core/Cache/MemoryBackend.php
+++ b/core/lib/Drupal/Core/Cache/MemoryBackend.php
@@ -80,7 +80,6 @@ protected function prepareItem($cache, $allow_invalid) {
// We must clone it as part of the preparation step so that the actual
// cache object is not affected by the unserialize() call or other
// manipulations of the returned object.
-
$prepared = clone $cache;
$prepared->data = unserialize($prepared->data);
diff --git a/core/lib/Drupal/Core/Database/Driver/pgsql/Upsert.php b/core/lib/Drupal/Core/Database/Driver/pgsql/Upsert.php
index 52e077a..7c080ca 100644
--- a/core/lib/Drupal/Core/Database/Driver/pgsql/Upsert.php
+++ b/core/lib/Drupal/Core/Database/Driver/pgsql/Upsert.php
@@ -69,7 +69,6 @@ public function execute() {
$this->insertValues = [];
// Transaction commits here where $transaction looses scope.
-
return TRUE;
}
diff --git a/core/lib/Drupal/Core/Database/Query/Insert.php b/core/lib/Drupal/Core/Database/Query/Insert.php
index 3af2d2a..fb14e0e 100644
--- a/core/lib/Drupal/Core/Database/Query/Insert.php
+++ b/core/lib/Drupal/Core/Database/Query/Insert.php
@@ -100,7 +100,6 @@ public function execute() {
$this->insertValues = [];
// Transaction commits here where $transaction looses scope.
-
return $last_insert_id;
}
diff --git a/core/lib/Drupal/Core/Database/Query/Select.php b/core/lib/Drupal/Core/Database/Query/Select.php
index b3426e6..5c5d419 100644
--- a/core/lib/Drupal/Core/Database/Query/Select.php
+++ b/core/lib/Drupal/Core/Database/Query/Select.php
@@ -916,7 +916,6 @@ public function __clone() {
// On cloning, also clone the dependent objects. However, we do not
// want to clone the database connection object as that would duplicate the
// connection itself.
-
$this->condition = clone($this->condition);
$this->having = clone($this->having);
foreach ($this->union as $key => $aggregate) {
diff --git a/core/lib/Drupal/Core/Database/StatementInterface.php b/core/lib/Drupal/Core/Database/StatementInterface.php
index a34bf61..f07a18d 100644
--- a/core/lib/Drupal/Core/Database/StatementInterface.php
+++ b/core/lib/Drupal/Core/Database/StatementInterface.php
@@ -38,7 +38,6 @@
* statements). The access type has to be protected.
*/
//protected function __construct(Connection $dbh);
-
/**
* Executes a prepared statement
*
diff --git a/core/lib/Drupal/Core/DependencyInjection/Compiler/TaggedHandlersPass.php b/core/lib/Drupal/Core/DependencyInjection/Compiler/TaggedHandlersPass.php
index 35b5bea..2ef9580 100644
--- a/core/lib/Drupal/Core/DependencyInjection/Compiler/TaggedHandlersPass.php
+++ b/core/lib/Drupal/Core/DependencyInjection/Compiler/TaggedHandlersPass.php
@@ -144,7 +144,6 @@ protected function processServiceCollectorPass(array $pass, $consumer_id, Contai
}
}
// Determine the ID.
-
if (!isset($interface)) {
throw new LogicException(vsprintf("Service consumer '%s' class method %s::%s() has to type-hint an interface.", [
$consumer_id,
diff --git a/core/lib/Drupal/Core/DrupalKernel.php b/core/lib/Drupal/Core/DrupalKernel.php
index 217d4dc..84ee203 100644
--- a/core/lib/Drupal/Core/DrupalKernel.php
+++ b/core/lib/Drupal/Core/DrupalKernel.php
@@ -973,7 +973,6 @@ public static function bootEnvironment($app_root = NULL) {
// Override PHP settings required for Drupal to work properly.
// sites/default/default.settings.php contains more runtime settings.
// The .htaccess file contains settings that cannot be changed at runtime.
-
// Use session cookies, not transparent sessions that puts the session id in
// the query string.
ini_set('session.use_cookies', '1');
diff --git a/core/lib/Drupal/Core/Entity/ContentEntityBase.php b/core/lib/Drupal/Core/Entity/ContentEntityBase.php
index c5421ed..afbadc6 100644
--- a/core/lib/Drupal/Core/Entity/ContentEntityBase.php
+++ b/core/lib/Drupal/Core/Entity/ContentEntityBase.php
@@ -510,7 +510,6 @@ protected function getTranslatedField($name, $langcode) {
}
// Non-translatable fields are always stored with
// LanguageInterface::LANGCODE_DEFAULT as key.
-
$default = $langcode == LanguageInterface::LANGCODE_DEFAULT;
if (!$default && !$definition->isTranslatable()) {
if (!isset($this->fields[$name][LanguageInterface::LANGCODE_DEFAULT])) {
@@ -1087,7 +1086,6 @@ public function __clone() {
// before cloning the fields. Otherwise calling
// $items->getEntity()->isNew(), for example, would return the
// $enforceIsNew value of the old entity.
-
// Ensure the translations array is actually cloned by overwriting the
// original reference with one pointing to a copy of the array.
$translations = $this->translations;
diff --git a/core/lib/Drupal/Core/Entity/EntityAccessControlHandler.php b/core/lib/Drupal/Core/Entity/EntityAccessControlHandler.php
index ac36411..025b478 100644
--- a/core/lib/Drupal/Core/Entity/EntityAccessControlHandler.php
+++ b/core/lib/Drupal/Core/Entity/EntityAccessControlHandler.php
@@ -88,7 +88,6 @@ public function access(EntityInterface $entity, $operation, AccountInterface $ac
// EntityAccessControlHandler::checkAccess(). Entities that have checks that
// need to be done before the hook is invoked should do so by overriding
// this method.
-
// We grant access to the entity if both of these conditions are met:
// - No modules say to deny access.
// - At least one module says to grant access.
@@ -243,7 +242,6 @@ public function createAccess($entity_bundle = NULL, AccountInterface $account =
// EntityAccessControlHandler::checkCreateAccess(). Entities that have
// checks that need to be done before the hook is invoked should do so by
// overriding this method.
-
// We grant access to the entity if both of these conditions are met:
// - No modules say to deny access.
// - At least one module says to grant access.
diff --git a/core/lib/Drupal/Core/Entity/EntityDefinitionUpdateManager.php b/core/lib/Drupal/Core/Entity/EntityDefinitionUpdateManager.php
index 70dec2c..a75f094 100644
--- a/core/lib/Drupal/Core/Entity/EntityDefinitionUpdateManager.php
+++ b/core/lib/Drupal/Core/Entity/EntityDefinitionUpdateManager.php
@@ -312,7 +312,6 @@ protected function getChangeList() {
// @todo Support deleting entity definitions when we support base field
// purging. See https://www.drupal.org/node/2282119.
-
$this->entityManager->useCaches(TRUE);
return array_filter($change_list);
diff --git a/core/lib/Drupal/Core/EventSubscriber/EntityRouteProviderSubscriber.php b/core/lib/Drupal/Core/EventSubscriber/EntityRouteProviderSubscriber.php
index 36a539d..227f31c 100644
--- a/core/lib/Drupal/Core/EventSubscriber/EntityRouteProviderSubscriber.php
+++ b/core/lib/Drupal/Core/EventSubscriber/EntityRouteProviderSubscriber.php
@@ -43,7 +43,6 @@ public function onDynamicRouteEvent(RouteBuildEvent $event) {
foreach ($this->entityManager->getRouteProviders($entity_type->id()) as $route_provider) {
// Allow to both return an array of routes or a route collection,
// like route_callbacks in the routing.yml file.
-
$routes = $route_provider->getRoutes($entity_type);
if ($routes instanceof RouteCollection) {
$routes = $routes->all();
diff --git a/core/lib/Drupal/Core/EventSubscriber/FinalExceptionSubscriber.php b/core/lib/Drupal/Core/EventSubscriber/FinalExceptionSubscriber.php
index 4317ff0..b9d2ff4 100644
--- a/core/lib/Drupal/Core/EventSubscriber/FinalExceptionSubscriber.php
+++ b/core/lib/Drupal/Core/EventSubscriber/FinalExceptionSubscriber.php
@@ -97,7 +97,6 @@ public function onException(GetResponseForExceptionEvent $event) {
if (!$this->isErrorLevelVerbose()) {
// Without verbose logging, use a simple message.
-
// We call SafeMarkup::format directly here, rather than use t() since
// we are in the middle of error handling, and we don't want t() to
// cause further errors.
@@ -105,7 +104,6 @@ public function onException(GetResponseForExceptionEvent $event) {
}
else {
// With verbose logging, we will also include a backtrace.
-
$backtrace_exception = $exception;
while ($backtrace_exception->getPrevious()) {
$backtrace_exception = $backtrace_exception->getPrevious();
diff --git a/core/lib/Drupal/Core/EventSubscriber/MaintenanceModeSubscriber.php b/core/lib/Drupal/Core/EventSubscriber/MaintenanceModeSubscriber.php
index c2688ed..f697c7a 100644
--- a/core/lib/Drupal/Core/EventSubscriber/MaintenanceModeSubscriber.php
+++ b/core/lib/Drupal/Core/EventSubscriber/MaintenanceModeSubscriber.php
@@ -110,7 +110,6 @@ public function onKernelRequestMaintenance(GetResponseEvent $event) {
if (!$this->maintenanceMode->exempt($this->account)) {
// Deliver the 503 page if the site is in maintenance mode and the
// logged in user is not allowed to bypass it.
-
// If the request format is not 'html' then show default maintenance
// mode page else show a text/plain page with maintenance message.
if ($request->getRequestFormat() !== 'html') {
diff --git a/core/lib/Drupal/Core/Extension/module.api.php b/core/lib/Drupal/Core/Extension/module.api.php
index 944d354..203a5fa 100644
--- a/core/lib/Drupal/Core/Extension/module.api.php
+++ b/core/lib/Drupal/Core/Extension/module.api.php
@@ -619,7 +619,6 @@ function hook_install_tasks_alter(&$tasks, $install_state) {
function hook_update_N(&$sandbox) {
// For non-batch updates, the signature can simply be:
// function hook_update_N() {
-
// Example function body for adding a field to a database table, which does
// not require a batch operation:
$spec = [
@@ -725,7 +724,6 @@ function hook_post_update_NAME(&$sandbox) {
foreach ($blocks as $block) {
// This block has had conditions removed due to an inability to resolve
// contexts in block_update_8001() so disable it.
-
// Disable currently enabled blocks.
if ($block_update_8001[$block->id()]['status']) {
$block->setStatus(FALSE);
diff --git a/core/lib/Drupal/Core/Field/Plugin/Field/FieldWidget/OptionsWidgetBase.php b/core/lib/Drupal/Core/Field/Plugin/Field/FieldWidget/OptionsWidgetBase.php
index bb773cd..35563f0 100644
--- a/core/lib/Drupal/Core/Field/Plugin/Field/FieldWidget/OptionsWidgetBase.php
+++ b/core/lib/Drupal/Core/Field/Plugin/Field/FieldWidget/OptionsWidgetBase.php
@@ -54,7 +54,6 @@ public function formElement(FieldItemListInterface $items, $delta, array $elemen
$element['#key_column'] = $this->column;
// The rest of the $element is built by child method implementations.
-
return $element;
}
@@ -75,7 +74,6 @@ public static function validateElement(array $element, FormStateInterface $form_
// Drupal\Core\Field\WidgetBase::submit() expects values as
// an array of values keyed by delta first, then by column, while our
// widgets return the opposite.
-
if (is_array($element['#value'])) {
$values = array_values($element['#value']);
}
diff --git a/core/lib/Drupal/Core/Form/form.api.php b/core/lib/Drupal/Core/Form/form.api.php
index 554e271..9efa854 100644
--- a/core/lib/Drupal/Core/Form/form.api.php
+++ b/core/lib/Drupal/Core/Form/form.api.php
@@ -246,7 +246,6 @@ function hook_form_FORM_ID_alter(&$form, \Drupal\Core\Form\FormStateInterface $f
// Modification for the form with the given form ID goes here. For example, if
// FORM_ID is "user_register_form" this code would run only on the user
// registration form.
-
// Add a checkbox to registration form about agreeing to terms of use.
$form['terms_of_use'] = [
'#type' => 'checkbox',
@@ -296,7 +295,6 @@ function hook_form_BASE_FORM_ID_alter(&$form, \Drupal\Core\Form\FormStateInterfa
// Modification for the form with the given BASE_FORM_ID goes here. For
// example, if BASE_FORM_ID is "node_form", this code would run on every
// node form, regardless of node type.
-
// Add a checkbox to the node form about agreeing to terms of use.
$form['terms_of_use'] = [
'#type' => 'checkbox',
diff --git a/core/lib/Drupal/Core/Language/Language.php b/core/lib/Drupal/Core/Language/Language.php
index 5c4068d..bd880f5 100644
--- a/core/lib/Drupal/Core/Language/Language.php
+++ b/core/lib/Drupal/Core/Language/Language.php
@@ -25,7 +25,6 @@ class Language implements LanguageInterface {
];
// Properties within the Language are set up as the default language.
-
/**
* The human readable English name.
*
diff --git a/core/lib/Drupal/Core/Lock/LockBackendAbstract.php b/core/lib/Drupal/Core/Lock/LockBackendAbstract.php
index 41ee27c..3a2097b 100644
--- a/core/lib/Drupal/Core/Lock/LockBackendAbstract.php
+++ b/core/lib/Drupal/Core/Lock/LockBackendAbstract.php
@@ -36,7 +36,6 @@ public function wait($name, $delay = 30) {
// concerns, begin waiting for 25ms, then add 25ms to the wait period each
// time until it reaches 500ms. After this point polling will continue every
// 500ms until $delay is reached.
-
// $delay is passed in seconds, but we will be using usleep(), which takes
// microseconds as a parameter. Multiply it by 1 million so that all
// further numbers are equivalent.
diff --git a/core/lib/Drupal/Core/Menu/LocalTaskDefault.php b/core/lib/Drupal/Core/Menu/LocalTaskDefault.php
index 30fba4f..6d1d1e3 100644
--- a/core/lib/Drupal/Core/Menu/LocalTaskDefault.php
+++ b/core/lib/Drupal/Core/Menu/LocalTaskDefault.php
@@ -51,7 +51,6 @@ public function getRouteParameters(RouteMatchInterface $route_match) {
// slugs in the path patterns. For example, if the route's path pattern is
// /filter/tips/{filter_format} and the path is /filter/tips/plain_text then
// $raw_variables->get('filter_format') == 'plain_text'.
-
$raw_variables = $route_match->getRawParameters();
foreach ($variables as $name) {
diff --git a/core/lib/Drupal/Core/Plugin/Context/ContextHandler.php b/core/lib/Drupal/Core/Plugin/Context/ContextHandler.php
index 2a13d36..a602024 100644
--- a/core/lib/Drupal/Core/Plugin/Context/ContextHandler.php
+++ b/core/lib/Drupal/Core/Plugin/Context/ContextHandler.php
@@ -70,7 +70,6 @@ public function applyContextMapping(ContextAwarePluginInterface $plugin, $contex
/** @var $contexts \Drupal\Core\Plugin\Context\ContextInterface[] */
$mappings += $plugin->getContextMapping();
// Loop through each of the expected contexts.
-
$missing_value = [];
foreach ($plugin->getContextDefinitions() as $plugin_context_id => $plugin_context_definition) {
diff --git a/core/lib/Drupal/Core/Render/Element/HtmlTag.php b/core/lib/Drupal/Core/Render/Element/HtmlTag.php
index 2413784..b747abf4 100644
--- a/core/lib/Drupal/Core/Render/Element/HtmlTag.php
+++ b/core/lib/Drupal/Core/Render/Element/HtmlTag.php
@@ -169,7 +169,6 @@ public static function preRenderConditionalComments($element) {
// browsers, use either the "downlevel-hidden" or "downlevel-revealed"
// technique. See http://wikipedia.org/wiki/Conditional_comment
// for details.
-
// Ensure what we are dealing with is safe.
// This would be done later anyway in drupal_render().
$prefix = isset($element['#prefix']) ? $element['#prefix'] : '';
diff --git a/core/lib/Drupal/Core/Render/RenderCache.php b/core/lib/Drupal/Core/Render/RenderCache.php
index 9ac3381..45ef1e2 100644
--- a/core/lib/Drupal/Core/Render/RenderCache.php
+++ b/core/lib/Drupal/Core/Render/RenderCache.php
@@ -206,7 +206,6 @@ public function set(array &$elements, array $pre_bubbling_elements) {
// towards that state by progressively merging the 'contexts' value
// across requests. That's the strategy employed below and tested in
// \Drupal\Tests\Core\Render\RendererBubblingTest::testConditionalCacheContextBubblingSelfHealing().
-
// Get the cacheability of this element according to the current (stored)
// redirecting cache item, if any.
$redirect_cacheability = new CacheableMetadata();
diff --git a/core/lib/Drupal/Core/Render/theme.api.php b/core/lib/Drupal/Core/Render/theme.api.php
index c12dd3d..74b00d7 100644
--- a/core/lib/Drupal/Core/Render/theme.api.php
+++ b/core/lib/Drupal/Core/Render/theme.api.php
@@ -552,7 +552,6 @@ function hook_preprocess(&$variables, $hook) {
static $hooks;
// Add contextual links to the variables, if the user has permission.
-
if (!\Drupal::currentUser()->hasPermission('access contextual links')) {
return;
}
diff --git a/core/lib/Drupal/Core/Routing/RouteCompiler.php b/core/lib/Drupal/Core/Routing/RouteCompiler.php
index e4aef43..c3e001c 100644
--- a/core/lib/Drupal/Core/Routing/RouteCompiler.php
+++ b/core/lib/Drupal/Core/Routing/RouteCompiler.php
@@ -49,7 +49,6 @@ public static function compile(Route $route) {
// The following parameters are what Symfony uses in
// \Symfony\Component\Routing\Matcher\UrlMatcher::matchCollection().
-
// Set the static prefix to an empty string since it is redundant to
// the matching in \Drupal\Core\Routing\RouteProvider::getRoutesByPath()
// and by skipping it we more easily make the routing case-insensitive.
diff --git a/core/lib/Drupal/Core/Template/TwigExtension.php b/core/lib/Drupal/Core/Template/TwigExtension.php
index 520a8be..69af028 100644
--- a/core/lib/Drupal/Core/Template/TwigExtension.php
+++ b/core/lib/Drupal/Core/Template/TwigExtension.php
@@ -481,7 +481,6 @@ public function escapeFilter(\Twig_Environment $env, $arg, $strategy = 'html', $
// This is a normal render array, which is safe by definition, with
// special simple cases already handled.
-
// Early return if this element was pre-rendered (no need to re-render).
if (isset($arg['#printed']) && $arg['#printed'] == TRUE && isset($arg['#markup']) && strlen($arg['#markup']) > 0) {
return $arg['#markup'];
diff --git a/core/lib/Drupal/Core/Theme/Registry.php b/core/lib/Drupal/Core/Theme/Registry.php
index a0af702..f6364f9 100644
--- a/core/lib/Drupal/Core/Theme/Registry.php
+++ b/core/lib/Drupal/Core/Theme/Registry.php
@@ -449,7 +449,6 @@ protected function processExtension(array &$cache, $name, $type, $theme, $path)
// $result[$hook] will only contain key/value pairs for information being
// overridden. Pull the rest of the information from what was defined by
// an earlier hook.
-
// Fill in the type and path of the module, theme, or engine that
// implements this theme function.
$result[$hook]['type'] = $type;
diff --git a/core/lib/Drupal/Core/Update/UpdateRegistry.php b/core/lib/Drupal/Core/Update/UpdateRegistry.php
index bb0b91f..5a8306b 100644
--- a/core/lib/Drupal/Core/Update/UpdateRegistry.php
+++ b/core/lib/Drupal/Core/Update/UpdateRegistry.php
@@ -122,7 +122,6 @@ public function getPendingUpdateFunctions() {
// We need a) the list of active modules (we get that from the config
// bootstrap factory) and b) the path to the modules, we use the extension
// discovery for that.
-
$this->scanExtensionsAndLoadUpdateFiles();
// First figure out which hook_{$this->updateType}_NAME got executed
diff --git a/core/modules/ban/tests/src/Functional/IpAddressBlockingTest.php b/core/modules/ban/tests/src/Functional/IpAddressBlockingTest.php
index 778f3db..60f68ec 100644
--- a/core/modules/ban/tests/src/Functional/IpAddressBlockingTest.php
+++ b/core/modules/ban/tests/src/Functional/IpAddressBlockingTest.php
@@ -75,7 +75,6 @@ public function testIPAddressValidation() {
// $edit['ip'] = \Drupal::request()->getClientIP();
// $this->drupalPostForm('admin/config/people/ban', $edit, t('Save'));
// $this->assertText(t('You may not ban your own IP address.'));
-
// Test duplicate ip address are not present in the 'blocked_ips' table.
// when they are entered programmatically.
$connection = Database::getConnection();
diff --git a/core/modules/block/block.install b/core/modules/block/block.install
index 6545a19..84553c5 100644
--- a/core/modules/block/block.install
+++ b/core/modules/block/block.install
@@ -23,12 +23,10 @@ function block_install() {
function block_update_8001() {
// This update function updates blocks for the change from
// https://www.drupal.org/node/2354889.
-
// Core visibility context plugins are updated automatically; blocks with
// unknown plugins are disabled and their previous visibility settings are
// saved in key value storage; see change record
// https://www.drupal.org/node/2527840 for more explanation.
-
// These are all the contexts that Drupal core provides.
$context_service_id_map = [
'node.node' => '@node.node_route_context:node',
diff --git a/core/modules/block/block.module b/core/modules/block/block.module
index ee6a4a1..46e2a48 100644
--- a/core/modules/block/block.module
+++ b/core/modules/block/block.module
@@ -172,7 +172,6 @@ function block_theme_suggestions_block(array $variables) {
// contains a hyphen, it will end up as an underscore after this conversion,
// and your function names won't be recognized. So, we need to convert
// hyphens to underscores in block deltas for the theme suggestions.
-
// We can safely explode on : because we know the Block plugin type manager
// enforces that delimiter for all derivatives.
$parts = explode(':', $variables['elements']['#plugin_id']);
diff --git a/core/modules/block/block.post_update.php b/core/modules/block/block.post_update.php
index d5f0852..19fd85e 100644
--- a/core/modules/block/block.post_update.php
+++ b/core/modules/block/block.post_update.php
@@ -36,7 +36,6 @@ function block_post_update_disable_blocks_with_missing_contexts() {
foreach ($blocks as $block) {
// This block has had conditions removed due to an inability to resolve
// contexts in block_update_8001() so disable it.
-
// Disable currently enabled blocks.
if ($block_update_8001[$block->id()]['status']) {
$block->setStatus(FALSE);
diff --git a/core/modules/book/src/BookManager.php b/core/modules/book/src/BookManager.php
index 3db359b..d9fdd51 100644
--- a/core/modules/book/src/BookManager.php
+++ b/core/modules/book/src/BookManager.php
@@ -1125,7 +1125,6 @@ public function bookSubtreeData($link) {
// Compute the real cid for book subtree data.
$tree_cid = 'book-links:subtree-data:' . hash('sha256', serialize($data));
// Cache the data, if it is not already in the cache.
-
if (!\Drupal::cache('data')->get($tree_cid)) {
\Drupal::cache('data')->set($tree_cid, $data, Cache::PERMANENT, ['bid:' . $link['bid']]);
}
diff --git a/core/modules/ckeditor/tests/modules/src/Kernel/CKEditorPluginManagerTest.php b/core/modules/ckeditor/tests/modules/src/Kernel/CKEditorPluginManagerTest.php
index eb02d37..eb8fa8b 100644
--- a/core/modules/ckeditor/tests/modules/src/Kernel/CKEditorPluginManagerTest.php
+++ b/core/modules/ckeditor/tests/modules/src/Kernel/CKEditorPluginManagerTest.php
@@ -31,7 +31,6 @@ protected function setUp() {
parent::setUp();
// Install the Filter module.
-
// Create text format, associate CKEditor.
$filtered_html_format = FilterFormat::create([
'format' => 'filtered_html',
diff --git a/core/modules/ckeditor/tests/modules/src/Kernel/CKEditorTest.php b/core/modules/ckeditor/tests/modules/src/Kernel/CKEditorTest.php
index 4fbead9..5f28955 100644
--- a/core/modules/ckeditor/tests/modules/src/Kernel/CKEditorTest.php
+++ b/core/modules/ckeditor/tests/modules/src/Kernel/CKEditorTest.php
@@ -39,7 +39,6 @@ protected function setUp() {
parent::setUp();
// Install the Filter module.
-
// Create text format, associate CKEditor.
$filtered_html_format = FilterFormat::create([
'format' => 'filtered_html',
diff --git a/core/modules/color/color.module b/core/modules/color/color.module
index 84abac1..4170961 100644
--- a/core/modules/color/color.module
+++ b/core/modules/color/color.module
@@ -544,7 +544,6 @@ function _color_rewrite_stylesheet($theme, &$info, &$paths, $palette, $style) {
}
else {
// Determine the most suitable base color for the next color.
-
// 'a' declarations. Use link.
if (preg_match('@[^a-z0-9_-](a)[^a-z0-9_-][^/{]*{[^{]+$@i', $chunk)) {
$base = 'link';
@@ -681,7 +680,6 @@ function _color_render_images($theme, &$info, &$paths, $palette) {
function _color_shift($given, $ref1, $ref2, $target) {
// We assume that ref2 is a blend of ref1 and target and find
// delta based on the length of the difference vectors.
-
// delta = 1 - |ref2 - ref1| / |white - ref1|
$target = _color_unpack($target, TRUE);
$ref1 = _color_unpack($ref1, TRUE);
diff --git a/core/modules/comment/comment.module b/core/modules/comment/comment.module
index 16949ff..36d1eb9 100644
--- a/core/modules/comment/comment.module
+++ b/core/modules/comment/comment.module
@@ -210,7 +210,6 @@ function comment_node_links_alter(array &$links, NodeInterface $node, array &$co
// can do so by implementing a new field formatter.
// @todo Make this configurable from the formatter. See
// https://www.drupal.org/node/1901110.
-
$comment_links = \Drupal::service('comment.link_builder')->buildCommentedEntityLinks($node, $context);
$links += $comment_links;
}
diff --git a/core/modules/comment/src/CommentStorage.php b/core/modules/comment/src/CommentStorage.php
index d164d8a..4029d4e 100644
--- a/core/modules/comment/src/CommentStorage.php
+++ b/core/modules/comment/src/CommentStorage.php
@@ -144,7 +144,6 @@ public function getNewCommentPageNumber($total_comments, $new_comments, Fieldabl
}
else {
// Threaded comments.
-
// 1. Find all the threads with a new comment.
$unread_threads_query = $this->database->select('comment_field_data', 'comment')
->fields('comment', ['thread'])
diff --git a/core/modules/comment/src/Entity/Comment.php b/core/modules/comment/src/Entity/Comment.php
index c2643fe..4250a6e 100644
--- a/core/modules/comment/src/Entity/Comment.php
+++ b/core/modules/comment/src/Entity/Comment.php
@@ -106,7 +106,6 @@ public function preSave(EntityStorageInterface $storage) {
else {
// This is a comment with a parent comment, so increase the part of
// the thread value at the proper depth.
-
// Get the parent comment:
$parent = $this->getParentComment();
// Strip the "/" from the end of the parent thread.
diff --git a/core/modules/comment/src/Tests/CommentPagerTest.php b/core/modules/comment/src/Tests/CommentPagerTest.php
index 9c48978..af8bc6a 100644
--- a/core/modules/comment/src/Tests/CommentPagerTest.php
+++ b/core/modules/comment/src/Tests/CommentPagerTest.php
@@ -167,7 +167,6 @@ public function testCommentOrderingThreading() {
// - 6
// - 2
// - 5
-
$this->setCommentSettings('default_mode', CommentManagerInterface::COMMENT_MODE_FLAT, 'Comment paging changed.');
$expected_order = [
@@ -262,7 +261,6 @@ public function testCommentNewPageIndicator() {
// - 3
// - 2
// - 5
-
$this->setCommentSettings('default_mode', CommentManagerInterface::COMMENT_MODE_FLAT, 'Comment paging changed.');
$expected_pages = [
diff --git a/core/modules/comment/src/Tests/CommentThreadingTest.php b/core/modules/comment/src/Tests/CommentThreadingTest.php
index 8f412b2..25c2105 100644
--- a/core/modules/comment/src/Tests/CommentThreadingTest.php
+++ b/core/modules/comment/src/Tests/CommentThreadingTest.php
@@ -158,7 +158,6 @@ protected function assertNoParentLink($cid) {
//
//
//
-
$pattern = "//a[@id='comment-$cid']/following-sibling::article//p[contains(@class, 'parent')]";
$this->assertNoFieldByXpath($pattern, NULL, format_string(
'Comment %cid does not have a link to a parent.',
diff --git a/core/modules/comment/src/Tests/Views/WizardTest.php b/core/modules/comment/src/Tests/Views/WizardTest.php
index a7f14c8..0deeff0 100644
--- a/core/modules/comment/src/Tests/Views/WizardTest.php
+++ b/core/modules/comment/src/Tests/Views/WizardTest.php
@@ -52,7 +52,6 @@ public function testCommentWizard() {
// If we update the type first we should get a selection of comment valid
// row plugins as the select field.
-
$this->drupalGet('admin/structure/views/add');
$this->drupalPostForm('admin/structure/views/add', $view, t('Update "of type" choice'));
diff --git a/core/modules/config/src/Tests/ConfigEntityTest.php b/core/modules/config/src/Tests/ConfigEntityTest.php
index a0c857b..5ff7be5 100644
--- a/core/modules/config/src/Tests/ConfigEntityTest.php
+++ b/core/modules/config/src/Tests/ConfigEntityTest.php
@@ -139,7 +139,6 @@ public function testCRUD() {
// Verify that a configuration entity can be saved with an ID of the
// maximum allowed length, but not longer.
-
// Test with a short ID.
$id_length_config_test = entity_create('config_test', [
'id' => $this->randomMachineName(8),
diff --git a/core/modules/config/src/Tests/ConfigImportAllTest.php b/core/modules/config/src/Tests/ConfigImportAllTest.php
index 459be7a..3efba6c 100644
--- a/core/modules/config/src/Tests/ConfigImportAllTest.php
+++ b/core/modules/config/src/Tests/ConfigImportAllTest.php
@@ -74,7 +74,6 @@ public function testInstallUninstall() {
// Delete every field on the site so all modules can be uninstalled. For
// example, if a comment field exists then module becomes required and can
// not be uninstalled.
-
$field_storages = \Drupal::entityManager()->getStorage('field_storage_config')->loadMultiple();
\Drupal::entityManager()->getStorage('field_storage_config')->delete($field_storages);
// Purge the data.
diff --git a/core/modules/config/src/Tests/ConfigImportUITest.php b/core/modules/config/src/Tests/ConfigImportUITest.php
index 772dada..678d24e 100644
--- a/core/modules/config/src/Tests/ConfigImportUITest.php
+++ b/core/modules/config/src/Tests/ConfigImportUITest.php
@@ -296,7 +296,6 @@ public function testImportDiff() {
// The following assertions do not use $this::assertEscaped() because
// \Drupal\Component\Diff\DiffFormatter adds markup that signifies what has
// changed.
-
// Changed values are escaped.
$this->assertText(Html::escape("foo: 'foobar
'"));
$this->assertText(Html::escape("foo: 'foobar
'"));
diff --git a/core/modules/config_translation/tests/src/Functional/ConfigTranslationListUiTest.php b/core/modules/config_translation/tests/src/Functional/ConfigTranslationListUiTest.php
index 586d197..17bd8c5 100644
--- a/core/modules/config_translation/tests/src/Functional/ConfigTranslationListUiTest.php
+++ b/core/modules/config_translation/tests/src/Functional/ConfigTranslationListUiTest.php
@@ -490,7 +490,6 @@ public function testTranslateOperationInListUi() {
$this->doFieldListTest();
// Views is tested in Drupal\config_translation\Tests\ConfigTranslationViewListUiTest
-
// Test the maintenance settings page.
$this->doSettingsPageTest('admin/config/development/maintenance');
// Test the site information settings page.
diff --git a/core/modules/content_translation/src/Controller/ContentTranslationController.php b/core/modules/content_translation/src/Controller/ContentTranslationController.php
index c18eab3..52ca978 100644
--- a/core/modules/content_translation/src/Controller/ContentTranslationController.php
+++ b/core/modules/content_translation/src/Controller/ContentTranslationController.php
@@ -337,7 +337,6 @@ public function add(LanguageInterface $source, LanguageInterface $target, RouteM
// @todo Provide a way to figure out the default form operation. Maybe like
// $operation = isset($info['default_operation']) ? $info['default_operation'] : 'default';
// See https://www.drupal.org/node/2006348.
-
// Use the add form handler, if available, otherwise default.
$operation = $entity->getEntityType()->hasHandlerClass('form', 'add') ? 'add' : 'default';
@@ -370,7 +369,6 @@ public function edit(LanguageInterface $language, RouteMatchInterface $route_mat
// @todo Provide a way to figure out the default form operation. Maybe like
// $operation = isset($info['default_operation']) ? $info['default_operation'] : 'default';
// See https://www.drupal.org/node/2006348.
-
// Use the edit form handler, if available, otherwise default.
$operation = $entity->getEntityType()->hasHandlerClass('form', 'edit') ? 'edit' : 'default';
diff --git a/core/modules/datetime/src/Plugin/views/filter/Date.php b/core/modules/datetime/src/Plugin/views/filter/Date.php
index afd629a..481e0d4 100644
--- a/core/modules/datetime/src/Plugin/views/filter/Date.php
+++ b/core/modules/datetime/src/Plugin/views/filter/Date.php
@@ -95,8 +95,6 @@ protected function opBetween($field) {
$b = intval(strtotime($this->value['max'], $origin));
// Formatting will vary on date storage.
-
-
// Convert to ISO format and format for query. UTC timezone is used since
// dates are stored in UTC.
$a = $this->query->getDateFormat("'" . $this->dateFormatter->format($a, 'custom', DATETIME_DATETIME_STORAGE_FORMAT, DATETIME_STORAGE_TIMEZONE) . "'", $this->dateFormat, TRUE);
diff --git a/core/modules/editor/tests/src/Kernel/EditorManagerTest.php b/core/modules/editor/tests/src/Kernel/EditorManagerTest.php
index d1d6eaf..9dc6206 100644
--- a/core/modules/editor/tests/src/Kernel/EditorManagerTest.php
+++ b/core/modules/editor/tests/src/Kernel/EditorManagerTest.php
@@ -31,7 +31,6 @@ protected function setUp() {
parent::setUp();
// Install the Filter module.
-
// Add text formats.
$filtered_html_format = FilterFormat::create([
'format' => 'filtered_html',
diff --git a/core/modules/editor/tests/src/Kernel/QuickEditIntegrationTest.php b/core/modules/editor/tests/src/Kernel/QuickEditIntegrationTest.php
index 276f559..08f7c20 100644
--- a/core/modules/editor/tests/src/Kernel/QuickEditIntegrationTest.php
+++ b/core/modules/editor/tests/src/Kernel/QuickEditIntegrationTest.php
@@ -67,7 +67,6 @@ protected function setUp() {
parent::setUp();
// Install the Filter module.
-
// Create a field.
$this->fieldName = 'field_textarea';
$this->createFieldWithStorage(
diff --git a/core/modules/editor/tests/src/Unit/EditorXssFilter/StandardTest.php b/core/modules/editor/tests/src/Unit/EditorXssFilter/StandardTest.php
index ce63092..55e591d 100644
--- a/core/modules/editor/tests/src/Unit/EditorXssFilter/StandardTest.php
+++ b/core/modules/editor/tests/src/Unit/EditorXssFilter/StandardTest.php
@@ -56,7 +56,6 @@ public function providerTestFilterXss() {
$data[] = ['Hello, world!
Pink Fairy Armadillotest', 'Hello, world!
Pink Fairy Armadillotest'];
// All cases listed on https://www.owasp.org/index.php/XSS_Filter_Evasion_Cheat_Sheet
-
// No Filter Evasion.
// @see https://www.owasp.org/index.php/XSS_Filter_Evasion_Cheat_Sheet#No_Filter_Evasion
$data[] = ['', ''];
@@ -178,7 +177,6 @@ public function providerTestFilterXss() {
// @see https://www.owasp.org/index.php/XSS_Filter_Evasion_Cheat_Sheet#Escaping_JavaScript_escapes
// This one is irrelevant for Drupal; we *never* output any JavaScript code
// that depends on the URL's query string.
-
// End title tag.
// @see https://www.owasp.org/index.php/XSS_Filter_Evasion_Cheat_Sheet#End_title_tag
$data[] = ['', 'alert("XSS");'];
@@ -390,7 +388,6 @@ public function providerTestFilterXss() {
// US-ASCII encoding.
// @see https://www.owasp.org/index.php/XSS_Filter_Evasion_Cheat_Sheet#US-ASCII_encoding
// This one is irrelevant for Drupal; Drupal *always* outputs UTF-8.
-
// META.
// @see https://www.owasp.org/index.php/XSS_Filter_Evasion_Cheat_Sheet#META
$data[] = ['', ''];
@@ -469,7 +466,6 @@ public function providerTestFilterXss() {
// @see https://www.owasp.org/index.php/XSS_Filter_Evasion_Cheat_Sheet#Locally_hosted_XML_with_embedded_JavaScript_that_is_generated_using_an_XML_data_island
// This one is irrelevant for Drupal; Drupal disallows XML uploads by
// default.
-
// HTML+TIME in XML.
// @see https://www.owasp.org/index.php/XSS_Filter_Evasion_Cheat_Sheet#HTML.2BTIME_in_XML
$data[] = ['">', '<?xml:namespace prefix="t" ns="urn:schemas-microsoft-com:time"><?import namespace="t" implementation="#default#time2">alert("XSS")">'];
@@ -482,7 +478,6 @@ public function providerTestFilterXss() {
// @see https://www.owasp.org/index.php/XSS_Filter_Evasion_Cheat_Sheet#IMG_Embedded_commands
// This one is irrelevant for Drupal; this is actually a CSRF, for which
// Drupal has CSRF protection. See https://www.drupal.org/node/178896.
-
// Cookie manipulation.
// @see https://www.owasp.org/index.php/XSS_Filter_Evasion_Cheat_Sheet#Cookie_manipulation
$data[] = ['', 'alert(\'XSS\')">'];
@@ -490,7 +485,6 @@ public function providerTestFilterXss() {
// UTF-7 encoding.
// @see https://www.owasp.org/index.php/XSS_Filter_Evasion_Cheat_Sheet#UTF-7_encoding
// This one is irrelevant for Drupal; Drupal *always* outputs UTF-8.
-
// XSS using HTML quote encapsulation.
// @see https://www.owasp.org/index.php/XSS_Filter_Evasion_Cheat_Sheet#XSS_using_HTML_quote_encapsulation
$data[] = ['', '" SRC="http://ha.ckers.org/xss.js">'];
@@ -506,7 +500,6 @@ public function providerTestFilterXss() {
// This one is irrelevant for Drupal; Drupal doesn't forbid linking to some
// sites, it only forbids linking to any protocols other than those that are
// whitelisted.
-
// Test XSS filtering on data-attributes.
// @see \Drupal\editor\EditorXssFilter::filterXssDataAttributes()
diff --git a/core/modules/field/field.module b/core/modules/field/field.module
index 1a18b91..c8236d5 100644
--- a/core/modules/field/field.module
+++ b/core/modules/field/field.module
@@ -230,7 +230,6 @@ function field_entity_bundle_delete($entity_type_id, $bundle) {
// \Drupal\Core\Field\Plugin\Field\FieldType\EntityReferenceItem::onDependencyRemoval()
// because we need to take into account bundles that are not provided by a
// config entity type so they are not part of the config dependencies.
-
// Gather a list of all entity reference fields.
$map = \Drupal::entityManager()->getFieldMapByFieldType('entity_reference');
$ids = [];
diff --git a/core/modules/field/src/Tests/FormTest.php b/core/modules/field/src/Tests/FormTest.php
index ff1b3ee..91ec001 100644
--- a/core/modules/field/src/Tests/FormTest.php
+++ b/core/modules/field/src/Tests/FormTest.php
@@ -121,7 +121,6 @@ public function testFieldFormSingle() {
$this->drupalPostForm(NULL, $edit, t('Save'));
$this->assertRaw(t('%name does not accept the value -1.', ['%name' => $this->field['label']]), 'Field validation fails with invalid input.');
// TODO : check that the correct field is flagged for error.
-
// Create an entity
$value = mt_rand(1, 127);
$edit = [
@@ -256,7 +255,6 @@ public function testFieldFormUnlimited() {
$this->assertFieldByName("{$field_name}[1][value]", '', 'New widget is displayed');
$this->assertNoField("{$field_name}[2][value]", 'No extraneous widget is displayed');
// TODO : check that non-field inputs are preserved ('title'), etc.
-
// Yet another time so that we can play with more values -> 3 widgets.
$this->drupalPostForm(NULL, [], t('Add another item'));
@@ -310,7 +308,6 @@ public function testFieldFormUnlimited() {
// value in the middle.
// Submit: check that the entity is updated with correct values
// Re-submit: check that the field can be emptied.
-
// Test with several multiple fields in a form
}
@@ -599,7 +596,6 @@ public function testHiddenField() {
$this->field->save();
// We explicitly do not assign a widget in a form display, so the field
// stays hidden in forms.
-
// Display the entity creation form.
$this->drupalGet($entity_type . '/add');
diff --git a/core/modules/field/src/Tests/String/StringFieldTest.php b/core/modules/field/src/Tests/String/StringFieldTest.php
index 97e470e..3b31082 100644
--- a/core/modules/field/src/Tests/String/StringFieldTest.php
+++ b/core/modules/field/src/Tests/String/StringFieldTest.php
@@ -37,7 +37,6 @@ protected function setUp() {
}
// Test fields.
-
/**
* Test widgets.
*/
diff --git a/core/modules/field/tests/src/Functional/EntityReference/EntityReferenceAutoCreateTest.php b/core/modules/field/tests/src/Functional/EntityReference/EntityReferenceAutoCreateTest.php
index 90895b1..2aea04f 100644
--- a/core/modules/field/tests/src/Functional/EntityReference/EntityReferenceAutoCreateTest.php
+++ b/core/modules/field/tests/src/Functional/EntityReference/EntityReferenceAutoCreateTest.php
@@ -211,7 +211,6 @@ public function testMultipleTargetBundles() {
// @todo Re-enable this test when WebTestBase::curlHeaderCallback() provides
// a way to catch and assert user-triggered errors.
-
// Test the case when the field config settings are inconsistent.
//unset($handler_settings['auto_create_bundle']);
//$field_config->setSetting('handler_settings', $handler_settings);
diff --git a/core/modules/field/tests/src/Kernel/EntityReference/Views/EntityReferenceRelationshipTest.php b/core/modules/field/tests/src/Kernel/EntityReference/Views/EntityReferenceRelationshipTest.php
index 2b0be85..c1a39ed 100644
--- a/core/modules/field/tests/src/Kernel/EntityReference/Views/EntityReferenceRelationshipTest.php
+++ b/core/modules/field/tests/src/Kernel/EntityReference/Views/EntityReferenceRelationshipTest.php
@@ -273,7 +273,6 @@ public function testDataTableRelationshipWithLongFieldName() {
// Test the forward relationship.
//$this->assertEqual($row->entity_test_entity_test_mul__field_data_test_id, 1);
-
// Test that the correct relationship entity is on the row.
$this->assertEqual($row->_relationship_entities['field_test_data_with_a_long_name']->id(), 1);
$this->assertEqual($row->_relationship_entities['field_test_data_with_a_long_name']->bundle(), 'entity_test');
diff --git a/core/modules/field/tests/src/Kernel/FieldCrudTest.php b/core/modules/field/tests/src/Kernel/FieldCrudTest.php
index 6407be7..5073a71 100644
--- a/core/modules/field/tests/src/Kernel/FieldCrudTest.php
+++ b/core/modules/field/tests/src/Kernel/FieldCrudTest.php
@@ -61,7 +61,6 @@ public function setUp() {
// - a minimal $field structure, check all default values are set
// defer actual $field comparison to a helper function, used for the two cases above,
// and for testUpdateField
-
/**
* Test the creation of a field.
*/
@@ -274,7 +273,6 @@ public function testUpdateField() {
public function testDeleteFieldNoData() {
// Deleting and purging fields with data is tested in
// \Drupal\Tests\field\Kernel\BulkDeleteTest.
-
// Create two fields for the same field storage so we can test that only one
// is deleted.
FieldConfig::create($this->fieldDefinition)->save();
diff --git a/core/modules/field/tests/src/Kernel/FieldImportDeleteTest.php b/core/modules/field/tests/src/Kernel/FieldImportDeleteTest.php
index f3408a5..cd65560 100644
--- a/core/modules/field/tests/src/Kernel/FieldImportDeleteTest.php
+++ b/core/modules/field/tests/src/Kernel/FieldImportDeleteTest.php
@@ -37,7 +37,6 @@ public function testImportDelete() {
// - field.field.entity_test.entity_test.field_test_import
// - field.field.entity_test.entity_test.field_test_import_2
// - field.field.entity_test.test_bundle.field_test_import_2
-
$field_name = 'field_test_import';
$field_storage_id = "entity_test.$field_name";
$field_name_2 = 'field_test_import_2';
diff --git a/core/modules/field/tests/src/Kernel/FieldStorageCrudTest.php b/core/modules/field/tests/src/Kernel/FieldStorageCrudTest.php
index 52eb4e4..17fc2c9 100644
--- a/core/modules/field/tests/src/Kernel/FieldStorageCrudTest.php
+++ b/core/modules/field/tests/src/Kernel/FieldStorageCrudTest.php
@@ -27,7 +27,6 @@ class FieldStorageCrudTest extends FieldKernelTestBase {
// - a full fledged $field structure, check that all the values are there
// - a minimal $field structure, check all default values are set
// defer actual $field comparison to a helper function, used for the two cases above
-
/**
* Test the creation of a field storage.
*/
@@ -287,7 +286,6 @@ public function testIndexes() {
public function testDeleteNoData() {
// Deleting and purging field storages with data is tested in
// \Drupal\Tests\field\Kernel\BulkDeleteTest.
-
// Create two fields (so we can test that only one is deleted).
$field_storage_definition = [
'field_name' => 'field_1',
diff --git a/core/modules/file/src/FileViewsData.php b/core/modules/file/src/FileViewsData.php
index 459a37e..43992cc 100644
--- a/core/modules/file/src/FileViewsData.php
+++ b/core/modules/file/src/FileViewsData.php
@@ -111,7 +111,6 @@ public function getViewsData() {
// relationships are type-restricted in the joins declared above, and
// file->entity relationships are type-restricted in the relationship
// declarations below.
-
// Describes relationships between files and nodes.
$data['file_usage']['file_to_node'] = [
'title' => $this->t('Content'),
diff --git a/core/modules/file/src/Tests/DownloadTest.php b/core/modules/file/src/Tests/DownloadTest.php
index c5189fa..37a7149 100644
--- a/core/modules/file/src/Tests/DownloadTest.php
+++ b/core/modules/file/src/Tests/DownloadTest.php
@@ -49,7 +49,6 @@ public function testPrivateFileTransferWithoutPageCache() {
*/
protected function doPrivateFileTransferTest() {
// Set file downloads to private so handler functions get called.
-
// Create a file.
$contents = $this->randomMachineName(8);
$file = $this->createFile(NULL, $contents, 'private');
diff --git a/core/modules/filter/tests/src/Kernel/FilterDefaultConfigTest.php b/core/modules/filter/tests/src/Kernel/FilterDefaultConfigTest.php
index e16bdda..27dbf3f 100644
--- a/core/modules/filter/tests/src/Kernel/FilterDefaultConfigTest.php
+++ b/core/modules/filter/tests/src/Kernel/FilterDefaultConfigTest.php
@@ -20,7 +20,6 @@ protected function setUp() {
// Drupal\filter\FilterPermissions::permissions() builds a URL to output
// a link in the description.
-
$this->installEntitySchema('user');
// Install filter_test module, which ships with custom default format.
diff --git a/core/modules/filter/tests/src/Kernel/FilterKernelTest.php b/core/modules/filter/tests/src/Kernel/FilterKernelTest.php
index cf72bbe..5647d85 100644
--- a/core/modules/filter/tests/src/Kernel/FilterKernelTest.php
+++ b/core/modules/filter/tests/src/Kernel/FilterKernelTest.php
@@ -555,7 +555,6 @@ public function testUrlFilter() {
// @todo Possible categories:
// - absolute, mail, partial
// - characters/encoding, surrounding markup, security
-
// Create a email that is too long.
$long_email = str_repeat('a', 254) . '@example.com';
$too_long_email = str_repeat('b', 255) . '@example.com';
diff --git a/core/modules/filter/tests/src/Kernel/Plugin/migrate/process/FilterSettingsTest.php b/core/modules/filter/tests/src/Kernel/Plugin/migrate/process/FilterSettingsTest.php
index 4250211..5422980 100644
--- a/core/modules/filter/tests/src/Kernel/Plugin/migrate/process/FilterSettingsTest.php
+++ b/core/modules/filter/tests/src/Kernel/Plugin/migrate/process/FilterSettingsTest.php
@@ -48,7 +48,6 @@ public function dataProvider() {
return [
// Tests that the transformed value is identical to the input value when
// destination is not the filter_html.
-
// Test with an empty source array.
[
[],
@@ -74,7 +73,6 @@ public function dataProvider() {
// Tests that the transformed value for 'allowed_html' is altered when the
// destination is filter_html.
-
// Test with an empty source array.
[
[],
diff --git a/core/modules/history/history.views.inc b/core/modules/history/history.views.inc
index a4dcd99..8699700 100644
--- a/core/modules/history/history.views.inc
+++ b/core/modules/history/history.views.inc
@@ -10,7 +10,6 @@
*/
function history_views_data() {
// History table
-
// We're actually defining a specific instance of the table, so let's
// alias it so that we can later add the real table for other purposes if we
// need it.
diff --git a/core/modules/history/src/Plugin/views/filter/HistoryUserTimestamp.php b/core/modules/history/src/Plugin/views/filter/HistoryUserTimestamp.php
index 92a5799..2bbe743 100644
--- a/core/modules/history/src/Plugin/views/filter/HistoryUserTimestamp.php
+++ b/core/modules/history/src/Plugin/views/filter/HistoryUserTimestamp.php
@@ -70,7 +70,6 @@ public function query() {
// Hey, Drupal kills old history, so nodes that haven't been updated
// since HISTORY_READ_LIMIT are bzzzzzzzt outta here!
-
$limit = REQUEST_TIME - HISTORY_READ_LIMIT;
$this->ensureMyTable();
diff --git a/core/modules/image/src/Tests/ImageAdminStylesTest.php b/core/modules/image/src/Tests/ImageAdminStylesTest.php
index 38fd984..2753207 100644
--- a/core/modules/image/src/Tests/ImageAdminStylesTest.php
+++ b/core/modules/image/src/Tests/ImageAdminStylesTest.php
@@ -97,7 +97,6 @@ public function testStyle() {
];
// Add style form.
-
$edit = [
'name' => $style_name,
'label' => $style_label,
@@ -112,7 +111,6 @@ public function testStyle() {
$this->assertLinkByHref($style_path . '/delete');
// Add effect form.
-
// Add each sample effect to the style.
foreach ($effect_edits as $effect => $edit) {
$edit_data = [];
@@ -161,7 +159,6 @@ public function testStyle() {
}
// Image style overview form (ordering and renaming).
-
// Confirm the order of effects is maintained according to the order we
// added the fields.
$effect_edits_order = array_keys($effect_edits);
@@ -228,7 +225,6 @@ public function testStyle() {
$this->assertTrue($order_correct, 'The order of the effects is correctly set by default.');
// Image effect deletion form.
-
// Create an image to make sure it gets flushed after deleting an effect.
$image_path = $this->createSampleImage($style);
$this->assertEqual($this->getImageCount($style), 1, format_string('Image style %style image %file successfully generated.', ['%style' => $style->label(), '%file' => $image_path]));
@@ -265,7 +261,6 @@ public function testStyle() {
$this->assertEqual(count($style->getEffects()), 6, 'Rotate effect with transparent background was added.');
// Style deletion form.
-
// Delete the style.
$this->drupalPostForm($style_path . '/delete', [], t('Delete'));
@@ -276,7 +271,6 @@ public function testStyle() {
$this->assertFalse(ImageStyle::load($style_name), format_string('Image style %style successfully deleted.', ['%style' => $style->label()]));
// Test empty text when there are no image styles.
-
// Delete all image styles.
foreach (ImageStyle::loadMultiple() as $image_style) {
$image_style->delete();
diff --git a/core/modules/image/src/Tests/ImageFieldDisplayTest.php b/core/modules/image/src/Tests/ImageFieldDisplayTest.php
index 8061526..27450e4 100644
--- a/core/modules/image/src/Tests/ImageFieldDisplayTest.php
+++ b/core/modules/image/src/Tests/ImageFieldDisplayTest.php
@@ -252,7 +252,6 @@ public function testImageFieldSettings() {
// We have to create the article first and then edit it because the alt
// and title fields do not display until the image has been attached.
-
// Create alt text for the image.
$alt = $this->randomMachineName();
@@ -385,7 +384,6 @@ public function testImageFieldDefaultImage() {
// Create a node with an image attached and ensure that the default image
// is not displayed.
-
// Create alt text for the image.
$alt = $this->randomMachineName();
diff --git a/core/modules/language/src/LanguageNegotiator.php b/core/modules/language/src/LanguageNegotiator.php
index a236a48..4c81e40 100644
--- a/core/modules/language/src/LanguageNegotiator.php
+++ b/core/modules/language/src/LanguageNegotiator.php
@@ -326,13 +326,11 @@ public function updateConfiguration(array $types) {
// settings are always considered non-configurable. In turn if default
// settings are missing, the language type is always considered
// configurable.
-
// If the language type is locked we can just store its default language
// negotiation settings if it has some, since it is not configurable.
if ($has_default_settings) {
$method_weights = [];
// Default settings are in $info['fixed'].
-
foreach ($info['fixed'] as $weight => $method_id) {
if (isset($method_definitions[$method_id])) {
$method_weights[$method_id] = $weight;
diff --git a/core/modules/language/src/Plugin/LanguageNegotiation/LanguageNegotiationContentEntity.php b/core/modules/language/src/Plugin/LanguageNegotiation/LanguageNegotiationContentEntity.php
index 9218899..405085c 100644
--- a/core/modules/language/src/Plugin/LanguageNegotiation/LanguageNegotiationContentEntity.php
+++ b/core/modules/language/src/Plugin/LanguageNegotiation/LanguageNegotiationContentEntity.php
@@ -103,7 +103,6 @@ public function processOutbound($path, &$options = [], Request $request = NULL,
// If appropriate, process outbound to add a query parameter to the url and
// remove the language option, so that url negotiator does not rewrite the
// url.
-
// First, check if processing conditions are met.
if (!($request && !empty($options['route']) && $this->hasLowerLanguageNegotiationWeight() && $this->meetsContentEntityRoutesCondition($options['route'], $request))) {
return $path;
diff --git a/core/modules/language/tests/src/Functional/LanguageListTest.php b/core/modules/language/tests/src/Functional/LanguageListTest.php
index 8c671cf..f3db76f 100644
--- a/core/modules/language/tests/src/Functional/LanguageListTest.php
+++ b/core/modules/language/tests/src/Functional/LanguageListTest.php
@@ -136,7 +136,6 @@ public function testLanguageList() {
$this->drupalGet('admin/config/regional/language/delete/fr');
$this->assertResponse(404, 'Language no longer found.');
// Make sure the "language_count" state has not changed.
-
// Ensure we can delete the English language. Right now English is the only
// language so we must add a new language and make it the default before
// deleting English.
diff --git a/core/modules/locale/locale.module b/core/modules/locale/locale.module
index b1298e4..5937423 100644
--- a/core/modules/locale/locale.module
+++ b/core/modules/locale/locale.module
@@ -1355,7 +1355,6 @@ function _locale_rebuild_js($langcode = NULL) {
$logger->warning('JavaScript translation file %file.js was lost.', ['%file' => $locale_javascripts[$language->getId()]]);
// Proceed to the 'created' case as the JavaScript translation file has
// been created again.
-
case 'created':
$logger->notice('Created JavaScript translation file for the language %language.', ['%language' => $language->getName()]);
return TRUE;
diff --git a/core/modules/locale/locale.translation.inc b/core/modules/locale/locale.translation.inc
index 571f7d2..3e3954b 100644
--- a/core/modules/locale/locale.translation.inc
+++ b/core/modules/locale/locale.translation.inc
@@ -229,7 +229,6 @@ function locale_translation_source_build($project, $langcode, $filename = NULL)
// Follow-up issue: https://www.drupal.org/node/1842380.
// Convert $source object to a TranslatableProject class and use a typed class
// for $source-file.
-
// Create a source object with data of the project object.
$source = clone $project;
$source->project = $project->name;
diff --git a/core/modules/locale/tests/src/Functional/LocaleTranslationUiTest.php b/core/modules/locale/tests/src/Functional/LocaleTranslationUiTest.php
index e7d28b3..39d59ab 100644
--- a/core/modules/locale/tests/src/Functional/LocaleTranslationUiTest.php
+++ b/core/modules/locale/tests/src/Functional/LocaleTranslationUiTest.php
@@ -229,7 +229,6 @@ public function testJavaScriptTranslation() {
$this->container->get('language_manager')->reset();
// Build the JavaScript translation file.
-
// Retrieve the source string of the first string available in the
// {locales_source} table and translate it.
$query = db_select('locales_source', 's');
@@ -307,7 +306,6 @@ public function testStringValidation() {
];
$this->drupalPostForm('admin/config/regional/translate', $search, t('Filter'));
// Find the edit path.
-
$textarea = current($this->xpath('//textarea'));
$lid = $textarea->getAttribute('name');
foreach ($bad_translations as $translation) {
diff --git a/core/modules/menu_ui/src/Tests/MenuTest.php b/core/modules/menu_ui/src/Tests/MenuTest.php
index dde9502..ee502f0 100644
--- a/core/modules/menu_ui/src/Tests/MenuTest.php
+++ b/core/modules/menu_ui/src/Tests/MenuTest.php
@@ -309,7 +309,6 @@ public function doMenuTests() {
// - item1
// -- item2
// --- item3
-
$this->assertMenuLink($item1->getPluginId(), [
'children' => [$item2->getPluginId(), $item3->getPluginId()],
'parents' => [$item1->getPluginId()],
@@ -349,7 +348,6 @@ public function doMenuTests() {
// - item4
// -- item5
// -- item6
-
$this->assertMenuLink($item4->getPluginId(), [
'children' => [$item5->getPluginId(), $item6->getPluginId()],
'parents' => [$item4->getPluginId()],
@@ -389,7 +387,6 @@ public function doMenuTests() {
// --- item2
// ---- item3
// -- item6
-
$this->assertMenuLink($item1->getPluginId(), [
'children' => [],
'parents' => [$item1->getPluginId()],
diff --git a/core/modules/node/node.api.php b/core/modules/node/node.api.php
index dd151a4..5e904bc 100644
--- a/core/modules/node/node.api.php
+++ b/core/modules/node/node.api.php
@@ -272,7 +272,6 @@ function hook_node_grants_alter(&$grants, \Drupal\Core\Session\AccountInterface
// content. Since some other node access modules might allow this
// permission, we expressly remove it by returning an empty $grants
// array for roles specified in our variable setting.
-
// Get our list of banned roles.
$restricted = \Drupal::config('example.settings')->get('restricted_roles');
diff --git a/core/modules/node/node.tokens.inc b/core/modules/node/node.tokens.inc
index 2fcf0dd..848e1be 100644
--- a/core/modules/node/node.tokens.inc
+++ b/core/modules/node/node.tokens.inc
@@ -140,7 +140,6 @@ function node_tokens($type, $tokens, array $data, array $options, BubbleableMeta
// A summary was requested.
if ($name == 'summary') {
// Generate an optionally trimmed summary of the body field.
-
// Get the 'trim_length' size used for the 'teaser' mode, if
// present, or use the default trim_length size.
$display_options = entity_get_display('node', $node->getType(), 'teaser')->getComponent('body');
diff --git a/core/modules/node/src/NodeViewsData.php b/core/modules/node/src/NodeViewsData.php
index b5aebf4..2b14c12 100644
--- a/core/modules/node/src/NodeViewsData.php
+++ b/core/modules/node/src/NodeViewsData.php
@@ -75,7 +75,6 @@ public function getViewsData() {
];
// Bogus fields for aliasing purposes.
-
// @todo Add similar support to any date field
// @see https://www.drupal.org/node/2337507
$data['node_field_data']['created_fulldate'] = [
diff --git a/core/modules/node/src/Plugin/Search/NodeSearch.php b/core/modules/node/src/Plugin/Search/NodeSearch.php
index c200f34..24c54c4 100644
--- a/core/modules/node/src/Plugin/Search/NodeSearch.php
+++ b/core/modules/node/src/Plugin/Search/NodeSearch.php
@@ -719,7 +719,6 @@ protected function parseAdvancedDefaults($f, $keys) {
}
// Split out the negative, phrase, and OR parts of keywords.
-
// For phrases, the form only supports one phrase.
$matches = [];
$keys = ' ' . $keys . ' ';
diff --git a/core/modules/node/src/Plugin/views/row/Rss.php b/core/modules/node/src/Plugin/views/row/Rss.php
index 9683879..bd755af 100644
--- a/core/modules/node/src/Plugin/views/row/Rss.php
+++ b/core/modules/node/src/Plugin/views/row/Rss.php
@@ -123,7 +123,6 @@ public function render($row) {
// The node gets built and modules add to or modify $node->rss_elements
// and $node->rss_namespaces.
-
$build_mode = $display_mode;
$build = node_view($node, $build_mode);
diff --git a/core/modules/node/src/Plugin/views/wizard/Node.php b/core/modules/node/src/Plugin/views/wizard/Node.php
index 34d4bcf..af58b46 100644
--- a/core/modules/node/src/Plugin/views/wizard/Node.php
+++ b/core/modules/node/src/Plugin/views/wizard/Node.php
@@ -199,7 +199,6 @@ protected function buildFilters(&$form, FormStateInterface $form_state) {
}
// Add the "tagged with" filter to the view.
-
// We construct this filter using taxonomy_index.tid (which limits the
// filtering to a specific vocabulary) rather than
// taxonomy_term_field_data.name (which matches terms in any vocabulary).
@@ -207,12 +206,10 @@ protected function buildFilters(&$form, FormStateInterface $form_state) {
// the autocomplete UI, and also to avoid confusion with other vocabularies
// on the site that may have terms with the same name but are not used for
// free tagging.
-
// The downside is that if there *is* more than one vocabulary on the site
// that is used for free tagging, the wizard will only be able to make the
// "tagged with" filter apply to one of them (see below for the method it
// uses to choose).
-
// Find all "tag-like" taxonomy fields associated with the view's
// entities. If a particular entity type (i.e., bundle) has been
// selected above, then we only search for taxonomy fields associated
diff --git a/core/modules/node/tests/src/Functional/NodeAccessLanguageAwareCombinationTest.php b/core/modules/node/tests/src/Functional/NodeAccessLanguageAwareCombinationTest.php
index e388afc..87af3b4 100644
--- a/core/modules/node/tests/src/Functional/NodeAccessLanguageAwareCombinationTest.php
+++ b/core/modules/node/tests/src/Functional/NodeAccessLanguageAwareCombinationTest.php
@@ -93,7 +93,6 @@ protected function setUp() {
// implemented by one or both modules to enforce that private nodes or
// translations are always private, but we want to test the default,
// additive behavior of node access).
-
// Create six Hungarian nodes with Catalan translations:
// 1. One public with neither language marked as private.
// 2. One private with neither language marked as private.
@@ -251,7 +250,6 @@ public function testNodeAccessLanguageAwareCombination() {
$this->assertNodeAccess($expected_node_access_no_access, $this->nodes['private_no_language_public'], $this->webUser);
// Query the node table with the node access tag in several languages.
-
// Query with no language specified. The fallback (hu or und) will be used.
$select = db_select('node', 'n')
->fields('n', ['nid'])
diff --git a/core/modules/node/tests/src/Functional/NodeAccessLanguageAwareTest.php b/core/modules/node/tests/src/Functional/NodeAccessLanguageAwareTest.php
index 48f9dfa..e6ec6d3 100644
--- a/core/modules/node/tests/src/Functional/NodeAccessLanguageAwareTest.php
+++ b/core/modules/node/tests/src/Functional/NodeAccessLanguageAwareTest.php
@@ -85,7 +85,6 @@ protected function setUp() {
// The node_access_test_language module allows individual translations of a
// node to be marked private (not viewable by normal users).
-
// Create six nodes:
// 1. Four Hungarian nodes with Catalan translations
// - One with neither language marked as private.
@@ -192,7 +191,6 @@ public function testNodeAccessLanguageAware() {
$this->assertNodeAccess($expected_node_access, $this->nodes['no_language_public'], $this->webUser);
// Query the node table with the node access tag in several languages.
-
// Query with no language specified. The fallback (hu) will be used.
$select = db_select('node', 'n')
->fields('n', ['nid'])
diff --git a/core/modules/options/tests/src/Functional/OptionsWidgetsTest.php b/core/modules/options/tests/src/Functional/OptionsWidgetsTest.php
index a2157f1..e2203e3 100644
--- a/core/modules/options/tests/src/Functional/OptionsWidgetsTest.php
+++ b/core/modules/options/tests/src/Functional/OptionsWidgetsTest.php
@@ -296,7 +296,6 @@ public function testSelectListSingle() {
$this->assertFieldValues($entity_init, 'card_1', []);
// Test optgroups.
-
$this->card1->setSetting('allowed_values', []);
$this->card1->setSetting('allowed_values_function', 'options_test_allowed_values_callback');
$this->card1->save();
@@ -392,7 +391,6 @@ public function testSelectListMultiple() {
$this->assertFieldValues($entity_init, 'card_2', []);
// Test the 'None' option.
-
// Check that the 'none' option has no effect if actual options are selected
// as well.
$edit = ['card_2[]' => ['_none' => '_none', 0 => 0]];
@@ -412,9 +410,7 @@ public function testSelectListMultiple() {
// We do not have to test that a required select list with one option is
// auto-selected because the browser does it for us.
-
// Test optgroups.
-
// Use a callback function defining optgroups.
$this->card2->setSetting('allowed_values', []);
$this->card2->setSetting('allowed_values_function', 'options_test_allowed_values_callback');
diff --git a/core/modules/rdf/tests/src/Functional/StandardProfileTest.php b/core/modules/rdf/tests/src/Functional/StandardProfileTest.php
index 76ae39b..cafd5d3 100644
--- a/core/modules/rdf/tests/src/Functional/StandardProfileTest.php
+++ b/core/modules/rdf/tests/src/Functional/StandardProfileTest.php
@@ -408,7 +408,6 @@ protected function assertRdfaArticleProperties($graph, $message_prefix) {
// Tag type.
// @todo Enable with https://www.drupal.org/node/2072791.
//$this->assertEqual($graph->type($this->termUri), 'schema:Thing', 'Tag type was found (schema:Thing).');
-
// Tag name.
$expected_value = [
'type' => 'literal',
diff --git a/core/modules/rest/src/RequestHandler.php b/core/modules/rest/src/RequestHandler.php
index 4b7020c..8001e5b 100644
--- a/core/modules/rest/src/RequestHandler.php
+++ b/core/modules/rest/src/RequestHandler.php
@@ -172,7 +172,6 @@ protected function createArgumentResolver(RouteMatchInterface $route_match, $uns
// not based on position but on name and typehint, specify commonly used
// names here. Similarly, those methods receive the original stored object
// as the first method argument.
-
$route_arguments_entity = NULL;
// Try to find a parameter which is an entity.
foreach ($route_arguments as $value) {
diff --git a/core/modules/rest/tests/src/Functional/EntityResource/Comment/CommentResourceTestBase.php b/core/modules/rest/tests/src/Functional/EntityResource/Comment/CommentResourceTestBase.php
index 5b51014..175605b 100644
--- a/core/modules/rest/tests/src/Functional/EntityResource/Comment/CommentResourceTestBase.php
+++ b/core/modules/rest/tests/src/Functional/EntityResource/Comment/CommentResourceTestBase.php
@@ -280,7 +280,6 @@ public function testPostDxWithoutCriticalBaseFields() {
$this->assertSame(['text/plain; charset=UTF-8'], $response->getHeader('Content-Type'));
$this->assertStringStartsWith('The website encountered an unexpected error. Please try again later.Symfony\Component\HttpKernel\Exception\HttpException: Internal Server Error in Drupal\rest\Plugin\rest\resource\EntityResource->post()', (string) $response->getBody());
//$this->assertResourceErrorResponse(422, "Unprocessable Entity: validation failed.\nentity_type: This value should not be null.\n", $response);
-
// DX: 422 when missing 'entity_id' field.
$request_options[RequestOptions::BODY] = $this->serializer->encode(array_diff_key($this->getNormalizedPostEntity(), ['entity_id' => TRUE]), static::$format);
// @todo Remove the try/catch in favor of the two commented lines in
@@ -296,7 +295,6 @@ public function testPostDxWithoutCriticalBaseFields() {
}
//$response = $this->request('POST', $url, $request_options);
//$this->assertResourceErrorResponse(422, "Unprocessable Entity: validation failed.\nentity_type: This value should not be null.\n", $response);
-
// DX: 422 when missing 'entity_type' field.
$request_options[RequestOptions::BODY] = $this->serializer->encode(array_diff_key($this->getNormalizedPostEntity(), ['field_name' => TRUE]), static::$format);
$response = $this->request('POST', $url, $request_options);
diff --git a/core/modules/rest/tests/src/Functional/Views/StyleSerializerTest.php b/core/modules/rest/tests/src/Functional/Views/StyleSerializerTest.php
index cb9fee2..d3b9e55 100644
--- a/core/modules/rest/tests/src/Functional/Views/StyleSerializerTest.php
+++ b/core/modules/rest/tests/src/Functional/Views/StyleSerializerTest.php
@@ -108,7 +108,6 @@ public function testSerializerResponses() {
$this->assertCacheContexts(['languages:language_interface', 'theme', 'request_format']);
// @todo Due to https://www.drupal.org/node/2352009 we can't yet test the
// propagation of cache max-age.
-
// Test the http Content-type.
$headers = $this->drupalGetHeaders();
$this->assertSame(['application/json'], $headers['Content-Type']);
diff --git a/core/modules/search/search.module b/core/modules/search/search.module
index ae99a87..b3ba1c3 100644
--- a/core/modules/search/search.module
+++ b/core/modules/search/search.module
@@ -445,7 +445,6 @@ function search_index($type, $sid, $langcode, $text) {
$split = preg_split('/\s*<([^>]+?)>\s*/', $text, -1, PREG_SPLIT_DELIM_CAPTURE);
// Note: PHP ensures the array consists of alternating delimiters and literals
// and begins and ends with a literal (inserting $null as required).
-
$tag = FALSE; // Odd/even counter. Tag or no tag.
$score = 1; // Starting score per word
$accum = ' '; // Accumulator for cleaned up data
diff --git a/core/modules/search/src/Tests/SearchCommentTest.php b/core/modules/search/src/Tests/SearchCommentTest.php
index 7047429..64bb092 100644
--- a/core/modules/search/src/Tests/SearchCommentTest.php
+++ b/core/modules/search/src/Tests/SearchCommentTest.php
@@ -194,7 +194,6 @@ public function testSearchResultsComment() {
// @todo Verify the actual search results.
// https://www.drupal.org/node/2551135
-
// Verify there is no script tag in search results.
$this->assertNoRaw('