skript-gui icon indicating copy to clipboard operation
skript-gui copied to clipboard

Is the is single method incorrect for next gui slot?

Open Fusezion opened this issue 2 years ago • 4 comments

Description

I ran into an error for using set {_slot} to next gui slot saying a single variable can't be set to more than one object, after looking at the code I've come to realize that the comparison for isSingle might be incorrect.

Code in Question https://github.com/APickledWalrus/skript-gui/blob/5e6c741218c55c4a22680d4f238bd721c2a7e846/src/main/java/io/github/apickledwalrus/skriptgui/elements/expressions/ExprNextGUISlot.java#L75-L78

My View the condition guis != null && guis.isSingle() should be guis == null || guis.isSignle(), since the system should output true if it's null otherwise return what the expression says it is.

Note

I'm aware this should have been a bug report, but didn't wanna log into my server to get the version information or screenshots

Fusezion avatar Oct 11 '23 03:10 Fusezion

Is that why I keep getting this message when sending the next gui slot? image

LohsMC avatar Nov 25 '23 23:11 LohsMC

No

Fusezion avatar Nov 26 '23 00:11 Fusezion

sounds like a bug :) thought i replied to this already but it is on my radar

APickledWalrus avatar Nov 29 '23 20:11 APickledWalrus