* init
* change logic
despawning of bosses is not really done the normal way, but it's forced with a set amount of guids. So we simply need to register the summons as guids as well
* small fixes
* Delete creature_scripts.sql
* Update boss_azgalor.cpp
* fix. fatal error: comparison of integers of different signs
* Update src/server/scripts/Kalimdor/CavernsOfTime/BattleForMountHyjal/boss_kazrogal.cpp
---------
Co-authored-by: Andrew <47818697+Nyeriah@users.noreply.github.com>
* (Instance/Culling of Stratholme) removed duplicated gossip_menus, fixed hardcoded texts
* Update culling_of_stratholme.sql
* Update culling_of_stratholme.sql
* (Instance/TrialOfTheCrusader) fixed hardcoded text from Jaraxxus text line
boss_twin_valkyr: removed unnececarry random txt line select (already exist groupid 6 id 0 and 1 in creature_text)
added ruRU locale for Acidmaw and Dreadscale GroupID 2
* move sqls
* Update culling_of_stratholme.sql
* Update data/sql/updates/pending_db_world/culling_of_stratholme.sql
Co-authored-by: Benjamin Jackson <38561765+heyitsbench@users.noreply.github.com>
* Update culling_of_stratholme.sql
* update naming of variables to better understand the sequence of events
* Change locale attribute to Locale
I'm not 100% sure, but since in Linux, uppercase and lowercase letters represent different values, I think that's why it doesn't pass the tests. In Windows nothing happens, but in Linux there are 2 different attributes.
* strange MySQL in Linux
* Update trial_of_crusader.sql
* updated sequence of txt lines with cityman before Malganis appear according to @Trinitycore enum RPEventLines2
combine all SQL in 1 file
---------
Co-authored-by: Benjamin Jackson <38561765+heyitsbench@users.noreply.github.com>
Co-authored-by: Walter Pagani <paganiwalter@gmail.com>
* arrays
* summon_groups
* part 2 lesgooo
* holy frick it works???
* drafted
* forgot to add co-author
Co-Authored-By: Saltgurka <22568446+saltgurka@users.noreply.github.com>
* thrall & tyrande
* Update hyjal.cpp
* trash
* winterchill & anetheron
* kazrogal & azgalor
* Update rev_1708737709108426300.sql
* entrance rp and scourge base
* it worked thank you nyeriah and ayamiss the hunter from AQ20 cause I remembered that it had a similar mechanic and i went to check how to deal with it. did you know ayamiss was my turning point in my emulation "career"? it was a very interesting fight and i spent many hours researching it, leading to my discovery into how the swarmers work, if i did it nowadays i'd have found out immediately but it was a journey nonetheless
Co-Authored-By: Andrew <47818697+Nyeriah@users.noreply.github.com>
* Update rev_1708737709108426300.sql
* no gem farming!
* infernals
* leftovers
* doors
* archimonde
* Update rev_1708737709108426300.sql
* jaina didnt reset gossip flag its over
* crashfix
Co-Authored-By: Andrew <47818697+Nyeriah@users.noreply.github.com>
* Update instance_hyjal.cpp
* no more dberrors
* fix archimonde model and size
scale was set to 0.4, also fixed the speed and attack time, as well as adding model info
* Update rev_1708737709108426300.sql
* Update rev_1708737709108426300.sql
* more stuff
* Update data/sql/updates/pending_db_world/rev_1708737709108426300.sql
* fix: warning
* fix: warning
* buildfix
don't need to assign anything, as long as it doesn't return null we're good
* buildfix
infernal has no DoAction behaviour so it's fine
* Update data/sql/updates/pending_db_world/rev_1708737709108426300.sql
* Update data/sql/updates/pending_db_world/rev_1708737709108426300.sql
* yay
Co-Authored-By: Dan <83884799+elthehablo@users.noreply.github.com>
* a
* looks nicer, less updates overall
* crashfix?
messy code gomenasai 🙇
* hide undesirables >:(
* Update instance_hyjal.cpp
Co-Authored-By: Anton Popovichenko <walkline.ua@gmail.com>
---------
Co-authored-by: Gultask <sagemochi@hotmail.com>
Co-authored-by: Saltgurka <22568446+saltgurka@users.noreply.github.com>
Co-authored-by: Andrew <47818697+Nyeriah@users.noreply.github.com>
Co-authored-by: Gultask <100873791+Gultask@users.noreply.github.com>
Co-authored-by: Dan <83884799+elthehablo@users.noreply.github.com>
Co-authored-by: Anton Popovichenko <walkline.ua@gmail.com>
* fix(Core/Creature): Prevent combat movement disabled creatures from repositioning if target moves within model boundary
* fixbuild
* Apply suggestions from code review
* Class Comparison Logic Encapsulation - Parity
* Add Context to IsClass
* Add Unit IsClass script hook
* Replace additional getClass with IsClass
* Update CanUseItem to replace getClass with IsClass
* Add separate context for pet vs ability
* Change Create to Init since not all referenced contexts are creation
* Align spacing in ClassContext
* Drop context on LFGManager max power
* Update IsClass context that wraps around Missle Barrage
* Rename context for swapping weapons
* Be more specific than CLASS_CONTEXT_TALENT
* Remove duplicate context
* Moved IsClass Hook to Player
* Removed unused parameter in virtual base function
* Added maybe_unused to IsClass virtual in order to compile
To match the override signature, the virtual base needs to include the parameter in question, so using [maybe_unused] to signal to the compiler to allow it
* Remove extra blank line
* Add ABILITY_REACTIVE context
* Add context for PET_CHARM
* Remove explicit nullopt check per review
* Code Readability - Change if to if else in pet
Due to the return pattern, this doesn't change functionality in any way
* Add OnPlayer to disambiguate
---------
Co-authored-by: NathanHandley <nathanhandley@protonmail.com>
* Changing hardcoded text in #define to enum member
- Changing #define to new enum member for AddGossipItemFor method
* Adding SQL Query File
* Forgot to add ';' in DELETE FROM
