Additional change: The answers have been changed from containing only names
to include the text ` is my answer.` from the matching lines in `broadcast_text`.
- added DB creature_text entries for Braug Dimspirit
- added DB gossip_menu_option entries for this NPC
- implemented NPC `Say` line in reply to the wrong answers
- avoiding gossip exploits by turning off npcflags during talk
- added conditions for showing gossip depending on quest status
- removed deprecated SD comments in zone_stonetalon_mountains.cpp
The core script only contains gossip handling + conditions.
- remove script from file the_black_morass.cpp
- thereby removing 1 line of hardcoded text
- remove redundant SD2 comments in header
- add SAI, gossip and conditions to DB
SmartAI uses that same method internally, so there's no need to handle it in two different ways.
Does not affect functionality, but prevents useless SAI error log in case the target creature is not using SAI.
True Believers and Still Believing (leftover from previous commit), A Terrible Purpose, Wrath of Neptulon (still missing the 100 gold attached), A Letter for Home (Alliance and Horde), Tactical Clemency, Abandoned Mail, Hammershout the Elder and various Lunar Festival mails.
Unit::IsControlledByPlayer is a expected value for TempSummons (like some triggers used in quests, summoned by spells).
Previous logic broke a lot of quests which use triggers to do stuff.
Allow JustSummoned and events linked from other events with flag set also in charmed (prevents breaking event chains)
Allow vehicles (needed for vehicles with SmartAI, eg Iron Rune Constructs and You: Rocket Jumping)
- Icy Talons, Holy Concentration, Surge of Light, Trauma, Glowing twilight scale: these auras have SPELL_ATTR3_CAN_PROC_WITH_TRIGGERED,
but spells triggering them don't have SPELL_ATTR2_TRIGGERED_CAN_TRIGGER_PROC or SPELL_ATTR3_TRIGGERED_CAN_TRIGGER_PROC_2, so I suspect a wrong flag implementation in the core. Setting the "can proc with triggered" attribute for now.
- Trauma, Glowing twilight scale: added spell_proc entry to override SPELLFAMILY_PALADIN in dbc (no spells from other spell family could trigger this)
- Light's Grace: Casted Holy Light was proccing the triggered aura and removing the charge instantly, Add "must use spell mod" attribute.
- Dislodged Foreign Object: should proc on damaging spell casts only (has positive spell proc flag in DBC)
Turns out epic and legendary items are excluded from this limitation (or it could be just instanced content that is immune to this). Other qualities are affected as normal. Sorry for the mishap.
