Issue #3427572 by alexpott: Standard Views attribute documentation for consistency and correctness

core/modules/views/src/Attribute/ViewsAccess.php

@@ -36,6 +36,7 @@ class ViewsAccess extends Plugin {
    * @param bool $no_ui
    *   (optional) Whether the plugin should be not selectable in the UI.
    *   If set to TRUE, you can still use it via the API in config files.
+   *   Defaults to FALSE.
    * @param class-string|null $deriver
    *   (optional) The deriver class.
    */

core/modules/views/src/Attribute/ViewsArgumentValidator.php

@@ -26,10 +26,12 @@ class ViewsArgumentValidator extends Plugin {
    *   The plugin title used in the views UI.
    * @param \Drupal\Core\StringTranslation\TranslatableMarkup|null $short_title
    *   (optional) The short title used in the views UI.
-   * @param string $entity_type
+   * @param string|null $entity_type
    *   (optional) Entity type.
    * @param bool $no_ui
-   *   Whether the plugin is selectable in the UI.
+   *   (optional) Whether the plugin should be not selectable in the UI.
+   *   If set to TRUE, you can still use it via the API in config files.
+   *   Defaults to FALSE.
    * @param class-string|null $deriver
    *   (optional) The deriver class.
    */

core/modules/views/src/Attribute/ViewsCache.php

@@ -31,11 +31,12 @@ class ViewsCache extends Plugin {
    *   For example the Feed display defines the type 'feed', so only rss style
    *   and row plugins can be used in the views UI.
    * @param string[] $base
-   *   The base tables on which this cache plugin can be used.
+   *   (optional) The base tables on which this cache plugin can be used.
    *   If no base table is specified the plugin can be used with all tables.
    * @param bool $no_ui
-   *   Whether the plugin should be not selectable in the UI.
+   *   (optional) Whether the plugin should be not selectable in the UI.
    *   If set to TRUE, you can still use it via the API in config files.
+   *   Defaults to FALSE.
    * @param class-string|null $deriver
    *   (optional) The deriver class.
    */

core/modules/views/src/Attribute/ViewsDisplayExtender.php

@@ -25,9 +25,11 @@ class ViewsDisplayExtender extends Plugin {
    * @param \Drupal\Core\StringTranslation\TranslatableMarkup|null $short_title
    *   (optional) The short title used in the views UI.
    * @param \Drupal\Core\StringTranslation\TranslatableMarkup|null $help
-   *   A short help string; this is displayed in the views UI.
+   *   (optional) A short help string; this is displayed in the views UI.
    * @param bool $no_ui
-   *   Whether the plugin is selectable in the UI.
+   *   (optional) Whether the plugin should be not selectable in the UI.
+   *   If set to TRUE, you can still use it via the API in config files.
+   *   Defaults to FALSE.
    * @param class-string|null $deriver
    *   (optional) The deriver class.
    */

core/modules/views/src/Attribute/ViewsQuery.php

@@ -22,21 +22,22 @@ class ViewsQuery extends Plugin {
    *
    * @param string $id
    *   The plugin ID.
-   * @param \Drupal\Core\StringTranslation\TranslatableMarkup|null $title
+   * @param \Drupal\Core\StringTranslation\TranslatableMarkup $title
    *   The plugin title used in the views UI.
    * @param \Drupal\Core\StringTranslation\TranslatableMarkup|null $short_title
    *   (optional) The short title used in the views UI.
    * @param \Drupal\Core\StringTranslation\TranslatableMarkup|null $help
-   *   A short help string; this is displayed in the views UI.
+   *   (optional) A short help string; this is displayed in the views UI.
    * @param bool $no_ui
-   *   Whether or not the plugin is selectable in the UI. If it's set to TRUE,
-   *   you can still use it via the API in config files.
+   *   (optional) Whether the plugin should be not selectable in the UI.
+   *   If set to TRUE, you can still use it via the API in config files.
+   *   Defaults to FALSE.
    * @param class-string|null $deriver
    *   (optional) The deriver class.
    */
   public function __construct(
     public readonly string $id,
-    public readonly ?TranslatableMarkup $title = NULL,
+    public readonly ?TranslatableMarkup $title,
     public readonly ?TranslatableMarkup $short_title = NULL,
     public readonly ?TranslatableMarkup $help = NULL,
     public readonly bool $no_ui = FALSE,