Problem/Motivation
Taxonomy Term has the field_ui_base_route set to entity.taxonomy_vocabulary.overview_form, in taxonomy.links.tasks.yml the base route for both the List and Edit tabs is also the overview_form route, this all makes sense.
Yet in #2934995: Add a "Manage permissions" tab for each bundle that has associated permissions we noted that the route paths are not consistent with this, to quote @benjifisher in #mr1296-note51248:
I think it is pretty odd to use the path
/admin/structure/taxonomy/manage/tags/overview/permissionsinstead of/admin/structure/taxonomy/manage/tags/permissions, but that is consistent with what the field_ui gives us.The oddness also affects the breadcrumbs.
Maybe we should update the taxonomy module to use
/admin/structure/taxonomy/manage/tagsfor the overview page and/admin/structure/taxonomy/manage/tags/editfor the edit form.
The links section of the taxonomy_vocabulary Entity Type shows:
* "overview-form" = "/admin/structure/taxonomy/manage/{taxonomy_vocabulary}/overview",
* "edit-form" = "/admin/structure/taxonomy/manage/{taxonomy_vocabulary}",
As noted above, this results in some oddities with generated paths and breadcrumbs, this is also not consistent with other core entity types.
Steps to reproduce
Proposed resolution
Update the taxonomy_vocabulary Entity Type links to be:
* "overview-form" = "/admin/structure/taxonomy/manage/{taxonomy_vocabulary}",
* "edit-form" = "/admin/structure/taxonomy/manage/{taxonomy_vocabulary}/edit",
This will also likely require some BC-layer if we do this in 9.3/9.4, but perhaps we can get away with doing this only in 10.0, as it would be a pretty awkward BC-layer for such a small code change (impact on tests TBC).
Comments
Comment #2
aaronmchaleComment #3
aaronmchale