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.From http://www.drupalusability.org/node/9:
The admin/by-module page generates links like:
<a href="/admin/user/permissions#module-my-module">Configure permissions</a>
But the admin/user/permissions page generates anchors like:
<td class="module" id="module-My Module" colspan="21">Comment</td>
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 392582.002.patch | 1.25 KB | karschsp |
| #1 | 392582.001.patch | 1.27 KB | karschsp |
Comments
Comment #1
karschsp CreditAttribution: karschsp commentedHere's a quick patch for this.
Comment #2
karschsp CreditAttribution: karschsp commentedforgot to change status.
Comment #3
karschsp CreditAttribution: karschsp commentedre-rolling patch to use ['#id'] rather than ['#anchorname']
Comment #4
Dave ReidAwesome. It's much better using #id, and consistant with current FormAPI conventions.
We should also get some quick tests written to make sure this doesn't break again. Do we have any tests that work with the admin/by-module page?
Comment #5
cburschkaThis is just two lines, and both looked okay at a glance. The internal module name is safe to use here, since they are guaranteed to be unique.
Comment #6
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD. Thanks!
Ideally, we'd have a generic broken link tester at some point.