Skip to content

Commit

Permalink
DB/Misc:
Browse files Browse the repository at this point in the history
Add Missing Sunreaver Guardian Mage (thx aokromes) Closes #9311
Add SAI for Mission: The Abyssal Shelf (thx Kirkhammett)
  • Loading branch information
ZxBiohazardZx committed Mar 3, 2013
1 parent a108a90 commit 13568b1
Showing 1 changed file with 13 additions and 0 deletions.
13 changes: 13 additions & 0 deletions sql/updates/world/2013_03_03_01_world_misc.sql
@@ -0,0 +1,13 @@
-- Issue 9311
-- Add Missing Sunreaver Guardian Mage inside Sunreaver Tavern (Thx Aokromes)
SET @CGUID := 40267; -- Set by TDB Team
DELETE FROM `creature` WHERE `guid` @CGUID;

This comment has been minimized.

Copy link
@rootor

rootor Mar 3, 2013

WHERE guid = @CGUID;

INSERT INTO `creature` (`guid`,`id`,`map`,`spawnMask`,`phaseMask`,`position_x`,`position_y`,`position_z`,`orientation`,`spawntimesecs`,`spawndist`,`MovementType`) VALUES
(@CGUID,29255,571,1,1,5900.299,505.75,641.6531,4.171337,120,0,0);

-- Issue 9322:
-- fix SAI to make the quest Mission: The Abyssal Shelf (10162) Horde completable (thx Kirkhammett)
UPDATE `creature_template` SET `AIName`= 'SmartAI' WHERE `entry`=19273;
DELETE FROM `smart_scripts` WHERE `entryorguid`=19273;
INSERT INTO `smart_scripts` (`entryorguid`,`source_type`,`id`,`link`,`event_type`,`event_phase_mask`,`event_chance`,`event_flags`,`event_param1`,`event_param2`,`event_param3`,`event_param4`,`action_type`,`action_param1`,`action_param2`,`action_param3`,`action_param4`,`action_param5`,`action_param6`,`target_type`,`target_param1`,`target_param2`,`target_param3`,`target_x`,`target_y`,`target_z`,`target_o`,`comment`) VALUES
(19273,0,0,0,19, 0,100,0,10162,0,0,0,15,10162,0,0,0,0,0,7,0,0,0,0,0,0,0, 'Commander To''arch - On Quest Accept - Call Areaexplored');

3 comments on commit 13568b1

@zaphod77
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that abyssal shelf fix is probably hack, but it's the same one i did. :) Of course that assumes that the spell is actually used.

and i see no evidence the spell with the complete effect is ever used. so seems that sanity check on specialflags is possibly wrong, and sometimes spell with quest complete effect is never used.

@filipboev
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is a hack, but it is a necessary hack. The spell used to make the quest eligibility flag active is client side, so this is one way of circumventing it.

@zaphod77
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My best guess is that this spell was a leftover and never actually used. I suspect originally one of those giant reaver was supposed to almost get you during the flight and get blown up by that spell.

So the quest shouldn't have the specialflags, and the core should ignore the unused spell, i think.

Please sign in to comment.