Skip to content

Commit

Permalink
It also works for non-permanent pets
Browse files Browse the repository at this point in the history
  • Loading branch information
Pewtro committed Oct 26, 2023
1 parent 1561b86 commit 37249c9
Showing 1 changed file with 0 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ import CritEffectBonus, { ValidEvents } from 'parser/shared/modules/helpers/Crit
import ItemDamageDone from 'parser/ui/ItemDamageDone';
import ItemHealingDone from 'parser/ui/ItemHealingDone';
import StatisticBox from 'parser/ui/StatisticBox';
import { isPermanentPet } from '../../pets/helpers';

export const CRIT_EFFECT = 0.02;

Expand Down Expand Up @@ -91,13 +90,6 @@ class MightOfTheMountain extends Analyzer {
if (!this.isApplicableDamage(event)) {
return;
}
const sourcePet = this.owner.playerPets.find(
(pet: { id: number | undefined }) => pet.id === event.sourceID,
);

if (!sourcePet || !isPermanentPet(sourcePet.guid)) {
return;
}

//These effects (dwarf & tauren crit mod) apply to both the player and the pet via Apply Player/Pet Aura (202) and the pet inherits from the
// player, effectively getting double the mod.
Expand Down

0 comments on commit 37249c9

Please sign in to comment.