Moved all of the Warnings to JobsBridge, just to have them all in once place for easier bumping in the future.
Changed the Warning's field names to match their IDs, just a standard used in core/Denizen.
Updated jobs (tag base) to the ListTag convertor constructor.
Cleaned up the jobs_job constructor tag's handling to use JobsJobTag as the param type instead of handling it manually.
Moved JobsCommand registration to be with the rest of the registrations instead of at the bottom.
Updated JobsCommand to modern command handling (with legacy handling for out-of-order args).
Minor meta correction in JobsCommand, new line after dot.
Added tab-complete for the job names in JobsCommand.
JobsCommand now uses the methods on PlayerManager instead of the ones on JobsPlayer, as these are the ones that handle saving the changes and all (reported on Discord).
Updated JobsCommand to the modern switch syntax.
Gave all jobs event a general update (modern determinations, switch syntax, meta, and using tryArgObject for JobsJobTag matching).
Corrected the UUID in JobsJobTag's meta example (it was meant to be mcmonkey's UUID, but the wrong mcmonkey was used. This is a very big problem).
Implemented advancedMatches and added Matchable meta to JobsJobTag.
Improved JobsJobTag#valueOf's handling of the player UUID input - It now parses it as just a UUID instead of a PlayerTag, mainly as stuff like PlayerName,job shouldn't be allowed.
Also changed the checking of whether the player is valid to trying to get it from jobs + a null check, instead of PlayerTag#isValid.
Improved the handling for the job name input to now give a proper error and return null instead of throwing an NPE for invalid names.
Added a starts with prefix -> true into JobsJobTag#matches, before doing a valueOf != null.
Changed JobsJobTag#get/setOwner to use a UUID instead of a PlayerTag.
Did a bit of method re-ordering in JobsJobTag, to have less relevant methods such as get/setPrefix at the bottom and stuff like register higher up.
Marked ElementTags as plain-text where relevant.
Changed JobsJobTag.xp/max_xp (tag) to use early return.
Changed JobsJobTag.level to use a ternary operator.
Updated JobsJobTag.xp (mechanism) to modern mech registration.
Marked JobsJobTag.tagProcessor as final.
Implemented JobsJobTag#debuggable.
Updated JobsPlayerProperties to a modern JobsPlayerExtensions extension class.
Replaced places that got a jobs player by name with getting it by UUID.
Removed the registerPropertyTagHandlers call in JobsJobTag#register, as it doesn't look like it has any properties?
Removed the weird contains("@") -> null check in JobsJobTag#valueOf, which seems like it was a mistake or something? don't see a reason to have that, but correct me if I'm wrong.
Changed JobsJobTag#identify to use the new JobsJobTag#identify(prefix, separator) method.
Additions
jobsSingleLineDescription deprecation warning for JobsJobTag.description, as it was deprecated but didn't have a proper warning.
JobsJobTag#identify(String prefix, String separator) - used by identify and debuggable to avoid duplicate logic.
Changes
Warnings toJobsBridge, just to have them all in once place for easier bumping in the future.Warning's field names to match their IDs, just a standard used in core/Denizen.jobs(tag base) to theListTagconvertor constructor.jobs_jobconstructor tag's handling to useJobsJobTagas the param type instead of handling it manually.JobsCommandregistration to be with the rest of the registrations instead of at the bottom.JobsCommandto modern command handling (with legacy handling for out-of-order args).JobsCommand, new line after dot.JobsCommand.JobsCommandnow uses the methods onPlayerManagerinstead of the ones onJobsPlayer, as these are the ones that handle saving the changes and all (reported on Discord).JobsCommandto the modern switch syntax.tryArgObjectforJobsJobTagmatching).JobsJobTag's meta example (it was meant to bemcmonkey's UUID, but the wrongmcmonkeywas used. This is a very big problem).advancedMatchesand addedMatchablemeta toJobsJobTag.JobsJobTag#valueOf's handling of the player UUID input - It now parses it as just a UUID instead of aPlayerTag, mainly as stuff likePlayerName,jobshouldn't be allowed. Also changed the checking of whether the player is valid to trying to get it from jobs + a null check, instead ofPlayerTag#isValid.starts with prefix -> trueintoJobsJobTag#matches, before doing avalueOf != null.JobsJobTag#get/setOwnerto use aUUIDinstead of aPlayerTag.JobsJobTag, to have less relevant methods such asget/setPrefixat the bottom and stuff likeregisterhigher up.ElementTags as plain-text where relevant.JobsJobTag.xp/max_xp (tag)to use early return.JobsJobTag.levelto use a ternary operator.JobsJobTag.xp (mechanism)to modern mech registration.JobsJobTag.tagProcessorasfinal.JobsJobTag#debuggable.JobsPlayerPropertiesto a modernJobsPlayerExtensionsextension class.registerPropertyTagHandlerscall inJobsJobTag#register, as it doesn't look like it has any properties?contains("@") -> nullcheck inJobsJobTag#valueOf, which seems like it was a mistake or something? don't see a reason to have that, but correct me if I'm wrong.JobsJobTag#identifyto use the newJobsJobTag#identify(prefix, separator)method.Additions
jobsSingleLineDescriptiondeprecation warning forJobsJobTag.description, as it was deprecated but didn't have a proper warning.JobsJobTag#identify(String prefix, String separator)- used byidentifyanddebuggableto avoid duplicate logic.