I was trying to wrap my head around this, but I'm pretty sure it's just not needed.

In block.routing.yml, block.admin_display provides an entity_type argument, but BlockListController overrides EntityListController::listing, removing entity_type from the arguments.

I tested by removing the argument and clearing cache. The admin display continued to load fine.

extra_block_routing_arg.patch445 bytesmmilano
PASSED: [[SimpleTest]]: [MySQL] 58,628 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more


mmilano’s picture

Status: Active » Needs review
jibran’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Quick fix, +Novice

@mmilano thanks nice catch.

alexpott’s picture

Assigned: Unassigned » tim.plunkett

Yes we can do this. However, all other entity list controllers use the entity_type key so I'm not sure we want to do this.

Assigning to Tim for a second opinion.

tim.plunkett’s picture

+1 for RTBC, that is not needed, and was left over from refactoring.

tim.plunkett’s picture

Assigned: tim.plunkett » Unassigned
webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

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