views_node_feed_delete_form takes an access argument (delete node feed), but there isn't a corresponding hook_perm implemented. The upshot: you can't delete views node feeds.

CommentFileSizeAuthor
#3 views_node_feed-560066.patch925 bytesphl3tch

Comments

andrewlevine’s picture

andrewlevine commented

The patch didn't seem to get attached to the issue. Make sure you hit the attach button before pressing save.

andrewlevine’s picture

andrewlevine commented
Status: Needs review » Needs work
phl3tch’s picture

phl3tch commented
StatusFileSize
new views_node_feed-560066.patch925 bytes

Oops. Trying it again...

andrewlevine’s picture

andrewlevine commented
Status: Needs work » Fixed

thanks! committed with a small change of the perm name as not to conflict with other modules that might use "node feed"

Please note I don't really have any time to work on the D6 branch but I will look at/commit patches

phl3tch’s picture

phl3tch commented

Thanks. I may well have more patches for you in the near future — my colleagues and I get a fair bit of use out of this module but we have some thoughts regarding its improvement. Assuming of course that we can find time ourselves. ;)

andrewlevine’s picture

andrewlevine commented

CrackWilding, thanks in advance for any work you put in on the module...can use all the help I can get with this one :0

Status: Fixed » Closed (fixed)

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