Update Fields and cleanup BukkitClasses

[TheLimeGlass]

Description

Adds short hand methods for singleton Fields objects, aswell as removing the forced Serializer deserialize method when instantiated == false, when canBeInstantiated is majority of the time returned false, and even if it's true, Yggdrasil will take over on primitives (See number serializers). This removes the need for defining the public void deserialize(Example object, Fields fields) { method.

Before:

public Fields serialize(Example example) {
	Fields fields = new Fields();
	fields.putObject("name", example.getName());
	return fields;
}

After:

public Fields serialize(Example example) {
	return Fields.singletonObject("name", example.getName());
}

Also cleaned up the formatting of BukkitClasses to be like the top 80% of the class.

There is also a new mapper method for collecting singleton objects from Fields

Before:

@Override
protected Example deserialize(Fields fields) throws StreamCorruptedException {
	UUID uuid = fields.getObject("uuid", UUID.class);
	assert uuid != null;
	return ExampleManager.getExample(uuid);
}

After:

@Override
protected Example deserialize(Fields fields) {
	return fields.mapObject("uuid", UUID.class, ExampleManager::getExample);
}

Testing Completed

Skript tests

Other

StreamCorruptedException should be removed in the future with something like an Optional, or having the logger pickup Skript.error. As the main usage is for error reporting. Having try catches on every serialized element is wasteful resources. This will also allow for out of scope erroring, since Functions can't throw to the origin call stack.

---

Completes: none
Related: none

[sovdeeth]

looks good to me