When you go to edit screen of menu's which are listed in menu_list_system_menus(), you still see the delete button. Clicking on it also returns a fatal error: Fatal error: Class 'AccessDeniedHttpException' not found.

Files: 
CommentFileSizeAuthor
#1 1920686-fail.patch995 bytesswentel
FAILED: [[SimpleTest]]: [MySQL] 51,365 pass(es), 2 fail(s), and 0 exception(s). View
#1 1920686-1.patch2.91 KBswentel
PASSED: [[SimpleTest]]: [MySQL] 51,475 pass(es). View

Comments

swentel’s picture

Status: Active » Needs review
FileSize
2.91 KB
PASSED: [[SimpleTest]]: [MySQL] 51,475 pass(es). View
995 bytes
FAILED: [[SimpleTest]]: [MySQL] 51,365 pass(es), 2 fail(s), and 0 exception(s). View
amateescu’s picture

Title: Deleting menu link gives fatal error + the delete button is shown on system menu's. » Deleting a menu gives fatal error + the delete button is shown on system menu's.

This title almost gave me a heart attack :D

Status: Needs review » Needs work

The last submitted patch, 1920686-1.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review

#1: 1920686-1.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 1920686-1.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review

#1: 1920686-1.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 1920686-1.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review

#1: 1920686-1.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 1920686-1.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review

#1: 1920686-1.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 1920686-1.patch, failed testing.

alexpott’s picture

Status: Needs work » Needs review

#1: 1920686-1.patch queued for re-testing.

alexpott’s picture

Status: Needs review » Reviewed & tested by the community

Code looks good and passes manual and automated testing...

tim.plunkett’s picture

Just found the same thing. Tests and code are both good.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Other than getting tripped up by #1929764: Rename #type => "actions" / EntityFormController::actions to "buttons" or similar, this looks good to me.

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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