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

Spawn point for jailed people #55

Closed
ghost opened this issue Mar 12, 2015 · 17 comments
Closed

Spawn point for jailed people #55

ghost opened this issue Mar 12, 2015 · 17 comments
Labels

Comments

@ghost
Copy link

ghost commented Mar 12, 2015

It'd also be nice to be able to set a spawn point for the people in a specific jail, or have the option for them to spawn inside the jail after they die

@graywolf336
Copy link
Owner

Does it not currently do that? Are the people in a cell or just the jail?
On Mar 12, 2015 1:03 PM, "motheman" notifications@github.com wrote:

It'd also be nice to be able to set a spawn point for the people in a
specific jail, or have the option for them to spawn inside the jail after
they die


Reply to this email directly or view it on GitHub
#55.

@ghost
Copy link
Author

ghost commented Mar 12, 2015

They are just in a jail. Haven't tested with a cell

@graywolf336
Copy link
Owner

And you're saying when they die while in jail they don't respawn back into the jail?

@ghost
Copy link
Author

ghost commented Mar 12, 2015

Correct

@graywolf336
Copy link
Owner

Do you have any other plugins which handle that? Because it currently does that: https://github.com/graywolf336/Jail/blob/master/src/main/java/com/graywolf336/jail/listeners/PlayerListener.java#L210

@ghost
Copy link
Author

ghost commented Mar 12, 2015

I have essentials installed so maybe that is messing with this plugins config? Not sure how the priority would work in that case.

@graywolf336
Copy link
Owner

Well, the plugin currently handles the event at medium priority which
allows for other plugins to handle it. Guess we should make this one higher
priority so we are the last ones to handle it as sending people back to
jail is rather important.

On Thu, Mar 12, 2015 at 1:23 PM, motheman notifications@github.com wrote:

I have essentials installed so maybe that is messing with this plugins
config? Not sure how the priority would work in that case.


Reply to this email directly or view it on GitHub
#55 (comment).

@ghost
Copy link
Author

ghost commented Mar 12, 2015

Is that a quick fix?

@graywolf336
Copy link
Owner

It'll be done tonight when I get home from work.

On Thu, Mar 12, 2015 at 1:29 PM, motheman notifications@github.com wrote:

Is that a quick fix?


Reply to this email directly or view it on GitHub
#55 (comment).

@ghost
Copy link
Author

ghost commented Mar 12, 2015

fantastic.

@ghost
Copy link
Author

ghost commented Mar 12, 2015

Think you could do that and include the broadcasting thing as well for me?

@graywolf336
Copy link
Owner

We'll see about the broadcast one, I've got a busy day and I would rather
do the broadcast with more thought put into it. So, no promises.
On Mar 12, 2015 1:32 PM, "motheman" notifications@github.com wrote:

Think you could do that and include the broadcasting thing as well for me?


Reply to this email directly or view it on GitHub
#55 (comment).

@ghost
Copy link
Author

ghost commented Mar 12, 2015

no worries. thanks anyway

graywolf336 added a commit that referenced this issue Mar 13, 2015
Since another plugin might interfere and teleport someone elsewhere
which isn't inside the Jail.
@graywolf336
Copy link
Owner

The build for this is building at the moment, if you don't mind downloading it and trying it out then let me know the results that'd be great! 😄

@ghost
Copy link
Author

ghost commented Mar 13, 2015

Won't be able to get to test it for a while but if you link me directly I can forward it to someone to test it for you.

On 13 Mar 2015, at 04:38, graywolf336 notifications@github.com wrote:

The build for this is building at the moment, if you don't mind downloading it and trying it out then let me know the results that'd be great!


Reply to this email directly or view it on GitHub.

@graywolf336
Copy link
Owner

Had a chance to test this yet?
On Mar 13, 2015 2:42 PM, "motheman" notifications@github.com wrote:

Won't be able to get to test it for a while but if you link me directly I
can forward it to someone to test it for you.

On 13 Mar 2015, at 04:38, graywolf336 notifications@github.com wrote:

The build for this is building at the moment, if you don't mind
downloading it and trying it out then let me know the results that'd be
great!


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub
#55 (comment).

@graywolf336
Copy link
Owner

Closing due to the changing being in for a while now and no status updates.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant