Follow on from #1939024: Convert action_admin_manage to a new-style Controller and blocked until that issue is committed in some form (because commits would otherwise conflict - issues are dependent). Probably also makes sense to include action_admin_delete_form in this conversion.

Convert this page callback to a new-style Controller, using the instructions on

#3 1944756-action-admin-configure-controller-3.patch9.73 KBkim.pepper
PASSED: [[SimpleTest]]: [MySQL] 54,263 pass(es). View


ayelet_Cr’s picture

Assigned:Unassigned» ayelet_Cr
kim.pepper’s picture

@ayelet_Cr are you still working on this issue?

kim.pepper’s picture

Assigned:ayelet_Cr» kim.pepper
Status:Active» Needs review
9.73 KB
PASSED: [[SimpleTest]]: [MySQL] 54,263 pass(es). View

Haven't had any activity on this for a week, so I've re-assigned to myself.

First attempt attached.

tim.plunkett’s picture

Status:Needs review» Reviewed & tested by the community

The moved code here is ugly as sin. But fixing that is not within scope.

The conversion is correct otherwise.

webchick’s picture

Status:Reviewed & tested by the community» Fixed

Committed and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary:View changes

Add another function for conversion which would make little sense as a separate issue.