ProjectSkyfire/SkyFireEMU_rebase — comment on commit 036f8fb1688b193c5bbd868b641b9248b7d5b3ce
File:  line 0
By Taff:

I tested this, it has the same problem as when i tried to fix it.
First, sic 'em rank 2 doesn't proc off.
Second problem, sic 'em rank 1, gives the pet every basic attack to cost reduced focus for entire aura duration.

i managed to get sic 'em rank 2 proc off, but it requires to write code in the core, this is what i've done so far:
unit.cpp
// Sic 'Em Rank 1
if (dummySpell->Id == 53340)
{
triggered_spell_id = 83359;
target = this;
break;
}
// Sic 'Em Rank 2
if (dummySpell->Id == 83356)
{
triggered_spell_id = 89388;
target = this;
break;
}

this way sic 'em rank 2 procs off too, but it has the same problem as rank 1, the focus cost is reduced by 100% not just for 1 attack, but for every basic attack until the aura duration is off.

This was one of the fixes i was about to post, but it wasn't complete.

what you think?