SunstriderEmu / BugTracker

Endless.gg bugtracker
https://endless.gg
30 stars 25 forks source link

Flare effect ignores line of sight. #1382

Closed Jerrythemagicdog closed 5 years ago

Jerrythemagicdog commented 5 years ago

Describe the bug Flare (1543) ignores Line of Sight restrictions when it's casted on the ground. The effect extends through walls and pillars.

Screenshots of current incorrect behaviour:

https://cdn.discordapp.com/attachments/525320741420531742/620237230992130048/unknown.png

https://cdn.discordapp.com/attachments/525320741420531742/620237676553175041/unknown.png

Expected behavior It should not extend through line of sight from the starting point.

woiwoz commented 5 years ago

Let me check this.

kelno commented 5 years ago

This affect a whole bunch of spells. Some spells to check, should all those ignore LoS?

https://db.endless.gg/?spell=27173 https://db.endless.gg/?spell=16914 https://db.endless.gg/?spell=4629 https://db.endless.gg/?spell=1510 https://db.endless.gg/?spell=13810 (shouldn't according to Jerry)

woiwoz commented 5 years ago

Don't look. It's the right thing to do. Including snowstorms. Nor should we attack targets behind walls.

Jerrythemagicdog commented 5 years ago

None of the other spells you have listed should ignore LoS. Not sure if Frost Trap should, this is just something common on private servers, need actual proof.

Jerrythemagicdog commented 5 years ago

Frost Trap going over pillar visually: https://youtu.be/VhCI3DDvGGU?t=72 (pretty much the entire video)

kelno commented 5 years ago

Okay so conclusion of our research:

Visual on retail doesn't matter and should be ignored, only the debuff should be looked at. Frost trap currently isn't affected by LoS and should be. Earthbind totem currently is and should be. Flare currently isn't affected and should. Blizzard currently is and should.

So what's wrong here is: spell that triggers other spells are correctly affected but area aura spells directly applying an aura are broken.

kelno commented 5 years ago

Fixed