Commit Graph

5304 Commits

Author SHA1 Message Date
jackpoz
b1d0855624 Core/Spells: Fix static analysis issues and a crash
Fix confusing NULL checks in Raise Ally check cast.
Fix a crash in Raise Ally triggered by executing ".cast back 61999" command targeting a pet in raid while being dead.
2016-01-08 23:13:15 +01:00
Saben65
ad425f9f17 Scripts/Shadowmoon Valley: Dragonmaw Peon
Fixes issue where NPC does not attack, even when aggro'd.
Fixes #16210
Closes #16204
2016-01-08 22:19:56 +01:00
Francesco Borzì
3e42d83e9c Merge pull request #16208 from ShinDarth/putricide-fix
[3.3.5] Scripts/ICC: Fixed Professor Putricide
2016-01-08 11:13:22 +01:00
ShinDarth
b1a89943a2 Scripts/ICC: Fixed Professor Putricide
- closes https://github.com/TrinityCore/TrinityCore/issues/16207
2016-01-08 10:33:16 +01:00
ccrs
e5d4005cc6 Core/Spells: Fixed Raise Ally
thx @Nayd for sniffs :)
thx @Shauren for helping to find all spell involved

* There is no implementation for stats update on Puppet class (only on Guardian) so same SummonProperty as Raise Dead Ghoul (non pet) is used.
(Default SummonProperties set category to SUMMON_CATEGORY_PUPPET)
* Override the Summon Effect to enable charm and stats scaling.
* PlayerAI is used to handle unaura on ghoul despawn or dead.
It's necessary due to the fact that all script hooks on ScriptedCreature are called on an unactive CreatureAI, resulting in creature being unable to handle unaura calls.
* Create UpdateAI call for Player class
* Stats scaling is based on forum and wowhead comments, they recall this ghoul as a copy of the other one (same stats).
* Spellscript for Ghoul spell Thrash

Closes #82
Closes #14830
2016-01-08 00:10:15 +01:00
velinath
4e4ef26108 Core/Scripts/DB: Migrate Wing Commander Dabir'ee to SAI 2016-01-05 19:23:48 -05:00
treeston
1d9d03b289 Merge remote-tracking branch 'velinath/sai-migrations-2' into 3.3.5-base 2016-01-05 23:36:57 +01:00
Andrew Letson
9e366b36ae Core/Scripts/DB: Migrate Skyguard Handler Deesak to SmartAI 2016-01-05 16:38:55 -05:00
velinath
f21199dbc9 Core/Scripts/DB: Migrate Skyguard Handler Irena to SAI 2016-01-04 20:23:01 -05:00
jackpoz
f22b1938db Scripts/Commands: Modify ".learn" command to target current Player by default 2016-01-04 22:44:19 +01:00
velinath
295f76d77c Core/Scripts: remove npc_taxi unused scripts 2016-01-04 09:14:03 -05:00
joschiwald
f377a5875c Merge pull request #16171 from velinath/sai-migrations
Core/Scripts/DB: move Gryphoneer Windbellow to SAI
2016-01-04 00:32:10 +01:00
velinath
caff65bdb1 Core/Scripts/DB: move Gryphoneer Windbellow to SAI 2016-01-03 18:30:47 -05:00
joschiwald
8a0bbc3869 Core/Spells: Fixed Enrage armor reduce part 2016-01-03 00:55:50 +01:00
joschiwald
35aed8902a Scripts/Taxi: addition to 660f7e72be 2016-01-02 23:57:56 +01:00
tkrokli
b0da73fc5d Core/Scripts/DB: move Wyrmrest Temple taxi FP to SAI
Closes #15957

Move NPC gossip menu options from core script to DB SAI script
for the 3 local Flight Path NPCs in Wyrmrest Temple:

- 26443, Tariolstrasz (Steward of Wyrmrest Temple)
- 26949, Torastrasza (Majordomo to the Ruling Council)
- 27575, Lord Afrasastrasz (Commander of Wyrmrest Temple Defenses)

Additional positive effect: the NPCs are now using verified Blizzlike text in their gossip.

Extra credits:  @velinath for relevant background information in the issue,
@Helias & @ShinDarth for their excellent Keira2 web-based database editor.
2016-01-02 22:50:18 +01:00
treeston
b6ed3d3fc2 Merge pull request #16126 from velinath/taxi-to-sai 2016-01-02 17:39:54 +01:00
Rushor
923a368ac7 Update copyright note for 2016 2016-01-01 12:02:33 +01:00
velinath
be38235306 Adds William Kielor to SAI 2015-12-29 21:00:45 -05:00
Killyana
13e509a622 Script/Creature: Imprisoned Beryl Sorcerer
Add missing emote state
Move the credit quest on the 5th hit instead of the 7th one
2015-12-30 02:05:24 +01:00
Killyana
79c9af11b9 Creature/script: William Kielar
Fix Gossips used for taxi
Closes #16112
2015-12-29 19:23:02 +01:00
treeston
403ff21324 Spells/Penance: If hostile target is not in front, now fails in the initial cast rather than the triggered cast, thus not erroneously causing a cooldown. 2015-12-19 11:32:25 +01:00
jackpoz
e6903fdffa Merge pull request #15852 from Golrag/icc_boss_aggro 2015-12-15 21:29:10 +01:00
tkrokli
4fbffab5f8 Scripts/Spells: Mine Sweeper achievement
Enable achievement 1428, "Mine Sweeper" in SpellScript

by @Re3os / @ccrs / @sirikfoll . based on original script from @Vincent-Michael

Closes #15938
2015-12-15 04:27:17 +01:00
jackpoz
b96704774b Merge pull request #15919 from Treeston/3.3.5-razuvious 2015-12-14 20:34:12 +01:00
treeston
54d583b0fb Scripts/Naxxramas: Instructor Razuvious rework:
- Fix missing yells
- Rearrange existing yells to match live
- Sniffed positions for adds
- Improve add respawn handling
- Make adds not fight the player for control (don't use their spells on their own etc.)
- Update adds' hotbar keys to match live (this will affect other creatures with nonsequential possess keys as well)
- Fix adds' movement when mind controlled, they no longer move automatically when right-clicking (this affects all mind-controlled creatures)
- Blizzlike timers
2015-12-14 20:11:57 +01:00
Vincent-Michael
54424aff30 Scripts/Pets: Fixed non pch build part 3 ... 2015-12-12 18:00:22 +01:00
Vincent-Michael
47d54ce859 Scripts/Pets: Fixed non pch build part 2 2015-12-12 17:56:03 +01:00
Vincent-Michael
677f281b9a Scripts/Pets: Fixed non pch build 2015-12-12 17:52:16 +01:00
jackpoz
263f127d07 Merge pull request #14896 from Ofinka/dupl 2015-12-12 17:43:35 +01:00
jackpoz
187e45d24b Merge pull request #15910 from Treeston/3.3.5-maexxna 2015-12-08 12:40:04 +01:00
jackpoz
3267c90102 Core/Unit: Add Unit::KillSelf() overload
Add Unit::KillSelf() function as overload of Unit::Kill(this) . Use KillSelf() whenever the killer and the victim are the same to clearly state the Unit is going to kill itself.
2015-12-07 20:06:02 +01:00
jackpoz
ee12982008 Merge pull request #15933 from Asterc/3.3.5
Core/Spells: Fix Misdirection exploit
2015-12-07 19:11:48 +01:00
jackpoz
1ddf44caf3 Core/Quest: Fix issues reported by static analysis 2015-12-07 18:31:53 +01:00
jackpoz
f1f4005ad8 Core/Quest: Fix issues reported by static analysis 2015-12-06 21:37:58 +01:00
Killyana
e60bbaa7d3 DB/Quest: Great Bear Spirit
Closes #15950
2015-12-06 20:04:13 +01:00
Asterc
f8f815dbc3 Correctly reset the redirected threat of the hunter's spell Misdirection in cases when it has RemoveMode AURA_REMOVE_BY_DEFAULT
Closes #15930
2015-12-02 19:30:12 -02:00
MitchesD
3b1f5f29c9 Merge pull request #15864 from tkrokli/hallowed_wands
Scripts/Spells: Hallow's End item Hallowed Wand
2015-12-01 15:05:15 +01:00
treeston
5dd6163387 Scripts/Naxxramas: Maexxna cleanup:
- Improved Web Wrap target selection
- Added missing text
- Cleanup of web wrap positioning - two players will never be flung to the same position (25-man)
- Further improvement to web wrap spawn behavior - targetable web wrap now only spawns after the player has finished being flung to the respective position (blizzlike)
- Add additional web wrap positions (sniff)
2015-11-30 20:18:55 +01:00
tkrokli
66073e4403 Core/Scripts: remove redundant script npc_anchorite_truuen
The cpp script npc_anchorite_truuen has become redundant
because it has been replaced by a SAI script in commit 2cecc7b.

This script currently causes an error line in DBErrors.log:
```
Script named 'npc_anchorite_truuen' does not have a script name assigned in database.
```
2015-11-27 04:02:30 +01:00
jackpoz
6e296ddaa2 Scripts/EyeOfEternity: Fix crash
Fix crash happening when a player would kill a Melee npc on the disks during Malygos fight and mount it before it ended the waypoint path.
Close #15900
2015-11-26 21:47:50 +01:00
MitchesD
3b90b0368a Core/Misc: fix warning and misc codestyle fixes 2015-11-26 19:28:53 +01:00
treeston
bf99630399 Scripts/Naxxramas: Thaddius follow-up:
- Reduce initial spawn timer on the encounter from 30 seconds to 5 seconds
- Rearrange respawn logic to fix the "petrified" visual from not showing properly
- Fix a potential infinite respawn loop that could prevent the encounter from initializing properly (fixes and closes #15898)
2015-11-26 18:57:15 +01:00
DevRival
3fdbcf18ae Scripts/Spells: Hallow's End item Hallowed Wand
Hallowed Wand items should cast a transformation on party members.

by @DevRival, closes  #3574

Before this patch, the following spells do not have any effect at all:
- 24717, Pirate Costume
- 24718, Ninja Costume
- 24719, Leper Gnome Costume
- 24720, Random Costume
- 24737, Ghost Costume

Details from testing on the current 3.3.5 branch (commit ecbbcc8):
- 24724 (Skeleton Costume) seems to work as intended
- 24733 (Bat Costume) only lasts 10 seconds instead of 60 minutes
- 24741 (Wisp Costume) also seems to work as intended

All the spells listed above are included in this PR.
2015-11-17 17:24:25 +01:00
Golrag
ce90e30a80 Scripts/ICC: * Bosses now enter when in aggro range.
* Removed some of the MoveInLineOfSight calls
* Added more AreaTriggerScripts
2015-11-16 16:24:10 +01:00
Shauren
ecbbcc8972 Core/Spells: Fixed Raise Dead cooldown
Closes #15862
2015-11-15 00:31:17 +01:00
Carbenium
f01ef901ef Script/Naxxramas: Fix a -Winconsistent-missing-override warning reported by clang 3.6 2015-11-14 19:48:05 +01:00
tkrokli
95d6f09243 Scripts/Thaddius: Coverity scan Control flow issues (MISSING_BREAK)
Coverity scan defects CID 1339315, 1339316 and 1339317:
- CID 1339315: boss_thaddius.cpp: 542
- CID 1339316: boss_thaddius.cpp: 811
- CID 1339317: boss_thaddius.cpp: 398

Added missing break; statements.
2015-11-14 05:12:25 +01:00
Vincent-Michael
2ee01a7b3d Core: Fix non pch build 2015-11-13 20:19:16 +01:00
MitchesD
e14e291ef6 Merge pull request #15523 from Treeston/3.3.5-thaddius
Scripts/Naxxramas: Thaddius: Complete rewrite
2015-11-12 19:46:46 +01:00