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

Pearls don't affect the pearled unless they're in a tileentity. They should affect while held, too #30

Open
Lazersmoke opened this issue Mar 4, 2018 · 6 comments

Comments

@Lazersmoke
Copy link
Contributor

From @GavJenks on March 3, 2018 23:37

  1. This is strictly a bug, since the instructions text on the config says "The radius around their pearl that an exiled player can't enter" which is definitely false as written.

  2. Even if rewritten, this is a feature we should have. yes, it may be annoying in some cases if a player is repeatedly teleported, but should be a feature anyway, because the plugin also allows for damage mode instead of teleporting, and it's pretty reasonable in that case. Even for teleporting, unless the pearler walks in exactly the direction of the victim, it shouldn't matter often.

Apologies if this is already a feature, but I looked through the config instructions and didn't see mention of it. Just "pearl_radius" alone

Copied from original issue: CivClassic#37

@Lazersmoke
Copy link
Contributor Author

@ProgrammerDan
Is there a reason why this exists?
https://github.com/CivClassic/ExilePearl/blob/32907821fefa6bfb3e0d218cf4933e77cbeeaad8/src/main/java/com/devotedmc/ExilePearl/core/PearlBoundaryTask.java#L149-L152
Is there something buggy that can happen or is it just to prevent weirdness like described above where you can get bumped back over and over?

@dquist
Copy link
Contributor

dquist commented Mar 4, 2018

I'm pretty sure Jon asked for this to be added after release to prevent the situation you're describing, and the config/docs were never updated.

It could probably be a config option.

@ProgrammerDan
Copy link
Contributor

Yeah the idea is it would be trivially easy to antagonistically use pearls in this case. Just load up a hot-potato player with pearls of a town you want to haze, run through, and enjoy the screams of frustration. This way, pearls had to be at least be marginally "fixed" in location; so yeah, you can still do functionally the same, but at least you have to do some effort.

So, not a bug; but, for sure not documented clearly.

@GavJenks
Copy link

GavJenks commented Mar 5, 2018

That's kind of the ONLY usage for pearls themselves excluding people though, on a server with bastions.

If you have bastions, why wouldn't you turn off pearls doing anything, and just exclude from bastions (or damage in bastions)? Is that an option yet?

@ProgrammerDan
Copy link
Contributor

I'm confused. You just have to put it in a chest. That's all...? They work as described then...?

@ProgrammerDan
Copy link
Contributor

Oh, now I understand you. Late. You can just make the radius super tiny (0 or 1) and that's accomplished. Been available since day 1.

Maxopoly pushed a commit to Maxopoly/ExilePearl that referenced this issue Jun 24, 2018
Diet-Cola pushed a commit to Diet-Cola/ExilePearl that referenced this issue Jul 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants