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].

Comments

juliangb’s picture

StatusFileSize
new1.05 KB

This 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.

juliangb’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
StatusFileSize
new960 bytes

Committed to D6.

D7 version attached.

juliangb’s picture

Committing 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?

juliangb’s picture

Status: Active » Needs review
StatusFileSize
new642 bytes

Patch against D7 version.

juliangb’s picture

Status: Needs review » Active

Pushed #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.

mlncn’s picture

Status: Active » Reviewed & tested by the community

Excellent!

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.

mlncn’s picture

Status: Reviewed & tested by the community » Active

Oop, i was responding to #4. Can't keep up.

juliangb’s picture

Ha! 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.

juliangb’s picture

Assigned: Unassigned » juliangb
Status: Active » Needs review
StatusFileSize
new801 bytes

Patch for D7 that checks the project on dependents and dependencies. Needs testing.

juliangb’s picture

Patch for D7 that checks the project on dependents and dependencies. Needs testing.

Status: Needs review » Needs work

The last submitted patch, em--766296-9.patch, failed testing.

juliangb’s picture

Status: Needs work » Needs review
StatusFileSize
new803 bytes

Amended patch.

juliangb’s picture

Status: Needs review » Needs work

This isn't quite there yet.