fix: material destroy bug (#1864)

Author: zhuxudong

packages/core/src/mesh/MeshRenderer.ts

@@ -87,6 +87,10 @@ export class MeshRenderer extends Renderer {
       Logger.error("mesh is null.");
       return;
     }
+    if (this._mesh.destroyed) {
+      Logger.error("mesh is destroyed.");
+      return;
+    }
     super._prepareRender(context);
   }
 

packages/core/src/shader/ShaderProgram.ts

@@ -380,10 +380,11 @@ export class ShaderProgram {
         case gl.BOOL_VEC3:
         case gl.INT_VEC3:
           if (isArray) {
+            shaderUniform.applyFunc = shaderUniform.upload3iv;
           } else {
+            shaderUniform.applyFunc = shaderUniform.upload3i;
+            shaderUniform.cacheValue = new Vector3(0, 0, 0);
           }
-          shaderUniform.applyFunc = isArray ? shaderUniform.upload3iv : shaderUniform.upload3i;
-          shaderUniform.cacheValue = new Vector3(0, 0, 0);
           break;
         case gl.BOOL_VEC4:
         case gl.INT_VEC4:

packages/core/src/sky/Sky.ts

@@ -64,13 +64,25 @@ export class Sky {
       Logger.warn("The material of sky is not defined.");
       return;
     }
+
+    if (material.destroyed) {
+      Logger.warn("The material of sky is destroyed.");
+      return;
+    }
+
     if (!mesh) {
       Logger.warn("The mesh of sky is not defined.");
       return;
     }
 
-    const { engine, aspectRatio, fieldOfView, viewMatrix, shaderData: cameraShaderData } = context.camera;
-    const sceneData = context.camera.scene.shaderData;
+    if (mesh.destroyed) {
+      Logger.warn("The mesh of sky is destroyed.");
+      return;
+    }
+
+    const { engine, scene, aspectRatio, fieldOfView, viewMatrix, shaderData: cameraShaderData } = context.camera;
+    const sceneData = scene.shaderData;
+
     const { _viewProjMatrix: viewProjMatrix, _projectionMatrix: projectionMatrix } = Sky;
     const rhi = engine._hardwareRenderer;
     const { shaderData: materialShaderData, shader, renderState } = material;