Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
If you install the module you can't find the Configure page from the module description.
Proposed resolution
Add the configure page to the .info.yml file
Read more at: Let Drupal 8 know about your module with an .info.yml file
Remaining tasks
None.
User interface changes
You will have the Configure link in the module section in the /admin/modules page
API changes
None.
Data model changes
None.
Comment | File | Size | Author |
---|---|---|---|
#3 | seo_checklist-add_link_configure_page-2916181-3-D8.patch | 304 bytes | adriancid |
Comments
Comment #2
TravisCarden CreditAttribution: TravisCarden as a volunteer commentedThat's odd. The configuration page is already specified in the
.info.yml
file aschecklistapi.checklists.seo_checklist
. That's definitely the correct route, and I'm sure it used to work. Maybe it doesn't work with dynamic routes and only used to work because we had to hardcode the route specification in the module due to an API bug for a while, and thus ceased working when we removed the hardcoded route. In any case, I'm afraid I don't have bandwidth to look into this, but if someone else figures it out I'll gladly commit a patch.Comment #3
adriancid@TravisCarden I think that is a core issue, I just create #2917439: info.yml file doesn't recognize dynamic routes meanwhile I think that we use the checklistapi.report route that refers to the admin/reports/checklistapi page.
If you have another idea I can made another patch.
Comment #4
tim.plunkettWhile building the links for /admin/modules, access checking is run.
checklistapi.checklists.seo_checklist is the route name
It is set up by \Drupal\checklistapi\Routing\ChecklistapiRoutes::routes()
Which adds
_checklistapi_access
as an access requirementWhich corresponds to \Drupal\checklistapi\Access\ChecklistapiAccessCheck
Which denies access without a
checklist_id
request attribute.Comment #5
tim.plunkett#2917760: ChecklistapiAccessCheck looks in the wrong place for 'checklist_id' and 'op' is the real bug here
Comment #6
adriancid