Fallback to id field when name is missing in Minecraft velo plugin parsing

[Zoriot]

Closes #285.

[Osiris-Team]

Hey thanks for the PR! Very nice, just some small things:

• please keep the comment you removed, it should still be applicable
• please add a check for null/empty json element and add a nice exception with a better error message than the default json one right now if it fails in either case. While we are at it.

[Zoriot]

I thought about leaving the comment in but decided strongly against it.
I don't think there is any reason why this comment should stay in. That should be self explainable that a Jar Name is not automatically the Name of the Plugin and should not be used as that. Such Comments more clutter the codebase then it helps imo

Generally afaik only jar's which have a id, version and main class are considered valid (maybe more is needed). At this point in code we can assume that, because we already checked if there is a Velo Plugin spec.
Why do you think additional validation and a better exception exception is needed?
I think that's a Point which likely will never fire off / or in weird broken Plugin edge cases where someone manually bricked it.

Lmk if you still demand the changes and which ones.

[Osiris-Team]

k no problem