Part of #2285413: [Meta] Standardize entity route names. See there for details.

CommentFileSizeAuthor
route-names-menulinkcontent.patch3.9 KBCrell
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community 
+++ b/core/modules/menu_link_content/menu_link_content.routing.yml
@@ -6,7 +6,7 @@ menu_link_content.link_add:
-menu_link_content.link_edit:
+entity.menu_link_content.canonical:
   path: '/admin/structure/menu/item/{menu_link_content}/edit'

Tbh. i prefered the previous way: the route has the edit in there, but the canonical linked pointed to but well, we maybe need to provide a proper route in the future anyway.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

  • webchick committed ad41b6f on 8.0.x 
    Issue #2314877 by Crell: Standardize menu link entity route names.

Status: Fixed » Closed (fixed)

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