Closed (fixed)
Project:
Wishlist Module
Version:
5.x-2.0
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
23 Nov 2007 at 17:52 UTC
Updated:
13 Dec 2007 at 13:31 UTC
Jump to comment: Most recent file
Comments
Comment #1
draco2002 commentedSorry haven't had my coffee yet. Diff-ed the files in the wrong order on the command line.
This one should work better.
Comment #2
scott.mclewin commentedA 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. :)
Comment #3
bittindm commentedHere'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.
Comment #4
scott.mclewin commentedAh 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.
Comment #5
scott.mclewin commented@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.
Comment #6
(not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.