Fix bug in LuaUi::Element::destroy() that sometimes leads to an infin…

…ite loop on UI cleanup
OpenMW · May 20, 2023 · c6eed2a · c6eed2a
1 parent 5f6ca2a
commit c6eed2a
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/components/lua_ui/element.cpp b/components/lua_ui/element.cpp
@@ -216,12 +216,12 @@ namespace LuaUi
 
     void Element::destroy()
     {
+        sAllElements.erase(this);
         if (!mRoot)
             return;
         destroyWidget(mRoot);
         mRoot = nullptr;
         mLayout = sol::make_object(mLayout.lua_state(), sol::nil);
-        sAllElements.erase(this);
     }
 
     void Element::attachToWidget(WidgetExtension* w)