Posted by stella on April 27, 2009 at 9:38pm
| Project: | Notify by Views |
| Version: | 5.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
Hi,
This module doesn't support default views provided by modules and so they don't appear in the list on admin/content/notify The attached patch fixes this issue.
Cheers,
Stella
Comments
#1
#2
Could you back up and make it so that it just gets all of the views using the proper API function, instead of the one db query plus this api call?
#3
That's just it, I couldn't find one. I had hoped to replace the db query with an api function, but couldn't find one that would either (a) return all custom views or (b) return both custom and default views. Views itself just seems to select directly from the db each time that information is required.
#4
OK, good to know I wasn't just being lazy when I first wrote that query. :-)
Thanks for the patch.
#5
Automatically closed -- issue fixed for 2 weeks with no activity.