From 56d06ad1c1469cf3a97d4dedd4d93e05579d64a7 Mon Sep 17 00:00:00 2001
From: LemonInTheDark <58055496+LemonInTheDark@users.noreply.github.com>
Date: Tue, 10 Sep 2024 14:16:18 -0700
Subject: [PATCH] Actually fixes MA issue w ifdefs (#86579)

## About The Pull Request

I FORGOT TO COMMIT AHHHH

#86538 WAS JUST THE REMOVAL
---
 .../debug_variable_appearance.dm              | 25 +++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/code/modules/admin/view_variables/debug_variable_appearance.dm b/code/modules/admin/view_variables/debug_variable_appearance.dm
index 6d87f6cc85d..c5a367e83a0 100644
--- a/code/modules/admin/view_variables/debug_variable_appearance.dm
+++ b/code/modules/admin/view_variables/debug_variable_appearance.dm
@@ -38,6 +38,9 @@
 	// can't use the name either for byond reasons
 	var/_vis_flags
 
+#if (MIN_COMPILER_VERSION > 515 || MIN_COMPILER_BUILD > 1643)
+#warn vis_flags should now be supported by mutable appearances so we can safely remove the weird copying in this code
+#endif
 // all alone at the end of the universe
 GLOBAL_DATUM_INIT(pluto, /atom/movable, new /atom/movable(null))
 
@@ -63,6 +66,28 @@ GLOBAL_DATUM_INIT(pluto, /atom/movable, new /atom/movable(null))
 		return FALSE
 	if(var_name == NAMEOF(src, realized_underlays))
 		return FALSE
+
+#if (MIN_COMPILER_VERSION >= 515 && MIN_COMPILER_BUILD >= 1643)
+#warn X/Y/Z and contents are now fully unviewable on our supported versions, remove the below check
+#endif
+
+// lummy removed these from the the MA/image type
+#if (DM_VERSION <= 515 && DM_BUILD < 1643)
+	// Filtering out the stuff I know we don't care about
+	if(var_name == NAMEOF(src, x))
+		return FALSE
+	if(var_name == NAMEOF(src, y))
+		return FALSE
+	if(var_name == NAMEOF(src, z))
+		return FALSE
+	#ifndef SPACEMAN_DMM // Spaceman doesn't believe in contents on appearances, sorry lads
+	if(var_name == NAMEOF(src, contents))
+		return FALSE
+	#endif
+	if(var_name == NAMEOF(src, loc))
+		return FALSE
+#endif
+	// Could make an argument for this but I think they will just confuse people, so yeeet
 	if(var_name == NAMEOF(src, vis_contents))
 		return FALSE
 	return ..()