ModReq 1.7 - Documentation, bug fixes, enhancements (including CommandHelper alias features) #19

Merged

totemo merged 10 commits from master into master

2015-11-16 07:41:34 +00:00

totemo commented

2015-11-15 20:34:17 +00:00

(Migrated from github.com)

Bug fixes:

Numerous NullPointerExceptions in the version currently on nerd.nu (1.6) were fixed. For example, referencing a non-existent request ("/check 99999" or "/reopen 99999") or "/unclaim"ing a request that is in the OPEN, rather than the CLAIMED state.

Enhancements:

Most commands now support "-" instead of a request number, to signify the request most recently claimed by the staff member, e.g. "/tpi -", "/done - flowing", "/check -".
Each staff member's most recently claimed request number is remembered across restarts with a file.
The /elevate command automatically unclaims, e.g. "/tpc 1234" then "/elevate -" works as you would hope.
Added commands to perform composite actions: /tpinfo (/tpi) and /tpc. These commands have the same effect as in the CommandHelper aliases, but /tpc correctly handles the failure to /claim and does not proceed if some other moderator claims first. Another pull request will be introduced to remove the CommandHelper aliases.
The date/time format is configurable and set to a more reasonable default.
Added more informative error messages for /mr-note.
The /check page size can now be configured separately from the maximum number of requests a player can have open.

Documentation:

The README is now in Markdown format and is comprehensive enough to serve as player-facing help.
Permissions and configuration are also documented extensively.

Other changes:

The version number was bumped to 1.7.
The pom.xml file now references the Spigot repository so that the plugin can be built. The Bukkit repository is not accessible following DMCA takedown.
Factored out duplicated code for concatenating arguments into strings.
Factored out duplicated code for looking up the request by number and handling errors related to that.
Fixed compiler warnings.

Bug fixes: - Numerous NullPointerExceptions in the version currently on nerd.nu (1.6) were fixed. For example, referencing a non-existent request ("/check 99999" or "/reopen 99999") or "/unclaim"ing a request that is in the OPEN, rather than the CLAIMED state. Enhancements: - Most commands now support "-" instead of a request number, to signify the request most recently claimed by the staff member, e.g. "/tpi -", "/done - flowing", "/check -". - Each staff member's most recently claimed request number is remembered across restarts with a file. - The /elevate command automatically unclaims, e.g. "/tpc 1234" then "/elevate -" works as you would hope. - Added commands to perform composite actions: /tpinfo (/tpi) and /tpc. These commands have the same effect as in the CommandHelper aliases, but /tpc correctly handles the failure to /claim and does not proceed if some other moderator claims first. Another pull request will be introduced to remove the CommandHelper aliases. - The date/time format is configurable and set to a more reasonable default. - Added more informative error messages for /mr-note. - The /check page size can now be configured separately from the maximum number of requests a player can have open. Documentation: - The README is now in Markdown format and is comprehensive enough to serve as player-facing help. - Permissions and configuration are also documented extensively. Other changes: - The version number was bumped to 1.7. - The pom.xml file now references the Spigot repository so that the plugin can be built. The Bukkit repository is not accessible following DMCA takedown. - Factored out duplicated code for concatenating arguments into strings. - Factored out duplicated code for looking up the request by number and handling errors related to that. - Fixed compiler warnings.