[patch] Shaman T10 Elemental 2P Bonus

[Den]

Кое-какая информация по опкоду SMSG_MODIFY_COOLDOWN , была взята отсюда: http://ru-mangos.ru/showthread.php?t=2708

Автор: ?
diff --git a/src/game/UnitAuraProcHandler.cpp b/src/game/UnitAuraProcHandler.cpp
index 92347ec..024a02b 100644
--- a/src/game/UnitAuraProcHandler.cpp
+++ b/src/game/UnitAuraProcHandler.cpp
@@ -2385,6 +2385,23 @@ SpellAuraProcResult Unit::HandleDummyAuraProc(Unit *pVictim, uint32 damage, Aura
basepoints[0] = int32(triggerAmount * damage / 100) / GetSpellAuraMaxTicks(triggered_spell_id);
break;
}
+ // Item - Shaman T10 Elemental 2P Bonus
+ case 70811:
+ {
+ // Lightning Bolt == 0x0000000000000001, Chain Lightning == 0x0000000000000002
+ if (procSpell->IsFitToFamilyMask(UI64LIT(0x0000000000000003)))
+ {
+ uint32 cooldown = ((Player*)this)->GetSpellCooldownDelay(16166);
+ ((Player*)this)->AddSpellCooldown(16166, 0, uint32(time(NULL) + cooldown));
+
+ WorldPacket data(SMSG_MODIFY_COOLDOWN, 4 + 8 + 4);
+ data << uint32(16166);
+ data << uint64(GetObjectGuid());
+ data << int32(-2000);
+ ((Player*)this)->GetSession()->SendPacket(&data);
+ return SPELL_AURA_PROC_OK;
+ }
+ }
}
// Storm, Earth and Fire
if (dummySpell->SpellIconID == 3063)

[Amaru]

не понятно.

спеллу выставляется такой же кулдаун, какой и был.
клиенту высылается обновление кулдауна, хотя это не произошло.
и что клиент будет думать, если оставшееся кд, например, 1 секунда, а ему приходит изменение на -2 сек?
конверт uint64(GetObjectGuid()) вроде бы не нужен, не так ли?
зачем обрабатывать прок, если кд на спелле нет?
не хватает break в case

2000 хранится в basepoints, а прок можно было бы запилить в spell_proc_event

[Den]

Ну попробуйте.

зачем break после return ? О_о

add
а прок можно было бы запилить в spell_proc_event

имхо лишние записи в бд, не к чему .

[Vipertv]

До использования таких вещей как ((Player*)this), возможно стоит проверять на соответствие GetTypeId к TYPEID_PLAYER. Это позволит избежать хоть и маловероятного но все-таки краша.

[Den]

Хм... это не мое "творение"... патч с ссылкой на http://ru-mangos.ru/showthread.php?t=2708 случайно залетел в папку /мои_патчи, и при чем довольна таки давно лежал там. Может быть с гетмангоса или с форка какого.

[Amaru]

Ну попробуйте.
зачем break после return ? О_о.
потому что если спелл не чайник и не болт, то при наличии кода после этого кейса на прок, свитч провалится дальше.

имхо, треш а не патч :bad:

[HuntsMan]

А почему бы момент с отсылкой пакета SMSG_MODIFY_COOLDOWN не перенести в AddSpellCooldown?