marcelo-mason / PreciousStones

Self-service protection for Minecraft servers
http://dev.bukkit.org/server-mods/preciousstones/
56 stars 66 forks source link

IDEA/BUG Multiple fields with same block type, dynamic config loading #1058

Open Omanoctoa opened 7 years ago

Omanoctoa commented 7 years ago

I've encountered an unexpected issue. I'm trying to create fields that give potion effects (works) and can be 'upgraded', by being crafted with a Nether Star (works). The problem...is PreciousStones is not registering the different variations. Here's a code snippet...

`

I've called them different names but they used to be called Crystal Lens of Fire, Fire II and Fire III. I assumed it was ignoring the repeating III and just reading the last one, but after some testing I realized it was actually ignoring the other 2 fields. It appears PS is not able to register different field types using the same block type, which kind of breaks my idea.

My second point, is to suggest dynamic configuration loading. Basically, when PS is reloaded or is restarting, read the fields individually instead of all at once. If it encounters a problem with a field, simply ignore that field, print a semi-useful error in Console (bonus!) and continue loading the rest. Craftbook, one of my favourite plugins, is incredibly easy to use because of this. If you could set up a similar system so if a field has errors it is simply skipped, this would be phenomenal and definitely make creating custom fields less frustrating. :)

marcelo-mason commented 7 years ago

You cannot use the same block type+data combo. You can use same block type with different data though, like different shades of grass etc. Unfortunately, this is one of the bedrock principles of the plugin, everything works around the uniqueness of type+data. This cannot change without re-writing the whole plugin from scratch.

The loading thing might be a good idea. Though limited use as wanting to load up a server with broken fields is a very rare thing.

Omanoctoa commented 7 years ago

I figured that was the case. The wiki/guide didn't mention that so I wasn't aware. It also details the incorrect format for listing potion effects and intensities. It details it as being a [single-line list], whereas it is actually supposed to be