[DiscordArchive] is it ok to rename GetAllMinionsByEntry to GetAllSummonsByEntry instead?
[DiscordArchive] is it ok to rename GetAllMinionsByEntry to GetAllSummonsByEntry instead?
Archived author: Telegrill • Posted: 2024-12-14T20:44:24.868000+00:00
Original source
while also maintaining certain helpers for ScriptedAI to use which cannot be accessed by player class
Archived author: Northstrider • Posted: 2024-12-14T20:44:58.565000+00:00
Original source
SAI is also suffering from the false summon implementation, ye
Archived author: Telegrill • Posted: 2024-12-14T20:47:25.757000+00:00
Original source
originally, I wanted to create a few helpers for Unit class to iterate through summons but since Player and Creature classes use different classes to store those, it'll end up being a ugly mess to check which class the accesor is and iterate through one or the other based on it
Archived author: Northstrider • Posted: 2024-12-14T20:49:14.203000+00:00
Original source
ideally you would override the TempSummon AddToWorld and RemoveFromWorld methods and register/unregister the summon's guid this way within the summoner's new container
Archived author: Telegrill • Posted: 2024-12-14T20:49:35.310000+00:00
Original source
like some sort of TempSummon unordered_map?
Archived author: Northstrider • Posted: 2024-12-14T20:50:20.234000+00:00
Original source
either a vector or set (you only need the guid because you access them via objectaccessor only)
Archived author: Northstrider • Posted: 2024-12-14T20:50:48.454000+00:00
Original source
storing pointers of other entities is a big nono
Archived author: Telegrill • Posted: 2024-12-14T20:51:21.610000+00:00
Original source
actually, I thought of a unonrdered_map using entry, GuidSet
Archived author: Telegrill • Posted: 2024-12-14T20:51:33.011000+00:00
Original source
since you can have multiple summons of the same entry
Archived author: Northstrider • Posted: 2024-12-14T20:52:14.073000+00:00
Original source
you can do that but this is a rather rare case where you would need to access them so better keep it as plain vector and instead build GuidSets on demand