Fix TextureEffect crash when clearing an existing texture

src/effects/TextureEffect.js

@@ -60,11 +60,12 @@ export class TextureEffect extends Effect {
 		if(prevTexture !== value) {
 
 			uniforms.get("map").value = value;
-			uniforms.get("uvTransform").value = value.matrix;
 			defines.delete("TEXTURE_PRECISION_HIGH");
 
 			if(value !== null) {
 
+				uniforms.get("uvTransform").value = value.matrix;
+
 				if(value.matrixAutoUpdate) {
 
 					defines.set("UV_TRANSFORM", "1");
@@ -89,6 +90,12 @@ export class TextureEffect extends Effect {
 
 				}
 
+			} else {
+
+				uniforms.get("uvTransform").value = null;
+				defines.delete("UV_TRANSFORM");
+				this.setVertexShader(null);
+
 			}
 
 		}

test/effects/TextureEffect.js

@@ -1,5 +1,6 @@
 import test from "ava";
-import { TextureEffect } from "postprocessing";
+import { Texture } from "three";
+import { EffectPass, TextureEffect } from "postprocessing";
 
 test("can be created and destroyed", t => {
 
@@ -16,3 +17,28 @@ test("can be updated", t => {
 	t.notThrows(() => effect.update());
 
 });
+
+test("can clear an existing texture", t => {
+
+	const effect = new TextureEffect({ texture: new Texture() });
+	t.notThrows(() => { effect.texture = null; });
+	t.is(effect.texture, null);
+
+});
+
+test("recompiles its host pass after clearing an existing texture", t => {
+
+	const texture = new Texture();
+	texture.matrixAutoUpdate = true;
+
+	const effect = new TextureEffect({ texture });
+	const pass = new EffectPass(null, effect);
+	pass.initialize(null, false, undefined);
+
+	t.true(pass.fullscreenMaterial.vertexShader.includes("UvTransform"));
+
+	t.notThrows(() => { effect.texture = null; });
+
+	t.false(pass.fullscreenMaterial.vertexShader.includes("UvTransform"));
+
+});