Commit Graph

27860 Commits

Author SHA1 Message Date
ariel-
05b42eef7e Core/Unit: don't clear low health aura states on death
- Confirmed with sniffs, healthless states aren't cleared
- Prevents some procs from happening

Closes #19754
2017-05-14 06:11:49 -03:00
Kittnz
d04bc1339a DB/GameObjects: Add missing gobs for map 450, 560, 329
Mostly chairs and random stuff
2017-05-13 19:17:05 +02:00
Killyana
d6a661681c DB/Creature: Fizzles
Taken  from aece8088f1/_develop/206_updatepack_mangos.sql
2017-05-13 17:10:01 +02:00
Aokromes
cacde7b9f5 Core/Logs: Reword some logs 2017-05-12 10:40:22 +02:00
Killyana
cadea4f183 DB/Gameobject: Add a missing cosmetic Yellow Teleport 2017-05-10 19:47:35 +02:00
Killyana
72447593f3 DB/Creature: Time Watchers
Remove a hack, the npc will despawn at the end of the mini event and respawn later in a correct position
Remaining issue #19744
2017-05-10 16:08:59 +02:00
Killyana
e036d2232d DB/Creature: Arcatraz Sentinel
Closes #10647
2017-05-10 02:41:50 +02:00
Killyana
ccf75008de DB/Creature: Fallen Earthen Warrior
Closes #16559
2017-05-09 23:01:33 +02:00
xinef1
538f60a880 Core/Quests: Display quest objective explore completion message
- Game client should show an on-screen message for explored quest objectives
  (AreaExploredOrEventHappens) when the objective has been completed.

Closes #19543
2017-05-07 20:15:19 +02:00
ariel-
1d905b78ec Core/Globals: fix array out of bounds crash on loading npc_text_locale
Closes #19556
2017-05-07 05:35:30 -03:00
Aokromes
c06f0bea9a DB/Misc: Fix PK for gossip_menu_option_locale 2017-05-07 08:15:52 +02:00
Aokromes
af411c2618 DB/Misc: Fix PK for npc_text_locale 2017-05-07 05:39:18 +02:00
ariel-
c3176a8143 Core/Players: fix null dereference crash 2017-05-06 15:49:56 -03:00
Aokromes
9ae53d2577 Typo fix 2017-05-06 04:45:53 +02:00
ariel-
97e608b053 Core/Entities: allow guardian pets to inherit spellmods 2017-05-05 19:27:28 -03:00
Chazy Chaz
4843544c21 Scripts/Commands: Update reload commands names 2017-05-05 05:52:49 +02:00
Mikhail Redko
8a5b998f48 Core/Spells: Move Big Blizzard Bear implementation to spell script (#19544) 2017-05-04 12:46:10 -03:00
Chazy Chaz
5ff8601d36 Core/Misc: Update locales_gossip_menu_option to simple system 2017-05-04 17:35:43 +02:00
Chazy Chaz
838b46e33e Core/Misc: Update locales_npc_text to simple system 2017-05-04 17:11:35 +02:00
Chazy Chaz
d17a45ee7e Core/Misc: Update locales_creature_text to simple system 2017-05-04 16:46:53 +02:00
tkrokli
4a0e51ad7a DB/SAI: run Marin Noggenfogger's script inside the quest
- change event type to On Quest Taken (19) instead of Finished (20)
- modify comment to reflect the event type used
- also modify comment to show the quest name instead of number
2017-05-03 23:55:05 +02:00
Shauren
011b8847d1 Core/Movement: Ported teleport packet fixes from master branch to get rid of ugly relocations 2017-05-03 17:48:42 +02:00
ccrs
b6274578b1 Core/Unit: revert recent changes
master cherry pick incomming :D
2017-05-03 15:14:41 +02:00
ccrs
d92132f1dc Core/Unit: 6fe4d37f8d followup 2017-05-03 14:14:37 +02:00
Kittnz
4d2a64cdf2 DB/Creature: Set correct Position Z height of some more Orgrimmar NPCs 2017-05-03 12:18:50 +02:00
Aokromes
5cdbbc29ab Core/Achievement: Fix startup 2017-05-03 05:49:31 +02:00
Chaouki Dhib
d625c0f619 Core/Spells: change to the way the spell effect SPELL_EFFECT_JUMP work 2017-05-02 23:59:21 +02:00
Chazy Chaz
309a7f7e2a Core/Misc: Update locales_achievement_reward to simple system
Closes #19528
2017-05-02 23:56:26 +02:00
ccrs
6fe4d37f8d Core/Unit: SendTeleportPacket corrections 2017-05-02 14:57:10 +02:00
ccrs
17579f8d91 Core/Creature: drop method SetPosition
eeeevil, use UpdatePosition. It was there just for old scripts compatibility.
2017-05-02 14:18:42 +02:00
ccrs
dd2151cc7a Core/Transport: fix logic error
justStopped was always set true, but when transports spawn and remain stationary timer, _currentFrame->ArriveTime and _currentFrame->DepartureTime are set to 0, making justStopped always remain true and triggering incorrect calls.
2017-05-02 13:47:51 +02:00
ccrs
b74264aef4 Core/Creature: IsMovementPreventedByCasting followup
Handle correctly move allowed channels: IsChannelActive checks if channel phase has started (after cast if any)
Must skip the other checks since all channels focus target.

ref #19532
2017-05-02 11:43:50 +02:00
tkrokli
cf8380a3a7 Fix missed apostrophe in previous commit
- beasts -> beast's
2017-05-02 02:16:16 +02:00
tkrokli
f6a4065f5e DB/Quests: minor correction in A Husband's Last Battle (6162)
Replacing the word 'head' with 'claw' in the quest progress gossip.
Check quest item 15879, it is a claw, not a head (obviously).

http://www.wowhead.com/item=15879/overlord-rors-claw
http://www.wowhead.com/quest=6162/deprecated-a-husbands-last-battle
2017-05-02 02:12:06 +02:00
ariel-
aa0b7dd152 Core/Misc: build fixes 2017-05-01 18:52:15 -03:00
ariel-
f913f3bb89 Core/Scripts: unified scripted gossip/quest api
- Changed self-accessor on GameObjectAI to "me", like UnitAI
- Moved all related functions to AI, now Unit and GameObject have the same function names with identical behaviour
- Remove "OnUpdate" from CreatureScript/GameObjectScript, was never used and we already have AI Update method
- Quest methods no longer return a bool, the return value was used to call the AI version if the ScriptMgr one returned false
- Implemented GameObjectAI::Destroyed hook (was never called), implemented Damaged method
- Rename OnStateChanged to OnLootStateChanged to reflect when it's really called, and created a new hook OnStateChanged that only gets called on GOState change
- Since the functions are now only getting called from AI, made GetAI methods full virtual. (CanSpawn method is anyways going to be used on creatures with AI)
2017-05-01 18:19:36 -03:00
tkrokli
e4481c016a DB/Quests: remove wrong $C in quest completion text (2284)
- Quest reward text should not change depending on player class.
- Quest: Necklace Recovery, Take 2 (ID 2284)
- NPC: Remains of a Paladin (entry 6912)

http://www.wowhead.com/quest=2284/necklace-recovery-take-2
2017-05-01 22:03:57 +02:00
tkrokli
a8f7c55d34 DB/Gossip: fix bad update in 2017_04_30_00_world_335.sql
* Properly remove text0_1 content from npc_text ID 16432

Ref. commit 88413de9d8

Ref. issue #19527
(DB/Gossip: npc_text contains "MISSING TEXT" and missing info in 48 rows)
2017-04-30 19:19:19 +02:00
tkrokli
9d3810c3c8 DB/Gossip: update npc_text info for Ravenholdt Guards
- all the information is available in broadcast_text
- npc_text updated both for ID 5934 and 5935
2017-04-30 11:03:44 +02:00
tkrokli
88413de9d8 Update 2017_04_30_00_world_335.sql
- remove BroadcastTextID0 for Jangdor Swiftstrider's gossip
- insert only text0_0 because NPC is a male
2017-04-30 08:23:31 +02:00
tkrokli
d4d85c764f DB/Gossip: update npc_text info for 3 NPCs
- set correct gossip_menu.text_id for Jangdor Swiftstrider
- add missing text for Terl Arakor and Griniblix the Spectator

* Thanks to MaNGOS Zero / billy1arm for providing the right clues.

Updates #19527
2017-04-30 07:50:40 +02:00
ariel-
231ec8331b Core/Entities: moved PetAura handling to Player where it belongs 2017-04-28 19:12:48 -03:00
ariel-
b6b59f6c23 Core/Scripts: remove OnDummyEffect hook/sOnDummyEffect ai hook
- Duplicated logic never used, sometimes only ScriptMgr version was called, sometimes only AI
- They only encourage bad scripting practices
- You can still use OnSpellHit or a SpellScript
2017-04-28 18:59:14 -03:00
ariel-
4c4dca6d69 Core/Misc: camelize GetFaction/SetFaction properly 2017-04-28 18:37:38 -03:00
Killyana
5a3a9381e4 DB/Creature: Add some missing spawns to Naxxanar area 2017-04-28 17:50:30 +02:00
ccrs
5a2f0ce29e Core/Unit: revert 3ea46e57af
After discussion we realized, ariel and me, that Creature and Unit IsMovementPreventedByCasting() have not the same checks (order matters)
2017-04-27 19:40:23 +02:00
Killyana
67a6e61908 DB: New line 2017-04-27 19:27:17 +02:00
Killyana
01d9df27d5 DB: Some Pathing on Coldarra area 2017-04-27 19:24:42 +02:00
ariel-
553103963f DB/Spell: fix Glyph of Shadow proc
- Fixes not proccing with Imp. Devouring Plague/Mind Flay/Mind Sear
2017-04-27 12:52:50 -03:00
ariel-
3ea46e57af Core/Unit: 229444b74a follow-up
- IsFocusing is made virtual again, so there's no need to keep a duplicated function

This reverts commit 5043639c56.
2017-04-27 11:00:01 -03:00