Bug 1767 - Browsing New Music -> Any Artist shows incorrect info on top line
: Browsing New Music -> Any Artist shows incorrect info on top line
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Player UI
: 6.1.0
: Macintosh All
: P2 normal (vote)
: ---
Assigned To: Dan Sully
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-07-06 08:20 UTC by Blackketter Dean
Modified: 2008-09-15 14:37 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Blackketter Dean 2005-07-06 08:20:37 UTC
The top line reads "Select item:", but it should show the artist name.
Comment 1 KDF 2005-07-06 11:10:27 UTC
This is due to the 'fake' that is the browse new music.  heirarchy is age,
track.  however, 'age' is really just a fake name for album.  This allows the
list to return albums, but then you drop down one level the head is created from
the list of the level above.  So, going down the levels, its called 'album',
going back up, it sees 'age' and thus, no header.

changing line 494 of Pages.pm to:
'nameTransform' => 'age',

fixes the header, but then returns no tracks for the list.
confusing, I know.
Comment 2 KDF 2005-07-06 12:51:26 UTC
ok, the problem is that the header is constructed by looking for the name found
for the level above.  In this case, the level above is 'age'.  however, the
items found are based on 'nameTransform', which is 'album' in this case.  So,
like we do for the find, we can check for 'nameTransform' before we take the
level above.

to fix this, change line 513 of BrowseDB.pm to:
$header = $names{$fieldInfo->{$levels[$level-1]}->{'nameTransform'} ||
$levels[$level-1]};

now...I go back to my lab and eye-straining soldering :)
Comment 3 Dan Sully 2005-07-06 15:59:25 UTC
Fixed in subversion change 3630.
Comment 4 Chris Owens 2006-06-16 14:41:22 UTC
There are 536 bugs in the database with targets of '---' that were fixed prior to new year 2006.  I am setting them to targets of 6.2.1 to keep them from showing up in my queries.