/unignore alias of /ignore, is unexpected behaviour - possible ignore system overhaul?

[accLarson]

Type of bug

Other unexpected behaviour

/ess dump all output

https://essentialsx.net/dump.html?id=528fd2d68c3a496a99d0a124efcd62bb

Error log (if applicable)

No response

Bug description

The current ignore system is not very intuitive with unignore being a alias of ignore. The method of viewing your currently ignore players is not very intuitive either /ignore.

I've tagged here a few additional ideas that I think are worth considering related to the essentials ignore system: https://github.com/EssentialsX/Essentials/discussions/5185 (/ignore <player> [time]) https://github.com/EssentialsX/Essentials/pull/3254 (Relevant pr relating link above) https://github.com/EssentialsX/Essentials/issues/785 (/ignore list instead of /ignore)

Steps to reproduce

1. Execute /unignore <player> to ignore a player
2. Observe that this command can ignore players.
3. Execute /ignore <player> to unignore a player
4. Observer that this command can unignore players.
5. Execute /ignore
6. Observer that this command displays a list of currently ignored players

Expected behaviour

/ignore <player> should be used to ignore and /unignore <player> should be used to unignore. /ignore should return an error/suggestion on usage. /ignore list should return a list of currently ignored players. /ignore <player> [time] should allow a player to ignore another player for a set amount of time.

Actual behaviour

/ignore <player> can ignore and unignore. /unignore <player> can ignore and unignore. /ignore returns a list of currently ignored players. /ignore <player> [time] should allow a player to ignore another player for a set amount of time.

[Evidentsinger14]

This is intended behavior, so rather than an issue for a bug report, it should be an enhancement. I do like the idea of having a proper ignore system though. May give this a go myself eventually.