Search playlist

Will Kirchheimer - May 20, 2008 - 21:02
Project:Station
Version:6.x-2.x-dev
Component:Playlists
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed
Description

Has anyone come up with a method for searching the playlists, song title, album, etc

#1

drewish - May 26, 2008 - 21:38

does drupal's built in search not work?

#2

Will Kirchheimer - May 27, 2008 - 14:00

It searches the list yes, but it is a fairly blunt object.

Ideally per field searching would be available.

"I want to know which dj played such-and-such song, where the song title is 'Jazz'"

#3

drewish - November 28, 2008 - 19:38
Version:5.x-1.5» 6.x-2.x-dev

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.

#4

drewish - June 1, 2009 - 06:26
Status:active» needs review

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

AttachmentSize
station_playlist_260798.patch 14.03 KB

#5

drewish - June 1, 2009 - 06:39

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

#6

drewish - June 11, 2009 - 15:53
Status:needs review» fixed

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.

#7

tim.plunkett - June 20, 2009 - 03:30
Status:fixed» needs work

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

#8

tim.plunkett - June 20, 2009 - 05:43
Status:needs work» needs review

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.

#9

drewish - June 22, 2009 - 20:25

mind rolling a patch?

#10

tim.plunkett - June 22, 2009 - 20:32

uhhh how's this?

AttachmentSize
station_260798.patch 673 bytes

#11

tim.plunkett - June 28, 2009 - 19:30

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

#12

drewish - June 28, 2009 - 21:09

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

#13

drewish - June 28, 2009 - 21:11
Status:needs review» fixed

oh forgot to mention committed ;)

#14

System Message - July 12, 2009 - 21:20
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.