Here's a patch that address the dizzying array of messiness that I just saw when installing the vote_up_down.module, which required the votingapi.module.

First of all, the Vote_Up_Down created a package called "Voting", even though it needed the Voting API to run.

Then the Voting API created another package for itself called "Voting API", and deposited one of it's own voting api modules into the "Voting" package with my vote_up_down stuff.

What a mess! Let's consolidate. There's no use in trying to use packages to separate the slight functionality differences between modules that all still accomplish the same thing. Voting.

CommentFileSizeAuthor
#2 votingapi_generate_change_info_package.patch677 bytessenpai
#1 voting_api_screenshot.png45.2 KBsenpai
votingapi_change_info_package.patch563 bytessenpai

Comments

senpai’s picture

senpai commented
StatusFileSize
new voting_api_screenshot.png45.2 KB

Here's a screenie.

senpai’s picture

senpai commented
StatusFileSize
new votingapi_generate_change_info_package.patch677 bytes

And here's one for the votingapi_generate.info to go along with the original patch.

senpai’s picture

senpai commented
Status: Needs review » Reviewed & tested by the community
Freso’s picture

Freso commented
Status: Reviewed & tested by the community » Needs work

I don't think the description should be changed in the same patch. Looks good apart from this. :)

(Also, it would obviously be extra crispy if the patches were in one patch.)

moshe weitzman’s picture

moshe weitzman commented
Status: Needs work » Reviewed & tested by the community

This has been committed to 6. Only 5 is needed.

eaton’s picture

eaton commented
Status: Reviewed & tested by the community » Closed (won't fix)

Support for the 5.x branch of VotingAPI has ended with the release of Drupal 7 and the upcoming release of VotingAPI 7.x-2.4.