module doesn't support default views

stella - April 27, 2009 - 21:38
Project:Notify by Views
Version:5.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

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

#1

stella - April 27, 2009 - 21:44
AttachmentSize
446652.patch 761 bytes

#2

matt2000 - April 28, 2009 - 17:27
Status:needs review» needs work

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

stella - April 28, 2009 - 18:07
Status:needs work» needs review

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

matt2000 - April 28, 2009 - 18:27
Status:needs review» fixed

OK, good to know I wasn't just being lazy when I first wrote that query. :-)

Thanks for the patch.

#5

System Message - May 12, 2009 - 18:30
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.