Paper icon indicating copy to clipboard operation
Paper copied to clipboard

Couple fixes & cleanup for the new teleport API

Open Machine-Maker opened this issue 3 years ago • 0 comments
trafficstars

Changes other calls to PlayerTeleportEvent to use the new dismount/relative flags api. Also fixes several teleport locations that were incorrectly telling the client to dismount when vanilla doesn't do that.


Future

PlayerPortalEvent extends PlayerTeleportEvent, yet neither of the new fields on PlayerTeleportEvent have anything to do with the PlayerPortalEvent. "dismount" and "relative flags" don't apply to PlayerPortalEvent since no actual teleporting is happening regarding that event.

Since its all still marked as experimental, I think those new fields should be moved to a new sub-event of PlayerTeleportEvent so they don't have to be included in PlayerPortalEvent.

Machine-Maker avatar Jul 26 '22 05:07 Machine-Maker