Search playlist

does drupal's built in search not work?

won't happen in the 5.x version. in the d6 code i'm planning on exposing the playlist track info so that you could build a view and use it's filtering.

This flushes out the playlist views support a bit more and converts the table listing on playlist nodes to a view.

i should add that you can can now filter by the playlist fields getting us close to that searching ideal.

I committed the patch to HEAD. I think you should be able to build a view with exposed filtering to find tracks the way you're describing.

I think there's an error in station_playlist_nodeapi(), line 393 of station_playlist.module.

402: $block = $view->preview($display_id);

That assigns a string of text to $block.

405: $node->content[$content_key]['#title'] = $block['subject'];

That attempts to assign an element of $block that does not exist.

I'm not sure what the intended result was, so I'm not sure the best way to fix it.

I'm using 6.x-2.x-dev of both Views and Station.

- Tim

I tried changing it from $block = $view->preview($display_id); back to $block = $view->execute_display($display_id);, and it solved the problem. But I'm not sure why it was changed in the first place, so this still needs addressing.

mind rolling a patch?

uhhh how's this?

Has anyone tried this out? It's only one line...

hadn't tried it but i don't remember why i changed it in the first place so no harm in rolling it back.

oh forgot to mention committed ;)

First of all, I finally learned the difference between preview() and execute_display(). preview() returns rendered HTML, while execute_display() returns an array of HTML. So switching back to execute_display() was correct.

However, if the view had nothing to display, it still printed an empty <div class="form-item"></div>.
I threw a check in there, this should FINALLY put this issue to rest.

Thanks, committed to HEAD.