Follow up for #663946: Merge "List links" page into "Edit menu" page

Updated: Comment #0

Problem/Motivation

+ $operatuins['edit']['href'] = $uri['path'];

typo

Proposed resolution

Simple fix typo
to
+ $operations['edit']['href'] = $uri['path'];

Remaining tasks

Discuss if this means test coverage is missing.

User interface changes

There is a side effect of doing this change.
Before this patch, the edit links from the menu list, used .../edit
like
admin/structure/menu/manage/admin/edit
After they are just
admin/structure/menu/manage/admin

API changes

No.

CommentFileSizeAuthor
#1 drupal8.menu-module.2019735-1.patch646 bytesyesct

Comments

yesct’s picture

yesct commented
StatusFileSize
new drupal8.menu-module.2019735-1.patch646 bytes
yesct’s picture

yesct commented
Assigned: yesct » Unassigned
Status: Active » Needs review
gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Needs review » Reviewed & tested by the community
tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Reviewed & tested by the community » Closed (duplicate)

We're solving this more holistically in #2010290: Editing a config entity from a listing page results in a 'page not found' (deleting that line altogether)