[DiscordArchive] So 6018 is Ur'Kyo, one of the many priest trainers... what if I changed his gossip_menu_id to that o
[DiscordArchive] So 6018 is Ur'Kyo, one of the many priest trainers... what if I changed his gossip_menu_id to that o
Archived author: sudlud • Posted: 2024-09-28T14:51:44.301000+00:00
Original source
no merge conflicts etc
Archived author: sudlud • Posted: 2024-09-28T14:51:56.278000+00:00
Original source
but for your minimal change it's probably not that critical
Archived author: sudlud • Posted: 2024-09-28T14:52:10.711000+00:00
Original source
<https://github.com/azerothcore/azerothcore-wotlk/blob/b07670f130a0e27747f27fa54502825c04db8a3b/src/server/game/Entities/Player/Player.cpp#L1282> here you can see where the hook is beeing called
Archived author: Agent • Posted: 2024-09-28T14:53:48.300000+00:00
Original source
Yes I found that line. Can I return True at line 1286 to achieve what I'm trying to do? I feel like it might be as simple as that
Archived author: Agent • Posted: 2024-09-28T14:54:12.362000+00:00
Original source
Hang on that won't work
Archived author: sudlud • Posted: 2024-09-28T14:55:01.185000+00:00
Original source
```
if (context == CLASS_CONTEXT_CLASS_TRAINER)
return true;
```
Archived author: sudlud • Posted: 2024-09-28T14:55:26.167000+00:00
Original source
should probably do it
Archived author: sudlud • Posted: 2024-09-28T14:57:19.797000+00:00
Original source
always returning `true` might break other contexts
Archived author: Agent • Posted: 2024-09-28T14:59:43.055000+00:00
Original source
Still parsing that. I feel like I should add that line above `if (scriptResult != std::nullopt)`
Yes?
Archived author: Agent • Posted: 2024-09-28T15:00:12.358000+00:00
Original source
So that the rest of the check will still go ahead if it's not a triner