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

Geez, this is an entity, too?

CommentFileSizeAuthor
#2 route-names-searchpage-1.patch3.03 KBundertext
route-names-searchpage.patch3.03 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 » Needs work 
+++ b/core/modules/search/search.routing.yml
@@ -22,7 +22,7 @@ search.add_type:
+entity.search_page.edit-form:

@@ -46,14 +46,14 @@ search.disable:
+entity.search_page.delete-form:

+++ b/core/modules/search/src/Entity/SearchPage.php
@@ -35,11 +35,11 @@
+ *     "edit-form" = "entity.search_page.edit-form",
+ *     "delete-form" = "entity.search_page.delete-form",

Missing underscore.

undertext’s picture

undertext CreditAttribution: undertext commented
Status: Needs work » Needs review
FileSize
route-names-searchpage-1.patch3.03 KB
dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

Fine.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed cc931ce and pushed to 8.0.x. Thanks!

  • alexpott committed cc931ce on 8.0.x 
    Issue #2314885 by undertext, Crell: Standardize search page entity route...
m1r1k’s picture

m1r1k CreditAttribution: m1r1k commented
Issue tags: +#ams2014contest

Status: Fixed » Closed (fixed)

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