From 2ad54439da07373f2bb5b5f79bfac5e1200c214e Mon Sep 17 00:00:00 2001
From: Sebastian Fieber <fallchildren@disroot.org>
Date: Tue, 29 Jul 2025 23:06:36 +0200
Subject: [PATCH] throw if vfxId is empty

---
 apps/openmw/mwlua/animationbindings.cpp | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/apps/openmw/mwlua/animationbindings.cpp b/apps/openmw/mwlua/animationbindings.cpp
index 07e2c4f133..cce00d49d3 100644
--- a/apps/openmw/mwlua/animationbindings.cpp
+++ b/apps/openmw/mwlua/animationbindings.cpp
@@ -303,8 +303,9 @@ namespace MWLua
         api["remove"] = [context](std::string vfxId) {
             context.mLuaManager->addAction(
                 [vfxId = vfxId] {
-                    if (!vfxId.empty())
-                        MWBase::Environment::get().getWorld()->removeEffect(vfxId);
+                    if (vfxId.empty())
+                        throw std::runtime_error("vfxId is empty");
+                    MWBase::Environment::get().getWorld()->removeEffect(vfxId);
                 },
                 "openmw.vfx.remove");
         };