Skip to content

Conversation

@arboriginal
Copy link

Fixes:

  • Prevent the use of /rules when in a bed
    (otherwise, player can drag and drops menu items into his inventory)
  • When reloading the plugin, rules menu is no more cleared
    (also before, user who didn't accept can then close it)

Improvements:

  • Complete the plugin.yml (add author urls, declare permissions, ...)
  • Add a new permission "rules.*"
  • Allows to localize messages

Misc:

  • I've change the configuration variable name in MenuManager.reload() because it was the same than confirmation inventory

arboriginal added 6 commits November 29, 2018 19:09
Fixes:
- Prevent the use of /rules when in a bed
(otherwise, player can drag and drops menu items into his inventory)
- When reloading the plugin, rules menu is no more cleared
(also before, user who didn't accept can then close it)

Improvements:
- Complete the plugin.yml (add author urls, declare permissions, ...)
- Add a new permission "rules.*"
- Allows to localize messages
Their slot number was not read from the configuration file,
but hard coded.
- Allows to clear accept status for a single player
  (/rules -clear <player>)
- Allows to flag a player as already having accepter
  (/rules -accept <player>)
- Allows to check the accept status of a player
  (/rules -status <player>)
- RulesConfirmedEvent is cancellable
- When using /rules -clear, open the menu for online players
  (or for the player cleared only)
- Add a cancellable event fired when player refuses
- Makes the kick action optional
- Add a new permission: rules.exempt
onJoin and onClose didn't consider it, now fixed.
@arboriginal
Copy link
Author

I've created an addon for your plugin using those additions:
https://github.com/arboriginal/RulesCommands

Fell free to link it to your Spigot page if you integrate this PR.
If you are OK, I can also create a Spigot page for it
(with link to your plugin obviously, because it's a hard dependency).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant