GriefPrevention / GriefPrevention

The official self-service anti-griefing Bukkit plugin for Minecraft servers since 2011.
https://griefprevention.com
GNU General Public License v3.0
295 stars 239 forks source link

Wither Spawn still breaks #1732

Open loothillRw opened 2 years ago

loothillRw commented 2 years ago

Observed Behavior

When you spawn a Wither inside a claim still breaks blocks (even with ClaimExplosions disabled), this only breaks if you are below y60

Expected Behavior

Wither spawn shouldn't break inside claims

Reproduction steps

1) Claim a random zone 2) Go below y60 3) Spawn a Wither 4) Wither will destroy on Spawn

Stack trace or error log

No response

Server version

Purpur version git-Purpur-1445 (MC: 1.18.1) (Implementing API version 1.18.1-R0.1-SNAPSHOT) (Git: 8183257 on HEAD)

GriefPrevention version

GriefPrevention version 16.18-RC1-1-g06cc566

Configuration

# Default values are perfect for most servers.  If you want to customize and have a question, look for the answer here first: http://dev.bukkit.org/bukkit-plugins/grief-prevention/pages/setup-and-configuration/
GriefPrevention:
  SeaLevelOverrides:
    World1: -1
    World1_nether: -1
    World1_the_end: -1
  Claims:
    Mode:
      World1_the_end: Disabled
      World1: Survival
      World1_nether: Survival
    PreventGlobalMonsterEggs: true
    PreventTheft: true
    ProtectCreatures: true
    PreventButtonsSwitches: true
    LockWoodenDoors: true
    LockTrapDoors: true
    LockFenceGates: true
    EnderPearlsRequireAccessTrust: true
    RaidTriggersRequireBuildTrust: true
    ProtectHorses: true
    ProtectDonkeys: true
    ProtectLlamas: true
    InitialBlocks: 300
    Claim Blocks Accrued Per Hour:
      Default: 70
    Max Accrued Claim Blocks:
      Default: 224000
    Accrued Idle Threshold: 0
    AccruedIdlePercent: 0
    AbandonReturnRatio: 1.0
    AutomaticNewPlayerClaimsRadius: 4
    AutomaticNewPlayerClaimsRadiusMinimum: 0
    ExtendIntoGroundDistance: 5
    MinimumWidth: 5
    MinimumArea: 100
    MaximumDepth: -64
    InvestigationTool: STICK
    ModificationTool: GOLDEN_SHOVEL
    Expiration:
      ChestClaimDays: 7
      UnusedClaimDays: 14
      AllClaims:
        DaysInactive: 90
        ExceptWhenOwnerHasTotalClaimBlocks: 10000
        ExceptWhenOwnerHasBonusClaimBlocks: 5000
      AutomaticNatureRestoration:
        SurvivalWorlds: false
    AllowTrappedInAdminClaims: false
    MaximumNumberOfClaimsPerPlayer: 0
    CreationRequiresWorldGuardBuildPermission: true
    VillagerTradingRequiresPermission: true
    CommandsRequiringAccessTrust: /sethome
    DeliverManuals: false
    ManualDeliveryDelaySeconds: 30
    RavagersBreakBlocks: true
    FireSpreadsInClaims: false
    FireDamagesInClaims: false
    LecternReadingRequiresAccessTrust: true
  Spam:
    Enabled: false
    LoginCooldownSeconds: 60
    LoginLogoutNotificationsPerMinute: 5
    ChatSlashCommands: /me;/global;/local
    WhisperSlashCommands: /tell;/pm;/r;/whisper;/msg
    WarningMessage: Please reduce your noise level.  Spammers will be banned.
    BanOffenders: true
    BanMessage: Banned for spam.
    AllowedIpAddresses: 1.2.3.4; 5.6.7.8
    DeathMessageCooldownSeconds: 120
    Logout Message Delay In Seconds: 0
  PvP:
    RulesEnabledInWorld:
      World1: true
      World1_nether: true
      World1_the_end: true
    ProtectFreshSpawns: false
    PunishLogout: true
    CombatTimeoutSeconds: 15
    AllowCombatItemDrop: false
    BlockedSlashCommands: /home;/vanish;/spawn;/tpa;/back;/tpaccept;/tpadeny;/homes;/g;/guild;/guilds;/claim;/g
      home;/guilds home
    ProtectPlayersInLandClaims:
      PlayerOwnedClaims: true
      AdministrativeClaims: true
      AdministrativeSubdivisions: true
    AllowLavaDumpingNearOtherPlayers:
      PvPWorlds: true
      NonPvPWorlds: false
    AllowFlintAndSteelNearOtherPlayers:
      PvPWorlds: true
      NonPvPWorlds: false
    ProtectPetsOutsideLandClaims: false
  Economy:
    ClaimBlocksMaxBonus: 0
    ClaimBlocksPurchaseCost: 0.0
    ClaimBlocksSellValue: 0.0
  ProtectItemsDroppedOnDeath:
    PvPWorlds: false
    NonPvPWorlds: true
  BlockLandClaimExplosions: true
  BlockSurfaceCreeperExplosions: true
  BlockSurfaceOtherExplosions: true
  LimitSkyTrees: true
  LimitTreeGrowth: false
  PistonMovement: EVERYWHERE
  PistonExplosionSound: true
  FireSpreads: false
  FireDestroys: false
  AdminsGetWhispers: true
  AdminsGetSignNotifications: true
  SmartBan: true
  Mute New Players Using Banned Words: true
  MaxPlayersPerIpAddress: 3
  SilenceBans: true
  Siege:
    Worlds: []
    BreakableBlocks:
    - GRASS_BLOCK
    - DIRT
    - COBBLESTONE
    - OAK_PLANKS
    - SPRUCE_PLANKS
    - BIRCH_PLANKS
    - JUNGLE_PLANKS
    - ACACIA_PLANKS
    - DARK_OAK_PLANKS
    - SAND
    - GRAVEL
    - GLASS
    - GRASS
    - FERN
    - DEAD_BUSH
    - WHITE_WOOL
    - ORANGE_WOOL
    - MAGENTA_WOOL
    - LIGHT_BLUE_WOOL
    - YELLOW_WOOL
    - LIME_WOOL
    - PINK_WOOL
    - GRAY_WOOL
    - LIGHT_GRAY_WOOL
    - CYAN_WOOL
    - PURPLE_WOOL
    - BLUE_WOOL
    - BROWN_WOOL
    - GREEN_WOOL
    - RED_WOOL
    - BLACK_WOOL
    - SNOW
    - GLASS_PANE
    DoorsOpenDelayInSeconds: 300
    CooldownEndInMinutes: 60
  EndermenMoveBlocks: false
  SilverfishBreakBlocks: false
  CreaturesTrampleCrops: false
  RabbitsEatCrops: true
  HardModeZombiesBreakDoors: false
  Database:
    URL: ''
    UserName: ''
    Password: ''
  UseBanCommand: false
  BanCommandPattern: ban %name% %reason%
  Advanced:
    fixNegativeClaimblockAmounts: true
    ClaimExpirationCheckRate: 60
    OfflinePlayer_cache_days: 90
  Abridged Logs:
    Days To Keep: 7
    Included Entry Types:
      Social Activity: true
      Suspicious Activity: true
      Administrative Activity: false
      Debug: false
      Muted Chat Messages: false

Plugin list

ajLeaderboards, AntiAFKPlus, AuctionHouse, BetterReports*, BetterRTP, BlockedCommands*, BungeeGuard, BuycraftX, ChatColor2, ChatSentinel, Chunky, Citizens, CMILib, ColoredAnvils, CommandPanels, CoreProtect, CrazyCrates, DiscordSRV, EconomyShopGUI, ElevatorSigns, Essentials, EssentialsChat, EssentialsSpawn, FastAsyncWorldEdit (WorldEdit), GriefPrevention, GSit, Guilds, GuiRedeemMCMMO*, Harbor, HoloExtension, HolographicDisplays, ImageMaps, InteractiveChat, InventoryRollbackPlus, Jobs, LiteBans, LuckPerms, MarriageMaster, mcMMO, Multiverse-Core, MVdWPlaceholderAPI*, PlaceholderAPI, PlayerKits, PlayerVaults, PlugManX (PlugMan), ProtocolLib, ProtocolSupport, RealScoreboard, SIR*, SkinsRestorer, spark, StackMob, StaffPlus, TAB, TerraformGenerator, Vault, VeryCubeCMDs, ViaBackwards, ViaVersion, Votifier, VotingPlugin, WorldBorder, WorldEdit, WorldGuard, XpBooster*

Running without GriefPrevention

Running with only GriefPrevention

Running on a fresh, clean server installation

Using unmodified client

We appreciate you taking the time to fill out a bug report!

YanisBft commented 2 years ago

I confirm this bug, withers break in claims only below layer 0 on my server

PirataGu commented 2 years ago

Even modifiyng the MaximumDepth it does not work with negative y !!!

Just noticed that if I modify the claimdata to a negative Y coordinate IT WORKS but the plugin doesn't know how to put the - (minus) behind the number LOOOOL

RoboMWM commented 2 years ago

@YanisBft wait so they don't break blocks in claims at coordinate y > 0?

YanisBft commented 2 years ago

@RoboMWM with claim explosions disabled, they don't break at y > 0 (which is intended I think?). Though they do break blocks below 0, hence the report

RoboMWM commented 2 years ago

Ok, because that's not what the reporter said, he basically said it causes explosions below sea level. So I'd ask for you to make a new bug report (if one doesn't already exist)

RoboMWM commented 2 years ago

@loothillRw Can you verify where the explosions occur? And if it happens with a more up-to-date purpur server version? Or let us know if this no longer occurs.