Create a wishlist version that works with Drupal version 7.

CommentFileSizeAuthor
#4 wishlist-drupal7-838866-5.patch24.47 KBpolynya

Comments

scott.mclewin’s picture

Assigned: cvdenzen » scott.mclewin

I've begun work on this.

hendrakieran’s picture

Awesome, anything that I can test so far?

scott.mclewin’s picture

Changes are on the master line and you are welcome to prod at it, although right now it's not worth your time. It's still mostly broken rather than working-but-with-errors. This module started out life with Drupal 4.5 before I shared it into the community. It has been long in need of cleanup and now is the time.

polynya’s picture

StatusFileSize
new24.47 KB

Thanks for a great module, I've been using it for a few years for Christmas lists.

I have fixed some of the remaining issues with the Drupal 7 version, which were mainly to convert SQL queries to use db_select.

I have also removed the requirement to unpublish nodes by adding a condition to wishlist_node_access. (I'm sorry if I have misunderstood the problem here.)

I have attached a patch (from Master on 22 May 2011.) Please let me know if you would like me to split it into smaller patches if they would be easier for you to test.

The module now works without generating any PHP errors (except the Verlanglijst_1 view which I haven't looked at yet), so hopefully a Dev version can be released soon!

Adrian.

scott.mclewin’s picture

Version: master » 7.x-2.x-dev
Status: Needs work » Closed (fixed)

This patch plus other local changes I was working on are both available on the 7.x-2.x-dev branch.

Install for testing and post issues as you find them.

I have tested lightly, and not given any focus to upgrade installations. The data format is unchanged from 6.x-2.4 to 7.x-2.x-dev, so I do not expect there to be migration problems. But the wishlist node type has changed to being extensible via CCK.

Closing this issue - open new issues independently for problems found with the D7 wishlist module.