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 commentedHere's a quick patch for this.
Comment #2
karschsp commentedforgot to change status.
Comment #3
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 commentedCommitted to CVS HEAD. Thanks!
Ideally, we'd have a generic broken link tester at some point.