- Creature: update current waypoint to store nodeId and pathId
- MotionMaster: change variable type on GetMotionSlotType and GetMotionSlot to keep consistency and prevent errors (ASSERT is now no longer needed)
- UnitAI: add new waypoint hooks WaypointPathStarted and WaypointPathEnded
- SAI: handle WAYPOINT related events if creature is no escorting
* SMART_EVENT_WAYPOINT_RESUMED still not implemented for no escorting
TODO: the new hooks can save, now duplicated, logic on EscortAI and SAI
closes#20777
updates #20310
updates 21bd52cb99
(cherry picked from commit e10d7dd45c)
remove home position set from creature PauseMovement, and use it when necessary
change log level on DelayedDelete
(cherry picked from commit cd4775420a)
Following the work done in #19361 this is the cleanup and improvement of the related logic of waypoint management.
Ref 28050f3#18020
(taking the good parts and ignoring the incomplete work)
(cherry picked from commit 7fff83d675)
- Get rid of fugly void pointers
- Streamlined AI selection code, using Permissible as it should've been instead of harcoded checks on the selector code
- Moved IdleMovementGenerator singleton to factory code
(cherry picked from commit ce3787f190)
Use own movement generator for creatures following on a formation.
First step of implementation, huge room for improvement.
Closes#19422
(cherry picked from commit 46221b6dc1)
Movement/SplineChain: Bump value range for DB chainId up to uint16 (0 to 65535) from uint8 (0 to 255). Turns out sniffs generate far more chains than I expected.
(cherry picked from commit 2170541a51)
Code style follow-up, I blame Notepad++.
(cherry picked from commit 7860da0de6)
(cherry picked from commit 6d00d3f283)
Merge remote-tracking branch 'Treeston/3.3.5-splinechains' into 3.3.5 (PR #17946)
(cherry picked from commit 20f483967f)
Core/Movement: Add a convenience default ctor to SplineChainResumeInfo, and fix PCH build in some configurations (zzz why do we even keep Appveyor and Travis around).
(cherry picked from commit 4fa646c0b2)
PCH build fix. Again.
(( Alright, you made me waste 20 minutes of my life on a full nonPCH rebuild of the core now. ))
(( I hope you're happy. ))
(cherry picked from commit 4a1a460241)
Also make sure the creature is alive before updating.
Fixes#12823.
(cherry picked from commit 2db9269ec1)
DB/Creature: World Trigger (Not Immune NPC) InhabitType update
Closes#17332 by Malcrom
(cherry picked from commit 78d08834c0)
DB/Item: Fix text in Rune-Inscribed Parchment
Closes#17329 by tkrokli
(cherry picked from commit 8879545f21)
DB/Quest: improve chaining for "The Ashenvale Hunt" quests
Closes#17323 by SnapperRy
(cherry picked from commit d034eedb81)
DB/trinity_string: wrong string about character customization + minor fix
Closes#17306 by lyn1337
(cherry picked from commit 0951e559eb)
DB/Quest: fix some completion/offer reward quest texts
Closes#17336 by SnapperRy
(cherry picked from commit e9025036b9)
DB/Quest: add queststarters for "Regthar Deathgate"
Closes#17337 by SnapperRy
(cherry picked from commit 0b8921a0ef)
DB/Quest: fix chaining for "The Real Threat"
Closes#17338 by SnapperRy
(cherry picked from commit c9cb86d6c5)
DB/Quest: fix repeatableness (or lack of) for some quests
Closes#17339 by SnapperRy
(cherry picked from commit b9f2e2a81b)
DB/Quest: improvements for "Fool's Stout"
Closes#17340 by SnapperRy
(cherry picked from commit 4ec753dc50)
Remove error message and allow spell cast or item use from
mounted state on ground, especially when sitting on
a landed flying mount. Current behavior is that the player
will get the error message "You can't do that while moving"
when trying to cast a spell or pick up an item
while sitting on a flying mount after having landed.
By joschiwald, closes#6236
(cherry picked from commit 19aac66bc2)
thanks @MitchesD for help
(cherry picked from commit df21162fe4)
(cherry picked from commit 3a4f54197c)
(cherry picked from commit f50df72820)
(cherry picked from commit 95bae7143f)
(cherry picked from commit 5406f82044)
