Fly is not turning off for players when they switch to a world where they don't have permission

Momshroom commented 10 months ago

Description of Issue

When players have /fly on in a world where they can turn it on and change worlds, they are able to keep flying in the new world, even though they do not have permission there (and can't even turn it off).

Version Information

[07:15:42 INFO]: --------------------------------------------------
[07:15:42 INFO]: CMI plugin version: 9.6.9.5 MySQL
[07:15:42 INFO]: CMILib: 1.4.4.3 
[07:15:42 INFO]: Server: Paper(68) 1.20.1-R0.1-SNAPSHOT+
[07:15:42 INFO]: CMI economy: Disabled Vault: 1.7.3-b131 CMI Chat: Disabled 
[07:15:42 INFO]: Modules -> 51 enabled 8 disabled: spawnerProximity, flightCharge, disabledEnchants, mirror, ranks, moneyCheque, spawners, spawnerCharge
[07:15:42 INFO]: --------------------------------------------------

Errors

none

Relevant Config Sections

none that I can find.
Noting here that fly is not completely blocked in the relevant worlds, just not all players have permission and fly is not being turned off for those that do not on world change.

Relevant Plugins

multiverse-core 4.3.1-b861

Agreements

[X] My server is supported by CMI.
[X] My version of CMI at the time of this report is up to date.
[X] I have searched the github and asked around before making this report.

1470429149 commented 10 months ago

me too

Momshroom commented 9 months ago

This is still very much a problem on our server in Minecraft 1.20.4 (paper) and the latest CMI/CMIlib (Note the output below bizarrely suggests downgrading is an update, but I am on the latest. I'm also clarifying that we do not use CMI's skins feature as that was mentioned in someone else's fly ticket Please look into this when you can and lmk if you have further questions.

18:37:16 INFO: CMI plugin version: 9.6.9.9 MySQL-> 9.6.9.8 18:37:16 INFO: CMILib: 1.4.4.7 18:37:16 INFO: Server: Paper(389) 1.20.4-R0.1-SNAPSHOT+ 18:37:16 INFO: CMI economy: Disabled Vault: 1.7.3-b131 CMI Chat: Disabled 18:37:16 INFO: Modules -> 51 enabled 8 disabled: spawnerProximity, flightCharge, disabledEnchants, mirror, ranks, moneyCheque, spawners, spawnerCharge

ver cmi [18:37:38 INFO]: CMI version 9.6.9.9 [18:37:38 INFO]: Powerful server management tool [18:37:38 INFO]: Website: https://www.spigotmc.org/resources/content-management-interface.3742/ [18:37:38 INFO]: Author: Zrips

Momshroom commented 9 months ago

For anyone else having this issue, with some help from Floris, I've found a workaround that gets me the desired behavior: in the EventCommands.yml file, set this:

playerWorldChange:
  Enabled: true
  Commands:
  - perm:cmi.command.fly@! asConsole! cmi fly [playerName] false -s

Players who don't have perms will be taken out of fly.

1470429149 commented 9 months ago

For anyone else having this issue, with some help from Floris, I've found a workaround that gets me the desired behavior: in the EventCommands.yml file, set this:
playerWorldChange:
  Enabled: true
  Commands:
  - perm:cmi.command.fly@! asConsole! cmi fly [playerName] false -s
Players who don't have perms will be taken out of fly.

thank you!

Zrips / CMI