azerothcore / azerothcore-wotlk

Complete Open Source and Modular solution for MMO
http://www.azerothcore.org
GNU Affero General Public License v3.0
6.63k stars 2.64k forks source link

Target is in line of sight through the Texture [$40 awarded] #281

Closed Irome closed 7 years ago

Irome commented 7 years ago

Description: If someone casts any spell, the texture is ignored, allowing you to attack or being attack, out of sight of the target. This happens in PvE and PvP.

Current behaviour: If a player is hidden behind a mountain without having a visual on the target, it can be hit without any problems through the texture.

Expected behaviour: Target is not in line of sight, so it wouldn't be possible to cast any spell.

Steps to reproduce the problem:

  1. Take two characters and bring them on any map.
  2. Hide a character under a hill or a mountain, and the other character above, without it being possible to see the hidden target.
  3. You have to cast any spell to the target over the mountain, or vice versa.

These two images should help you understand the problem:

  1. http://tinypic.com/view.php?pic=24v4bb9&s=9#.WFJPhqnrxwE
  2. http://tinypic.com/view.php?pic=2a4wfh5&s=9#.WFJPQ6nrxwF

In these images we understand that none of the characters should be attacked because it is out of the target view.

When he was tested: it was tested on 14.12.2016 with the presence of a staff member to confirm the obvious problem.

AC hash/commit: 9d0c8dc71bb867218a949644b72592b452a7e62b

Operating system: Windows 10 64bit

Bountysource

--- The **[$40 bounty](https://www.bountysource.com/issues/40153074-target-is-in-line-of-sight-through-the-texture?utm_campaign=plugin&utm_content=tracker%2F40032087&utm_medium=issues&utm_source=github)** on this issue has been claimed at [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F40032087&utm_medium=issues&utm_source=github).
Yehonal commented 7 years ago

Could someone confirm it?

Il 17/Dic/2016 02:04, "Irome" notifications@github.com ha scritto:

Description: If someone casts any spell, the texture is ignored, allowing you to attack or being attack, out of sight of the target. This happens in PvE and PvP.

Current behaviour: If a player is hidden behind a mountain without having a visual on the target, it can be hit without any problems through the texture.

Expected behaviour: Target is not in line of sight, so it wouldn't be possible to cast any spell.

Steps to reproduce the problem:

  1. Take two characters and bring them on any map.
  2. Hide a character under a hill or a mountain, and the other character above, without it being possible to see the hidden target.
  3. You have to cast any spell to the target over the mountain, or vice versa.

These two images should help you understand the problem:

  1. http://tinypic.com/view.php?pic=24v4bb9&s=9#.WFJPhqnrxwE
  2. http://tinypic.com/view.php?pic=2a4wfh5&s=9#.WFJPQ6nrxwF

In these images we understand that none of the characters should be attacked because it is out of the target view.

When he was tested: it was tested on 14.12.2016 with the presence of a staff member to confirm the obvious problem.

AC hash/commit: 9d0c8dc https://github.com/azerothcore/azerothcore-wotlk/commit/9d0c8dc71bb867218a949644b72592b452a7e62b

Operating system: Windows 10 64bit

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/azerothcore/azerothcore-wotlk/issues/281, or mute the thread https://github.com/notifications/unsubscribe-auth/AAI-lDS7D-7XjdrmqOhrRQl_5Fwu0S30ks5rIzUXgaJpZM4LPuqx .

sucofog commented 7 years ago

Confirmed @Irome @Yehonal

In my case i created a warlock, and casted Shadow Bolt trough texture in Warsong Gulch.

Expected behaviour: As you said @Irome target is not in LoS, so target 1 cannot cast spells on target 2 if it's not on LoS.

Yehonal commented 7 years ago

If someone has latest rev of trinitycore to do a test , we can compare it eventually and checking for a solution

falzard commented 7 years ago

Maybe related to https://github.com/azerothcore/azerothcore-wotlk/issues/250 (read comments, there is a partial fix)

Yehonal commented 7 years ago

thanks @falzard , this is the fix : https://bitbucket.org/maczuga/sunwellcore/commits/1749037808d1de4f963b65aef117decacf78a8aa

we should try it and create a PR

sucofog commented 7 years ago

@Yehonal I'm going to test @Yehonal but the problem is in that commit there are also other files edited so how can we figure out what are the changes related to this concrete issue? Also, the mmaps/vmaps/maps extractors of Azerothcore are broken as fuck i think, maczuga is updating this with a working extractors i think, correct me if i'm wrong =P

Yehonal commented 7 years ago

@Maczuga could you help us to create this PR?

ghost commented 7 years ago

I also backported tc's vmaps etc and didin't run into this issue, i'll see if I can make a PR this week but no promise, kind of busy

sucofog commented 7 years ago

If you do, you get the 20$ bounty @Blumfield

falzard commented 7 years ago

Someone?

falzard commented 7 years ago

Just wondering if it happens on trinitycore

Viste commented 7 years ago

yes it happens to but cases when it happen to loww like tree on warsong and etc small objects

Yehonal commented 7 years ago

it happens with hills, climbs, objects and even walls

it's quite critical. However, just FYI, when it will be fixed we also have to import a fix about hit calculation on npc area perimeter and not center to avoid bugs such as Sindragosa Tombs

I explained it here: https://github.com/TrinityCore/TrinityCore/issues/6541#issuecomment-147715990

Viste commented 7 years ago

In that issue golrag fix los calculation for GO https://github.com/TrinityCore/TrinityCore/pull/15807/files

Yehonal commented 7 years ago

Yes i know, but this is a fix that is needed if the LOS on gobject and structure is working.

Currently our issue is in the "root".

BarbzYHOOL commented 7 years ago

Some say it's blizzlike, some say it's not, and some say it's blizzlike but it's shit (me).

Dany proposed to make it a config option if fixed.

Estaira commented 7 years ago

@Yehonal try this:

diff --git a/src/server/game/Entities/Object/Object.cpp b/src/server/game/Entities/Object/Object.cpp index e4d9e61..bf23e41 100644 --- a/src/server/game/Entities/Object/Object.cpp +++ b/src/server/game/Entities/Object/Object.cpp @@ -1147,7 +1147,10 @@ bool WorldObject::IsWithinLOSInMap(const WorldObject* obj) const return false;

 float ox, oy, oz;

UPDATE creature_model_info SET BoundingRadius=7.5, CombatReach=6 WHERE DisplayID=30890;

Fix: https://github.com/TrinityCore/TrinityCore/issues/6541#issuecomment-147715990 "Golrag"

https://github.com/TrinityCore/TrinityCore/issues/15807

Yehonal commented 7 years ago

@Kizarr it's another kind of issue

Yehonal commented 7 years ago

I suspect that we can't fix it, event if as a config, because the vmap information are missing in client

So maybe you should edit maps