Issue #2317981: The Breadcrumb for block content entities is not useful
1 unresolved thread
Merge request reports
Activity
added 2 commits
added 116 commits
-
cdeb1ce8...d1faffc7 - 110 commits from branch
project:10.1.x
- 9002a13c - Update block content paths to extend /admin/content/block
- b90fefee - Add BC redirects for the edit and delete paths
- 84d2f042 - Add a test for the BC redirects
- d0268b98 - Update /block/ to /admin/content/block-content/ in tests
- bc7eaeb8 - Fix more tests
- f5372c7b - Update after #3318549
Toggle commit list-
cdeb1ce8...d1faffc7 - 110 commits from branch
53 67 block_content: \d+ 54 68 69 # @todo Deprecate this route once 70 # https://www.drupal.org/project/drupal/issues/3159210 is fixed, or remove 71 # it in Drupal 11. 72 # @see https://www.drupal.org/node/2317981 73 entity.block_content.edit_form.bc: 74 path: '/block/{block_content}' 75 defaults: 76 _controller: '\Drupal\block_content\Controller\BlockContentController::editRedirect' 77 options: 78 _admin_route: TRUE 79 requirements: 80 _entity_access: 'block_content.update' 81 block_content: \d+ 82 Yes, they are duplicates. So are
entity.block_content.edit_form
andentity.block_content.canonical
.I suppose there is some reason for the existing duplicate. There are link templates for the canonical route and for the edit form, and something might go wrong if one of those is missing. That reason does not apply to the BC route.
This MR uses
entity.block_content.edit_form.bc
in the redirect controller, so I removed the other one (BC version of the canonical route). Let's see what the testbot thinks.
Please register or sign in to reply