Hopeful fix to items having threaded update called when in inventory

Causeless · Causeless · commit 528c4bc7f601 · 2024-02-08T16:34:43.000Z
diff --git a/Source/Managers/MovableMan.cpp b/Source/Managers/MovableMan.cpp
@@ -1334,7 +1334,9 @@ void MovableMan::Update() {
 			                                                     g_LuaMan.SetThreadLuaStateOverride(&luaState);
 
 			                                                     for (MovableObject* mo: luaState.GetRegisteredMOs()) {
-				                                                     mo->RunScriptedFunctionInAppropriateScripts(threadedUpdate, false, false, {}, {}, {});
+				                                                     if (ValidMO(mo->GetRootParent())) {
+					                                                     mo->RunScriptedFunctionInAppropriateScripts(threadedUpdate, false, false, {}, {}, {});
+				                                                     }
 			                                                     }
 
 			                                                     g_LuaMan.SetThreadLuaStateOverride(nullptr);

Original file line number	Diff line number	Diff line change
`@@ -1334,7 +1334,9 @@ void MovableMan::Update() {`
`1334`	`1334`	`g_LuaMan.SetThreadLuaStateOverride(&luaState);`
`1335`	`1335`
`1336`	`1336`	`for (MovableObject* mo: luaState.GetRegisteredMOs()) {`
`1337`		`- mo->RunScriptedFunctionInAppropriateScripts(threadedUpdate, false, false, {}, {}, {});`
	`1337`	`+ if (ValidMO(mo->GetRootParent())) {`
	`1338`	`+ mo->RunScriptedFunctionInAppropriateScripts(threadedUpdate, false, false, {}, {}, {});`
	`1339`	`+ }`
`1338`	`1340`	`}`
`1339`	`1341`
`1340`	`1342`	`g_LuaMan.SetThreadLuaStateOverride(nullptr);`