SmithKettlewellRERC / DVX-YouDescribe-issues

Issue tracking repo for DVX & YouDescribe (Automatically exported from code.google.com/p/sk-dvx)
http://youdescribe.org/
0 stars 0 forks source link

Need improved handling of invalid video ID's in player.php #85

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Currently, player.php does not give the user any indication that something is 
wrong when invalid video ID's are specified.
To reproduce:
Go to the urll for the old Pilates Twist video described by Owen: 
http://youdescribe.org/player.php?v=TaB0eT7nC3M

Note that no title is displayed, but Owen is specified as the describer in the 
Describer Combo box. Interestingly, if a garbage video ID is specified, the 
Describer Combo box is populated with mvdiaz as the describer. For example:
http://youdescribe.org/player.php?v=abc123DEF45

In either case, player.php should not display the controls or the player. It 
should print a message saying "This video cannot be found on YouTube. Please 
check the video ID and try again."

Original issue reported on code.google.com by joshua.a...@gmail.com on 26 Sep 2013 at 6:43

GoogleCodeExporter commented 9 years ago
This is actually two problems; 

1/. Videos not being properly detected as deleted, Private or invalid on 
YouTube.

2/. Videos without description, including valid videos, show mvdiaz in the 
Describer Combo box. This means that when an un-described video is played in 
the YouDescribe player, playback freezes at 3 seconds while it tries to play 
the mvdiaz clip. Playback continues if the Play/Pause button is pressed.

Original comment by Owen.R.E...@gmail.com on 26 Sep 2013 at 8:14

GoogleCodeExporter commented 9 years ago
Josh,

Can you look at http://youdescribe.ski.org/owen/player.php?v=TaB0eT7nC3M and 
confirm if what you see is what you'd expected. I added a little extra text 
about the video potentially being deleted or private on YouTube.

-Owen

Original comment by Owen.R.E...@gmail.com on 26 Sep 2013 at 8:31

GoogleCodeExporter commented 9 years ago

Original comment by Owen.R.E...@gmail.com on 1 Nov 2013 at 3:45

GoogleCodeExporter commented 9 years ago
Fixed in release YouDescribe-release-20131220.

Original comment by Owen.R.E...@gmail.com on 21 Dec 2013 at 5:27