Move WhenStrikes skill success to a more appropriate place

apps/openmw/mwclass/npc.cpp

@@ -603,10 +603,7 @@ namespace MWClass
             {
                 MWMechanics::CastSpell cast(ptr, victim);
                 cast.mHitPosition = hitPosition;
-                bool success = cast.cast(weapon);
-
-                if (ptr.getRefData().getHandle() == "player" && success)
-                    skillUsageSucceeded (ptr, ESM::Skill::Enchant, 3);
+                cast.cast(weapon);
             }
         }
 

apps/openmw/mwmechanics/spellcasting.cpp

@@ -554,7 +554,10 @@ namespace MWMechanics
         else if (enchantment->mData.mType != ESM::Enchantment::WhenStrikes)
         {
             if (mCaster.getRefData().getHandle() == "player")
+            {
                 MWBase::Environment::get().getWindowManager()->setSelectedEnchantItem(item); // Set again to show the modified charge
+                mCaster.getClass().skillUsageSucceeded (mCaster, ESM::Skill::Enchant, 3);
+            }
         }
 
         inflict(mCaster, mCaster, enchantment->mEffects, ESM::RT_Self);