diff --git a/core/lib/Drupal/Component/Plugin/Exception/InvalidPluginDefinitionException.php b/core/lib/Drupal/Component/Plugin/Exception/InvalidPluginDefinitionException.php index 6b205dbbd29d6cd777ba60202454c1565ce6b27d..684f71afe9714fc7e27bfa4cacdacfc4db0b8e1f 100644 --- a/core/lib/Drupal/Component/Plugin/Exception/InvalidPluginDefinitionException.php +++ b/core/lib/Drupal/Component/Plugin/Exception/InvalidPluginDefinitionException.php @@ -17,10 +17,14 @@ class InvalidPluginDefinitionException extends PluginException { /** * Constructs an InvalidPluginDefinitionException. * - * For the remaining parameters see \Exception. - * * @param string $plugin_id * The plugin ID of the mapper. + * @param string $message + * The exception message. + * @param int $code + * The exception code. + * @param \Exception|null $previous + * The previous throwable used for exception chaining. * * @see \Exception */ diff --git a/core/lib/Drupal/Component/Plugin/Exception/PluginNotFoundException.php b/core/lib/Drupal/Component/Plugin/Exception/PluginNotFoundException.php index 3c78fa93b3a27d5cac2291ee4b49fa9a917f5ee4..7c0da9fe942e52e13df05d8a106366be98a12149 100644 --- a/core/lib/Drupal/Component/Plugin/Exception/PluginNotFoundException.php +++ b/core/lib/Drupal/Component/Plugin/Exception/PluginNotFoundException.php @@ -10,10 +10,14 @@ class PluginNotFoundException extends PluginException { /** * Construct a PluginNotFoundException exception. * - * For the remaining parameters see \Exception. - * * @param string $plugin_id * The plugin ID that was not found. + * @param string $message + * The exception message. + * @param int $code + * The exception code. + * @param \Exception|null $previous + * The previous throwable used for exception chaining. * * @see \Exception */ diff --git a/core/lib/Drupal/Core/Asset/LibraryDiscoveryParser.php b/core/lib/Drupal/Core/Asset/LibraryDiscoveryParser.php index 8aff0c9375416a4be1547fdcf124ab2e6ec5bd61..983baec6167427eae8a0b3b21cd47936c82f4dbd 100644 --- a/core/lib/Drupal/Core/Asset/LibraryDiscoveryParser.php +++ b/core/lib/Drupal/Core/Asset/LibraryDiscoveryParser.php @@ -452,10 +452,12 @@ protected function isValidUri($string) { * The containing library definition. * @param array $sub_key * An array containing the sub-keys specifying the library asset, e.g. - * @code['js']@endcode or @code['css', 'component']@endcode + * ['js'] or ['css', 'component']. * @param array $overrides * Specifies the overrides, this is an array where the key is the asset to * be overridden while the value is overriding asset. + * @param string $theme_path + * The theme or base theme. */ protected function setOverrideValue(array &$library, array $sub_key, array $overrides, $theme_path) { foreach ($overrides as $original => $replacement) { diff --git a/core/lib/Drupal/Core/Config/UnmetDependenciesException.php b/core/lib/Drupal/Core/Config/UnmetDependenciesException.php index 573ec54849a02828ea7035ae5650fab5d252f0a0..8217aa5ade7dec58141100f2709e839320deae4f 100644 --- a/core/lib/Drupal/Core/Config/UnmetDependenciesException.php +++ b/core/lib/Drupal/Core/Config/UnmetDependenciesException.php @@ -64,6 +64,8 @@ public function getExtension() { * * @param \Drupal\Core\StringTranslation\TranslationInterface $string_translation * The string translation service. + * @param string $extension + * The name of the extension that is being installed. * * @return string */ diff --git a/core/lib/Drupal/Core/Database/Driver/pgsql/Schema.php b/core/lib/Drupal/Core/Database/Driver/pgsql/Schema.php index 1e7c4b6dca7dc9bd3acbea0830dfd2bd98d5a5a3..106add49efb010b52ee2ff83765b1c183ef3c2bf 100644 --- a/core/lib/Drupal/Core/Database/Driver/pgsql/Schema.php +++ b/core/lib/Drupal/Core/Database/Driver/pgsql/Schema.php @@ -103,13 +103,16 @@ protected function ensureIdentifiersLength($table_identifier_part, $column_ident * We introspect the database to collect the information required by insert * and update queries. * - * @param $table_name + * @param string $table * The non-prefixed name of the table. * - * @return + * @return mixed|object * An object with two member variables: - * - 'blob_fields' that lists all the blob fields in the table. - * - 'sequences' that lists the sequences used in that table. + * - 'blob_fields' that lists all the blob fields in the table. + * - 'sequences' that lists the sequences used in that table. + * + * @throws \Exception + * Exception thrown when the query for the table information fails. */ public function queryTableInformation($table) { // Generate a key to reference this table's information on. @@ -233,6 +236,9 @@ protected function resetTableInformation($table) { * * @return array * An array containing all the constraint names for the field. + * + * @throws \Exception + * Exception thrown when the query for the table information fails. */ public function queryFieldInformation($table, $field, $constraint_type = 'c') { assert(in_array($constraint_type, ['c', 'f', 'p', 'u', 't', 'x'])); @@ -267,12 +273,12 @@ public function queryFieldInformation($table, $field, $constraint_type = 'c') { /** * Generate SQL to create a new table from a Drupal schema definition. * - * @param $name + * @param string $name * The name of the table to create. - * @param $table + * @param array $table * A Schema API table definition array. * - * @return + * @return array * An array of SQL statements to create the table. */ protected function createTableSql($name, $table) { diff --git a/core/lib/Drupal/Core/Database/Schema.php b/core/lib/Drupal/Core/Database/Schema.php index fe932d3bce9b58b3e7efd43663c0c7f52abe5606..9083b2749298da084a615860d890b768c7876ad1 100644 --- a/core/lib/Drupal/Core/Database/Schema.php +++ b/core/lib/Drupal/Core/Database/Schema.php @@ -72,13 +72,13 @@ public function nextPlaceholder() { /** * Get information about the table name and schema from the prefix. * - * @param + * @param string $table * Name of table to look prefix up for. Defaults to 'default' because that's * default key for prefix. - * @param $add_prefix + * @param bool $add_prefix * Boolean that indicates whether the given table name should be prefixed. * - * @return + * @return array * A keyed array with information about the schema, table name and prefix. */ protected function getPrefixInfo($table = 'default', $add_prefix = TRUE) { @@ -238,9 +238,9 @@ public function findTables($table_expression) { /** * Check if a column exists in the given table. * - * @param $table + * @param string $table * The name of the table in drupal (no prefixing). - * @param $column + * @param string $column * The name of the column. * * @return diff --git a/core/lib/Drupal/Core/Entity/EntityFormInterface.php b/core/lib/Drupal/Core/Entity/EntityFormInterface.php index a6e8d2db39e5f12b920028ee814d19a6eacc328d..5c762daa5b2d75470fb2ca0f2332aedff2a79e49 100644 --- a/core/lib/Drupal/Core/Entity/EntityFormInterface.php +++ b/core/lib/Drupal/Core/Entity/EntityFormInterface.php @@ -77,8 +77,6 @@ public function getEntityFromRouteMatch(RouteMatchInterface $route_match, $entit * the submitted form values are copied to entity properties. The form's * entity remains unchanged. * - * @see \Drupal\Core\Entity\EntityFormInterface::getEntity() - * * @param array $form * A nested array form elements comprising the form. * @param \Drupal\Core\Form\FormStateInterface $form_state @@ -86,6 +84,8 @@ public function getEntityFromRouteMatch(RouteMatchInterface $route_match, $entit * * @return \Drupal\Core\Entity\EntityInterface * An updated copy of the form's entity object. + * + * @see \Drupal\Core\Entity\EntityFormInterface::getEntity() */ public function buildEntity(array $form, FormStateInterface $form_state); diff --git a/core/lib/Drupal/Core/Entity/Plugin/DataType/Deriver/EntityDeriver.php b/core/lib/Drupal/Core/Entity/Plugin/DataType/Deriver/EntityDeriver.php index 888a40562dc2d7ae2026e0c9d517410745725efb..ec7b5049828dd8ea1ee1f1a268bd6b904ec5eae0 100644 --- a/core/lib/Drupal/Core/Entity/Plugin/DataType/Deriver/EntityDeriver.php +++ b/core/lib/Drupal/Core/Entity/Plugin/DataType/Deriver/EntityDeriver.php @@ -50,6 +50,8 @@ class EntityDeriver implements ContainerDeriverInterface { * The base plugin ID. * @param \Drupal\Core\Entity\EntityTypeManagerInterface $entity_type_manager * The entity type manager. + * @param \Drupal\Core\Entity\EntityTypeBundleInfoInterface $bundle_info_service + * The bundle info service. */ public function __construct($base_plugin_id, EntityTypeManagerInterface $entity_type_manager, EntityTypeBundleInfoInterface $bundle_info_service) { $this->basePluginId = $base_plugin_id; diff --git a/core/lib/Drupal/Core/Entity/Query/ConditionAggregateInterface.php b/core/lib/Drupal/Core/Entity/Query/ConditionAggregateInterface.php index 43226805349e042ab8408db0cdfb0603a9f39876..0c49c2811a3f0ce772ac6710d990876c95ec7e65 100644 --- a/core/lib/Drupal/Core/Entity/Query/ConditionAggregateInterface.php +++ b/core/lib/Drupal/Core/Entity/Query/ConditionAggregateInterface.php @@ -34,10 +34,11 @@ public function condition($field, $function = NULL, $value = NULL, $operator = N /** * Queries for the existence of a field. * - * @param $field + * @param string $field + * @param string $function * @param string $langcode * - * @return ConditionInterface + * @return \Drupal\Core\Entity\Query\ConditionInterface * * @see \Drupal\Core\Entity\Query\QueryInterface::exists() */ @@ -47,8 +48,10 @@ public function exists($field, $function, $langcode = NULL); * Queries for the nonexistence of a field. * * @param string $field + * @param string $function + * @param string $langcode * - * @return ConditionInterface + * @return \Drupal\Core\Entity\Query\ConditionInterface * * @see \Drupal\Core\Entity\Query\QueryInterface::notExists() */ diff --git a/core/lib/Drupal/Core/Entity/Query/ConditionInterface.php b/core/lib/Drupal/Core/Entity/Query/ConditionInterface.php index 7b5f513f0fded7d29f5cd368df912a60626b0b54..d0817fc7935440631daa471fe3f30e4b518ab0da 100644 --- a/core/lib/Drupal/Core/Entity/Query/ConditionInterface.php +++ b/core/lib/Drupal/Core/Entity/Query/ConditionInterface.php @@ -54,6 +54,8 @@ public function exists($field, $langcode = NULL); * * @param string $field * @param string $langcode + * (optional) For which language the entity should be prepared, defaults to + * the current content language. * * @return $this * diff --git a/core/lib/Drupal/Core/Entity/Query/Sql/Tables.php b/core/lib/Drupal/Core/Entity/Query/Sql/Tables.php index 4fe64dbc62bddaf5994e4aad1e23a7508703b8c0..cb535d8c5c8c77d7ab8b3422dcbfc76e2e29a990 100644 --- a/core/lib/Drupal/Core/Entity/Query/Sql/Tables.php +++ b/core/lib/Drupal/Core/Entity/Query/Sql/Tables.php @@ -369,14 +369,32 @@ protected function ensureEntityTable($index_prefix, $property, $type, $langcode, } /** - * Join field table if necessary. + * Ensure the field table is joined if necessary. * - * @param $field_name - * Name of the field. + * @param string $index_prefix + * The table array index prefix. For a base table this will be empty, + * for a target entity reference like 'field_tags.entity:taxonomy_term.name' + * this will be 'entity:taxonomy_term.target_id.'. + * @param \Drupal\Core\Field\FieldStorageDefinitionInterface &$field + * The field storage definition for the field being joined. + * @param string $type + * The join type. + * @param string $langcode + * The langcode we use on the join. + * @param string $base_table + * The table to join to. It can be either the table name, its alias or the + * 'base_table' placeholder. + * @param string $entity_id_field + * The name of the ID field/property for the current entity. For instance: + * tid, nid, etc. + * @param string $field_id_field + * The column representing the id for the field. For example, 'revision_id' + * or 'entity_id'. + * @param string $delta + * A delta which should be used as additional condition. * * @return string - * - * @throws \Drupal\Core\Entity\Query\QueryException + * The alias of the joined table. */ protected function ensureFieldTable($index_prefix, &$field, $type, $langcode, $base_table, $entity_id_field, $field_id_field, $delta) { $field_name = $field->getName(); @@ -435,6 +453,7 @@ protected function addJoin($type, $table, $join_condition, $langcode, $delta = N * * @param string $table * The table name. + * @param string $entity_type_id * * @return array|false * An associative array of table field mapping for the given table, keyed by diff --git a/core/lib/Drupal/Core/Entity/TypedData/EntityDataDefinition.php b/core/lib/Drupal/Core/Entity/TypedData/EntityDataDefinition.php index 74463ee2c3c7f84ac7dae440c998855f8dbc137e..3e362317b574974fa32ab8a143ace00b1bf85818 100644 --- a/core/lib/Drupal/Core/Entity/TypedData/EntityDataDefinition.php +++ b/core/lib/Drupal/Core/Entity/TypedData/EntityDataDefinition.php @@ -15,6 +15,9 @@ class EntityDataDefinition extends ComplexDataDefinitionBase implements EntityDa * @param string $entity_type_id * (optional) The ID of the entity type, or NULL if the entity type is * unknown. Defaults to NULL. + * @param string $bundle + * (optional) The bundle of the entity type, or NULL if the bundle is + * unknown. Defaults to NULL. * * @return static */ diff --git a/core/lib/Drupal/Core/EventSubscriber/ConfigSnapshotSubscriber.php b/core/lib/Drupal/Core/EventSubscriber/ConfigSnapshotSubscriber.php index 2d78d20ec3464708f7427ba9b59bc4a7eee4b7eb..20e0cd6773f012fa9599bfcf1adbbfa1c1a4ecfb 100644 --- a/core/lib/Drupal/Core/EventSubscriber/ConfigSnapshotSubscriber.php +++ b/core/lib/Drupal/Core/EventSubscriber/ConfigSnapshotSubscriber.php @@ -37,6 +37,8 @@ class ConfigSnapshotSubscriber implements EventSubscriberInterface { /** * Constructs the ConfigSnapshotSubscriber object. * + * @param \Drupal\Core\Config\ConfigManagerInterface $config_manager + * The configuration manager. * @param \Drupal\Core\Config\StorageInterface $source_storage * The source storage used to discover configuration changes. * @param \Drupal\Core\Config\StorageInterface $snapshot_storage diff --git a/core/lib/Drupal/Core/Installer/Form/SiteSettingsForm.php b/core/lib/Drupal/Core/Installer/Form/SiteSettingsForm.php index c37b5b714232f42c8de167d1559a153ecd0a5723..d98c4820fce37f427ed407d061a90f824744b5d2 100644 --- a/core/lib/Drupal/Core/Installer/Form/SiteSettingsForm.php +++ b/core/lib/Drupal/Core/Installer/Form/SiteSettingsForm.php @@ -37,6 +37,8 @@ class SiteSettingsForm extends FormBase { * * @param string $site_path * The site path. + * @param \Drupal\Core\Render\RendererInterface $renderer + * The renderer. */ public function __construct($site_path, RendererInterface $renderer) { $this->sitePath = $site_path; diff --git a/core/lib/Drupal/Core/Site/Settings.php b/core/lib/Drupal/Core/Site/Settings.php index bcc0e4eba402663bc2ac541b33839133ad29f63e..dac9714d1f9293a8a43344ff39109bf355893b30 100644 --- a/core/lib/Drupal/Core/Site/Settings.php +++ b/core/lib/Drupal/Core/Site/Settings.php @@ -212,9 +212,13 @@ public static function getHashSalt() { * module directories setting apcu_ensure_unique_prefix would allow the sites * to share APCu cache items. * - * @param $identifier + * @param string $identifier * An identifier for the prefix. For example, 'class_loader' or * 'cache_backend'. + * @param string $root + * The app root. + * @param string $site_path + * (optional) The site path. Defaults to an empty string. * * @return string * The prefix for APCu user cache keys. diff --git a/core/lib/Drupal/Core/TypedData/Plugin/DataType/ItemList.php b/core/lib/Drupal/Core/TypedData/Plugin/DataType/ItemList.php index 4c83afc630470e7b606b72d4218713765ffd38ef..733a4e86d2feb48713355b54cbc64fe3a66264f8 100644 --- a/core/lib/Drupal/Core/TypedData/Plugin/DataType/ItemList.php +++ b/core/lib/Drupal/Core/TypedData/Plugin/DataType/ItemList.php @@ -48,6 +48,9 @@ public function getValue() { * * @param array|null $values * An array of values of the field items, or NULL to unset the field. + * @param bool $notify + * (optional) Whether to notify the parent object of the change. Defaults to + * TRUE. */ public function setValue($values, $notify = TRUE) { if (!isset($values) || $values === []) { diff --git a/core/lib/Drupal/Core/TypedData/Plugin/DataType/Map.php b/core/lib/Drupal/Core/TypedData/Plugin/DataType/Map.php index aee25bc00db132c7930a72bb8897d480114604bd..cadebd7d2a1dcd85a4ee36515c8c1d6576078046 100644 --- a/core/lib/Drupal/Core/TypedData/Plugin/DataType/Map.php +++ b/core/lib/Drupal/Core/TypedData/Plugin/DataType/Map.php @@ -70,6 +70,10 @@ public function getValue() { * * @param array|null $values * An array of property values. + * @param bool $notify + * (optional) Whether to notify the parent object of the change. Defaults to + * TRUE. If a property is updated from a parent object, set it to FALSE to + * avoid being notified again. */ public function setValue($values, $notify = TRUE) { if (isset($values) && !is_array($values)) { @@ -214,6 +218,8 @@ public function __clone() { /** * {@inheritdoc} * + * @param $property_name + * The name of the property. * @param bool $notify * (optional) Whether to forward the notification to the parent. Defaults to * TRUE. By passing FALSE, overrides of this method can re-use the logic diff --git a/core/modules/action/src/Form/ActionAddForm.php b/core/modules/action/src/Form/ActionAddForm.php index 54a2c4c037685d71dc77b2fb24d04e3c029aea38..23dcb558bc80856c55736906b2981e41beb5fe2f 100644 --- a/core/modules/action/src/Form/ActionAddForm.php +++ b/core/modules/action/src/Form/ActionAddForm.php @@ -14,6 +14,10 @@ class ActionAddForm extends ActionFormBase { /** * {@inheritdoc} * + * @param array $form + * An associative array containing the structure of the form. + * @param \Drupal\Core\Form\FormStateInterface $form_state + * The current state of the form. * @param string $action_id * The action ID. */ diff --git a/core/modules/ban/src/Form/BanAdmin.php b/core/modules/ban/src/Form/BanAdmin.php index d71be8d40259d3910744b2ba30ca474164071ae2..9a55d02dbe27016a41048daeb1e6ed608c5138f6 100644 --- a/core/modules/ban/src/Form/BanAdmin.php +++ b/core/modules/ban/src/Form/BanAdmin.php @@ -48,6 +48,10 @@ public function getFormId() { /** * {@inheritdoc} * + * @param array $form + * A nested array form elements comprising the form. + * @param \Drupal\Core\Form\FormStateInterface $form_state + * The current state of the form. * @param string $default_ip * (optional) IP address to be passed on to * \Drupal::formBuilder()->getForm() for use as the default value of the IP diff --git a/core/modules/ban/src/Form/BanDelete.php b/core/modules/ban/src/Form/BanDelete.php index 9a3ddd48cfd17446bf0734f51316a33d6580247c..e016e8dd7ff8203bf104607b96cfa0a73bbfa160 100644 --- a/core/modules/ban/src/Form/BanDelete.php +++ b/core/modules/ban/src/Form/BanDelete.php @@ -80,6 +80,10 @@ public function getCancelUrl() { /** * {@inheritdoc} * + * @param array $form + * A nested array form elements comprising the form. + * @param \Drupal\Core\Form\FormStateInterface $form_state + * The current state of the form. * @param string $ban_id * The IP address record ID to unban. */ diff --git a/core/modules/block/src/BlockListBuilder.php b/core/modules/block/src/BlockListBuilder.php index eb38b44ec036451aba70432ec3fc8a585ebd4e6b..aa05ffc07b132cc1eddd0e817f0e84ae478d4b39 100644 --- a/core/modules/block/src/BlockListBuilder.php +++ b/core/modules/block/src/BlockListBuilder.php @@ -70,6 +70,8 @@ class BlockListBuilder extends ConfigEntityListBuilder implements FormInterface * The theme manager. * @param \Drupal\Core\Form\FormBuilderInterface $form_builder * The form builder. + * @param \Drupal\Core\Messenger\MessengerInterface $messenger + * The messenger service. */ public function __construct(EntityTypeInterface $entity_type, EntityStorageInterface $storage, ThemeManagerInterface $theme_manager, FormBuilderInterface $form_builder, MessengerInterface $messenger) { parent::__construct($entity_type, $storage); diff --git a/core/modules/comment/src/CommentForm.php b/core/modules/comment/src/CommentForm.php index 48c72712df6ce432423dd08f312aa5e9da64cee1..0f9c7dbd6c860e25e490da66e5b3ebac33c6262d 100644 --- a/core/modules/comment/src/CommentForm.php +++ b/core/modules/comment/src/CommentForm.php @@ -74,6 +74,8 @@ public static function create(ContainerInterface $container) { * The entity type bundle service. * @param \Drupal\Component\Datetime\TimeInterface $time * The time service. + * @param \Drupal\Core\Entity\EntityFieldManagerInterface $entity_field_manager + * The entity field manager service. */ public function __construct(EntityRepositoryInterface $entity_repository, AccountInterface $current_user, RendererInterface $renderer, EntityTypeBundleInfoInterface $entity_type_bundle_info = NULL, TimeInterface $time = NULL, EntityFieldManagerInterface $entity_field_manager = NULL) { parent::__construct($entity_repository, $entity_type_bundle_info, $time); diff --git a/core/modules/content_moderation/src/EntityTypeInfo.php b/core/modules/content_moderation/src/EntityTypeInfo.php index f073e25a389eaada15928c9fd4933aaf9983d7e0..56ba9094c46192b256b8d7c323993085b40f3ee7 100644 --- a/core/modules/content_moderation/src/EntityTypeInfo.php +++ b/core/modules/content_moderation/src/EntityTypeInfo.php @@ -95,6 +95,8 @@ class EntityTypeInfo implements ContainerInjectionInterface { * Bundle information service. * @param \Drupal\Core\Session\AccountInterface $current_user * Current user. + * @param \Drupal\content_moderation\StateTransitionValidationInterface $validator + * State transition validator. */ public function __construct(TranslationInterface $translation, ModerationInformationInterface $moderation_information, EntityTypeManagerInterface $entity_type_manager, EntityTypeBundleInfoInterface $bundle_info, AccountInterface $current_user, StateTransitionValidationInterface $validator) { $this->stringTranslation = $translation; diff --git a/core/modules/content_moderation/src/Plugin/WorkflowType/ContentModeration.php b/core/modules/content_moderation/src/Plugin/WorkflowType/ContentModeration.php index 5182c6b3fd9ae0f3e6d40209307f0aff1a686475..b585961d2dfdb1c726c39fe78629abccbf67ca20 100644 --- a/core/modules/content_moderation/src/Plugin/WorkflowType/ContentModeration.php +++ b/core/modules/content_moderation/src/Plugin/WorkflowType/ContentModeration.php @@ -67,6 +67,8 @@ class ContentModeration extends WorkflowTypeBase implements ContentModerationInt * The plugin implementation definition. * @param \Drupal\Core\Entity\EntityTypeManagerInterface $entity_type_manager * The entity type manager. + * @param \Drupal\Core\Entity\EntityTypeBundleInfoInterface $entity_type_bundle_info + * The entity type bundle info. * @param \Drupal\content_moderation\ModerationInformationInterface $moderation_info * Moderation information service. */ diff --git a/core/modules/content_translation/src/ContentTranslationHandler.php b/core/modules/content_translation/src/ContentTranslationHandler.php index 12a579ec0429ffd56c99116129acbeb089d55802..ff1938f2640d8468994d04c4b8b6b36c42cd6069 100644 --- a/core/modules/content_translation/src/ContentTranslationHandler.php +++ b/core/modules/content_translation/src/ContentTranslationHandler.php @@ -116,6 +116,8 @@ class ContentTranslationHandler implements ContentTranslationHandlerInterface, E * The messenger service. * @param \Drupal\Core\Datetime\DateFormatterInterface $date_formatter * The date formatter service. + * @param \Drupal\Core\Entity\EntityLastInstalledSchemaRepositoryInterface $entity_last_installed_schema_repository + * The installed entity definition repository service. */ public function __construct(EntityTypeInterface $entity_type, LanguageManagerInterface $language_manager, ContentTranslationManagerInterface $manager, EntityTypeManagerInterface $entity_type_manager, AccountInterface $current_user, MessengerInterface $messenger, DateFormatterInterface $date_formatter, EntityLastInstalledSchemaRepositoryInterface $entity_last_installed_schema_repository) { $this->entityTypeId = $entity_type->id(); @@ -667,6 +669,10 @@ protected function addTranslatabilityClue(&$element) { * The type of the entity. * @param \Drupal\Core\Entity\EntityInterface $entity * The entity whose form is being built. + * @param array $form + * A nested array form elements comprising the form. + * @param \Drupal\Core\Form\FormStateInterface $form_state + * The current state of the form. * * @see \Drupal\content_translation\ContentTranslationHandler::entityFormAlter() */ diff --git a/core/modules/content_translation/tests/src/Kernel/ContentTranslationHandlerTest.php b/core/modules/content_translation/tests/src/Kernel/ContentTranslationHandlerTest.php index 80c4ed19cf5edf932b3bb851c250809b65b6ac4a..22c411127fdeb7643d624a97bb2d4171aa285b91 100644 --- a/core/modules/content_translation/tests/src/Kernel/ContentTranslationHandlerTest.php +++ b/core/modules/content_translation/tests/src/Kernel/ContentTranslationHandlerTest.php @@ -93,8 +93,14 @@ protected function setUp(): void { * Whether or not the entity is the default translation. * @param bool $translation_form * Whether or not the form is a translation form. + * @param bool $is_submitted + * Whether or not the form should be marked as submitted. + * @param bool $is_rebuilding + * Whether or not the form should be flagged for rebuild. * @param array $expected * The expected altered element. + * @param bool $display_warning + * Whether or not the warning message should be displayed. * * @dataProvider providerTestEntityFormSharedElements * diff --git a/core/modules/editor/src/Form/EditorImageDialog.php b/core/modules/editor/src/Form/EditorImageDialog.php index 489011dfc5b698cd84d0035cffb29593f4b8edee..6308aea773139cff16bc55ea13997c1bb9e22d8f 100644 --- a/core/modules/editor/src/Form/EditorImageDialog.php +++ b/core/modules/editor/src/Form/EditorImageDialog.php @@ -57,6 +57,10 @@ public function getFormId() { /** * {@inheritdoc} * + * @param array $form + * An associative array containing the structure of the form. + * @param \Drupal\Core\Form\FormStateInterface $form_state + * The current state of the form. * @param \Drupal\editor\Entity\Editor $editor * The text editor to which this dialog corresponds. */ diff --git a/core/modules/editor/src/Form/EditorLinkDialog.php b/core/modules/editor/src/Form/EditorLinkDialog.php index 5f34e1078695241bba24c50e9a390a146970f177..1b1735a2e7816098076a08bf0d5103afab992ca0 100644 --- a/core/modules/editor/src/Form/EditorLinkDialog.php +++ b/core/modules/editor/src/Form/EditorLinkDialog.php @@ -27,8 +27,14 @@ public function getFormId() { /** * {@inheritdoc} * + * @param array $form + * An associative array containing the structure of the form. + * @param \Drupal\Core\Form\FormStateInterface $form_state + * The current state of the form. * @param \Drupal\editor\Entity\Editor $editor * The text editor to which this dialog corresponds. + * + * @return array */ public function buildForm(array $form, FormStateInterface $form_state, Editor $editor = NULL) { // The default values are set directly from \Drupal::request()->request, diff --git a/core/modules/field/src/Entity/FieldConfig.php b/core/modules/field/src/Entity/FieldConfig.php index 909c5b882e8039e482eae9d74b0f5dffb62055d1..9df0ce9a3219d6a38e4af8734504d5c498eba01a 100644 --- a/core/modules/field/src/Entity/FieldConfig.php +++ b/core/modules/field/src/Entity/FieldConfig.php @@ -90,6 +90,9 @@ class FieldConfig extends FieldConfigBase implements FieldConfigInterface { * bundle to which the field is attached to. Other array elements will be * used to set the corresponding properties on the class; see the class * property documentation for details. + * @param string $entity_type + * (optional) The entity type on which the field should be created. + * Defaults to "field_config". */ public function __construct(array $values, $entity_type = 'field_config') { // Allow either an injected FieldStorageConfig object, or a field_name and diff --git a/core/modules/field/src/Entity/FieldStorageConfig.php b/core/modules/field/src/Entity/FieldStorageConfig.php index 7d2fd51f1aef1f27c82c4210f89fc8243a823b7a..c05caf084c2bb7d4a179102cdc49f49a7d5d9eb0 100644 --- a/core/modules/field/src/Entity/FieldStorageConfig.php +++ b/core/modules/field/src/Entity/FieldStorageConfig.php @@ -245,6 +245,9 @@ class FieldStorageConfig extends ConfigEntityBase implements FieldStorageConfigI * a 'field_name' property can be accepted in place of 'id'. * - entity_type: required. * - type: required. + * @param string $entity_type + * (optional) The entity type on which the field should be created. + * Defaults to "field_storage_config". */ public function __construct(array $values, $entity_type = 'field_storage_config') { // Check required properties. diff --git a/core/modules/field_ui/src/FieldStorageConfigListBuilder.php b/core/modules/field_ui/src/FieldStorageConfigListBuilder.php index 600d41809d9508e74a2ee217f0a03629075ac912..03fd5b6a8b1e466c152c18df8ee042190ac6c203 100644 --- a/core/modules/field_ui/src/FieldStorageConfigListBuilder.php +++ b/core/modules/field_ui/src/FieldStorageConfigListBuilder.php @@ -56,6 +56,8 @@ class FieldStorageConfigListBuilder extends ConfigEntityListBuilder { * The entity type manager. * @param \Drupal\Core\Field\FieldTypePluginManagerInterface $field_type_manager * The 'field type' plugin manager. + * @param \Drupal\Core\Entity\EntityTypeBundleInfoInterface $bundle_info_service + * The bundle info service. */ public function __construct(EntityTypeInterface $entity_type, EntityTypeManagerInterface $entity_type_manager, FieldTypePluginManagerInterface $field_type_manager, EntityTypeBundleInfoInterface $bundle_info_service) { parent::__construct($entity_type, $entity_type_manager->getStorage($entity_type->id())); diff --git a/core/modules/field_ui/src/Form/FieldStorageConfigEditForm.php b/core/modules/field_ui/src/Form/FieldStorageConfigEditForm.php index e50ddd0302257157b9a815a860013527d5eac0a8..bf1994c1f93cd1f2f6ac84e8668152702d57c41f 100644 --- a/core/modules/field_ui/src/Form/FieldStorageConfigEditForm.php +++ b/core/modules/field_ui/src/Form/FieldStorageConfigEditForm.php @@ -41,6 +41,10 @@ public function getEntityFromRouteMatch(RouteMatchInterface $route_match, $entit /** * {@inheritdoc} * + * @param array $form + * A nested array form elements comprising the form. + * @param \Drupal\Core\Form\FormStateInterface $form_state + * The current state of the form. * @param string $field_config * The ID of the field config whose field storage config is being edited. */ diff --git a/core/modules/image/src/Form/ImageEffectFormBase.php b/core/modules/image/src/Form/ImageEffectFormBase.php index ccf03c2253bd64491cc8a4e7d01c17bc8387e7e2..69ce18ad78fc2aa77294f2edd143f6bf1e85e823 100644 --- a/core/modules/image/src/Form/ImageEffectFormBase.php +++ b/core/modules/image/src/Form/ImageEffectFormBase.php @@ -39,6 +39,10 @@ public function getFormId() { /** * {@inheritdoc} * + * @param array $form + * A nested array form elements comprising the form. + * @param \Drupal\Core\Form\FormStateInterface $form_state + * The current state of the form. * @param \Drupal\image\ImageStyleInterface $image_style * The image style. * @param string $image_effect diff --git a/core/modules/image/tests/src/Unit/ImageStyleTest.php b/core/modules/image/tests/src/Unit/ImageStyleTest.php index 2256cdf0a429cb144f0c397d4ce4498c74f5e107..e38d9152c13f29c8228db2e765e8c1c44d3d7bec 100644 --- a/core/modules/image/tests/src/Unit/ImageStyleTest.php +++ b/core/modules/image/tests/src/Unit/ImageStyleTest.php @@ -40,6 +40,8 @@ class ImageStyleTest extends UnitTestCase { * The image effect ID. * @param \Drupal\image\ImageEffectInterface|\PHPUnit\Framework\MockObject\MockObject $image_effect * The image effect used for testing. + * @param array $stubs + * An array of additional method names to mock. * * @return \Drupal\image\ImageStyleInterface * The mocked image style. diff --git a/core/modules/jsonapi/src/Controller/TemporaryJsonapiFileFieldUploader.php b/core/modules/jsonapi/src/Controller/TemporaryJsonapiFileFieldUploader.php index 4751d9caaca325bf3d3165c33823a2708c72ae81..39ef17f38704e9230cf37bf8fc57e70f9c1ebc65 100644 --- a/core/modules/jsonapi/src/Controller/TemporaryJsonapiFileFieldUploader.php +++ b/core/modules/jsonapi/src/Controller/TemporaryJsonapiFileFieldUploader.php @@ -122,6 +122,8 @@ class TemporaryJsonapiFileFieldUploader { * The lock service. * @param \Drupal\Core\Config\ConfigFactoryInterface $config_factory * The config factory. + * @param \Symfony\Contracts\EventDispatcher\EventDispatcherInterface $event_dispatcher + * (optional) The event dispatcher. */ public function __construct(LoggerInterface $logger, FileSystemInterface $file_system, $mime_type_guesser, Token $token, LockBackendInterface $lock, ConfigFactoryInterface $config_factory, EventDispatcherInterface $event_dispatcher = NULL) { $this->logger = $logger; diff --git a/core/modules/language/src/Entity/ContentLanguageSettings.php b/core/modules/language/src/Entity/ContentLanguageSettings.php index 509dbaf3c7110f56d50bdeb9f9300001ebf59445..d3751a0eaa6fca404a3fb64513eea38de3eb07a4 100644 --- a/core/modules/language/src/Entity/ContentLanguageSettings.php +++ b/core/modules/language/src/Entity/ContentLanguageSettings.php @@ -86,6 +86,9 @@ class ContentLanguageSettings extends ConfigEntityBase implements ContentLanguag * - target_bundle: The bundle. * Other array elements will be used to set the corresponding properties on * the class; see the class property documentation for details. + * @param string $entity_type + * (optional) The entity type ID that is being created. Defaults to + * "language_content_settings". */ public function __construct(array $values, $entity_type = 'language_content_settings') { if (empty($values['target_entity_type_id'])) { diff --git a/core/modules/language/src/LanguageNegotiator.php b/core/modules/language/src/LanguageNegotiator.php index 4791102493552e0ff959e69440a8011d5e7c47c4..40a0d0296d7498eabca92c119f6f27d81c652847 100644 --- a/core/modules/language/src/LanguageNegotiator.php +++ b/core/modules/language/src/LanguageNegotiator.php @@ -81,6 +81,8 @@ class LanguageNegotiator implements LanguageNegotiatorInterface { * The configuration factory. * @param \Drupal\Core\Site\Settings $settings * The settings instance. + * @param \Symfony\Component\HttpFoundation\RequestStack $requestStack + * The request stack service. */ public function __construct(ConfigurableLanguageManagerInterface $language_manager, PluginManagerInterface $negotiator_manager, ConfigFactoryInterface $config_factory, Settings $settings, RequestStack $requestStack) { $this->languageManager = $language_manager; diff --git a/core/modules/language/tests/language_test/src/Controller/LanguageTestController.php b/core/modules/language/tests/language_test/src/Controller/LanguageTestController.php index 2737f31cbf42bea1ef1cf66acab02cafd00a2477..9166dae384464c89eec3d3c08394b2c050120e1c 100644 --- a/core/modules/language/tests/language_test/src/Controller/LanguageTestController.php +++ b/core/modules/language/tests/language_test/src/Controller/LanguageTestController.php @@ -37,6 +37,8 @@ class LanguageTestController implements ContainerInjectionInterface { * * @param \Symfony\Component\HttpKernel\HttpKernelInterface $httpKernel * An HTTP kernel. + * @param \Drupal\Core\Language\LanguageManagerInterface $language_manager + * The language manager. */ public function __construct(HttpKernelInterface $httpKernel, LanguageManagerInterface $language_manager) { $this->httpKernel = $httpKernel; diff --git a/core/modules/locale/src/LocaleDefaultConfigStorage.php b/core/modules/locale/src/LocaleDefaultConfigStorage.php index fb4e352069e9ec4f5b027d8e1802fd253b9867c6..d68d71804106397452457486b357324741bde217 100644 --- a/core/modules/locale/src/LocaleDefaultConfigStorage.php +++ b/core/modules/locale/src/LocaleDefaultConfigStorage.php @@ -56,6 +56,8 @@ class LocaleDefaultConfigStorage { * The storage object to use for reading configuration data. * @param \Drupal\language\ConfigurableLanguageManagerInterface $language_manager * The language manager. + * @param string $install_profile + * The current installation profile. */ public function __construct(StorageInterface $config_storage, ConfigurableLanguageManagerInterface $language_manager, $install_profile) { $this->configStorage = $config_storage; diff --git a/core/modules/media/src/Form/EditorMediaDialog.php b/core/modules/media/src/Form/EditorMediaDialog.php index fe7e07d5d43a857146b83adc4b39235f537c3c67..1b52044dbd70768b74b62e2c838c961b5f555454 100644 --- a/core/modules/media/src/Form/EditorMediaDialog.php +++ b/core/modules/media/src/Form/EditorMediaDialog.php @@ -77,6 +77,10 @@ public function getFormId() { /** * {@inheritdoc} * + * @param array $form + * A nested array form elements comprising the form. + * @param \Drupal\Core\Form\FormStateInterface $form_state + * The current state of the form. * @param \Drupal\editor\EditorInterface $editor * The text editor to which this dialog corresponds. */ diff --git a/core/modules/media/tests/src/Functional/MediaUiFunctionalTest.php b/core/modules/media/tests/src/Functional/MediaUiFunctionalTest.php index 34c342a024d50e295a459a3f3dd1dd45487a8e21..6155a1acbde95667a0960133a4eb910635bf5e1a 100644 --- a/core/modules/media/tests/src/Functional/MediaUiFunctionalTest.php +++ b/core/modules/media/tests/src/Functional/MediaUiFunctionalTest.php @@ -247,6 +247,8 @@ public function providerTestMediaReferenceWidget() { * one that allows the user to create media and a second that does not. * @param bool $list_access * Whether to grant the test user access to list media. + * @param string $widget_id + * The widget ID to test. * * @see media_field_widget_entity_reference_autocomplete_form_alter() * @see media_field_widget_multiple_entity_reference_autocomplete_form_alter() diff --git a/core/modules/menu_ui/src/Form/MenuLinkEditForm.php b/core/modules/menu_ui/src/Form/MenuLinkEditForm.php index 1afafd64449b01f912a8b2ec4e49a1ab2f6c0600..fafb293dbed716e7144283742a087788604045de 100644 --- a/core/modules/menu_ui/src/Form/MenuLinkEditForm.php +++ b/core/modules/menu_ui/src/Form/MenuLinkEditForm.php @@ -55,6 +55,10 @@ public function getFormId() { /** * {@inheritdoc} * + * @param array $form + * An associative array containing the structure of the form. + * @param \Drupal\Core\Form\FormStateInterface $form_state + * The current state of the form. * @param \Drupal\Core\Menu\MenuLinkInterface $menu_link_plugin * The plugin instance to use for this form. */ diff --git a/core/modules/migrate/src/Plugin/Exception/BadPluginDefinitionException.php b/core/modules/migrate/src/Plugin/Exception/BadPluginDefinitionException.php index 5387f97117eac6a9baa44c67b8975ef2e714e484..081d4c01d536825fbca5bd010e733dc534b56271 100644 --- a/core/modules/migrate/src/Plugin/Exception/BadPluginDefinitionException.php +++ b/core/modules/migrate/src/Plugin/Exception/BadPluginDefinitionException.php @@ -12,12 +12,14 @@ class BadPluginDefinitionException extends InvalidPluginDefinitionException { /** * Constructs a BadPluginDefinitionException. * - * For the remaining parameters see \Exception. - * * @param string $plugin_id * The plugin ID of the mapper. * @param string $property * The name of the property that is missing from the plugin. + * @param int $code + * (optional) The exception code. Defaults to 0. + * @param \Exception|null $previous + * The previous throwable used for exception chaining. * * @see \Exception */ diff --git a/core/modules/migrate/tests/src/Unit/MigrateSourceTest.php b/core/modules/migrate/tests/src/Unit/MigrateSourceTest.php index b923d1fea6bdd9cdcd9cbcaee9668ae45e8f3645..8212c7a45222ee041baf4016783a78916cf3ab36 100644 --- a/core/modules/migrate/tests/src/Unit/MigrateSourceTest.php +++ b/core/modules/migrate/tests/src/Unit/MigrateSourceTest.php @@ -74,6 +74,8 @@ class MigrateSourceTest extends MigrateTestCase { * @param int $status * (optional) The default status for the new rows to be imported. Defaults * to MigrateIdMapInterface::STATUS_NEEDS_UPDATE. + * @param int $high_water_value + * (optional) The high water mark to start from, if set. * * @return \Drupal\migrate\Plugin\MigrateSourceInterface * A mocked source plugin. diff --git a/core/modules/node/tests/src/Kernel/Migrate/d7/MigrateNodeTypeTest.php b/core/modules/node/tests/src/Kernel/Migrate/d7/MigrateNodeTypeTest.php index 4d2f61feada4a94b7f8b3cfd9a7367481327a4be..8a3c8a4a6e3786e55f0b61a7910226400a17fbee 100644 --- a/core/modules/node/tests/src/Kernel/Migrate/d7/MigrateNodeTypeTest.php +++ b/core/modules/node/tests/src/Kernel/Migrate/d7/MigrateNodeTypeTest.php @@ -43,6 +43,11 @@ protected function setUp(): void { * The expected node type description. * @param string $help * The expected help text. + * @param bool $display_submitted + * @param bool $new_revision + * @param array $expected_available_menus + * @param string $expected_parent + * @param string|null $body_label */ protected function assertEntity($id, $label, $description, $help, $display_submitted, $new_revision, $expected_available_menus, $expected_parent, $body_label = NULL) { /** @var \Drupal\node\NodeTypeInterface $entity */ diff --git a/core/modules/quickedit/tests/src/FunctionalJavascript/QuickEditJavascriptTestBase.php b/core/modules/quickedit/tests/src/FunctionalJavascript/QuickEditJavascriptTestBase.php index 22ddc63ddaacbb9febd9be19b8fd2821e0d0fa98..de77528560aeba7d6fb35beed0052177a7ee88d2 100644 --- a/core/modules/quickedit/tests/src/FunctionalJavascript/QuickEditJavascriptTestBase.php +++ b/core/modules/quickedit/tests/src/FunctionalJavascript/QuickEditJavascriptTestBase.php @@ -156,7 +156,10 @@ protected function awaitEntityInstanceFieldState($entity_type_id, $entity_id, $e * Asserts the state of the Quick Edit entity toolbar. * * @param string $expected_entity_label - * The expected label in the Quick Edit Entity Toolbar. + * The expected entity label in the Quick Edit Entity Toolbar. + * @param string|null $expected_field_label + * The expected field label in the Quick Edit Entity Toolbar, or NULL + * if no field label is expected. */ protected function assertQuickEditEntityToolbar($expected_entity_label, $expected_field_label) { $quickedit_entity_toolbar = $this->getSession()->getPage()->findById('quickedit-entity-toolbar'); diff --git a/core/modules/rdf/tests/src/Functional/CommentAttributesTest.php b/core/modules/rdf/tests/src/Functional/CommentAttributesTest.php index 67091357e84b52a8ce3a0b0d15019e6406c31753..b8b8e95b718174b20013840411b394e93c28ca4f 100644 --- a/core/modules/rdf/tests/src/Functional/CommentAttributesTest.php +++ b/core/modules/rdf/tests/src/Functional/CommentAttributesTest.php @@ -249,10 +249,11 @@ public function testCommentReplyOfRdfaMarkup() { * * Tests the current page for basic comment RDFa markup. * - * @param $comment + * @param \Drupal\comment\CommentInterface $comment * Comment object. - * @param $account - * An array containing information about an anonymous user. + * @param array|null $account + * (optional) An array containing information about an anonymous user. + * Defaults to NULL. */ public function _testBasicCommentRdfaMarkup(CommentInterface $comment, $account = NULL) { $this->drupalGet($this->node->toUrl()); diff --git a/core/modules/rest/tests/src/Functional/ResourceTestBase.php b/core/modules/rest/tests/src/Functional/ResourceTestBase.php index 531226dc7c71a94b22cc8842c793c926c3901bb1..f9cfb1e169f8a114ef99eaff0b3f5489975506c2 100644 --- a/core/modules/rest/tests/src/Functional/ResourceTestBase.php +++ b/core/modules/rest/tests/src/Functional/ResourceTestBase.php @@ -143,6 +143,8 @@ public function setUp() { * The allowed formats for this resource. * @param string[] $authentication * The allowed authentication providers for this resource. + * @param string[] $methods + * The allowed methods for this resource. */ protected function provisionResource($formats = [], $authentication = [], array $methods = ['GET', 'POST', 'PATCH', 'DELETE']) { $this->resourceConfigStorage->create([ diff --git a/core/modules/serialization/tests/serialization_test/src/SerializationTestNormalizer.php b/core/modules/serialization/tests/serialization_test/src/SerializationTestNormalizer.php index 2c8a96265ee9ef3e1dca5d37db6387f0bb99df31..35300d8a8cdcd3784ee55bdb4b54e11034a5ba3d 100644 --- a/core/modules/serialization/tests/serialization_test/src/SerializationTestNormalizer.php +++ b/core/modules/serialization/tests/serialization_test/src/SerializationTestNormalizer.php @@ -18,8 +18,11 @@ class SerializationTestNormalizer implements NormalizerInterface { * * @param object $object * Object to normalize. - * @param string $format - * Format the normalization result will be encoded as. + * @param string|null $format + * (optional) Format the normalization result will be encoded as. Defaults + * to NULL + * @param array $context + * (optional) The context data. Defaults to an empty array. * * @return array * An array containing a normalized representation of $object, appropriate diff --git a/core/modules/statistics/src/NodeStatisticsDatabaseStorage.php b/core/modules/statistics/src/NodeStatisticsDatabaseStorage.php index 68a36dac63b4f09f1b9e542fd9807bdf40ed65af..82a57eef4f931aaa6168cb9764df774438f79301 100644 --- a/core/modules/statistics/src/NodeStatisticsDatabaseStorage.php +++ b/core/modules/statistics/src/NodeStatisticsDatabaseStorage.php @@ -39,6 +39,8 @@ class NodeStatisticsDatabaseStorage implements StatisticsStorageInterface { * The database connection for the node view storage. * @param \Drupal\Core\State\StateInterface $state * The state service. + * @param \Symfony\Component\HttpFoundation\RequestStack $request_stack + * The request stack. */ public function __construct(Connection $connection, StateInterface $state, RequestStack $request_stack) { $this->connection = $connection; diff --git a/core/modules/statistics/src/Plugin/Block/StatisticsPopularBlock.php b/core/modules/statistics/src/Plugin/Block/StatisticsPopularBlock.php index fbd5b1c76dd2679c5519003ce2843156b372aeea..53e39c3333a4eaaeae6487155e73f121b4567729 100644 --- a/core/modules/statistics/src/Plugin/Block/StatisticsPopularBlock.php +++ b/core/modules/statistics/src/Plugin/Block/StatisticsPopularBlock.php @@ -64,6 +64,8 @@ class StatisticsPopularBlock extends BlockBase implements ContainerFactoryPlugin * The entity repository service * @param \Drupal\statistics\StatisticsStorageInterface $statistics_storage * The storage for statistics. + * @param \Drupal\Core\Render\RendererInterface $renderer + * The renderer configuration array. */ public function __construct(array $configuration, $plugin_id, $plugin_definition, EntityTypeManagerInterface $entity_type_manager, EntityRepositoryInterface $entity_repository, StatisticsStorageInterface $statistics_storage, RendererInterface $renderer) { parent::__construct($configuration, $plugin_id, $plugin_definition); diff --git a/core/modules/system/src/Controller/DbUpdateController.php b/core/modules/system/src/Controller/DbUpdateController.php index b8ea203de6c8773d0a59d9dfa30d6f8792fb9e3a..05af92706f8cd3fbc8f8f59af635ec73ebf6180d 100644 --- a/core/modules/system/src/Controller/DbUpdateController.php +++ b/core/modules/system/src/Controller/DbUpdateController.php @@ -510,6 +510,10 @@ protected function results(Request $request) { /** * Renders a list of requirement errors or warnings. * + * @param $severity + * The severity of the message, as per RFC 3164. + * @param array $requirements + * The array of requirement values. * @param \Symfony\Component\HttpFoundation\Request $request * The current request. * diff --git a/core/modules/system/src/Form/ThemeSettingsForm.php b/core/modules/system/src/Form/ThemeSettingsForm.php index 04900d9a24e038818ccbd6439df5d37ee10c3a5b..03e4a4a33c3e6bf957d5f4008e962db501690273 100644 --- a/core/modules/system/src/Form/ThemeSettingsForm.php +++ b/core/modules/system/src/Form/ThemeSettingsForm.php @@ -123,6 +123,10 @@ protected function getEditableConfigNames() { /** * {@inheritdoc} * + * @param array $form + * An associative array containing the structure of the form. + * @param \Drupal\Core\Form\FormStateInterface $form_state + * The current state of the form. * @param string $theme * The theme name. */ diff --git a/core/modules/system/tests/modules/error_service_test/src/MonkeysInTheControlRoom.php b/core/modules/system/tests/modules/error_service_test/src/MonkeysInTheControlRoom.php index 60617048b899829c7df2c1ee19b62353f73f6837..4595a055926686bdcc70839415bb4d0e395542f6 100644 --- a/core/modules/system/tests/modules/error_service_test/src/MonkeysInTheControlRoom.php +++ b/core/modules/system/tests/modules/error_service_test/src/MonkeysInTheControlRoom.php @@ -25,6 +25,8 @@ class MonkeysInTheControlRoom implements HttpKernelInterface { * * @param \Symfony\Component\HttpKernel\HttpKernelInterface $app * The wrapper HTTP kernel. + * @param \Drupal\Core\Site\Settings $settings + * The settings object. */ public function __construct(HttpKernelInterface $app, Settings $settings) { $this->app = $app; diff --git a/core/modules/system/tests/src/Functional/Common/UrlTest.php b/core/modules/system/tests/src/Functional/Common/UrlTest.php index 73a1490e357d65d1a6e39334a50d84248eae2573..35272cfbcf7124780a18e194b7712d3f8d91129d 100644 --- a/core/modules/system/tests/src/Functional/Common/UrlTest.php +++ b/core/modules/system/tests/src/Functional/Common/UrlTest.php @@ -204,6 +204,7 @@ public function testLinkRenderArrayText() { /** * Checks for class existence in link. * + * @param $attribute * @param $link * URL to search. * @param $class diff --git a/core/modules/system/tests/src/Functional/Form/ElementsTableSelectTest.php b/core/modules/system/tests/src/Functional/Form/ElementsTableSelectTest.php index ce0f3d84ea0a103407818eaf870bc354813cc9b1..2ed48f04988c80269365cf6144b746d48a9b534a 100644 --- a/core/modules/system/tests/src/Functional/Form/ElementsTableSelectTest.php +++ b/core/modules/system/tests/src/Functional/Form/ElementsTableSelectTest.php @@ -196,13 +196,16 @@ public function testMultipleFalseOptionchecker() { /** * Helper function for the option check test to submit a form while collecting errors. * - * @param $form_element - * A form element to test. - * @param $edit + * @param array $form + * Nested array of form elements that comprise the form. + * @param array $edit * An array containing post data. * - * @return + * @return array * An array containing the processed form, the form_state and any errors. + * + * @internal param $form_element A form element to test. + * A form element to test. */ private function formSubmitHelper($form, $edit) { $form_id = $this->randomMachineName(); diff --git a/core/modules/views/src/Plugin/views/argument/ArgumentPluginBase.php b/core/modules/views/src/Plugin/views/argument/ArgumentPluginBase.php index 8e5a7c7172ecc3bd81bec8d2502c921ce7a6715c..628b6f7a58e1723298699170df4439e3f6f8f25a 100644 --- a/core/modules/views/src/Plugin/views/argument/ArgumentPluginBase.php +++ b/core/modules/views/src/Plugin/views/argument/ArgumentPluginBase.php @@ -926,6 +926,9 @@ public function summaryBasics($count_field = TRUE) { * * @param $order * The order selected in the UI. + * @param string|null $by + * (optional) This parameter sets the direction for which to order. + * Defaults to NULL. */ public function summarySort($order, $by = NULL) { $this->query->addOrderBy(NULL, NULL, $order, (!empty($by) ? $by : $this->name_alias)); diff --git a/core/modules/views/src/Plugin/views/relationship/EntityReverse.php b/core/modules/views/src/Plugin/views/relationship/EntityReverse.php index 1510d22e7de83be3cf8bd2708a2f1cd8bf2264ed..7f6d26b81b5a1027724220552dbe6488d18229cb 100644 --- a/core/modules/views/src/Plugin/views/relationship/EntityReverse.php +++ b/core/modules/views/src/Plugin/views/relationship/EntityReverse.php @@ -18,6 +18,12 @@ class EntityReverse extends RelationshipPluginBase { /** * Constructs an EntityReverse object. * + * @param array $configuration + * A configuration array containing information about the plugin instance. + * @param string $plugin_id + * The plugin_id for the plugin instance. + * @param array $plugin_definition + * The plugin implementation definition. * @param \Drupal\views\Plugin\ViewsHandlerManager $join_manager * The views plugin join manager. */ diff --git a/core/modules/views_ui/src/ViewEditForm.php b/core/modules/views_ui/src/ViewEditForm.php index 373fd428e44c47ccd0b72c786acf94f03cec4d63..c5cd2381dc3c5c0c290d2b40a2415ebd1f4b2a03 100644 --- a/core/modules/views_ui/src/ViewEditForm.php +++ b/core/modules/views_ui/src/ViewEditForm.php @@ -391,7 +391,9 @@ public function getDisplayTab($view) { /** * Helper function to get the display details section of the edit UI. * - * @param $display + * @param \Drupal\views_ui\ViewUI $view + * The ViewUI entity. + * @param array $display * * @return array * A renderable page build array. diff --git a/core/modules/views_ui/src/ViewFormBase.php b/core/modules/views_ui/src/ViewFormBase.php index 1797f0aa19652d8f8368ed3aa8c36f24d64b7485..c04628bfdabc9a660c95cc8af1c2b947f37dbfe3 100644 --- a/core/modules/views_ui/src/ViewFormBase.php +++ b/core/modules/views_ui/src/ViewFormBase.php @@ -81,8 +81,11 @@ protected function prepareEntity() { * having them as secondary local tasks isn't desired. The caller is responsible * for setting the active tab's #active property to TRUE. * - * @param $display_id - * The display_id which is edited on the current request. + * @param \Drupal\views_ui\ViewUI $view + * The ViewUI entity. + * + * @return array + * An array of tab definitions. */ public function getDisplayTabs(ViewUI $view) { $executable = $view->getExecutable(); diff --git a/core/modules/workspaces/src/WorkspaceListBuilder.php b/core/modules/workspaces/src/WorkspaceListBuilder.php index 69de069fcef05332d10ddb0c8eebe38beb455e9e..6361f5fd3c0130c83e216af160929d18e98c7fa1 100644 --- a/core/modules/workspaces/src/WorkspaceListBuilder.php +++ b/core/modules/workspaces/src/WorkspaceListBuilder.php @@ -51,6 +51,8 @@ class WorkspaceListBuilder extends EntityListBuilder { * The entity storage class. * @param \Drupal\workspaces\WorkspaceManagerInterface $workspace_manager * The workspace manager service. + * @param \Drupal\workspaces\WorkspaceRepositoryInterface $workspace_repository + * The workspace repository service. * @param \Drupal\Core\Render\RendererInterface $renderer * The renderer service. */ diff --git a/core/modules/workspaces/src/WorkspacePublisher.php b/core/modules/workspaces/src/WorkspacePublisher.php index 1df156efd70fe197e3d8de8302b3171af6193d68..1d336cfe1ec7f7ca3258192db6f1a916c6ae05f6 100644 --- a/core/modules/workspaces/src/WorkspacePublisher.php +++ b/core/modules/workspaces/src/WorkspacePublisher.php @@ -62,6 +62,8 @@ class WorkspacePublisher implements WorkspacePublisherInterface { * The workspace manager. * @param \Drupal\workspaces\WorkspaceAssociationInterface $workspace_association * The workspace association service. + * @param \Drupal\workspaces\WorkspaceInterface $source + * The source workspace entity. */ public function __construct(EntityTypeManagerInterface $entity_type_manager, Connection $database, WorkspaceManagerInterface $workspace_manager, WorkspaceAssociationInterface $workspace_association, WorkspaceInterface $source) { $this->entityTypeManager = $entity_type_manager; diff --git a/core/phpcs.xml.dist b/core/phpcs.xml.dist index 4c70d11b4345cdfbd3e43b8bf09473fd56c6c7cd..74b73b5e2aec46f9370752f29cb69f20e526a695 100644 --- a/core/phpcs.xml.dist +++ b/core/phpcs.xml.dist @@ -70,7 +70,6 @@ <exclude name="Drupal.Commenting.FunctionComment.MissingReturnComment"/> <exclude name="Drupal.Commenting.FunctionComment.MissingReturnType"/> <exclude name="Drupal.Commenting.FunctionComment.ParamCommentFullStop"/> - <exclude name="Drupal.Commenting.FunctionComment.ParamMissingDefinition"/> <exclude name="Drupal.Commenting.FunctionComment.TypeHintMissing"/> </rule> <rule ref="Drupal.Commenting.HookComment"/> diff --git a/core/profiles/demo_umami/modules/demo_umami_content/src/InstallHelper.php b/core/profiles/demo_umami/modules/demo_umami_content/src/InstallHelper.php index 0b5b0a65cb79028526e00594407c697ea45268ed..c4764faddd31ae58ebdfde6664184ed3ef37cd12 100644 --- a/core/profiles/demo_umami/modules/demo_umami_content/src/InstallHelper.php +++ b/core/profiles/demo_umami/modules/demo_umami_content/src/InstallHelper.php @@ -427,6 +427,8 @@ protected function processPage(array $data, $langcode) { * * @param array $data * Data of line that was read from the file. + * @param string $langcode + * Current language code. * * @return array * Data structured as a recipe node. diff --git a/core/tests/Drupal/FunctionalTests/Bootstrap/UncaughtExceptionTest.php b/core/tests/Drupal/FunctionalTests/Bootstrap/UncaughtExceptionTest.php index f60f1fc46e7f6138bd5bfbbdc36b4f063a9353f1..5323d2511a7c1df0dac3f9a1ddd1dd4eb484f4e0 100644 --- a/core/tests/Drupal/FunctionalTests/Bootstrap/UncaughtExceptionTest.php +++ b/core/tests/Drupal/FunctionalTests/Bootstrap/UncaughtExceptionTest.php @@ -265,7 +265,7 @@ public function testLoggerException() { $this->assertStringContainsString('Failed to log error', $errors[0], 'The error handling logs when an error could not be logged to the logger.'); $expected_path = \Drupal::root() . '/core/modules/system/tests/modules/error_service_test/src/MonkeysInTheControlRoom.php'; - $expected_line = 59; + $expected_line = 61; $expected_entry = "Failed to log error: Exception: Deforestation in Drupal\\error_service_test\\MonkeysInTheControlRoom->handle() (line ${expected_line} of ${expected_path})"; $this->assertStringContainsString($expected_entry, $errors[0], 'Original error logged to the PHP error log when an exception is thrown by a logger'); diff --git a/core/tests/Drupal/KernelTests/Config/DefaultConfigTest.php b/core/tests/Drupal/KernelTests/Config/DefaultConfigTest.php index 15018db232ffd4f548be7437bdb1078cf7f2699e..ac990870ecb31dc2ec5bc58053ceec0d90d3c406 100644 --- a/core/tests/Drupal/KernelTests/Config/DefaultConfigTest.php +++ b/core/tests/Drupal/KernelTests/Config/DefaultConfigTest.php @@ -135,6 +135,8 @@ public function themeListDataProvider() { * * @param \Drupal\Core\Config\StorageInterface $default_config_storage * The default config storage to test. + * @param string $module + * The module that is being tested. */ protected function doTestsOnConfigStorage(StorageInterface $default_config_storage, $module) { /** @var \Drupal\Core\Config\ConfigManagerInterface $config_manager */ diff --git a/core/tests/Drupal/KernelTests/Core/Database/SchemaTest.php b/core/tests/Drupal/KernelTests/Core/Database/SchemaTest.php index 78ab1b38a1e71f92001bf6afbccb900f6b8e2424..a1ab3ec618436ae9823b002592e4c31bede33939 100644 --- a/core/tests/Drupal/KernelTests/Core/Database/SchemaTest.php +++ b/core/tests/Drupal/KernelTests/Core/Database/SchemaTest.php @@ -1011,6 +1011,8 @@ public function testSchemaChangeFieldDefaultInitial() { * The beginning field specification. * @param $new_spec * The ending field specification. + * @param $test_data + * (optional) A test value to insert and test, if specified. */ protected function assertFieldChange($old_spec, $new_spec, $test_data = NULL) { $table_name = 'test_table_' . ($this->counter++); diff --git a/core/tests/Drupal/KernelTests/Core/Entity/EntityLanguageTestBase.php b/core/tests/Drupal/KernelTests/Core/Entity/EntityLanguageTestBase.php index f0ac841811246eed2a4868ee1ed6237f0074d164..73768a452d2343ddfbe71904b744cc0eb7210ff4 100644 --- a/core/tests/Drupal/KernelTests/Core/Entity/EntityLanguageTestBase.php +++ b/core/tests/Drupal/KernelTests/Core/Entity/EntityLanguageTestBase.php @@ -118,6 +118,8 @@ protected function setUp() { * * @param string $entity_type * The type of the entity fields are attached to. + * @param string $bundle + * The bundle of the entity. */ protected function toggleFieldTranslatability($entity_type, $bundle) { $fields = [$this->fieldName, $this->untranslatableFieldName]; diff --git a/core/tests/Drupal/KernelTests/Core/Entity/EntityQueryAggregateTest.php b/core/tests/Drupal/KernelTests/Core/Entity/EntityQueryAggregateTest.php index e408aeca9594ecbb19045f8a44bc60d03030519c..1fcabc2687dcf6baaf9caf0c2b33895299be7806 100644 --- a/core/tests/Drupal/KernelTests/Core/Entity/EntityQueryAggregateTest.php +++ b/core/tests/Drupal/KernelTests/Core/Entity/EntityQueryAggregateTest.php @@ -582,6 +582,7 @@ public function testRepeatedExecution() { * * @param array $expected * An array of the expected results. + * @param bool $sorted */ protected function assertResults($expected, $sorted = FALSE) { $found = TRUE; diff --git a/core/tests/Drupal/Tests/Component/Datetime/DateTimePlusTest.php b/core/tests/Drupal/Tests/Component/Datetime/DateTimePlusTest.php index 1d3c4ce5baa66dff479934e3915457ffd0768ee8..30330574a4541b8f448d67cceca95067dc0afc89 100644 --- a/core/tests/Drupal/Tests/Component/Datetime/DateTimePlusTest.php +++ b/core/tests/Drupal/Tests/Component/Datetime/DateTimePlusTest.php @@ -187,8 +187,7 @@ public function testDateTimestamp($input, array $initial, array $transform) { * * @param \Drupal\Component\Datetime\DateTimePlus $date * DateTimePlus to test. - * - * @input mixed $input + * @param string $input * The original input passed to the test method. * @param array $initial * @see testTimestamp() @@ -231,6 +230,7 @@ public function assertDateTimestamp($date, $input, $initial, $transform) { * Input argument for DateTimePlus. * @param string $timezone * Timezone argument for DateTimePlus. + * @param string $format * @param string $format_date * Format argument for DateTimePlus::format(). * @param string $expected diff --git a/core/tests/Drupal/Tests/Component/ProxyBuilder/ProxyBuilderTest.php b/core/tests/Drupal/Tests/Component/ProxyBuilder/ProxyBuilderTest.php index 5ea17d617bec60be8c894a0e23997c0790de2d78..2ff8083fee46b669543a661bdc22b901a276b517 100644 --- a/core/tests/Drupal/Tests/Component/ProxyBuilder/ProxyBuilderTest.php +++ b/core/tests/Drupal/Tests/Component/ProxyBuilder/ProxyBuilderTest.php @@ -312,8 +312,12 @@ public function typehintSelf(?\Drupal\Tests\Component\ProxyBuilder\TestServiceNu /** * Constructs the expected class output. * + * @param string $class + * The class name that is being built. * @param string $expected_methods_body * The expected body of decorated methods. + * @param string $interface_string + * (optional) The expected "implements" clause of the class definition. * * @return string * The code of the entire proxy. diff --git a/core/tests/Drupal/Tests/Core/Entity/Sql/DefaultTableMappingTest.php b/core/tests/Drupal/Tests/Core/Entity/Sql/DefaultTableMappingTest.php index 80e903a27a8ca3a3393b07945a768a08af3ce6ec..113543d702c25da7bec4514724fa6db49f275147 100644 --- a/core/tests/Drupal/Tests/Core/Entity/Sql/DefaultTableMappingTest.php +++ b/core/tests/Drupal/Tests/Core/Entity/Sql/DefaultTableMappingTest.php @@ -578,6 +578,9 @@ public function providerTestGetDedicatedTableName() { * The field name. * @param array $column_names * An array of column names for the storage definition. + * @param bool $base_field + * Flag indicating whether the field should be treated as a base or bundle + * field. * * @return \Drupal\Core\Field\FieldStorageDefinitionInterface|\PHPUnit\Framework\MockObject\MockObject */ diff --git a/core/tests/Drupal/Tests/Core/ProxyBuilder/ProxyBuilderTest.php b/core/tests/Drupal/Tests/Core/ProxyBuilder/ProxyBuilderTest.php index aa940b839ff891a54aa1c0e0d26d6d42dc97c623..33bdef6f220c72ee98cbe9db53469edcd22babca 100644 --- a/core/tests/Drupal/Tests/Core/ProxyBuilder/ProxyBuilderTest.php +++ b/core/tests/Drupal/Tests/Core/ProxyBuilder/ProxyBuilderTest.php @@ -62,8 +62,12 @@ public function complexMethod($parameter, callable $function, ?\Drupal\Tests\Cor /** * Constructs the expected class output. * + * @param string $class + * The class name that is being built. * @param string $expected_methods_body * The expected body of decorated methods. + * @param string $interface_string + * (optional) The expected "implements" clause of the class definition. * * @return string * The code of the entire proxy. diff --git a/core/tests/Drupal/Tests/Core/Render/RendererTestBase.php b/core/tests/Drupal/Tests/Core/Render/RendererTestBase.php index 4a243dca74fd3153b33bd509d86408b26384c2e4..651219463181798d3f2257f5fa3ff90b6bc87260 100644 --- a/core/tests/Drupal/Tests/Core/Render/RendererTestBase.php +++ b/core/tests/Drupal/Tests/Core/Render/RendererTestBase.php @@ -265,6 +265,9 @@ class PlaceholdersTest implements TrustedCallbackInterface { * * @param string $animal * An animal. + * @param bool $use_animal_as_array_key + * TRUE if the $animal parameter should be used as an array key, FALSE + * if it should be used as a plain string. * * @return array * A renderable array. diff --git a/core/tests/Drupal/Tests/Core/TypedData/RecursiveContextualValidatorTest.php b/core/tests/Drupal/Tests/Core/TypedData/RecursiveContextualValidatorTest.php index 4fd896bbdd8666ca2349054043a017d559240bc6..25703e7b734c3123439fda339c1e2936e1fa7c2c 100644 --- a/core/tests/Drupal/Tests/Core/TypedData/RecursiveContextualValidatorTest.php +++ b/core/tests/Drupal/Tests/Core/TypedData/RecursiveContextualValidatorTest.php @@ -188,6 +188,7 @@ public function testPropertiesValidateWithMultipleLevels() { * * @param array $tree * An array of value, constraints and properties. + * @param string $name * * @return \Drupal\Core\TypedData\TypedDataInterface|\PHPUnit\Framework\MockObject\MockObject */