benjaminforras / AdvancedKitsReloaded

AdvancedKitsReloaded is a Kit managing plugin for Minecraft servers which runs Bukkit, CraftBukkit, Spigot and Paper (>1.8). With this plugin you can easily create/edit/delete your kits and you can customize it as much as you'd like.
10 stars 4 forks source link

Cant load kit after restart #74

Open melodicaxe opened 6 years ago

melodicaxe commented 6 years ago

What steps will reproduce the problem?

  1. I created a daily kit. here is the paste bin of it https://pastebin.com/3GvZDQmh

What were you expecting to happen? What happened instead?

I expected that after server restart kit would work but it didn't. It works if I create the kit without of restarting. But after restart its all gone :( and here is the error:

[AdvancedKits] Error when trying to load 1-Daily

What version of the plugin are you using? Type /version <Plugin Name>

AdvancedKits Reloaded v1.1.14

What Spigot version are you using? Type /version

1.12.2-R0.1-SNAPSHOT

What plugins are you using? Type /plugins

Over 100 custom plugins

Do you have an error log? Use pastebin.com. If you're not sure, upload your whole server log

[AdvancedKits] Error when trying to load 1-Daily

Additional information? (Did it work in previous versions? etc.)

I am using mythic mobs and have some custom potions which are colorful perhaps the kits plugin can't store custom colored potions after restarts?

benjaminforras commented 6 years ago

Hello

It is definitely a bug in the plugin. I can only assume that the item serialization is broken. Sadly, my pc is broken and I can't update or maintain my plugins until I get a new pc.

Thanks for reporting.

melodicaxe commented 6 years ago

OH man that really sucks is there no way of you working on a temp laptop or anything. I can run test builds on my server?

benjaminforras commented 6 years ago

I don't have anything else then just a phone. Will fix it once I get myself a pc.

On Thu, Jan 4, 2018, 10:55 AM melodicaxe notifications@github.com wrote:

OH man that really sucks is there no way of you working on a temp laptop or anything. I can run test builds on my server?

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/TryHardDood/AdvancedKitsReloaded/issues/74#issuecomment-355241788, or mute the thread https://github.com/notifications/unsubscribe-auth/AJ4n4LehJqGZEp8XG_d-io1eqPL0qVYfks5tHKAdgaJpZM4RRwLf .

melodicaxe commented 6 years ago

Do you have a eta?

benjaminforras commented 6 years ago

At least a month

On Thu, Jan 4, 2018, 4:51 PM melodicaxe notifications@github.com wrote:

Do you have a eta?

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/TryHardDood/AdvancedKitsReloaded/issues/74#issuecomment-355317662, or mute the thread https://github.com/notifications/unsubscribe-auth/AJ4n4GU8SRorvrQEKek3jFIPBJ9hBp3bks5tHPNmgaJpZM4RRwLf .

melodicaxe commented 6 years ago

Hi will this bug be ever resolved?

Mick00 commented 6 years ago

The name of the file containing the kit is incorrect. Looking at the KitManager class, the file name should contain only letters and numbers and looking at your log it seems that you are using "-" in one of your kit which is not in the pattern[^A-Za-z0-9_]+. So try renaming the kit file and it should work. If you really want to use a "-" in your kit name you cant set the display name using the command /kit flag displayname