EntityRouteEnhancer provides the default route _entity_list which entities can use to define a route to the page that lists all available entities. This is uses throughout core, except for the Node and Filter modules, which still point directly to the method that returns the list: EntityListController::listing().

To be consistent we should use _entity_list everywhere.

Beta phase evaluation

Reference: https://www.drupal.org/core/beta-changes
Issue category Task, because its a cleanup of code
Issue priority Normal, given that its not that much cleanup.
Disruption No disruption
CommentFileSizeAuthor
#2 2552403-2.patch1.1 KBpfrenssen
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pfrenssen created an issue. See original summary.

pfrenssen’s picture

Component: routing system » entity system
Status: Active » Needs review
FileSize
1.1 KB

Here's a patch. Also moving to the entity system, that is probably more appropriate.

dawehner’s picture

Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Nice!

pfrenssen’s picture

Status: Reviewed & tested by the community » Needs review

Wow that was quick! I edited the issue to add the beta evaluation and was very confused for a moment that it was already there :)

pfrenssen’s picture

Status: Needs review » Reviewed & tested by the community
dawehner’s picture

Wow that was quick! I edited the issue to add the beta evaluation and was very confused for a moment that it was already there :)

You know, sometimes I just think its not even worth to ask people for one ... just do it, so everyone saves time.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Zero disruption change to routing.yml files. Seems okay to do during beta. Committed 1234fa6 and pushed to 8.0.x. Thanks!

  • alexpott committed 1234fa6 on 8.0.x
    Issue #2552403 by pfrenssen: Convert remaining routes to use...

Status: Fixed » Closed (fixed)

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