Bug 3738 – "status" returns stringified hash as album name

Bug 3738 - "status" returns stringified hash as album name

Summary:

"status" returns stringified hash as album name

Status:	RESOLVED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	CLI
Version:	6.5b1
Platform:	PC Windows XP

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

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2006-07-12 11:37 UTC by Michael Herger
Modified:	2008-09-15 14:39 UTC (History)
CC List:	1 user (show)

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 Michael Herger 2006-07-12 11:37:44 UTC

Status returns a stringified hash reference or something instead of the album name:

67%3A1d%3A9b%3Abb%3Aa2%3A85 status - 1 tags:l
67%3A1d%3A9b%3Abb%3Aa2%3A85 status - 1 tags%3Al player_name%3A192.168.0.122 player_connected%3A1 power%3A1 mode%3Astop time%3A0 rate%3A1 mixer%20volume%3A50 playlist%20repeat%3A2 playlist%20shuffle%3A0 playlist_cur_index%3A2 playlist_tracks%3A12 playlist%20index%3A2 id%3A117 title%3ASecret%20World album%3ASlim%3A%3ASchema%3A%3AAlbum%3DHASH(0x51f5898)

Comment 1 KDF 2006-07-12 13:02:26 UTC

$album is now an object.  query handler needs to use $album->title()

perhaps change Queries.pm line 1796 from:
'l' => 'album', 
to:
'l' => 'album->title', 
?

Comment 2 Fred 2006-07-12 17:17:51 UTC

Yes, all DB code in the current 6.5 is rather messy. Tracks/playlist should start to be better with my latest patch tonight. Working on it.

Comment 3 Michael Herger 2006-07-12 23:37:26 UTC

Fred, in some thread you asked me what was broken for me. Shall I fill in here whenever I find something, or shall I wait until dust on 6.5 & CLI has settled down a bit?

Comment 4 Dan Sully 2006-07-13 13:14:19 UTC

Fred - I'm going to assign this to you, since you're in the midst of changes.

Comment 5 Fred 2006-07-13 16:37:12 UTC

Dan -- please do.
Michael -- yes, wait a bit. My request was premature, sorry.

Comment 6 Michael Herger 2006-07-13 23:54:35 UTC

Thanks! Working fine for me.