PermissionsEx

PermissionsEx

14M Downloads

[PEX 1.23.4][MC 1.12.1] 'group set' command puts user into the wrong group.

Stormbow opened this issue · 6 comments

commented

froger599 (Minecraft username: NishiPlays) on the Curse website reported that the command /pex user <name> group set <rank> is putting people into the wrong groups.

This server is running CraftBukkit version git-Spigot-edd0396-2a927e8 (MC: 1.12.1) (Implementing API version 1.12.1-R0.1-SNAPSHOT)

Permissions.yml file: https://pastebin.com/j0ehvr2j

/pex user NishiPlays group set Apollo
     Puts NishiPlays into the group named "Zeus".
/pex user NishiPlays group set Aphrodite
     Puts NishiPlays into the group named "Acolytes".

No errors appear in the console when the above commands were issued.

I'm setting up a 1.12.1 server right now to test this issue.

commented

As usual, I have to run BuildTools a thousand times before it finally creates the Spigot.jar file... It frequently fails after 5 minutes or more of running because something "didn't apply right".

commented

I downloaded the Spigot.jar file from froger599/NishiPlays and I was unable to recreate this issue:

https://pastebin.com/cXCef33i

Test results follow:


Line 3: Stormbow is displayed in the group named "Guest". (Working as intended.)

Line 12: Stormbow is set to the group named "Apollo".

Line 15: PermissionsEx reports that "Stormbow" is a member of "Apollo". (Working as intended.)

Line 24: Stormbow is set to the group named "Aphrodite".

Line 27: PermissionsEx reports that "Stormbow" is a member of "Aphrodite". (Working as intended.)


commented

I suspect that the chat management plugin for the server was programmed incorrectly— a result of copying and pasting possibly. This would result in the wrong prefix being displayed when a user is set to "Apollo" or "Aphrodite" and would subsequently deceive someone into believing the user was put in the wrong group, when they talk in chat, because they see "Zeus" or "Acolytes", respectively, instead of "Apollo" or "Aphrodite".

commented

I'm awaiting a reply from froger/Nishi to confirm or deny this theory.

commented

On closer inspection of your permissions.yml file, neither Apollo nor Aphrodite have prefixes assigned.

commented

Since the {DISPLAYNAME} variable is the user's group's PREFIX+USERNAME+SUFFIX, altogether, and since "Apollo" and "Aphrodite" haven't been assigned prefixes, they're inheriting the prefixes of other groups in their inheritance chains.

  Apollo:
    inheritance:
    - Zeus
    - Aphrodite
    - Persephone
    - Hades
    - Poseidon
 Aphrodite:
    inheritance:
    - Acolytes

"Apollo" is showing the prefix of the first group it inherits from ("Zeus") and so is "Aphrodite"— which only has 1 inheritance, so it's using the "Acolytes" group's prefix.