Spin-off issue from #3042774-18: Drupal 9 Deprecated Code Report
Several Feed classes implement ConfigurablePluginInterface. These should be replaced with ConfigurableInterface and possibly DependentPluginInterface.
Fetching class constant class of deprecated class Drupal\Component\Plugin\ConfigurablePluginInterface:
Drupal\Component\Plugin\ConfigurablePluginInterface is deprecated
in Drupal 8.7.0 and will be removed before Drupal 9.0.0. You should implement
ConfigurableInterface and/or DependentPluginInterface directly as needed. If
you implement ConfigurableInterface you may choose to implement
ConfigurablePluginInterface in Drupal 8 as well for maximum compatibility,
however this must be removed prior to Drupal 9.
Patch will follow.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | interdiff-3093515-2-3.txt | 1.04 KB | megachriz |
| #4 | feeds-replace-ConfigurablePluginInterface-3093515-3.patch | 3.06 KB | megachriz |
Comments
Comment #2
megachrizI'm not sure yet if DependentPluginInterface should be added everywhere. Also I need to check if ConfigurablePluginInterface should perhaps still be used at some places.
Comment #3
megachrizComment #4
megachrizOkay, I inspected the code and I think it will be best for action plugins to check for both ConfigurableInterface and ConfigurablePluginInterface.
There are also deprecated actions listed that would need to be removed from the list. This would also require a database update. I'll open a new issue for that.
Comment #6
megachrizCommitted #4.
Comment #7
kristen polPer a Slack discussion with Gábor Hojtsy regarding usage of D9 tags (Drupal 9, Drupal 9 compatibility, Drupal 9 readiness, etc.), "Drupal 9 compatibility" should be used for contributed projects that need updating and "Drupal 9" was the old tag for D8 issues before the D9 branch was ready. Doing tag cleanup here based on that discussion.