@woosh/meep-engine 2.42.4 → 2.42.5

package/engine/graphics/micron/render/v1/MaterialContext.js

@@ -69,7 +69,7 @@ export class MaterialContext {
 
         const rewrite_material = this.adapter.rewriteMaterial.bind(this.adapter);
 
-        if (material[MATERIAL_FLAG] === undefined) {
+        if (this.material[MATERIAL_FLAG] === undefined) {
             material.onBeforeCompile = composeCompile(this.material.onBeforeCompile, rewrite_material);
             material.needsUpdate = true;
 

package/engine/graphics/render/forward_plus/plugin/MaterialTransformer.js

@@ -6,6 +6,12 @@ import { fp_build_fragment_shader } from "../materials/fp_build_fragment_shader.
 import { fp_build_vertex_shader } from "../materials/fp_build_vertex_shader.js";
 import { fp_build_vertex_lighting_shared } from "../materials/fp_build_vertex_lighting_shared.js";
 
+/**
+ * @readonly
+ * @type {string}
+ */
+const PROPERTY_TRANSFORMER_MARKER = '@forward-plus-material-transformer';
+
 export class MaterialTransformer extends AbstractMaterialTransformer {
 
     /**
@@ -88,6 +94,17 @@ export class MaterialTransformer extends AbstractMaterialTransformer {
     }
 
     transform(source) {
+        if (source.hasOwnProperty(PROPERTY_TRANSFORMER_MARKER)) {
+            // already transformed
+
+            if (source[PROPERTY_TRANSFORMER_MARKER] !== this) {
+                throw new Error('The material is already transformed, but is associated with a different transformer instance');
+            } else {
+                return source;
+            }
+
+        }
+
         let result = source;
 
         if (isLitMaterial(source)) {
@@ -96,12 +113,16 @@ export class MaterialTransformer extends AbstractMaterialTransformer {
             // inherit uniforms directly
 
             if (source.isShaderMaterial) {
-                result.uniforms = source.uniforms;
-                result.defines = source.defines;
+                // TODO use Proxy
+                result.uniforms = Object.assign({}, source.uniforms);
+                result.defines = Object.assign({}, source.defines);
             }
 
             result.onBeforeCompile = composeCompile(source.onBeforeCompile, this.__on_before_compile);
 
+
+            result[PROPERTY_TRANSFORMER_MARKER] = this;
+
         }
 
         return result;

package/package.json

@@ -5,7 +5,7 @@
   "productName": "Meep",
   "description": "production-ready JavaScript game engine based on Entity Component System Architecture",
   "author": "Alexander Goldring",
-  "version": "2.42.4",
+  "version": "2.42.5",
   "dependencies": {
     "gl-matrix": "3.4.3",
     "fast-levenshtein": "2.0.6",