[BUG] Skript ignores event cancellation.

[ghost]

Skript/Server Version

Server Version: git-Paper-448 (MC: 1.19.3) 
Skript Version: 2.6.4 
Installed Skript Addons:
- Skellet v2.0.9 
-skRayFall v1.9.28 
- SkQuery v4.1.9 
- SkBee v2.8.2 
Installed dependencies: 
- Vault v1.7.3-b131 
- WorldGuard v7.0.7+216b061

Bug Description

Skript ignores event cancellation. When a plugin like CommandWhitelist tries to cancel the event, it doesn’t work. Source: YouHaveTrouble/CommandWhitelist/issues/73

Expected Behavior

Skript's no permission message shouldn't show.

Steps to Reproduce

1. Get CommandWhitelist for Bukkit (https://www.spigotmc.org/resources/commandwhitelist-spigot-waterfall-velocity.81326/).
2. Put CommandWhitelist's .jar file in the plugins folder.
3. Make this code with the file name test.sk:

command /test:
  permission: permission.test
  trigger:
    send "test" to player

4. Save your file and do /sk reload test.
5. CommandWhitelist's configured no such command message will popup and Skript's no permission message.

Errors or Screenshots

No response

Other

No response

Agreement

• I have read the guidelines above and affirm I am following them with this report.

[TheLimeGlass]

This is because Skript is NOT actually making a server registered command. Skript just listens within a on chat/command event when the phase is said /test. This is how you can dynamically make commands just by reloading a script.

If you wish for Skript to take priority over the plugin in question. Change the event priority of Skript in the config.sk to something higher or you can even use the priority pattern after any event with [with priority (lowest|low|normal|high|highest|monitor)]

Please explain what you are trying to do with Skript in this regards, as we do not know the purpose of CommandWhitelist.

[ghost]

CommandWhitelist (https://www.spigotmc.org/resources/commandwhitelist-spigot-waterfall-velocity.81326/) is for blocking command execution and tab completion. Skript will still show the no permission message when CommandWhitelist cancels the event. I tried to make skript lower priority which didn't work.

[sovdeeth]

Yes, Skript ignores all cancellations when listening for commands to run. I don't think it should do this, but I haven't investigated as to why it ignores cancellations yet.

[sovdeeth]

@JeffreyRCGithub Try this nightly jar for 2.7: https://github.com/SkriptLang/Skript/actions/runs/6066909722
I can make you one for 2.6.4 if you can't test on 2.7 for some reason.

[UnderscoreTud]

We usually leave issues open until a version that contains the fix/feature is released.

[sovdeeth]

We usually leave issues open until a version that contains the fix/feature is released.

It should actually have been included in 2.7.1 I believe, it was cherrypicked from the feature branch to the patch branch. So it should be good to close now. I forgot about it in favor of the other issue #5966 fixed.