Merge branch 'fatiguecrit' into 'master'

Only play critical hit sound for health damage (#8207)

Closes #8207

See merge request OpenMW/openmw!4427

Author: psi29a

CHANGELOG.md

@@ -198,6 +198,7 @@
     Bug #8172: Openmw-cs crashes when viewing `Dantooine, Sea`
     Bug #8187: Intervention effects should use Chebyshev distance to determine the closest marker
     Bug #8191: NiRollController does not work for sheath meshes
+    Bug #8207: Using hand-to-hand while sneaking plays the critical hit sound when the target is not getting hurt
     Feature #1415: Infinite fall failsafe
     Feature #2566: Handle NAM9 records for manual cell references
     Feature #3501: OpenMW-CS: Instance Editing - Shortcuts for axial locking

apps/openmw/mwclass/npc.cpp

@@ -671,8 +671,11 @@ namespace MWClass
             {
                 damage *= store.find("fCombatCriticalStrikeMult")->mValue.getFloat();
                 MWBase::Environment::get().getWindowManager()->messageBox("#{sTargetCriticalStrike}");
-                MWBase::Environment::get().getSoundManager()->playSound3D(
-                    victim, ESM::RefId::stringRefId("critical damage"), 1.0f, 1.0f);
+                if (healthdmg)
+                {
+                    MWBase::Environment::get().getSoundManager()->playSound3D(
+                        victim, ESM::RefId::stringRefId("critical damage"), 1.0f, 1.0f);
+                }
             }
         }