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

bskyblock.admin.tp permission when given as a standalone doesn't allow /bsbadmin tp {playername} #2571

Open
MacTh3Mac opened this issue Dec 14, 2024 · 3 comments
Assignees
Labels
Type: Documentation Type: Not a bug Works as intended / not a BentoBox bug.

Comments

@MacTh3Mac
Copy link

Expected behavior

When staff member with permission bskyblock.admin.tp true runs the command /bsbadmin tp {player name} they should teleport to the players island

Observed/Actual behavior

They get a you dont have permission dialogue

error

Screenshot 2024-12-14 at 19 13 53

Steps/models to reproduce

have attempted also to apply the node directly to the player in a global and a server specific context.

Note: ops unaffected, also higher ranked staff with bskyblock.admin.* are affected too

BentoBox version

14.12 19:32:42 [Server] INFO Running PAPER 1.21.3.
14.12 19:32:42 [Server] INFO (1.21.3-2358-16ce24a (MC: 1.21.3))
14.12 19:32:42 [Server] INFO BentoBox version: 3.0.1
14.12 19:32:42 [Server] INFO Database: JSON
14.12 19:32:42 [Server] INFO Loaded Game Worlds:
14.12 19:32:42 [Server] INFO bskyblock_world (BSkyBlock): Overworld, Nether, The End
14.12 19:32:42 [Server] INFO Loaded Addons:
14.12 19:32:42 [Server] INFO Biomes 2.2.0 (ENABLED)
14.12 19:32:42 [Server] INFO Border 4.4.0 (ENABLED)
14.12 19:32:42 [Server] INFO BSkyBlock 1.19.0 (ENABLED)
14.12 19:32:42 [Server] INFO Challenges 1.4.0 (ENABLED)
14.12 19:32:42 [Server] INFO Chat 1.3.0 (ENABLED)
14.12 19:32:42 [Server] INFO ControlPanel 1.14.0 (ENABLED)
14.12 19:32:42 [Server] INFO ExtraMobs 1.14.0 (ENABLED)
14.12 19:32:42 [Server] INFO Level 2.17.0 (ENABLED)
14.12 19:32:42 [Server] INFO Likes 2.5.0 (ENABLED)
14.12 19:32:42 [Server] INFO Limits 1.26.0 (ENABLED)
14.12 19:32:42 [Server] INFO MagicCobblestoneGenerator 2.6.0 (ENABLED)
14.12 19:32:42 [Server] INFO Warps 1.16.0 (ENABLED)

Plugin list

luckperms 5.4.141

Other

No response

@tastybento tastybento self-assigned this Dec 14, 2024
@tastybento tastybento transferred this issue from BentoBoxWorld/BSkyBlock Dec 14, 2024
@tastybento tastybento added the Status: Under investigation Investigating the interest and the feasability of the issue. label Dec 14, 2024
@tastybento
Copy link
Member

This is a problem with perms that are hierarchical - bskyblock.admin.tp requires the bskyblock.admin perm, which will give access to all admin commands, and then you need to disable admin perms the commands you don't want to give access It's a pain, sorry, but I don't think there's a way to avoid it. You can get a list of all the perms in the console by running bbox perms

@MacTh3Mac
Copy link
Author

We have been using bbox game modes since way back when bskyblock was launched, and this seems slightly a backwards step. What version / build did you change the permissions to require giving an elevated node then denying access to all specific nodes?

This hasn't ever been an issue for many years and doesn't seem logical?

@tastybento
Copy link
Member

Sorry, I gave bad advice. You just need to give the player the missing perm, in this case bskyblock.admin. Players need the parent perm as well and the child one now. This is safe, they won't be able to do anything else unless you give them specific perms. Here I am with bskyblock.admin.tp and bskyblock.admin perms:

Screenshot 2024-12-15 at 8 49 54 AM
> lp user tastybentotoo permission info
[08:51:37 INFO]: [LP] tastybentotoo's Permissions:  (page 1 of 1 - 2 entries)
[08:51:37 INFO]: > bskyblock.admin 
[08:51:37 INFO]: > bskyblock.admin.tp 

The change was made in August 2022 here, which was to address this issue, which was that with the default island command, as it actually did something (made an island/teleported you) everyone had that perm, but then also by default had all sub-perms and subcommands. This change requires players to have the full perm chain, so the parent(s) and the child.

Anyway, in summary, just give your staff member bskyblock.admin perm and it should work. Sorry about the confusion!

@tastybento tastybento added Type: Documentation Type: Not a bug Works as intended / not a BentoBox bug. and removed Status: Under investigation Investigating the interest and the feasability of the issue. labels Dec 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Documentation Type: Not a bug Works as intended / not a BentoBox bug.
Projects
None yet
Development

No branches or pull requests

2 participants