Bug 2371 - Selecting Year when browsing by year adds entire library
: Selecting Year when browsing by year adds entire library
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 6.2.0
: PC Windows XP
: P2 normal (vote)
: ---
Assigned To: Jacob Potter
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-10-25 18:03 UTC by Mike Cappella
Modified: 2008-09-15 14:37 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 Mike Cappella 2005-10-25 18:03:42 UTC
Selecting a year when browsing by year adds all songs from the library, not just 
those from that year.

See also: http://forums.slimdevices.com/showthread.php?t=17409
Comment 1 Mike Cappella 2005-10-25 18:11:19 UTC
Correction.  This occurs in the ExBrowse3 skin, not Fishbone or Default2.

To further clarify, selecting the year drills down into the year correctly.  Its 
the selecting a play or add to playlist button that fails.
Comment 2 Dan Sully 2005-10-25 18:16:40 UTC
This one looks like yours, Jacob.
Comment 3 Mike Cappella 2005-10-25 18:17:29 UTC
It also occurs in SB2 as well.
Comment 4 KDF 2005-10-25 21:45:18 UTC
fixed player-side problem at change 4821
Comment 5 KDF 2005-10-25 23:42:13 UTC
2005-10-25 23:38:14.5050  Executing command 00:04:20:05:32:e2: playlistcontrol
(cmd:load) (year_id:1967) () () () () ()
2005-10-25 23:38:29.1125  Returning array: playlistcontrol (cmd:load)
(year_id:1967) () () () () ()

year is a special case.  the items are not objects, so the id's don't work.

the other skins use loadtracks field=item, where field is the current level of
the heirarchy.  either that, or the CLI or the skin js should recognise a
non-object case and handle it.

hope that helps out a bit, jacob.
Comment 6 KDF 2005-10-25 23:57:25 UTC
actually, looking at the CLI, it doesn't seem to handle the year_id param. 
I've put in a fix in change 4825 in the trunk, with a doc note at change 4826. 
merges to 6.2.1 at change 4827
Comment 7 Mike Cappella 2005-10-26 09:17:22 UTC
All fixed, thanks!