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 ]


Status:Active» Needs review

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

@mmilano thanks nice catch.

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.

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

Assigned:tim.plunkett» Unassigned

Status:Reviewed & tested by the community» Fixed

Committed and pushed to 8.x. Thanks!

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