- Issue created by @anthonyroundtree
- First commit to issue fork.
- @bhumikavarshney opened merge request.
- 🇮🇳India Kanchan Bhogade
Hi
I have tested issue on Drupal 11.xTesting steps:
- Install Project Browser as normal.
- Go to /admin/modules of your site.
- Search for Project Browser
- Click on Help link, , issue reprodused
- Applied MR and check for the same
After applied MR, the Help link broken on Modules page.
Attaching screenshot
Moving to Needs work
- 🇺🇸United States chrisfromredfin Portland, Maine
The issue here is line 24 of project_browser.module which is using Url::fromRoute to link to a route which now requires a parameter.
I think the right thing here is to instead iterate over all enabled sources, and generate a link to each one:
$output .= '<dd>' . t('Users who have the <em>Administer modules</em> can browse modules from the <a href=":project_browser_admin">Browse projects page</a>.', [':project_browser_admin' => Url::fromRoute('project_browser.browse', [])->toString()]) . '</dd>';
So it says something like:
Users who have the Administer modules ^permission can browse projects from the Extend section:
- Contrib modules
- Recipes
Something like that.
- First commit to issue fork.
- 🇮🇳India libbna New Delhi, India
The error occurred because the source parameter was missing when generating the route URL for project_browser.browse in the project_browser_help() function. I fixed this by explicitly passing 'source' => 'official' to Url::fromRoute(). After this change, the error no longer appears.
- Status changed to Needs review
22 days ago 8:37am 2 June 2025 - 🇮🇳India Kanchan Bhogade
Hi
I have tested issue with MR on Drupal 11.x
The MR is applied successfully...Help link working on Modules page.
- 🇮🇳India libbna New Delhi, India
Hey @phenaproxima → I'm not sure I fully understand the reason for removing this line. We added the route because there wasn’t an existing route that redirects to the Project Browser help page. I referenced similar implementations in other modules when adding it. Could you please clarify why this line needs to be removed?
$output .= '<dd>' . t('Users who have the <em>Administer modules</em> can browse modules from the <em>Browse projects</em> page.') . '</dd>';
Thanks. - 🇺🇸United States phenaproxima Massachusetts
It's not that we want to remove the line, we want to remove the link. We should remove the link because the link cannot be generated unless you have the ID of an enabled source plugin. You can compute that dynamically, but it's sort of pointless to do in the context of the Help page.
- 🇺🇸United States phenaproxima Massachusetts
Looks like the controller class is still there, but I'm not sure it's needed anymore...?
- 🇮🇳India roshanibhangale
Hi
I have tested issue on Drupal 11.xTesting steps:
- Install Project Browser as normal.
- Go to /admin/modules of your site.
- Search for Project Browser
- Click on Help link, , issue reprodused
After applied MR, the Help link is redirecting to the help page correctly.
Attaching screenshot
Moving to RTBC+1