Add to changelog, authors. Move variable declaration inside block

AUTHORS.md

@@ -96,6 +96,7 @@ Programmers
     Jan Borsodi (am0s)
     Jason Hooks (jhooks)
     jeaye
+    jefetienne
     Jeffrey Haines (Jyby)
     Jengerer
     Jiří Kuneš (kunesj)

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
 

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, "");