Bug 821 – browseID3 playermode broken

Bug 821 - browseID3 playermode broken

Summary:

browseID3 playermode broken

Status:	RESOLVED INVALID

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Player UI
Version:	6.0.0
Platform:	All All

Importance:	P2 normal (vote)
Target Milestone:	---
Assigned To:	Dan Sully

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2005-02-02 00:54 UTC by KDF
Modified:	2005-02-03 10:00 UTC (History)
CC List:	0 users

See Also:
Category:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description KDF 2005-02-02 00:54:36 UTC

The BrowseID3 mode still uses the old methods for retrieving data from the
infocache, as well as the Slim::Music::Info::songs() which is deprecated AND
still uses the old Control::Command API.  as such, adding artist, genres or
albums will fail to add anything to the current playlist.

Comment 1 Robert Moser II 2005-02-03 08:40:31 UTC

BrowseID3.pm doesn't use Slim::Music::Info::songs(), but it does use
Slim::Control::Command::execute(['playlist','(add|play|insert)album', which then
uses S::M::I::songs().  Using Softsqueeze, this seemed to still work fine.

Comment 2 KDF 2005-02-03 10:00:15 UTC

This was based on a user report, which at the time I had been able to confirm. 
However, I'm now just getting the backtrace and it does seem to be adding. 
Instant mix, however, does not produce a list, but tht's antoher bug.