croxis / Lift

Minecraft/Bukkit elevator plugin
23 stars 26 forks source link

Add TeleportCause #181

Closed WMGameLive closed 4 years ago

WMGameLive commented 4 years ago

Add TeleportCause to make other plugins know what cause the teleport. Because Lift is just moving up and down, not really "teleport", so set it to UNKNOWN.

croxis commented 4 years ago

I've been out of the minecraft/bukkit/spigot loop for a while. what effect does teleportcause have with other plugins? I do plan on merging, just making sure I know what I am merging in :)

WMGameLive commented 4 years ago

For example, maybe some plugins will listen to PlayerTeleportEvent to get the teleport location and then do something. Lift just using teleport for moving up and down, not really teleport to somewhere, so it is better to add TeleportCause.UNKNOW to avoid other plugins thinking that players are teleport to somewhere.