Bug 1813 - Wrong linking of "all songs" at the bottom of the hierachy during browsing
: Wrong linking of "all songs" at the bottom of the hierachy during browsing
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 6.1.0
: PC Windows XP
: P2 normal (vote)
: ---
Assigned To: KDF
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-07-13 05:17 UTC by Dieter
Modified: 2008-08-18 10:54 UTC (History)
0 users

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dieter 2005-07-13 05:17:43 UTC
When selecting in SlimServer "browse albums" and then "all songs" on top of the
song list again a hyperlink "all songs" is shown. Clicking on this hyperlinks
should probably do nothing (since we are already in the "all songs" list) but
does in fact lead list of all genres. On top of this genre list there is again a
hyperlink for "all songs" but clicking on this link does nothing (so still all
genres are shown but not all songs).

The same is true when starting with "browse artists", "browse genres", "browse
years", "new songs" (I don't have any artwork so I cannot comment "browse artwork").

Further, when I select "browse music folder" and then click on "all songs" shown
on top of the list no song is shown on the next page but only the following
lines are displayed:

Play this song
Add this song to the playlist
Title:  	 Musik
File Format: 	
Location: 	C:\temp\Musik (Download)
Date Modified: 	Dienstag, Juli 5, 2005, 12:52:42
Comment 1 KDF 2005-07-13 09:12:22 UTC
I think I'm getting to the poitn where I'd rather see every 'all' link
disappear.  WAY more trouble than they are worth.
Comment 2 KDF 2005-07-13 14:15:49 UTC
All songs is not always for descending via a link, it is also a shortcut for
playing all.  However, sometimes the hierarchy ends up blank (when we've hit the
bottom of a particular path).  If their is no hierarchy, the best way to deal
with this is to disable the descending link and keeping only the play/add links
active.

framing the link in [% IF hierarchy %] should take care of that.

Dan,assign to me if you'd like to take care of that for 6.1
Comment 3 KDF 2005-07-13 15:42:02 UTC
ok, will patch this tonight so leaving a note for myself for later.  Seems that
a lot of skins will need a tweak.  BMF needs two changes to Pages.pm to mark the
top level for Descend, and to undefine the hierarchy to that the 'all songs'
link will just be static text.
Comment 4 KDF 2005-07-13 19:17:25 UTC
committed to trunk at change 3698.  part of nightly build for july 14.  please
re-open if any specific related issues remain.
Comment 5 Chris Owens 2008-03-11 11:28:28 UTC
This bug was marked resolved in Slimserver 6.1, which is several versions ago.  If you're still seeing this bug, please re-open it.  Thanks!