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.

Files: 
CommentFileSizeAuthor
extra_block_routing_arg.patch445 bytesmmilano
PASSED: [[SimpleTest]]: [MySQL] 58,628 pass(es).
[ View ]

Comments

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.