Bug 8776 - Power off doesn't work when pressed between preset button and time it takes to initiate stream
: Power off doesn't work when pressed between preset button and time it takes t...
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Front Panel
: 7.3.0
: PC Windows XP
: P5 normal (vote)
: 7.x
Assigned To: Alan Young
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-07-17 11:45 UTC by Ross Levine
Modified: 2009-07-31 10:24 UTC (History)
4 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ross Levine 2008-07-17 11:45:36 UTC
Thanks Dave for spotting this one. Stream music, press a preset button to play a favorite (I have it set to radio time streams) and before the stream starts playing hit the power button. Boom will come back on and stream despite your split-second change of mind.
Comment 1 Michael Herger 2008-07-17 14:09:48 UTC
unassigning to get more eyes on this
Comment 2 KDF 2008-07-17 17:38:50 UTC
so, power button does actually work?  If so, then the problem is the response to the buffer full event or the response from the remote server triggering playback.  Sounds vaguely like the problem where random mix was also causing players to turn back on.  Looks like a need for normalising the behaviour of powering off so that async events like this get cancelled.
Comment 3 Ross Levine 2008-07-17 17:57:36 UTC
(In reply to comment #2)
> so, power button does actually work?  

Correct, it displays date/time then suddenly it's on and streaming. 
Comment 4 Michael Herger 2008-07-21 08:13:34 UTC
Ross - do you still see this with the new streaming branch?
Comment 5 Chris Owens 2008-07-21 10:45:51 UTC
Andy notes this behavior has actually gotten worse with new streaming
Comment 6 Andy Grundman 2008-07-31 13:26:44 UTC
Fixed in change 22279.
Comment 7 Alan Young 2008-08-15 01:37:03 UTC
Hmm, I cannot reproduce this with new-streaming. Looking at the code, I think that it should already work correctly. If Song.pm is doing the async getTrackInfo stuff while Player::power() is invoked, then playing() will be true and, as it is a remote stream, 'stop' will be executed which will cancel the results of the async fetch.

I guess that there could be a window caused by the deferred execution of the stop command, but it would be very small.
Comment 8 Ross Levine 2008-08-15 12:12:00 UTC
I'm still able to reproduce this in 7.2 - 22623. I'm using a station local here via Radio Time, it takes about 3 seconds to start playing after I hit the preset button. If I hit the preset button then the power button within that 3 seconds, I can reproduce this. 
Comment 9 Ross Levine 2008-10-09 18:10:07 UTC
Still able to reproduce this with steps in comment #8. 
Comment 10 Alan Young 2008-10-15 02:30:52 UTC
With 7.3?
Comment 11 Ross Levine 2008-10-15 11:09:55 UTC
Sorry I should have been more clear, I'm able to reproduce with 7.2.1 - 23530. 
Comment 12 Alan Young 2008-10-24 03:06:56 UTC
Should already be fixed in 7.3
Comment 13 Ross Levine 2008-10-31 16:50:44 UTC
7.3 - 23752, this is actually a little bit worse. If you are playing some music, and you press a preset and immediately hit power the off screen saver shows but the music starts playing. From this state you can't pause or adjust volume you have to press power first. 
Comment 14 Andy Grundman 2008-11-10 08:19:02 UTC
Cannot reproduce in 7.3, works fine for me.
Comment 15 Andy Grundman 2008-11-10 08:19:24 UTC
Oops, figures, now I do see it.
Comment 16 Andy Grundman 2008-11-10 08:23:32 UTC
Alan: can you take a look?

It seems it works as intended if you do this:

1. Player stopped and powered on.
2. Press preset assigned to slow-scanning radio station.
3. Power off.

But the bug occurs if you do this:

1. Player playing a local track.
2. Press preset assigned to slow-scanning radio station.
3. Power off.
Comment 17 Alan Young 2008-11-10 10:03:48 UTC
Change 23888
Comment 18 James Richardson 2008-12-15 12:05:25 UTC
This bug has been fixed in the 7.3.0 release version of SqueezeCenter!

Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.
Comment 19 Chris Owens 2009-07-31 10:24:56 UTC
Reduce number of active targets for SC