Fix some idiosyncracies with animation damage

- Adjust Damage.IsDealtByInvoker=true code path fallbacks for house dealing damage: InvokerHouse -> invoker's owner -> anim owner -> fallbacks mentioned in second bullet
- Restore fallback for house dealing damage to use owner objects (attachee) or parent building's owning house if available, separately from invoker logic
- Clarify some of the behaviour in docs

Author: Starkku

docs/Fixed-or-Improved-Logics.md

@@ -328,7 +328,7 @@ LandingDir=     ; Direction type (integers from 0-255). Accepts negative values
 
 - `Weapon` can be set to a WeaponType, to create a projectile and immediately detonate it instead of simply dealing `Damage` by `Warhead`. This allows weapon effects to be applied.
 - `Damage.Delay` determines delay between two applications of `Damage`. Requires `Damage` to be set to 1.0 or above. Value of 0 disables the delay. Keep in mind that this is measured in animation frames, not game frames. Depending on `Rate`, animation may or may not advance animation frames on every game frame.
-- `Damage.DealtByInvoker`, if set to true, makes any `Damage` dealt to be considered as coming from the animation's invoker (f.ex, firer of the weapon if it is Warhead `AnimList/SplashList` animation, the destroyed vehicle if it is `DestroyAnim` animation or the object the animation is attached to). If invoker has died or does not exist, the house the invoker belonged to is still used to deal damage and apply Phobos-introduced Warhead effects. Does not affect which house the `Damage` dealt by `Warhead` is dealt by.
+- `Damage.DealtByInvoker`, if set to true, makes any `Damage` dealt to be considered as coming from the animation's invoker (f.ex, firer of the weapon if it is Warhead `AnimList/SplashList` animation, the destroyed vehicle if it is `DestroyAnim` animation or the object the animation is attached to). If invoker has died or does not exist, the house the invoker belonged to is still used to deal damage and apply Phobos-introduced Warhead effects etc. If not set, the animation's owner house, or failing that, owning house of object it is attached to or the building it belongs to is used to deal the damage.
   - `Damage.ApplyFirepowerMult` determines whether or not firepower modifiers from the animation's invoker are applied on the damage dealt from this animation, if exists.
 - `Damage.ApplyOncePerLoop`, if set to true, makes `Damage` be dealt only once per animation loop (on single loop animations, only once, period) instead of on every frame or intervals defined by `Damage.Delay`. The frame on which it is dealt is determined by `Damage.Delay`, defaulting to after the first animation frame.
 

src/Ext/Anim/Hooks.cpp

@@ -100,9 +100,6 @@ DEFINE_HOOK(0x42453E, AnimClass_AI_Damage, 0x6)
 		const auto pExt = AnimExt::ExtMap.Find(pThis);
 		pInvoker = pExt->Invoker;
 
-		if (pExt->InvokerHouse)
-			pOwner = pExt->InvokerHouse;
-
 		if (!pInvoker)
 		{
 			if (pThis->OwnerObject)
@@ -111,16 +108,29 @@ DEFINE_HOOK(0x42453E, AnimClass_AI_Damage, 0x6)
 				pInvoker = pExt->ParentBuilding;
 		}
 
+		if (pExt->InvokerHouse)
+			pOwner = pExt->InvokerHouse;
+
 		if (pInvoker)
 		{
-			if (!pOwner)
+			if (!pExt->InvokerHouse)
 				pOwner = pInvoker->Owner;
 
 			if (pTypeExt->Damage_ApplyFirepowerMult)
 				appliedDamage = static_cast<int>(appliedDamage * pInvoker->FirepowerMultiplier * TechnoExt::ExtMap.Find(pInvoker)->AE.FirepowerMultiplier);
 		}
 	}
 
+	// Jun 29, 2025 - Starkku: Owner != Invoker. Previously OwnerObject / ParentBuilding fallback only existed for Warheads
+	// but if we are unifying the approaches it needs to be available even without and separately from invoker.
+	if (!pOwner)
+	{
+		if (pThis->OwnerObject)
+			pOwner = pThis->OwnerObject->GetOwningHouse();
+		else if (pThis->IsBuildingAnim)
+			pOwner = AnimExt::ExtMap.Find(pThis)->ParentBuilding->Owner;
+	}
+
 	if (pTypeExt->Weapon)
 	{
 		WeaponTypeExt::DetonateAt(pTypeExt->Weapon, pThis->GetCoords(), pInvoker, appliedDamage, pOwner);