diff --git a/core/includes/file.inc b/core/includes/file.inc index 5d7837208188..77a743eab044 100644 --- a/core/includes/file.inc +++ b/core/includes/file.inc @@ -99,7 +99,6 @@ function file_stream_wrapper_valid_scheme($scheme) { return \Drupal::service('file_system')->validScheme($scheme); } - /** * Returns the part of a URI after the schema. * @@ -926,7 +925,6 @@ function file_unmanaged_delete_recursive($path, $callback = NULL) { return file_unmanaged_delete($path); } - /** * Moves an uploaded file to a new location. * diff --git a/core/includes/install.inc b/core/includes/install.inc index 0296e9663056..6e6db7aa4eea 100644 --- a/core/includes/install.inc +++ b/core/includes/install.inc @@ -380,7 +380,6 @@ function _drupal_rewrite_settings_is_simple($type, $value) { return $is_integer || $is_float || $is_string || $is_boolean_or_null; } - /** * Helper for drupal_rewrite_settings(). * @@ -451,7 +450,6 @@ function _drupal_rewrite_settings_dump($variable, $variable_name) { return $return; } - /** * Helper for drupal_rewrite_settings(). * diff --git a/core/includes/unicode.inc b/core/includes/unicode.inc index 6a157dc98cc4..e9cc76cd17a9 100644 --- a/core/includes/unicode.inc +++ b/core/includes/unicode.inc @@ -10,6 +10,7 @@ use Drupal\Component\Utility\Unicode; /** * Returns Unicode library status and errors. */ + /** * Moves unicode_requirements() logic to system_requirements(). * diff --git a/core/lib/Drupal.php b/core/lib/Drupal.php index e0b08670b78e..0b37ca2a8ee8 100644 --- a/core/lib/Drupal.php +++ b/core/lib/Drupal.php @@ -141,7 +141,6 @@ class Drupal { return static::$container !== NULL; } - /** * Retrieves a service from the container. * diff --git a/core/lib/Drupal/Component/Assertion/Inspector.php b/core/lib/Drupal/Component/Assertion/Inspector.php index dbf3f511c18e..681f19264828 100644 --- a/core/lib/Drupal/Component/Assertion/Inspector.php +++ b/core/lib/Drupal/Component/Assertion/Inspector.php @@ -333,7 +333,6 @@ class Inspector { return FALSE; } - /** * Asserts that all members are strings matching a regular expression. * diff --git a/core/lib/Drupal/Component/Datetime/DateTimePlus.php b/core/lib/Drupal/Component/Datetime/DateTimePlus.php index e4148a6de885..cb23cba57867 100644 --- a/core/lib/Drupal/Component/Datetime/DateTimePlus.php +++ b/core/lib/Drupal/Component/Datetime/DateTimePlus.php @@ -477,7 +477,6 @@ class DateTimePlus { return $format; } - /** * Examines getLastErrors() to see what errors to report. * diff --git a/core/lib/Drupal/Component/DependencyInjection/Dumper/OptimizedPhpArrayDumper.php b/core/lib/Drupal/Component/DependencyInjection/Dumper/OptimizedPhpArrayDumper.php index dfd2a431ccb6..b4c1da83bf6e 100644 --- a/core/lib/Drupal/Component/DependencyInjection/Dumper/OptimizedPhpArrayDumper.php +++ b/core/lib/Drupal/Component/DependencyInjection/Dumper/OptimizedPhpArrayDumper.php @@ -282,7 +282,6 @@ class OptimizedPhpArrayDumper extends Dumper { return $code; } - /** * Dumps a collection to a PHP array. * diff --git a/core/lib/Drupal/Component/Utility/Random.php b/core/lib/Drupal/Component/Utility/Random.php index cf3483509d73..d1f25ff36c62 100644 --- a/core/lib/Drupal/Component/Utility/Random.php +++ b/core/lib/Drupal/Component/Utility/Random.php @@ -259,7 +259,6 @@ class Random { return $output; } - /** * Create a placeholder image. * diff --git a/core/lib/Drupal/Component/Uuid/Com.php b/core/lib/Drupal/Component/Uuid/Com.php index f9305f5014c1..c18b39a60c4d 100644 --- a/core/lib/Drupal/Component/Uuid/Com.php +++ b/core/lib/Drupal/Component/Uuid/Com.php @@ -8,6 +8,7 @@ namespace Drupal\Component\Uuid; * @see http://php.net/com_create_guid */ class Com implements UuidInterface { + /** * {@inheritdoc} */ diff --git a/core/lib/Drupal/Core/Access/AccessResultForbidden.php b/core/lib/Drupal/Core/Access/AccessResultForbidden.php index 3edae61af34c..dbdbafea3304 100644 --- a/core/lib/Drupal/Core/Access/AccessResultForbidden.php +++ b/core/lib/Drupal/Core/Access/AccessResultForbidden.php @@ -24,7 +24,6 @@ class AccessResultForbidden extends AccessResult implements AccessResultReasonIn $this->reason = $reason; } - /** * {@inheritdoc} */ diff --git a/core/lib/Drupal/Core/Access/CheckProvider.php b/core/lib/Drupal/Core/Access/CheckProvider.php index d6ba1cf6dd3c..51423bdab4ed 100644 --- a/core/lib/Drupal/Core/Access/CheckProvider.php +++ b/core/lib/Drupal/Core/Access/CheckProvider.php @@ -142,6 +142,7 @@ class CheckProvider implements CheckProviderInterface, ContainerAwareInterface { return $checks; } + /** * Compiles a mapping of requirement keys to access checker service IDs. */ diff --git a/core/lib/Drupal/Core/Access/CheckProviderInterface.php b/core/lib/Drupal/Core/Access/CheckProviderInterface.php index a94a92c97842..35e3c18b822a 100644 --- a/core/lib/Drupal/Core/Access/CheckProviderInterface.php +++ b/core/lib/Drupal/Core/Access/CheckProviderInterface.php @@ -15,7 +15,6 @@ use Symfony\Component\Routing\RouteCollection; */ interface CheckProviderInterface { - /** * For each route, saves a list of applicable access checks to the route. * diff --git a/core/lib/Drupal/Core/Ajax/OpenModalDialogCommand.php b/core/lib/Drupal/Core/Ajax/OpenModalDialogCommand.php index 53d6e823eab8..a55b0eaf602c 100644 --- a/core/lib/Drupal/Core/Ajax/OpenModalDialogCommand.php +++ b/core/lib/Drupal/Core/Ajax/OpenModalDialogCommand.php @@ -8,6 +8,7 @@ namespace Drupal\Core\Ajax; * @ingroup ajax */ class OpenModalDialogCommand extends OpenDialogCommand { + /** * Constructs an OpenModalDialog object. * diff --git a/core/lib/Drupal/Core/Config/DatabaseStorage.php b/core/lib/Drupal/Core/Config/DatabaseStorage.php index 0ac6134c3400..2560a461e76d 100644 --- a/core/lib/Drupal/Core/Config/DatabaseStorage.php +++ b/core/lib/Drupal/Core/Config/DatabaseStorage.php @@ -228,7 +228,6 @@ class DatabaseStorage implements StorageInterface { ->execute(); } - /** * Implements Drupal\Core\Config\StorageInterface::rename(). * diff --git a/core/lib/Drupal/Core/Config/Entity/ThirdPartySettingsInterface.php b/core/lib/Drupal/Core/Config/Entity/ThirdPartySettingsInterface.php index 7b2ee060f658..990a5b4fe16b 100644 --- a/core/lib/Drupal/Core/Config/Entity/ThirdPartySettingsInterface.php +++ b/core/lib/Drupal/Core/Config/Entity/ThirdPartySettingsInterface.php @@ -41,7 +41,6 @@ interface ThirdPartySettingsInterface { */ public function getThirdPartySetting($module, $key, $default = NULL); - /** * Gets all third-party settings of a given module. * diff --git a/core/lib/Drupal/Core/Controller/ControllerResolver.php b/core/lib/Drupal/Core/Controller/ControllerResolver.php index 3bae3c53b652..35d433a2abdb 100644 --- a/core/lib/Drupal/Core/Controller/ControllerResolver.php +++ b/core/lib/Drupal/Core/Controller/ControllerResolver.php @@ -80,7 +80,6 @@ class ControllerResolver extends BaseControllerResolver implements ControllerRes return $callable; } - /** * {@inheritdoc} */ diff --git a/core/lib/Drupal/Core/Database/Driver/sqlite/Connection.php b/core/lib/Drupal/Core/Database/Driver/sqlite/Connection.php index a7c1496c7a8f..bbf86681fb51 100644 --- a/core/lib/Drupal/Core/Database/Driver/sqlite/Connection.php +++ b/core/lib/Drupal/Core/Database/Driver/sqlite/Connection.php @@ -165,7 +165,6 @@ class Connection extends DatabaseConnection { return $pdo; } - /** * Destructor for the SQLite connection. * diff --git a/core/lib/Drupal/Core/Database/Driver/sqlite/Select.php b/core/lib/Drupal/Core/Database/Driver/sqlite/Select.php index 6d7caab24bcb..5eaa5157d860 100644 --- a/core/lib/Drupal/Core/Database/Driver/sqlite/Select.php +++ b/core/lib/Drupal/Core/Database/Driver/sqlite/Select.php @@ -8,6 +8,7 @@ use Drupal\Core\Database\Query\Select as QuerySelect; * SQLite implementation of \Drupal\Core\Database\Query\Select. */ class Select extends QuerySelect { + public function forUpdate($set = TRUE) { // SQLite does not support FOR UPDATE so nothing to do. return $this; diff --git a/core/lib/Drupal/Core/Database/Driver/sqlite/Truncate.php b/core/lib/Drupal/Core/Database/Driver/sqlite/Truncate.php index c58ff7d38c08..386912f87801 100644 --- a/core/lib/Drupal/Core/Database/Driver/sqlite/Truncate.php +++ b/core/lib/Drupal/Core/Database/Driver/sqlite/Truncate.php @@ -11,6 +11,7 @@ use Drupal\Core\Database\Query\Truncate as QueryTruncate; * exactly the effect (it is implemented by DROPing the table). */ class Truncate extends QueryTruncate { + public function __toString() { // Create a sanitized comment string to prepend to the query. $comments = $this->connection->makeComment($this->comments); diff --git a/core/lib/Drupal/Core/Datetime/DateHelper.php b/core/lib/Drupal/Core/Datetime/DateHelper.php index b3a8ce75d603..168ef62740a7 100644 --- a/core/lib/Drupal/Core/Datetime/DateHelper.php +++ b/core/lib/Drupal/Core/Datetime/DateHelper.php @@ -334,7 +334,6 @@ class DateHelper { return !$required ? $none + $range : $range; } - /** * Constructs an array of hours. * diff --git a/core/lib/Drupal/Core/Entity/Enhancer/EntityRouteEnhancer.php b/core/lib/Drupal/Core/Entity/Enhancer/EntityRouteEnhancer.php index 850d404baa75..f383fbc98640 100644 --- a/core/lib/Drupal/Core/Entity/Enhancer/EntityRouteEnhancer.php +++ b/core/lib/Drupal/Core/Entity/Enhancer/EntityRouteEnhancer.php @@ -68,7 +68,6 @@ class EntityRouteEnhancer implements EnhancerInterface { return $defaults; } - /** * Update defaults for an entity list. * diff --git a/core/lib/Drupal/Core/Entity/EntityType.php b/core/lib/Drupal/Core/Entity/EntityType.php index 2ef984d6623d..e28fdb0da439 100644 --- a/core/lib/Drupal/Core/Entity/EntityType.php +++ b/core/lib/Drupal/Core/Entity/EntityType.php @@ -836,7 +836,6 @@ class EntityType extends PluginDefinition implements EntityTypeInterface { return $this->group; } - /** * {@inheritdoc} */ diff --git a/core/lib/Drupal/Core/Entity/Query/Sql/Query.php b/core/lib/Drupal/Core/Entity/Query/Sql/Query.php index 50b81bba91d3..7e5db5b9a05c 100644 --- a/core/lib/Drupal/Core/Entity/Query/Sql/Query.php +++ b/core/lib/Drupal/Core/Entity/Query/Sql/Query.php @@ -64,7 +64,6 @@ class Query extends QueryBase implements QueryInterface { $this->connection = $connection; } - /** * {@inheritdoc} */ diff --git a/core/lib/Drupal/Core/Entity/Query/Sql/QueryAggregate.php b/core/lib/Drupal/Core/Entity/Query/Sql/QueryAggregate.php index c791bb555298..f865cf76ef81 100644 --- a/core/lib/Drupal/Core/Entity/Query/Sql/QueryAggregate.php +++ b/core/lib/Drupal/Core/Entity/Query/Sql/QueryAggregate.php @@ -65,7 +65,6 @@ class QueryAggregate extends Query implements QueryAggregateInterface { return $this->conditionAggregate->notExists($field, $function, $langcode); } - /** * Adds the aggregations to the query. * @@ -126,7 +125,6 @@ class QueryAggregate extends Query implements QueryAggregateInterface { return $this; } - /** * Overrides \Drupal\Core\Entity\Query\Sql\Query::finish(). * diff --git a/core/lib/Drupal/Core/EventSubscriber/ActiveLinkResponseFilter.php b/core/lib/Drupal/Core/EventSubscriber/ActiveLinkResponseFilter.php index d87fdf1375c3..b32cf84e22a4 100644 --- a/core/lib/Drupal/Core/EventSubscriber/ActiveLinkResponseFilter.php +++ b/core/lib/Drupal/Core/EventSubscriber/ActiveLinkResponseFilter.php @@ -97,7 +97,6 @@ class ActiveLinkResponseFilter implements EventSubscriberInterface { )); } - /** * Sets the "is-active" class on relevant links. * diff --git a/core/lib/Drupal/Core/EventSubscriber/EnforcedFormResponseSubscriber.php b/core/lib/Drupal/Core/EventSubscriber/EnforcedFormResponseSubscriber.php index 351642617d33..3d7821503f61 100644 --- a/core/lib/Drupal/Core/EventSubscriber/EnforcedFormResponseSubscriber.php +++ b/core/lib/Drupal/Core/EventSubscriber/EnforcedFormResponseSubscriber.php @@ -34,7 +34,6 @@ class EnforcedFormResponseSubscriber implements EventSubscriberInterface { } } - /** * {@inheritdoc} */ diff --git a/core/lib/Drupal/Core/EventSubscriber/Fast404ExceptionHtmlSubscriber.php b/core/lib/Drupal/Core/EventSubscriber/Fast404ExceptionHtmlSubscriber.php index 5b838d49e70f..99c7b021f567 100644 --- a/core/lib/Drupal/Core/EventSubscriber/Fast404ExceptionHtmlSubscriber.php +++ b/core/lib/Drupal/Core/EventSubscriber/Fast404ExceptionHtmlSubscriber.php @@ -43,7 +43,6 @@ class Fast404ExceptionHtmlSubscriber extends HttpExceptionSubscriberBase { $this->httpKernel = $http_kernel; } - /** * {@inheritdoc} */ diff --git a/core/lib/Drupal/Core/Extension/ModuleHandler.php b/core/lib/Drupal/Core/Extension/ModuleHandler.php index 2a2113ff13df..a3555cd98dc6 100644 --- a/core/lib/Drupal/Core/Extension/ModuleHandler.php +++ b/core/lib/Drupal/Core/Extension/ModuleHandler.php @@ -429,7 +429,6 @@ class ModuleHandler implements ModuleHandlerInterface { return $result; } - /** * Triggers an E_USER_DEPRECATED error if any module implements the hook. * diff --git a/core/lib/Drupal/Core/Extension/ThemeHandlerInterface.php b/core/lib/Drupal/Core/Extension/ThemeHandlerInterface.php index a82be6a1d2bd..00433f0b9664 100644 --- a/core/lib/Drupal/Core/Extension/ThemeHandlerInterface.php +++ b/core/lib/Drupal/Core/Extension/ThemeHandlerInterface.php @@ -92,7 +92,6 @@ interface ThemeHandlerInterface { */ public function listInfo(); - /** * Adds a theme extension to the internal listing. * diff --git a/core/lib/Drupal/Core/Field/FieldConfigBase.php b/core/lib/Drupal/Core/Field/FieldConfigBase.php index a32ff423beac..9bb5228441b0 100644 --- a/core/lib/Drupal/Core/Field/FieldConfigBase.php +++ b/core/lib/Drupal/Core/Field/FieldConfigBase.php @@ -263,7 +263,6 @@ abstract class FieldConfigBase extends ConfigEntityBase implements FieldConfigIn return $changed; } - /** * {@inheritdoc} */ diff --git a/core/lib/Drupal/Core/Field/Plugin/Field/FieldType/DecimalItem.php b/core/lib/Drupal/Core/Field/Plugin/Field/FieldType/DecimalItem.php index 00fe4098c764..22da3be42026 100644 --- a/core/lib/Drupal/Core/Field/Plugin/Field/FieldType/DecimalItem.php +++ b/core/lib/Drupal/Core/Field/Plugin/Field/FieldType/DecimalItem.php @@ -105,7 +105,6 @@ class DecimalItem extends NumericItemBase { return $constraints; } - /** * {@inheritdoc} */ @@ -154,7 +153,6 @@ class DecimalItem extends NumericItemBase { return $values; } - /** * Helper method to get the number of decimal digits out of a decimal number. * diff --git a/core/lib/Drupal/Core/Field/WidgetPluginManager.php b/core/lib/Drupal/Core/Field/WidgetPluginManager.php index c7c2d796da74..80e9718769c6 100644 --- a/core/lib/Drupal/Core/Field/WidgetPluginManager.php +++ b/core/lib/Drupal/Core/Field/WidgetPluginManager.php @@ -125,7 +125,6 @@ class WidgetPluginManager extends DefaultPluginManager { return new $plugin_class($plugin_id, $plugin_definition, $configuration['field_definition'], $configuration['settings'], $configuration['third_party_settings']); } - /** * Merges default values for widget configuration. * diff --git a/core/lib/Drupal/Core/ImageToolkit/ImageToolkitBase.php b/core/lib/Drupal/Core/ImageToolkit/ImageToolkitBase.php index a91069328a8e..ce0e283c8efc 100644 --- a/core/lib/Drupal/Core/ImageToolkit/ImageToolkitBase.php +++ b/core/lib/Drupal/Core/ImageToolkit/ImageToolkitBase.php @@ -46,7 +46,6 @@ abstract class ImageToolkitBase extends PluginBase implements ImageToolkitInterf */ protected $logger; - /** * Constructs an ImageToolkitBase object. * diff --git a/core/lib/Drupal/Core/Menu/InaccessibleMenuLink.php b/core/lib/Drupal/Core/Menu/InaccessibleMenuLink.php index 39090e23d9fb..1f163023596f 100644 --- a/core/lib/Drupal/Core/Menu/InaccessibleMenuLink.php +++ b/core/lib/Drupal/Core/Menu/InaccessibleMenuLink.php @@ -69,7 +69,6 @@ class InaccessibleMenuLink extends MenuLinkBase { return $this->wrappedLink->getCacheMaxAge(); } - /** * {@inheritdoc} */ diff --git a/core/lib/Drupal/Core/Menu/MenuLinkManager.php b/core/lib/Drupal/Core/Menu/MenuLinkManager.php index d6a60943489c..af2e4a594d5d 100644 --- a/core/lib/Drupal/Core/Menu/MenuLinkManager.php +++ b/core/lib/Drupal/Core/Menu/MenuLinkManager.php @@ -93,7 +93,6 @@ class MenuLinkManager implements MenuLinkManagerInterface { */ protected $moduleHandler; - /** * Constructs a \Drupal\Core\Menu\MenuLinkManager object. * diff --git a/core/lib/Drupal/Core/PathProcessor/PathProcessorManager.php b/core/lib/Drupal/Core/PathProcessor/PathProcessorManager.php index 2a3181ee2edb..55a952213adb 100644 --- a/core/lib/Drupal/Core/PathProcessor/PathProcessorManager.php +++ b/core/lib/Drupal/Core/PathProcessor/PathProcessorManager.php @@ -86,7 +86,6 @@ class PathProcessorManager implements InboundPathProcessorInterface, OutboundPat return $this->sortedInbound; } - /** * Adds an outbound processor object to the $outboundProcessors property. * diff --git a/core/lib/Drupal/Core/Plugin/Context/Context.php b/core/lib/Drupal/Core/Plugin/Context/Context.php index 49571206c408..a33eca2a454e 100644 --- a/core/lib/Drupal/Core/Plugin/Context/Context.php +++ b/core/lib/Drupal/Core/Plugin/Context/Context.php @@ -123,7 +123,6 @@ class Context extends ComponentContext implements ContextInterface { return $this->contextData; } - /** * {@inheritdoc} */ diff --git a/core/lib/Drupal/Core/Queue/QueueFactory.php b/core/lib/Drupal/Core/Queue/QueueFactory.php index 6df7c5e6026a..cfb072538adf 100644 --- a/core/lib/Drupal/Core/Queue/QueueFactory.php +++ b/core/lib/Drupal/Core/Queue/QueueFactory.php @@ -27,7 +27,6 @@ class QueueFactory implements ContainerAwareInterface { */ protected $settings; - /** * Constructs a queue factory. */ diff --git a/core/lib/Drupal/Core/Routing/AccessAwareRouterInterface.php b/core/lib/Drupal/Core/Routing/AccessAwareRouterInterface.php index 31ce1c4f499e..7b0c1c1fd320 100644 --- a/core/lib/Drupal/Core/Routing/AccessAwareRouterInterface.php +++ b/core/lib/Drupal/Core/Routing/AccessAwareRouterInterface.php @@ -24,7 +24,6 @@ interface AccessAwareRouterInterface extends RouterInterface, RequestMatcherInte */ public function matchRequest(Request $request); - /** * {@inheritdoc} * diff --git a/core/lib/Drupal/Core/StreamWrapper/LocalReadOnlyStream.php b/core/lib/Drupal/Core/StreamWrapper/LocalReadOnlyStream.php index fa12ab488586..29277393ff14 100644 --- a/core/lib/Drupal/Core/StreamWrapper/LocalReadOnlyStream.php +++ b/core/lib/Drupal/Core/StreamWrapper/LocalReadOnlyStream.php @@ -69,7 +69,6 @@ abstract class LocalReadOnlyStream extends LocalStream { return TRUE; } - /** * Support for fwrite(), file_put_contents() etc. * diff --git a/core/lib/Drupal/Core/Theme/ThemeAccessCheck.php b/core/lib/Drupal/Core/Theme/ThemeAccessCheck.php index ed596e8b92d6..f073f46f7cd5 100644 --- a/core/lib/Drupal/Core/Theme/ThemeAccessCheck.php +++ b/core/lib/Drupal/Core/Theme/ThemeAccessCheck.php @@ -27,6 +27,7 @@ class ThemeAccessCheck implements AccessInterface { public function __construct(ThemeHandlerInterface $theme_handler) { $this->themeHandler = $theme_handler; } + /** * Checks access to the theme for routing. * diff --git a/core/lib/Drupal/Core/Validation/ConstraintManager.php b/core/lib/Drupal/Core/Validation/ConstraintManager.php index a2bd604c0a15..79da1d6e8a2f 100644 --- a/core/lib/Drupal/Core/Validation/ConstraintManager.php +++ b/core/lib/Drupal/Core/Validation/ConstraintManager.php @@ -56,7 +56,6 @@ class ConstraintManager extends DefaultPluginManager { return $this->discovery; } - /** * Creates a validation constraint. * diff --git a/core/modules/aggregator/src/Entity/Item.php b/core/modules/aggregator/src/Entity/Item.php index b637171da12f..205252542fdc 100644 --- a/core/modules/aggregator/src/Entity/Item.php +++ b/core/modules/aggregator/src/Entity/Item.php @@ -239,7 +239,6 @@ class Item extends ContentEntityBase implements ItemInterface { return Feed::load($this->getFeedId())->getCacheTags(); } - /** * Entity URI callback. */ diff --git a/core/modules/aggregator/src/Plugin/Block/AggregatorFeedBlock.php b/core/modules/aggregator/src/Plugin/Block/AggregatorFeedBlock.php index f4c8e2a57d18..18390c73ff7f 100644 --- a/core/modules/aggregator/src/Plugin/Block/AggregatorFeedBlock.php +++ b/core/modules/aggregator/src/Plugin/Block/AggregatorFeedBlock.php @@ -57,7 +57,6 @@ class AggregatorFeedBlock extends BlockBase implements ContainerFactoryPluginInt $this->itemStorage = $item_storage; } - /** * {@inheritdoc} */ @@ -71,7 +70,6 @@ class AggregatorFeedBlock extends BlockBase implements ContainerFactoryPluginInt ); } - /** * {@inheritdoc} */ diff --git a/core/modules/aggregator/src/Plugin/Field/FieldFormatter/AggregatorTitleFormatter.php b/core/modules/aggregator/src/Plugin/Field/FieldFormatter/AggregatorTitleFormatter.php index 25e33e9774fc..de8f703508dc 100644 --- a/core/modules/aggregator/src/Plugin/Field/FieldFormatter/AggregatorTitleFormatter.php +++ b/core/modules/aggregator/src/Plugin/Field/FieldFormatter/AggregatorTitleFormatter.php @@ -21,6 +21,7 @@ use Drupal\Core\Url; * ) */ class AggregatorTitleFormatter extends FormatterBase { + /** * {@inheritdoc} */ diff --git a/core/modules/aggregator/tests/src/Functional/DeleteFeedItemTest.php b/core/modules/aggregator/tests/src/Functional/DeleteFeedItemTest.php index a8527f6b1c62..0708a9ea5a31 100644 --- a/core/modules/aggregator/tests/src/Functional/DeleteFeedItemTest.php +++ b/core/modules/aggregator/tests/src/Functional/DeleteFeedItemTest.php @@ -8,6 +8,7 @@ namespace Drupal\Tests\aggregator\Functional; * @group aggregator */ class DeleteFeedItemTest extends AggregatorTestBase { + /** * Tests running "delete items" from 'admin/config/services/aggregator' page. */ diff --git a/core/modules/aggregator/tests/src/Functional/UpdateFeedItemTest.php b/core/modules/aggregator/tests/src/Functional/UpdateFeedItemTest.php index 0f76eee70f87..a7ec0edd5a33 100644 --- a/core/modules/aggregator/tests/src/Functional/UpdateFeedItemTest.php +++ b/core/modules/aggregator/tests/src/Functional/UpdateFeedItemTest.php @@ -10,6 +10,7 @@ use Drupal\aggregator\Entity\Feed; * @group aggregator */ class UpdateFeedItemTest extends AggregatorTestBase { + /** * Tests running "update items" from 'admin/config/services/aggregator' page. */ diff --git a/core/modules/aggregator/tests/src/Functional/UpdateFeedTest.php b/core/modules/aggregator/tests/src/Functional/UpdateFeedTest.php index a81e0aefd1f9..0c76f978a981 100644 --- a/core/modules/aggregator/tests/src/Functional/UpdateFeedTest.php +++ b/core/modules/aggregator/tests/src/Functional/UpdateFeedTest.php @@ -8,6 +8,7 @@ namespace Drupal\Tests\aggregator\Functional; * @group aggregator */ class UpdateFeedTest extends AggregatorTestBase { + /** * Creates a feed and attempts to update it. */ diff --git a/core/modules/block/src/BlockAccessControlHandler.php b/core/modules/block/src/BlockAccessControlHandler.php index 35af61ed5429..265225116e48 100644 --- a/core/modules/block/src/BlockAccessControlHandler.php +++ b/core/modules/block/src/BlockAccessControlHandler.php @@ -67,7 +67,6 @@ class BlockAccessControlHandler extends EntityAccessControlHandler implements En $this->contextRepository = $context_repository; } - /** * {@inheritdoc} */ diff --git a/core/modules/block_content/tests/src/Functional/Views/BlockContentFieldFilterTest.php b/core/modules/block_content/tests/src/Functional/Views/BlockContentFieldFilterTest.php index a016ab9141b7..a77c2e1b297c 100644 --- a/core/modules/block_content/tests/src/Functional/Views/BlockContentFieldFilterTest.php +++ b/core/modules/block_content/tests/src/Functional/Views/BlockContentFieldFilterTest.php @@ -31,7 +31,6 @@ class BlockContentFieldFilterTest extends BlockContentTestBase { */ public $blockContentInfos = []; - /** * {@inheritdoc} */ diff --git a/core/modules/book/src/BookOutlineStorageInterface.php b/core/modules/book/src/BookOutlineStorageInterface.php index c09c52afe6fa..820bc1b2a7c6 100644 --- a/core/modules/book/src/BookOutlineStorageInterface.php +++ b/core/modules/book/src/BookOutlineStorageInterface.php @@ -110,7 +110,6 @@ interface BookOutlineStorageInterface { */ public function insert($link, $parents); - /** * Updates book reference for links that were moved between books. * diff --git a/core/modules/comment/src/Tests/CommentPagerTest.php b/core/modules/comment/src/Tests/CommentPagerTest.php index 726a8e9ca67d..5bbf49c2b473 100644 --- a/core/modules/comment/src/Tests/CommentPagerTest.php +++ b/core/modules/comment/src/Tests/CommentPagerTest.php @@ -12,6 +12,7 @@ use Drupal\node\Entity\Node; * @group comment */ class CommentPagerTest extends CommentTestBase { + /** * Confirms comment paging works correctly with flat and threaded comments. */ diff --git a/core/modules/comment/tests/src/Functional/CommentThreadingTest.php b/core/modules/comment/tests/src/Functional/CommentThreadingTest.php index 9fe7aea09bae..e70629e51bb7 100644 --- a/core/modules/comment/tests/src/Functional/CommentThreadingTest.php +++ b/core/modules/comment/tests/src/Functional/CommentThreadingTest.php @@ -10,6 +10,7 @@ use Drupal\comment\CommentManagerInterface; * @group comment */ class CommentThreadingTest extends CommentTestBase { + /** * Tests the comment threading. */ diff --git a/core/modules/comment/tests/src/Functional/CommentTitleTest.php b/core/modules/comment/tests/src/Functional/CommentTitleTest.php index 65c4937e39ea..9cd7946895bf 100644 --- a/core/modules/comment/tests/src/Functional/CommentTitleTest.php +++ b/core/modules/comment/tests/src/Functional/CommentTitleTest.php @@ -9,6 +9,7 @@ namespace Drupal\Tests\comment\Functional; * @group comment */ class CommentTitleTest extends CommentTestBase { + /** * Tests markup for comments with empty titles. */ diff --git a/core/modules/comment/tests/src/Functional/CommentUninstallTest.php b/core/modules/comment/tests/src/Functional/CommentUninstallTest.php index 6e0c8a2b30f8..96d7eeae8daf 100644 --- a/core/modules/comment/tests/src/Functional/CommentUninstallTest.php +++ b/core/modules/comment/tests/src/Functional/CommentUninstallTest.php @@ -52,7 +52,6 @@ class CommentUninstallTest extends BrowserTestBase { } } - /** * Tests if uninstallation succeeds if the field has been deleted beforehand. */ diff --git a/core/modules/comment/tests/src/Functional/Views/CommentRestExportTest.php b/core/modules/comment/tests/src/Functional/Views/CommentRestExportTest.php index 17e04b5c5cf7..c6e39f83169e 100644 --- a/core/modules/comment/tests/src/Functional/Views/CommentRestExportTest.php +++ b/core/modules/comment/tests/src/Functional/Views/CommentRestExportTest.php @@ -46,7 +46,6 @@ class CommentRestExportTest extends CommentTestBase { $this->drupalLogin($user); } - /** * Test comment row. */ diff --git a/core/modules/comment/tests/src/Functional/Views/WizardTest.php b/core/modules/comment/tests/src/Functional/Views/WizardTest.php index 62fada9fd041..9c02fccdac85 100644 --- a/core/modules/comment/tests/src/Functional/Views/WizardTest.php +++ b/core/modules/comment/tests/src/Functional/Views/WizardTest.php @@ -23,7 +23,6 @@ class WizardTest extends WizardTestBase { */ public static $modules = ['node', 'comment']; - /** * {@inheritdoc} */ diff --git a/core/modules/comment/tests/src/Unit/CommentLinkBuilderTest.php b/core/modules/comment/tests/src/Unit/CommentLinkBuilderTest.php index 8aaae7560981..662f5f332e25 100644 --- a/core/modules/comment/tests/src/Unit/CommentLinkBuilderTest.php +++ b/core/modules/comment/tests/src/Unit/CommentLinkBuilderTest.php @@ -324,7 +324,9 @@ class CommentLinkBuilderTest extends UnitTestCase { namespace Drupal\comment; if (!function_exists('history_read')) { + function history_read() { return 0; } + } diff --git a/core/modules/config/src/Tests/ConfigInstallProfileUnmetDependenciesTest.php b/core/modules/config/src/Tests/ConfigInstallProfileUnmetDependenciesTest.php index 6683580eab74..23199e063299 100644 --- a/core/modules/config/src/Tests/ConfigInstallProfileUnmetDependenciesTest.php +++ b/core/modules/config/src/Tests/ConfigInstallProfileUnmetDependenciesTest.php @@ -79,7 +79,6 @@ class ConfigInstallProfileUnmetDependenciesTest extends InstallerTestBase { // This step is not reached due to the exception. } - /** * Confirms that the installation succeeded. */ diff --git a/core/modules/config/tests/src/Functional/ConfigInstallProfileOverrideTest.php b/core/modules/config/tests/src/Functional/ConfigInstallProfileOverrideTest.php index 1f5232c939ef..187155f70eae 100644 --- a/core/modules/config/tests/src/Functional/ConfigInstallProfileOverrideTest.php +++ b/core/modules/config/tests/src/Functional/ConfigInstallProfileOverrideTest.php @@ -24,7 +24,6 @@ class ConfigInstallProfileOverrideTest extends BrowserTestBase { */ protected $profile = 'testing_config_overrides'; - /** * Tests install profile config changes. */ diff --git a/core/modules/contact/tests/modules/contact_storage_test/contact_storage_test.module b/core/modules/contact/tests/modules/contact_storage_test/contact_storage_test.module index cef976c6ffef..3b49f0aef65c 100644 --- a/core/modules/contact/tests/modules/contact_storage_test/contact_storage_test.module +++ b/core/modules/contact/tests/modules/contact_storage_test/contact_storage_test.module @@ -27,7 +27,6 @@ function contact_storage_test_entity_base_field_info(EntityTypeInterface $entity } } - /** * Implements hook_entity_type_alter(). */ @@ -56,6 +55,7 @@ function contact_storage_test_form_contact_form_form_alter(&$form, FormStateInte ]; $form['#entity_builders'][] = 'contact_storage_test_contact_form_form_builder'; } + /** * Entity builder for the contact form edit form with third party options. * diff --git a/core/modules/content_moderation/src/EntityTypeInfo.php b/core/modules/content_moderation/src/EntityTypeInfo.php index c03cf4922036..80ce9d55df52 100644 --- a/core/modules/content_moderation/src/EntityTypeInfo.php +++ b/core/modules/content_moderation/src/EntityTypeInfo.php @@ -119,7 +119,6 @@ class EntityTypeInfo implements ContainerInjectionInterface { ); } - /** * Adds Moderation configuration to appropriate entity types. * diff --git a/core/modules/content_translation/content_translation.admin.inc b/core/modules/content_translation/content_translation.admin.inc index f7d5a0d21d0c..998359d9ddb9 100644 --- a/core/modules/content_translation/content_translation.admin.inc +++ b/core/modules/content_translation/content_translation.admin.inc @@ -164,6 +164,7 @@ function _content_translation_form_language_content_settings_form_alter(array &$ $form['#validate'][] = 'content_translation_form_language_content_settings_validate'; $form['#submit'][] = 'content_translation_form_language_content_settings_submit'; } + /** * Checks whether translatability should be configurable for a field. * diff --git a/core/modules/editor/src/EditorXssFilter/Standard.php b/core/modules/editor/src/EditorXssFilter/Standard.php index 363497f8d22e..754a014ca626 100644 --- a/core/modules/editor/src/EditorXssFilter/Standard.php +++ b/core/modules/editor/src/EditorXssFilter/Standard.php @@ -116,7 +116,6 @@ class Standard extends Xss implements EditorXssFilterInterface { return $html; } - /** * Get all allowed tags from a restrictions data structure. * diff --git a/core/modules/field/field.api.php b/core/modules/field/field.api.php index 40b42e1dee34..64e2893e71b9 100644 --- a/core/modules/field/field.api.php +++ b/core/modules/field/field.api.php @@ -43,7 +43,6 @@ * @see plugin_api */ - /** * Perform alterations on Field API field types. * diff --git a/core/modules/field/src/FieldUninstallValidator.php b/core/modules/field/src/FieldUninstallValidator.php index 6ee521e00267..62daf4525a67 100644 --- a/core/modules/field/src/FieldUninstallValidator.php +++ b/core/modules/field/src/FieldUninstallValidator.php @@ -85,7 +85,6 @@ class FieldUninstallValidator implements ModuleUninstallValidatorInterface { return $this->fieldStorageConfigStorage->loadByProperties(['module' => $module, 'include_deleted' => TRUE]); } - /** * Returns the label for a specified field type. * diff --git a/core/modules/field/tests/modules/field_test/field_test.module b/core/modules/field/tests/modules/field_test/field_test.module index 4458672f67ec..2c827c2def3f 100644 --- a/core/modules/field/tests/modules/field_test/field_test.module +++ b/core/modules/field/tests/modules/field_test/field_test.module @@ -130,7 +130,6 @@ function field_test_field_widget_multivalue_test_field_widget_multiple_single_va _field_test_alter_widget("hook_field_widget_multivalue_WIDGET_TYPE_form_alter", $elements, $form_state, $context); } - /** * Sets up alterations for widget alter tests. * @@ -164,7 +163,6 @@ function field_test_query_efq_table_prefixing_test_alter(&$query) { $query->join('entity_test', 'et2', '%alias.id = entity_test.id'); } - /** * Implements hook_query_TAG_alter() for tag 'efq_metadata_test'. * diff --git a/core/modules/field/tests/src/Functional/EntityReference/EntityReferenceFieldTranslatedReferenceViewTest.php b/core/modules/field/tests/src/Functional/EntityReference/EntityReferenceFieldTranslatedReferenceViewTest.php index 7ded590c6a3c..cf53a7402a3e 100644 --- a/core/modules/field/tests/src/Functional/EntityReference/EntityReferenceFieldTranslatedReferenceViewTest.php +++ b/core/modules/field/tests/src/Functional/EntityReference/EntityReferenceFieldTranslatedReferenceViewTest.php @@ -302,7 +302,6 @@ class EntityReferenceFieldTranslatedReferenceViewTest extends BrowserTestBase { return $node; } - /** * Create the referenced entity. */ diff --git a/core/modules/field/tests/src/Kernel/EntityReference/EntityReferenceFormatterTest.php b/core/modules/field/tests/src/Kernel/EntityReference/EntityReferenceFormatterTest.php index 1473375b1990..dd9e7d4c0860 100644 --- a/core/modules/field/tests/src/Kernel/EntityReference/EntityReferenceFormatterTest.php +++ b/core/modules/field/tests/src/Kernel/EntityReference/EntityReferenceFormatterTest.php @@ -332,7 +332,6 @@ class EntityReferenceFormatterTest extends EntityKernelTestBase { $this->assertEquals($expected_occurrences, $actual_occurrences); } - /** * Tests the label formatter. */ diff --git a/core/modules/file/src/Plugin/migrate/source/d6/File.php b/core/modules/file/src/Plugin/migrate/source/d6/File.php index 714dec96358b..9ec4887671ee 100644 --- a/core/modules/file/src/Plugin/migrate/source/d6/File.php +++ b/core/modules/file/src/Plugin/migrate/source/d6/File.php @@ -88,6 +88,7 @@ class File extends DrupalSqlBase { 'is_public' => $this->t('TRUE if the files directory is public otherwise FALSE.'), ]; } + /** * {@inheritdoc} */ diff --git a/core/modules/file/src/Tests/DownloadTest.php b/core/modules/file/src/Tests/DownloadTest.php index 8f85f09f9601..2864ed22e81b 100644 --- a/core/modules/file/src/Tests/DownloadTest.php +++ b/core/modules/file/src/Tests/DownloadTest.php @@ -8,6 +8,7 @@ namespace Drupal\file\Tests; * @group file */ class DownloadTest extends FileManagedTestBase { + protected function setUp() { parent::setUp(); // Clear out any hook calls. diff --git a/core/modules/file/src/Tests/FileFieldPathTest.php b/core/modules/file/src/Tests/FileFieldPathTest.php index 300b92c3ca26..bd3bcbf36a48 100644 --- a/core/modules/file/src/Tests/FileFieldPathTest.php +++ b/core/modules/file/src/Tests/FileFieldPathTest.php @@ -10,6 +10,7 @@ use Drupal\file\Entity\File; * @group file */ class FileFieldPathTest extends FileFieldTestBase { + /** * Tests the normal formatter display on node display. */ diff --git a/core/modules/file/src/Tests/FileFieldRevisionTest.php b/core/modules/file/src/Tests/FileFieldRevisionTest.php index 04fbcb8767f6..4fd1d36601bb 100644 --- a/core/modules/file/src/Tests/FileFieldRevisionTest.php +++ b/core/modules/file/src/Tests/FileFieldRevisionTest.php @@ -10,6 +10,7 @@ use Drupal\file\Entity\File; * @group file */ class FileFieldRevisionTest extends FileFieldTestBase { + /** * Tests creating multiple revisions of a node and managing attached files. * diff --git a/core/modules/file/src/Tests/FileManagedFileElementTest.php b/core/modules/file/src/Tests/FileManagedFileElementTest.php index f48532e8d8e0..de9b46111051 100644 --- a/core/modules/file/src/Tests/FileManagedFileElementTest.php +++ b/core/modules/file/src/Tests/FileManagedFileElementTest.php @@ -10,6 +10,7 @@ namespace Drupal\file\Tests; * that aren't related to fields into it. */ class FileManagedFileElementTest extends FileFieldTestBase { + /** * Tests the managed_file element type. */ diff --git a/core/modules/file/src/Tests/FileTokenReplaceTest.php b/core/modules/file/src/Tests/FileTokenReplaceTest.php index 5bfa07b9ab29..e7c7815bb877 100644 --- a/core/modules/file/src/Tests/FileTokenReplaceTest.php +++ b/core/modules/file/src/Tests/FileTokenReplaceTest.php @@ -13,6 +13,7 @@ use Drupal\file\Entity\File; * @group file */ class FileTokenReplaceTest extends FileFieldTestBase { + /** * Creates a file, then tests the tokens generated from it. */ diff --git a/core/modules/file/tests/src/Kernel/CopyTest.php b/core/modules/file/tests/src/Kernel/CopyTest.php index 3193cb661912..0df63c86b270 100644 --- a/core/modules/file/tests/src/Kernel/CopyTest.php +++ b/core/modules/file/tests/src/Kernel/CopyTest.php @@ -10,6 +10,7 @@ use Drupal\file\Entity\File; * @group file */ class CopyTest extends FileManagedUnitTestBase { + /** * Test file copying in the normal, base case. */ diff --git a/core/modules/file/tests/src/Kernel/DeleteTest.php b/core/modules/file/tests/src/Kernel/DeleteTest.php index de5a5bc045d8..9ab4c51e0de0 100644 --- a/core/modules/file/tests/src/Kernel/DeleteTest.php +++ b/core/modules/file/tests/src/Kernel/DeleteTest.php @@ -10,6 +10,7 @@ use Drupal\file\Entity\File; * @group file */ class DeleteTest extends FileManagedUnitTestBase { + /** * Tries deleting a normal file (as opposed to a directory, symlink, etc). */ diff --git a/core/modules/file/tests/src/Kernel/LoadTest.php b/core/modules/file/tests/src/Kernel/LoadTest.php index 9d715395b0ae..413086ad1f49 100644 --- a/core/modules/file/tests/src/Kernel/LoadTest.php +++ b/core/modules/file/tests/src/Kernel/LoadTest.php @@ -10,6 +10,7 @@ use Drupal\file\Entity\File; * @group file */ class LoadTest extends FileManagedUnitTestBase { + /** * Try to load a non-existent file by fid. */ diff --git a/core/modules/file/tests/src/Kernel/MoveTest.php b/core/modules/file/tests/src/Kernel/MoveTest.php index 3183ecfe42d8..72f6801a5516 100644 --- a/core/modules/file/tests/src/Kernel/MoveTest.php +++ b/core/modules/file/tests/src/Kernel/MoveTest.php @@ -10,6 +10,7 @@ use Drupal\file\Entity\File; * @group file */ class MoveTest extends FileManagedUnitTestBase { + /** * Move a normal file. */ diff --git a/core/modules/file/tests/src/Kernel/SaveDataTest.php b/core/modules/file/tests/src/Kernel/SaveDataTest.php index 747c7b1ca546..8d03e3656d02 100644 --- a/core/modules/file/tests/src/Kernel/SaveDataTest.php +++ b/core/modules/file/tests/src/Kernel/SaveDataTest.php @@ -10,6 +10,7 @@ use Drupal\file\Entity\File; * @group file */ class SaveDataTest extends FileManagedUnitTestBase { + /** * Test the file_save_data() function when no filename is provided. */ diff --git a/core/modules/file/tests/src/Kernel/SaveTest.php b/core/modules/file/tests/src/Kernel/SaveTest.php index 193c5684ec1c..73e8b25a5327 100644 --- a/core/modules/file/tests/src/Kernel/SaveTest.php +++ b/core/modules/file/tests/src/Kernel/SaveTest.php @@ -10,6 +10,7 @@ use Drupal\file\Entity\File; * @group file */ class SaveTest extends FileManagedUnitTestBase { + public function testFileSave() { // Create a new file entity. $file = File::create([ diff --git a/core/modules/file/tests/src/Kernel/SpaceUsedTest.php b/core/modules/file/tests/src/Kernel/SpaceUsedTest.php index 650a289cfa9a..fd5c90cac5ed 100644 --- a/core/modules/file/tests/src/Kernel/SpaceUsedTest.php +++ b/core/modules/file/tests/src/Kernel/SpaceUsedTest.php @@ -10,6 +10,7 @@ use Drupal\file\Entity\File; * @group file */ class SpaceUsedTest extends FileManagedUnitTestBase { + protected function setUp() { parent::setUp(); diff --git a/core/modules/file/tests/src/Kernel/UsageTest.php b/core/modules/file/tests/src/Kernel/UsageTest.php index f95e9cd48821..854360e1c65c 100644 --- a/core/modules/file/tests/src/Kernel/UsageTest.php +++ b/core/modules/file/tests/src/Kernel/UsageTest.php @@ -15,6 +15,7 @@ use Drupal\node\Entity\NodeType; * @group file */ class UsageTest extends FileManagedUnitTestBase { + /** * Tests \Drupal\file\FileUsage\DatabaseFileUsageBackend::listUsage(). */ diff --git a/core/modules/file/tests/src/Kernel/ValidateTest.php b/core/modules/file/tests/src/Kernel/ValidateTest.php index 0f348d69bd1c..3e9eb07a8ae5 100644 --- a/core/modules/file/tests/src/Kernel/ValidateTest.php +++ b/core/modules/file/tests/src/Kernel/ValidateTest.php @@ -8,6 +8,7 @@ namespace Drupal\Tests\file\Kernel; * @group file */ class ValidateTest extends FileManagedUnitTestBase { + /** * Test that the validators passed into are checked. */ diff --git a/core/modules/file/tests/src/Kernel/ValidatorTest.php b/core/modules/file/tests/src/Kernel/ValidatorTest.php index b645f3913920..ab0d85229c75 100644 --- a/core/modules/file/tests/src/Kernel/ValidatorTest.php +++ b/core/modules/file/tests/src/Kernel/ValidatorTest.php @@ -137,7 +137,6 @@ class ValidatorTest extends FileManagedUnitTestBase { $this->assertEqual(count($errors), 1, 'An error reported for 0 length filename.', 'File'); } - /** * Test file_validate_size(). */ diff --git a/core/modules/filter/tests/src/Kernel/FilterKernelTest.php b/core/modules/filter/tests/src/Kernel/FilterKernelTest.php index 7b0c029b1b58..1838cfea79f4 100644 --- a/core/modules/filter/tests/src/Kernel/FilterKernelTest.php +++ b/core/modules/filter/tests/src/Kernel/FilterKernelTest.php @@ -388,7 +388,6 @@ class FilterKernelTest extends KernelTestBase { } } - /** * Tests filter settings, defaults, access restrictions and similar. * diff --git a/core/modules/forum/tests/src/Functional/Views/ForumIntegrationTest.php b/core/modules/forum/tests/src/Functional/Views/ForumIntegrationTest.php index 19c000012d7c..4114db5efc16 100644 --- a/core/modules/forum/tests/src/Functional/Views/ForumIntegrationTest.php +++ b/core/modules/forum/tests/src/Functional/Views/ForumIntegrationTest.php @@ -34,7 +34,6 @@ class ForumIntegrationTest extends ViewTestBase { ViewTestData::createTestViews(get_class($this), ['forum_test_views']); } - /** * Tests the integration. */ diff --git a/core/modules/hal/hal.api.php b/core/modules/hal/hal.api.php index 197785cbc390..648de018794f 100644 --- a/core/modules/hal/hal.api.php +++ b/core/modules/hal/hal.api.php @@ -33,7 +33,6 @@ function hook_hal_type_uri_alter(&$uri, $context = []) { } } - /** * Alter the HAL relation URI. * diff --git a/core/modules/hal/src/Normalizer/EntityReferenceItemNormalizer.php b/core/modules/hal/src/Normalizer/EntityReferenceItemNormalizer.php index 1a4ef54e12a8..de9fdc73610a 100644 --- a/core/modules/hal/src/Normalizer/EntityReferenceItemNormalizer.php +++ b/core/modules/hal/src/Normalizer/EntityReferenceItemNormalizer.php @@ -169,7 +169,6 @@ class EntityReferenceItemNormalizer extends FieldItemNormalizer implements UuidR return $normalized; } - /** * {@inheritdoc} */ diff --git a/core/modules/hal/tests/src/Kernel/FileNormalizeTest.php b/core/modules/hal/tests/src/Kernel/FileNormalizeTest.php index 355c09d1a67e..edcf3dca1f84 100644 --- a/core/modules/hal/tests/src/Kernel/FileNormalizeTest.php +++ b/core/modules/hal/tests/src/Kernel/FileNormalizeTest.php @@ -26,7 +26,6 @@ class FileNormalizeTest extends NormalizerTestBase { $this->installEntitySchema('file'); } - /** * Tests the normalize function. */ diff --git a/core/modules/help/src/HelpSectionPluginInterface.php b/core/modules/help/src/HelpSectionPluginInterface.php index 8a725f97941f..cf377c420221 100644 --- a/core/modules/help/src/HelpSectionPluginInterface.php +++ b/core/modules/help/src/HelpSectionPluginInterface.php @@ -17,7 +17,6 @@ use Drupal\Core\Cache\CacheableDependencyInterface; */ interface HelpSectionPluginInterface extends PluginInspectionInterface, CacheableDependencyInterface { - /** * Returns the title of the help section. * diff --git a/core/modules/help/tests/modules/help_page_test/src/Plugin/HelpSection/EmptyHelpSection.php b/core/modules/help/tests/modules/help_page_test/src/Plugin/HelpSection/EmptyHelpSection.php index 6fa2ffd20da2..db390285bbfd 100644 --- a/core/modules/help/tests/modules/help_page_test/src/Plugin/HelpSection/EmptyHelpSection.php +++ b/core/modules/help/tests/modules/help_page_test/src/Plugin/HelpSection/EmptyHelpSection.php @@ -14,6 +14,7 @@ use Drupal\help\Plugin\HelpSection\HelpSectionPluginBase; * ) */ class EmptyHelpSection extends HelpSectionPluginBase { + /** * {@inheritdoc} */ diff --git a/core/modules/image/src/Form/ImageStyleDeleteForm.php b/core/modules/image/src/Form/ImageStyleDeleteForm.php index f49d6b867e25..a0aea333b5cb 100644 --- a/core/modules/image/src/Form/ImageStyleDeleteForm.php +++ b/core/modules/image/src/Form/ImageStyleDeleteForm.php @@ -25,6 +25,7 @@ class ImageStyleDeleteForm extends EntityDeleteForm { public function getQuestion() { return $this->t('Optionally select a style before deleting %style', ['%style' => $this->entity->label()]); } + /** * {@inheritdoc} */ diff --git a/core/modules/image/src/ImageStyleInterface.php b/core/modules/image/src/ImageStyleInterface.php index 7dec36107553..8430fdc2f704 100644 --- a/core/modules/image/src/ImageStyleInterface.php +++ b/core/modules/image/src/ImageStyleInterface.php @@ -42,7 +42,6 @@ interface ImageStyleInterface extends ConfigEntityInterface { */ public function setName($name); - /** * Returns the URI of this image when using this style. * diff --git a/core/modules/language/src/Form/NegotiationBrowserDeleteForm.php b/core/modules/language/src/Form/NegotiationBrowserDeleteForm.php index 1c7cdcbc28f6..fc87845eb610 100644 --- a/core/modules/language/src/Form/NegotiationBrowserDeleteForm.php +++ b/core/modules/language/src/Form/NegotiationBrowserDeleteForm.php @@ -29,7 +29,6 @@ class NegotiationBrowserDeleteForm extends ConfirmFormBase { return ['language.mappings']; } - /** * {@inheritdoc} */ diff --git a/core/modules/language/src/HttpKernel/PathProcessorLanguage.php b/core/modules/language/src/HttpKernel/PathProcessorLanguage.php index 8bb4f838e91d..9feae277c7f8 100644 --- a/core/modules/language/src/HttpKernel/PathProcessorLanguage.php +++ b/core/modules/language/src/HttpKernel/PathProcessorLanguage.php @@ -60,7 +60,6 @@ class PathProcessorLanguage implements InboundPathProcessorInterface, OutboundPa */ protected $configSubscriber; - /** * Constructs a PathProcessorLanguage object. * diff --git a/core/modules/language/src/Plugin/Block/LanguageBlock.php b/core/modules/language/src/Plugin/Block/LanguageBlock.php index 8e7ee918d50b..be8571b975f3 100644 --- a/core/modules/language/src/Plugin/Block/LanguageBlock.php +++ b/core/modules/language/src/Plugin/Block/LanguageBlock.php @@ -57,7 +57,6 @@ class LanguageBlock extends BlockBase implements ContainerFactoryPluginInterface $this->pathMatcher = $path_matcher; } - /** * {@inheritdoc} */ @@ -71,7 +70,6 @@ class LanguageBlock extends BlockBase implements ContainerFactoryPluginInterface ); } - /** * {@inheritdoc} */ diff --git a/core/modules/language/tests/src/Kernel/Migrate/d7/MigrateLanguageContentSettingsTest.php b/core/modules/language/tests/src/Kernel/Migrate/d7/MigrateLanguageContentSettingsTest.php index d7a4551d6f34..fdba37073e4f 100644 --- a/core/modules/language/tests/src/Kernel/Migrate/d7/MigrateLanguageContentSettingsTest.php +++ b/core/modules/language/tests/src/Kernel/Migrate/d7/MigrateLanguageContentSettingsTest.php @@ -17,6 +17,7 @@ class MigrateLanguageContentSettingsTest extends MigrateDrupal7TestBase { * {@inheritdoc} */ public static $modules = ['node', 'text', 'language', 'content_translation', 'menu_ui']; + /** * {@inheritdoc} */ diff --git a/core/modules/language/tests/src/Unit/LanguageNegotiationUrlTest.php b/core/modules/language/tests/src/Unit/LanguageNegotiationUrlTest.php index 825ca7d863dc..981228956ea6 100644 --- a/core/modules/language/tests/src/Unit/LanguageNegotiationUrlTest.php +++ b/core/modules/language/tests/src/Unit/LanguageNegotiationUrlTest.php @@ -256,7 +256,9 @@ class LanguageNegotiationUrlTest extends UnitTestCase { namespace Drupal\language\Plugin\LanguageNegotiation; if (!function_exists('base_path')) { + function base_path() { return '/'; } + } diff --git a/core/modules/link/src/Plugin/Field/FieldWidget/LinkWidget.php b/core/modules/link/src/Plugin/Field/FieldWidget/LinkWidget.php index 281634c46c8c..7570fe92414a 100644 --- a/core/modules/link/src/Plugin/Field/FieldWidget/LinkWidget.php +++ b/core/modules/link/src/Plugin/Field/FieldWidget/LinkWidget.php @@ -398,7 +398,6 @@ class LinkWidget extends WidgetBase { return $values; } - /** * {@inheritdoc} * diff --git a/core/modules/link/tests/src/Functional/LinkFieldTest.php b/core/modules/link/tests/src/Functional/LinkFieldTest.php index e5c8cb3fa171..fc6c9fee2655 100644 --- a/core/modules/link/tests/src/Functional/LinkFieldTest.php +++ b/core/modules/link/tests/src/Functional/LinkFieldTest.php @@ -662,7 +662,6 @@ class LinkFieldTest extends BrowserTestBase { $this->assertEqual($form[$field_name]['widget'][0]['uri']['#link_type'], $link_type); } - /** * Tests editing a link to a non-node entity. */ diff --git a/core/modules/locale/locale.module b/core/modules/locale/locale.module index dbc6dfd724ec..8bc05fc75fa6 100644 --- a/core/modules/locale/locale.module +++ b/core/modules/locale/locale.module @@ -308,7 +308,6 @@ function locale_get_plural($count, $langcode = NULL) { return $plural_indexes[$langcode][$count]; } - /** * Implements hook_modules_installed(). */ @@ -1372,6 +1371,7 @@ function _locale_rebuild_js($langcode = NULL) { return TRUE; } } + /** * Form element callback: After build changes to the language update table. * diff --git a/core/modules/locale/src/Form/ImportForm.php b/core/modules/locale/src/Form/ImportForm.php index 708a121de69a..122093ae69d2 100644 --- a/core/modules/locale/src/Form/ImportForm.php +++ b/core/modules/locale/src/Form/ImportForm.php @@ -46,6 +46,7 @@ class ImportForm extends FormBase { $container->get('language_manager') ); } + /** * Constructs a form for language import. * diff --git a/core/modules/locale/src/SourceString.php b/core/modules/locale/src/SourceString.php index a1af509c2215..dd59954bf7c1 100644 --- a/core/modules/locale/src/SourceString.php +++ b/core/modules/locale/src/SourceString.php @@ -10,6 +10,7 @@ namespace Drupal\locale; * value, and is assumed to be in English language. */ class SourceString extends StringBase { + /** * {@inheritdoc} */ diff --git a/core/modules/locale/tests/src/Kernel/LocaleConfigSubscriberTest.php b/core/modules/locale/tests/src/Kernel/LocaleConfigSubscriberTest.php index 3a12efdf2473..0368350f0adb 100644 --- a/core/modules/locale/tests/src/Kernel/LocaleConfigSubscriberTest.php +++ b/core/modules/locale/tests/src/Kernel/LocaleConfigSubscriberTest.php @@ -187,7 +187,6 @@ class LocaleConfigSubscriberTest extends KernelTestBase { $this->assertNoTranslation($config_name, $langcode); } - /** * Sets up a configuration string with a translation. * diff --git a/core/modules/locale/tests/src/Kernel/LocaleTranslationProjectsTest.php b/core/modules/locale/tests/src/Kernel/LocaleTranslationProjectsTest.php index 370c0146de02..8b9b7974dc59 100644 --- a/core/modules/locale/tests/src/Kernel/LocaleTranslationProjectsTest.php +++ b/core/modules/locale/tests/src/Kernel/LocaleTranslationProjectsTest.php @@ -41,7 +41,6 @@ class LocaleTranslationProjectsTest extends KernelTestBase { \Drupal::state()->set('locale.remove_core_project', TRUE); } - /** * Tests locale_translation_clear_cache_projects(). */ diff --git a/core/modules/locale/tests/src/Unit/StringBaseTest.php b/core/modules/locale/tests/src/Unit/StringBaseTest.php index 96bb41bc22b7..7436c70072aa 100644 --- a/core/modules/locale/tests/src/Unit/StringBaseTest.php +++ b/core/modules/locale/tests/src/Unit/StringBaseTest.php @@ -21,7 +21,6 @@ class StringBaseTest extends UnitTestCase { $string->save(); } - /** * @covers ::delete */ diff --git a/core/modules/media/src/Plugin/media/Source/File.php b/core/modules/media/src/Plugin/media/Source/File.php index 0f7631ec2771..480667760f8d 100644 --- a/core/modules/media/src/Plugin/media/Source/File.php +++ b/core/modules/media/src/Plugin/media/Source/File.php @@ -36,7 +36,6 @@ class File extends MediaSourceBase { */ const METADATA_ATTRIBUTE_SIZE = 'filesize'; - /** * {@inheritdoc} */ diff --git a/core/modules/menu_link_content/src/Entity/MenuLinkContent.php b/core/modules/menu_link_content/src/Entity/MenuLinkContent.php index 129404407bbe..0b8e7ccae0e9 100644 --- a/core/modules/menu_link_content/src/Entity/MenuLinkContent.php +++ b/core/modules/menu_link_content/src/Entity/MenuLinkContent.php @@ -193,6 +193,7 @@ class MenuLinkContent extends ContentEntityBase implements MenuLinkContentInterf $this->setRequiresRediscovery(FALSE); } } + /** * {@inheritdoc} */ diff --git a/core/modules/menu_link_content/tests/src/Kernel/PathAliasMenuLinkContentTest.php b/core/modules/menu_link_content/tests/src/Kernel/PathAliasMenuLinkContentTest.php index 046deb4a00fd..6a7cf4a363d1 100644 --- a/core/modules/menu_link_content/tests/src/Kernel/PathAliasMenuLinkContentTest.php +++ b/core/modules/menu_link_content/tests/src/Kernel/PathAliasMenuLinkContentTest.php @@ -44,7 +44,6 @@ class PathAliasMenuLinkContentTest extends KernelTestBase { ->addTag('path_processor_inbound', ['priority' => 100]); } - /** * Tests the path aliasing changing. */ diff --git a/core/modules/menu_ui/menu_ui.module b/core/modules/menu_ui/menu_ui.module index cb1e85307f5c..ab084375ec54 100644 --- a/core/modules/menu_ui/menu_ui.module +++ b/core/modules/menu_ui/menu_ui.module @@ -496,7 +496,6 @@ function menu_ui_preprocess_block(&$variables) { } } - /** * Implements hook_system_breadcrumb_alter(). */ diff --git a/core/modules/migrate/src/Plugin/Discovery/AnnotatedClassDiscoveryAutomatedProviders.php b/core/modules/migrate/src/Plugin/Discovery/AnnotatedClassDiscoveryAutomatedProviders.php index 180be0a3aae5..a1f973746b2c 100644 --- a/core/modules/migrate/src/Plugin/Discovery/AnnotatedClassDiscoveryAutomatedProviders.php +++ b/core/modules/migrate/src/Plugin/Discovery/AnnotatedClassDiscoveryAutomatedProviders.php @@ -48,7 +48,6 @@ class AnnotatedClassDiscoveryAutomatedProviders extends AnnotatedClassDiscovery $this->finder = new ClassFinder(); } - /** * {@inheritdoc} */ diff --git a/core/modules/migrate/src/Plugin/MigrateIdMapInterface.php b/core/modules/migrate/src/Plugin/MigrateIdMapInterface.php index 52a57b69f7d2..3482e4d3b8d4 100644 --- a/core/modules/migrate/src/Plugin/MigrateIdMapInterface.php +++ b/core/modules/migrate/src/Plugin/MigrateIdMapInterface.php @@ -103,7 +103,6 @@ interface MigrateIdMapInterface extends \Iterator, PluginInspectionInterface { */ public function importedCount(); - /** * Returns a count of items which are marked as needing update. * diff --git a/core/modules/migrate/src/Plugin/Migration.php b/core/modules/migrate/src/Plugin/Migration.php index 3007815d041f..5ff495b266c7 100644 --- a/core/modules/migrate/src/Plugin/Migration.php +++ b/core/modules/migrate/src/Plugin/Migration.php @@ -554,7 +554,6 @@ class Migration extends PluginBase implements MigrationInterface, RequirementsIn return $this; } - /** * {@inheritdoc} */ diff --git a/core/modules/migrate/src/Plugin/MigrationPluginManager.php b/core/modules/migrate/src/Plugin/MigrationPluginManager.php index 74f151999bd4..d06341fcc89d 100644 --- a/core/modules/migrate/src/Plugin/MigrationPluginManager.php +++ b/core/modules/migrate/src/Plugin/MigrationPluginManager.php @@ -160,7 +160,6 @@ class MigrationPluginManager extends DefaultPluginManager implements MigrationPl return $plugin_ids; } - /** * {@inheritdoc} */ diff --git a/core/modules/migrate/tests/src/Unit/destination/EntityRevisionTest.php b/core/modules/migrate/tests/src/Unit/destination/EntityRevisionTest.php index 8a6fe9a43dbc..7c331910e210 100644 --- a/core/modules/migrate/tests/src/Unit/destination/EntityRevisionTest.php +++ b/core/modules/migrate/tests/src/Unit/destination/EntityRevisionTest.php @@ -176,7 +176,6 @@ class EntityRevisionTest extends UnitTestCase { $this->assertEquals([1234], $destination->save($entity->reveal(), [])); } - /** * Helper method to create an entity revision destination with mock services. * diff --git a/core/modules/migrate/tests/src/Unit/destination/PerComponentEntityDisplayTest.php b/core/modules/migrate/tests/src/Unit/destination/PerComponentEntityDisplayTest.php index f450984d4889..5c4a856c0b11 100644 --- a/core/modules/migrate/tests/src/Unit/destination/PerComponentEntityDisplayTest.php +++ b/core/modules/migrate/tests/src/Unit/destination/PerComponentEntityDisplayTest.php @@ -53,13 +53,16 @@ class PerComponentEntityDisplayTest extends MigrateTestCase { class TestPerComponentEntityDisplay extends ComponentEntityDisplayBase { const MODE_NAME = 'view_mode'; protected $testValues; + public function __construct($entity) { $this->entity = $entity; } + protected function getEntity($entity_type, $bundle, $view_mode) { $this->testValues = func_get_args(); return $this->entity; } + public function getTestValues() { return $this->testValues; } diff --git a/core/modules/migrate/tests/src/Unit/destination/PerComponentEntityFormDisplayTest.php b/core/modules/migrate/tests/src/Unit/destination/PerComponentEntityFormDisplayTest.php index 157d45e3393e..218d9a6585c1 100644 --- a/core/modules/migrate/tests/src/Unit/destination/PerComponentEntityFormDisplayTest.php +++ b/core/modules/migrate/tests/src/Unit/destination/PerComponentEntityFormDisplayTest.php @@ -53,13 +53,16 @@ class PerComponentEntityFormDisplayTest extends MigrateTestCase { class TestPerComponentEntityFormDisplay extends PerComponentEntityFormDisplay { const MODE_NAME = 'form_mode'; protected $testValues; + public function __construct($entity) { $this->entity = $entity; } + protected function getEntity($entity_type, $bundle, $form_mode) { $this->testValues = func_get_args(); return $this->entity; } + public function getTestValues() { return $this->testValues; } diff --git a/core/modules/migrate/tests/src/Unit/process/ConcatTest.php b/core/modules/migrate/tests/src/Unit/process/ConcatTest.php index e7873fee3988..8acbc11573a1 100644 --- a/core/modules/migrate/tests/src/Unit/process/ConcatTest.php +++ b/core/modules/migrate/tests/src/Unit/process/ConcatTest.php @@ -53,6 +53,7 @@ class ConcatTest extends MigrateProcessTestCase { } class TestConcat extends Concat { + public function __construct() { } diff --git a/core/modules/migrate/tests/src/Unit/process/GetTest.php b/core/modules/migrate/tests/src/Unit/process/GetTest.php index 5bc6247e7d8b..67f56b0a78b7 100644 --- a/core/modules/migrate/tests/src/Unit/process/GetTest.php +++ b/core/modules/migrate/tests/src/Unit/process/GetTest.php @@ -114,8 +114,10 @@ class GetTest extends MigrateProcessTestCase { namespace Drupal\migrate\Plugin\migrate\process; class TestGet extends Get { + public function __construct() { } + public function setSource($source) { $this->configuration['source'] = $source; } diff --git a/core/modules/node/node.module b/core/modules/node/node.module index 5822cb07f7ec..b26be926846b 100644 --- a/core/modules/node/node.module +++ b/core/modules/node/node.module @@ -1035,7 +1035,6 @@ function node_access_view_all_nodes($account = NULL) { return $access[$account->id()]; } - /** * Implements hook_query_TAG_alter(). * diff --git a/core/modules/node/src/Entity/Node.php b/core/modules/node/src/Entity/Node.php index ccee95b3a663..827c459f724e 100644 --- a/core/modules/node/src/Entity/Node.php +++ b/core/modules/node/src/Entity/Node.php @@ -212,7 +212,6 @@ class Node extends EditorialContentEntityBase implements NodeInterface { return $this->get('created')->value; } - /** * {@inheritdoc} */ diff --git a/core/modules/node/src/NodeAccessControlHandler.php b/core/modules/node/src/NodeAccessControlHandler.php index a8a0bc6241e0..5ccc6425e36d 100644 --- a/core/modules/node/src/NodeAccessControlHandler.php +++ b/core/modules/node/src/NodeAccessControlHandler.php @@ -50,7 +50,6 @@ class NodeAccessControlHandler extends EntityAccessControlHandler implements Nod ); } - /** * {@inheritdoc} */ diff --git a/core/modules/node/src/Plugin/views/filter/Access.php b/core/modules/node/src/Plugin/views/filter/Access.php index 71e8e95e5aac..38f16377c424 100644 --- a/core/modules/node/src/Plugin/views/filter/Access.php +++ b/core/modules/node/src/Plugin/views/filter/Access.php @@ -16,7 +16,9 @@ use Drupal\views\Plugin\views\filter\FilterPluginBase; class Access extends FilterPluginBase { public function adminSummary() {} + protected function operatorForm(&$form, FormStateInterface $form_state) {} + public function canExpose() { return FALSE; } diff --git a/core/modules/node/tests/src/Functional/PageViewTest.php b/core/modules/node/tests/src/Functional/PageViewTest.php index 9f7e41402c9c..1f796d83c7cf 100644 --- a/core/modules/node/tests/src/Functional/PageViewTest.php +++ b/core/modules/node/tests/src/Functional/PageViewTest.php @@ -10,6 +10,7 @@ use Drupal\node\Entity\Node; * @group node */ class PageViewTest extends NodeTestBase { + /** * Tests an anonymous and unpermissioned user attempting to edit the node. */ diff --git a/core/modules/node/tests/src/Kernel/NodeListBuilderTest.php b/core/modules/node/tests/src/Kernel/NodeListBuilderTest.php index 0a0f145dd9a4..420b30661e5a 100644 --- a/core/modules/node/tests/src/Kernel/NodeListBuilderTest.php +++ b/core/modules/node/tests/src/Kernel/NodeListBuilderTest.php @@ -23,7 +23,6 @@ class NodeListBuilderTest extends KernelTestBase { $this->installEntitySchema('node'); } - /** * Tests that the correct cache contexts are set. */ diff --git a/core/modules/options/tests/src/Functional/OptionsDynamicValuesValidationTest.php b/core/modules/options/tests/src/Functional/OptionsDynamicValuesValidationTest.php index 9205011ed8a9..7ef535c96c16 100644 --- a/core/modules/options/tests/src/Functional/OptionsDynamicValuesValidationTest.php +++ b/core/modules/options/tests/src/Functional/OptionsDynamicValuesValidationTest.php @@ -8,6 +8,7 @@ namespace Drupal\Tests\options\Functional; * @group options */ class OptionsDynamicValuesValidationTest extends OptionsDynamicValuesTestBase { + /** * Test that allowed values function gets the entity. */ diff --git a/core/modules/options/tests/src/Functional/OptionsSelectDynamicValuesTest.php b/core/modules/options/tests/src/Functional/OptionsSelectDynamicValuesTest.php index b661c0d53625..e94126504062 100644 --- a/core/modules/options/tests/src/Functional/OptionsSelectDynamicValuesTest.php +++ b/core/modules/options/tests/src/Functional/OptionsSelectDynamicValuesTest.php @@ -8,6 +8,7 @@ namespace Drupal\Tests\options\Functional; * @group options */ class OptionsSelectDynamicValuesTest extends OptionsDynamicValuesTestBase { + /** * Tests the 'options_select' widget (single select). */ diff --git a/core/modules/options/tests/src/Functional/OptionsWidgetsTest.php b/core/modules/options/tests/src/Functional/OptionsWidgetsTest.php index a2157f1f918c..a6cee048a87f 100644 --- a/core/modules/options/tests/src/Functional/OptionsWidgetsTest.php +++ b/core/modules/options/tests/src/Functional/OptionsWidgetsTest.php @@ -35,7 +35,6 @@ class OptionsWidgetsTest extends FieldTestBase { */ protected $card2; - protected function setUp() { parent::setUp(); diff --git a/core/modules/rdf/rdf.module b/core/modules/rdf/rdf.module index 8d4329884d60..dbffb20c017b 100644 --- a/core/modules/rdf/rdf.module +++ b/core/modules/rdf/rdf.module @@ -38,6 +38,7 @@ function rdf_help($route_name, RouteMatchInterface $route_match) { * into variables available to theme functions and templates. All Drupal core * themes are coded to be RDFa compatible. */ + /** * Returns the RDF mapping object associated with a bundle. * @@ -293,7 +294,6 @@ function _rdf_set_field_rel_attribute(&$variables) { } } - /** * Implements hook_preprocess_HOOK() for node templates. */ diff --git a/core/modules/rdf/tests/src/Kernel/RdfaAttributesTest.php b/core/modules/rdf/tests/src/Kernel/RdfaAttributesTest.php index 39f4831223a0..4023ef26b279 100644 --- a/core/modules/rdf/tests/src/Kernel/RdfaAttributesTest.php +++ b/core/modules/rdf/tests/src/Kernel/RdfaAttributesTest.php @@ -73,7 +73,6 @@ class RdfaAttributesTest extends KernelTestBase { $this->_testAttributes($expected_attributes, $mapping, $date); } - /** * Test attribute creation for mappings which use data converters. */ diff --git a/core/modules/responsive_image/src/Tests/ResponsiveImageFieldDisplayTest.php b/core/modules/responsive_image/src/Tests/ResponsiveImageFieldDisplayTest.php index 9196135afdc9..04efb7b80371 100644 --- a/core/modules/responsive_image/src/Tests/ResponsiveImageFieldDisplayTest.php +++ b/core/modules/responsive_image/src/Tests/ResponsiveImageFieldDisplayTest.php @@ -151,6 +151,7 @@ class ResponsiveImageFieldDisplayTest extends ImageFieldTestBase { ->save(); } } + /** * Test responsive image formatters on node display. * diff --git a/core/modules/rest/rest.api.php b/core/modules/rest/rest.api.php index 6c09bbff530b..0890e3bbf202 100644 --- a/core/modules/rest/rest.api.php +++ b/core/modules/rest/rest.api.php @@ -56,7 +56,6 @@ function hook_rest_type_uri_alter(&$uri, $context = []) { } } - /** * Alter the REST relation URI. * diff --git a/core/modules/rest/src/Plugin/views/display/RestExport.php b/core/modules/rest/src/Plugin/views/display/RestExport.php index 52daee05f040..93605b8e9501 100644 --- a/core/modules/rest/src/Plugin/views/display/RestExport.php +++ b/core/modules/rest/src/Plugin/views/display/RestExport.php @@ -166,6 +166,7 @@ class RestExport extends PathPluginBase implements ResponseDisplayPluginInterfac $container->getParameter('serializer.format_providers') ); } + /** * {@inheritdoc} */ diff --git a/core/modules/rest/tests/src/Functional/EntityResource/BlockContent/BlockContentResourceTestBase.php b/core/modules/rest/tests/src/Functional/EntityResource/BlockContent/BlockContentResourceTestBase.php index 260511a34651..cdc89d396147 100644 --- a/core/modules/rest/tests/src/Functional/EntityResource/BlockContent/BlockContentResourceTestBase.php +++ b/core/modules/rest/tests/src/Functional/EntityResource/BlockContent/BlockContentResourceTestBase.php @@ -161,7 +161,6 @@ abstract class BlockContentResourceTestBase extends EntityResourceTestBase { ]; } - /** * {@inheritdoc} */ diff --git a/core/modules/rest/tests/src/Functional/EntityResource/XmlEntityNormalizationQuirksTrait.php b/core/modules/rest/tests/src/Functional/EntityResource/XmlEntityNormalizationQuirksTrait.php index b69394ba1420..3064ea3a50f0 100644 --- a/core/modules/rest/tests/src/Functional/EntityResource/XmlEntityNormalizationQuirksTrait.php +++ b/core/modules/rest/tests/src/Functional/EntityResource/XmlEntityNormalizationQuirksTrait.php @@ -107,7 +107,6 @@ trait XmlEntityNormalizationQuirksTrait { return $normalization; } - /** * Applies the XML config entity encoding quirks that remain after decoding. * diff --git a/core/modules/rest/tests/src/Kernel/RequestHandlerTest.php b/core/modules/rest/tests/src/Kernel/RequestHandlerTest.php index de75f3ea2cec..a791838f9fdd 100644 --- a/core/modules/rest/tests/src/Kernel/RequestHandlerTest.php +++ b/core/modules/rest/tests/src/Kernel/RequestHandlerTest.php @@ -97,8 +97,11 @@ class RequestHandlerTest extends KernelTestBase { class StubRequestHandlerResourcePlugin extends ResourceBase { public function get($example, Request $request) {} + public function post() {} + public function patch($example_original, Request $request) {} + public function delete() {} } diff --git a/core/modules/search/tests/src/Functional/SearchExactTest.php b/core/modules/search/tests/src/Functional/SearchExactTest.php index 04163918e540..fc9cec2d562d 100644 --- a/core/modules/search/tests/src/Functional/SearchExactTest.php +++ b/core/modules/search/tests/src/Functional/SearchExactTest.php @@ -8,6 +8,7 @@ namespace Drupal\Tests\search\Functional; * @group search */ class SearchExactTest extends SearchTestBase { + /** * Tests that the correct number of pager links are found for both keywords and phrases. */ diff --git a/core/modules/search/tests/src/Functional/SearchSimplifyTest.php b/core/modules/search/tests/src/Functional/SearchSimplifyTest.php index 1c30c2e3639e..399f4ba0d851 100644 --- a/core/modules/search/tests/src/Functional/SearchSimplifyTest.php +++ b/core/modules/search/tests/src/Functional/SearchSimplifyTest.php @@ -8,6 +8,7 @@ namespace Drupal\Tests\search\Functional; * @group search */ class SearchSimplifyTest extends SearchTestBase { + /** * Tests that all Unicode characters simplify correctly. */ diff --git a/core/modules/search/tests/src/Unit/SearchPageRepositoryTest.php b/core/modules/search/tests/src/Unit/SearchPageRepositoryTest.php index 69dffd87cc84..5321a51d819d 100644 --- a/core/modules/search/tests/src/Unit/SearchPageRepositoryTest.php +++ b/core/modules/search/tests/src/Unit/SearchPageRepositoryTest.php @@ -273,11 +273,13 @@ class SearchPageRepositoryTest extends UnitTestCase { } class TestSearchPage extends SearchPage { + public function __construct(array $values) { foreach ($values as $key => $value) { $this->$key = $value; } } + public function label($langcode = NULL) { return $this->label; } diff --git a/core/modules/serialization/tests/src/Unit/Encoder/XmlEncoderTest.php b/core/modules/serialization/tests/src/Unit/Encoder/XmlEncoderTest.php index a347982b372f..cbff14a9b45b 100644 --- a/core/modules/serialization/tests/src/Unit/Encoder/XmlEncoderTest.php +++ b/core/modules/serialization/tests/src/Unit/Encoder/XmlEncoderTest.php @@ -97,6 +97,7 @@ class XmlEncoderTest extends UnitTestCase { } class TestObject { + public function getA() { return 'A'; } diff --git a/core/modules/simpletest/simpletest.api.php b/core/modules/simpletest/simpletest.api.php index 62563a0893dd..d8f820946015 100644 --- a/core/modules/simpletest/simpletest.api.php +++ b/core/modules/simpletest/simpletest.api.php @@ -62,7 +62,6 @@ function hook_test_group_finished() { function hook_test_finished($results) { } - /** * @} End of "addtogroup hooks". */ diff --git a/core/modules/simpletest/src/KernelTestBase.php b/core/modules/simpletest/src/KernelTestBase.php index 12113bce9b6a..c8b8669d542b 100644 --- a/core/modules/simpletest/src/KernelTestBase.php +++ b/core/modules/simpletest/src/KernelTestBase.php @@ -478,7 +478,6 @@ EOD; ])); } - /** * Installs the storage schema for a specific entity type. * diff --git a/core/modules/simpletest/tests/src/Functional/MailCaptureTest.php b/core/modules/simpletest/tests/src/Functional/MailCaptureTest.php index f610eaf429de..8cb0904bf996 100644 --- a/core/modules/simpletest/tests/src/Functional/MailCaptureTest.php +++ b/core/modules/simpletest/tests/src/Functional/MailCaptureTest.php @@ -15,6 +15,7 @@ class MailCaptureTest extends BrowserTestBase { use AssertMailTrait { getMails as drupalGetMails; } + /** * Test to see if the wrapper function is executed correctly. */ diff --git a/core/modules/statistics/src/StatisticsViewsResult.php b/core/modules/statistics/src/StatisticsViewsResult.php index ef0db9775e79..f3253e5b84ca 100644 --- a/core/modules/statistics/src/StatisticsViewsResult.php +++ b/core/modules/statistics/src/StatisticsViewsResult.php @@ -37,7 +37,6 @@ class StatisticsViewsResult { return $this->totalCount; } - /** * Total number of times the entity has been viewed "today". * @@ -47,7 +46,6 @@ class StatisticsViewsResult { return $this->dayCount; } - /** * Timestamp of when the entity was last viewed. * diff --git a/core/modules/statistics/tests/src/Functional/StatisticsTokenReplaceTest.php b/core/modules/statistics/tests/src/Functional/StatisticsTokenReplaceTest.php index 8402d221af0e..50616af23aad 100644 --- a/core/modules/statistics/tests/src/Functional/StatisticsTokenReplaceTest.php +++ b/core/modules/statistics/tests/src/Functional/StatisticsTokenReplaceTest.php @@ -9,6 +9,7 @@ namespace Drupal\Tests\statistics\Functional; * @group statistics */ class StatisticsTokenReplaceTest extends StatisticsTestBase { + /** * Creates a node, then tests the statistics tokens generated from it. */ diff --git a/core/modules/system/src/CronController.php b/core/modules/system/src/CronController.php index 915a0c887eb1..2626bf3a0591 100644 --- a/core/modules/system/src/CronController.php +++ b/core/modules/system/src/CronController.php @@ -36,7 +36,6 @@ class CronController extends ControllerBase { return new static($container->get('cron')); } - /** * Run Cron once. * diff --git a/core/modules/system/src/Form/SiteMaintenanceModeForm.php b/core/modules/system/src/Form/SiteMaintenanceModeForm.php index 58b1f1a9d62c..dae500df4840 100644 --- a/core/modules/system/src/Form/SiteMaintenanceModeForm.php +++ b/core/modules/system/src/Form/SiteMaintenanceModeForm.php @@ -56,6 +56,7 @@ class SiteMaintenanceModeForm extends ConfigFormBase { $container->get('user.permissions') ); } + /** * {@inheritdoc} */ diff --git a/core/modules/system/src/Plugin/migrate/process/d6/TimeZone.php b/core/modules/system/src/Plugin/migrate/process/d6/TimeZone.php index 3dc88810d575..7d11beea1941 100644 --- a/core/modules/system/src/Plugin/migrate/process/d6/TimeZone.php +++ b/core/modules/system/src/Plugin/migrate/process/d6/TimeZone.php @@ -14,6 +14,7 @@ use Drupal\migrate\Row; * ) */ class TimeZone extends ProcessPluginBase { + /** * {@inheritdoc} */ diff --git a/core/modules/system/src/Tests/Ajax/AjaxInGroupTest.php b/core/modules/system/src/Tests/Ajax/AjaxInGroupTest.php index 4116f5f892b3..39af58be5a51 100644 --- a/core/modules/system/src/Tests/Ajax/AjaxInGroupTest.php +++ b/core/modules/system/src/Tests/Ajax/AjaxInGroupTest.php @@ -8,6 +8,7 @@ namespace Drupal\system\Tests\Ajax; * @group Ajax */ class AjaxInGroupTest extends AjaxTestBase { + protected function setUp() { parent::setUp(); diff --git a/core/modules/system/src/Tests/Ajax/CommandsTest.php b/core/modules/system/src/Tests/Ajax/CommandsTest.php index 6ae97ae1341a..b91ac98271ca 100644 --- a/core/modules/system/src/Tests/Ajax/CommandsTest.php +++ b/core/modules/system/src/Tests/Ajax/CommandsTest.php @@ -29,6 +29,7 @@ use Symfony\Component\HttpKernel\HttpKernelInterface; * @group Ajax */ class CommandsTest extends AjaxTestBase { + /** * Tests the various Ajax Commands. */ diff --git a/core/modules/system/src/Tests/Ajax/ElementValidationTest.php b/core/modules/system/src/Tests/Ajax/ElementValidationTest.php index d20f453be48f..169a38900be3 100644 --- a/core/modules/system/src/Tests/Ajax/ElementValidationTest.php +++ b/core/modules/system/src/Tests/Ajax/ElementValidationTest.php @@ -8,6 +8,7 @@ namespace Drupal\system\Tests\Ajax; * @group Ajax */ class ElementValidationTest extends AjaxTestBase { + /** * Tries to post an Ajax change to a form that has a validated element. * diff --git a/core/modules/system/src/Tests/Ajax/FormValuesTest.php b/core/modules/system/src/Tests/Ajax/FormValuesTest.php index 9f4b52175253..358db893388b 100644 --- a/core/modules/system/src/Tests/Ajax/FormValuesTest.php +++ b/core/modules/system/src/Tests/Ajax/FormValuesTest.php @@ -10,6 +10,7 @@ use Drupal\Core\Ajax\DataCommand; * @group Ajax */ class FormValuesTest extends AjaxTestBase { + protected function setUp() { parent::setUp(); diff --git a/core/modules/system/src/Tests/Ajax/FrameworkTest.php b/core/modules/system/src/Tests/Ajax/FrameworkTest.php index 14bae0644293..507493b7d09b 100644 --- a/core/modules/system/src/Tests/Ajax/FrameworkTest.php +++ b/core/modules/system/src/Tests/Ajax/FrameworkTest.php @@ -16,6 +16,7 @@ use Drupal\Core\Asset\AttachedAssets; * @group Ajax */ class FrameworkTest extends AjaxTestBase { + /** * Verifies the Ajax rendering of a command in the settings. */ diff --git a/core/modules/system/src/Tests/Form/ElementsTableSelectTest.php b/core/modules/system/src/Tests/Form/ElementsTableSelectTest.php index 9cbda92cf6da..030526eef9fa 100644 --- a/core/modules/system/src/Tests/Form/ElementsTableSelectTest.php +++ b/core/modules/system/src/Tests/Form/ElementsTableSelectTest.php @@ -163,7 +163,6 @@ class ElementsTableSelectTest extends WebTestBase { $this->assertNoFieldByXPath('//th[@class="select-all"]', NULL, 'Do not display a "Select all" checkbox when #multiple is FALSE, even when #js_select is TRUE.'); } - /** * Test the whether the option checker gives an error on invalid tableselect values for checkboxes. */ @@ -187,7 +186,6 @@ class ElementsTableSelectTest extends WebTestBase { } - /** * Test the whether the option checker gives an error on invalid tableselect values for radios. */ @@ -211,7 +209,6 @@ class ElementsTableSelectTest extends WebTestBase { $this->assertTrue(isset($errors['tableselect']), 'Option checker disallows invalid values for radio buttons.'); } - /** * Helper function for the option check test to submit a form while collecting errors. * diff --git a/core/modules/system/src/Tests/Path/UrlAliasFixtures.php b/core/modules/system/src/Tests/Path/UrlAliasFixtures.php index 71cd736380e0..bb57c31ba128 100644 --- a/core/modules/system/src/Tests/Path/UrlAliasFixtures.php +++ b/core/modules/system/src/Tests/Path/UrlAliasFixtures.php @@ -71,7 +71,6 @@ class UrlAliasFixtures { ]; } - /** * Returns the table definition for the URL alias fixtures. * diff --git a/core/modules/system/system.post_update.php b/core/modules/system/system.post_update.php index 038d4b82909b..0662853339d4 100644 --- a/core/modules/system/system.post_update.php +++ b/core/modules/system/system.post_update.php @@ -51,7 +51,6 @@ function system_post_update_add_region_to_entity_displays() { array_map($entity_save, EntityFormDisplay::loadMultiple()); } - /** * Force caches using hashes to be cleared (Twig, render cache, etc.). */ diff --git a/core/modules/system/tests/modules/ajax_test/src/Form/AjaxTestDialogForm.php b/core/modules/system/tests/modules/ajax_test/src/Form/AjaxTestDialogForm.php index daedcf17069f..eafe8a98a7a1 100644 --- a/core/modules/system/tests/modules/ajax_test/src/Form/AjaxTestDialogForm.php +++ b/core/modules/system/tests/modules/ajax_test/src/Form/AjaxTestDialogForm.php @@ -67,7 +67,6 @@ class AjaxTestDialogForm extends FormBase { $form_state->setRedirect('ajax_test.dialog_contents'); } - /** * AJAX callback handler for AjaxTestDialogForm. */ @@ -82,7 +81,6 @@ class AjaxTestDialogForm extends FormBase { return $this->dialog(FALSE); } - /** * Util to render dialog in ajax callback. * diff --git a/core/modules/system/tests/modules/database_test/database_test.module b/core/modules/system/tests/modules/database_test/database_test.module index 1eaa103fa598..210998444a25 100644 --- a/core/modules/system/tests/modules/database_test/database_test.module +++ b/core/modules/system/tests/modules/database_test/database_test.module @@ -38,7 +38,6 @@ function database_test_query_alter(AlterableInterface $query) { } } - /** * Implements hook_query_TAG_alter(). * diff --git a/core/modules/system/tests/modules/dialog_renderer_test/src/Render/MainContent/WideModalRenderer.php b/core/modules/system/tests/modules/dialog_renderer_test/src/Render/MainContent/WideModalRenderer.php index 2fd982818031..4b3aee92d8a3 100644 --- a/core/modules/system/tests/modules/dialog_renderer_test/src/Render/MainContent/WideModalRenderer.php +++ b/core/modules/system/tests/modules/dialog_renderer_test/src/Render/MainContent/WideModalRenderer.php @@ -38,7 +38,6 @@ class WideModalRenderer extends ModalRenderer { $this->mode = $mode; } - /** * {@inheritdoc} */ diff --git a/core/modules/system/tests/modules/entity_test/src/Controller/EntityTestController.php b/core/modules/system/tests/modules/entity_test/src/Controller/EntityTestController.php index a69348849f44..fa69e783d08a 100644 --- a/core/modules/system/tests/modules/entity_test/src/Controller/EntityTestController.php +++ b/core/modules/system/tests/modules/entity_test/src/Controller/EntityTestController.php @@ -95,7 +95,6 @@ class EntityTestController extends ControllerBase { ]; } - /** * Empty list of entities of the given entity type. * diff --git a/core/modules/system/tests/modules/js_webassert_test/src/Form/JsWebAssertTestForm.php b/core/modules/system/tests/modules/js_webassert_test/src/Form/JsWebAssertTestForm.php index d46db774d68f..1bc25bb557ab 100644 --- a/core/modules/system/tests/modules/js_webassert_test/src/Form/JsWebAssertTestForm.php +++ b/core/modules/system/tests/modules/js_webassert_test/src/Form/JsWebAssertTestForm.php @@ -139,6 +139,7 @@ class JsWebAssertTestForm extends FormBase { ]; return $form; } + /** * Ajax callback for the "Add field" button. */ @@ -179,7 +180,6 @@ class JsWebAssertTestForm extends FormBase { return $form; } - /** * Ajax callback for the "Test waitForElementVisible" button. */ diff --git a/core/modules/system/tests/modules/menu_test/src/Controller/MenuTestController.php b/core/modules/system/tests/modules/menu_test/src/Controller/MenuTestController.php index fe6566cddb32..8472b99a183c 100644 --- a/core/modules/system/tests/modules/menu_test/src/Controller/MenuTestController.php +++ b/core/modules/system/tests/modules/menu_test/src/Controller/MenuTestController.php @@ -71,7 +71,6 @@ class MenuTestController extends ControllerBase { return ['#markup' => 'This is the menuTestCallback content.']; } - /** * A title callback method for test routes. * diff --git a/core/modules/system/tests/modules/module_autoload_test/src/SomeClass.php b/core/modules/system/tests/modules/module_autoload_test/src/SomeClass.php index 6cd97a0cb65a..11993b71f48e 100644 --- a/core/modules/system/tests/modules/module_autoload_test/src/SomeClass.php +++ b/core/modules/system/tests/modules/module_autoload_test/src/SomeClass.php @@ -3,6 +3,7 @@ namespace Drupal\module_autoload_test; class SomeClass { + public function testMethod() { return 'Drupal\\module_autoload_test\\SomeClass::testMethod() was invoked.'; } diff --git a/core/modules/system/tests/modules/plugin_test/src/Plugin/MockBlockManager.php b/core/modules/system/tests/modules/plugin_test/src/Plugin/MockBlockManager.php index 443ab5d04090..78a02a95dd16 100644 --- a/core/modules/system/tests/modules/plugin_test/src/Plugin/MockBlockManager.php +++ b/core/modules/system/tests/modules/plugin_test/src/Plugin/MockBlockManager.php @@ -12,6 +12,7 @@ use Drupal\Core\Plugin\Context\ContextDefinition; * Defines a plugin manager used by Plugin API derivative unit tests. */ class MockBlockManager extends PluginManagerBase { + public function __construct() { // Create the object that can be used to return definitions for all the diff --git a/core/modules/system/tests/modules/plugin_test/src/Plugin/TestPluginManager.php b/core/modules/system/tests/modules/plugin_test/src/Plugin/TestPluginManager.php index ec464a5b7ae8..d044bafcc135 100644 --- a/core/modules/system/tests/modules/plugin_test/src/Plugin/TestPluginManager.php +++ b/core/modules/system/tests/modules/plugin_test/src/Plugin/TestPluginManager.php @@ -10,6 +10,7 @@ use Drupal\Component\Plugin\Factory\DefaultFactory; * Defines a plugin manager used by Plugin API unit tests. */ class TestPluginManager extends PluginManagerBase { + public function __construct() { // Create the object that can be used to return definitions for all the diff --git a/core/modules/system/tests/modules/theme_test/src/ThemeTestController.php b/core/modules/system/tests/modules/theme_test/src/ThemeTestController.php index f1ed8f3a991b..c9a939000a71 100644 --- a/core/modules/system/tests/modules/theme_test/src/ThemeTestController.php +++ b/core/modules/system/tests/modules/theme_test/src/ThemeTestController.php @@ -119,7 +119,6 @@ class ThemeTestController extends ControllerBase { return ['#theme' => 'theme_test_function_suggestions']; } - /** * Menu callback for testing includes with suggestion alter hooks. */ diff --git a/core/modules/system/tests/modules/update_test_schema/update_test_schema.install b/core/modules/system/tests/modules/update_test_schema/update_test_schema.install index 972f9017616f..eef6b255aa09 100644 --- a/core/modules/system/tests/modules/update_test_schema/update_test_schema.install +++ b/core/modules/system/tests/modules/update_test_schema/update_test_schema.install @@ -31,6 +31,7 @@ function update_test_schema_schema() { $schema_version = \Drupal::state()->get('update_test_schema_version', 8000); if ($schema_version >= 8001) { + /** * Schema version 8001. */ @@ -45,4 +46,5 @@ if ($schema_version >= 8001) { // Add a column. db_add_index('update_test_schema_table', 'test', ['a'], $table); } + } diff --git a/core/modules/system/tests/src/Functional/Batch/ProcessingTest.php b/core/modules/system/tests/src/Functional/Batch/ProcessingTest.php index 2d7c7ae01c52..97dc6e7a5f69 100644 --- a/core/modules/system/tests/src/Functional/Batch/ProcessingTest.php +++ b/core/modules/system/tests/src/Functional/Batch/ProcessingTest.php @@ -175,7 +175,6 @@ class ProcessingTest extends BrowserTestBase { $this->assertText('Redirection successful.', 'Redirection after batch execution is correct.'); } - /** * Triggers a pass if the texts were found in order in the raw content. * diff --git a/core/modules/system/tests/src/Functional/Entity/ConfigEntityImportTest.php b/core/modules/system/tests/src/Functional/Entity/ConfigEntityImportTest.php index 6c5a9d64c59e..b69960375501 100644 --- a/core/modules/system/tests/src/Functional/Entity/ConfigEntityImportTest.php +++ b/core/modules/system/tests/src/Functional/Entity/ConfigEntityImportTest.php @@ -42,6 +42,7 @@ class ConfigEntityImportTest extends BrowserTestBase { $this->doImageStyleUpdate(); $this->doSearchPageUpdate(); } + /** * Tests updating a action during import. */ diff --git a/core/modules/system/tests/src/Functional/Entity/Update/UpdateApiEntityDefinitionUpdateTest.php b/core/modules/system/tests/src/Functional/Entity/Update/UpdateApiEntityDefinitionUpdateTest.php index f3285ae5d5ec..1c3c6e3e0aee 100644 --- a/core/modules/system/tests/src/Functional/Entity/Update/UpdateApiEntityDefinitionUpdateTest.php +++ b/core/modules/system/tests/src/Functional/Entity/Update/UpdateApiEntityDefinitionUpdateTest.php @@ -34,7 +34,6 @@ class UpdateApiEntityDefinitionUpdateTest extends BrowserTestBase { */ protected $updatesManager; - /** * {@inheritdoc} */ diff --git a/core/modules/system/tests/src/Functional/Mail/HtmlToTextTest.php b/core/modules/system/tests/src/Functional/Mail/HtmlToTextTest.php index 82410fd52a7c..02bfca844126 100644 --- a/core/modules/system/tests/src/Functional/Mail/HtmlToTextTest.php +++ b/core/modules/system/tests/src/Functional/Mail/HtmlToTextTest.php @@ -13,6 +13,7 @@ use Drupal\Tests\BrowserTestBase; * @group Mail */ class HtmlToTextTest extends BrowserTestBase { + /** * Converts a string to its PHP source equivalent for display in test messages. * diff --git a/core/modules/system/tests/src/Functional/Module/DependencyTest.php b/core/modules/system/tests/src/Functional/Module/DependencyTest.php index 0fc78caa953a..d2938721a861 100644 --- a/core/modules/system/tests/src/Functional/Module/DependencyTest.php +++ b/core/modules/system/tests/src/Functional/Module/DependencyTest.php @@ -10,6 +10,7 @@ use Drupal\Component\Utility\Unicode; * @group Module */ class DependencyTest extends ModuleTestBase { + /** * Checks functionality of project namespaces for dependencies. */ diff --git a/core/modules/system/tests/src/Functional/Module/HookRequirementsTest.php b/core/modules/system/tests/src/Functional/Module/HookRequirementsTest.php index a873d9652a27..2ada3a2854d9 100644 --- a/core/modules/system/tests/src/Functional/Module/HookRequirementsTest.php +++ b/core/modules/system/tests/src/Functional/Module/HookRequirementsTest.php @@ -8,6 +8,7 @@ namespace Drupal\Tests\system\Functional\Module; * @group Module */ class HookRequirementsTest extends ModuleTestBase { + /** * Assert that a module cannot be installed if it fails hook_requirements(). */ diff --git a/core/modules/system/tests/src/Functional/System/AdminMetaTagTest.php b/core/modules/system/tests/src/Functional/System/AdminMetaTagTest.php index dbd82cda02f0..6fcc4b848d18 100644 --- a/core/modules/system/tests/src/Functional/System/AdminMetaTagTest.php +++ b/core/modules/system/tests/src/Functional/System/AdminMetaTagTest.php @@ -10,6 +10,7 @@ use Drupal\Tests\BrowserTestBase; * @group system */ class AdminMetaTagTest extends BrowserTestBase { + /** * Verify that the meta tag HTML is generated correctly. */ diff --git a/core/modules/system/tests/src/Functional/System/IndexPhpTest.php b/core/modules/system/tests/src/Functional/System/IndexPhpTest.php index 2bf098931d61..4675fed0faa0 100644 --- a/core/modules/system/tests/src/Functional/System/IndexPhpTest.php +++ b/core/modules/system/tests/src/Functional/System/IndexPhpTest.php @@ -10,6 +10,7 @@ use Drupal\Tests\BrowserTestBase; * @group system */ class IndexPhpTest extends BrowserTestBase { + protected function setUp() { parent::setUp(); } diff --git a/core/modules/system/tests/src/Functional/System/RetrieveFileTest.php b/core/modules/system/tests/src/Functional/System/RetrieveFileTest.php index ef12a51adeba..cf64f079adc3 100644 --- a/core/modules/system/tests/src/Functional/System/RetrieveFileTest.php +++ b/core/modules/system/tests/src/Functional/System/RetrieveFileTest.php @@ -10,6 +10,7 @@ use Drupal\Tests\BrowserTestBase; * @group system */ class RetrieveFileTest extends BrowserTestBase { + /** * Invokes system_retrieve_file() in several scenarios. */ diff --git a/core/modules/system/tests/src/Functional/Update/RouterIndexOptimizationTest.php b/core/modules/system/tests/src/Functional/Update/RouterIndexOptimizationTest.php index a3bfad256292..0a952c576b3e 100644 --- a/core/modules/system/tests/src/Functional/Update/RouterIndexOptimizationTest.php +++ b/core/modules/system/tests/src/Functional/Update/RouterIndexOptimizationTest.php @@ -10,6 +10,7 @@ use Drupal\FunctionalTests\Update\UpdatePathTestBase; * @group Update */ class RouterIndexOptimizationTest extends UpdatePathTestBase { + /** * {@inheritdoc} */ diff --git a/core/modules/system/tests/src/Kernel/Common/SystemListingTest.php b/core/modules/system/tests/src/Kernel/Common/SystemListingTest.php index b0412b4336fd..221daeeaeb87 100644 --- a/core/modules/system/tests/src/Kernel/Common/SystemListingTest.php +++ b/core/modules/system/tests/src/Kernel/Common/SystemListingTest.php @@ -11,6 +11,7 @@ use Drupal\KernelTests\KernelTestBase; * @group Common */ class SystemListingTest extends KernelTestBase { + /** * Tests that files in different directories take precedence as expected. */ diff --git a/core/modules/taxonomy/src/Plugin/views/field/TermName.php b/core/modules/taxonomy/src/Plugin/views/field/TermName.php index 0a777e9c1cfe..eefc3e3d3fe1 100644 --- a/core/modules/taxonomy/src/Plugin/views/field/TermName.php +++ b/core/modules/taxonomy/src/Plugin/views/field/TermName.php @@ -31,7 +31,6 @@ class TermName extends EntityField { return $items; } - /** * {@inheritdoc} */ diff --git a/core/modules/text/tests/src/Kernel/TextWithSummaryItemTest.php b/core/modules/text/tests/src/Kernel/TextWithSummaryItemTest.php index a48c191b5ee7..1f4a5bfdfed9 100644 --- a/core/modules/text/tests/src/Kernel/TextWithSummaryItemTest.php +++ b/core/modules/text/tests/src/Kernel/TextWithSummaryItemTest.php @@ -37,7 +37,6 @@ class TextWithSummaryItemTest extends FieldKernelTestBase { */ protected $field; - protected function setUp() { parent::setUp(); diff --git a/core/modules/tracker/src/Plugin/Menu/UserTrackerTab.php b/core/modules/tracker/src/Plugin/Menu/UserTrackerTab.php index 95264803f742..db0f2b463250 100644 --- a/core/modules/tracker/src/Plugin/Menu/UserTrackerTab.php +++ b/core/modules/tracker/src/Plugin/Menu/UserTrackerTab.php @@ -32,7 +32,6 @@ class UserTrackerTab extends LocalTaskDefault { return $this->currentUser; } - /** * {@inheritdoc} */ diff --git a/core/modules/user/src/Entity/User.php b/core/modules/user/src/Entity/User.php index 979819fc8cdf..7f4f4cdc78eb 100644 --- a/core/modules/user/src/Entity/User.php +++ b/core/modules/user/src/Entity/User.php @@ -355,6 +355,7 @@ class User extends ContentEntityBase implements UserInterface { public function isAuthenticated() { return $this->id() > 0; } + /** * {@inheritdoc} */ diff --git a/core/modules/user/src/Plugin/Block/UserLoginBlock.php b/core/modules/user/src/Plugin/Block/UserLoginBlock.php index a9341f9b7f79..8649494b929c 100644 --- a/core/modules/user/src/Plugin/Block/UserLoginBlock.php +++ b/core/modules/user/src/Plugin/Block/UserLoginBlock.php @@ -64,7 +64,6 @@ class UserLoginBlock extends BlockBase implements ContainerFactoryPluginInterfac ); } - /** * {@inheritdoc} */ diff --git a/core/modules/user/src/Plugin/migrate/process/d6/UserUpdate7002.php b/core/modules/user/src/Plugin/migrate/process/d6/UserUpdate7002.php index 0efae5d6b3d5..858fcfc9906c 100644 --- a/core/modules/user/src/Plugin/migrate/process/d6/UserUpdate7002.php +++ b/core/modules/user/src/Plugin/migrate/process/d6/UserUpdate7002.php @@ -54,6 +54,7 @@ class UserUpdate7002 extends ProcessPluginBase implements ContainerFactoryPlugin $container->get('config.factory')->get('system.date') ); } + /** * {@inheritdoc} */ diff --git a/core/modules/user/src/Plugin/migrate/source/d6/UserPicture.php b/core/modules/user/src/Plugin/migrate/source/d6/UserPicture.php index 0d5bd6bcb310..288cf6cac4c3 100644 --- a/core/modules/user/src/Plugin/migrate/source/d6/UserPicture.php +++ b/core/modules/user/src/Plugin/migrate/source/d6/UserPicture.php @@ -37,6 +37,7 @@ class UserPicture extends DrupalSqlBase { 'picture' => "Path to the user's uploaded picture.", ]; } + /** * {@inheritdoc} */ diff --git a/core/modules/user/src/Plugin/migrate/source/d6/UserPictureFile.php b/core/modules/user/src/Plugin/migrate/source/d6/UserPictureFile.php index 72d18fd08f36..4414198e3ae1 100644 --- a/core/modules/user/src/Plugin/migrate/source/d6/UserPictureFile.php +++ b/core/modules/user/src/Plugin/migrate/source/d6/UserPictureFile.php @@ -68,6 +68,7 @@ class UserPictureFile extends DrupalSqlBase { 'filename' => 'The picture filename.', ]; } + /** * {@inheritdoc} */ diff --git a/core/modules/user/src/Plugin/views/access/Permission.php b/core/modules/user/src/Plugin/views/access/Permission.php index 8d0c43dfcef7..291eb1ca8edf 100644 --- a/core/modules/user/src/Plugin/views/access/Permission.php +++ b/core/modules/user/src/Plugin/views/access/Permission.php @@ -100,7 +100,6 @@ class Permission extends AccessPluginBase implements CacheableDependencyInterfac return $this->t($this->options['perm']); } - protected function defineOptions() { $options = parent::defineOptions(); $options['perm'] = ['default' => 'access content']; diff --git a/core/modules/user/src/Plugin/views/access/Role.php b/core/modules/user/src/Plugin/views/access/Role.php index e35daa1eea93..d1fe8abcc240 100644 --- a/core/modules/user/src/Plugin/views/access/Role.php +++ b/core/modules/user/src/Plugin/views/access/Role.php @@ -96,7 +96,6 @@ class Role extends AccessPluginBase implements CacheableDependencyInterface { } } - protected function defineOptions() { $options = parent::defineOptions(); $options['role'] = ['default' => []]; diff --git a/core/modules/user/src/Plugin/views/field/UserData.php b/core/modules/user/src/Plugin/views/field/UserData.php index 365ab3b746a6..1a4bb82acdd5 100644 --- a/core/modules/user/src/Plugin/views/field/UserData.php +++ b/core/modules/user/src/Plugin/views/field/UserData.php @@ -34,7 +34,6 @@ class UserData extends FieldPluginBase { */ protected $moduleHandler; - /** * {@inheritdoc} */ diff --git a/core/modules/user/tests/src/Functional/Views/HandlerFilterUserNameTest.php b/core/modules/user/tests/src/Functional/Views/HandlerFilterUserNameTest.php index c2b15ff58cbf..fd6fdd2af183 100644 --- a/core/modules/user/tests/src/Functional/Views/HandlerFilterUserNameTest.php +++ b/core/modules/user/tests/src/Functional/Views/HandlerFilterUserNameTest.php @@ -66,7 +66,6 @@ class HandlerFilterUserNameTest extends ViewTestBase { } } - /** * Tests just using the filter. */ diff --git a/core/modules/user/tests/src/Kernel/UserInstallTest.php b/core/modules/user/tests/src/Kernel/UserInstallTest.php index f61811be79ff..be3336f78027 100644 --- a/core/modules/user/tests/src/Kernel/UserInstallTest.php +++ b/core/modules/user/tests/src/Kernel/UserInstallTest.php @@ -28,7 +28,6 @@ class UserInstallTest extends KernelTestBase { user_install(); } - /** * Test that the initial users have correct values. */ diff --git a/core/modules/user/tests/src/Unit/PrivateTempStoreTest.php b/core/modules/user/tests/src/Unit/PrivateTempStoreTest.php index be9376ad1dc1..2d8e092c1217 100644 --- a/core/modules/user/tests/src/Unit/PrivateTempStoreTest.php +++ b/core/modules/user/tests/src/Unit/PrivateTempStoreTest.php @@ -96,7 +96,6 @@ class PrivateTempStoreTest extends UnitTestCase { $this->otherObject->owner = 2; } - /** * Tests the get() method. * diff --git a/core/modules/views/src/Analyzer.php b/core/modules/views/src/Analyzer.php index 6e176293072e..eb09269e059e 100644 --- a/core/modules/views/src/Analyzer.php +++ b/core/modules/views/src/Analyzer.php @@ -31,7 +31,6 @@ class Analyzer { $this->moduleHandler = $module_handler; } - /** * Analyzes a review and return the results. * diff --git a/core/modules/views/src/Plugin/Derivative/DefaultWizardDeriver.php b/core/modules/views/src/Plugin/Derivative/DefaultWizardDeriver.php index f6a045cf32df..de9bb26c8dc2 100644 --- a/core/modules/views/src/Plugin/Derivative/DefaultWizardDeriver.php +++ b/core/modules/views/src/Plugin/Derivative/DefaultWizardDeriver.php @@ -11,6 +11,7 @@ use Drupal\views\Views; * The derivatives store all base table plugin information. */ class DefaultWizardDeriver extends DeriverBase { + /** * {@inheritdoc} */ diff --git a/core/modules/views/src/Plugin/Menu/ViewsMenuLink.php b/core/modules/views/src/Plugin/Menu/ViewsMenuLink.php index b4f078fc684e..3cbfc84e0a18 100644 --- a/core/modules/views/src/Plugin/Menu/ViewsMenuLink.php +++ b/core/modules/views/src/Plugin/Menu/ViewsMenuLink.php @@ -126,7 +126,6 @@ class ViewsMenuLink extends MenuLinkBase implements ContainerFactoryPluginInterf return (bool) $this->loadView()->display_handler->getOption('menu')['expanded']; } - /** * {@inheritdoc} */ diff --git a/core/modules/views/src/Plugin/views/HandlerBase.php b/core/modules/views/src/Plugin/views/HandlerBase.php index 35689ee077dc..28dfbfcab7d4 100644 --- a/core/modules/views/src/Plugin/views/HandlerBase.php +++ b/core/modules/views/src/Plugin/views/HandlerBase.php @@ -313,6 +313,7 @@ abstract class HandlerBase extends PluginBase implements ViewsHandlerInterface { public function usesGroupBy() { return TRUE; } + /** * Provide a form for aggregation settings. */ diff --git a/core/modules/views/src/Plugin/views/argument/ArgumentPluginBase.php b/core/modules/views/src/Plugin/views/argument/ArgumentPluginBase.php index 9bcd5ba699c9..96a41f3544c0 100644 --- a/core/modules/views/src/Plugin/views/argument/ArgumentPluginBase.php +++ b/core/modules/views/src/Plugin/views/argument/ArgumentPluginBase.php @@ -398,7 +398,6 @@ abstract class ArgumentPluginBase extends HandlerBase implements CacheableDepend return $output; } - public function validateOptionsForm(&$form, FormStateInterface $form_state) { $option_values = &$form_state->getValue('options'); if (empty($option_values)) { @@ -721,6 +720,7 @@ abstract class ArgumentPluginBase extends HandlerBase implements CacheableDepend $info = $this->defaultActions($this->options['validate']['fail']); return $this->defaultAction($info); } + /** * Default action: ignore. * diff --git a/core/modules/views/src/Plugin/views/cache/None.php b/core/modules/views/src/Plugin/views/cache/None.php index d05abd2adb33..fc9b58b238bb 100644 --- a/core/modules/views/src/Plugin/views/cache/None.php +++ b/core/modules/views/src/Plugin/views/cache/None.php @@ -19,7 +19,6 @@ class None extends CachePluginBase { return $this->t('None'); } - /** * Overrides \Drupal\views\Plugin\views\cache\CachePluginBase::cacheGet(). * diff --git a/core/modules/views/src/Plugin/views/display/DisplayPluginBase.php b/core/modules/views/src/Plugin/views/display/DisplayPluginBase.php index 567bb4f95a69..29d188c187d2 100644 --- a/core/modules/views/src/Plugin/views/display/DisplayPluginBase.php +++ b/core/modules/views/src/Plugin/views/display/DisplayPluginBase.php @@ -962,7 +962,6 @@ abstract class DisplayPluginBase extends PluginBase implements DisplayPluginInte return $this->dependencies; } - /** * {@inheritdoc} */ diff --git a/core/modules/views/src/Plugin/views/display/DisplayPluginInterface.php b/core/modules/views/src/Plugin/views/display/DisplayPluginInterface.php index f8630af08fba..80e4783164fe 100644 --- a/core/modules/views/src/Plugin/views/display/DisplayPluginInterface.php +++ b/core/modules/views/src/Plugin/views/display/DisplayPluginInterface.php @@ -457,7 +457,6 @@ interface DisplayPluginInterface { */ public function execute(); - /** * Builds a basic render array which can be properly render cached. * diff --git a/core/modules/views/src/Plugin/views/display/Feed.php b/core/modules/views/src/Plugin/views/display/Feed.php index c664c053e256..db1bc1c73ddd 100644 --- a/core/modules/views/src/Plugin/views/display/Feed.php +++ b/core/modules/views/src/Plugin/views/display/Feed.php @@ -73,7 +73,6 @@ class Feed extends PathPluginBase implements ResponseDisplayPluginInterface { return $response; } - /** * {@inheritdoc} */ diff --git a/core/modules/views/src/Plugin/views/field/EntityOperations.php b/core/modules/views/src/Plugin/views/field/EntityOperations.php index 6f0882979a4d..d31cd1ea3068 100644 --- a/core/modules/views/src/Plugin/views/field/EntityOperations.php +++ b/core/modules/views/src/Plugin/views/field/EntityOperations.php @@ -158,6 +158,7 @@ class EntityOperations extends FieldPluginBase { protected function getLanguageManager() { return $this->languageManager; } + /** * {@inheritdoc} */ diff --git a/core/modules/views/src/Plugin/views/filter/BooleanOperator.php b/core/modules/views/src/Plugin/views/filter/BooleanOperator.php index fa0d730f8365..81a31880c320 100644 --- a/core/modules/views/src/Plugin/views/filter/BooleanOperator.php +++ b/core/modules/views/src/Plugin/views/filter/BooleanOperator.php @@ -48,7 +48,6 @@ class BooleanOperator extends FilterPluginBase { // Whether to accept NULL as a false value or not public $accept_null = FALSE; - /** * {@inheritdoc} */ diff --git a/core/modules/views/src/Plugin/views/filter/FilterPluginBase.php b/core/modules/views/src/Plugin/views/filter/FilterPluginBase.php index fd273ddfa838..b49190b05511 100644 --- a/core/modules/views/src/Plugin/views/filter/FilterPluginBase.php +++ b/core/modules/views/src/Plugin/views/filter/FilterPluginBase.php @@ -656,7 +656,6 @@ abstract class FilterPluginBase extends HandlerBase implements CacheableDependen return FALSE; } - /** * Validate the build group options form. */ @@ -835,7 +834,6 @@ abstract class FilterPluginBase extends HandlerBase implements CacheableDependen } } - /** * Render our chunk of the exposed filter form when selecting * @@ -1165,7 +1163,6 @@ abstract class FilterPluginBase extends HandlerBase implements CacheableDependen $form_state->get('force_build_group_options', TRUE); } - /** * Make some translations to a form item to make it more suitable to * exposing. @@ -1207,7 +1204,6 @@ abstract class FilterPluginBase extends HandlerBase implements CacheableDependen } } - /** * Sanitizes the HTML select element's options. * diff --git a/core/modules/views/src/Plugin/views/filter/GroupByNumeric.php b/core/modules/views/src/Plugin/views/filter/GroupByNumeric.php index 577b2dfde4f1..d68970bd5a29 100644 --- a/core/modules/views/src/Plugin/views/filter/GroupByNumeric.php +++ b/core/modules/views/src/Plugin/views/filter/GroupByNumeric.php @@ -20,6 +20,7 @@ class GroupByNumeric extends NumericFilter { $this->{$info[$this->operator]['method']}($field); } } + protected function opBetween($field) { $placeholder_min = $this->placeholder(); $placeholder_max = $this->placeholder(); diff --git a/core/modules/views/src/Plugin/views/filter/ManyToOne.php b/core/modules/views/src/Plugin/views/filter/ManyToOne.php index 86eb1f3361ba..1b3810cfe6f2 100644 --- a/core/modules/views/src/Plugin/views/filter/ManyToOne.php +++ b/core/modules/views/src/Plugin/views/filter/ManyToOne.php @@ -103,6 +103,7 @@ class ManyToOne extends InOperator { } protected $valueFormType = 'select'; + protected function valueForm(&$form, FormStateInterface $form_state) { parent::valueForm($form, $form_state); diff --git a/core/modules/views/src/Plugin/views/filter/NumericFilter.php b/core/modules/views/src/Plugin/views/filter/NumericFilter.php index cdfbb06bb318..ed19bee6ae67 100644 --- a/core/modules/views/src/Plugin/views/filter/NumericFilter.php +++ b/core/modules/views/src/Plugin/views/filter/NumericFilter.php @@ -190,6 +190,7 @@ class NumericFilter extends FilterPluginBase { return $options; } + /** * Provide a simple textfield for equality */ diff --git a/core/modules/views/src/Plugin/views/join/JoinPluginBase.php b/core/modules/views/src/Plugin/views/join/JoinPluginBase.php index ccf40f64e9fc..a7ac37d85f71 100644 --- a/core/modules/views/src/Plugin/views/join/JoinPluginBase.php +++ b/core/modules/views/src/Plugin/views/join/JoinPluginBase.php @@ -281,6 +281,7 @@ class JoinPluginBase extends PluginBase implements JoinPluginInterface { $select_query->addJoin($this->type, $right_table, $table['alias'], $condition, $arguments); } + /** * Adds the extras to the join condition. * diff --git a/core/modules/views/src/Plugin/views/pager/SqlBase.php b/core/modules/views/src/Plugin/views/pager/SqlBase.php index d683202d1758..5195acff0df6 100644 --- a/core/modules/views/src/Plugin/views/pager/SqlBase.php +++ b/core/modules/views/src/Plugin/views/pager/SqlBase.php @@ -233,7 +233,6 @@ abstract class SqlBase extends PagerPluginBase implements CacheableDependencyInt $this->view->query->setOffset($offset); } - /** * Set the current page. * diff --git a/core/modules/views/tests/modules/views_test_data/src/Plugin/views/join/JoinTest.php b/core/modules/views/tests/modules/views_test_data/src/Plugin/views/join/JoinTest.php index b57c970fb627..ed96decd57de 100644 --- a/core/modules/views/tests/modules/views_test_data/src/Plugin/views/join/JoinTest.php +++ b/core/modules/views/tests/modules/views_test_data/src/Plugin/views/join/JoinTest.php @@ -35,7 +35,6 @@ class JoinTest extends JoinPluginBase { $this->joinValue = $join_value; } - /** * {@inheritdoc} */ diff --git a/core/modules/views/tests/modules/views_test_data/src/Plugin/views/query/QueryTest.php b/core/modules/views/tests/modules/views_test_data/src/Plugin/views/query/QueryTest.php index 8868f75de667..230e4285036d 100644 --- a/core/modules/views/tests/modules/views_test_data/src/Plugin/views/query/QueryTest.php +++ b/core/modules/views/tests/modules/views_test_data/src/Plugin/views/query/QueryTest.php @@ -74,7 +74,6 @@ class QueryTest extends QueryPluginBase { $this->orderBy = ['field' => $field, 'order' => $order]; } - public function ensureTable($table, $relationship = NULL, JoinPluginBase $join = NULL) { // There is no concept of joins. } diff --git a/core/modules/views/tests/src/Functional/Handler/AreaTest.php b/core/modules/views/tests/src/Functional/Handler/AreaTest.php index d631373d23ef..cf877d137493 100644 --- a/core/modules/views/tests/src/Functional/Handler/AreaTest.php +++ b/core/modules/views/tests/src/Functional/Handler/AreaTest.php @@ -48,7 +48,6 @@ class AreaTest extends ViewTestBase { return $data; } - /** * Tests the generic UI of a area handler. */ diff --git a/core/modules/views/tests/src/Functional/Plugin/ArgumentDefaultTest.php b/core/modules/views/tests/src/Functional/Plugin/ArgumentDefaultTest.php index 57eb2320b8d2..816af901fde0 100644 --- a/core/modules/views/tests/src/Functional/Plugin/ArgumentDefaultTest.php +++ b/core/modules/views/tests/src/Functional/Plugin/ArgumentDefaultTest.php @@ -81,7 +81,6 @@ class ArgumentDefaultTest extends ViewTestBase { $this->assertIdenticalResultset($view, $expected_result, ['views_test_data_name' => 'name']); } - /** * Tests the use of a default argument plugin that provides no options. */ diff --git a/core/modules/views/tests/src/Functional/Plugin/DisplayAttachmentTest.php b/core/modules/views/tests/src/Functional/Plugin/DisplayAttachmentTest.php index 8c326baa0589..44668447c783 100644 --- a/core/modules/views/tests/src/Functional/Plugin/DisplayAttachmentTest.php +++ b/core/modules/views/tests/src/Functional/Plugin/DisplayAttachmentTest.php @@ -36,7 +36,6 @@ class DisplayAttachmentTest extends ViewTestBase { $this->drupalLogin($admin_user); } - /** * Tests the attachment plugin. */ diff --git a/core/modules/views/tests/src/Functional/ViewRenderTest.php b/core/modules/views/tests/src/Functional/ViewRenderTest.php index aaf894cd23e0..24eee7e71326 100644 --- a/core/modules/views/tests/src/Functional/ViewRenderTest.php +++ b/core/modules/views/tests/src/Functional/ViewRenderTest.php @@ -24,7 +24,6 @@ class ViewRenderTest extends ViewTestBase { $this->enableViewsTestModule(); } - /** * Tests render functionality. */ diff --git a/core/modules/views/tests/src/Functional/ViewsFormMultipleTest.php b/core/modules/views/tests/src/Functional/ViewsFormMultipleTest.php index 03d5bcee8ce5..c4c709e3bfc6 100644 --- a/core/modules/views/tests/src/Functional/ViewsFormMultipleTest.php +++ b/core/modules/views/tests/src/Functional/ViewsFormMultipleTest.php @@ -38,7 +38,6 @@ class ViewsFormMultipleTest extends ViewTestBase { return $data; } - /** * Tests the a page with multiple View forms in it. */ diff --git a/core/modules/views/tests/src/Kernel/Handler/ComputedFieldTest.php b/core/modules/views/tests/src/Kernel/Handler/ComputedFieldTest.php index 8f85f6901816..ed00f68a066d 100644 --- a/core/modules/views/tests/src/Kernel/Handler/ComputedFieldTest.php +++ b/core/modules/views/tests/src/Kernel/Handler/ComputedFieldTest.php @@ -28,7 +28,6 @@ class ComputedFieldTest extends ViewsKernelTestBase { */ public static $modules = ['entity_test']; - /** * {@inheritdoc} */ diff --git a/core/modules/views/tests/src/Kernel/Handler/FilterEqualityTest.php b/core/modules/views/tests/src/Kernel/Handler/FilterEqualityTest.php index d78f7c92e2b7..4f8c4961e390 100644 --- a/core/modules/views/tests/src/Kernel/Handler/FilterEqualityTest.php +++ b/core/modules/views/tests/src/Kernel/Handler/FilterEqualityTest.php @@ -146,7 +146,6 @@ class FilterEqualityTest extends ViewsKernelTestBase { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - protected function getGroupedExposedFilters() { $filters = [ 'name' => [ diff --git a/core/modules/views/tests/src/Kernel/Handler/FilterStringTest.php b/core/modules/views/tests/src/Kernel/Handler/FilterStringTest.php index 03c1b878a507..5d8f71f1723a 100644 --- a/core/modules/views/tests/src/Kernel/Handler/FilterStringTest.php +++ b/core/modules/views/tests/src/Kernel/Handler/FilterStringTest.php @@ -222,7 +222,6 @@ class FilterStringTest extends ViewsKernelTestBase { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - public function testFilterStringGroupedExposedContains() { $filters = $this->getGroupedExposedFilters(); $view = $this->getBasicPageView(); @@ -245,7 +244,6 @@ class FilterStringTest extends ViewsKernelTestBase { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - public function testFilterStringWord() { $view = Views::getView('test_view'); $view->setDisplay(); @@ -334,7 +332,6 @@ class FilterStringTest extends ViewsKernelTestBase { $this->assertIdenticalResultset($view, $resultset); } - public function testFilterStringGroupedExposedWord() { $filters = $this->getGroupedExposedFilters(); $view = $this->getBasicPageView(); @@ -618,7 +615,6 @@ class FilterStringTest extends ViewsKernelTestBase { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - public function testFilterStringGroupedExposedNot() { $filters = $this->getGroupedExposedFilters(); $view = $this->getBasicPageView(); @@ -741,7 +737,6 @@ class FilterStringTest extends ViewsKernelTestBase { $this->assertIdenticalResultset($view, $resultset, $this->columnMap); } - public function testFilterStringEmpty() { $view = Views::getView('test_view'); $view->setDisplay(); diff --git a/core/modules/views/tests/src/Kernel/Plugin/CacheTest.php b/core/modules/views/tests/src/Kernel/Plugin/CacheTest.php index 9f6066eff314..708b4046c805 100644 --- a/core/modules/views/tests/src/Kernel/Plugin/CacheTest.php +++ b/core/modules/views/tests/src/Kernel/Plugin/CacheTest.php @@ -61,7 +61,6 @@ class CacheTest extends ViewsKernelTestBase { return $data; } - /** * Tests time based caching. * diff --git a/core/modules/views/tests/src/Kernel/QueryGroupByTest.php b/core/modules/views/tests/src/Kernel/QueryGroupByTest.php index e4a088459fed..25d8b4a4c1dc 100644 --- a/core/modules/views/tests/src/Kernel/QueryGroupByTest.php +++ b/core/modules/views/tests/src/Kernel/QueryGroupByTest.php @@ -51,7 +51,6 @@ class QueryGroupByTest extends ViewsKernelTestBase { ConfigurableLanguage::createFromLangcode('it')->save(); } - /** * Tests aggregate count feature. */ diff --git a/core/modules/views/tests/src/Kernel/ViewsKernelTestBase.php b/core/modules/views/tests/src/Kernel/ViewsKernelTestBase.php index 4bbe4028637d..af69ba1873db 100644 --- a/core/modules/views/tests/src/Kernel/ViewsKernelTestBase.php +++ b/core/modules/views/tests/src/Kernel/ViewsKernelTestBase.php @@ -48,6 +48,7 @@ abstract class ViewsKernelTestBase extends KernelTestBase { ViewTestData::createTestViews(get_class($this), ['views_test_config']); } } + /** * Sets up the configuration and schema of views and views_test_data modules. * diff --git a/core/modules/views/tests/src/Unit/EntityViewsDataTest.php b/core/modules/views/tests/src/Unit/EntityViewsDataTest.php index 405979588eb3..a581c862a187 100644 --- a/core/modules/views/tests/src/Unit/EntityViewsDataTest.php +++ b/core/modules/views/tests/src/Unit/EntityViewsDataTest.php @@ -1134,16 +1134,20 @@ class TestEntityType extends ContentEntityType { namespace Drupal\entity_test\Entity; if (!function_exists('t')) { + function t($string, array $args = []) { return strtr($string, $args); } + } namespace Drupal\Core\Entity; if (!function_exists('t')) { + function t($string, array $args = []) { return strtr($string, $args); } + } diff --git a/core/modules/views/tests/src/Unit/Plugin/Block/ViewsBlockTest.php b/core/modules/views/tests/src/Unit/Plugin/Block/ViewsBlockTest.php index f9bd6d2f1b72..e7cd98477b33 100644 --- a/core/modules/views/tests/src/Unit/Plugin/Block/ViewsBlockTest.php +++ b/core/modules/views/tests/src/Unit/Plugin/Block/ViewsBlockTest.php @@ -204,6 +204,8 @@ class ViewsBlockTest extends UnitTestCase { namespace Drupal\views\Plugin\Block; if (!function_exists('views_add_contextual_links')) { + function views_add_contextual_links() { } + } diff --git a/core/modules/views/tests/src/Unit/Plugin/field/FieldPluginBaseTest.php b/core/modules/views/tests/src/Unit/Plugin/field/FieldPluginBaseTest.php index a7bcf81de6f2..71ab347aa5b4 100644 --- a/core/modules/views/tests/src/Unit/Plugin/field/FieldPluginBaseTest.php +++ b/core/modules/views/tests/src/Unit/Plugin/field/FieldPluginBaseTest.php @@ -775,7 +775,9 @@ class FieldPluginBaseTestField extends FieldPluginBase { namespace Drupal\views\Plugin\views\field; if (!function_exists('base_path')) { + function base_path() { return '/'; } + } diff --git a/core/modules/views/tests/src/Unit/Routing/ViewPageControllerTest.php b/core/modules/views/tests/src/Unit/Routing/ViewPageControllerTest.php index 9242545b7ad3..6ced52ee8471 100644 --- a/core/modules/views/tests/src/Unit/Routing/ViewPageControllerTest.php +++ b/core/modules/views/tests/src/Unit/Routing/ViewPageControllerTest.php @@ -182,6 +182,8 @@ class ViewPageControllerTest extends UnitTestCase { namespace Drupal\views\Routing; if (!function_exists('views_add_contextual_links')) { + function views_add_contextual_links() { } + } diff --git a/core/modules/workflows/tests/src/Unit/WorkflowTest.php b/core/modules/workflows/tests/src/Unit/WorkflowTest.php index dc77a3d2f622..c10ddf95a94d 100644 --- a/core/modules/workflows/tests/src/Unit/WorkflowTest.php +++ b/core/modules/workflows/tests/src/Unit/WorkflowTest.php @@ -417,7 +417,6 @@ class WorkflowTest extends UnitTestCase { $this->assertArrayEquals([], array_keys($workflow->getTypePlugin()->getTransitions([]))); } - /** * @covers ::getTransition */ @@ -474,7 +473,6 @@ class WorkflowTest extends UnitTestCase { $this->assertEquals(['archive'], array_keys($workflow->getTypePlugin()->getTransitionsForState('archived', 'to'))); } - /** * @covers ::getTransitionFromStateToState * @covers ::hasTransitionFromStateToState diff --git a/core/phpcs.xml.dist b/core/phpcs.xml.dist index 29308e9ca599..128b84efa666 100644 --- a/core/phpcs.xml.dist +++ b/core/phpcs.xml.dist @@ -307,6 +307,11 @@ + + + + + diff --git a/core/scripts/update-countries.sh b/core/scripts/update-countries.sh index abcc6994d45c..539d7321a93b 100755 --- a/core/scripts/update-countries.sh +++ b/core/scripts/update-countries.sh @@ -37,9 +37,11 @@ USAGE; // Fake the t() function used in CountryManager.php instead of attempting a full // Drupal bootstrap of core/includes/bootstrap.inc (where t() is declared). if (!function_exists('t')) { + function t($string) { return $string; } + } // Read in existing codes. diff --git a/core/tests/Drupal/FunctionalJavascriptTests/JSWebAssert.php b/core/tests/Drupal/FunctionalJavascriptTests/JSWebAssert.php index 281cb783487a..ce5167ae31fa 100644 --- a/core/tests/Drupal/FunctionalJavascriptTests/JSWebAssert.php +++ b/core/tests/Drupal/FunctionalJavascriptTests/JSWebAssert.php @@ -100,6 +100,7 @@ JS; return $result; } + /** * Waits for a button (input[type=submit|image|button|reset], button) with * specified locator and returns it. diff --git a/core/tests/Drupal/FunctionalTests/Image/ToolkitTest.php b/core/tests/Drupal/FunctionalTests/Image/ToolkitTest.php index 789369ceeb7f..17c91158c265 100644 --- a/core/tests/Drupal/FunctionalTests/Image/ToolkitTest.php +++ b/core/tests/Drupal/FunctionalTests/Image/ToolkitTest.php @@ -8,6 +8,7 @@ namespace Drupal\FunctionalTests\Image; * @group Image */ class ToolkitTest extends ToolkitTestBase { + /** * Check that ImageToolkitManager::getAvailableToolkits() only returns * available toolkits. diff --git a/core/tests/Drupal/FunctionalTests/Installer/DistributionProfileTranslationTest.php b/core/tests/Drupal/FunctionalTests/Installer/DistributionProfileTranslationTest.php index 6e87495693e2..49e7193ae88e 100644 --- a/core/tests/Drupal/FunctionalTests/Installer/DistributionProfileTranslationTest.php +++ b/core/tests/Drupal/FunctionalTests/Installer/DistributionProfileTranslationTest.php @@ -91,7 +91,6 @@ class DistributionProfileTranslationTest extends InstallerTestBase { parent::setUpSettings(); } - /** * Confirms that the installation succeeded. */ diff --git a/core/tests/Drupal/KernelTests/Core/Database/BasicSyntaxTest.php b/core/tests/Drupal/KernelTests/Core/Database/BasicSyntaxTest.php index 53a87b7dbf9c..17f8ebb92c94 100644 --- a/core/tests/Drupal/KernelTests/Core/Database/BasicSyntaxTest.php +++ b/core/tests/Drupal/KernelTests/Core/Database/BasicSyntaxTest.php @@ -12,6 +12,7 @@ namespace Drupal\KernelTests\Core\Database; * @group Database */ class BasicSyntaxTest extends DatabaseTestBase { + /** * Tests string concatenation. */ diff --git a/core/tests/Drupal/KernelTests/Core/Database/CaseSensitivityTest.php b/core/tests/Drupal/KernelTests/Core/Database/CaseSensitivityTest.php index cd241e5145d6..37b876c8f84d 100644 --- a/core/tests/Drupal/KernelTests/Core/Database/CaseSensitivityTest.php +++ b/core/tests/Drupal/KernelTests/Core/Database/CaseSensitivityTest.php @@ -8,6 +8,7 @@ namespace Drupal\KernelTests\Core\Database; * @group Database */ class CaseSensitivityTest extends DatabaseTestBase { + /** * Tests BINARY collation in MySQL. */ diff --git a/core/tests/Drupal/KernelTests/Core/Database/InvalidDataTest.php b/core/tests/Drupal/KernelTests/Core/Database/InvalidDataTest.php index 371800b73d23..f6af91fef88d 100644 --- a/core/tests/Drupal/KernelTests/Core/Database/InvalidDataTest.php +++ b/core/tests/Drupal/KernelTests/Core/Database/InvalidDataTest.php @@ -11,6 +11,7 @@ use Drupal\Core\Database\IntegrityConstraintViolationException; * @group Database */ class InvalidDataTest extends DatabaseTestBase { + /** * Tests aborting of traditional SQL database systems with invalid data. */ diff --git a/core/tests/Drupal/KernelTests/Core/Database/SelectComplexTest.php b/core/tests/Drupal/KernelTests/Core/Database/SelectComplexTest.php index cd3ec4a16384..23ec66494a9e 100644 --- a/core/tests/Drupal/KernelTests/Core/Database/SelectComplexTest.php +++ b/core/tests/Drupal/KernelTests/Core/Database/SelectComplexTest.php @@ -245,7 +245,6 @@ class SelectComplexTest extends DatabaseTestBase { $this->assertEqual($count, 4, 'Counted the correct number of records.'); } - /** * Tests that countQuery properly removes fields and expressions. */ diff --git a/core/tests/Drupal/KernelTests/Core/Database/SerializeQueryTest.php b/core/tests/Drupal/KernelTests/Core/Database/SerializeQueryTest.php index 357b39feb2bd..b3299d6005ae 100644 --- a/core/tests/Drupal/KernelTests/Core/Database/SerializeQueryTest.php +++ b/core/tests/Drupal/KernelTests/Core/Database/SerializeQueryTest.php @@ -8,6 +8,7 @@ namespace Drupal\KernelTests\Core\Database; * @group Database */ class SerializeQueryTest extends DatabaseTestBase { + /** * Confirms that a query can be serialized and unserialized. */ diff --git a/core/tests/Drupal/KernelTests/Core/File/ScanDirectoryTest.php b/core/tests/Drupal/KernelTests/Core/File/ScanDirectoryTest.php index 0bd546bbef14..a096ae852677 100644 --- a/core/tests/Drupal/KernelTests/Core/File/ScanDirectoryTest.php +++ b/core/tests/Drupal/KernelTests/Core/File/ScanDirectoryTest.php @@ -129,7 +129,6 @@ class ScanDirectoryTest extends FileTestBase { $this->assertEqual(2, count($files), 'With recursion we found the expected javascript files.'); } - /** * Check that the min_depth options lets us ignore files in the starting * directory. diff --git a/core/tests/Drupal/KernelTests/Core/File/UnmanagedCopyTest.php b/core/tests/Drupal/KernelTests/Core/File/UnmanagedCopyTest.php index d3c5cd4e62a6..655bb5f398a9 100644 --- a/core/tests/Drupal/KernelTests/Core/File/UnmanagedCopyTest.php +++ b/core/tests/Drupal/KernelTests/Core/File/UnmanagedCopyTest.php @@ -11,6 +11,7 @@ use Drupal\Core\File\FileSystem; * @group File */ class UnmanagedCopyTest extends FileTestBase { + /** * Copy a normal file. */ diff --git a/core/tests/Drupal/KernelTests/Core/File/UnmanagedDeleteRecursiveTest.php b/core/tests/Drupal/KernelTests/Core/File/UnmanagedDeleteRecursiveTest.php index 64045fea767a..05acac413af4 100644 --- a/core/tests/Drupal/KernelTests/Core/File/UnmanagedDeleteRecursiveTest.php +++ b/core/tests/Drupal/KernelTests/Core/File/UnmanagedDeleteRecursiveTest.php @@ -8,6 +8,7 @@ namespace Drupal\KernelTests\Core\File; * @group File */ class UnmanagedDeleteRecursiveTest extends FileTestBase { + /** * Delete a normal file. */ diff --git a/core/tests/Drupal/KernelTests/Core/File/UnmanagedDeleteTest.php b/core/tests/Drupal/KernelTests/Core/File/UnmanagedDeleteTest.php index d1cfb2e7dcee..6871f1936b8d 100644 --- a/core/tests/Drupal/KernelTests/Core/File/UnmanagedDeleteTest.php +++ b/core/tests/Drupal/KernelTests/Core/File/UnmanagedDeleteTest.php @@ -8,6 +8,7 @@ namespace Drupal\KernelTests\Core\File; * @group File */ class UnmanagedDeleteTest extends FileTestBase { + /** * Delete a normal file. */ diff --git a/core/tests/Drupal/KernelTests/Core/File/UnmanagedMoveTest.php b/core/tests/Drupal/KernelTests/Core/File/UnmanagedMoveTest.php index 8619a1507d07..2e3f50c2f098 100644 --- a/core/tests/Drupal/KernelTests/Core/File/UnmanagedMoveTest.php +++ b/core/tests/Drupal/KernelTests/Core/File/UnmanagedMoveTest.php @@ -11,6 +11,7 @@ use Drupal\Core\File\FileSystem; * @group File */ class UnmanagedMoveTest extends FileTestBase { + /** * Move a normal file. */ diff --git a/core/tests/Drupal/KernelTests/Core/File/UnmanagedSaveDataTest.php b/core/tests/Drupal/KernelTests/Core/File/UnmanagedSaveDataTest.php index 549332338825..86ea0293158c 100644 --- a/core/tests/Drupal/KernelTests/Core/File/UnmanagedSaveDataTest.php +++ b/core/tests/Drupal/KernelTests/Core/File/UnmanagedSaveDataTest.php @@ -8,6 +8,7 @@ namespace Drupal\KernelTests\Core\File; * @group File */ class UnmanagedSaveDataTest extends FileTestBase { + /** * Test the file_unmanaged_save_data() function. */ diff --git a/core/tests/Drupal/KernelTests/Core/Theme/TwigEnvironmentTest.php b/core/tests/Drupal/KernelTests/Core/Theme/TwigEnvironmentTest.php index 854c25c8dc63..c1be007b163e 100644 --- a/core/tests/Drupal/KernelTests/Core/Theme/TwigEnvironmentTest.php +++ b/core/tests/Drupal/KernelTests/Core/Theme/TwigEnvironmentTest.php @@ -155,7 +155,6 @@ class TwigEnvironmentTest extends KernelTestBase { ->addTag('twig.loader'); } - /** * Test template invalidation. */ diff --git a/core/tests/Drupal/Tests/Component/Assertion/InspectorTest.php b/core/tests/Drupal/Tests/Component/Assertion/InspectorTest.php index f41824416c4f..717e32749a9e 100644 --- a/core/tests/Drupal/Tests/Component/Assertion/InspectorTest.php +++ b/core/tests/Drupal/Tests/Component/Assertion/InspectorTest.php @@ -256,6 +256,7 @@ class InspectorTest extends TestCase { * Quick class for testing for objects with __toString. */ class StringObject { + /** * {@inheritdoc} */ diff --git a/core/tests/Drupal/Tests/Component/Plugin/PluginBaseTest.php b/core/tests/Drupal/Tests/Component/Plugin/PluginBaseTest.php index de684885ffa6..94cf51a0f5b2 100644 --- a/core/tests/Drupal/Tests/Component/Plugin/PluginBaseTest.php +++ b/core/tests/Drupal/Tests/Component/Plugin/PluginBaseTest.php @@ -63,7 +63,6 @@ class PluginBaseTest extends TestCase { ]; } - /** * @dataProvider providerTestGetDerivativeId * @covers ::getDerivativeId diff --git a/core/tests/Drupal/Tests/Component/Serialization/JsonTest.php b/core/tests/Drupal/Tests/Component/Serialization/JsonTest.php index d12ec61e177f..036fa6639549 100644 --- a/core/tests/Drupal/Tests/Component/Serialization/JsonTest.php +++ b/core/tests/Drupal/Tests/Component/Serialization/JsonTest.php @@ -32,7 +32,6 @@ class JsonTest extends TestCase { */ protected $htmlUnsafeEscaped; - /** * {@inheritdoc} */ diff --git a/core/tests/Drupal/Tests/Component/Utility/ArgumentsResolverTest.php b/core/tests/Drupal/Tests/Component/Utility/ArgumentsResolverTest.php index 9099d6fa582a..70cbd4a0aa88 100644 --- a/core/tests/Drupal/Tests/Component/Utility/ArgumentsResolverTest.php +++ b/core/tests/Drupal/Tests/Component/Utility/ArgumentsResolverTest.php @@ -206,6 +206,7 @@ class ArgumentsResolverTest extends TestCase { * Provides a test class. */ class TestClass { + public function access($foo) { } diff --git a/core/tests/Drupal/Tests/Core/Access/AccessManagerTest.php b/core/tests/Drupal/Tests/Core/Access/AccessManagerTest.php index 231f80906e98..f6214d6f30a3 100644 --- a/core/tests/Drupal/Tests/Core/Access/AccessManagerTest.php +++ b/core/tests/Drupal/Tests/Core/Access/AccessManagerTest.php @@ -567,6 +567,7 @@ class AccessManagerTest extends UnitTestCase { * Defines an interface with a defined access() method for mocking. */ interface TestAccessCheckInterface extends AccessCheckInterface { + public function access(); } diff --git a/core/tests/Drupal/Tests/Core/Access/AccessResultTest.php b/core/tests/Drupal/Tests/Core/Access/AccessResultTest.php index 4467e79cbccc..f8245892292d 100644 --- a/core/tests/Drupal/Tests/Core/Access/AccessResultTest.php +++ b/core/tests/Drupal/Tests/Core/Access/AccessResultTest.php @@ -985,6 +985,7 @@ class UncacheableTestAccessResult implements AccessResultInterface { public function __construct($value) { $this->value = $value; } + /** * {@inheritdoc} */ diff --git a/core/tests/Drupal/Tests/Core/Asset/AssetResolverTest.php b/core/tests/Drupal/Tests/Core/Asset/AssetResolverTest.php index 6ed63aa952eb..95cacea45c6d 100644 --- a/core/tests/Drupal/Tests/Core/Asset/AssetResolverTest.php +++ b/core/tests/Drupal/Tests/Core/Asset/AssetResolverTest.php @@ -162,6 +162,7 @@ if (!defined('JS_DEFAULT')) { } class TestMemoryBackend extends MemoryBackend { + public function getAllCids() { return array_keys($this->cache); } diff --git a/core/tests/Drupal/Tests/Core/Asset/CssCollectionRendererUnitTest.php b/core/tests/Drupal/Tests/Core/Asset/CssCollectionRendererUnitTest.php index 4ff4b8312884..ba0b4db532f3 100644 --- a/core/tests/Drupal/Tests/Core/Asset/CssCollectionRendererUnitTest.php +++ b/core/tests/Drupal/Tests/Core/Asset/CssCollectionRendererUnitTest.php @@ -473,9 +473,11 @@ class CssCollectionRendererUnitTest extends UnitTestCase { * Component/Utility. */ if (!function_exists('Drupal\Tests\Core\Asset\file_create_url')) { + function file_create_url($uri) { return 'file_create_url:' . $uri; } + } /** @@ -483,9 +485,11 @@ if (!function_exists('Drupal\Tests\Core\Asset\file_create_url')) { * Component/Utility. */ if (!function_exists('Drupal\Tests\Core\Asset\file_url_transform_relative')) { + function file_url_transform_relative($uri) { return 'file_url_transform_relative:' . $uri; } + } /** diff --git a/core/tests/Drupal/Tests/Core/Asset/CssOptimizerUnitTest.php b/core/tests/Drupal/Tests/Core/Asset/CssOptimizerUnitTest.php index 59ddca0e0ceb..f2e70b9fd595 100644 --- a/core/tests/Drupal/Tests/Core/Asset/CssOptimizerUnitTest.php +++ b/core/tests/Drupal/Tests/Core/Asset/CssOptimizerUnitTest.php @@ -269,9 +269,11 @@ class CssOptimizerUnitTest extends UnitTestCase { * Component/Utility. */ if (!function_exists('Drupal\Tests\Core\Asset\file_create_url')) { + function file_create_url($uri) { return 'file_create_url:' . $uri; } + } /** @@ -279,9 +281,11 @@ if (!function_exists('Drupal\Tests\Core\Asset\file_create_url')) { * Component/Utility. */ if (!function_exists('Drupal\Tests\Core\Asset\file_url_transform_relative')) { + function file_url_transform_relative($uri) { return 'file_url_transform_relative:' . $uri; } + } /** diff --git a/core/tests/Drupal/Tests/Core/Asset/LibraryDependencyResolverTest.php b/core/tests/Drupal/Tests/Core/Asset/LibraryDependencyResolverTest.php index 6e499d9eaaea..bbd55a64beda 100644 --- a/core/tests/Drupal/Tests/Core/Asset/LibraryDependencyResolverTest.php +++ b/core/tests/Drupal/Tests/Core/Asset/LibraryDependencyResolverTest.php @@ -64,7 +64,6 @@ class LibraryDependencyResolverTest extends UnitTestCase { $this->libraryDependencyResolver = new LibraryDependencyResolver($this->libraryDiscovery); } - /** * Provides test data for ::testGetLibrariesWithDependencies(). */ diff --git a/core/tests/Drupal/Tests/Core/Asset/LibraryDiscoveryParserTest.php b/core/tests/Drupal/Tests/Core/Asset/LibraryDiscoveryParserTest.php index cbb06668c045..7a758cdf6493 100644 --- a/core/tests/Drupal/Tests/Core/Asset/LibraryDiscoveryParserTest.php +++ b/core/tests/Drupal/Tests/Core/Asset/LibraryDiscoveryParserTest.php @@ -211,7 +211,6 @@ class LibraryDiscoveryParserTest extends UnitTestCase { $this->assertEquals(\Drupal::VERSION, $libraries['core-versioned']['js'][0]['version']); } - /** * Tests that the version property of external libraries is handled. * @@ -399,6 +398,7 @@ class LibraryDiscoveryParserTest extends UnitTestCase { $this->assertEquals(FALSE, $library['js'][0]['minified']); $this->assertEquals(TRUE, $library['js'][1]['minified']); } + /** * Tests that an exception is thrown when license is missing when 3rd party. * diff --git a/core/tests/Drupal/Tests/Core/Batch/PercentagesTest.php b/core/tests/Drupal/Tests/Core/Batch/PercentagesTest.php index 0c905e6d9801..382ae29da804 100644 --- a/core/tests/Drupal/Tests/Core/Batch/PercentagesTest.php +++ b/core/tests/Drupal/Tests/Core/Batch/PercentagesTest.php @@ -23,6 +23,7 @@ class PercentagesTest extends UnitTestCase { $actual_result = Percentage::format($total, $current); $this->assertEquals($actual_result, $expected_result, sprintf('The expected the batch api percentage at the state %s/%s is %s%% and got %s%%.', $current, $total, $expected_result, $actual_result)); } + /** * Provide data for batch unit tests. * diff --git a/core/tests/Drupal/Tests/Core/Cache/CacheCollectorTest.php b/core/tests/Drupal/Tests/Core/Cache/CacheCollectorTest.php index 0c06a4b86528..50b1eee1c53f 100644 --- a/core/tests/Drupal/Tests/Core/Cache/CacheCollectorTest.php +++ b/core/tests/Drupal/Tests/Core/Cache/CacheCollectorTest.php @@ -59,7 +59,6 @@ class CacheCollectorTest extends UnitTestCase { $this->getContainerWithCacheTagsInvalidator($this->cacheTagsInvalidator); } - /** * Tests the resolve cache miss function. */ @@ -85,7 +84,6 @@ class CacheCollectorTest extends UnitTestCase { $this->assertEquals($value, $this->collector->get($key)); } - /** * Makes sure that NULL is a valid value and is collected. */ @@ -192,7 +190,6 @@ class CacheCollectorTest extends UnitTestCase { $this->collector->destruct(); } - /** * Tests updating the cache when the lock acquire fails. */ diff --git a/core/tests/Drupal/Tests/Core/Cache/CacheTest.php b/core/tests/Drupal/Tests/Core/Cache/CacheTest.php index 446afcd883cc..5d7f26948741 100644 --- a/core/tests/Drupal/Tests/Core/Cache/CacheTest.php +++ b/core/tests/Drupal/Tests/Core/Cache/CacheTest.php @@ -53,7 +53,6 @@ class CacheTest extends UnitTestCase { $this->assertNull(Cache::validateTags($tags)); } - /** * Provides a list of pairs of cache tags arrays to be merged. * @@ -102,7 +101,6 @@ class CacheTest extends UnitTestCase { ]; } - /** * @covers ::mergeMaxAges * diff --git a/core/tests/Drupal/Tests/Core/Controller/ControllerResolverTest.php b/core/tests/Drupal/Tests/Core/Controller/ControllerResolverTest.php index a14f338aeb5b..227027b3933b 100644 --- a/core/tests/Drupal/Tests/Core/Controller/ControllerResolverTest.php +++ b/core/tests/Drupal/Tests/Core/Controller/ControllerResolverTest.php @@ -190,6 +190,7 @@ class ControllerResolverTest extends UnitTestCase { ['Drupal\Tests\Core\Controller\MockInvokeController', 'This used __invoke().'], ]; } + /** * Tests getControllerFromDefinition() without a callable. */ @@ -252,6 +253,7 @@ class ControllerResolverTest extends UnitTestCase { } class MockController { + public function getResult() { return 'This is a regular controller.'; } @@ -262,6 +264,7 @@ class MockController { } class MockControllerPsr7 { + public function getResult() { return ['#markup' => 'This is a regular controller']; } @@ -274,12 +277,15 @@ class MockControllerPsr7 { class MockContainerInjection implements ContainerInjectionInterface { protected $result; + public function __construct($result) { $this->result = $result; } + public static function create(ContainerInterface $container) { return new static('This used injection.'); } + public function getResult() { return $this->result; } @@ -287,12 +293,14 @@ class MockContainerInjection implements ContainerInjectionInterface { } class MockContainerAware implements ContainerAwareInterface { use ContainerAwareTrait; + public function getResult() { return 'This is container aware.'; } } class MockInvokeController { + public function __invoke() { return 'This used __invoke().'; } diff --git a/core/tests/Drupal/Tests/Core/Database/EmptyStatementTest.php b/core/tests/Drupal/Tests/Core/Database/EmptyStatementTest.php index 19548cd79df2..7cea8076e174 100644 --- a/core/tests/Drupal/Tests/Core/Database/EmptyStatementTest.php +++ b/core/tests/Drupal/Tests/Core/Database/EmptyStatementTest.php @@ -12,6 +12,7 @@ use Drupal\Tests\UnitTestCase; * @group Database */ class EmptyStatementTest extends UnitTestCase { + /** * Tests that the empty result set behaves as empty. */ diff --git a/core/tests/Drupal/Tests/Core/DependencyInjection/Compiler/ProxyServicesPassTest.php b/core/tests/Drupal/Tests/Core/DependencyInjection/Compiler/ProxyServicesPassTest.php index 638bd1512e88..44d45fba915c 100644 --- a/core/tests/Drupal/Tests/Core/DependencyInjection/Compiler/ProxyServicesPassTest.php +++ b/core/tests/Drupal/Tests/Core/DependencyInjection/Compiler/ProxyServicesPassTest.php @@ -29,7 +29,6 @@ class ProxyServicesPassTest extends UnitTestCase { $this->proxyServicesPass = new ProxyServicesPass(); } - /** * @covers ::process */ diff --git a/core/tests/Drupal/Tests/Core/DependencyInjection/Compiler/TaggedHandlersPassTest.php b/core/tests/Drupal/Tests/Core/DependencyInjection/Compiler/TaggedHandlersPassTest.php index 848c458a7514..30a515c78b76 100644 --- a/core/tests/Drupal/Tests/Core/DependencyInjection/Compiler/TaggedHandlersPassTest.php +++ b/core/tests/Drupal/Tests/Core/DependencyInjection/Compiler/TaggedHandlersPassTest.php @@ -433,26 +433,34 @@ class TaggedHandlersPassTest extends UnitTestCase { interface HandlerInterface { } class ValidConsumer { + public function addHandler(HandlerInterface $instance, $priority = 0) { } + public function addNoPriority(HandlerInterface $instance) { } + public function addWithId(HandlerInterface $instance, $id, $priority = 0) { } } class InvalidConsumer { + public function addHandler($instance, $priority = 0) { } } class ValidConsumerWithExtraArguments { + public function addHandler(HandlerInterface $instance, $priority = 0, $extra1 = '', $extra2 = '') { } + public function addNoPriority(HandlerInterface $instance, $extra) { } + public function addWithId(HandlerInterface $instance, $id, $priority = 0, $extra1 = '', $extra2 = NULL) { } + public function addWithDifferentOrder(HandlerInterface $instance, $extra1, $priority = 0, $extra2 = 'default2', $extra3 = 'default3') { } diff --git a/core/tests/Drupal/Tests/Core/DrupalKernel/DrupalKernelTest.php b/core/tests/Drupal/Tests/Core/DrupalKernel/DrupalKernelTest.php index 9ca4ffff8599..da041bda4e49 100644 --- a/core/tests/Drupal/Tests/Core/DrupalKernel/DrupalKernelTest.php +++ b/core/tests/Drupal/Tests/Core/DrupalKernel/DrupalKernelTest.php @@ -245,8 +245,10 @@ EOD; namespace { if (!function_exists('drupal_valid_test_ua')) { + function drupal_valid_test_ua($new_prefix = NULL) { return FALSE; } + } } diff --git a/core/tests/Drupal/Tests/Core/Entity/Access/EntityFormDisplayAccessControlHandlerTest.php b/core/tests/Drupal/Tests/Core/Entity/Access/EntityFormDisplayAccessControlHandlerTest.php index a92d76abb70d..01d9ae2560af 100644 --- a/core/tests/Drupal/Tests/Core/Entity/Access/EntityFormDisplayAccessControlHandlerTest.php +++ b/core/tests/Drupal/Tests/Core/Entity/Access/EntityFormDisplayAccessControlHandlerTest.php @@ -69,7 +69,6 @@ class EntityFormDisplayAccessControlHandlerTest extends UnitTestCase { */ protected $entity; - /** * Returns a mock Entity Type Manager. * diff --git a/core/tests/Drupal/Tests/Core/Entity/BaseFieldDefinitionTest.php b/core/tests/Drupal/Tests/Core/Entity/BaseFieldDefinitionTest.php index 616f6ba0a32b..81069c35d3fd 100644 --- a/core/tests/Drupal/Tests/Core/Entity/BaseFieldDefinitionTest.php +++ b/core/tests/Drupal/Tests/Core/Entity/BaseFieldDefinitionTest.php @@ -29,7 +29,6 @@ class BaseFieldDefinitionTest extends UnitTestCase { */ protected $fieldTypeDefinition; - /** * {@inheritdoc} */ diff --git a/core/tests/Drupal/Tests/Core/Entity/EntityListBuilderTest.php b/core/tests/Drupal/Tests/Core/Entity/EntityListBuilderTest.php index 55928b98a6e2..6662cb22779c 100644 --- a/core/tests/Drupal/Tests/Core/Entity/EntityListBuilderTest.php +++ b/core/tests/Drupal/Tests/Core/Entity/EntityListBuilderTest.php @@ -154,6 +154,7 @@ class EntityListBuilderTest extends UnitTestCase { } class TestEntityListBuilder extends EntityTestListBuilder { + public function buildOperations(EntityInterface $entity) { return []; } diff --git a/core/tests/Drupal/Tests/Core/Entity/Routing/DefaultHtmlRouteProviderTest.php b/core/tests/Drupal/Tests/Core/Entity/Routing/DefaultHtmlRouteProviderTest.php index 4e25e4e3b57a..8f89eaabc217 100644 --- a/core/tests/Drupal/Tests/Core/Entity/Routing/DefaultHtmlRouteProviderTest.php +++ b/core/tests/Drupal/Tests/Core/Entity/Routing/DefaultHtmlRouteProviderTest.php @@ -356,15 +356,19 @@ class TestDefaultHtmlRouteProvider extends DefaultHtmlRouteProvider { public function getEntityTypeIdKeyType(EntityTypeInterface $entity_type) { return parent::getEntityTypeIdKeyType($entity_type); } + public function getAddPageRoute(EntityTypeInterface $entity_type) { return parent::getAddPageRoute($entity_type); } + public function getAddFormRoute(EntityTypeInterface $entity_type) { return parent::getAddFormRoute($entity_type); } + public function getCanonicalRoute(EntityTypeInterface $entity_type) { return parent::getCanonicalRoute($entity_type); } + public function getCollectionRoute(EntityTypeInterface $entity_type) { return parent::getCollectionRoute($entity_type); } diff --git a/core/tests/Drupal/Tests/Core/EventSubscriber/PsrResponseSubscriberTest.php b/core/tests/Drupal/Tests/Core/EventSubscriber/PsrResponseSubscriberTest.php index f5855be83757..8529f33dc0fe 100644 --- a/core/tests/Drupal/Tests/Core/EventSubscriber/PsrResponseSubscriberTest.php +++ b/core/tests/Drupal/Tests/Core/EventSubscriber/PsrResponseSubscriberTest.php @@ -54,7 +54,6 @@ class PsrResponseSubscriberTest extends UnitTestCase { $this->psrResponseSubscriber->onKernelView($event); } - /** * Tests altering and finished event. * diff --git a/core/tests/Drupal/Tests/Core/Extension/ExtensionListTest.php b/core/tests/Drupal/Tests/Core/Extension/ExtensionListTest.php index 80a0c7669588..3b9d09a12a8a 100644 --- a/core/tests/Drupal/Tests/Core/Extension/ExtensionListTest.php +++ b/core/tests/Drupal/Tests/Core/Extension/ExtensionListTest.php @@ -155,7 +155,6 @@ class ExtensionListTest extends UnitTestCase { $this->assertEquals('vfs://drupal_root/example/test_name/test_name.info.yml', $pathname); } - /** * @covers ::setPathname * @covers ::getPathname diff --git a/core/tests/Drupal/Tests/Core/Field/FieldDefinitionListenerTest.php b/core/tests/Drupal/Tests/Core/Field/FieldDefinitionListenerTest.php index 8bedf0750f03..1312f40c80ce 100644 --- a/core/tests/Drupal/Tests/Core/Field/FieldDefinitionListenerTest.php +++ b/core/tests/Drupal/Tests/Core/Field/FieldDefinitionListenerTest.php @@ -232,7 +232,6 @@ class FieldDefinitionListenerTest extends UnitTestCase { $this->fieldDefinitionListener->onFieldDefinitionDelete($field_definition->reveal()); } - /** * @covers ::onFieldDefinitionDelete */ diff --git a/core/tests/Drupal/Tests/Core/Form/FormBuilderTest.php b/core/tests/Drupal/Tests/Core/Form/FormBuilderTest.php index 76b7584186d8..8d20db5c3f1d 100644 --- a/core/tests/Drupal/Tests/Core/Form/FormBuilderTest.php +++ b/core/tests/Drupal/Tests/Core/Form/FormBuilderTest.php @@ -881,6 +881,7 @@ class FormBuilderTest extends FormTestBase { } class TestForm implements FormInterface { + public function getFormId() { return 'test_form'; } @@ -888,11 +889,14 @@ class TestForm implements FormInterface { public function buildForm(array $form, FormStateInterface $form_state) { return test_form_id(); } + public function validateForm(array &$form, FormStateInterface $form_state) {} + public function submitForm(array &$form, FormStateInterface $form_state) {} } class TestFormInjected extends TestForm implements ContainerInjectionInterface { + public static function create(ContainerInterface $container) { return new static(); } diff --git a/core/tests/Drupal/Tests/Core/Form/FormCacheTest.php b/core/tests/Drupal/Tests/Core/Form/FormCacheTest.php index 3ae9d6c139fb..f089bc37678c 100644 --- a/core/tests/Drupal/Tests/Core/Form/FormCacheTest.php +++ b/core/tests/Drupal/Tests/Core/Form/FormCacheTest.php @@ -417,7 +417,6 @@ class FormCacheTest extends UnitTestCase { $this->formCache->setCache($form_build_id, $form, $form_state); } - /** * @covers ::deleteCache */ diff --git a/core/tests/Drupal/Tests/Core/Form/FormStateDecoratorBaseTest.php b/core/tests/Drupal/Tests/Core/Form/FormStateDecoratorBaseTest.php index de7baa996d7a..ae42e511a27a 100644 --- a/core/tests/Drupal/Tests/Core/Form/FormStateDecoratorBaseTest.php +++ b/core/tests/Drupal/Tests/Core/Form/FormStateDecoratorBaseTest.php @@ -1119,6 +1119,7 @@ class FormStateDecoratorBaseTest extends UnitTestCase { $this->assertSame($values, $this->formStateDecoratorBase->getValues()); } + /** * @covers ::getValue */ diff --git a/core/tests/Drupal/Tests/Core/Form/FormStateTest.php b/core/tests/Drupal/Tests/Core/Form/FormStateTest.php index 23579a12a9b8..aa60d61006ad 100644 --- a/core/tests/Drupal/Tests/Core/Form/FormStateTest.php +++ b/core/tests/Drupal/Tests/Core/Form/FormStateTest.php @@ -434,12 +434,15 @@ class FormStateTest extends UnitTestCase { * A test form used for the prepareCallback() tests. */ class PrepareCallbackTestForm implements FormInterface { + public function getFormId() { return 'test_form'; } public function buildForm(array $form, FormStateInterface $form_state) {} + public function validateForm(array &$form, FormStateInterface $form_state) {} + public function submitForm(array &$form, FormStateInterface $form_state) {} } diff --git a/core/tests/Drupal/Tests/Core/Mail/MailManagerTest.php b/core/tests/Drupal/Tests/Core/Mail/MailManagerTest.php index 9ba2fb52b70c..ea523028a8dc 100644 --- a/core/tests/Drupal/Tests/Core/Mail/MailManagerTest.php +++ b/core/tests/Drupal/Tests/Core/Mail/MailManagerTest.php @@ -135,7 +135,6 @@ class MailManagerTest extends UnitTestCase { $this->assertInstanceOf('Drupal\Core\Mail\Plugin\Mail\TestMailCollector', $instance); } - /** * Tests that mails are sent in a separate render context. * @@ -163,6 +162,7 @@ class MailManagerTest extends UnitTestCase { * Provides a testing version of MailManager with an empty constructor. */ class TestMailManager extends MailManager { + /** * Sets the discovery for the manager. * diff --git a/core/tests/Drupal/Tests/Core/Menu/LocalTaskDefaultTest.php b/core/tests/Drupal/Tests/Core/Menu/LocalTaskDefaultTest.php index 84b91afe97d7..cc5514a76be4 100644 --- a/core/tests/Drupal/Tests/Core/Menu/LocalTaskDefaultTest.php +++ b/core/tests/Drupal/Tests/Core/Menu/LocalTaskDefaultTest.php @@ -317,6 +317,7 @@ class LocalTaskDefaultTest extends UnitTestCase { } class TestLocalTaskDefault extends LocalTaskDefault { + public function setRouteProvider(RouteProviderInterface $route_provider) { $this->routeProvider = $route_provider; return $this; diff --git a/core/tests/Drupal/Tests/Core/Password/PasswordHashingTest.php b/core/tests/Drupal/Tests/Core/Password/PasswordHashingTest.php index a4e57eaada35..5c81b7eb2b46 100644 --- a/core/tests/Drupal/Tests/Core/Password/PasswordHashingTest.php +++ b/core/tests/Drupal/Tests/Core/Password/PasswordHashingTest.php @@ -76,7 +76,6 @@ class PasswordHashingTest extends UnitTestCase { $this->assertEquals(PhpassHashedPassword::MAX_HASH_COUNT, $hasher->enforceLog2Boundaries(100), "Max hash count enforced"); } - /** * Test a password needs update. * diff --git a/core/tests/Drupal/Tests/Core/Path/AliasManagerTest.php b/core/tests/Drupal/Tests/Core/Path/AliasManagerTest.php index ffd52dc4af43..109c7cc2c2a9 100644 --- a/core/tests/Drupal/Tests/Core/Path/AliasManagerTest.php +++ b/core/tests/Drupal/Tests/Core/Path/AliasManagerTest.php @@ -145,7 +145,6 @@ class AliasManagerTest extends UnitTestCase { $this->assertEquals($path, $this->aliasManager->getPathByAlias($alias, 'de')); } - /** * Tests the getAliasByPath method for a path that is not in the whitelist. * diff --git a/core/tests/Drupal/Tests/Core/Plugin/Context/ContextDefinitionIsSatisfiedTest.php b/core/tests/Drupal/Tests/Core/Plugin/Context/ContextDefinitionIsSatisfiedTest.php index d74607584f37..53d16ffe2187 100644 --- a/core/tests/Drupal/Tests/Core/Plugin/Context/ContextDefinitionIsSatisfiedTest.php +++ b/core/tests/Drupal/Tests/Core/Plugin/Context/ContextDefinitionIsSatisfiedTest.php @@ -346,7 +346,9 @@ class ContextDefinitionIsSatisfiedTest extends UnitTestCase { namespace Drupal\Core\Validation; if (!function_exists('t')) { + function t($string, array $args = []) { return strtr($string, $args); } + } diff --git a/core/tests/Drupal/Tests/Core/Plugin/ContextHandlerTest.php b/core/tests/Drupal/Tests/Core/Plugin/ContextHandlerTest.php index 5cf8d9cc8bf2..f1ca26ecf7fd 100644 --- a/core/tests/Drupal/Tests/Core/Plugin/ContextHandlerTest.php +++ b/core/tests/Drupal/Tests/Core/Plugin/ContextHandlerTest.php @@ -408,7 +408,6 @@ class ContextHandlerTest extends UnitTestCase { $this->contextHandler->applyContextMapping($plugin, $contexts); } - /** * @covers ::applyContextMapping */ diff --git a/core/tests/Drupal/Tests/Core/Plugin/Discovery/HookDiscoveryTest.php b/core/tests/Drupal/Tests/Core/Plugin/Discovery/HookDiscoveryTest.php index 9f4eb95dfcaf..8eb4b3d99854 100644 --- a/core/tests/Drupal/Tests/Core/Plugin/Discovery/HookDiscoveryTest.php +++ b/core/tests/Drupal/Tests/Core/Plugin/Discovery/HookDiscoveryTest.php @@ -135,6 +135,7 @@ class HookDiscoveryTest extends UnitTestCase { 'test_id_2' => ['class' => 'Drupal\plugin_test\Plugin\plugin_test\fruit\Orange'], ]; } + protected function hookDiscoveryTest2TestPlugin() { return [ 'test_id_3' => ['class' => 'Drupal\plugin_test\Plugin\plugin_test\fruit\Cherry'], diff --git a/core/tests/Drupal/Tests/Core/Render/BubbleableMetadataTest.php b/core/tests/Drupal/Tests/Core/Render/BubbleableMetadataTest.php index a881e7c2ce72..040b1f988e1a 100644 --- a/core/tests/Drupal/Tests/Core/Render/BubbleableMetadataTest.php +++ b/core/tests/Drupal/Tests/Core/Render/BubbleableMetadataTest.php @@ -641,7 +641,6 @@ class BubbleableMetadataTest extends UnitTestCase { ]; } - /** * @covers ::addCacheableDependency * @dataProvider providerTestMerge diff --git a/core/tests/Drupal/Tests/Core/Render/Element/MachineNameTest.php b/core/tests/Drupal/Tests/Core/Render/Element/MachineNameTest.php index 0dcdc5829035..db463d8b40e7 100644 --- a/core/tests/Drupal/Tests/Core/Render/Element/MachineNameTest.php +++ b/core/tests/Drupal/Tests/Core/Render/Element/MachineNameTest.php @@ -42,7 +42,6 @@ class MachineNameTest extends UnitTestCase { return $data; } - /** * @covers ::processMachineName */ @@ -112,7 +111,9 @@ class MachineNameTest extends UnitTestCase { namespace Drupal\Core\Render\Element; if (!function_exists('t')) { + function t($string, array $args = []) { return strtr($string, $args); } + } diff --git a/core/tests/Drupal/Tests/Core/Render/RendererPlaceholdersTest.php b/core/tests/Drupal/Tests/Core/Render/RendererPlaceholdersTest.php index 253a3580dc35..8cfa0fda7c8a 100644 --- a/core/tests/Drupal/Tests/Core/Render/RendererPlaceholdersTest.php +++ b/core/tests/Drupal/Tests/Core/Render/RendererPlaceholdersTest.php @@ -574,6 +574,7 @@ class RendererPlaceholdersTest extends RendererTestBase { $this->assertEquals($expected_data, $cached_element, 'The correct data is cached: the stored #markup and #attached properties are not affected by the placeholder being replaced.'); } } + /** * @covers ::render * @covers ::doRender diff --git a/core/tests/Drupal/Tests/Core/Routing/RouteBuilderTest.php b/core/tests/Drupal/Tests/Core/Routing/RouteBuilderTest.php index d075934100d6..974df439d9bc 100644 --- a/core/tests/Drupal/Tests/Core/Routing/RouteBuilderTest.php +++ b/core/tests/Drupal/Tests/Core/Routing/RouteBuilderTest.php @@ -311,11 +311,13 @@ class TestRouteBuilder extends RouteBuilder { * Provides a callback for route definition. */ class TestRouteSubscriber { + public function routesFromArray() { return [ 'test_route.1' => new Route('/test-route/1'), ]; } + public function routesFromCollection() { $collection = new RouteCollection(); $collection->add('test_route.2', new Route('/test-route/2')); diff --git a/core/tests/Drupal/Tests/Core/Routing/RouteMatchTestBase.php b/core/tests/Drupal/Tests/Core/Routing/RouteMatchTestBase.php index 4d2a914bf76e..a47ec73210fa 100644 --- a/core/tests/Drupal/Tests/Core/Routing/RouteMatchTestBase.php +++ b/core/tests/Drupal/Tests/Core/Routing/RouteMatchTestBase.php @@ -26,7 +26,6 @@ abstract class RouteMatchTestBase extends UnitTestCase { */ abstract protected function getRouteMatch($name, Route $route, array $parameters, array $raw_parameters); - /** * Provide sets of parameters and expected parameters for parameter tests. */ diff --git a/core/tests/Drupal/Tests/Core/Routing/RoutePreloaderTest.php b/core/tests/Drupal/Tests/Core/Routing/RoutePreloaderTest.php index 90b4a9ddb66c..4b7c7410a64a 100644 --- a/core/tests/Drupal/Tests/Core/Routing/RoutePreloaderTest.php +++ b/core/tests/Drupal/Tests/Core/Routing/RoutePreloaderTest.php @@ -97,7 +97,6 @@ class RoutePreloaderTest extends UnitTestCase { $this->preloader->onFinishedRoutes(new Event()); } - /** * Tests onAlterRoutes with admin routes and non admin routes. */ diff --git a/core/tests/Drupal/Tests/Core/Session/AccountProxyTest.php b/core/tests/Drupal/Tests/Core/Session/AccountProxyTest.php index 0e575a07ecc6..3e6d501723d5 100644 --- a/core/tests/Drupal/Tests/Core/Session/AccountProxyTest.php +++ b/core/tests/Drupal/Tests/Core/Session/AccountProxyTest.php @@ -47,7 +47,9 @@ class AccountProxyTest extends UnitTestCase { namespace Drupal\Core\Session; if (!function_exists('drupal_get_user_timezone')) { + function drupal_get_user_timezone() { return date_default_timezone_get(); } + } diff --git a/core/tests/Drupal/Tests/Core/Session/PermissionsHashGeneratorTest.php b/core/tests/Drupal/Tests/Core/Session/PermissionsHashGeneratorTest.php index aa980f8c951f..ef1253118711 100644 --- a/core/tests/Drupal/Tests/Core/Session/PermissionsHashGeneratorTest.php +++ b/core/tests/Drupal/Tests/Core/Session/PermissionsHashGeneratorTest.php @@ -250,6 +250,7 @@ namespace Drupal\Core\Session; // @todo remove once user_role_permissions() can be injected. if (!function_exists('user_role_permissions')) { + function user_role_permissions(array $roles) { $role_permissions = []; foreach ($roles as $rid) { @@ -257,4 +258,5 @@ if (!function_exists('user_role_permissions')) { } return $role_permissions; } + } diff --git a/core/tests/Drupal/Tests/Core/StackMiddleware/NegotiationMiddlewareTest.php b/core/tests/Drupal/Tests/Core/StackMiddleware/NegotiationMiddlewareTest.php index 7b3a0d45da89..10d2df8b8ba2 100644 --- a/core/tests/Drupal/Tests/Core/StackMiddleware/NegotiationMiddlewareTest.php +++ b/core/tests/Drupal/Tests/Core/StackMiddleware/NegotiationMiddlewareTest.php @@ -142,6 +142,7 @@ class NegotiationMiddlewareTest extends UnitTestCase { } class StubNegotiationMiddleware extends NegotiationMiddleware { + public function getContentType(Request $request) { return parent::getContentType($request); } diff --git a/core/tests/Drupal/Tests/Core/TempStore/PrivateTempStoreTest.php b/core/tests/Drupal/Tests/Core/TempStore/PrivateTempStoreTest.php index 0144919b2422..c289b89b58b2 100644 --- a/core/tests/Drupal/Tests/Core/TempStore/PrivateTempStoreTest.php +++ b/core/tests/Drupal/Tests/Core/TempStore/PrivateTempStoreTest.php @@ -93,7 +93,6 @@ class PrivateTempStoreTest extends UnitTestCase { $this->otherObject->owner = 2; } - /** * Tests the get() method. * diff --git a/core/tests/Drupal/Tests/Core/UnroutedUrlTest.php b/core/tests/Drupal/Tests/Core/UnroutedUrlTest.php index 680e2fde182b..fb4b84cc934c 100644 --- a/core/tests/Drupal/Tests/Core/UnroutedUrlTest.php +++ b/core/tests/Drupal/Tests/Core/UnroutedUrlTest.php @@ -73,7 +73,6 @@ class UnroutedUrlTest extends UnitTestCase { $this->assertInstanceOf('Drupal\Core\Url', $url); } - /** * Data provider for testFromUri(). */ diff --git a/core/tests/Drupal/Tests/Listeners/HtmlOutputPrinter.php b/core/tests/Drupal/Tests/Listeners/HtmlOutputPrinter.php index 80219898682f..49618b8172b6 100644 --- a/core/tests/Drupal/Tests/Listeners/HtmlOutputPrinter.php +++ b/core/tests/Drupal/Tests/Listeners/HtmlOutputPrinter.php @@ -20,6 +20,7 @@ else { */ class HtmlOutputPrinter extends ResultPrinter { use HtmlOutputPrinterTrait; + /** * {@inheritdoc} */ diff --git a/core/tests/Drupal/Tests/PhpunitCompatibilityTraitTest.php b/core/tests/Drupal/Tests/PhpunitCompatibilityTraitTest.php index 145980bfe531..790831a6d7d0 100644 --- a/core/tests/Drupal/Tests/PhpunitCompatibilityTraitTest.php +++ b/core/tests/Drupal/Tests/PhpunitCompatibilityTraitTest.php @@ -52,6 +52,7 @@ class PhpunitCompatibilityTraitTest extends UnitTestCase { * Test class for \PHPUnit\Framework\TestCase in PHPUnit 4. */ class Phpunit4TestClass { + public function getMock($originalClassName) { return 'PHPUnit 4'; } @@ -62,6 +63,7 @@ class Phpunit4TestClass { * Test class for \PHPUnit\Framework\TestCase in PHPUnit 6. */ class Phpunit6TestClass { + public function createMock($originalClassName) { return 'PHPUnit 6'; } @@ -76,6 +78,7 @@ class Phpunit6TestClass { * Test double for PHPUnit_Framework_MockObject_MockBuilder. */ class Mockbuilder { + public function __call($name, $arguments) { return $this; }