try catch around youtube to prevent errors

MapcreatorIO · Apr 12, 2016 · 4b8feed · 4b8feed
1 parent 998ead4
commit 4b8feed
Showing 3 changed files with 42 additions and 34 deletions.
diff --git a/app/js/classes/popup.js b/app/js/classes/popup.js
@@ -174,22 +174,26 @@ Popup.prototype.stopVideo = function() {
  * Start the video inside the popup
  */
 Popup.prototype.startVideo = function() {
-	switch(this.media_type) {
-		case "youtube":
-			if(
-				this.youtube && [
-					YT.PlayerState.UNSTARTED,
-					YT.PlayerState.PAUSED,
-					YT.PlayerState.CUED,
-					YT.PlayerState.BUFFERING
-				].indexOf(this.youtube.getPlayerState()) !== -1
-			) {
-				this.youtube.playVideo();
-			}
-			break;
-		case "video":
-			document.getElementById(this.video_id).play();
-			break;
+	try { // TODO fix youtube check
+		switch(this.media_type) {
+			case "youtube":
+				if(
+					this.youtube && [
+						YT.PlayerState.UNSTARTED,
+						YT.PlayerState.PAUSED,
+						YT.PlayerState.CUED,
+						YT.PlayerState.BUFFERING
+					].indexOf(this.youtube.getPlayerState()) !== -1
+				) {
+					this.youtube.playVideo();
+				}
+				break;
+			case "video":
+				document.getElementById(this.video_id).play();
+				break;
+		}
+	} catch(e) {
+		console.warn("YouTube wasn't ready, yet!", e);
 	}
 };
 

diff --git a/dist/m4n.js b/dist/m4n.js
@@ -1297,22 +1297,26 @@ var M4nInteractive = (function(options, container, callback) {
      * Start the video inside the popup
      */
     Popup.prototype.startVideo = function() {
-        switch (this.media_type) {
-            case "youtube":
-                if (
-                    this.youtube && [
-                        YT.PlayerState.UNSTARTED,
-                        YT.PlayerState.PAUSED,
-                        YT.PlayerState.CUED,
-                        YT.PlayerState.BUFFERING
-                    ].indexOf(this.youtube.getPlayerState()) !== -1
-                ) {
-                    this.youtube.playVideo();
-                }
-                break;
-            case "video":
-                document.getElementById(this.video_id).play();
-                break;
+        try { // TODO fix youtube check
+            switch (this.media_type) {
+                case "youtube":
+                    if (
+                        this.youtube && [
+                            YT.PlayerState.UNSTARTED,
+                            YT.PlayerState.PAUSED,
+                            YT.PlayerState.CUED,
+                            YT.PlayerState.BUFFERING
+                        ].indexOf(this.youtube.getPlayerState()) !== -1
+                    ) {
+                        this.youtube.playVideo();
+                    }
+                    break;
+                case "video":
+                    document.getElementById(this.video_id).play();
+                    break;
+            }
+        } catch (e) {
+            console.warn("YouTube wasn't ready, yet!", e);
         }
     };
 

diff --git a/dist/m4n.min.js b/dist/m4n.min.js