New movement -> Mutate -> Finish current for that slot that happens to be the same type, Follow -> Finish method on the removed movement -> UpdateSpeed call -> search for top -> crash
Closes#21861
(cherry picked from commit bcab5bc4e7ac3866e86a54a1aa17e1174672ce75)
- Chase to angle is now functional. Pets use this to chase behind the target. Closes#19925.
- Chase to arbitrary range interval works. Not used anywhere, but you can technically make hunter-like mobs.
- Pets now follow the hunter cleanly and without stutter stepping. Also fix some other things. Closes#8924.
(cherry picked from commit 2a84562dc85516f432bb1e5de9add23c28b26ce4)
* Core/Entities: Fix some weird movement due to los issues
- Made LoS check use collisionHeight instead of midsection. Value was too low.
- Gnomes will now have a breath bar more quickly than for example a tauren.
- Changes have been made to checking for ground z as well, some tweeking might be needed but removed most of the scattered +2.0f/+0.5f/we
* Add 0.05f to isInAir check in Creature::UpdateMovementFlags
(cherry picked from commit e42903ec16f4849b71bc8736e3151a1e11d3cd25)
- Use Midsection height for LoS checking.
- Changed MotionMaster::MoveJumpTo to use correct z. This change also makes sure the _owner will jump towards the given angle. Instead of jumping to a unintended angle if the first one is not in LoS.
(cherry picked from commit 95456ab5d8bd623481d315ae55dfbb44b45ba9f5)
- 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 e10d7dd45cfc073c791cd5e5f3f964d14b37df3d)
since MOTION_SLOT_IDLE cannot be expired, signal path done and behave like IdleMotionGenerator
(cherry picked from commit eb2769996f01bfd7d7118503a44705711eb309a3)
remove home position set from creature PauseMovement, and use it when necessary
change log level on DelayedDelete
(cherry picked from commit cd4775420a4894012616d54467eb3827430245b5)
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 7fff83d67526efff63867d41b9e036a19a9287b3)
- 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 ce3787f190d28c1e981598eef2cd24c622a2cbae)
Use own movement generator for creatures following on a formation.
First step of implementation, huge room for improvement.
Closes#19422
(cherry picked from commit 46221b6dc1272989e0de85a4563fc49dba8e7851)
And restore broken _cleanFlag check/use on update (eca3645d8bb655b88e39f05e36b435171f738f70)
(cherry picked from commit 96f4c7c187e03aeb61e905421751f3d5a1df060e)
Fixed HP drop bug on zeppelins
Fixed possible packet relocation problems on huge transports
(cherrypicked from b3f04ec1570c7fd6ef0d1dacfb7ff9f8ab00a43d)
(cherry picked from commit d3214a0012cf93a5e66d1dc405ac90c4abc1b8c3)
d3214a0 follow-up for code style.
(cherry picked from commit 4deeee66bda390d2b35b64aaeb5924dac4170e10)
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 2170541a51ced3c15675b8854e0ae49461884f8c)
Code style follow-up, I blame Notepad++.
(cherry picked from commit 7860da0de6e53c7740862cae18840c9d399dda7f)
(cherry picked from commit 6d00d3f28380dd0811a18913450b1dc4f07fef48)
Merge remote-tracking branch 'Treeston/3.3.5-splinechains' into 3.3.5 (PR #17946)
(cherry picked from commit 20f483967fb3a430fbd54148c08b6d8e8937daac)
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 4fa646c0b2ae3261e7fab249f4177900d4c8d013)
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 4a1a460241acf511467decd92d2a30e19927d74d)
Also make sure the creature is alive before updating.
Fixes#12823.
(cherry picked from commit 2db9269ec1c06b5fccc45dc0a5b4f714f9b48804)
DB/Creature: World Trigger (Not Immune NPC) InhabitType update
Closes#17332 by Malcrom
(cherry picked from commit 78d08834c0afe0be09e4ec071a6e6855aa18e2c8)
DB/Item: Fix text in Rune-Inscribed Parchment
Closes#17329 by tkrokli
(cherry picked from commit 8879545f21fe13650872358741b687a524cf8cb9)
DB/Quest: improve chaining for "The Ashenvale Hunt" quests
Closes#17323 by SnapperRy
(cherry picked from commit d034eedb811912546025d11a5e9335bcd6600464)
DB/trinity_string: wrong string about character customization + minor fix
Closes#17306 by lyn1337
(cherry picked from commit 0951e559ebe6b76abdfef55619244578d9917002)
DB/Quest: fix some completion/offer reward quest texts
Closes#17336 by SnapperRy
(cherry picked from commit e9025036b9848a5a684c57e3e066687c29fde321)
DB/Quest: add queststarters for "Regthar Deathgate"
Closes#17337 by SnapperRy
(cherry picked from commit 0b8921a0ef81fb7c4741248301b17a86fa0ffb69)
DB/Quest: fix chaining for "The Real Threat"
Closes#17338 by SnapperRy
(cherry picked from commit c9cb86d6c53ceb7331a1a5c1ff8a69f569cca426)
DB/Quest: fix repeatableness (or lack of) for some quests
Closes#17339 by SnapperRy
(cherry picked from commit b9f2e2a81b7f0a269da3a8f9597a4eca17899781)
DB/Quest: improvements for "Fool's Stout"
Closes#17340 by SnapperRy
(cherry picked from commit 4ec753dc50927e358a63a49a56cef7f5a72f41eb)
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 19aac66bc262a835cae288fc3d0ccdc8e93d0fb9)
ccrs