Dungeon texture cache treats a failed preload as complete #1045

New Issue

2026-08-13T05:23:59Z

spikerj commented

2026-08-13 05:23:59 +00:00

TextureCache.preloadAll swallows per-texture failures (console.warn + continue) and then unconditionally sets isPreloaded = true.

A later preloadAll therefore returns immediately, so a 404'd wall/floor/tent never retries for the rest of the session. get() stays undefined; getOrLoad may start a sync loader that also fails silently.

Pinned by texture-cache.spec.ts ("preloadAll still marks isPreloaded after a per-texture failure and does not retry").

try {
  await this.loadTexture(path);
} catch (error) {
  console.warn(`Failed to preload texture: ${path}`, error);
}
// ...
this.isPreloaded = true;

`TextureCache.preloadAll` swallows per-texture failures (`console.warn` + continue) and then unconditionally sets `isPreloaded = true`. A later `preloadAll` therefore returns immediately, so a 404'd wall/floor/tent never retries for the rest of the session. `get()` stays undefined; `getOrLoad` may start a *sync* loader that also fails silently. Pinned by `texture-cache.spec.ts` ("preloadAll still marks isPreloaded after a per-texture failure and does not retry"). ```ts try { await this.loadTexture(path); } catch (error) { console.warn(`Failed to preload texture: ${path}`, error); } // ... this.isPreloaded = true; ```

spikerj commented

2026-08-14 04:24:39 +00:00

Fixed in spikersoft-angular PR #823 (a0f0ccd9). Closing.

Fixed in `spikersoft-angular` [PR #823](https://git.spikersoft.com/spikerj/spikersoft-angular/pulls/823) (`a0f0ccd9`). Closing.

spikerj closed this issue

2026-08-14 04:24:46 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: spikerj/spikersoft-issues#1045