Trinitycore: DB/Creature: Klaven Mortwake

[ZenoX92]

Description:

Klaven Mortwake:

• Add aggro text
• Add emotes for some of his texts
• Fix react state

Branch(es):

3.3.5

TC rev. hash/commit:

TrinityCore rev. 1cc3d33 2018-02-19 20:21:57 +0100 (3.3.5 branch) (Win64, RelWithDebInfo, Static)

TDB version: 335.64

Operating system: Windows 7

Suggested Fix:
``sql -- Klaven Mortwake UPDATEcreature_textSETEmote=5 WHERECreatureID=7053 ANDGroupID=2; UPDATEcreature_textSETEmote=11 WHERECreatureID=7053 ANDGroupID=1; DELETE FROMcreature_textWHERECreatureID=7053 ANDGroupID=5; INSERT INTOcreature_text(CreatureID,GroupID,ID,Text,Type,Language,Probability,Emote,Duration,Sound,BroadcastTextId,TextRange,comment`) VALUES
(7053,5,0,"You cannot hide from me, rogue. I can smell your fear.",12,0,100,0,0,0,3092,0,"Klaven Mortwake");

UPDATE smart_scripts SET action_param6=2 WHERE entryorguid=7053 AND source_type=0 AND id=1;
DELETE FROM smart_scripts WHERE entryorguid=7053 AND source_type=0 AND id=3;
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
(7053,0,3,0,4,0,100,0,0,0,0,0,1,5,0,0,0,0,0,1,0,0,0,0,0,0,0,"Klaven Mortwake - On Aggro - Say Line 5");

[illfated]

I just visited his tower with a level 30 warrior now.
I did not expect him to call a warrior a rogue.
Maybe this quote section would explain the issue?

---

https://wow.gamepedia.com/Klaven_Mortwake#Quotes

Upon detecting stealth:

• You cannot hide from me, Rogue! I can smell your fear!

---

I suspect his text should be
You cannot hide from me, $c. I can smell your fear.
(unless his AI could differentiate between classes and say the line only when a rogue is present).

[offl]

He's a rogue quest NPC

[illfated]

OK, maybe Blizzard never intended for any other classes to visit that tower or that NPC.

[offl]

Or maybe condition is required :)
Who knows
I even can't find a video with this quest

[illfated]

Fair enough. At least you tried. 👍
Anyway, it is a very cosmetic issue.

[Killyana]

Try:
``sql DELETE FROMconditionsWHERESourceTypeOrReferenceId=22 ANDSourceEntryIN(7053) ANDSourceGroup=4; INSERT INTOconditions(SourceTypeOrReferenceId,SourceGroup,SourceEntry,SourceId,ElseGroup,ConditionTypeOrReference,ConditionTarget,ConditionValue1,ConditionValue2,ConditionValue3,NegativeCondition,ErrorType,ErrorTextId,ScriptName,Comment`) VALUES
(22, 4, 7053, 0, 0, 1, 0, 1784, 0, 0, 0, 0, 0, '', 'Klaven Mortwake - run SAI event 4 only if the player is stealthed'),
(22, 4, 7053, 0, 1, 1, 0, 1785, 0, 0, 0, 0, 0, '', 'Klaven Mortwake - run SAI event 4 only if the player is stealthed'),
(22, 4, 7053, 0, 2, 1, 0, 1786, 0, 0, 0, 0, 0, '', 'Klaven Mortwake - run SAI event 4 only if the player is stealthed'),
(22, 4, 7053, 0, 3, 1, 0, 1787, 0, 0, 0, 0, 0, '', 'Klaven Mortwake - run SAI event 4 only if the player is stealthed');

[illfated]

Very good point. 👍 The text is after all meant to be said when he detects stealth.
(When player is stealthed, at least at a level Klaven should be able to detect.)
BTW, suggestion for the English wording in the condition comments:

'Klaven Mortwake - run SAI event 4 only if the player is stealthed'

[illfated]

Tested the SQL just now (had to restart worldserver) and it worked just fine.
Klaven did not say that line to me, so it worked as intended. 👍

TC>server debug
TrinityCore rev. ba9943d8a422 2018-07-25 15:35:03 +0200 (3.3.5 branch) (Win64, Release, Static)
Using SSL version: OpenSSL 1.1.0h 27 Mar 2018 (library: OpenSSL 1.1.0h 27 Mar 2018)
Using Boost version: 1.65.1
Using MySQL version: 5.6.38
Using CMake version: 3.10.1
Compiled on: Windows 10.0.17134
Automatic database updates are enabled for the following databases: Auth, Characters, World
Worldserver listening connections on port 8085
Realmlist (Realm Id: 1) configured in port 8085
VMAPs status: Enabled. LineOfSight: 1, getHeight: 1, indoorCheck: 1
MMAPs status: Enabled
maps directory located in ./maps. Total size: 252191207 bytes
vmaps directory located in ./vmaps. Total size: 588247501 bytes
mmaps directory located in ./mmaps. Total size: 2152621724 bytes
Using enUS DBC Locale as default. All available DBC locales: enUS
Using World DB: TDB 335.64

[ZenoX92]

Seems I missed this when I added the text. I'm glad @illfated and @Killyana were around to notice it and fix it.

I /bow to you. :)