[DiscordArchive] is it ok to rename GetAllMinionsByEntry to GetAllSummonsByEntry instead?
[DiscordArchive] is it ok to rename GetAllMinionsByEntry to GetAllSummonsByEntry instead?
Archived author: Northstrider • Posted: 2024-12-14T20:21:10.432000+00:00
Original source
SetMinion is not using masks like this. It's using updatefields
Archived author: Northstrider • Posted: 2024-12-14T20:21:20.759000+00:00
Original source
(which are also wrongly set)
Archived author: Telegrill • Posted: 2024-12-14T20:22:42.160000+00:00
Original source
mhm
Archived author: Northstrider • Posted: 2024-12-14T20:23:12.705000+00:00
Original source
in other terms: don't meddle with it unless you want to re-implement everything from the ground up. summons are a giant house of cards
Archived author: Telegrill • Posted: 2024-12-14T20:24:06.289000+00:00
Original source
well i wanted to introduce helpers for player class to iterate through its own summons
Archived author: Northstrider • Posted: 2024-12-14T20:28:01.366000+00:00
Original source
then implement a new helper and a new container for tracking them. better not touch minion logic
Archived author: Telegrill • Posted: 2024-12-14T20:28:30.833000+00:00
Original source
it's rather weird, because player class uses m_Controlled which is ControlList, but creatures use SummonList
Archived author: Northstrider • Posted: 2024-12-14T20:29:00.799000+00:00
Original source
SummonList is tied to bossAI, not to creatures themselves
Archived author: Northstrider • Posted: 2024-12-14T20:29:16.823000+00:00
Original source
ScriptedAI needs to define it as member
Archived author: Northstrider • Posted: 2024-12-14T20:29:57.460000+00:00
Original source
m_Controlled is also currently filled with wrong data (it was originally meant for charmed units only, not for every friendly summon)