Originally raised in #422262: Patch adds configurable block to list selected modules.
This block provides a link to a project page on drupal.org, but for a module, and formulates the url as http://drupal.org/project/[module]. This should be http://drupal.org/project/[project].
| Comment | File | Size | Author |
|---|---|---|---|
| #12 | em--766296-11.patch | 803 bytes | juliangb |
| #9 | em--766296-9.patch | 801 bytes | juliangb |
| #4 | em--module-project-change.patch | 642 bytes | juliangb |
| #2 | enabled_modules--766296-2.patch | 960 bytes | juliangb |
| #1 | enabled_modules--766296-1.patch | 1.05 KB | juliangb |
Comments
Comment #1
juliangb commentedThis patch should fix the link for the module in question, but it is going to be more difficult to get the correct link for the dependencies/dependents.
Comment #2
juliangb commentedCommitted to D6.
D7 version attached.
Comment #3
juliangb commentedCommitting of D7 version delayed due to #768276: "Key Site Modules" block not shown on D7.
For dependents / dependencies, we have the following options:
- Show a link to /project/[module-name], even though this might be wrong (if the project name != the module name - current behaviour.
- Don't show links, just module name.
- User input? This would add a whole lot of unnecessary? code.
Thoughts?
Comment #4
juliangb commentedPatch against D7 version.
Comment #5
juliangb commentedPushed #4.
Now have to sort out dependents / dependencies links. I'm going to change it to link to the project name if the project is known, else show no link - just the module name text.
Comment #6
mlncn commentedExcellent!
At some point we could consider making blocks render element with theme functions for D7 but i think that's a lot of overhead for an admin-facing module.
Comment #7
mlncn commentedOop, i was responding to #4. Can't keep up.
Comment #8
juliangb commentedHa! Have a bit of time so thought I should get some of these old issues sorted.
Do you fancy opening a new issue for the theme rendering? Although I'm wondering whether this block should be deprecated and recreated using views anyway - which would make it configurable by the site admin.
Comment #9
juliangb commentedPatch for D7 that checks the project on dependents and dependencies. Needs testing.
Comment #10
juliangb commentedPatch for D7 that checks the project on dependents and dependencies. Needs testing.
Comment #12
juliangb commentedAmended patch.
Comment #13
juliangb commentedThis isn't quite there yet.