Implementation of CommandFilters, replacing cooldowns and costs

[FrankHeijden]

Basically implements #3200, with just a small difference in yaml format: made the name required, as a key, just like kits.
Automatic converter is included, and will create the new yaml format from existing nodes.
Formats for command will be the same as with the command-costs, formats for pattern will be the same as in command-cooldowns.

Closes #3200, #3847

[Bobcat00]

How does this affect PR #3744 ?

[FrankHeijden]

How does this affect PR #3744 ?

They should go hand in hand with each other, defining a pattern filter in this version will behave the same as a regular command-cooldown in that PR

[Bobcat00]

They should go hand in hand with each other, defining a pattern filter in this version will behave the same as a regular command-cooldown in that PR

So this PR needs #3744 ? Just asking because 3744 hasn't been merged yet.

[FrankHeijden]

They should go hand in hand with each other, defining a pattern filter in this version will behave the same as a regular command-cooldown in that PR

So this PR needs #3744 ? Just asking because 3744 hasn't been merged yet.

I mean they are independent features -- #3744 fixes a bug where commands.yml defined commands aren't recognised by EssX, this one refactors the cooldowns/costs into a separate yml file as described per #3200 (and by doing that fixing a bug where periods couldn't be used in the regexes, because yml prohibits periods in keys).

[FrankHeijden]

So I just came across a thing which prevented AsyncTeleport's from applying command cooldowns - this is due to the fact AsyncTeleport creates a new Trade without the command defined in it. I added a new cooldownCommand parameter for the Trade#charge() command, so this method can know what command we should apply cooldowns on.

[JRoy]

Conflicts need to be addressed, please reopen when done so.

[FrankHeijden]

Cant reopen this one, opening #4224 instead

[FrankHeijden]

What’s the status on this PR? 😅

[JRoy]

What’s the status on this PR? 😅

This likely won't make it in time for 2.19 but 100% 2.20/2.19.x

[JRoy]

Needs to be updated to configurate (use EssentialsConfiguration over EssentialsConf)

[FrankHeijden]

Needs to be updated to configurate (use EssentialsConfiguration over EssentialsConf)

Done

[mdcfe]

Just a quick heads-up: I have a pending review for this PR, but I likely won't have time to properly review this for a few weeks.

[JRoy]

Remaining errors (EssentialsConf and logger stuff) need to be fixed. It's almost been 2 years since this PR was made, and I appreciate your persistence 😅. This pull request will be merged for 2.20 which is HOPEFULLY before the new year.

[FrankHeijden]

@JRoy Some build dependencies are timing out because they redirect to jcenter, which closed down. Do they need to be updated or is there a repo missing for these dependencies?