EssentialsX

EssentialsX

2M Downloads

Add intermediate level of teleportation toggle

RealSuperRyn opened this issue ยท 4 comments

commented

Feature description

Allows players to request teleportation (e.g. /tpa) but be unable to teleport without permission (e.g. /tp). TP being completely disabled would be /tptoggle low, generic teleportation being disabled with teleportation asking being enabled would be /tptoggle mid, TP being completely enabled would be /tptoggle high

How the feature is useful

Would remove the need for alternative commands to /tpa (e.g. /ride) when someone's TP is disabled. You would still be able to disable requests, but you would also be able to keep requests on.

Scenario A (current implementation):

Person A: /tpa B

B has teleportation disabled.

Person A: B, can you toggle TP on rq?

Person B: Sure!

Person B: /tptoggle enable

Troll: /tp B

Person A: /tpa B

Person B: /tptoggle disable

(The troll proceeds to grief the area he teleported into)

Scenario B (proposed implementation):

Person B: /tptoggle mid

Person A: /tpa B

B: /tpaccept

Troll: /tp B

B has instant teleportation disabled.

Troll: /tpa B

B: /tpdeny

(The troll does not manage to grief B's area.)

commented

It seems like the main problem here is that you're giving players access to /tp by default. Removing permission for that fixes your example scenario since an untrusted 3rd party will not be able to directly teleport.

You shouldn't think of /tptoggle as a way to protect players from teleports. It is simply a way of saying "I don't want others to teleport to me right now."

If the purpose of granting /tp was to avoid requiring teleportation requests, that can already be done using the /tpauto command, which automatically accepts teleportation requests. This practically makes /tpa the same as /tp if the player is okay with others instantly teleporting to them.

commented

It seems like the main problem here is that you're giving players access to /tp by default. Removing permission for that fixes your example scenario since an untrusted 3rd party will not be able to directly teleport.

You shouldn't think of /tptoggle as a way to protect players from teleports. It is simply a way of saying "I don't want others to teleport to me right now."

If the purpose of granting /tp was to avoid requiring teleportation requests, that can already be done using the /tpauto command, which automatically accepts teleportation requests. This practically makes /tpa the same as /tp if the player is okay with others instantly teleporting to them.

I guess a better way to phrase this then would be as an option like this: "/tp and /tpa", "/tpa only", "no teleports"

commented

I guess a better way to phrase this then would be as an option like this: "/tp and /tpa", "/tpa only", "no teleports"

You don't need to rephrase, I completely understand the suggestion here, but you will need to provide a better argument for why this is a necessary feature if you want it considered. The point I was trying to make above is that the kind of setup you have (granting both /tp and /tpa) is not standard practice and suitable alternatives already exist. If those alternatives are not acceptable, can you explain why?

commented

Closing since there is no further response here. If the provided solution is not sufficient, let us know.