Age | Commit message (Collapse) | Author |
|
|
|
|
|
Crash fix in .gps command
|
|
|
|
readability
Original code by @Shocker (6.x branch)
|
|
* Assignment of function parameter has no effect outside the function. Did you forget dereferencing it?
-> This argument is passed by value, it makes no sence so set it to null here.
* Member variable 'instance_violet_hold_InstanceMapScript::bWiped' is not initialized in the constructor.
(cherry picked from commit 433bc289c2c82b7377ad0189e415a3ca070d220f)
|
|
* fixes detecting support of <initializer_list>
* fixes build on solaris
(cherry picked from commit e40a5bf0b1483ce4275464a0a75a3d99035ca71c)
|
|
* probably fixes gcc compile after adding cppformat
(cherry picked from commit a610dea85c77da9ce6c2be5b10f8259e3f315227)
|
|
(cherry picked from commit d64e6ffade392be32f634b32658a6e0353f369c1)
|
|
by @untaught closes #3977
|
|
closes #1815
* Add missing Waypoint
* Improve Aggrobehaviour
* Add BroascasttextIDs
* Update to current GUID-Handling
Original Patch by @trickerer
|
|
|
|
(cherry picked from commit 3eaace1508e818c4f651eaaf8792cc60a766890f)
|
|
|
|
Type safe logging
(cherry picked from commit cc0c9add5a3f3a17e19029ddcaa7e711040f4fc5)
Conflicts:
src/server/bnetserver/CMakeLists.txt
src/server/game/Server/WorldSocketMgr.cpp
src/server/shared/Logging/AppenderDB.cpp
src/server/worldserver/CMakeLists.txt
|
|
* Possible inefficient checking for 'list' emptiness.
* Variable '_scheduledScripts' is assigned in constructor body.
Consider performing initialization in initialization list.
* Variable 'Text' is assigned in constructor body.
Consider performing initialization in initialization list.
* Possible inefficient checking for 'm_loadedScripts' emptiness.
(cherry picked from commit 9a61049f88835fd6bdb75905f042583d038d0667)
Conflicts:
src/server/game/Server/Packets/TicketPackets.cpp
|
|
* Either inefficient or wrong usage of string::find(). string::compare() will
be faster if string::find's result is compared with 0, because it will not scan the whole string.
If your intention is to check that there are no findings in the string,
you should compare with std::string::npos.
* C-style pointer casting detected. C++ offers four different kinds of casts as replacements:
static_cast, const_cast, dynamic_cast and reinterpret_cast.
(cherry picked from commit 8882a6ca78e90d2980a3cd8d53acea9d2137f8b8)
|
|
missing things to Journey to Undercity's quest end event
Closes https://github.com/TrinityCore/TrinityCore/pull/14094
Thans to @Killyana @Rushor @jaredjones and all others who helped with this
|
|
|
|
* Fordring statue will only be visible on a realm where the realm first is rewarded for http://www.wowhead.com/achievement=4576/realm-first-fall-of-the-lich-king otherwise this is replaced with fountain
* Dedication of honor will play lich king video to a player who has killed the lich king
* Correct flags for fountain object.
|
|
Core/Condition: Realm Achievement condition
|
|
By Malcrom (no, he is not back)
|
|
by @Killyana + @danlapps
closes #10234
|
|
This was already fixed with updates but changing here so this wont cause problems if people try to commit later.
|
|
By Malcrom (no, he is not back)
|
|
By Malcrom (no, he is not back)
|
|
- Partly Fixes https://github.com/TrinityCore/TrinityCore/issues/5754
@DRJ has SQL examples
|
|
By Malcrom (no, he is not back)
|
|
By Malcrom (no he is not back)
|
|
|
|
|
|
By Killyana, updates #13707
|
|
Core: Move FillCirclePath function to MotionMaster
Closes #14371
|
|
closes #6702
|
|
|
|
Ref 170bc318b34d393ea08e1607b6a4de87fcb2e6d7
|
|
closes #1678
|
|
#closes 12775
|
|
by pete318 closes #14355
|
|
* thx @Kittnz for providing the datas
closes #14363
|
|
closes #14356
|
|
* Events for Creatures on the 'Wyrmskull Bridge'
closes #14358
|
|
* Flameevent for Portals
closes #14359
|
|
closes #14360
|
|
closes #14361
|
|
closes #14364
|
|
closes #14367
|
|
|
|
|
|
Scripts/SunkenTemple: Add Eventsupport for NPC: Malfurion Stormrage
|