I need help with this last thing if anyone’s down to help.

[slim wasp]

I'm having a problem there are no errors, but I just don’t know how to make it so people can’t take out items from my GUI. I tried making it so if someone is in a GUI with a specific name, then it cancels the inventory click, but for some reason that makes the whole GUI break. If anyone knows how to help, please let me know. Here’s the code:

this uses:

skript, skbee, skquery

newest versions

[trim kayak]

delete skquery

and you also need to provide more details:

1. Instead of saying you are "having a problem" and "makes the whole GUI break", actually explain these issues

• if you have no errors, debug.

2. instead of asking us to look through 280 lines and magically find an issue, after debugging, if you still cant figure it out, send a (small) snippet instead

can you also please tell me why all of these are identical?

you can refactor this code and save like 150 lines..

[slim wasp]

okay so first of all

i will not delete skquery

becacuse i use it for guis

it makes them easier

[slim wasp]

beacuse i havent puit the recipes in yet

and second of all

the problems are this:
you can take stuff out
if you add the cancel event thing then it doesnt let yoi click on the papers

that are named yk credits and recipoes and that

[slim wasp]

and sorry but i didnt know what snippet of code to send so i sent the whole thing

[trim kayak]

No. Skript itself has gui syntax, or use the skript-gui addon. Skquery is outdated and known to cause bugs in your code and even Skript.

[trim kayak]

you should still take the duplicate lines and run them beforehand:

if x == 1:
  set y to 1000
  set z to 5
else if x == 2:
  set y to 1000
  set z to 7``` ```# GOOD CODE

set y to 1000
if x == 1:
  set z to 5
else if x == 2:
  set z to 7```

[slim wasp]

im sorry i dont really understand

[slim wasp]

dude but skquery makes it more easy

[slim wasp]

you do know that i spent a while making that gui since im not amazing with them and now you want me to remake it?

[trim kayak]

Its definitely worth it now, rather than continuing to rely on a broken addon and need to re-write a whole lot more in the future

And honestly, half the syntax you use now is base skript anyways

[daring mauve]

have u ran sk reload all by chance?

since that makes all skquery guis stealable iirc

skript-gui is much better if u want easy & readable guis

https://github.com/APickledWalrus/skript-gui/

[slim wasp]

what? do you mean like when i reload cuz yes after a change i always do /rr - its a custom command made by me that reloads all scripts

if i can remove skquery then id rather just stick to skript and skbee i dont wnat a lot of addons

[daring mauve]

individual reload is fine but reload all will break skquery guis

[daring mauve]

https://forums.skunity.com/threads/vanilla-guis.8939/