Zones: Allow creating arbitrary subsets of a claim with different permission settings, etc. Resolves #49. DRAFT #54
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.
I added this since as a draft since:
This is a refactor adding an additional "scs_zones" table so that permissions (settings) can be changed for any size zone. A zone is a user-defined bounding box with corners at any X, Y, and Z. This feature is not #37 as it does not allow multiple owners, but it can at least change permissions.
scs_zonestable).Tasks:
getSelectedBBmethod that returns selection (or null if both corners are not selected for given player)scs_zonestable (CREATE TABLE, and CRUD methods)Zonesubclass ofClaimzonestoClaimthrow UnsupportedOperationExceptionif trying to doZoneCRUD operations or operations involving Claim-level settings on aZoneinstanceChunkmenu buttons to doZoneoperations if in a ZonegetString) and SettingsGui access to Zone version (with zone-based ids for the values; this prevents needing twice as many YML files in the guis folder) if in a Zone (make private and add/change methods to require either a Zone instance or null)addzone(mostly done, just need to check permissions)delzone(mostly done, just need to check permissions)setnamesetdescriptionPlease do not modify or make a secondary PR into this PR until I confirm I am ready, as I likely have pending local changes at any point. Please do not merge until I remove DRAFT from the title.
Please answer questions I have on #49 which are critical to making this work. Otherwise, for one thing, I may refactor
setnamesince it seems to have illogical/obfuscatedifconditions under it.