From c06d3dd8d79ced9b2e815cc799bfee53bfe8521d Mon Sep 17 00:00:00 2001
From: Joachim Bauch <bauch@struktur.de>
Date: Mon, 27 Apr 2015 17:26:39 +0200
Subject: [PATCH] Implement error handling for YouTube playback.

---
 static/js/directives/youtubevideo.js | 56 ++++++++++++++++++++--------
 static/js/sandboxes/youtube.js       | 12 ++++++
 2 files changed, 53 insertions(+), 15 deletions(-)

diff --git a/static/js/directives/youtubevideo.js b/static/js/directives/youtubevideo.js
index 44021e18..dc152f20 100644
--- a/static/js/directives/youtubevideo.js
+++ b/static/js/directives/youtubevideo.js
@@ -128,6 +128,12 @@ define(['require', 'jquery', 'underscore', 'moment', 'text!partials/youtubevideo
 								isYouTubeIframeAPIReadyDefer.resolve();
 							});
 							break;
+						case "youtube.error":
+							$scope.$apply(function(scope) {
+								console.log("YouTube error", data);
+								scope.$emit("youtube.error", data.msgid);
+							});
+							break;
 						case "youtube.playerReady":
 							$scope.$apply(function() {
 								playerReady.resolve();
@@ -163,6 +169,7 @@ define(['require', 'jquery', 'underscore', 'moment', 'text!partials/youtubevideo
 			$scope.$on("$destroy", function() {
 				if (player) {
 					player.destroy();
+					player = null;
 				}
 				if (sandboxApi) {
 					sandboxApi.destroy();
@@ -170,14 +177,6 @@ define(['require', 'jquery', 'underscore', 'moment', 'text!partials/youtubevideo
 				}
 			});
 
-			var errorIds = {
-				"2": "invalidParameter",
-				"5": "htmlPlayerError",
-				"100": "videoNotFound",
-				"101": "notAllowedEmbedded",
-				"150": "notAllowedEmbedded"
-			};
-
 			$scope.isPublisher = null;
 			$scope.playbackActive = false;
 			$scope.hideControlsBar = true;
@@ -194,13 +193,6 @@ define(['require', 'jquery', 'underscore', 'moment', 'text!partials/youtubevideo
 				});
 			});
 
-			var onPlayerError = function(event) {
-				var error = errorIds[event.data] || "unknownError";
-				$scope.$apply(function(scope) {
-					scope.$emit("youtube.error", error);
-				});
-			};
-
 			var getYouTubeId = function(url) {
 				/*
 				 * Supported URLs:
@@ -288,6 +280,40 @@ define(['require', 'jquery', 'underscore', 'moment', 'text!partials/youtubevideo
 				});
 			});
 
+			$scope.$on("youtube.error", function($event, msgid) {
+				var message;
+				switch (msgid) {
+				case "loadScriptFailed":
+					message = translation._("Could not load YouTube player API, please check your network / firewall settings.");
+					break;
+				case "invalidParameter":
+					message = translation._("The request contains an invalid parameter value. Please check the URL of the video you want to share and try again.");
+					break;
+				case "htmlPlayerError":
+					message = translation._("The requested content cannot be played in an HTML5 player or another error related to the HTML5 player has occurred. Please try again later.");
+					break;
+				case "videoNotFound":
+					message = translation._("The video requested was not found. Please check the URL of the video you want to share and try again.");
+					break;
+				case "notAllowedEmbedded":
+					message = translation._("The owner of the requested video does not allow it to be played in embedded players.");
+					break;
+				default:
+					if (msgid) {
+						message = translation._("An unknown error occurred while playing back the video (%s). Please try again later.", msgid);
+					} else {
+						message = translation._("An unknown error occurred while playing back the video. Please try again later.");
+					}
+					break;
+				}
+				if (player) {
+					player.destroy();
+					player = null;
+				}
+				alertify.dialog.alert(message);
+			});
+
+
 			var playVideo = function(id, position, state) {
 				playerReady.done(function() {
 					$scope.playbackActive = true;
diff --git a/static/js/sandboxes/youtube.js b/static/js/sandboxes/youtube.js
index 4a182730..b76f4213 100644
--- a/static/js/sandboxes/youtube.js
+++ b/static/js/sandboxes/youtube.js
@@ -73,6 +73,14 @@
                 "5": "youtube.videocued"
             };
 
+            var errorIds = {
+                "2": "invalidParameter",
+                "5": "htmlPlayerError",
+                "100": "videoNotFound",
+                "101": "notAllowedEmbedded",
+                "150": "notAllowedEmbedded"
+            };
+
             var playerVars = params.playerVars || {};
             delete playerVars.origin;
             this.player = new this.window.YT.Player("youtubeplayer", {
@@ -108,6 +116,10 @@
                         }
 
                         that.postMessage("youtube.event", {"event": msg, "state": event.data, "position": that.player.getCurrentTime()});
+                    },
+                    "onError": function(event) {
+                        var error = errorIds[event.data] || "unknownError";
+                        that.postMessage("youtube.error", {"msgid": error, "code": event.data});
                     }
                 }
             });