* feat(Core/ChatHandler): Flood Control Addon Messages
Original Closed PR by @Kitzunu
https://github.com/azerothcore/azerothcore-wotlk/pull/11675
Cherrypicked TC:
cherry-pick commit (TrinityCore/TrinityCore@5384fc1)
cherry-pick commit (stoneharry/TrinityCore@a0629fc)
Co-Authored-By: stoneharry 3818405+stoneharry@users.noreply.github.com
Co-Authored-By: Shauren shauren.trinity@gmail.com
Co-Authored-By: stoneharry <3818405+stoneharry@users.noreply.github.com>
Co-Authored-By: Shauren <shauren.trinity@gmail.com>
Co-Authored-By: Kitzunu <24550914+Kitzunu@users.noreply.github.com>
* update: (pr): Review Response
* update: (core): Chathandler
No reason why this case should even exist. Previous or current. This cast is under two sided interations which will always lead to you being in a group of some sorts. there is literally zero sense having a above or equal to level 80 if statement here.
* update (cleanup): Code Style
suggestion per @Nefertumm
Co-Authored-By: Angelo Venturini <nefertum.dev@protonmail.com>
Co-authored-by: stoneharry <3818405+stoneharry@users.noreply.github.com>
Co-authored-by: Shauren <shauren.trinity@gmail.com>
Co-authored-by: Kitzunu <24550914+Kitzunu@users.noreply.github.com>
Co-authored-by: Angelo Venturini <nefertum.dev@protonmail.com>
* fix(Core): Fix Instance Reset Exploit
* Whitespace fix
* Remove unnecessary include
* Check player instance save IDs when adding to map
* Only compare playerBind during login
* Config option
* Extra check
* Fixes
Updated some comments
Removed the config option from the first if since it's not necessary (afaik)
* REVERT
Reverting the if to what I had it at originally until I can get some input on it...
* Updated if
Got some input and this is working as intended
* Update World.cpp
* Update src/server/worldserver/worldserver.conf.dist
Co-authored-by: Kitzunu <24550914+Kitzunu@users.noreply.github.com>
Co-authored-by: Kitzunu <24550914+Kitzunu@users.noreply.github.com>
* fix\feat: (Core/PacketIO): updated sound and creature addon
updated SMSG_PLAY_SOUND, SMSG_PLAY_MUSIC and SMSG_PLAY_OBJECT_SOUND via tc cherry pick
https://github.com/TrinityCore/TrinityCore/pull/2363 and 0f1f7ef401 by @joschiwald and @ForesterDev
This so far a attempt to align atleast with sound with tc, and correct a potentional issue of hearing creature sounds that is not in visible range. I notice PlaySound was being defined in some weird dependency as it isnt with tc so I renamed it to Playsound. Notice a isLarge still being used in the creature addon when it was depreciated, so i removed that a that seem to of interfered with visibilitydistanceType.
Co-Authored-By: joschiwald <736792+joschiwald@users.noreply.github.com>
Co-Authored-By: ForesterDev <11771800+ForesterDev@users.noreply.github.com>
* Change TO ThreadUNSAFE
This will process the far sight packet as a is not thread-safe - process it in World::UpdateSessions() compared to the former packet is thread-safe - process it in Map::Update().
This matches up with tc in everything we have with the HandleFarSightOpcode.
* Further Adjustments
* Move to Debug Loggin
TC has it as move to Debug Logging. Stating its a weird opcode handling issue which is not really known why.
