Commit Graph

21991 Commits

Author SHA1 Message Date
ccrs
a58ed271c9 Scripts/PoS: a7a96d4 followup
cause edge cases may exist
ty Aokromes
2019-10-27 02:28:28 +01:00
ccrs
06c6b30ed4 Scripts/ScarletMonastery: instance script cleanup (#23889)
Adapt InstanceScript into newer structure and style
Adapt existing boss scripts into newer structure and style
Improve existing boss related definitions

Should make each boss more functionally reliable, but nothing new has been added
2019-10-26 23:33:30 +02:00
jackpoz
00703ee238 Core/Misc: Enable MaxCoreStuckTime setting by default
Enable MaxCoreStuckTime setting by default to force worldserver to exit in case of infinite loops.
Note that this will not affect existing setups with the setting MaxCoreStuckTime set to 0 in the local worldserver.conf .
2019-10-26 19:49:41 +02:00
ccrs
fcbcb43b2b Scripts/PoS: a7a96d4 followup
we cant despawn all summons since vehicle spawn their passengers as regular summons
ty Aokromes
2019-10-24 10:51:27 +02:00
ccrs
40116f1025 Scripts/PoS: a7a96d4 followup 2019-10-23 17:39:25 +02:00
ccrs
a7a96d4271 Scripts/PoS: little implementation cleanup in boss_krickandick with, you guessed it, movement related corrections
also, never forgetti  -->
                                     ¨¨Ç*^¿?_:;_^!"·Ç!Ñ·$%/(/)/()(=)(=]]{
Unit::StopMoving() is evil, and so is MotionMaster::Clear()

the first can really mess up existing movement generators
the second can interrupt "controlled" movements... can do it literally mid-air
2019-10-23 17:29:34 +02:00
ccrs
8fd33dfd79 Build: fix nopch - add missing declaration 2019-10-23 14:06:17 +02:00
ccrs
d27fd717c1 Scripts/PoS: move boss_krickandick into new script definition 2019-10-23 12:53:46 +02:00
ccrs
c9c6871659 Core/Movement: prevent any Initialize queue till AddToWorld is signaled 2019-10-23 03:28:57 +02:00
ccrs
bc9a9ad7e1 Core/Movement: d033032 followup
now THATS a typo... sorry folks
2019-10-23 03:26:49 +02:00
ccrs
5d056f37c7 Core/Movement: d033032 followup
ty jackpoz
2019-10-22 15:48:00 +02:00
ccrs
bb9af06cd4 Core/AI: 06443e3 followup 2019-10-22 13:36:23 +02:00
ccrs
fe7c2c3772 Core/AI: add missing public tag & correct alignment 2019-10-22 12:46:20 +02:00
Gildor
e1df7b751e Core/Spells - Rogue: Fix Unfair Advantage range (#23877)
Closes #23873
2019-10-21 19:34:17 +02:00
ccrs
bc507209a7 Core/Movement: d033032 followup
ty jackpoz
2019-10-21 16:49:25 +02:00
ccrs
a71c5f4c7f Core/AI: 06443e3 followup
cause its not "that" horrible
2019-10-21 16:43:39 +02:00
ccrs
d033032d33 Core/Movement: delay movement update in MotionMaster until its owner signals AddToWorld
ref #23199
closes #23876
2019-10-21 13:55:23 +02:00
Aokromes
69b7958ebc Core/Totem: Change log level of a log
To much spam of people doing quests with totem-like creatures
2019-10-21 10:19:52 +02:00
jackpoz
84937337da Revert "Core/Movement: Fix a crash on login"
This reverts commit 9941b4c2ed.
2019-10-20 12:25:14 +02:00
jackpoz
9941b4c2ed Core/Movement: Fix a crash on login
Fix a crash in MotionMaster happening when logging in with an aura of type SPELL_AURA_MOD_CONFUSE
2019-10-19 23:05:22 +02:00
jackpoz
90bb10aa3a Core/GameObject: Fix battleground flags not showing burning animation
Fix battleground flags not displaying the typical burning animation when they "despawn" (they don't actually despawn, they just change state)
2019-10-17 20:18:25 +02:00
jackpoz
30823f9b79 Core/Misc: Fix C++17 build errors 2019-10-13 20:12:36 +02:00
ForesterDev
c03c88d0c0 Core/Defines: define UNIT_FLAG_SWIMMING (#23630) 2019-10-13 17:43:23 +02:00
Giacomo Pozzoni
fed030da19 Scripts/Kologarn: Fix "stuck in combat" issues (#23866)
Fix players stuck in combat with NPC_ARM_SWEEP_STALKER and/or both arms after killing the boss.
2019-10-13 17:37:52 +02:00
ForesterDev
f4be544629 Core/Achievements: fixed bug with counting honorless units for achievement type ACHIEVEMENT_CRITERIA_TYPE_SPECIAL_PVP_KILL (#23844)
* Core/Achievements: fixed bug with counting honorless units for achievement type ACHIEVEMENT_CRITERIA_TYPE_SPECIAL_PVP_KILL

* Remove codestyle changes

* Remove codestyle changes
2019-10-12 20:51:31 +02:00
jackpoz
06443e38e7 Core/AI: Add new method to force creatures around to stop combat 2019-10-12 20:29:39 +02:00
Ovah
7fcb77ecd1 Core/Taxis: teleport players to the destination taxi node location instead of their current ground position (#23835)
* Core/Taxis: teleport players to the destination taxi node location instead of straight to their current ground position

* fixed a typo
2019-10-08 11:11:24 +02:00
Giacomo Pozzoni
abcad7cda5 Core/LFG: Fix re-queueing while in a dungeon (#23862)
Fix an issue that wouldn't allow to re-queue after joining a dungeon, having 1 member leave/get kicked, queue, leave, queue.
In this case a message "One or more dungeons was not valid" would be displayed because LFG would try to queue the group as "continue dungeon" but the current group was not marked as "in dungeon" state anymore.
2019-10-06 22:24:13 +02:00
jackpoz
ada4e431ae Core/LFG: Remove Dungeon Cooldown when getting kicked 2019-10-05 20:40:17 +02:00
jackpoz
88317a4ec8 Core/Misc: Fix issues reported by static analysis
Fix issues reported by Coverity Scan
2019-10-04 19:16:45 +02:00
Keader
a108f2ba61 Scripts/ICC: Fixed some issues with Rotface Oozes (#23845) 2019-10-04 09:35:22 -03:00
Keader
8210b4a539 Scripts/ICC: Fixed Death Plague (Frost Giant) (#23847) 2019-10-04 09:33:14 -03:00
Keader
34ee2effc8 Scripts/Forge of Souls: Fixed spell Soul Sickness (#23843) 2019-10-03 08:49:05 -03:00
Keader
16fde1ffad Scripts/ICC: Fixed Tear Gas not getting removed of Abomination in phase change of PP (#23846) 2019-10-02 16:50:45 -03:00
Giacomo Pozzoni
b3b8ec811d Core/Pet: Fix guardians low damage (#23840)
Fix guardians low damage for unhandled cases in Guardian::InitStatsForLevel()
2019-10-02 13:59:52 +02:00
Giacomo Pozzoni
ef694cc251 Scripts: Fix crossfaction combat issue (#23818)
* Scripts/PitOfSaron: Fix Martin Victus combat in crossfaction groups

Fix Martin Victus attacking players in crossfaction groups because it was spawned as summon of the boss, triggering DoCombatInZone()

* Scripts/ForgeOfSouls: Fix guards combat in crossfaction groups

Fix Devourer of Souls guardians attacking players in crossfaction groups because they were spawned as summons of the boss, triggering DoCombatInZone()

* Scripts/Misc: Fix no PCH build
2019-09-28 15:45:25 +02:00
Ovah
c1e3d79a4f Core/Movement: fixed taxi pathings getting finalized earlier than intended (#23830)
* Core/Movement: fixed taxi pathings getting finalized earlier than intended

Since we handle the intial starting point of a spline properly now the update logic was still expecting the old handling which resulted in incrementing the current node index earlier than intended

* apply a missing change
2019-09-28 09:24:41 +02:00
Giacomo Pozzoni
ca8953a0ec Core/Misc: Fix static analysis issues
Fix static analysis issues reported by Coverity
2019-09-27 14:12:10 +02:00
Keader
746a1d89b3 Scripts/Tempest Keep: Fixed a typo in Nethermancer Sepethrea
Closes #23828 
Ty Rizztazz
2019-09-25 16:56:03 -03:00
Giacomo Pozzoni
ed08754663 Core/Misc: Fix static analysis issue
Fix static analysis issue
2019-09-25 11:44:08 +02:00
tarwyn
c8dc9fdc41 Fix inconsistent NPC aggro on spell miss (#23816) 2019-09-23 08:01:28 +02:00
Ovah
cf3277c641 Core/Movement: provide an extra spline path vertex for taxi pathings … (#23817)
* Core/Movement: provide an extra spline path vertex for taxi pathings so the first path point wont get lost.

* Update FlightPathMovementGenerator.cpp
2019-09-22 17:12:59 +02:00
Killyana
8e3862cfbd Instance/Script: Gorkun Ironskull & Martin Victus 2019-09-21 21:52:33 +02:00
jackpoz
f9d6c57006 Core/Quest: Fix quests not being shareable 2019-09-21 18:26:55 +02:00
jackpoz
42ab6010f8 Core/Map: Move a log from ERROR to WARN
Move "Could not load MMAP" log from ERROR to WARN as 99% of times is a false positive
2019-09-21 18:05:19 +02:00
Gildor
a392216d20 Scripts/Icecrown Citadel: Remove duplicated Spell enum (#23810) 2019-09-21 13:17:45 +02:00
Giacomo Pozzoni
fb059722fd Core/Calendar: Add some additional validation when creating events (#23797)
* Core/Calendar: Add some additional validation when creating events

Allow only 30 player events and 100 guild events to be created.
Don't allow to create guild events if player is not in guild.
Send some more error messages to the client (not blizzlike errors but better than nothing).

* Core/Calendar: Add some additional validation/checks

Add guild id check in GetPlayerEvents().
Change error message in HandleCalendarCopyEvent() to be the same as in HandleCalendarAddEvent() when creating an event in the past.

* Core/Calendar: Add some additional validation/checks

Reduce the number of CMSG_CALENDAR_ADD_EVENT packets a seconds allowed from 10 to 3.

* Core/Calendar: Add some additional validation/checks

Implement 5 seconds cooldown between the creation of calendar events

* Core/Calendar: Add some additional validation/checks

Don't allow to copy events of a different player/guild

* Core/Calendar: Implement automatic deletion of old events

Implement automatic deletion of events older than 1 month (30 days).
Fix debug assertion triggered when deleting a character with calendar events.
Avoid double std::set lookup when deleting events when deleting a character.

NB: The whole CalendarMgr/CalendarHandler code should be checked line by line for bugs/crashes/exploits.
2019-09-17 09:11:56 +02:00
Kelno
6fc7ab3448 Core/Commands: Fixed wrong permissions for .lookup map commands (#23800) 2019-09-16 17:41:55 +02:00
jackpoz
215a6cee24 Core/Movement: Fix LoS issue of NPCs chasing targets
Add Line of Sight checks to ChaseMovementGenerator::Update(), fixing to ChaseMovementGenerator not reaching a point with valid LoS to the target.

Fix re-implements 8927a04253 after 2a84562dc8 partially removed it.

Fix #23724
2019-09-13 21:06:28 +02:00
Gildor
b98f858cd2 Spells/DK: Fix wrong class name in Rime AuraScript (#23783)
(Introduced in this commit: 2ff855054f)
2019-09-10 16:29:38 +02:00