make setting props on a null renderer only a warning

al2me6 · al2me6 · commit 4b24c3c2b0c6 · 2025-06-10T21:16:11.000-04:00
diff --git a/Source/DynamicProperties/MaterialPropertyManager.cs b/Source/DynamicProperties/MaterialPropertyManager.cs
@@ -46,7 +46,7 @@ private void OnDestroy()
 		MaterialColorUpdaterPatch.temperatureColorProps.Clear();
 		ModuleColorChangerPatch.mccProps.Clear();
 
-		this.LogDebug("destroyed");
+		this.LogMessage("destroyed");
 	}
 
 	#endregion
@@ -56,7 +56,7 @@ private void OnDestroy()
 	public bool Set(Renderer renderer, Props props)
 	{
 		if (renderer == null) {
-			Log.LogError(this, $"cannot set property on null renderer {renderer.GetHashCode()}");
+			this.LogWarning($"cannot set property on null renderer {renderer.GetHashCode()}");
 			return false;
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ private void OnDestroy()`
`46`	`46`	`MaterialColorUpdaterPatch.temperatureColorProps.Clear();`
`47`	`47`	`ModuleColorChangerPatch.mccProps.Clear();`
`48`	`48`
`49`		`- this.LogDebug("destroyed");`
	`49`	`+ this.LogMessage("destroyed");`
`50`	`50`	`}`
`51`	`51`
`52`	`52`	`#endregion`
`@@ -56,7 +56,7 @@ private void OnDestroy()`
`56`	`56`	`public bool Set(Renderer renderer, Props props)`
`57`	`57`	`{`
`58`	`58`	`if (renderer == null) {`
`59`		`- Log.LogError(this, $"cannot set property on null renderer {renderer.GetHashCode()}");`
	`59`	`+ this.LogWarning($"cannot set property on null renderer {renderer.GetHashCode()}");`
`60`	`60`	`return false;`
`61`	`61`	`}`
`62`	`62`