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

CommentFileSizeAuthor
route-names-category.patch2.21 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/contact/src/Entity/Category.php
@@ -35,8 +35,8 @@
diff --git a/core/modules/contact/src/Entity/Message.php b/core/modules/contact/src/Entity/Message.php

diff --git a/core/modules/contact/src/Entity/Message.php b/core/modules/contact/src/Entity/Message.php
index 2f5e140..b52b32a 100644

index 2f5e140..b52b32a 100644
--- a/core/modules/contact/src/Entity/Message.php

--- a/core/modules/contact/src/Entity/Message.php
+++ b/core/modules/contact/src/Entity/Message.php

+++ b/core/modules/contact/src/Entity/Message.php
+++ b/core/modules/contact/src/Entity/Message.php
@@ -32,7 +32,7 @@

@@ -32,7 +32,7 @@
  *   bundle_entity_type = "contact_category",
  *   fieldable = TRUE,
  *   links = {
- *     "admin-form" = "contact.category_edit"
+ *     "admin-form" = "entity.contact_category.edit_form"
  *   }
  * )
  */

Do we have a follow up for this double linking as well?

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 de764ba on 8.0.x 
    Issue #2314865 by Crell: Standardize contact entity route names.
Crell’s picture

Crell CreditAttribution: Crell commented

Yes we do: #2309187: Fix double-link-entry between Entity and Entity Type classes

Status: Fixed » Closed (fixed)

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