Cazador talento id: 155228 [Lobo solitario] bug

[yudian1944]

Función: Cuando escoges este talento tu daño aumenta un 18% pero a cambio dejas de invocar a tu mascota.
Error: Cuando tienes el talento activo puedes usar [Domesticar bestia]id:1515, Domesticar una nueva mascota y seguir usándola.

[Elgrimm]

add PlayerScripts
void OnUpdate(Player* player, uint32 diff) override
{
if (player->HasSpell(155228))
{
player->RemoveAurasDueToSpell(1515);
}
}

[Elgrimm]

@yudian1944
class playerScriptLearnAndRemoveNew : public PlayerScript
{
public:
playerScriptLearnAndRemoveNew () : PlayerScript("playerScriptLearnAndRemoveNew ") { }

void OnUpdate(Player* player, uint32 diff) override
{
if (player->HasSpell(155228))
{
player->RemoveAurasDueToSpell(1515);
}
}
};

[Elgrimm]

in ->
void AddSC_player_special_scripts()
{
new playerScriptLearnNew();
...........
};

[yudian1944]

in -> void AddSC_player_special_scripts() { new playerScriptLearnNew(); ........... };

Deberia ser asi: new playerScriptLearnAndRemoveNew();

[yudian1944]

Ya añadí las líneas, y con el talento no deja domesticar nuevas mascotas pero el problema no viene solo por ahí, también sucede que cuando corregimos el issue #131 al modificar estas líneas :
*`void Spell::EffectSummonDeadPet(SpellEffIndex /effIndex/)
{...
Pet pet = player->GetPet();
-- if (!pet)
++ if (pet && pet->isAlive())
++ return;
-- return;

-- if (pet->isAlive())
-- return;
if (damage < 0)
return;

float x, y, z;
player->GetPosition(x, y, z);

++ if (!pet)
++ {
++ player->SummonPet(0, x, y, z, player->GetOrientation(), SUMMON_PET, 0);
++ pet = player->GetPet();
++ }
++ if (!pet)
++ return;
player->GetMap()->CreatureRelocation(pet, x, y, z, player->GetOrientation());`*

Spell Id: 982 [Revivir mascota] ahora está invocando la mascota, o sea está haciendo la función de (llamar a mascota)

[Elgrimm]

@yudian1944 You can register in playerscript
in \src\server\scripts\World\player_special.cpp
........

• class playerScriptLearnAndRemoveNew : public PlayerScript

• {

• public:

• playerScriptLearnAndRemoveNew () : PlayerScript("playerScriptLearnAndRemoveNew ") { }

• void OnUpdate(Player* player, uint32 diff) override

• {

• if (player->HasSpell(155228))

• {

• player->RemoveAurasDueToSpell(1515);

• player->RemoveAurasDueToSpell(982);

• }

• }

• };
  void AddSC_player_special_scripts()
  {
  .....

• new playerScriptLearnNew();
  ...........
  };

[Elgrimm]

This repository is abandoned, apparently I will make a new branch soon and I will fix it there already

[yudian1944]

[Elgrimm]

@yudian1944
void AddSC_player_special_scripts()
{
.....
delete- -> new playerScriptLearnNew();
++ new playerScriptLearnAndRemoveNew();
...........
};

[yudian1944]

Ya lo he hecho pero aún sigue sin funcionar te dejo un video para que veas donde está el problema https://youtu.be/GgRMBHyfcW0

[Elgrimm]

@yudian1944
-- void OnUpdate(Player* player, uint32 diff) override
++ void OnSpellLearned(Player* player, uint32 spellID) override
{

-- if (player->HasSpell(155228))
++ if (spellID == 155228)

{

-- player->RemoveAurasDueToSpell(1515);
++ player->removeSpell(1515);
-- player->RemoveAurasDueToSpell(982);
++ player->removeSpell(982);

}

}

};
void AddSC_player_special_scripts()

[yudian1944]

@Funciona pero al relogear el cliente los hechizos 1515 y 982 vuelven a estar disponibles y se pueden seguir utilizando con el talento.

[Elgrimm]

@yudian1944 ok, Changing the code by adding the update function

[Elgrimm]

void OnSpellLearned(Player* player, uint32 spellID) override
{
if (spellID == 155228)
{
player->removeSpell(1515);
player->removeSpell(982);

}

}

++ void OnUpdate(Player* player, uint32 diff) override

++ {

++ if (player->HasSpell(155228))

++ {

++ player->removeSpell(1515);
++ player->removeSpell(982);

++ }

++ }

};

[yudian1944]

Bueno ahora si ha quedado solucionado el uso de este bug, gracias nuevamente @Elgrimm por el arreglo y sierro tema.