Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
MythicMobs MythicMobs
  • Project overview
    • Project overview
    • Details
    • Activity
  • Issues 145
    • Issues 145
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Operations
    • Operations
    • Incidents
  • Analytics
    • Analytics
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Members
    • Members
  • Activity
  • Create a new issue
  • Issue Boards
Collapse sidebar
  • MythicCraft
  • MythicMobsMythicMobs
  • Issues
  • #1503

Closed
Open
Created Feb 23, 2024 by Mykindos@Mykindos

Blockmask onlyair attribute includes all non-occluding blocks

Summary

Even with onlyair set to true in blockmask, blocks such as glass and stained glass will also be masked. The documentation specifies it is only air, but that just isn't the case.

Steps to reproduce

Call blockmask with onlyair set to true, create a sphere around current location and have the floor made of stained glass

Current behavior

The non-air blocks are masked

Intended correct behavior

The non-air blocks are not masked

Proposed fixes

Change logic in BlockMaskEffect.java to match other classes that use MaskEffect.ONLY_AIR

BukkitAdapter.adapt(newloc).getBlock().getType() == Material.AIR or block.getType().isAir()

instead of

!newloc.getBlock().getType().isOccluding()

Edited Feb 23, 2024 by Mykindos
Assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking