Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 300130

Adopt PluginExists validator in relevant places

$
0
0

Problem/Motivation

#2920682: Add config validation for plugin IDs added the PluginExists constraint and used it in 2 places.

There are many more places where we need to use this.

StatusManagerSchemaInterface
In patchplugin.manager.actionaction.configuration.\Drupal\Core\Action\ActionInterface
N/Aplugin.manager.archiver
Doneplugin.manager.block
plugin.manager.ckeditor
plugin.manager.conditioncondition.plugin.
plugin.manager.config_translation.mapper
plugin.manager.core.layoutlayout_plugin.settings.\Drupal\Core\Layout\LayoutInterface
N/Aplugin.manager.display_variant
Doneplugin.manager.editor
N/Aplugin.manager.element_info
plugin.manager.entity_reference_selection
plugin.manager.field.field_type
plugin.manager.field.formatter
plugin.manager.field.widget
In patchplugin.manager.filter
plugin.manager.help_section
plugin.manager.help_section_topics
plugin.manager.help_topic
In patchplugin.manager.image.effectimage.style.effects.\Drupal\image\ImageEffectInterface
plugin.manager.language_negotiation_method
plugin.manager.layout_builder.section_storage
plugin.manager.link_relation_type
plugin.manager.mail
In patchplugin.manager.media.sourcemedia.type.\Drupal\media\MediaSourceInterface
plugin.manager.menu.contextual_link
plugin.manager.menu.link
plugin.manager.menu.local_action
plugin.manager.menu.local_task
plugin.manager.migrate.destination
plugin.manager.migrate.field
plugin.manager.migrate.id_map
plugin.manager.migrate.process
plugin.manager.migrate.source
plugin.manager.migration
plugin.manager.queue_worker
In patchplugin.manager.rest
In patchplugin.manager.searchsearch.page.Drupal\search\Plugin\SearchInterface
plugin.manager.tour.tip
plugin.manager.views.access
In patchplugin.manager.views.area
In patchplugin.manager.views.argument
plugin.manager.views.argument_default
plugin.manager.views.argument_validator
plugin.manager.views.cache
In patchplugin.manager.views.display
plugin.manager.views.display_extender
In patchplugin.manager.views.exposed_form
In patchplugin.manager.views.field
In patchplugin.manager.views.filter
plugin.manager.views.join
In patchplugin.manager.views.pager
In patchplugin.manager.views.query
plugin.manager.views.relationship
In patchplugin.manager.views.row
In patchplugin.manager.views.sort
In patchplugin.manager.views.style
plugin.manager.views.wizard
plugin.manager.workflows.type

Steps to reproduce

Proposed resolution

  1. Adopt it in all relevant places.
  2. Write test that verifies that all default config in Drupal core complies with the validation constraints, by adding a new testValidity() method to \Drupal\KernelTests\Core\Config\DefaultConfigTest

Remaining tasks

See above.

User interface changes

None.

API changes

None.

Data model changes

None.

Release notes snippet

N/A


Viewing all articles
Browse latest Browse all 300130

Latest Images

Trending Articles



Latest Images

<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>