Skip to content
Snippets Groups Projects
Commit 99b53d92 authored by catch's avatar catch
Browse files

Issue #2360647 by quietone, alexpott, Manuel Garcia, jhodgdon: Documentation... 

Issue #2360647 by quietone, alexpott, Manuel Garcia, jhodgdon: Documentation in ConfigDependencyManager conflates plugin dependencies and config dependencies

(cherry picked from commit 3219336f)
parent dfba5cb9
No related branches found
No related tags found
17 merge requests!10663Issue #3495778: Update phpdoc in FileSaveHtaccessLoggingTest,!10451Issue #3472458 by watergate, smustgrave: CKEditor 5 show blocks label is not translated,!103032838547 Fix punctuation rules for inline label suffix colon with CSS only,!10150Issue #3467294 by quietone, nod_, smustgrave, catch, longwave: Change string...,!10130Resolve #3480321 "Second level menu",!9936Issue #3483087: Check the module:// prefix in the translation server path and replace it with the actual module path,!9933Issue #3394728 by ankondrat4: Undefined array key "#prefix" and deprecated function: explode() in Drupal\file\Element\ManagedFile::uploadAjaxCallback(),!9914Issue #3451136 by quietone, gapple, ghost of drupal past: Improve...,!9882Draft: Issue #3481777 In bulk_form ensure the triggering element is the bulk_form button,!9839Issue #3445469 by pooja_sharma, smustgrave: Add additional test coverage for...,!9815Issue #3480025: There is no way to remove entity cache items,!9757Issue #3478869 Add "All" or overview links to parent links,!9752Issue #3439910 by pooja_sharma, vensires: Fix Toolbar tests that rely on UID1's super user behavior,!9749Issue #3439910 by pooja_sharma, vensires: Fix Toolbar tests that rely on UID1's super user behavior,!9678Issue #3465132 by catch, Spokje, nod_: Show test run time by class in run-tests.sh output,!9578Issue #3304746 by scott_euser, casey, smustgrave: BigPipe cannot handle (GET)...,!9449Issue #3344041: Allow textarea widgets to be used for text (formatted) fields
Pipeline #237149 passed
Stage: 🪄 Lint
Stage: 🗜️ Test
Pipeline: drupal

#237166

    Pipeline: drupal

    #237160

      Pipeline: drupal

      #237152

        Hide whitespace changes
        Inline Side-by-side
        core/lib/Drupal/Core/Config/Entity/ConfigDependencyManager.php
        + 15
        − 4
        View file @ 99b53d92
        ...@@ -61,10 +61,20 @@ ...@@ -61,10 +61,20 @@
        * dependencies. Implementations should call the base class implementation to * dependencies. Implementations should call the base class implementation to
        * inherit the generic functionality. * inherit the generic functionality.
        * *
        * Classes for configurable plugins are a special case. They can either declare * Some configuration entities have dependencies from plugins and third-party
        * their configuration dependencies using the calculateDependencies() method * settings; these dependencies can be collected by
        * described in the paragraph above, or if they have only static dependencies, * \Drupal\Core\Config\Entity\ConfigEntityBase::calculateDependencies().
        * these can be declared using the 'config_dependencies' annotation key. * Entities with third-party settings need to implement
        * \Drupal\Core\Config\Entity\ThirdPartySettingsInterface in order to trigger
        * this generic dependency collection. Entities with plugin dependencies need to
        * implement \Drupal\Core\Entity\EntityWithPluginCollectionInterface; this
        * causes the base calculateDependencies() method to add the plugins' providers
        * as dependencies, as well as dependencies declared in the
        * "config_dependencies" key from the plugins' definitions. In addition, plugins
        * that implement \Drupal\Component\Plugin\ConfigurablePluginInterface can
        * declare additional dependencies using
        * \Drupal\Component\Plugin\DependentPluginInterface::calculateDependencies(),
        * and these will also be collected by the base method.
        * *
        * If an extension author wants a configuration entity to depend on something * If an extension author wants a configuration entity to depend on something
        * that is not calculable then they can add these dependencies to the enforced * that is not calculable then they can add these dependencies to the enforced
        ...@@ -112,6 +122,7 @@ ...@@ -112,6 +122,7 @@
        * @see \Drupal\Core\Config\Entity\ConfigEntityInterface::getDependencies() * @see \Drupal\Core\Config\Entity\ConfigEntityInterface::getDependencies()
        * @see \Drupal\Core\Config\Entity\ConfigEntityInterface::onDependencyRemoval() * @see \Drupal\Core\Config\Entity\ConfigEntityInterface::onDependencyRemoval()
        * @see \Drupal\Core\Config\Entity\ConfigEntityBase::addDependency() * @see \Drupal\Core\Config\Entity\ConfigEntityBase::addDependency()
        * @see \Drupal\Core\Config\Entity\ConfigEntityBase::calculateDependencies()
        * @see \Drupal\Core\Config\ConfigInstallerInterface::installDefaultConfig() * @see \Drupal\Core\Config\ConfigInstallerInterface::installDefaultConfig()
        * @see \Drupal\Core\Config\ConfigManagerInterface::uninstall() * @see \Drupal\Core\Config\ConfigManagerInterface::uninstall()
        * @see \Drupal\Core\Config\Entity\ConfigEntityDependency * @see \Drupal\Core\Config\Entity\ConfigEntityDependency
        ......
        0% or .
        You are about to add 0 people to the discussion. Proceed with caution.
        Finish editing this message first!
        Please register or to comment