Bug 4499 – wma files with album art attached not showing up in SlimServer

Bug 4499 - wma files with album art attached not showing up in SlimServer

Summary:

wma files with album art attached not showing up in SlimServer

Status:	RESOLVED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Display
Version:	6.5.1
Platform:	PC Windows XP

Importance:	P4 minor (vote)
Target Milestone:	---
Assigned To:	Chris Owens

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2006-11-08 12:31 UTC by Ross Levine
Modified:	2008-12-18 11:11 UTC (History)
CC List:	3 users (show)

See Also:
Category:	---

Attachments
WMP w/ album art attached not displaying in SS (1) (3.75 MB, application/octet-stream) 2006-11-08 12:32 UTC, Ross Levine	Details
WMP w/ album art attached not displaying in SS (2) (2.84 MB, application/octet-stream) 2006-11-08 12:32 UTC, Ross Levine	Details
scan log with wlm on debug (12.24 KB, text/plain) 2006-11-08 18:25 UTC, KDF	Details
View All 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 Ross Levine 2006-11-08 12:31:13 UTC

I'll attach the two files in a minute. The album art appears fine in Windows Media Player, but nothing in SlimServer.

Comment 1 Ross Levine 2006-11-08 12:32:21 UTC

Created attachment 1700 [details]
WMP w/ album art attached not displaying in SS (1)

Comment 2 Ross Levine 2006-11-08 12:32:50 UTC

Created attachment 1701 [details]
WMP w/ album art attached not displaying in SS (2)

Comment 3 KDF 2006-11-08 16:03:14 UTC

there is a patch in bug 4269 that helps there.  Is this the same problem?

Comment 4 KDF 2006-11-08 18:25:26 UTC

Created attachment 1702 [details]
scan log with wlm on debug

for the record, this is the debug output from the wma module when scanning the two supplied files.  As you can see, only the second file apears to have any artwork.  The patch from the other bug does appear to extract the artwork correctly from the SECOND file.

Comment 5 Ross Levine 2006-11-09 11:52:16 UTC

KDF, you're amazing. 

My customer (tmid 13061) tried it out and he says it works perfectly! I'd think that means we should consider implementing this in SlimServer?

Comment 6 KDF 2006-11-09 14:45:05 UTC

Chris, how do you feel about this one?  Seems to cover both bug 4269 and this one.
I can merge it in any time, but I don't want to just shove it in if QA isn't on board.

can assign either of the bugs to me and I'll take care of it, asap.  I'll flag it as 6.5.1 targetted for now, but it will get punted if it's not appropriate.

Comment 7 Chris Owens 2006-11-22 16:50:06 UTC

It doesn't look risky to me.  Dan, what do you think?  Can KDF put this in 6.5.1?

Comment 8 Dan Sully 2006-11-22 17:38:00 UTC

Sounds good to me.

Comment 9 KDF 2006-11-22 19:28:57 UTC

committed to trunk at change 10750 
and to 6.5.1 at change 10751