TypeError: Cannot read property 'forEach' of undefined

Hello there. General Kenobi.

I created a PNG Thumbnails Batch Renderer with your awesome project and sometimes it stops just because source model file has something like ‘problems with materials’ i think… BUT. I can see the model… Idk how to organize try catch with model-viewer just with vanilla js… sorry))

Can you handle this case without type error please?

model-viewer.js:56233 Uncaught (in promise) TypeError: Cannot read property 'forEach' of undefined
    at new Sm (model-viewer.js:56233)
    at HTMLElement.[onModelLoad] (model-viewer.js:56342)
    at vc.<anonymous> (model-viewer.js:48752)
    at vc.dispatchEvent (model-viewer.js:1689)
    at vc.setupScene (model-viewer.js:44589)
    at vc.setSource (model-viewer.js:44577)
    at async HTMLElement.[updateSource] (model-viewer.js:49041)
    at async HTMLElement.[updateSource] (model-viewer.js:53334)

Screens:

From:

class Model {
    constructor(correlatedSceneGraph, onUpdate = () => { }) {
        this[_a] = [];
        const { gltf, gltfElementMap } = correlatedSceneGraph;
        gltf.materials.forEach(material => {
            this[$materials].push(new Material(onUpdate, gltf, material, gltfElementMap.get(material)));
        });
    }

source glb (14mb) (not mine) https://cloudflare-ipfs.com/ipfs/QmeqKjz7KLigLis1AJGbuu22uXdgGdNaVneFF1MiDkoaPJ

from object 6137 at hic et nunc gallery.

I’ve checked source file and don’t see any “materials” property.

[elalish]

Since toBlob() returns a promise, I believe you can catch errors this way: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise/catch

[elalish]

This looks like a regression; we have code specifically to handle this case where the glTF only has the default material.