Skip to content

protection-claims.md

Latest commit

 

History

History
631 lines (467 loc) · 30.5 KB

protection-claims.md

File metadata and controls

631 lines (467 loc) · 30.5 KB
Raw

Claim Protection

Version: 0.10.0 | Source: protection/ProtectionChecker.java, data/FactionPermissions.java

How faction territory claims are protected. For zone protection, see protection-zones.md. For cross-cutting concerns (wilderness, explosions, fire), see protection-global.md.

Overview

Faction claims are chunk-based territory owned by factions via /f claim. Protection priority:

Zone > Claim > Wilderness
  • Zone: Admin SafeZone/WarZone with 52 flags — checked first, overrides claims
  • Claim: Faction territory with 57 permission flags — checked only when NOT in a zone
  • Wilderness: Unclaimed land — no protection, all interactions allowed

Source: ProtectionChecker.canInteractChunk()

Faction Permissions (57 Flags)

Source: FactionPermissions.java ALL_FLAGS constant

Per-Level Interaction Flags (4 levels x 12 = 48 flags)

Each level has the same 12 flag suffixes. Flag name = {level}{Suffix} (e.g., memberBreak, allyDoorUse).

Suffix Controls Parent
Break Block breaking —
Place Block placement —
Interact General block interaction (parent) —
DoorUse Doors and gates {level}Interact
ContainerUse Chests and storage {level}Interact
BenchUse Crafting tables {level}Interact
ProcessingUse Furnaces, smelters {level}Interact
SeatUse Seats and mounts {level}Interact
TransportUse Teleporters and portals {level}Interact
CrateUse Capture crate pickup and placement (mixin) —
NpcTame F-key NPC taming (mixin) —
PveDamage Damage non-player entities (mobs) —

Levels: outsider, ally, member, officer

Default Values

Flag Suffix Outsider Ally Member Officer
Break false false true true
Place false false true true
Interact false true true true
DoorUse false true true true
ContainerUse false false true true
BenchUse false false true true
ProcessingUse false false true true
SeatUse false true true true
TransportUse false true true true
CrateUse false false true true
NpcTame false false true true
PveDamage false true true true

Summary: Members/officers get full access. Allies can interact, use doors/seats/transport, and damage mobs but cannot break/place/access containers/benches/furnaces/crates/taming. Outsiders are denied everything.

Mob Spawning Flags (4 flags)

Flag Default Parent
mobSpawning true —
hostileMobSpawning true mobSpawning
passiveMobSpawning true mobSpawning
neutralMobSpawning true mobSpawning

Global Flags (2 flags)

Flag Default Notes
pvpEnabled true Per-faction territory PvP toggle
officersCanEdit false Whether officers can modify faction permissions

Treasury Flags (3 flags)

Flag Default Notes
treasuryDeposit true Members can deposit into treasury
treasuryWithdraw false Officers can withdraw from treasury
treasuryTransfer false Officers can transfer to other factions

Treasury flags are exposed in the TreasurySettingsPage GUI (accessible via /f treasury settings or /f settings treasury tab). They can also be configured via faction-permissions.json defaults/locks. See Managing Permissions.

Parent-Child Flag Hierarchy

Source: FactionPermissions.getParentFlag(), get()

When a flag has a parent, FactionPermissions.get() checks the parent first. If the parent is false, the child returns false regardless of its stored value.

// FactionPermissions.get()
public boolean get(@NotNull String flagName) {
    String parent = getParentFlag(flagName);
    if (parent != null && !getRaw(parent)) {
        return false;  // Parent is false → child is implicitly false
    }
    return getRaw(flagName);  // Otherwise return stored value
}

Relationships

{level}Interact (parent)
├── {level}DoorUse
├── {level}ContainerUse
├── {level}BenchUse
├── {level}ProcessingUse
├── {level}SeatUse
└── {level}TransportUse

mobSpawning (parent)
├── hostileMobSpawning
├── passiveMobSpawning
└── neutralMobSpawning

Gotcha Example

A faction sets memberInteract=false but memberDoorUse=true:

get("memberDoorUse")
  → parent = getParentFlag("memberDoorUse") = "memberInteract"
  → getRaw("memberInteract") = false
  → return false  (parent disabled — door access DENIED despite doorUse=true)

The GUI handles this correctly by disabling child toggles when the parent is off.

Interaction Check Flow

Source: ProtectionChecker.canInteractChunk()

When a player performs any action in claimed territory, this is the exact check order:

Step 1: Admin Bypass

Is player admin? (has "hyperfactions.admin.use")
  → YES: Is admin bypass toggle ON? (/f admin bypass)
    → YES: ALLOWED_BYPASS (skip all checks)
    → NO: Fall through to zone/claim checks (admins with bypass OFF get NO special treatment)
  → NO: Continue to Step 2

Key: Admins do NOT use standard bypass permissions. Only the explicit toggle matters.

Step 2: Standard Bypass Permissions

Non-admin players only. Checked by interaction type:

InteractionType Bypass Permission
BUILD hyperfactions.bypass.build
INTERACT, DOOR, BENCH, PROCESSING, SEAT, LIGHT, MOUNT, TELEPORTER, PORTAL, CRATE_PICKUP, CRATE_PLACE, NPC_TAME, NPC_INTERACT, ITEM_DROP, ITEM_PICKUP hyperfactions.bypass.interact
CONTAINER hyperfactions.bypass.container
DAMAGE, PVE_DAMAGE hyperfactions.bypass.damage
USE hyperfactions.bypass.use

Wildcard hyperfactions.bypass.* bypasses all types. If granted → ALLOWED_BYPASS.

Step 3: Zone Check

If the chunk is in a zone, zone flags take precedence:

  • Zone flag disabled → DENIED_SAFEZONE or DENIED_WARZONE
  • WarZone with flag allowed → ALLOWED_WARZONE (returns immediately, skips claim checks)
  • SafeZone with flag allowed → falls through to claim check below

Step 4: Claim Ownership

claimManager.getClaimOwner(world, chunkX, chunkZ)
  → NULL: ALLOWED_WILDERNESS or ALLOWED_SAFEZONE (if in SafeZone)
  → UUID: Continue to faction permission checks

Step 5: Same Faction

Player is in the owning faction. Role determines which flag level is checked:

boolean isOfficerOrLeader = factionMember.role().getLevel() >= FactionRole.OFFICER.getLevel();
  • Officer/Leader (role level >= 2): Checked against officer{Suffix} flags
  • Member (role level 1): Checked against member{Suffix} flags

If the appropriate flag denies the action → DENIED_NO_PERMISSION. If allowed → ALLOWED_OWN_CLAIM.

Step 6: Ally Check

Player has a faction and that faction has ALLY relation with claim owner:

  • Check ally{Suffix} flags
  • If allowed → ALLOWED_ALLY_CLAIM
  • If denied → DENIED_NO_PERMISSION

Step 7: Outsider Check

Player is not in the owning faction and not allied:

  • Check outsider{Suffix} flags
  • If allowed → ALLOWED

Step 8: Default Deny

If outsider flags deny the action:

  • Player is ENEMY → DENIED_ENEMY_CLAIM
  • Otherwise → DENIED_NEUTRAL_CLAIM (includes factionless players)

InteractionType to Flag Mapping

Source: ProtectionChecker.checkPermission()

When the checker evaluates a faction permission, it maps the action type to specific flag name(s):

InteractionType Flag(s) Checked Resolution
BUILD {level}Break OR {level}Place Either flag grants permission
INTERACT {level}Interact Parent flag — gates 6 children
USE {level}Interact Same as INTERACT
DOOR {level}DoorUse Child of Interact
CONTAINER {level}ContainerUse Child of Interact
BENCH {level}BenchUse Child of Interact
PROCESSING {level}ProcessingUse Child of Interact
SEAT {level}SeatUse Child of Interact
LIGHT {level}Interact Shares general interact permission
TELEPORTER {level}TransportUse Child of Interact
PORTAL {level}TransportUse Child of Interact
CRATE_PICKUP {level}CrateUse Capture crate pickup
CRATE_PLACE {level}CrateUse Capture crate release
NPC_TAME {level}NpcTame F-key NPC taming
NPC_INTERACT {level}NpcInteract NPC shops/dialogue (note: no matching FactionPermissions flag)
MOUNT {level}SeatUse Shares seat permission
PVE_DAMAGE {level}PveDamage Mob/entity damage by player
DAMAGE Hardcoded: non-outsiders always allowed Outsiders always denied
ITEM_DROP {level}Interact Shares general interact permission
ITEM_PICKUP {level}Interact Shares general interact permission

BUILD note: checkPermission() uses perms.get(level + "Break") || perms.get(level + "Place"). This means having EITHER break OR place permission allows the BUILD interaction type. This is because the ECS systems may route both break and place events through the same BUILD type.

DAMAGE note: Entity damage (non-player, via DAMAGE type) uses !"outsider".equals(level) — members, officers, and allies can always damage entities in claims. Outsiders cannot. There is no configurable flag for this. For explicit PvE damage control, the PVE_DAMAGE type checks {level}PveDamage which IS a configurable per-level flag.

NPC_INTERACT note: The checkPermission() method checks {level}NpcInteract but there is no NpcInteract suffix in FactionPermissions.LEVEL_SUFFIXES. This means perms.get(level + "NpcInteract") returns false (unknown flag defaults to false). NPC interaction in claims is effectively always denied unless bypassed.

Faction Roles

Source: FactionRole.java

Role Level Permission Check Level
LEADER 3 officer flags (same as officer)
OFFICER 2 officer flags
MEMBER 1 member flags

Leaders and officers both use officer{Suffix} flags. There are no separate leader-specific permission flags.

PvP in Claimed Territory

Source: ProtectionChecker.canDamagePlayerChunk()

Check Order

  1. Spawn protection: Defender has spawn protection → DENIED_SPAWN_PROTECTED
  2. Break attacker spawn protection: If config spawnProtection.breakOnAttack=true and attacker has spawn protection, it's cleared
  3. Zone PvP: If in a zone, zone flags override (including friendly fire hierarchy)
  4. Territory PvP flag: Claim owner's pvpEnabled faction permission checked. If false → DENIED_TERRITORY_NO_PVP
  5. Same faction: ConfigManager.isFactionDamage(world) — per-world override support. If false → DENIED_SAME_FACTION
  6. Ally check: ConfigManager.isAllyDamage(world) — per-world override support. If false → DENIED_ALLY
  7. Outsider PvP damage config: In claimed territory, outsider damage is checked against 3 config flags: factionlessDamageAllowed, enemyDamageAllowed, neutralDamageAllowed. If the attacker's relation type is denied → DENIED_TERRITORY_NO_PVP
  8. Default allow: ALLOWED

PvP Configuration Layers

Setting Type Scope Default Where Set
pvpEnabled Faction permission Per-faction, per-territory true /f settings GUI
factionDamage Server config Global, per-world override false config/factions.json
allyDamage Server config Global, per-world override false config/factions.json

Note: pvpEnabled is a faction permission flag and can be toggled by faction officers. factionDamage and allyDamage are server configs in config/factions.json that factions cannot control. Per-world overrides for faction/ally damage use ConfigManager.isFactionDamage(world) and ConfigManager.isAllyDamage(world).

Item Pickup in Claims

Source: ProtectionChecker.canPickupItem()

Pickup checks are faction-relationship-based, not permission-flag-based:

  1. Admin bypass toggle → allow
  2. hyperfactions.bypass.pickup or hyperfactions.bypass.* → allow
  3. Zone flag (ITEM_PICKUP for auto, ITEM_PICKUP_MANUAL for F-key) → if zone denies, blocked
  4. Wilderness (no claim owner) → allow
  5. Same faction → always allow
  6. Ally faction → always allow
  7. Outsider/enemy/neutral → controlled by claims.outsiderPickupAllowed in config/factions.json

The outsiderPickupAllowed setting (default: true, migrated servers: false) controls whether outsiders can pick up items in claimed territory. Previously this was hardcoded to deny.

Mob Spawning in Claims

Source: ProtectionChecker.shouldBlockSpawn() (see protection-systems.md for line reference)

  1. Zone flags checked first (ZoneFlags.MOB_SPAWNING / ZoneFlags.NPC_SPAWNING)
  2. Claim owner's faction permissions checked: mobSpawning parent + sub-type children
  3. Parent-child resolution applies: mobSpawning=false blocks all spawning types
  4. Wilderness → no restriction

Server-Wide Configuration

Note: As of v0.10.0 (Migration V5 -> V6), config.json has been split into config/factions.json (faction gameplay settings) and config/server.json (server behavior settings). The old config.json is deleted after migration. CoreConfig is deprecated in favor of FactionsConfig and ServerConfig.

FactionsConfig (config/factions.json)

Source: FactionsConfig.java

Claim Settings

Key Type Default Purpose
claims.maxClaims int 100 Maximum claims per faction
claims.onlyAdjacent bool false Claims must be adjacent to existing claims
claims.preventDisconnect bool false Prevent claim gaps in faction territory
claims.decayEnabled bool true Auto-remove inactive faction claims
claims.decayDaysInactive int 30 Days of inactivity before decay
claims.worldWhitelist List [] Worlds where claiming is allowed (if non-empty, overrides blacklist)
claims.worldBlacklist List [] Worlds where claiming is forbidden

Claim Protection Settings (9 flags)

These flags control server-wide protection behavior in claimed territory. They are NOT faction permissions — factions cannot change them.

Key Type Default Migrated Default Purpose
claims.outsiderPickupAllowed bool true false Whether outsiders can pick up items in claims
claims.outsiderDropAllowed bool true — Whether outsiders can drop items in claims
claims.factionlessExplosionsAllowed bool false — Allow explosion damage from factionless sources in claims
claims.enemyExplosionsAllowed bool false — Allow explosion damage from enemy faction sources in claims
claims.neutralExplosionsAllowed bool false — Allow explosion damage from neutral faction sources in claims
claims.fireSpreadAllowed bool true false Allow fire to spread in claimed territory
claims.factionlessDamageAllowed bool true — Allow entity damage from factionless players in claims
claims.enemyDamageAllowed bool true — Allow entity damage from enemy faction players in claims
claims.neutralDamageAllowed bool true — Allow entity damage from neutral faction players in claims

Migrated Default column: when a server is migrated from V5 to V6, outsiderPickupAllowed and fireSpreadAllowed are set to false to preserve the previous hardcoded behavior. Fresh installs use the regular defaults.

Explosion limitation: Explosion hooks do not provide a player context, so shouldBlockExplosion() performs a combined 3-way check: if ANY of the three explosion flags (factionlessExplosionsAllowed, enemyExplosionsAllowed, neutralExplosionsAllowed) is true, explosions are allowed. All three must be false to block explosions. This replaces the old single allowExplosionsInClaims flag.

Combat Settings

Key Type Default Purpose
combat.factionDamage bool false Same-faction PvP allowed
combat.allyDamage bool false Ally faction PvP allowed
combat.tagDurationSeconds int 15 Combat tag duration
combat.taggedLogoutPenalty bool true Penalize tagged player logout
combat.logoutPowerLoss double 1.0 Power lost on tagged logout
combat.neutralAttackPenalty double 0.0 Power lost attacking neutrals

Spawn Protection

Key Type Default Purpose
combat.spawnProtection.enabled bool true Enable spawn protection
combat.spawnProtection.durationSeconds int 5 Duration after respawn
combat.spawnProtection.breakOnAttack bool true Remove if player attacks
combat.spawnProtection.breakOnMove bool true Remove if player moves

Power Settings

Key Type Default Purpose
power.powerLossOnMobDeath bool true Power lost to mob kills
power.powerLossOnEnvironmentalDeath bool true Power lost to fall/drown

FactionPermissionsConfig (config/faction-permissions.json)

Source: FactionPermissionsConfig.java, loaded at ConfigManager.java

Separate file from config/factions.json. Three sections:

defaults Section

Server-wide defaults for new factions. Also the value used when a flag is locked.

{
  "defaults": {
    "outsider": {
      "break": false, "place": false, "interact": false,
      "doorUse": false, "containerUse": false, "benchUse": false,
      "processingUse": false, "seatUse": false, "transportUse": false,
      "crateUse": false, "npcTame": false, "pveDamage": false
    },
    "ally": {
      "break": false, "place": false, "interact": true,
      "doorUse": true, "containerUse": false, "benchUse": false,
      "processingUse": false, "seatUse": true, "transportUse": true,
      "crateUse": false, "npcTame": false, "pveDamage": true
    },
    "member": {
      "break": true, "place": true, "interact": true,
      "doorUse": true, "containerUse": true, "benchUse": true,
      "processingUse": true, "seatUse": true, "transportUse": true,
      "crateUse": true, "npcTame": true, "pveDamage": true
    },
    "officer": {
      "break": true, "place": true, "interact": true,
      "doorUse": true, "containerUse": true, "benchUse": true,
      "processingUse": true, "seatUse": true, "transportUse": true,
      "crateUse": true, "npcTame": true, "pveDamage": true
    },
    "mobSpawning": {
      "enabled": true, "hostile": true,
      "passive": true, "neutral": true
    },
    "pvpEnabled": true,
    "officersCanEdit": false,
    "treasury": {
      "deposit": true,
      "withdraw": false,
      "transfer": false
    }
  }
}

Supports both nested JSON format (above) and legacy flat format ("outsiderBreak": false).

treasury Section

Treasury permission flags are also configured in faction-permissions.json under both defaults and locks. The treasury key controls deposit, withdraw, and transfer permissions:

Key Default Purpose
treasury.deposit true Whether members can deposit into faction treasury
treasury.withdraw false Whether officers can withdraw from treasury
treasury.transfer false Whether officers can transfer to other factions

These flags are exposed in-game via the TreasurySettingsPage GUI.

locks Section

Which flags are server-locked (factions cannot change them). Structure mirrors defaults. Any flag set to true here forces the default value.

{
  "locks": {
    "outsider": { "break": false },
    "pvpEnabled": false
  }
}

All locks default to false (unlocked).

Lock Resolution

Source: FactionPermissionsConfig.getEffectiveFactionPermissions()

public FactionPermissions getEffectiveFactionPermissions(FactionPermissions factionPerms) {
    Map<String, Boolean> effective = new HashMap<>(factionPerms.toMap());
    for (Map.Entry<String, Boolean> entry : locks.entrySet()) {
        if (entry.getValue()) {
            // Flag is locked — override with server default
            effective.put(entry.getKey(), defaults.getOrDefault(entry.getKey(), false));
        }
    }
    return new FactionPermissions(effective);
}

When locked: The faction's stored value is ignored, server default is used. When unlocked: Faction's stored value is used.

Managing Faction Permissions

Source: FactionSettingsPage.java

GUI Access

  • Command: /f settings
  • Required role: Officer or higher (role level >= 2)
  • Officers see: All 54 flags (48 per-level + 4 mob spawning + pvpEnabled + officersCanEdit as disabled)
  • Leaders see: All 54 flags including officersCanEdit as editable
  • Treasury flags: 3 treasury flags (treasuryDeposit, treasuryWithdraw, treasuryTransfer) are exposed in the TreasurySettingsPage GUI (accessible via /f treasury settings or /f settings treasury tab)

No CLI Command

There is no /f perm or similar CLI command for editing faction permissions. The settings GUI is the only in-game interface.

Server Lock Behavior

When a flag is locked via faction-permissions.json:

  • The toggle appears disabled in the GUI
  • Attempting to toggle shows: "This setting is locked by the server."
  • The effective value is always the server default

Mixin-Dependent Claim Protections

Some protections require a mixin system to be installed. Without one, certain protections have no enforcement at all.

Source: HyperProtectIntegration.java, OrbisMixinsIntegration.java

Protection No Mixin OrbisGuard-Mixins HyperProtect-Mixin Checks Claims?
Block break ECS system Mixin hook Mixin hook (slot 0) Yes — faction perms
Block place ECS system Mixin hook Mixin hook (slot 18) Yes — faction perms
Block use (doors, etc.) ECS system Mixin hook Mixin hook (slot 20) Yes — faction perms
Container access ECS system — Mixin hook (slot 7) Yes — faction perms
Container open — — Mixin hook (slot 17) Yes — faction perms
Item pickup (auto) ECS system Mixin hook Mixin hook (slot 4) Yes — faction relationship
Item pickup (F-key) ECS system Mixin hook Mixin hook (slot 4) Yes — faction relationship
Crop harvest Codec replacement Mixin hook Mixin hook Yes — via canInteract()
Hammer use None Mixin hook Mixin hook (slot 19) Yes — faction perms
Builder tools None — Mixin hook (slot 3) Yes — faction perms
Explosions None Mixin hook Mixin hook (slot 1) Yes — 3-way config
Fire spread None — Mixin hook (slot 2) Yes — configurable
Teleporter use None — Mixin hook (slot 9) Yes — faction perms
Portal use None — Mixin hook (slot 10) Yes — faction perms
Entity damage ECS system — Mixin hook (slot 16) Yes — faction perms
Seat protection ECS system Mixin hook Mixin hook (slot 21) Yes — faction perms
Mob spawn suppression None Mixin hook Mixin hook (slot 8) Yes — faction perms
Respawn override — — Mixin hook (slot 22) Yes
Item drop Zone flag only Zone flag only Zone flag only Partial — zone flag + outsiderDropAllowed config
Keep inventory None Mixin hook Mixin hook (slot 5) No — zone-only
Durability loss None Mixin hook Mixin hook (slot 6) No — zone-only
Command blocking — Mixin hook Mixin hook (slot 11) No — combat tag only

"None" means no enforcement exists for that feature without a mixin. The protection simply doesn't happen.

Known Limitations & Gaps

Zone-Only Protections (No Claim Equivalent)

These protections work in admin zones but have no faction permission or config equivalent for claims. In claimed territory outside a zone, these features are unprotected.

Feature Zone Support Claim Support Source
Keep inventory on death KEEP_INVENTORY zone flag None — returns false shouldKeepInventory()
Durability prevention INVINCIBLE_ITEMS zone flag None — returns false shouldPreventDurability()
Item drop prevention ITEM_DROP zone flag Partial — outsiderDropAllowed config ItemDropProtectionSystem
Fall damage prevention FALL_DAMAGE zone flag None — zone-only FallDamageProtection
Environmental damage prevention ENVIRONMENTAL_DAMAGE zone flag None — zone-only EnvironmentalDamageProtection
Projectile damage prevention PROJECTILE_DAMAGE zone flag None — zone-only ProjectileDamageProtection
Mob damage prevention MOB_DAMAGE zone flag None — zone-only MobDamageProtection

Asymmetries & Known Limitations

Issue Details Source
Pickup vs Drop Item PICKUP checks faction relationships + outsiderPickupAllowed config. Item DROP checks zone flags + outsiderDropAllowed config. canPickupItem() vs ItemDropProtectionSystem
Explosion source attribution (KNOWN LIMITATION) Explosion hooks do not provide a player UUID, so shouldBlockExplosion() cannot determine the source faction. Instead it performs a combined 3-way check: if ANY of factionlessExplosionsAllowed, enemyExplosionsAllowed, or neutralExplosionsAllowed is true, explosions are allowed. All three must be false to block. This is a platform limitation, not a bug. shouldBlockExplosion()
DAMAGE type for outsiders The DAMAGE InteractionType (non-player entity damage) is hardcoded: non-outsiders always allowed, outsiders always denied. For configurable PvE damage, the PVE_DAMAGE type checks {level}PveDamage flags. The 3 config flags (factionlessDamageAllowed, enemyDamageAllowed, neutralDamageAllowed) control outsider PvP damage in claims, not entity damage. checkPermission(), canDamagePlayerChunk()

Config vs Faction Permission Confusion

Some claim protections are per-faction (toggleable by officers), others are server-wide (only admin can change), and some are hardcoded:

Protection Control Type Who Controls
Block interactions Faction permission (48 per-level flags) Faction officers via GUI
PvP in territory Faction permission (pvpEnabled) Faction officers via GUI
Mob spawning Faction permission (4 flags) Faction officers via GUI
Same-faction PvP Server config (combat.factionDamage) Server admin via config/factions.json
Ally PvP Server config (combat.allyDamage) Server admin via config/factions.json
Explosions Server config (3-way: claims.factionlessExplosionsAllowed, claims.enemyExplosionsAllowed, claims.neutralExplosionsAllowed) Server admin via config/factions.json
Fire spread Server config (claims.fireSpreadAllowed) Server admin via config/factions.json
Outsider PvP damage Server config (3-way: claims.factionlessDamageAllowed, claims.enemyDamageAllowed, claims.neutralDamageAllowed) Server admin via config/factions.json
Item pickup Server config (claims.outsiderPickupAllowed) Server admin via config/factions.json
Item drop Server config (claims.outsiderDropAllowed) + zone flags Server admin via config/factions.json / zone admins
Keep inventory Zone-only (no claim protection) Zone admins only
Durability Zone-only (no claim protection) Zone admins only

Accessor Methods

Only two convenience accessor methods remain on FactionPermissions:

  • pvpEnabled() — calls get(PVP_ENABLED) (uses parent-child logic)
  • officersCanEdit() — calls get(OFFICERS_CAN_EDIT) (uses parent-child logic)

All per-level accessor methods (e.g., outsiderBreak(), memberInteract()) have been removed. The main checkPermission() in ProtectionChecker uses get() for all flag lookups, which correctly applies parent-child resolution.

Code Links

Class Path
FactionPermissions data/FactionPermissions.java
FactionRole data/FactionRole.java
ProtectionChecker protection/ProtectionChecker.java
FactionPermissionsConfig config/modules/FactionPermissionsConfig.java
CoreConfig (deprecated) config/CoreConfig.java
FactionsConfig config/modules/FactionsConfig.java
ServerConfig config/modules/ServerConfig.java
ConfigManager config/ConfigManager.java
FactionSettingsPage gui/faction/page/FactionSettingsPage.java