Download & Extend

Adding Initial Views Support to Wishlist

Project:Wishlist Module
Version:5.x-2.0
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

This adds the ability to show the Priority number in a view and Sort by that number.

Just getting ready to use the wishlist for the holidays.

Would be nice to show the get/bought column as well, but we'll take baby steps.

AttachmentSize
wishlist_view.patch874 bytes

Comments

#1

Sorry haven't had my coffee yet. Diff-ed the files in the wrong order on the command line.

This one should work better.

AttachmentSize
wishlist_view.patch 874 bytes

#2

Status:needs work» fixed

A short but sweet patch - thanks. It has been incorporated into HEAD and DRUPAL_5 for the wishlist module.

As you flesh out further views support keep the patches coming in. And don't always expect a few hour turnaround time. I happened to be reviewing the wishlist issue queue when you posted this. :)

#3

Here's a patch with more Views support made from today's version of HEAD. I added an Filter for item is public even though it seems like the Public / Private status is meant to set the node's publish status in which case you could just use the Node: Published filter. I don't think this is apparent unless you look in to the module code and database. As noted in this issue http://drupal.org/node/172751, there does seem to be an existing bug when going from public to private then back to public.

AttachmentSize
wishlist_views_112707.patch 1.47 KB

#4

Status:fixed» reviewed & tested by the community

Ah good, I am glad somebody is giving the D6 version some playtime. I'll blend this patch in shortly (perhaps tonight). The patch itself looks simple enough.

#5

Status:reviewed & tested by the community» fixed

@bittindm - your patch has been committed to head. I only made sure it parsed correctly, I did not test whether it worked. But looking at it, I'm pretty sure it works just fine.

#6

Status:fixed» closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

nobody click here