Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invisible Item Frames WorldGuard #86

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

garrettsummerfi3ld
Copy link
Member

Information

Add WorldGuard functionality for Invisible Item Frames. Such as disabling toggling them on/off in a specified region.

Demonstration

image
This is the test region for the flags.
image
There is a specified flag that can be used to deny changing Item Frames in a region.
image
If you do not have the standard WorldGuard bypass permission node, you cannot change the item frame and a message is thrown to the user.

Added flags for WorldGuard to allow/disallow users to change invisibility of item frames. This is helpful for protected regions.
Added a check if a player is able to bypass the protection.
@garrettsummerfi3ld garrettsummerfi3ld self-assigned this Dec 4, 2023
@garrettsummerfi3ld garrettsummerfi3ld added the enhancement New feature or request label Dec 4, 2023
Copy link
Member

@josefbaltz josefbaltz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please review comments made, overall well written code just an optimization and a bit more documentation and should be good.

build.gradle Show resolved Hide resolved
src/main/java/tv/galaxe/smp/cmd/InvisibleItemFrame.java Outdated Show resolved Hide resolved
garrettsummerfi3ld added a commit that referenced this pull request Dec 11, 2023
Previously the plugin would throw a ClassCastException when trying to cast a WorldGuard region to a World. This was due to checking if the player had a bypass check and directly casting a normal player to the location, rather than WorldGuard's LocalPlayer. This wasn't documented well from WorldGuard's side, but it is now fixed.

Fixed issue where on startup the plugin would throw an error due to the entirety of the plugin had WorldGuard embedded into it, rather than just the WorldGuard dependency. This is now fixed.

Removed a maven repository due to it conflicting with other dependencies being used.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants