Added command to fix name changes #1
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added command /ta resident <name> Added command /ta resident <name> rename <newname> (to be used to fix name change issues) Added permission node towny.command.townyadmin.resident.* Added permission node towny.command.townyadmin.resident.rename Added event RenameResidentEvent(String oldname, Resident resident) Signed-off-by: backcab <[email protected]>
|
Manually merged to form 0.89.2.2 |
LlmDl
pushed a commit
that referenced
this pull request
Jul 17, 2017
Fixed many translation flaws and added some missing variables
LlmDl
pushed a commit
that referenced
this pull request
Feb 28, 2018
Update local repository with head.
ghost
mentioned this pull request
LlmDl
added a commit
that referenced
this pull request
Nov 28, 2020
- Mostly formatting things. - Probably caught a 2nd bug with the mysql implementation of ruins' hours.
LlmDl
added a commit
that referenced
this pull request
Nov 29, 2020
- Adds TownReclaimedEvent & TownRuinedEvent. - Cleans up TownRuinUtil to use the API in places it did not. - Changes PlotCommand test to check the townblock's town and not the player running the command. - Maxes out the RemainingHours at 1000.
LlmDl
added a commit
that referenced
this pull request
Nov 29, 2020
* Merges in the Ruined Town feature from Goosius' SiegeWar. - Only slight modifications made so far in the TownRuinUtil. * Cleanup pass #1 - Mostly formatting things. - Probably caught a 2nd bug with the mysql implementation of ruins' hours. * Clean up pass #2. - Adds a permission node to /town reclaim. * Polishing commit #1 - Adds TownReclaimedEvent & TownRuinedEvent. - Cleans up TownRuinUtil to use the API in places it did not. - Changes PlotCommand test to check the townblock's town and not the player running the command. - Maxes out the RemainingHours at 1000. * Polishing Commit #2 - Remove unnecessary isRuined() tests in DailyTimerTask. - the town.hasUpkeep() is false for all NPC mayor'd towns. * Final polishing commit #3 - Fix events running in async. - Add onLogin message warning of ruin status. - Fix ruined towns still paying upkeep. - Somehow they are not storing the hasUpkeep=false when their mayor is set to NPC. - Add some more language strings. - Move delete message to centralized removeTown() method. - Add remaining hours to the Town status screen. - Moved the config section to above the war section. - Town ruining is not dependant on war being used at all. * Actual final polishing commit #4 - Switchs out an int representing hours remaining for a ruined town, with a long representing the time the town went into ruin. - Allows for easier calculation of remaining hours in messages. - Means we aren't saving every ruined town with remaining hours on each newHour task. - Expanded messaging seen on town status screen to inlude how long until reclamation is possible. * Make sure reclamation only shows on the status screen if it is an option.
LlmDl
added a commit
that referenced
this pull request
Apr 26, 2021
* Trying out eclipse * Implement perm node for bypassing this * tweaks and adds limited commands * adds perm node to plugin.yml * try to get this to work * fix maven crying a river because nre was not caught * why are u crying about the javadocs * add fixes made by llmdl * adds checks that were missing * Change getTownLimitedCommands() to getPlayerOwnedPlotLimdCommands * remove unused import * add tests for owntown and townowned nodes * fix commands being blocked even if they shouldn't * limited cmds actually work now lol * requested changes (#1) * requested changes (#2) * fix bypass not checking for res township (req. changes) * remove repeated check left from older commits (req changes) * mvn exception not caught * Rebase off master and make a couple polishing changes. * Cleanup Commit & Requested Changes. - Requests from Warriorrrr are made. - Moves town-blocked commands above the personally-owned commands test to reduce the required returns later on. - Switches to the PermissionSource hasOwnTown and hasTownOwned permission tests. Co-authored-by: Llm Dl <[email protected]>
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added command /ta resident
Added command /ta resident
Added command /ta resident rename (to be used to fix
name change issues)
Added permission node towny.command.townyadmin.resident.*
Added permission node towny.command.townyadmin.resident.rename
Added event RenameResidentEvent(String oldname, Resident resident)
Signed-off-by: backcab [email protected]