supernginx / flowplayer-core

Automatically exported from code.google.com/p/flowplayer-core
0 stars 0 forks source link

Audio plugin: onStart and onFinish works incorrect #569

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
1. openhttp://flowplayer.org/demos/plugins/flash/audio.html 
2. open JavaScript console and enter this:
$f().stop();
$f().onStart(function(){
console.log('start');
});
$f().onFinish(function(){
console.log('finish');
});
$f().play({url: 'http://releases.flowplayer.org/data/fake_empire.mp3'});

as you can see there is no 'start' message display
same problem with onFinish

$f().pause(); doesn't work too

Original issue reported on code.google.com by siria...@gmail.com on 24 May 2012 at 12:49

GoogleCodeExporter commented 8 years ago
please try this version

http://dl.dropbox.com/u/3394987/flowplayer.audio-3.2.9.zip

And a different mp3 file for comparison. 

Original comment by dani...@electroteque.org on 25 May 2012 at 8:49

GoogleCodeExporter commented 8 years ago
Any luck ?

Original comment by electrot...@gmail.com on 27 May 2012 at 2:59

GoogleCodeExporter commented 8 years ago
Any luck ?

Original comment by electrot...@gmail.com on 27 May 2012 at 3:00

GoogleCodeExporter commented 8 years ago
I'll try tomorrow this version and when post here results

Original comment by siria...@gmail.com on 27 May 2012 at 6:17

GoogleCodeExporter commented 8 years ago
onStart still doesn't triggered if clip.url is equal to previous

Original comment by siria...@gmail.com on 28 May 2012 at 7:17

GoogleCodeExporter commented 8 years ago

Original comment by electrot...@gmail.com on 1 Jun 2012 at 7:01

GoogleCodeExporter commented 8 years ago
please provide demo page, your exact code is triggering to reload a brand new 
file which will trigger on start. hitting replay will also now trigger on 
start. 

Original comment by electrot...@gmail.com on 1 Jun 2012 at 7:05

GoogleCodeExporter commented 8 years ago
http://dl.dropbox.com/u/3394987/flowplayer.audio-3.2.9.zip just so we're on the 
same page use this file if possible. 

Original comment by electrot...@gmail.com on 1 Jun 2012 at 7:06

GoogleCodeExporter commented 8 years ago
http://sirian.su/flowplayer/
try to $f().play('1.mp3')

Original comment by siria...@gmail.com on 1 Jun 2012 at 7:25

GoogleCodeExporter commented 8 years ago
I don't know what you are trying to achieve there, place the first clip into 
the clip config. $f().play("http://sirian.su/flowplayer/1.mp3"); by doing this 
you are resetting the playlist also. If you want to replay the file 
$f().play(0);  says to replay that file in the index. I do see an issue though 
resetting the playlist with the same file though. 

Original comment by dani...@electroteque.org on 2 Jun 2012 at 8:17

GoogleCodeExporter commented 8 years ago
This needs a special case because you are resetting the playlist not replaying 
the clip so the clip duration setting starts back to zero, and its an already 
buffered file it won't estimate the duration / trigger on start. Trying to work 
around this. 

Original comment by dani...@electroteque.org on 2 Jun 2012 at 8:34

GoogleCodeExporter commented 8 years ago
http://dl.dropbox.com/u/3394987/flowplayer.audio-3.2.9.zip

please try this it will now send the start method correctly. what is happening 
you are resetting the playlist and because the clip is already buffered there 
is a restriction on sending the events while obtaining the duration and because 
the clip is already in the buffer this range was 0. It will send start properly 
now. 

Original comment by dani...@electroteque.org on 2 Jun 2012 at 9:52

GoogleCodeExporter commented 8 years ago
correction the latest change should work better.

Original comment by dani...@electroteque.org on 2 Jun 2012 at 6:21

GoogleCodeExporter commented 8 years ago
thx, now it works as expected

Original comment by siria...@gmail.com on 4 Jun 2012 at 7:36

GoogleCodeExporter commented 8 years ago
ok not a problem I will close this for now. 

Original comment by dani...@electroteque.org on 4 Jun 2012 at 8:44