Commit Graph
100 Commits
Author SHA1 Message Date
QAston 5365bd25fa Core/Spells: remove some workarounds about transform spells in corpse explosion script. 2012-02-09 23:27:59 +01:00
QAston 800cf73750 Core/Spells: Fix some problems with spells summoning to many objects. 2012-02-09 20:27:26 +01:00
QAston 012be16796 Merge pull request #4726 from ille/baa2252bd31718e93ccc3e97aa81afeecb5ec9c3
Core/GameObject: implement stealth/invisibility for traps
2012-02-09 09:38:13 -08:00
QAston b4022acdf3 Core/Spell: move conditions check to Spell::CheckCast. 2012-02-09 18:35:52 +01:00
QAston 0b1da6c0ad Core/Db/Conditions: update error log. 2012-02-09 18:18:06 +01:00
QAston e0c699b0f3 Core/Db/Conditions: allow CONDITION_SOURCE_TYPE_ITEM_REQUIRED_TARGET to be used for spells with any spell using unit as explicit target instead of just TARGET_UNIT_TARGET_ANY and TARGET_UNIT_TARGET_ENEMY. 2012-02-09 18:12:47 +01:00
QAston dc326131a3 Core/Entities: don't save aura caster's guid for pets to db. 2012-02-09 17:19:24 +01:00
QAston a27758207f Merge pull request #5096 from devilcoredev/fix_005
Core/AI: Fix a bug on some pets introdouced in commit 14bc551862
2012-02-07 23:30:41 -08:00
QAston 84fa96efc6 Once again - a typo fix in 22bd06fc57 - rush is not the best advisor. 2012-02-05 09:18:39 +01:00
QAston a2494f569e Merge pull request #5013 from PKX/fix_auras
Core/Auras: Fix Pet & Totem aura stacking
2012-02-05 00:14:53 -08:00
QAston 7be73a0667 Merge pull request #5075 from devilcoredev/fix_004
Core/AI: Fix pet and other types of minion pet attack on targets with Crowd Controls.
2012-02-05 00:13:56 -08:00
QAston 22bd06fc57 Typofix. Sorry for that :( 2012-02-04 18:28:56 +01:00
QAston 9d185c5d66 Core/Spells: Fix immunity checks on apply aura - original patch by devilcoredev, changed a bit so immunity checks are grouped together instead of being spread. 2012-02-04 15:27:47 +01:00
QAston 6b12f39b35 Merge pull request #4821 from devilcoredev/fix_001
Core/AuraScript: Add OnDispel and AfterDispel hook to AuraScript
2012-02-04 06:09:57 -08:00
QAston 92ad375864 Core/Entities: clarify comment added in c7ea952a47 2012-02-03 22:56:42 +01:00
QAston 7e99e5f094 Core/Spells:
- rename SpellInfo::IsAOE to SpellInfo::IsTargetingArea due to ambiguity of old name.
- add IsAffectingArea function to check whenever spell or it's effects are affecting objects in area and use new function to fix problems with hunter flares.
Closes #3310. Thanks to Kaelima and Warpten for research.
2012-02-03 22:24:03 +01:00
QAston aae833a759 Core/Spells: fix GameObject spell cast on enemies, original patch by Svannon, thanks to horn for research. 2012-02-03 20:07:57 +01:00
QAston c7ea952a47 Core/Entities: Rename UNIT_FLAG_OOC_NOT_ATTACKABLE and UNIT_FLAG_PASSIVE to UNIT_FLAG_IMMUNE_TO_PC and UNIT_FLAG_IMMUNE_TO_NPC respectively. New names tell much more about what those flags do. 2012-02-03 18:46:05 +01:00
QAston ad4cf93e08 Merge pull request #4785 from Warpten/lastStanding
Core/Arenas: Fix achievement [The Last Standing].
2012-01-29 05:50:16 -08:00
QAston 72359b3d11 Db/Quests: fix q13129 quest reward - by Jildor - closes #4633 2012-01-29 14:43:11 +01:00
QAston 1c3064080d Db/Spells: Correct spell_bonus_data for Empowered Renew - by Wallker, sql formatted by kandera:) 2012-01-29 14:00:05 +01:00
QAston 3b80efe204 Core/Spells: Fix spell Gushing Wound - by kandera - closes #4628. 2012-01-29 13:52:52 +01:00
QAston b754f7c91d Core/Auras: Add some documentation to new proc system code to avoid confusion. 2012-01-17 21:35:27 +01:00
QAston 798677ca54 Merge pull request #4800 from Chaplain/cleanup
Core/Spells: Fix issue in CAST_FLAG_ADJUST_MISSILE handling. Thx rsa (R2) for pointing.
2012-01-14 04:43:08 -08:00
QAston 95ad9a992d Merge pull request #4771 from devilcoredev/fix_001
Core/Spells: Fix the 'Trueshot Aura' on pet
2012-01-12 08:33:32 -08:00
QAston afc18461d7 Core/SpellScripts: Add BeforeCast, OnCast and AfterCast hooks to SpellScripts. 2011-12-31 13:11:12 +01:00
QAston 15bd8f3cd5 Core/Spells: Fix logic in AnyDeadUnitSpellTargetInRangeCheck - spells using this should now correctly check target requirements. This is an alternative fix to #4112. 2011-12-24 08:40:30 +01:00
QAston eecc4b58a7 Core/Spells: Use better check for target type in Spell::EffectSendEvent - SpellEffect::GetProvidedTargetMask() is much more accurate than hardcoded dbc index. 2011-12-23 23:52:22 +01:00
QAston 8dd88fa43c Core/Spells: fix a typo which caused to display unneeded error message (invalid target) during cast of some spells (like warrior charge). 2011-09-28 18:59:55 +02:00
QAston 5f043e6291 Core/Spells: Bring back workaround removed in cd7060c65a - it's needed for compatibility with some scripts. 2011-09-27 22:04:26 +02:00
QAston 7fa2a64290 Merge branch 'master' of https://github.com/Chaplain/TrinityCore 2011-09-27 19:19:25 +02:00
QAston ee2af420e7 Merge pull request #3235 from Chaplain/cleanup
Core/Spell: Hymn of Hope fix.
2011-09-27 10:11:35 -07:00
QAston 23ba776767 Core/Entities: use Unit::GetReactionTo in Unit::IsFriendlyTo and Unit::IsHostileTo 2011-09-27 01:29:20 +02:00
QAston 75aeb15e90 Core/Spells: Allow AOE spells to hit stealthed targets. 2011-09-27 01:13:32 +02:00
QAston a409cc226c Core/Spells: Fix Vanish's stealth trigger and Demonic Empowerment's invisibility trigger
Those spells were failing to pass default effect trigger spell due to their custom implementation (changing triggered_spell_id instead of adding serverside spells with proper targeting).
2011-09-27 00:36:23 +02:00
QAston a2633bf7fc Scripts: fix spell_generic_quest_update_entry script after e949ad3adb. 2011-09-27 00:12:45 +02:00
QAston 210a770958 Core/Spells: Fix totem passive area auras stacking bug. 2011-09-25 19:00:37 +03:00
QAston e949ad3adb Core/Spells: spell effect handling improvements
* Call spell effect handlers in 4 modes:
   - SPELL_EFFECT_HANDLE_LAUNCH - called when spell is launched (cast just finished)
   - SPELL_EFFECT_HANDLE_LAUNCH_TARGET - called when spell is launched for each target in spell target map
   - SPELL_EFFECT_HANDLE_HIT - called when spell hits its destination
   - SPELL_EFFECT_HANDLE_HIT_TARGET - called when spell hits it's target from spell target map
*Correctly implement SPELL_EFFECT_TRIGGER_SPELL, SPELL_EFFECT_TRIGGER_SPELL_WITH_VALUE, SPELL_EFFECT_TRIGGER_MISSILE_SPELL_WITH_VALUE, SPELL_EFFECT_TRIGGER_MISSILE_SPELL
*Remove spell system hacks which became obsolete with this commit
Core/SpellScripts:
add OnEffectLaunch, OnEffectLaunchTarget, OnEffectHit, OnEffectHitTarget hooks for new effect handle modes and remove OnEffect hook.
A generic rule of thumb how to update your scripts (will work for nearly all cases) for spell system noobs:
if your spell script used GetHitXXXX function, you need to use OnEffectHitTarget, otherwise use
OnEffectHit
2011-09-25 13:29:50 +02:00
QAston cd7060c65a Core/Spells: Add more generic overload for Unit::CastSpell function and make all cast functions use it as a base. 2011-09-24 16:41:50 +02:00
QAston d01a761f53 Core/Spells: Drop EFFECT_IMPLICIT_TARGET_PET and use EFFECT_IMPLICIT_TARGET_EXPLICIT instead. 2011-09-24 02:02:39 +02:00
QAston 71bbbaa21b Merge pull request #3146 from Havenard/fix0001
Reimplement WorldObject::IsInBetween() to work as intented.
2011-09-23 06:04:17 -07:00
QAston 413b542cd9 Add a tempfix for trigger missile spell effects after rev 13a1d90130. 2011-09-23 13:09:31 +02:00
QAston 13a1d90130 Core/Spells: Improvements in selecting default targets based on effect type. 2011-09-23 11:58:40 +02:00
QAston 29fef71137 Merge pull request #2915 from idostyle/armorPen
Core/Mechanics: Cap armorPen to maxArmorPen - prevents having armor penetration rating greater than 100%.
2011-09-22 02:03:14 -07:00
QAston a0030ef334 Really fix build (sorry for that:() 2011-09-21 11:06:31 +02:00
QAston 87d9a1d384 Scripts/TwinValkyr: Fix build. 2011-09-21 10:40:11 +02:00
QAston 1881765540 Correct names of todays update SQLs. 2011-09-21 10:37:49 +02:00
QAston e2e784e7ea Merge pull request #3086 from LihO/master
Core/Groups: Refactored sending of update packets.
Closes #1042.
2011-09-21 01:05:53 -07:00
QAston d628145d95 Merge pull request #3143 from megamage/master
Threat manager. Fix crashes #2396 and #2992.
2011-09-21 01:03:53 -07:00
QAston 82bc917265 Merge pull request #2908 from Drethek/TwinsValk
Scripts:
Twins Valk - Cleanup / Remove Hardcoded Bullet Spawn (Need DB Support - Stalkers Spawns)
Acidmaw - Use Correct Submerge Spell
Trial Of Crusader - Correct Coordinate Spawn
Trial Of Crusader - Correct Open/Close Main Door Timer
Trial Of Crusader - Open/Close On Boss EnterEvadeMode
2011-09-21 00:27:20 -07:00
QAston 8c2e58d036 Core/Entities: Allow 2 side parties to work. 2011-09-17 13:45:07 +02:00
QAston a2d9aabbb3 Core/Entities: Allow creatures to attack players while in sanctuary. 2011-09-17 01:50:53 +02:00
QAston fe25ce4fa3 Core/AI: Replace many Unit::IsHostileTo with Unit::IsValidAttackTarget or Creature::canCreatureAttack. 2011-09-17 00:56:59 +02:00
QAston 580264b5a5 Core/Entities: Remove obsolete function Unit::canAttack. 2011-09-16 23:51:01 +02:00
QAston c0b1823524 Core/Entities: Replace Unit::canAttack usage with more accurate Unit::IsValidAttackTarget 2011-09-16 23:49:31 +02:00
QAston 3a3f2d927f Core/Misc: Whitespace and a minor typo fix (thanks Kapoeira) 2011-09-16 22:37:49 +02:00
QAston 9bbb4ef583 Core/Spells: Use Unit::IsValidAttack/AssistTarget functions to validate spell explicit targets. 2011-09-16 22:17:49 +02:00
QAston 31e755c291 Core/Entities: Add Unit::IsValidAssistTarget and Unit::IsValidAttackTarget functions for proper serverside checks of related entity flags. 2011-09-16 22:17:48 +02:00
QAston e07535c3e3 Core/Entities: Add new functions: Unit::GetReactionTo and Unit::GetFactionReactionTo for more direct reaction checks than Unit::IsFriendly/HostileTo (those functions have many duplicated code and have many checks unrelated to unit reaction). 2011-09-15 17:44:03 +02:00
QAston decb20f079 Merge pull request #2991 from megamage/master
Core/Entities: Do not call AddObjectToRemoveList(Pet*) in Map::MoveAllCreaturesInMoveList() - Closes #2892.
Scripts: Fix a crash in npc_brunnhildar_prisoner script. Closes #2825.
2011-09-14 14:27:21 -07:00
QAston ec7da6f911 Core/ScriptedAI: remove ScriptedAI::CanCast function which is not functioning properly (compare that func with Spell::CheckCast to see how it should work) and is unused. Closes #2275. 2011-09-12 00:20:37 +02:00
QAston 1d38ccaca6 Core/Spells: Remove redundant explicit target checks from Spell::cast 2011-09-11 22:42:29 +02:00
QAston 606fd07cb1 Merge pull request #2966 from Chaplain/cleanup
Core/Auras: Improvements in SPELL_AURA_MOD_DAMAGE_PERCENT_DONE handling:
2011-09-11 13:29:50 -07:00
QAston 7c85617847 Core/Spells: Don't check cast requirements twice for instant cast spells. 2011-09-11 22:27:21 +02:00
QAston 06b9ae6e11 Core/Unit: Define UNIT_FLAG_UNK_0 as UNIT_FLAG_SERVER_CONTROLLED and remove related redundant creature_template data. 2011-09-11 11:41:55 +02:00
QAston 1fef67187b Core/Protocol: Define more opcode names - thanks to TOM_RUS. 2011-09-11 11:41:55 +02:00
QAston dad9d07045 Merge pull request #2920 from Chaplain/cleanup
Core/Auras: Improvements in SPELL_AURA_MOD_DAMAGE_PERCENT_DONE handling: 
* Weapon specific case for MOD_DAMAGE_DONE_PCT shouldn't change PLAYER_FIELD_MOD_DAMAGE_DONE_PCT, because PLAYER_FIELD_MOD_DAMAGE_DONE_PCT applyed for _all_ damage that deal target.
* Fixed applying SPELL_AURA_MOD_DAMAGE_PERCENT_DONE for SpellDamageBonus.
2011-09-06 02:37:21 -07:00
QAston 94126e0369 Core/Spells: Fix some talents with incorrect target entries in dbcs. This is an alternative fix for #2848. Closes #2846 Closes #2848. 2011-09-06 11:21:40 +02:00
QAston fab78d5e02 Scripts/Vault of Archavon: Earth, Wind & Fire achievement - by idostyle@zoit, with some spell attribute correction to the patch by me. 2011-09-03 21:16:12 +02:00
QAston 81a841ed20 Database/Spells: Add AttributesEx6 and AttributesEx7 columns to spell_dbc table. 2011-09-03 21:05:26 +02:00
QAston ddacd0de94 Core/Spells: Fix pick lock - thanks to cyberbrest for pointing the reason of this issue. 2011-09-03 20:16:15 +02:00
QAston f31e676199 Fix build after recent pull merges. 2011-09-03 18:07:27 +02:00
QAston df19a0c31f Correct sql file naming for b40bf69319. 2011-09-03 17:49:53 +02:00
QAston dc01541111 Merge pull request #2852 from Chaplain/cleanup2
Core/Misc: Cleanup - remove unused variables, add some const modifiers to functions, fix process of args in vmap3_extractor
2011-09-03 08:34:02 -07:00
QAston 82e7b11df1 Merge pull request #2778 from KoffeinKaio/master
Fixed spellid's from Zul'Gurub encounters High Priest Thekal and Bloodlord Mandokir (&adds).
2011-09-03 08:27:37 -07:00
QAston d23b03271d Merge pull request #2777 from KoffeinKaio/fishing
Core/Spells: Fixed fishing bobber landing angle.
2011-09-03 07:52:47 -07:00
QAston 7871ead798 Merge pull request 'Fix quest New Recruit(13143) & Fix quest King of the Mountain(13280&13283)' from Kiperr/TrinityCore 2011-09-03 16:44:05 +02:00
QAston 414f048bd5 Merge pull request #2735 from hacknowledge/fix_2734
Core/Spells: Fixes ammo consumption for Hunter's Volley
2011-09-03 07:29:31 -07:00
QAston 0ba79e0078 Merge pull request: The Val'kyr Twins Clean up and Implements from 'Drethek/master' 2011-09-03 16:26:49 +02:00
QAston 68037cdc3c Scripts/Commands: Allow c++ style comments in opcode.txt file for .debug send opcode command. 2011-09-01 10:46:15 +02:00
QAston 71b9e0d8de Core/Spells: Make mind sear aoe no longet target unit at which caster is channeling the spell. 2011-09-01 09:07:14 +02:00
QAston f16520fa25 Merge pull request #2838 from wonopon/patch-7
Rotface should only cast Mutated Infection on one player.
2011-08-31 14:34:38 -07:00
QAston 534ca8e23f Merge pull request #2821 from Chaplain/cleanup2
Core/Misc: Cleanup
2011-08-31 11:33:26 -07:00
QAston 048ae9cfab Merge pull request #2792 from wonopon/patch-6
Scripts/Taldaram: Make the boss cast Bloodthirst on self instead of on victim.
2011-08-31 11:31:11 -07:00
QAston ac9701afa7 Merge pull request #2762 from Kapoeira/master
Core/Spells: Prevent deterrance from parrying spells that require the attacker to be behind the hunter, like Shred or Ambush.
2011-08-31 11:24:45 -07:00
QAston d863562507 Core/Spells: Make core be a little less strict about some spell attributes SPELL_ATTR0_CASTABLE_WHILE_DEAD and SPELL_ATTR2_CAN_TARGET_NOT_IN_LOS checks. 2011-08-30 00:13:53 +02:00
QAston 2f79241237 Core/Spells: check unattackable flag only for negative spells. 2011-08-29 23:26:47 +02:00
QAston 8754166071 Core/Spells: Send expected type of spell implicit target to client. 2011-08-27 12:59:57 +02:00
QAston 751cbaf307 Core/Spells: Fix a typo (!GetDst() instead of !HasDst()) which prevented from automatical lconverion of spell unit target to destination target (this is only used in script casts). Closes #2761. 2011-08-26 16:30:56 +02:00
QAston 79427354aa Fix build. Damn. 2011-08-26 15:40:11 +02:00
QAston 265c909666 Core/Spells: small correction to e2d592f380, don't interrupt passive auras. 2011-08-26 14:43:19 +02:00
QAston e2d592f380 Core/Spells: Interrupt periodic trigger auras when triggered spells fails to cast. Closes #2598 2011-08-26 14:23:23 +02:00
QAston 2ee88d76da Core/Spells: remove one obsolete spell proc target check. 2011-08-26 12:48:12 +02:00
QAston 8e392440cf Core/Spells: Fix a regression of 7f30ac5a95 - spells with TARGET_SELECT_CATEGORY_CHANNEL target type (like mind sear) should be triggered by triggering spell caster 2011-08-26 12:39:50 +02:00
QAston 5c48d9ec8a Core/Spells: Store explicit unit, corpse and GO targets in one variable, because both client and server can't handle more than one explicit object target; Add spell explicit target validation. 2011-08-26 12:06:13 +02:00
QAston dfa179b427 Scripts/Achievement: fix a typo. 2011-08-25 13:16:09 +02:00
QAston 5ebb4ba4c1 Core/Packets: Update opcodes enum - research by TOM_RUS, also remove some useless/obsolete comments from opcodes.h 2011-08-25 12:48:51 +02:00
QAston c0f3122b9c Fix build after revert. 2011-08-25 08:01:56 +02:00
QAston 188fdb3e4c Core/Spells: Allow spells with TARGET_FLAG_CORPSE to target players which didn't hit release button yet. 2011-08-25 06:51:57 +02:00
QAston 62d913bb35 Revert "Core/Spells: Also Death Runes itself should be able to turn into a Death Rune. They should not turn back into their original rune when using a spell that turns the used runes into Death Runes." - it is causing crashes.
This reverts commit 17b89ea8ab.
2011-08-25 06:40:56 +02:00