Geyser users

[FamousL]

Hey, had a highly niche use case, one of my users is limited to bedrock only, which ordinarily would not be an issue, however we also have geyser/floodgate installed allowing bedrock connections.

The issue comes when attempting to start the Tardis seed, bedrock doesn't have a dedicated "use" key, and so long pressing is both "use" and "break", resulting in the seed being broken/reclaimed as an item.

To further add complexity, we are running as a world/user setup (multiverse, each tardis has its own world). As a result generating an abandoned/junk tardis is not possible (both are enabled in config, but disable during startup it seems).

I would like to suggest the ability to /tardisadmin create (user) (schematic) (etc) in this particular edge case, where the user themselves may be unable grow the seed.

[eccentricdevotion]

Bedrock on a device that doesn't have a mouse - ie tablet, game console? I can right-click in Bedrock on a PC...

[FamousL]

Correct, the device in question is a tablet. Edit to add: May have found a weakness in my suggestion, MV doesn't support names with special characters, floodgate users tend to have a special character to avoid collision with java users.

[eccentricdevotion]

Added, but untested in build 2447

[eccentricdevotion]

/tadmin create [player] [schematic] <wall> <floor> while targeting the block the TARDIS should grow on

[FamousL]

Made an attempt, but looks like MV is preventing the world from registering, since worlds cannot have special characters. Removing special characters from floodgate/geyser users could cause collisions with java users (In the below example, my java account is Famous_Longwing, and the bedrock account I tested is *Famous_Longwing. The bedrock account I opened this issue for is a different one, but figured this feedback would help)

org.bukkit.command.CommandException: Unhandled exception executing command 'tadmin' in plugin TARDIS v4.10.2-b2447
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:47) ~[paper-api-1.18.2-R0.1-SNAPSHOT.jar:?]
        at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:159) ~[paper-api-1.18.2-R0.1-SNAPSHOT.jar:?]
        at org.bukkit.craftbukkit.v1_18_R2.CraftServer.dispatchCommand(CraftServer.java:906) ~[paper-1.18.2.jar:git-Paper-341]
        at net.minecraft.server.network.ServerGamePacketListenerImpl.handleCommand(ServerGamePacketListenerImpl.java:2306) ~[?:?]
        at net.minecraft.server.network.ServerGamePacketListenerImpl.handleChat(ServerGamePacketListenerImpl.java:2117) ~[?:?]
        at net.minecraft.server.network.ServerGamePacketListenerImpl.handleChat(ServerGamePacketListenerImpl.java:2098) ~[?:?]
        at net.minecraft.network.protocol.game.ServerboundChatPacket.handle(ServerboundChatPacket.java:46) ~[?:?]
        at net.minecraft.network.protocol.game.ServerboundChatPacket.a(ServerboundChatPacket.java:6) ~[?:?]
        at net.minecraft.network.protocol.PacketUtils.lambda$ensureRunningOnSameThread$1(PacketUtils.java:51) ~[?:?]
        at net.minecraft.server.TickTask.run(TickTask.java:18) ~[paper-1.18.2.jar:git-Paper-341]
        at net.minecraft.util.thread.BlockableEventLoop.doRunTask(BlockableEventLoop.java:153) ~[?:?]
        at net.minecraft.util.thread.ReentrantBlockableEventLoop.doRunTask(ReentrantBlockableEventLoop.java:24) ~[?:?]
        at net.minecraft.server.MinecraftServer.doRunTask(MinecraftServer.java:1399) ~[paper-1.18.2.jar:git-Paper-341]
        at net.minecraft.server.MinecraftServer.d(MinecraftServer.java:188) ~[paper-1.18.2.jar:git-Paper-341]
        at net.minecraft.util.thread.BlockableEventLoop.pollTask(BlockableEventLoop.java:126) ~[?:?]
        at net.minecraft.server.MinecraftServer.pollTaskInternal(MinecraftServer.java:1376) ~[paper-1.18.2.jar:git-Paper-341]
        at net.minecraft.server.MinecraftServer.pollTask(MinecraftServer.java:1369) ~[paper-1.18.2.jar:git-Paper-341]
        at net.minecraft.util.thread.BlockableEventLoop.managedBlock(BlockableEventLoop.java:136) ~[?:?]
        at net.minecraft.server.MinecraftServer.tickServer(MinecraftServer.java:1447) ~[paper-1.18.2.jar:git-Paper-341]
        at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1225) ~[paper-1.18.2.jar:git-Paper-341]
        at net.minecraft.server.MinecraftServer.lambda$spin$0(MinecraftServer.java:315) ~[paper-1.18.2.jar:git-Paper-341]
        at java.lang.Thread.run(Thread.java:833) ~[?:?]
Caused by: java.lang.IllegalArgumentException: Invalid key. Must be [a-z0-9/._-]: tardis_world_*famous_longwing
        at com.google.common.base.Preconditions.checkArgument(Preconditions.java:220) ~[guava-31.0.1-jre.jar:?]
        at org.bukkit.NamespacedKey.<init>(NamespacedKey.java:52) ~[paper-api-1.18.2-R0.1-SNAPSHOT.jar:?]
        at org.bukkit.NamespacedKey.minecraft(NamespacedKey.java:145) ~[paper-api-1.18.2-R0.1-SNAPSHOT.jar:?]
        at org.bukkit.WorldCreator.<init>(WorldCreator.java:34) ~[paper-api-1.18.2-R0.1-SNAPSHOT.jar:?]
        at org.bukkit.WorldCreator.name(WorldCreator.java:472) ~[paper-api-1.18.2-R0.1-SNAPSHOT.jar:?]
        at me.eccentric_nz.TARDIS.planets.TARDISSpace.getTardisWorld(TARDISSpace.java:54) ~[TARDIS.jar:?]
        at me.eccentric_nz.TARDIS.builders.TARDISSeedBlockProcessor.processBlock(TARDISSeedBlockProcessor.java:117) ~[TARDIS.jar:?]
        at me.eccentric_nz.TARDIS.commands.admin.TARDISCreateCommand.buildTARDIS(TARDISCreateCommand.java:80) ~[TARDIS.jar:?]
        at me.eccentric_nz.TARDIS.commands.admin.TARDISAdminCommands.onCommand(TARDISAdminCommands.java:133) ~[TARDIS.jar:?]
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:45) ~[paper-api-1.18.2-R0.1-SNAPSHOT.jar:?]
        ... 21 more

[eccentricdevotion]

Change the floodgate username prefix to a ., _ or -?

[eccentricdevotion]

Or I could change the prefix when naming the TARDIS world...

[eccentricdevotion]

Build 2448

[eccentricdevotion]

Seems the default floodgate prefix is now a period? Just did a fresh install...

username-prefix: "."

[Chew]

Yep, it's been this for a few months now.

[FamousL]

Sorry, just had a chance to test this, the update is able to create a tardis for my bedrock user. However the box never materialized, and it looks like the map didn't spawn

Edit:
Purged the first attempt, and the second attempt worked correctly

[eccentricdevotion]

In my testing the exterior didn't materialise on the block I was looking at but about 8 blocks away. I haven't had a chance to find out why yet...

On another note, I installed MCPE on my iPad and joined my test server - it's a very different experience playing on a tablet as the chest GUIs are all out of whack :p

[FamousL]

indeed, it has been a challenge navigating ars, after that post i did notice a shell about the same distance away and thought another tardis on the server was to blame.

I wasn't paying much attention to where my bedrock user was looking, did it maybe use their line of sight instead of mine?

[eccentricdevotion]

Ah, you are entirely correct! I was using the player's line of sight rather than the command sender's. Fixed :)