is online/offline not working with variables

[AyhamAl-Ali]

Description

using is online/offline condition with variables is not working.

Steps to Reproduce

set {_p} to player
send "1" if {_p} is online # should send 1 but doesn't send anything
send "0" if {_p} is offline # should send 0 but doesn't send anything

Expected Behavior

Should send 1 if online and 0 if offline

Errors / Screenshots

None

Server Information

• Server version/platform: 1.16.5 Paper-771
• Skript version: 2.5.3

Additional Context

[TPGamesNL]

Likely addon conflict, probably DiSky, try without it

[AyhamAl-Ali]

Likely addon conflict, probably DiSky, try without it

Not using that.
My current addons

[TPGamesNL]

One of them has a type named 'onlinestatus', check which

[AyhamAl-Ali]

okay I will let you know as soon as I can.

[OGContent]

Description

using is online/offline condition with variables is not working.

Steps to Reproduce

set {_p} to player
send "1" if {_p} is online # should send 1 but doesn't send anything
send "0" if {_p} is offline # should send 0 but doesn't send anything

Expected Behavior

Should send 1 if online and 0 if offline

Errors / Screenshots

None

Server Information

• Server version/platform: 1.16.5 Paper-771
• Skript version: 2.5.3

Additional Context

the else didn't work somehow
git-Paper-"e04368045" mc: 1.16.5
skript version 2.5.3

[AyhamAl-Ali]

Can confirm that it works as expected without addons only TuSKe. (haven't yet found which addon conflict with this)

the else didn't work somehow

Can confirm as well. there was a fix for EffDoIf for stopping the whole trigger instead of the line itself in 2.6.0 not sure if this is related.

[TPGamesNL]

It's Vixio, see https://github.com/iBlitzkriegi/Vixio/blob/9cca56fa1fa4cb6253000b1175e300e4bf2efe21/src/main/java/me/iblitzkriegi/vixio/registration/Types.java#L224-L241

[AyhamAl-Ali]

It's Vixio, see https://github.com/iBlitzkriegi/Vixio/blob/9cca56fa1fa4cb6253000b1175e300e4bf2efe21/src/main/java/me/iblitzkriegi/vixio/registration/Types.java#L224-L241

I suspected that from the beginning. Is there a way to maybe add a setting in each script to make Skript's vanilla effects, expressions, types etc. have a higher priority than addons? or should I make another suggestion for that?

[TPGamesNL]

They are parsed first, but CondCompare comes before CondIsOnline, so that wouldn't matter here. It would be possible to make CondCompare be attempted last, which would also solve #4123 (comment)

[AyhamAl-Ali]

They are parsed first, but CondCompare comes before CondIsOnline, so that wouldn't matter here. It would be possible to make CondCompare be attempted last, which would also solve #4123 (comment)

Indeed that would be good I think.