updated registerSound and registerManifest to return true if a source has already been loaded.

* Updated libs & docs

Signed-off-by: Lanny McNie <lanny@gskinner.com>

Author: Lanny McNie

VERSIONS.txt

@@ -1,4 +1,4 @@
-Version NEXT [Not Released]
+Version 0.5.0 [Not Released]
 ************************************************************************************************************************
 CRITICAL (may break existing content):
 - removed all onEvent handlers (ex. onClick, onTick, onAnimationEnd, etc)
@@ -16,6 +16,7 @@ CRITICAL (may break existing content):
 	ie play("music", {loop: -1, volume: 0.5})
 - Changed WebAudioPlugin to test if XHR is available for local files rather than assuming it is not.
 - implement basePath support for local loading and with PreloadJS
+- updated registerSound and registerManifest to return true if a source has already been loaded.
 - added getter/setter to volume and pan of SoundInstance, to allow tweening.
 - added polyfil for Array.indexOf to allow IE8 support
 - fixed an issue with Sound.removeSound not stopping audio with that src.

docs/soundjs_docs-NEXT.zip

@@ -718,8 +718,9 @@ this.createjs = this.createjs || {};
 	 * @param {Boolean} [preload=true] If the sound should be internally preloaded so that it can be played back
 	 * without an external preloader.
 	 * @param {string} basePath Set a path that will be prepending to src for loading.
-	 * @return {Object} An object with the modified values that were passed in, which defines the sound. Returns false
-	 * if the source cannot be parsed.
+	 * @return {Object} An object with the modified values that were passed in, which defines the sound.
+	 * Returns false if the source cannot be parsed or no plugins can be initialized.
+	 * Returns true if the source is already loaded.
 	 * @static
 	 * @since 0.4.0
 	 */
@@ -728,7 +729,6 @@ this.createjs = this.createjs || {};
 			return false;
 		}
 
-
 		if (src instanceof Object) {
 			basePath = id;	//this assumes preload will not be passed in as a property // OJR check if arguments == 3
 			//?? preload = src.preload;
@@ -746,7 +746,7 @@ this.createjs = this.createjs || {};
 			s.idHash[id] = details.src;
 		}
 
-		var numChannels = null; // null will set all SoundChannel to set this to it's internal maxDefault
+		var numChannels = null; // null tells SoundChannel to set this to it's internal maxDefault
 		if (data != null) {
 			if (!isNaN(data.channels)) {
 				numChannels = parseInt(data.channels);
@@ -791,10 +791,13 @@ this.createjs = this.createjs || {};
 			}  // we do this so we can store multiple id's and data if needed
 			s.preloadHash[details.src].push({src:src, id:id, data:data});  // keep this data so we can return it in fileload event
 			if (s.preloadHash[details.src].length == 1) {
+				// if already loaded once, don't load a second time  // OJR note this will disallow reloading a sound if loading fails or the source changes
 				if (basePath == null) {basePath = "";}
 				s.activePlugin.preload(details.src, loader, basePath);
+			} else {
+				// if src already loaded successfully, return true
+				if (s.preloadHash[details.src][0] == true) {return true;}
 			}
-			// if already loaded once, don't load a second time  // OJR note this will disallow reloading a sound if loading fails or the source changes
 		}
 
 		return details;
@@ -819,8 +822,9 @@ this.createjs = this.createjs || {};
 	 * {{#crossLink "Sound/registerSound"}}{{/crossLink}}: <code>{src:srcURI, id:ID, data:Data, preload:UseInternalPreloader}</code>
 	 * with "id", "data", and "preload" being optional.
 	 * @param {string} basePath Set a path that will be prepending to each src when loading.
-	 * @return {Object} An array of objects with the modified values that were passed in, which defines each sound. It
-	 * will return false for any values that the source cannot be parsed.
+	 * @return {Object} An array of objects with the modified values that were passed in, which defines each sound.
+	 * Like registerSound, it will return false for any values that the source cannot be parsed or if no plugins can be initialized.
+	 * Also, it will returns true for any values that the source is already loaded.
 	 * @static
 	 * @since 0.4.0
 	 */

lib/soundjs-NEXT.min.js

@@ -27,6 +27,6 @@ this.createjs = this.createjs || {};
 	 * @type String
 	 * @static
 	 **/
-	s.buildDate = /*date*/"Wed, 11 Sep 2013 18:29:40 GMT"; // injected by build process
+	s.buildDate = /*date*/"Thu, 12 Sep 2013 21:45:41 GMT"; // injected by build process
 
 })();