Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jumpdrive area protection check bugs #654

Open
Klaranth opened this issue May 14, 2021 · 3 comments
Open

jumpdrive area protection check bugs #654

Klaranth opened this issue May 14, 2021 · 3 comments
Labels
bug 🐛 Something isn't working

Comments

@Klaranth
Copy link

Apparently a JD cant bring a ship to jump onto the edge of a protected area.

Please consult Fexor and Huhhila for more information.

@S-S-X S-S-X transferred this issue from pandorabox-io/in-game May 15, 2021
@S-S-X S-S-X added the bug 🐛 Something isn't working label May 15, 2021
@FeXoR-o-Illuria
Copy link

FeXoR-o-Illuria commented May 17, 2021

Yes, the if check in line 160-162 is not sufficient.

My proposal would be to rewrite areas:canInteractInArea to:

  • Determine the areas overlapping with the target area
  • Segment the target area for each face, edge and corner of those areas inside it (Yea, that's the harder part) and store the owners in a list with each segment and make each subsegment "inherit" those. Early continue for areas of that user and open areas if allow_open.
  • Check for each segment if the user is in the segment's owner list (or open andallow_open or owner list empty)

Other ideas welcome ;)

(And why does none of those functions have an optional argument "areas" defaulting to self.areas? :D)

@OgelGames
Copy link
Contributor

eshattow

Area Protect with faction area open set enabled, other faction members cannot use jumpdrive; seems like a bug.

S-S-X

Factions are not compatible with jumpdrive and not really yet ready for wide use that's not only protection problem factions currently have

@OgelGames OgelGames changed the title JumpDrives unable to jump on end of protected area jumpdrive area protection check bugs Jun 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants