ccshiro / cc-buglist

Corecraft's issue tracker
43 stars 40 forks source link

Polymorph Wandering #806

Open YourNewfriend opened 9 years ago

YourNewfriend commented 9 years ago

On Corecraft, polymorphed targets don't wander like they should.

Retail: https://www.youtube.com/watch?v=ROnfPE7h4Kw&feature=youtu.be&t=1m39s

This video isn't sped up at all. Compare it to the current Corecraft polymorph. Maybe some more recent TBC footage is necessary.

ccnim commented 9 years ago

Some TBC footage would be nice, yes.

YourNewfriend commented 9 years ago

https://youtu.be/1xoawQ4gLak?t=26s

The guy is spinning around in circles at some points like a dog chasing its own tail, and that definitely doesn't look anything like the sheep on CC.

Jigodian commented 9 years ago

Thank you for reporting this game-breaking bug!

darkcranio commented 9 years ago

Fuck your shitposting, every bug is a bug some high some are low priority.

YourNewfriend commented 9 years ago

Creature: https://www.youtube.com/watch?v=Msd24voy-kw https://youtu.be/qLSdgKiY9f4?t=14s https://youtu.be/bNps4nlwdG8?t=36s https://www.youtube.com/watch?v=spClTnD2N1o

Player: https://www.youtube.com/watch?v=YIRHaRLKOzk https://www.youtube.com/watch?v=jPQfnsDvdQM https://youtu.be/K8UQSBujguA?t=44s https://youtu.be/Erohy5g_jws?t=12m42s https://youtu.be/qLSdgKiY9f4?t=4m36s

YourNewfriend commented 9 years ago

https://youtu.be/IEnEbZsKPj8?t=8s

robinsch commented 9 years ago

My research, doesn't include Polymorph: // Spells with SPELL_AURA_MOD_CONFUSE and BasePoints == 1 (or higher) should skip finalized check in movement function // This will effect Scatter Shot, Blind movementgen

just added a bool at ConfuseMovement which will skip the check for destinationHolder.HasArrived.