kroimon/bukkit-alphachest

A fork of the original AlphaChest plugin by Mast3rPlan

bukkit-alphachest's Issues

Deprecated code

'ItemStack(int, int, short)' is deprecated in InventoryIO
Mojang are moving away from IDs and so it would be wise to update this soon :)

load(InputStream stream) deprecated

Utf8YamlConfiguration

load(InputStream steam) overrides a deprecated method in 'org.bukkit.configuration.file.FileConfiguration'

AlphaChest Major Update

Alright so I'm creating this as a sort of plan, request for a possible AlphaChest v2.0.0. I want to get people's opinions/suggestions for this and especially kroimon and Brianetta's thoughts.

I know that AlphaChest is still only at 1.18.0 in terms of versioning but because this idea may involve a fair few changes I thought it might be worth bumping up the version to something like 1.2.0 or even 2.0.0.

I've put together some ideas I had in mind below.

Core:

• Add world restrictions via something like permissions
• Add MySQL as an additional method of storing saved chests (keep flat-file storage as the default method though)
• Add an (/alphachest) base command with several subcommands such as help, reload, info... etc

Configuration:

• Add more configurable options
• Implement common YAML standards/conventions (e.g. changing silentAutosave option to silent-autosave)

Permissions:

• Remove uppercased letters in permission nodes
• Split some permissions into 'groups' (e.g. alphachest.keepOnDeath -> alphachest.death.keep... etc)
• Add separate permission nodes for opening other player's chests and clearing other player's chests (e.g. alphachest.admin.chest & alphachest.admin.clear)
• Change alphachest.* to something like alphachest.basic as the current might seem like it would give ALL AlphaChest permissions

Error when i type /chest (bukkit #1143)

16:52:06 [INFO] [OKLogger] [COMMAND] <Charant(188.233.22.221)>@world(-588,77,-139) : /chest
16:52:06 [SEVERE] null
org.bukkit.command.CommandException: Unhandled exception executing command 'chest' in plugin AlphaChest v1.7.2
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:41)
        at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:163)
        at org.bukkit.craftbukkit.CraftServer.dispatchCommand(CraftServer.java:353)
        at net.minecraft.server.NetServerHandler.handleCommand(NetServerHandler.java:755)
        at net.minecraft.server.NetServerHandler.chat(NetServerHandler.java:720)
        at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:713)
        at net.minecraft.server.Packet3Chat.a(Packet3Chat.java:33)
        at net.minecraft.server.NetworkManager.b(NetworkManager.java:226)
        at net.minecraft.server.NetServerHandler.a(NetServerHandler.java:91)
        at org.getspout.spout.SpoutNetServerHandler.a(SpoutNetServerHandler.java:500)
        at net.minecraft.server.NetworkListenThread.a(SourceFile:108)
        at net.minecraft.server.MinecraftServer.h(MinecraftServer.java:464)
        at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:374)
        at net.minecraft.server.ThreadServerApplication.run(SourceFile:417)
Caused by: java.lang.NullPointerException
        at net.minecraft.server.TileEntityChest.e(TileEntityChest.java:219)
        at net.minecraft.server.InventoryLargeChest.e(InventoryLargeChest.java:71)
        at net.minecraft.server.ContainerChest.<init>(SourceFile:14)
        at net.minecraft.server.EntityPlayer.a(EntityPlayer.java:423)
        at com.mast3rplan.alphachest.commands.ChestCommands.performChestCommand(ChestCommands.java:55)
        at com.mast3rplan.alphachest.commands.ChestCommands.onCommand(ChestCommands.java:29)
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:39)
        ... 13 more
16:52:06 [INFO] Charant lost connection: disconnect.endOfStream

WIndow Names

Window names are limited to a maximum of 16 characters. Creating a new chest will crash clients because "'s Virtual Chest" is too long.

UUID conversions don't work on Cauldron

Since this works completely fine on Spigot and Bukkit, could you please explain to the Cauldron team what needs fixing?

Thanks

MinecraftPortCentral/Cauldron#1556

Failing to convert old chest file names to UUID

I am using the latest Spigot version and have learnt that the plugin is failing to convert the old username files to the new UUID file names.

I will try and add some of my own debugging to find more specific problems here.