From 94cb83c8bd89d5ba6e655c65313bb3e9f4d1eeb2 Mon Sep 17 00:00:00 2001
From: Brian Chirls <brian@chirls.com>
Date: Wed, 8 Mar 2017 11:03:07 -0500
Subject: [PATCH] Added `ready` state to PuppetShow; Play button enabled when
 all audio loaded #12

---
 src/js/index.js       |  8 +++++---
 src/js/puppet-show.js | 42 +++++++++++++++++++++++++++++++++++++++++-
 2 files changed, 46 insertions(+), 4 deletions(-)

diff --git a/src/js/index.js b/src/js/index.js
index 972723e..e660d2b 100644
--- a/src/js/index.js
+++ b/src/js/index.js
@@ -298,6 +298,7 @@ function updateButtons() {
 	document.getElementById('edit-buttons').style.display = isEditing ? '' : 'none';
 	document.getElementById('sound-effects').style.display = isEditing ? '' : 'none';
 	document.getElementById('editing-label').style.display = puppetShow.isCreator ? '' : 'none';
+	document.getElementById('play').disabled = !puppetShow.ready;
 
 	if (isEditing) {
 		const recordButton = document.getElementById('record');
@@ -311,8 +312,6 @@ function updateButtons() {
 			recordButton.innerHTML = 'Reset';
 		}
 	}
-
-
 }
 
 function initializeEditor() {
@@ -453,7 +452,10 @@ puppetShow
 	.on('error', id => {
 		console.log('error loading puppet show', id);
 		// todo: clear stage, force redraw and report error
-	});
+	})
+	.on('ready', updateButtons)
+	.on('unready', updateButtons);
+
 
 
 // Request animation frame loop function
diff --git a/src/js/puppet-show.js b/src/js/puppet-show.js
index e64e5ee..0cb03b3 100644
--- a/src/js/puppet-show.js
+++ b/src/js/puppet-show.js
@@ -100,6 +100,7 @@ function sortEvents(a, b) {
 }
 
 function PuppetShow(options) {
+	const me = this;
 	const {audioContext} = options;
 
 	let showId = '';
@@ -107,8 +108,10 @@ function PuppetShow(options) {
 	let showCreatorId = '';
 	let audioAssetsRef = null;
 	let title = '';
-	let loaded = false;
+	let loaded = false; // loaded = data loaded, enough to edit
+	let ready = false; // ready = ready to play
 	let duration = 0;
+	let assetsToLoad = 0;
 	/*
 	todo:
 	- set/get methods for metadata (arbitrary key/value)
@@ -127,6 +130,13 @@ function PuppetShow(options) {
 	const audioAssets = new Map();
 	const events = [];
 
+	function checkReady() {
+		if (!ready && !assetsToLoad && loaded) {
+			ready = true;
+			me.emit('ready');
+		}
+	}
+
 	eventEmitter(this);
 
 	this.authenticate = authenticate;
@@ -165,11 +175,14 @@ function PuppetShow(options) {
 	this.unload = () => {
 		const id = showId;
 		const wasLoaded = loaded;
+		const wasReady = ready;
 
 		showId = '';
 		showCreatorId = '';
 		showRef = null;
 		loaded = false;
+		ready = false;
+		assetsToLoad = 0;
 		duration = 0;
 		/*
 		todo:
@@ -183,6 +196,10 @@ function PuppetShow(options) {
 		audioAssets.clear();
 		events.length = 0;
 
+		if (wasReady) {
+			this.emit('unready');
+		}
+
 		if (wasLoaded) {
 			this.emit('unload', id);
 		}
@@ -260,6 +277,9 @@ function PuppetShow(options) {
 							console.log('loaded buffer', url, decodedBuffer);
 
 							duration = Math.max(duration, audioObject.time + decodedBuffer.duration);
+
+							assetsToLoad--;
+							checkReady();
 						});
 					};
 					xhr.onerror = e => {
@@ -271,6 +291,8 @@ function PuppetShow(options) {
 				}).catch(err => {
 					console.error('Error accessing file', err, auth.currentUser);
 				});
+
+				assetsToLoad++;
 			});
 
 			loaded = true;
@@ -306,6 +328,7 @@ function PuppetShow(options) {
 		audioAssets.clear();
 		events.length = 0;
 		duration = 0;
+		assetsToLoad = 0;
 
 		// erasing all media assets and events from server
 		showRef.child('duration').set(0);
@@ -372,6 +395,10 @@ function PuppetShow(options) {
 
 		audioAssets.set(id, audioObject);
 
+		const wasReady = ready;
+		ready = false;
+		assetsToLoad++;
+
 		// todo: Decode and add to audioObject.buffer
 		const fileReader = new FileReader();
 		fileReader.onloadend = () => {
@@ -387,6 +414,9 @@ function PuppetShow(options) {
 
 				duration = Math.max(duration, audioObject.time + decodedBuffer.duration);
 				showRef.child('duration').set(duration);
+
+				assetsToLoad--;
+				checkReady();
 			});
 		};
 		fileReader.readAsArrayBuffer(encodedBlob);
@@ -406,12 +436,22 @@ function PuppetShow(options) {
 		});
 
 		// todo: add to list of events
+
+		if (wasReady) {
+			this.emit('unready');
+		}
 	};
 
 	Object.defineProperties(this, {
 		id: {
 			get: () => showId
 		},
+		loaded: {
+			get: () => loaded
+		},
+		ready: {
+			get: () => ready
+		},
 		duration: {
 			get: () => duration
 		},