From bc6f46465f9d3bf09aebbd44689898673998efe4 Mon Sep 17 00:00:00 2001
From: jefetienne <etienneayoub@gmail.com>
Date: Wed, 18 Nov 2020 14:30:41 -0500
Subject: [PATCH] Add to changelog, authors. Move variable declaration inside
 block

---
 AUTHORS.md                       | 1 +
 CHANGELOG.md                     | 1 +
 apps/openmw/mwgui/spellmodel.cpp | 4 ++--
 3 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/AUTHORS.md b/AUTHORS.md
index 25004078e0..09ab784125 100644
--- a/AUTHORS.md
+++ b/AUTHORS.md
@@ -96,6 +96,7 @@ Programmers
     Jan Borsodi (am0s)
     Jason Hooks (jhooks)
     jeaye
+    jefetienne
     Jeffrey Haines (Jyby)
     Jengerer
     Jiří Kuneš (kunesj)
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 51babd907b..8f909a19fd 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -87,6 +87,7 @@
     Feature #5642: Ability to attach arrows to actor skeleton instead of bow mesh
     Feature #5649: Skyrim SE compressed BSA format support
     Feature #5672: Make stretch menu background configuration more accessible
+    Feature #5692: Improve spell/magic item search to factor in magic effect names
     Task #5480: Drop Qt4 support
     Task #5520: Improve cell name autocompleter implementation
 
diff --git a/apps/openmw/mwgui/spellmodel.cpp b/apps/openmw/mwgui/spellmodel.cpp
index 2ae92e33fe..136547c4c1 100644
--- a/apps/openmw/mwgui/spellmodel.cpp
+++ b/apps/openmw/mwgui/spellmodel.cpp
@@ -51,11 +51,11 @@ namespace MWGui
         for (unsigned int i = 0; i < effects.mList.size(); ++i)
         {
             short effectId = effects.mList[i].mEffectID;
-            const ESM::MagicEffect *magicEffect =
-                store.get<ESM::MagicEffect>().search(effectId);
 
             if (effectId != -1)
             {
+                const ESM::MagicEffect *magicEffect =
+                    store.get<ESM::MagicEffect>().search(effectId);
                 std::string effectIDStr = ESM::MagicEffect::effectIdToString(effectId);
                 std::string fullEffectName = wm->getGameSettingString(effectIDStr, "");