EssentialsX

EssentialsX

2M Downloads

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

accLarson opened this issue ยท 1 comments

commented

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:
#5185 (/ignore <player> [time])
#3254 (Relevant pr relating link above)
#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.

commented

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.