@woosh/meep-engine 2.131.9 → 2.131.10

package/package.json

@@ -5,7 +5,7 @@
   "description": "Pure JavaScript game engine. Fully featured and production ready.",
   "type": "module",
   "author": "Alexander Goldring",
-  "version": "2.131.9",
+  "version": "2.131.10",
   "main": "build/meep.module.js",
   "module": "build/meep.module.js",
   "exports": {

package/src/engine/asset/preloader/AssetPreloader.d.ts

@@ -1,6 +1,8 @@
 /**
  * Utility for loading multiple assets together.
  *
+ * NOTE: A pre-loader is not intended to be re-used. You have one collection of assets to be loaded, use one {@link AssetPreloader} instance for it. For another batch - use a separate instance.
+ *
  * @example
  *  const loader = new AssetPreloader();
  *
@@ -35,7 +37,6 @@ export class AssetPreloader {
         error: Signal<any, any, any, any, any, any, any, any>;
         loadStart: Signal<any, any, any, any, any, any, any, any>;
         completed: Signal<any, any, any, any, any, any, any, any>;
-        resolved: Signal<any, any, any, any, any, any, any, any>;
     };
     /**
      *

package/src/engine/asset/preloader/AssetPreloader.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"AssetPreloader.d.ts","sourceRoot":"","sources":["../../../../../src/engine/asset/preloader/AssetPreloader.js"],"names":[],"mappings":"AAqDA;;;;;;;;;;;;;;;GAeG;AACH;IACI;;;OAGG;IACH,iBAFU,MAAM,CAEI;IAEpB;;;OAGG;IACH,iBAFU,aAAa,EAAE,EAAE,CAEf;IAEZ;;OAEG;IACH;;;;;;;;MAQE;IAaF;;;;;;;OAOG;IACH,SANW,MAAM,QACN,MAAM,UACN,MAAM,aACN,MAAM,GACL,cAAc,CAqCzB;IAED;;;OAGG;IACH,aAFW;QAAC,GAAG,EAAC,MAAM,CAAC;QAAC,IAAI,EAAC,MAAM,CAAC;QAAC,KAAK,CAAC,EAAC,MAAM,CAAA;KAAC,EAAE,QAQpD;IAED;;;;OAIG;IACH,kCAFY,cAAc,CAyHzB;CACJ;AAED;;;GAGG;AACH,wBAFU,cAAc,CAEgB;AAjSxC;;GAEG;AACH;IA+BI;;;;OAIG;IACH,4BAFY,aAAa,CAQxB;IAxCG;;;OAGG;IACH,YAAe;IACf;;;OAGG;IACH,aAAgB;IAChB;;;OAGG;IACH,OAFU,MAAM,GAAC,UAAU,CAEK;IAChC;;;OAGG;IACH,UAFU,MAAM,CAEC;IAGrB;;;;;aAKC;CAcJ;mBAlDkB,uCAAuC;uBAEnC,iBAAiB"}
+{"version":3,"file":"AssetPreloader.d.ts","sourceRoot":"","sources":["../../../../../src/engine/asset/preloader/AssetPreloader.js"],"names":[],"mappings":"AAsDA;;;;;;;;;;;;;;;;;GAiBG;AACH;IACI;;;OAGG;IACH,iBAFU,MAAM,CAEI;IAEpB;;;OAGG;IACH,iBAFU,aAAa,EAAE,EAAE,CAEf;IAEZ;;OAEG;IACH;;;;;;;MAOE;IAaF;;;;;;;OAOG;IACH,SANW,MAAM,QACN,MAAM,UACN,MAAM,aACN,MAAM,GACL,cAAc,CAqCzB;IAED;;;OAGG;IACH,aAFW;QAAC,GAAG,EAAC,MAAM,CAAC;QAAC,IAAI,EAAC,MAAM,CAAC;QAAC,KAAK,CAAC,EAAC,MAAM,CAAA;KAAC,EAAE,QAQpD;IAED;;;;OAIG;IACH,kCAFY,cAAc,CAmIzB;CACJ;AAED;;;GAGG;AACH,wBAFU,cAAc,CAEgB;AA5SxC;;GAEG;AACH;IA+BI;;;;OAIG;IACH,4BAFY,aAAa,CAQxB;IAxCG;;;OAGG;IACH,YAAe;IACf;;;OAGG;IACH,aAAgB;IAChB;;;OAGG;IACH,OAFU,MAAM,GAAC,UAAU,CAEK;IAChC;;;OAGG;IACH,UAFU,MAAM,CAEC;IAGrB;;;;;aAKC;CAcJ;mBAlDkB,uCAAuC;uBAEnC,iBAAiB"}

package/src/engine/asset/preloader/AssetPreloader.js

@@ -1,4 +1,5 @@
 import { assert } from "../../../core/assert.js";
+import { array_push_if_unique } from "../../../core/collection/array/array_push_if_unique.js";
 import Signal from "../../../core/events/signal/Signal.js";
 import { clamp01 } from "../../../core/math/clamp01.js";
 import AssetLevel from "./AssetLevel.js";
@@ -54,6 +55,8 @@ class AssetLoadSpec {
 /**
  * Utility for loading multiple assets together.
  *
+ * NOTE: A pre-loader is not intended to be re-used. You have one collection of assets to be loaded, use one {@link AssetPreloader} instance for it. For another batch - use a separate instance.
+ *
  * @example
  *  const loader = new AssetPreloader();
  *
@@ -90,7 +93,6 @@ export class AssetPreloader {
         error: new Signal(),
         loadStart: new Signal(),
         completed: new Signal(),
-        resolved: new Signal(),
     };
 
     constructor() {
@@ -167,6 +169,9 @@ export class AssetPreloader {
      * @return {AssetPreloader}
      */
     load(assetManager) {
+        assert.defined(assetManager, 'assetManager');
+        assert.equal(assetManager.isAssetManager, true, 'assetManager.isAssetManager !== true');
+
         const on = this.on;
 
         //current level being processed
@@ -188,7 +193,7 @@ export class AssetPreloader {
          * submit requests in batches in order of importance
          * @param {number} level
          */
-        function loadBatch(level) {
+        function load_level(level) {
             // filter out assets of the specified level
             const batch = assets[level];
 
@@ -243,7 +248,7 @@ export class AssetPreloader {
                 return b.priotity - a.priotity;
             });
 
-            batch.forEach(function (def) {
+            for (const def of batch) {
 
                 assetManager.get({
                     path: def.uri,
@@ -252,14 +257,17 @@ export class AssetPreloader {
                     failure: assetLoadFailed
                 });
 
-            });
+            }
         }
 
+        /**
+         *
+         * @type {number[]}
+         */
         const levels = [];
+
         for (let level in assets) {
-            if (assets.hasOwnProperty(level)) {
-                levels.push(level);
-            }
+            array_push_if_unique(levels, level);
         }
 
         let lastLoadedLevel = 0;
@@ -267,17 +275,21 @@ export class AssetPreloader {
         function prod() {
 
             if (lastLoadedLevel < levels.length) {
+                
+                // load next
 
                 const levelToLoad = lastLoadedLevel;
                 lastLoadedLevel++;
                 on.levelFinished.addOne(prod);
 
-                //console.log('requesting load of level ',levelToLoad);
-
-                loadBatch(levelToLoad);
+                load_level(levelToLoad);
 
             } else {
+
+                // we're done
+
                 on.completed.send0();
+
             }
 
         }