Essential-Commands icon indicating copy to clipboard operation
Essential-Commands copied to clipboard

Player warps

Open Soham52 opened this issue 4 years ago • 4 comments

Normal Player should also be able to set warps using the command /warp set [name] There can also be a config to limit the number of warps a player can have.

Soham52 avatar Aug 10 '21 05:08 Soham52

this would be great to have!

pahanakun avatar Jan 11 '22 05:01 pahanakun

In my mind this is better understood as a "public homes" system than "player warps" (mostly because warps are stored at the World level, whereas homes are stored at the Player level).

This leads me to a question:

Which of these do you think is more intuitive to use?

  1. Homes with a public/private flag
  2. A separate root command for public player homes. (e.g. /phome or /pwarp)

If 2, I'm open to suggestions for the root command name, as I'm not the biggest fan of the two above...

John-Paul-R avatar Feb 06 '22 20:02 John-Paul-R

I might take a crack at this. At the moment I am leaning toward simply adding it as another nbt compound in the existing world_data.dat over a dedicated pwarp.dat file.

It would have this structure:

pwarps:
-<name>:
     -headYaw: <f>
     -pitch: <f>
     -x: <f>
     -y: <f>
     -z: <f>
     -WorldRegistryKey: <s>
     -owner: [<i>]
     -private: <b>
     -blacklist: [<s>]
     -whitelist: [<s>]

SirReal314 avatar Mar 01 '23 14:03 SirReal314

Great, that structure seems reasonable.

I'd recommend extending (or, at least using internally) MinecraftLocation for storing these, handles some of the nbt interactions for you

John-Paul-R avatar Mar 02 '23 13:03 John-Paul-R