Issue #2721901 by Mile23, anoopjohn: Fix Drupal.Commenting.FunctionComment.MissingParamName

core/lib/Drupal/Component/Assertion/Inspector.php

@@ -108,7 +108,7 @@ public static function assertAllStringable($traversable) {
    * Use this instead of is_string() alone unless the argument being an object
    * in any way will cause a problem.
    *
-   * @param mixed string
+   * @param mixed $string
    *   Variable to be examined
    *
    * @return bool

core/lib/Drupal/Component/Gettext/PoItem.php

@@ -169,7 +169,7 @@ function setComment($comment) {
   /**
    * Create the PoItem from a structured array.
    *
-   * @param array values
+   * @param array $values
    */
   public function setFromArray(array $values = array()) {
     if (isset($values['context'])) {

core/lib/Drupal/Core/Asset/CssCollectionOptimizer.php

@@ -40,13 +40,13 @@ class CssCollectionOptimizer implements AssetCollectionOptimizerInterface {
   /**
    * Constructs a CssCollectionOptimizer.
    *
-   * @param \Drupal\Core\Asset\AssetCollectionGrouperInterface
+   * @param \Drupal\Core\Asset\AssetCollectionGrouperInterface $grouper
    *   The grouper for CSS assets.
-   * @param \Drupal\Core\Asset\AssetOptimizerInterface
+   * @param \Drupal\Core\Asset\AssetOptimizerInterface $optimizer
    *   The optimizer for a single CSS asset.
-   * @param \Drupal\Core\Asset\AssetDumperInterface
+   * @param \Drupal\Core\Asset\AssetDumperInterface $dumper
    *   The dumper for optimized CSS assets.
-   * @param \Drupal\Core\State\StateInterface
+   * @param \Drupal\Core\State\StateInterface $state
    *   The state key/value store.
    */
   public function __construct(AssetCollectionGrouperInterface $grouper, AssetOptimizerInterface $optimizer, AssetDumperInterface $dumper, StateInterface $state) {

core/lib/Drupal/Core/Asset/CssCollectionRenderer.php

@@ -63,7 +63,7 @@ class CssCollectionRenderer implements AssetCollectionRendererInterface {
   /**
    * Constructs a CssCollectionRenderer.
    *
-   * @param \Drupal\Core\State\StateInterface
+   * @param \Drupal\Core\State\StateInterface $state
    *   The state key/value store.
    */
   public function __construct(StateInterface $state) {

core/lib/Drupal/Core/Asset/JsCollectionOptimizer.php

@@ -41,13 +41,13 @@ class JsCollectionOptimizer implements AssetCollectionOptimizerInterface {
   /**
    * Constructs a JsCollectionOptimizer.
    *
-   * @param \Drupal\Core\Asset\AssetCollectionGrouperInterface
+   * @param \Drupal\Core\Asset\AssetCollectionGrouperInterface $grouper
    *   The grouper for JS assets.
-   * @param \Drupal\Core\Asset\AssetOptimizerInterface
+   * @param \Drupal\Core\Asset\AssetOptimizerInterface $optimizer
    *   The optimizer for a single JS asset.
-   * @param \Drupal\Core\Asset\AssetDumperInterface
+   * @param \Drupal\Core\Asset\AssetDumperInterface $dumper
    *   The dumper for optimized JS assets.
-   * @param \Drupal\Core\State\StateInterface
+   * @param \Drupal\Core\State\StateInterface $state
    *   The state key/value store.
    */
   public function __construct(AssetCollectionGrouperInterface $grouper, AssetOptimizerInterface $optimizer, AssetDumperInterface $dumper, StateInterface $state) {

core/lib/Drupal/Core/Asset/JsCollectionRenderer.php

@@ -20,7 +20,7 @@ class JsCollectionRenderer implements AssetCollectionRendererInterface {
   /**
    * Constructs a JsCollectionRenderer.
    *
-   * @param \Drupal\Core\State\StateInterface
+   * @param \Drupal\Core\State\StateInterface $state
    *   The state key/value store.
    */
   public function __construct(StateInterface $state) {

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

@@ -24,7 +24,7 @@ class ConfigCollectionInfo extends Event {
    *
    * @param string $collection
    *   Collection name to add.
-   * @param \Drupal\Core\Config\ConfigFactoryOverrideInterface
+   * @param \Drupal\Core\Config\ConfigFactoryOverrideInterface $override_service
    *   (optional) The configuration factory override service responsible for the
    *   collection.
    *

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

@@ -19,7 +19,7 @@ class ConfigCrudEvent extends Event {
   /**
    * Constructs a configuration event object.
    *
-   * @param \Drupal\Core\Config\Config
+   * @param \Drupal\Core\Config\Config $config
    *   Configuration object.
    */
   public function __construct(Config $config) {

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

@@ -40,7 +40,7 @@ public function installDefaultConfig($type, $name);
    * - it's a configuration entity.
    * - its dependencies can be met.
    *
-   * @param \Drupal\Core\Config\StorageInterface
+   * @param \Drupal\Core\Config\StorageInterface $storage
    *   (optional) The configuration storage to search for optional
    *   configuration. If not provided, all enabled extension's optional
    *   configuration directories will be searched.

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

@@ -37,7 +37,7 @@ class ConfigModuleOverridesEvent extends Event {
    *
    * @param array $names
    *   A list of configuration names.
-   * @param \Drupal\Core\Language\LanguageInterface
+   * @param \Drupal\Core\Language\LanguageInterface $language
    *   (optional) The language for this configuration.
    */
   public function __construct(array $names, LanguageInterface $language = NULL) {

core/lib/Drupal/Core/Cron.php

@@ -83,7 +83,7 @@ class Cron implements CronInterface {
    *    The account switching service.
    * @param \Psr\Log\LoggerInterface $logger
    *   A logger instance.
-   * @param \Drupal\Core\Queue\QueueWorkerManagerInterface
+   * @param \Drupal\Core\Queue\QueueWorkerManagerInterface $queue_manager
    *   The queue plugin manager.
    */
   public function __construct(ModuleHandlerInterface $module_handler, LockBackendInterface $lock, QueueFactory $queue_factory, StateInterface $state, AccountSwitcherInterface $account_switcher, LoggerInterface $logger, QueueWorkerManagerInterface $queue_manager) {

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

@@ -30,7 +30,7 @@ class EntityRouteAlterSubscriber implements EventSubscriberInterface {
   /**
    * Constructs an EntityRouteAlterSubscriber instance.
    *
-   * @param \Drupal\Core\Entity\EntityResolverManager
+   * @param \Drupal\Core\Entity\EntityResolverManager $entity_resolver_manager
    *   The entity resolver manager.
    */
   public function __construct(EntityResolverManager $entity_resolver_manager) {

core/lib/Drupal/Core/Field/FieldItemListInterface.php

@@ -245,7 +245,7 @@ public function defaultValuesFormSubmit(array $element, array &$form, FormStateI
    * in order to be a valid runtime value for the field type; e.g., a date field
    * could process the defined value of 'NOW' to a valid date.
    *
-   * @param array
+   * @param array $default_value
    *   The unprocessed default value defined for the field, as a numerically
    *   indexed array of items, each item being an array of property/value pairs.
    * @param \Drupal\Core\Entity\FieldableEntityInterface $entity

core/lib/Drupal/Core/Field/FieldTypePluginManager.php

@@ -34,7 +34,7 @@ class FieldTypePluginManager extends DefaultPluginManager implements FieldTypePl
    *   keyed by the corresponding namespace to look for plugin implementations.
    * @param \Drupal\Core\Cache\CacheBackendInterface $cache_backend
    *   Cache backend instance to use.
-   * @param \Drupal\Core\Extension\ModuleHandlerInterface
+   * @param \Drupal\Core\Extension\ModuleHandlerInterface $module_handler
    *   The module handler.
    * @param \Drupal\Core\TypedData\TypedDataManagerInterface $typed_data_manager
    *   The typed data manager.

core/lib/Drupal/Core/Form/FormStateInterface.php

@@ -780,7 +780,7 @@ public function setRequestMethod($method);
   /**
    * Returns the HTTP form method.
    *
-   * @param string
+   * @param string $method_type
    *   The HTTP form method.
    *
    * @return bool

core/lib/Drupal/Core/Language/ContextProvider/CurrentLanguageContext.php

@@ -26,7 +26,7 @@ class CurrentLanguageContext implements ContextProviderInterface {
   /**
    * Constructs a new CurrentLanguageContext.
    *
-   * @param \Drupal\Core\Language\LanguageManagerInterface
+   * @param \Drupal\Core\Language\LanguageManagerInterface $language_manager
    *   The language manager.
    */
   public function __construct(LanguageManagerInterface $language_manager) {

core/lib/Drupal/Core/Render/PageDisplayVariantSelectionEvent.php

@@ -54,7 +54,7 @@ class PageDisplayVariantSelectionEvent extends Event implements RefinableCacheab
   /**
    * Constructs the page display variant plugin selection event.
    *
-   * @param string
+   * @param string $plugin_id
    *   The ID of the page display variant plugin to use by default.
    * @param \Drupal\Core\Routing\RouteMatchInterface $route_match
    *   The current route match, for context.

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

@@ -36,7 +36,7 @@ public function getParentRouteMatch();
   /**
    * Returns a route match from a given request, if possible.
    *
-   * @param \Symfony\Component\HttpFoundation\Request
+   * @param \Symfony\Component\HttpFoundation\Request $request
    *   The request.
    *
    * @return \Drupal\Core\Routing\RouteMatchInterface|null

core/lib/Drupal/Core/Session/PermissionsHashGenerator.php

@@ -40,7 +40,7 @@ class PermissionsHashGenerator implements PermissionsHashGeneratorInterface {
    *   The private key service.
    * @param \Drupal\Core\Cache\CacheBackendInterface $cache
    *   The cache backend interface to use for the persistent cache.
-   * @param \Drupal\Core\Cache\CacheBackendInterface
+   * @param \Drupal\Core\Cache\CacheBackendInterface $static
    *   The cache backend interface to use for the static cache.
    */
   public function __construct(PrivateKey $private_key, CacheBackendInterface $cache, CacheBackendInterface $static) {

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

@@ -192,7 +192,7 @@ public function getValidationConstraintManager();
    * The validation constraint manager is used to instantiate validation
    * constraint plugins.
    *
-   * @param \Drupal\Core\Validation\ConstraintManager
+   * @param \Drupal\Core\Validation\ConstraintManager $constraintManager
    *   The constraint manager to set.
    */
   public function setValidationConstraintManager(ConstraintManager $constraintManager);

core/lib/Drupal/Core/Url.php

@@ -845,7 +845,7 @@ protected function unroutedUrlAssembler() {
   /**
    * Sets the URL generator.
    *
-   * @param \Drupal\Core\Routing\UrlGeneratorInterface
+   * @param \Drupal\Core\Routing\UrlGeneratorInterface $url_generator
    *   (optional) The URL generator, specify NULL to reset it.
    *
    * @return $this
@@ -859,7 +859,7 @@ public function setUrlGenerator(UrlGeneratorInterface $url_generator = NULL) {
   /**
    * Sets the unrouted URL assembler.
    *
-   * @param \Drupal\Core\Utility\UnroutedUrlAssemblerInterface
+   * @param \Drupal\Core\Utility\UnroutedUrlAssemblerInterface $url_assembler
    *   The unrouted URL assembler.
    *
    * @return $this

core/modules/action/src/Plugin/Action/EmailAction.php

@@ -84,9 +84,9 @@ class EmailAction extends ConfigurableActionBase implements ContainerFactoryPlug
    *   The entity manager.
    * @param \Psr\Log\LoggerInterface $logger
    *   A logger instance.
-   * @param \Drupal\Core\Mail\MailManagerInterface
+   * @param \Drupal\Core\Mail\MailManagerInterface $mail_manager
    *   The mail manager.
-   * @param \Drupal\Core\Language\LanguageManagerInterface
+   * @param \Drupal\Core\Language\LanguageManagerInterface $language_manager
    *   The language manager.
    * @param \Egulias\EmailValidator\EmailValidator $email_validator
    *   The email validator.

core/modules/book/src/BookManager.php

@@ -138,7 +138,7 @@ public function getParentDepthLimit(array $book_link) {
   /**
    * Determine the relative depth of the children of a given book link.
    *
-   * @param array
+   * @param array $book_link
    *   The book link.
    *
    * @return int

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

@@ -124,7 +124,7 @@ class ConfigSync extends FormBase {
    *   The module installer.
    * @param \Drupal\Core\Extension\ThemeHandlerInterface $theme_handler
    *   The theme handler.
-   * @param \Drupal\Core\Render\RendererInterface
+   * @param \Drupal\Core\Render\RendererInterface $renderer
    *   The renderer.
    */
   public function __construct(StorageInterface $sync_storage, StorageInterface $active_storage, StorageInterface $snapshot_storage, LockBackendInterface $lock, EventDispatcherInterface $event_dispatcher, ConfigManagerInterface $config_manager, TypedConfigManagerInterface $typed_config, ModuleHandlerInterface $module_handler, ModuleInstallerInterface $module_installer, ThemeHandlerInterface $theme_handler, RendererInterface $renderer) {

core/modules/config_translation/src/ConfigNamesMapper.php

@@ -109,7 +109,7 @@ class ConfigNamesMapper extends PluginBase implements ConfigMapperInterface, Con
    *   The locale configuration manager.
    * @param \Drupal\config_translation\ConfigMapperManagerInterface $config_mapper_manager
    *   The mapper plugin discovery service.
-   * @param \Drupal\Core\Routing\RouteProviderInterface
+   * @param \Drupal\Core\Routing\RouteProviderInterface $route_provider
    *   The route provider.
    * @param \Drupal\Core\StringTranslation\TranslationInterface $string_translation
    *   The string translation manager.

core/modules/content_translation/src/Controller/ContentTranslationController.php

@@ -26,7 +26,7 @@ class ContentTranslationController extends ControllerBase {
   /**
    * Initializes a content translation controller.
    *
-   * @param \Drupal\content_translation\ContentTranslationManagerInterface
+   * @param \Drupal\content_translation\ContentTranslationManagerInterface $manager
    *   A content translation manager instance.
    */
   public function __construct(ContentTranslationManagerInterface $manager) {
@@ -319,7 +319,7 @@ public function overview(RouteMatchInterface $route_match, $entity_type_id = NUL
    * @param \Drupal\Core\Language\LanguageInterface $target
    *   The language of the translated values. Defaults to the current content
    *   language.
-   * @param \Drupal\Core\Routing\RouteMatchInterface
+   * @param \Drupal\Core\Routing\RouteMatchInterface $route_match
    *   The route match object from which to extract the entity type.
    * @param string $entity_type_id
    *   (optional) The entity type ID.
@@ -354,7 +354,7 @@ public function add(LanguageInterface $source, LanguageInterface $target, RouteM
    * @param \Drupal\Core\Language\LanguageInterface $language
    *   The language of the translated values. Defaults to the current content
    *   language.
-   * @param \Drupal\Core\Routing\RouteMatchInterface
+   * @param \Drupal\Core\Routing\RouteMatchInterface $route_match
    *   The route match object from which to extract the entity type.
    * @param string $entity_type_id
    *   (optional) The entity type ID.

core/modules/datetime/src/Plugin/views/filter/Date.php

@@ -58,7 +58,7 @@ class Date extends NumericDate implements ContainerFactoryPluginInterface {
    *   The plugin implementation definition.
    * @param \Drupal\Core\Datetime\DateFormatterInterface $date_formatter
    *   The date formatter service.
-   * @param \Symfony\Component\HttpFoundation\RequestStack
+   * @param \Symfony\Component\HttpFoundation\RequestStack $request_stack
    *   The request stack used to determine the current time.
    */
   public function __construct(array $configuration, $plugin_id, $plugin_definition, DateFormatterInterface $date_formatter, RequestStack $request_stack) {

core/modules/field/src/FieldStorageConfigStorage.php

@@ -64,7 +64,7 @@ class FieldStorageConfigStorage extends ConfigEntityStorage {
    *   The module handler.
    * @param \Drupal\Core\State\StateInterface $state
    *   The state key value store.
-   * @param \Drupal\Component\Plugin\PluginManagerInterface\FieldTypePluginManagerInterface
+   * @param \Drupal\Component\Plugin\PluginManagerInterface\FieldTypePluginManagerInterface $field_type_manager
    *   The field type plugin manager.
    */
   public function __construct(EntityTypeInterface $entity_type, ConfigFactoryInterface $config_factory, UuidInterface $uuid_service, LanguageManagerInterface $language_manager, EntityManagerInterface $entity_manager, ModuleHandlerInterface $module_handler, StateInterface $state, FieldTypePluginManagerInterface $field_type_manager) {

core/modules/hal/src/Normalizer/ContentEntityNormalizer.php

@@ -190,7 +190,7 @@ public function denormalize($data, $class, $format = NULL, array $context = arra
   /**
    * Constructs the entity URI.
    *
-   * @param \Drupal\Core\Entity\EntityInterface
+   * @param \Drupal\Core\Entity\EntityInterface $entity
    *   The entity.
    * @return string
    *   The entity URI.

core/modules/migrate_drupal/tests/src/Unit/source/DrupalSqlBaseTest.php

@@ -83,7 +83,7 @@ public function query() {
   /**
    * Tweaks DrupalSqlBase to set a new database connection for tests.
    *
-   * @param \Drupal\Core\Database\Connection
+   * @param \Drupal\Core\Database\Connection $database
    *   The new connection to use.
    *
    * @see \Drupal\Tests\migrate\Unit\MigrateSourceSqlTestCase

core/modules/migrate_drupal/tests/src/Unit/source/d6/Drupal6SqlBaseTest.php

@@ -170,7 +170,7 @@ public function query() {
   /**
    * Tweaks Drupal6SqlBase to set a new database connection for tests.
    *
-   * @param \Drupal\Core\Database\Connection
+   * @param \Drupal\Core\Database\Connection $database
    *   The new connection to use.
    *
    * @see \Drupal\Tests\migrate\Unit\MigrateSqlTestCase
@@ -182,7 +182,7 @@ public function setDatabase(Connection $database) {
   /**
    * Tweaks Drupal6SqlBase to set a new module handler for tests.
    *
-   * @param \Drupal\Core\Extension\ModuleHandlerInterface
+   * @param \Drupal\Core\Extension\ModuleHandlerInterface $module_handler
    *   The new module handler to use.
    *
    * @see \Drupal\Tests\migrate\Unit\MigrateSqlTestCase

core/modules/node/src/Controller/NodeController.php

@@ -285,7 +285,7 @@ public function addPageTitle(NodeTypeInterface $node_type) {
   /**
    * Gets a list of node revision IDs for a specific node.
    *
-   * @param \Drupal\node\NodeInterface
+   * @param \Drupal\node\NodeInterface $node
    *   The node entity.
    * @param \Drupal\node\NodeStorageInterface $node_storage
    *   The node storage handler.

core/modules/node/src/NodeStorageInterface.php

@@ -14,7 +14,7 @@ interface NodeStorageInterface extends ContentEntityStorageInterface {
   /**
    * Gets a list of node revision IDs for a specific node.
    *
-   * @param \Drupal\node\NodeInterface
+   * @param \Drupal\node\NodeInterface $node
    *   The node entity.
    *
    * @return int[]
@@ -36,7 +36,7 @@ public function userRevisionIds(AccountInterface $account);
   /**
    * Counts the number of revisions in the default language.
    *
-   * @param \Drupal\node\NodeInterface
+   * @param \Drupal\node\NodeInterface $node
    *   The node entity.
    *
    * @return int

core/modules/node/src/Plugin/migrate/source/d6/Node.php

@@ -261,7 +261,7 @@ public function getIds() {
   /**
    * Adapt our query for translations.
    *
-   * @param \Drupal\Core\Database\Query\SelectInterface
+   * @param \Drupal\Core\Database\Query\SelectInterface $query
    *   The generated query.
    */
   protected function handleTranslations(SelectInterface $query) {

core/modules/node/src/Plugin/views/argument/Vid.php

@@ -39,7 +39,7 @@ class Vid extends NumericArgument {
    *   The plugin implementation definition.
    * @param \Drupal\Core\Database\Connection $database
    *   Database Service Object.
-   * @param \Drupal\node\NodeStorageInterface
+   * @param \Drupal\node\NodeStorageInterface $node_storage
    *   The node storage.
    */
   public function __construct(array $configuration, $plugin_id, $plugin_definition, Connection $database, NodeStorageInterface $node_storage) {

core/modules/path/tests/src/Kernel/Migrate/d6/MigrateUrlAliasTest.php

@@ -29,7 +29,7 @@ protected function setUp() {
   /**
    * Assert a path.
    *
-   * @param string pid
+   * @param string $pid
    *   The path id.
    * @param array $conditions
    *   The path conditions.