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.