| Age | Commit message (Collapse) | Author |
|
|
|
- Sound playback is handled via creature_text
- remove `DoPlaySoundToSet()` to avoid duplicate playback
- remove redundant 'me' parameter from `Talk()` entries
|
|
|
|
Replacing more hardcoded text with DB entries.
A little bit extra enum alignment and sorting.
|
|
Closes: #16533
|
|
Core/Scripts/boss_headless_horseman: remove hardcoded text
|
|
Remove hardcoded (as well as deprecated) text in the following places:
- blackrock_depths.cpp
- zone_burning_steppes.cpp
- zone_stormwind_city.cpp
Updates https://github.com/velinath/TrinityCore/issues/2
|
|
Remove the old
`#define EMOTE_LAUGHS "Headless Horseman laughs"`
No emote is ever used on retail. It would have created log spam.
|
|
|
|
Change an assert to a NULL check on Flame Leviathan vehicle kit, it can be NULL on map unload.
Close #16662
|
|
(PR #16680)
|
|
Core/Scripts: zone_terokkar_forest cleanup
|
|
This removes the current core script npc_tirion_fordring
and replaces it with database entries in:
- `conditions` 9 (quest taken) + 41 (unit_state)
- `gossip_menu_option` (3502,3681,3682,3683)
- `smart_scripts` (SmartAI)
Gossip text / `npc_text` is already in the database.
Closes #16430 (my old attempt at removing hardcoded text)
|
|
* Thanks @joschiwald pointing this out
|
|
|
|
Core/UnitAI: Rework creature-controlled player behavior.
|
|
- moved spell, quest and gossip IDs into enum
- replaced hardcoded text with DB gossip IDs
- restored correct gossip option order for Skull Pile
Replaces the crashed PR attempt in #16668
|
|
- Removed hacked control mechanism, use proper PlayerAI instead
- Port old hacky code to new SimpleCharmedPlayerAI class
- Make adjustments to aforementioned code to fix bugs:
- Properly clean up movement after charm ends
- Only try to attack a target if charmer is engaged in combat
|
|
through sai now
* Ref 546c8a5b8ead9d198
|
|
* Each subdirectory contains it's own translation unit now
which is responsible for loading it's directory
* Improves merging & decoupling between 3.3.5 <-> 6.x
* Removes unused Battleground loader
* Ref #15671
(cherry picked from commit 5534915f743c707c07cb977bca67c2ff15487597)
|
|
This is an attempt to proofread standard error messages and system messages
from both core and DB. The corrections span typos, grammar and punctuation.
Because some of these messages can have multiple meanings depending on context,
this PR will stay in "WIP" status until the corrections have been validated and approved.
You are welcome to suggest improvements and files not yet included in this list.
Thanks to @Kinzcool for suggesting this line of work. :)
|
|
[3.3.5] Core/Spells: Fix Glyph of Arcane Shot
|
|
|
|
Core/Spells Implemented SpellScripts Needed for Achievement Pilgrim's…
|
|
Scripts/Deadmines: Fix #6624 Mr. Smite's event text, and phase switch…
|
|
|
|
* Fixes Mr. Smite announcing to instance once doors have been destroyed.
* Add missing dialogue to encounter phases.
* Now properly runs in front of his chest, kneels down and equips items, stands, and reengages like he's supposed to.
* Fix weapon model for first phase when he switched to double axes.
|
|
Eliminationzx/FixWrongSealOfRighteousnessCalculation" while we figure out more details (see discussion in the PR).
This reverts commit 1a213bd9214bee65e887d9a20766032af0cbdb80, reversing
changes made to 657adc270a32da4ef943b443c1fdcf2814466909.
|
|
Eliminationzx/FixWrongSealOfRighteousnessCalculation
[3.3.5] Core/Spells: Fix incorrect Seal of Righteousness calculation.
|
|
|
|
Core/Spells: Disallow druid flight form while shapeshifted
|
|
Closes #16594
Closes #15915
|
|
|
|
|
|
Implement _DespawnAtEvade functionality to boss Lich King
|
|
prevent mounting. No more swimming zombies. Closes #16354.
|
|
|
|
|
|
|
|
3.3.5 (PR #16450) with some minor CS adjustments
|
|
[3.3.5] Add target boundary check to BossAI CanCreatureAttack call
|
|
|
|
|
|
Source http://wowwiki.wikia.com/wiki/Spell_power_coefficient
|
|
Bah
(cherry picked from commit 781152a30c526ff94638a2b10845d1686edb5e62)
|
|
|
|
Scryer's Orb
- Added missing waypoints, spells, emotes, proper timing... and prevent him from being summoned multiple times
Closes #13208, #16522
|
|
|
|
Moves the entire NPC script to the database and corrects a logic error where players who have rewarded the quest Scarlet Subterfuge and lost the disguise before taking the quest In Dreams were no longer able to get the disguise.
|
|
- Add missing const attribute on CheckBoundary arg.
- Prevent AI from attacking a target that is out of bounds. This prevents exploids such as killing Gluth/Deathbringer from out of bounds.
|