fix: prevent duplicate action mounting from double-click

[People-Sea]

Description

Fixes: #19483

resolveTableAction() doesn't check $parentActions like resolveAction() does,
so duplicate table actions from double-clicks always resolve successfully instead
of failing and being cleaned up by unmountAction().

Visual changes

Before

before.mp4

After

after.mp4

Functional changes

• Code style has been fixed by running the composer cs command.
• Changes have been tested to not break existing functionality.
• Documentation is up-to-date.

[danharrin]

So I would have thought the unmountAction() (for when the mounted action cannot be resolved) would have accounted for this, is there a reason it doesn't catch it? I'm just a bit concerned that this current solution breaks the case where an action is embedded within an action modal with the same name and arguments?

I think the ideal solution is to attempt to resolve the action, and if its not resolvable then remove it from the mounted actions stack, I would have thought that the unmount action line handled that.

[People-Sea]

Got it, thanks @danharrin! Updated in the latest commit.

The issue was resolveTableAction() ignored $parentActions, so duplicates always resolved successfully and unmountAction() cleanup never kicked in. Now it matches what
resolveAction() already does.

For nested same-named actions in modals, they won't have context['table'], so they go through resolveAction(), not affected.