Bug 7643 - CLI current duration value wrong for some plugins
: CLI current duration value wrong for some plugins
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: CLI
: 7.0.1
: PC Windows XP
: -- normal (vote)
: 7.x
Assigned To: Squeezebox QA Team email alias
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-03-27 12:49 UTC by Tom Hale
Modified: 2009-07-31 10:18 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tom Hale 2008-03-27 12:49:54 UTC
On a status with K,a,l,d,u tags with Rhapsody playing.

The duration returned after the rate tag in the list of tags before the individual songs:

<rate tag>
duration:224    (Not the duration of the current playing song.)
<mixer volume tag>


The duration of the 3 tracks in order received (Note: These are the correct durations of the songs):

duration:277    (playlist index:0)

duration:184    (playlist index:1)

duration:224    (playlist index:2)
Comment 1 Michael Herger 2008-03-28 01:36:17 UTC
Could you please provide us with the full CLI sequence you're using? Just copy the output of the console to this bug report.
Comment 2 Tom Hale 2008-03-31 09:38:47 UTC
(In reply to comment #1)
> Could you please provide us with the full CLI sequence you're using? Just copy
> the output of the console to this bug report.
> 



What I send.

00:04:20:06:56:00 status - 3 subscribe:0 tags:K,a,l,d,u<LF>


What I receive.(I kinda separated it out for readability)
In This case the first duration returned is 191 which matches the last entry. Should be 435.

00%3A04%3A20%3A06%3A56%3A00 status - 3 subscribe%3A0 
tags%3AK%2Ca%2Cl%2Cd%2Cu 
player_name%3ATom's%20Office%20Player 
player_connected%3A1 
power%3A1 
signalstrength%3A0 
mode%3Astop 
remote%3A1 
current_title%3A 
time%3A0 
rate%3A1 
duration%3A191 
mixer%20volume%3A84 
playlist%20repeat%3A0 
playlist%20shuffle%3A0 
playlist_cur_index%3A1 
playlist_timestamp%3A1206906433.84639 
playlist_tracks%3A16 

playlist%20index%3A1 
id%3A5144 
title%3ALucky%20Star artwork_url%3Ahttp%3A%2F%2Fimage.listen.com%2Fimg%2F170x170%2F4%2F0%2F3%2F1%2F751304_170x170.jpg 
artist%3AMadonna 
album%3AThe%20Immaculate%20Collection 
duration%3A435 
url%3Arhapd%3A%2F%2FTra.1888310.wma 

playlist%20index%3A2 
id%3A5145 
title%3ABorderline artwork_url%3Ahttp%3A%2F%2Fimage.listen.com%2Fimg%2F170x170%2F4%2F0%2F3%2F1%2F751304_170x170.jpg 
artist%3AMadonna 
album%3AThe%20Immaculate%20Collection 
duration%3A318 
url%3Arhapd%3A%2F%2FTra.1888311.wma 

playlist%20index%3A3 
id%3A5146 
title%3ALike%20A%20Virgin artwork_url%3Ahttp%3A%2F%2Fimage.listen.com%2Fimg%2F170x170%2F4%2F0%2F3%2F1%2F751304_170x170.jpg 
artist%3AMadonna 
album%3AThe%20Immaculate%20Collection 
duration%3A191 
url%3Arhapd%3A%2F%2FTra.1888312.wma

Comment 3 Michael Herger 2008-04-23 04:45:57 UTC
Andy - this bug is due to the change you did to fix bug 6943 (Queries.pm, line 5035). That fix seems to be needed for Pandora because it's using one single track object, but it's breaking Rhapsody, as Rhapsody can handle more than one track. It's overwriting Rhapsody's correct value.

_songData probably should not directly overwrite the result's duration, but only the value for the current item within the loop?
Comment 4 Andy Grundman 2008-04-23 06:00:28 UTC
Fixed in 7.0.1 change 19078.
Comment 5 James Richardson 2008-05-12 14:50:29 UTC
Tom:  Can you verify that this issue is resolved for you using the latest daily build of SC 7.0.1
Comment 6 James Richardson 2008-05-15 12:25:28 UTC
This bug has recently been fixed in the latest release of SqueezeCenter 7.0.1

Please try that version, if you still see the error, then reopen this bug.

To download this version, please navigate to: http://www.slimdevices.com/su_downloads.html
Comment 7 Michael Herger 2009-04-15 02:15:09 UTC
QA/Tom - I've checked in a code cleanup which touches the fix Andy added for this workaround (change 25960). Please test with the latest builds to make sure I didn't break anything. Rhapsody is blocking me again :-(.
Comment 8 Chris Owens 2009-07-31 10:18:36 UTC
Reduce number of active targets for SC