Documentation in ConfigDependencyManager conflates plugin dependencies and config dependencies making it confusing

* Classes for configurable plugins are a special case. They can either declare * their configuration dependencies using the calculateDependencies() method * described in the paragraph above, or if they have only static dependencies, * these can be declared using the 'config_dependencies' annotation key.

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Comment 11 months ago →
🇳🇿New Zealand quietone New Zealand
Add novice tag
Status changed to Needs review 11 months ago1:41pm 10 August 2023
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.2 & MySQL 8
last update 11 months ago
29,958 pass
Comment 11 months ago →
🇮🇳India sourabhjain
I have updated the changes mentioned in #4. Please review.
Status changed to RTBC 11 months ago6:03pm 16 August 2023
Comment 11 months ago →
🇺🇸United States smustgrave
#4 does appear to have been copied over.

RTBC+1
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.2 & MySQL 8
last update 11 months ago
29,961 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.2 & MySQL 8
last update 11 months ago
30,047 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.2 & MySQL 8
last update 10 months ago
30,052 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.2 & MySQL 8
last update 10 months ago
30,056 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.2 & MySQL 8
last update 10 months ago
30,058 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.2 & MySQL 8
last update 10 months ago
30,060 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.2 & MySQL 8
last update 10 months ago
30,060 pass
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.2 & MySQL 8
39:33
37:30
Running
Status changed to Needs work 10 months ago4:17am 31 August 2023
Comment 10 months ago →
🇳🇿New Zealand quietone New Zealand
I'm triaging RTBC issues → . I re-read the IS and the comments.

My comment in #18 was a simple read of the suggested text, I did not check it for accuracy. In hindsight, that looks like a mistake.

It is a 9 year old comment and it does need to read and checked for accuracy. I did a bit of that by searching for each interface or method referred to in the comment. It turns out that \Drupal\Component\Plugin\ConfigurablePluginInterface no longer exists. It was removed in #3081145: Remove BC layers in the Plugin component → .

Setting back to Needs work and removing the novice tag.

Documentation in ConfigDependencyManager conflates plugin dependencies and config dependencies making it confusing

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Comments & Activities