Use Context system for actions

Open on Drupal.org →

Created on 3 June 2013, over 11 years ago

Updated 25 June 2023, over 1 year ago

Problems

Having a single parameter for actions is not sufficient, nor is it possible to add further metadata to the single parameter (e.g. to define whether it is required)
We need to support actions without an entity paramter.
Actions should follow the same API as conditions if there are no reasons to do it different.
The action subsystem is coupled to entities.
Finally, this is required if the action system should be useful to Rules as well.

Proposed resolution

Make the action sub-system using context, such as conditions.

Remaining tasks

Patch review.

User interface changes

-

API changes

API changes in this issue are approved by the comments on ActionInterface.php.

The 'type' parameter of action annotations is replaced by a proper context definition.
The execute() method on ActionConfigEntityInterface is removed.
The executeMultiple() method signature on ActionInterface is changed to additionally take a context name argument as first parameter.

Original report by tim.plunkett

See #1846172: Replace the actions API → , it currently passes entities around directly, and uses hardcoded type = node

✨ Feature request

Status

Needs work

Version

11.0 🔥

Component

Last updated about 3 hours ago

Maintained by
🇺🇸United States @effulgentsia
🇬🇧United Kingdom @catch
🇬🇧United Kingdom @alexpott
🇦🇺Australia @larowlan
🇺🇸United States @bnjmnm
🇫🇷France @nod_
🇪🇸Spain @ckrina
🇬🇧United Kingdom @justafish

Created by

🇺🇸United States tim.plunkett Philadelphia

Live updates comments and jobs are added and updated live.

Rules 8.x

d8rules

Needs tests
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

Incomplete comments

Sign in to follow issues

Comments & Activities

Not all content is available!

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

Comment almost 2 years ago →
🇳🇿New Zealand quietone
Action module is approved for removal. See 🌱 [Policy] Deprecate Action (UI) module in D10 and move to contrib in D11 Fixed

This is now Postponed. The status is set according to two policies. The Remove a core extension and move it to a contributed project → and the Extensions approved for removal → policies.

It will be moved to the contributed Action project when the Drupal 11 branch is open.
Comment almost 2 years ago →
🇦🇺Australia dpi Perth, Australia
@quietone is this issue affected? its just actions UI due for removal?
Status changed to Needs work almost 2 years ago2:34am 22 February 2023
Comment almost 2 years ago →
🇳🇿New Zealand quietone
Oops, you are right. I was on 'automatic'. Feel free to change the status on any others I did incorrectly. I think I'll take a break and then come back to this task.
Comment over 1 year ago →
🇳🇿New Zealand quietone
Moving Non UI issue to the base system

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024