Problem/Motivation

In #2946122: Deprecate ConfigurablePluginInterface and replace with an interface that doesn't extend DependentPluginInterface We added an isConfigurable() method on PluginBase. this makes the isConfigurable method on Action unnecessary, as you can call isConfigurable directly on the plugin.

Proposed resolution

Deprecated this function and remove in 9.0.0.

Remaining tasks

Do it.

User interface changes

None

API changes

ActionConfigEntityInterface::isConfigurable() is removed and you should call ActionConfigEntityInterface->getPlugin()->isConfigurable() instead

Data model changes

None.

Release notes snippet

Comments

mikelutz created an issue. See original summary.

vdenis’s picture

vdenis commented
Assigned: Unassigned » vdenis
mikelutz’s picture

mikelutz
He/Him
Primary language English
Location Michigan, USA
commented
Status: Active » Postponed

I should have set this to postponed, as the original issue hasn't been committed yet.

vdenis’s picture

vdenis commented
Assigned: vdenis » Unassigned

Ok, @mikelutz. Agree with you.

mikelutz’s picture

mikelutz
He/Him
Primary language English
Location Michigan, USA
commented
Status: Postponed » Closed (outdated)

We had to revert and remove the isConfigurable method, and replaced it with a PluginHelper static class, that I think we will deprecate in Drupal 9 anyway, so I think this should stay as it is for now.