I've tried adding a non-NH view into the node, which works fine, paging works and everything. However, if I add an argument of RSS: Feed Selector, when I visit the node with the embedded view, there is an RSS Selector, but it points to /node/x/feed, instead of the view's RSS feed. That feed does nothing.
I've also tried overriding the nodehierarchy_children_list view, and adding RSS feeds in there, but that doesn't work either. I wasn't really expecting it to, but I can't think of anything else to try to get this to work.
I've followed the code thru which adds the RSS feeds to the node. I don't really know how it is working when it creates the node feed, to be honest, but it's not the desired result, as far as I can tell - not for me anyway!
Comments
Comment #1
ronan commentedThis is fixed in HEAD. Actually I fixed the argument handling code in general, so other arguments should work now too.
Comment #2
niklp commentedFixed indeed, what a legend! :)
Any chance you can give a brief rundown of what was wrong and what you did, with some line numbers or something? This could be useful info for someone else I think...
Cheers!
Comment #3
ronan commentedIt was actually really simple. For developers, the solution is on line 118 of nodehierarchy_views.module and is fairly well documented in code.
Here's the embed function:
Comment #4
niklp commentedAll good - not sure, shouldn't you be using arg() to get the GET values rather than accessing GET directly? Security issue maybe?
Also, I saw the node content line there earlier, but I can't see the value in my output as a variable (using Devel) - is it trivial to make this views data available for theming? Then we can move the Views data about as we see fit...
Comment #5
ronan commentedNope,
arg()does not do any input sanitizing (http://api.drupal.org/api/function/arg/5). In this case it's up to the views module to filter the input. There is also no way to get a list of all the arguments with arg().The embedded view content is in:
$node->content['nodehierarchy_children']['#value']when you go to theme the node.Comment #6
niklp commentedre 1: ok my mistake, and learning material there too! :)
The embedded view content is not visible in the devel module's output of the node load/render - I suspect this is due to the module's weighting? If so, if you could set the module weight lighter than devel that would help - I didn't think this was working but further inspection proved that it is...
Comment #7
(not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.