aboutsummaryrefslogtreecommitdiff
path: root/src
AgeCommit message (Collapse)Author
2019-07-17Core/Achievement: don't increase ACHIEVEMENT_CRITERIA_TYPE_RECEIVE_EPIC_ITEM ↵Wyrserth
counters for items bought back from vendors.
2019-07-17Core/Script: fix logical mistake in spell_gen_clone_weapon_aura.Wyrserth
2019-07-17Core/Pet: generate a new name for Death Knights' Ghouls when a new one is ↵Wyrserth
summoned. Unlike Warlock pets, whose names are generated only once because they're the same demon that is summoned over and over, Death Knight pets are different entities generated from different corpses (or corpse dust). So each has their own, randomly generated, name.
2019-07-17Script/BoreanTundra: move Pile of Fake Furs' script (related to quest Unfit ↵Wyrserth
for Death) from ItemScript to SpellScript. Closes #21890.
2019-07-17Core/GameObject: do not allow players to interact with gameobjects that use ↵Wyrserth
the "Point" icon. Updates #21890.
2019-07-17Core/Quest: when receiving an item required for a quest, update every quest ↵Wyrserth
in the quest log instead of stopping at the first objective that matches the received item. Closes #10586.
2019-07-17Core/Player: always set the player's dugeon/raid difficulty to the group's ↵Wyrserth
dungeon/raid difficulty, regardless of level. Fixes cases where players could enter heroic dungeons without meeting the heroic version requirements.
2019-07-17Core/Player: don't send errors to client from Satisfy* methods when ↵Wyrserth
Player::CanRewardQuest() is called with msg = false. Fixes cases where people would get strange errors like "You are not high enough level for that quest" when entering WotLK instances.
2019-07-17Core/DB: Add RBAC for .debug poolstatus, because I need it for the pooling ↵Treeston
branch.
2019-07-16Core/Misc: Fix no-pch buildjackpoz
2019-07-16Core/DB: Merge pool_creature, pool_gameobject and pool_pool into a single ↵Treeston
pool_members table. (More pooling prep, I really don't want to have any sql update files in that PR if I can help it.)
2019-07-16Core/Addons: Fixed Blizzard_* addons insecure lua errorsShauren
Closes #23599
2019-07-16Scripts/Commands: Add .debug questreset to force daily/weekly/monthly quest ↵Treeston
reset.
2019-07-16Core/Chat: Properly reload saved channel settings from DB after 8c16f31.Treeston
2019-07-16Core/DB: Add a world state for daily quest reset time to `characters`. Not ↵Treeston
used right now, but written to allow easier transition later.
2019-07-15Core/Authserver: Fixed crash happening when sending packets larger than 4096 ↵Shauren
bytes (giant realmlist basically) Closes #23597
2019-07-15Fix clang buildShauren
2019-07-15Core/Addons: Improve WorldSession::ReadAddonsInfoShauren
* Only allow built-in addons to be marked as secure * Better protect against malformed packets
2019-07-15Core/Spells: Spell pushback now only affects players (as it should)Treeston
2019-07-15Core/Misc: Various dynspawn cleanup and refactors split off from pooling ↵Treeston
rewrite: - Map::RemoveRespawnTime(SpawnObjectType, LowType, doRespawn) split into Map::Respawn and Map::RemoveRespawnTime, without the extra boolean - Map::RemoveRespawnTime(RespawnInfo*) merged into Map::DeleteRespawnInfo(RespawnInfo*) and is now private - Map::DeleteRespawnInfo(void) renamed to Map::UnloadAllRespawnInfos to properly describe what it does - Map::ProcessRespawns now actually saves the delayed respawn time to DB if the respawn was delayed - Map::AddRespawnInfo now takes const reference, and returns success as a boolean - Map::AddRespawnInfo no longer offers an unused "replace" parameter - Map::DeleteRespawnInfo no longer offers a variety of unused private overloads - Map::SaveRespawnTime no longer offers a tantalizing writeDB parameter. Parameter is now called "startup" to properly describe what it does. - Map::SaveRespawnInfoDB now takes RespawnInfo reference instead of all the various fields. Still public because compatibility mode. QQ. - Map::GetWorldObjectBySpawnId sanitized - Map::GetXRespawnTime methods sanitized to all go through Map::GetRespawnTime
2019-07-15Core/Spawns: Exterminate CONFIG_SAVE_RESPAWN_TIME_IMMEDIATELY with extreme ↵Treeston
prejudice. (It didn't work anyway.)
2019-07-14Core/DB: Unify `creature_respawn` and `gameobject_respawn` into a single ↵Treeston
`respawn` table
2019-07-14Build: fix build errorccrs
In file included from /home/circleci/project/src/server/game/Entities/Player/Player.cpp:19: In file included from /home/circleci/project/src/server/game/Entities/Player/Player.h:22: In file included from /home/circleci/project/src/server/game/Entities/Unit/Unit.h:22: In file included from /home/circleci/project/src/server/game/Entities/Object/Object.h:22: In file included from /home/circleci/project/src/common/Common.h:23: In file included from /usr/bin/../lib/gcc/x86_64-linux-gnu/8/../../../../include/c++/8/memory:63: In file included from /usr/bin/../lib/gcc/x86_64-linux-gnu/8/../../../../include/c++/8/bits/allocator.h:46: In file included from /usr/bin/../lib/gcc/x86_64-linux-gnu/8/../../../../include/x86_64-linux-gnu/c++/8/bits/c++allocator.h:33: /usr/bin/../lib/gcc/x86_64-linux-gnu/8/../../../../include/c++/8/ext/new_allocator.h:136:23: fatal error: no matching constructor for initialization of 'WorldPackets::Spells::ResyncRune' { ::new((void *)__p) _Up(std::forward<_Args>(__args)...); } ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~ /usr/bin/../lib/gcc/x86_64-linux-gnu/8/../../../../include/c++/8/bits/alloc_traits.h:475:8: note: in instantiation of function template specialization '__gnu_cxx::new_allocator<WorldPackets::Spells::ResyncRune>::construct<WorldPackets::Spells::ResyncRune, unsigned char &, unsigned int &>' requested here { __a.construct(__p, std::forward<_Args>(__args)...); } ^ /usr/bin/../lib/gcc/x86_64-linux-gnu/8/../../../../include/c++/8/bits/vector.tcc:103:21: note: in instantiation of function template specialization 'std::allocator_traits<std::allocator<WorldPackets::Spells::ResyncRune> >::construct<WorldPackets::Spells::ResyncRune, unsigned char &, unsigned int &>' requested here _Alloc_traits::construct(this->_M_impl, this->_M_impl._M_finish, ^ /home/circleci/project/src/server/game/Entities/Player/Player.cpp:24611:22: note: in instantiation of function template specialization 'std::vector<WorldPackets::Spells::ResyncRune, std::allocator<WorldPackets::Spells::ResyncRune> >::emplace_back<unsigned char &, unsigned int &>' requested here packet.Runes.emplace_back(type, cooldown); ^ /home/circleci/project/src/server/game/Server/Packets/SpellPackets.h:117:16: note: candidate constructor (the implicit copy constructor) not viable: requires 1 argument, but 2 were provided struct ResyncRune ^ /home/circleci/project/src/server/game/Server/Packets/SpellPackets.h:117:16: note: candidate constructor (the implicit move constructor) not viable: requires 1 argument, but 2 were provided /home/circleci/project/src/server/game/Server/Packets/SpellPackets.h:117:16: note: candidate constructor (the implicit default constructor) not viable: requires 0 arguments, but 2 were provided 1 error generated.
2019-07-14Core/Server: 3785a35 followupccrs
2019-07-14Core/DB: Improved terrible assertion messageShauren
2019-07-13Server/Packets: implement SMSG_RESYNC_RUNES packet structureccrs
2019-07-13Core/Commands: Add .debug guidlimits <optional map id> chat commandjackpoz
Add .debug guidlimits <optional map id> chat command to show the highest low guid counter for Creature and GameObject, useful when reaching Respawn.GuidWarnLevel
2019-07-13Scripts/DK: correctly handle Blood Tapccrs
2019-07-13Core/Chat: Rewrite some custom channel handling. Channel creation now ↵Treeston
properly saves passwords. Closes #23589.
2019-07-13Core/Calendar: Fix issue with invite status being set to the wrong invitejackpoz
Consider the case of Player A inviting Player B to a Calendar Event. After Player B clicks on Accepts/Tentative/Reject, the calendar shows that status for Player A instead.
2019-07-13Core/Player: apply LevelReq.Trade only to the player initiating a trade, ↵Wyrserth
aligning it with the other LevelReq.* config options.
2019-07-13Core/Player: prevent trade system from getting stuck in an invalid state ↵Wyrserth
when the LevelReq.Trade config is used.
2019-07-12Scripts/ScarletEnclave: 4ee84bb followupccrs
2019-07-12Scripts/ScarletEnclave: 4ee84bb followupccrs
2019-07-12Core/Unit: movement related corrections prior to getting Charmedccrs
2019-07-12Script/ScarletEnclave: use spline movement for the lauch phase of the Eye of ↵Wyrserth
Acherus. Closes #22802.
2019-07-12Battleground/Arena: Properly check RBAC arena join permission before ↵Treeston
allowing queue. Closes #23000.
2019-07-12Scripts/Commands: .account ban no longer fails if an account containing the ↵Treeston
specified account name is already banned (f.ex. attempting to ban 'test' if 'test2' is already banned) - closes #22990
2019-07-12typo fix, duplicate assignmentTreeston
2019-07-11Entities/GO: FindNearestGameObject no longer incorrectly returns despawned ↵Treeston
gameobjects. Closes #23262.
2019-07-11DB/Quest: Seeds of the Blacksouled KeepersKillyana
2019-07-10Core/Weather: 6eecb68 followupccrs
missing IntervalTimer::Update call
2019-07-10Core/ChatCommands: Fix .wp reload crashGiacomo Pozzoni
2019-07-10DB/Quest: restore ExclusiveGroup data for breadcrumb quests, as they're ↵Wyrserth
mutually exclusive. Also restore PrevQuestId/NextQuestId error logs involving breadcrumb quests. Special cases must be handled via conditions.
2019-07-09Core/Quest: remove arbitrary error logs related to breadcrumb quest chaining.Wyrserth
There are complex cases where breadcrumb quests are mutually exclusive with other breadcrumbs leading to other quests, chaining them shouldn't generate log errors.
2019-07-09Scripts: Update boss_akilzon weather packetShauren
2019-07-09Core/Weather: Fixed weather updates to be thread safeShauren
(cherry picked from commit 6eb997394722fcd4b5248646b5abfa185a7ec58f)
2019-07-09Core/Packets: Update and enable SMSG_WEATHERNayd
(cherry picked from commit 6f7d048765f102110cbf9ad4653c078fbadaa105)
2019-07-09Scripts/World: Rewrite air force trigger bots script, add hostility check ↵Treeston
along the way. Closes #23307.
2019-07-09Core/Movement: add missing parenthesis on MotionMaster::Sizeccrs
pretty sure this can cause unexpected behaviour