Attempt 5 of seeking proffesional help | skUnity | Page 1

cedar rapids Jul 23, 2023, 3:15 AM

#

Please someone help me ive been stuck on this for weeks now there has to be a solution somewhere it just doesnt make sense

Intended outcome:

I close the gui
Class Gui reopens
I close the gui
Class Gui reopens
Continued

Reality:

I close the gui
Class Gui reopens
I close the gui
Nothing.

Event:

on inventory close:
    if name of event-inventory contains "Choose your class":
        classgui(player)

Function:

function classgui(p: player):
    set {ingui::%uuid of {_p}%} to {_p}'s uuid    
    if team of {_p} is {r} or {b}:
        if team of {_p} is {r}:
            set {_guinamecolor} to "&4&l"
        if team of {_p} is {b}:
            set {_guinamecolor} to "&1&l"    
        else:
            set {_guinamecolor} to "&f&l"        
        set {_classgui} to a new chest inventory with 1 row with name "%{_guinamecolor}%Choose your class"

#

before someone sends a solution allow me to present the failed solution board:

gleaming cosmos Jul 23, 2023, 3:29 AM

#

Why is debug crossed off?

#

I told you to do that in the other thread, then you told me you created this thread and that i was “free to go”, instead of providing results of debugging. So either you didnt do it and should, or you are disregarding important information that could help.

#

As I also mentioned the first condition in the function is redundant

gleaming cosmos Jul 23, 2023, 3:46 AM

#

gleaming cosmos As I also mentioned the first condition in the function is redundant

And if its passed theres no need for the else. You should also change the b one to an else if

cedar rapids Jul 23, 2023, 3:57 AM

#

gleaming cosmos Why is debug crossed off?

the results didnt prove anything helpful

gleaming cosmos Jul 23, 2023, 3:58 AM

#

as I also said you didnt show results

cedar rapids Jul 23, 2023, 3:59 AM

#

#

i really dont know what you are expecting

gleaming cosmos Jul 23, 2023, 3:59 AM

#

thats not what I asked you to do

cedar rapids Jul 23, 2023, 3:59 AM

#

i dont understand the term "debug it" even means

#

no one will tell me

#

they just say "debug it"

gleaming cosmos Jul 23, 2023, 4:01 AM

#

it means find the specific line/portion that the issue originates from. add broadcast x to check. If I have an issue I will add broadcast "a = b" after a condition if a = b:. If its not broadcasted I know I didnt set a or something.

gleaming cosmos Jul 23, 2023, 4:01 AM

#

cedar rapids no one will tell me

You could have googled it.

gleaming cosmos Jul 23, 2023, 4:02 AM

#

cedar rapids i dont understand the term "debug it" even means

Then why did you cross it off on the list???

cedar rapids Jul 23, 2023, 4:03 AM

#

gleaming cosmos You could have googled it.

???

#

that woulda just made an even bigger rabbit hole is skript debugging wasnt the same as what came up on google

gleaming cosmos Jul 23, 2023, 4:05 AM

#

google says "the proccess of finding and fixing errors in code"

#

but regardless of what google says, now that you know what I mean, try it out

#

brodacast things

#

the player's team, any variables, etc

cedar rapids Jul 23, 2023, 4:08 AM

#

gleaming cosmos it means find the specific line/portion that the issue originates from. add `bro...

#

results

cedar rapids Jul 23, 2023, 4:09 AM

#

gleaming cosmos the player's team, any variables, etc

i have commmands for that already

cedar rapids Jul 23, 2023, 4:29 AM

#

on inventory close:
    broadcast "on inventory close:"
    if metadata "class gui" of player is event-inventory:
        broadcast "if metadata 'class gui' of player is event-inventory:"
        broadcast "&oclassgui(player)&e running"
        classgui(event-player)
        broadcast "&oclassgui(player)&2 done"

#

function classgui(p: player):
    set {_guinamecolor} to "&f&l" 
    broadcast "guiname &fset"
    if team of {_p} is {r}:
        set {_guinamecolor} to "&4&l"
        broadcast "guiname &4red"
    else if team of {_p} is {b}:
        set {_guinamecolor} to "&1&l"     
        broadcast "guiname &1blue"
    set {classgui} to a new chest inventory with 1 row with name "%{_guinamecolor}%Choose your class"
    set metadata "class gui" of {_p} to {classgui}
    broadcast "&2metadata 'class gui' &rset to {classgui}"
    set {ingui::%uuid of {_p}%} to {_p}'s uuid
    open {classgui} to {_p}
    broadcast "&6{_classgui} opened"

this is so confusing how does it only broadcast the last line of the function

gleaming cosmos Jul 23, 2023, 4:39 AM

#

how come it did earlier?

gleaming cosmos Jul 23, 2023, 4:39 AM

#

cedar rapids

.

#

    set {_guinamecolor} to "&f&l" 
    if team of {_p} is {r}:
        set {_guinamecolor} to "&4&l"
    else if team of {_p} is {b}:
        set {_guinamecolor} to "&1&l"     
    set {_classgui} to a new chest inventory with 1 row with name "%{_guinamecolor}%Choose your class"
    broadcast unformatted "name: %{_guinamecolor}%Choose your class"
    set {ingui::%uuid of {_p}%} to {_p}'s uuid
    open {_classgui} to {_p}
    broadcast "&6{_classgui} opened"

on inventory close:
  broadcast unformatted "%name of event-inventory% was closed"```

#

see what this does

cedar rapids Jul 23, 2023, 3:25 PM

#

gleaming cosmos ```function classgui(p: player): set {_guinamecolor} to "&f&l" if team ...

Oooohhh

#

Okay

#

I didn't know you can do that

#

That will help

#

In a few hours when I arrive home

cedar rapids Jul 24, 2023, 6:35 PM

#

i cant even open the {_classgui} anymore

#

wtf

gleaming cosmos Jul 24, 2023, 7:00 PM

#

send_code

graceful nova Jul 24, 2023, 7:19 PM

#

there is an alternative to doing all of this name checking. you can set a metadata tag of the player to the gui, then open that gui. and when you close the inventory you can check if the metadata tag is set or if the event-inventory is the one from the metadata tag and if it is: wait a tick, reopen the metadata tag gui

#

this sounds more confusing, but it'll make checking the inventory much easier

cedar rapids Jul 24, 2023, 8:29 PM

#

@gleaming cosmos

#

function classgui(p: player):
    set {ingui::%uuid of {_p}%} to {_p}'s uuid    
    set metadata "class gui" of {_p} to {_classgui}
    if team of {_p} is {r}:
        set {_guinamecolor} to "&4&l"
    if team of {_p} is {b}:
        set {_guinamecolor} to "&1&l"    
    else:
        set {_guinamecolor} to "&f&l"        
    set {_classgui} to a new chest inventory with 1 row with name "%{_guinamecolor}%Choose your class"
    open {_classgui} to {_p}
    broadcast "&6{_classgui} open"

on inventory close:
    if name of event-inventory contains "Choose your class":
#    if metadata "class gui" of player is event-inventory:
#        broadcast "&2'class gui' &finventory closed"
        broadcast "&8'Choose your class' &finventory closed"
        broadcast "&oclassgui(player)&e running"
        classgui(event-player)
        broadcast "&oclassgui(player)&2 done"

#

it spams like a hundred times

gleaming cosmos Jul 24, 2023, 8:34 PM

#

#

You cant use it before its set if its a local variable.

#

Also change the second team condition to else if

#

What happens if you use open event-inventory to player?

graceful nova Jul 25, 2023, 2:40 AM

#

gleaming cosmos

you are using a local variable outside of where it's created. You can't do that

#

if you're gonna do that, create the gui in the function
otherwise, set the gui to the metadata tag instead of a variable

gleaming cosmos Jul 25, 2023, 2:43 AM

#

im not op

gleaming cosmos Jul 25, 2023, 2:43 AM

#

gleaming cosmos You cant use it before its set if its a local variable.

.

#

im telling him that lol

graceful nova Jul 25, 2023, 2:43 AM

#

oh, I read that too fast then xD

#

sorry

gleaming cosmos Jul 25, 2023, 2:43 AM

#

no problem

graceful nova Jul 25, 2023, 2:52 AM

#

cedar rapids ```vb function classgui(p: player): set {ingui::%uuid of {_p}%} to {_p}'s uu...

You have the right idea, but the wrong approach
Example command /test: trigger: set metadata tag "SomeGUI" to a new chest inventory with 1 row named "Name" set slot 1 of metadata tag "SomeGUI" to 3 stone named "Some Stone" open metadata tag "SomeGUI" to player
Using this method, you can then reference the gui anywhere and it will be on the player.
Just note when the player logs off, the metadata will still be there; but if the server shuts down, I believe it will dissapear

cedar rapids Jul 25, 2023, 3:46 AM

#

okay will do

cedar rapids Jul 25, 2023, 3:52 AM

#

gleaming cosmos What happens if you use `open event-inventory to player`?

it would work but it wouldnt maintian the properties that the class gui gives it

#

like the name tag color

gleaming cosmos Jul 25, 2023, 3:52 AM

#

the inventory's name?

#

shouldnt it?

cedar rapids Jul 25, 2023, 4:01 AM

#

graceful nova You have the right idea, but the wrong approach __Example__ ``` command /test: ...

this doesnt work

#

#

#

i added the {_p} later after it already didnt work

graceful nova Jul 25, 2023, 4:04 AM

#

metadata tag "SomeTag" of player

#

you had the syntax backwards

cedar rapids Jul 25, 2023, 4:04 AM

#

graceful nova metadata tag "SomeTag" **of** player

#

nice eye

#

new error : (

#

im entierly just a baby waiting to be spoon fed cuz i dont understand metadatas at all

#

got the meta datas working

#

just had to tweak the syntax around

graceful nova Jul 25, 2023, 4:13 AM

#

you're still backwards

#

open (your metadata tag info here, the one I told you to fix) to {_p}

cedar rapids Jul 25, 2023, 4:16 AM

#

graceful nova `open (your metadata tag info here, the one I told you to fix) to {_p}`

it works regardless

#

👍

#

however

#

there is one thing i cant find for syntax on the internet

#

on inventory close:
    if metadata "Class gui" of player is event-inventory:

#

this is not detecting

set metadata tag "Class Gui" of {_p} to chest inventory with 1 row named "%{_guinamecolor} ? "&f&l"%Choose your class"

this inventory

graceful nova Jul 25, 2023, 4:17 AM

#

it might just be the wording. you can try
if event-inventory is metadata tag "Your tag" of player:

cedar rapids Jul 25, 2023, 4:18 AM

#

nice it detects now

#

but..............

#

the exact same thing is happening since the begining of this bug

#

Intended outcome:

I close the gui
Class Gui reopens
I close the gui
Class Gui reopens
Continued

Reality:

I close the gui
Class Gui reopens
I close the gui
Nothing.

graceful nova Jul 25, 2023, 4:19 AM

#

so it detects the gui just fine, right?

#

what is the code to reopen the gui?

cedar rapids Jul 25, 2023, 4:20 AM

#

heres the vid of whats happening

#

cedar rapids Jul 25, 2023, 4:20 AM

#

graceful nova what is the code to reopen the gui?

classgui(player)

#

im gonna manually reopen it now

#

hope it works

graceful nova Jul 25, 2023, 4:21 AM

#

I meant send the code

cedar rapids Jul 25, 2023, 4:22 AM

#

same outcome

cedar rapids Jul 25, 2023, 4:22 AM

#

graceful nova I meant send the code

k

#

on inventory close:
    if event-inventory is metadata tag "Class Gui" of player:
        broadcast "&2'Class Gui' &finventory closed"
        set {ingui::%uuid of player%} to player's uuid    
        if team of player is {r}:
            set {_guinamecolor} to "&4&l"
        else if team of player is {b}:
            set {_guinamecolor} to "&1&l"     
        set metadata tag "Class Gui" of player to chest inventory with 1 row named "%{_guinamecolor} ? "&f&l"%Choose your class"
        set slot 0 of metadata tag "Class Gui" of player to paper "alot more nbt in these slots"
        open (metadata tag "Class Gui" of player) to player
        broadcast "&2'Class Gui'&f open"

graceful nova Jul 25, 2023, 4:24 AM

#

before opening the gui, waitatick

cedar rapids Jul 25, 2023, 4:24 AM

#

graceful nova before opening the gui, <:waitatick:297214779720990720>

GENIUS

#

IT WORKS

#

this whole time the answer was sitting in the emoji list

silhouette-businessman-broken-chains-sunset-image-96811830.png

graceful nova Jul 25, 2023, 4:25 AM

#

figured that was the issue

cedar rapids Jul 25, 2023, 4:25 AM

#

now all my gui fromats are converted to meta data tags for no reason lol

graceful nova Jul 25, 2023, 4:26 AM

#

all of that is happening in the same tick, so opening the gui as soon as the inventory was closed, not enough time passes to create a new instance. So, wait a tick and you're good

cedar rapids Jul 25, 2023, 4:26 AM

#

lets goo

graceful nova Jul 25, 2023, 4:26 AM

#

cedar rapids now all my gui fromats are converted to meta data tags for no reason lol

true xD
But it can sometimes be easier to use these tags instead of having a bunch of unneccessary variables

cedar rapids Jul 25, 2023, 4:29 AM

#

yeah but now immma have to refer to this code everytime i wanna remake a gui

#

i like vairables

#

what evs its done now

#

if metadata value "pyromaniac" of player is false:
if metadata value "shadowwalker" of player is false:
if metadata value "quickfoot" of player is false:
if metadata value "frontliner" of player is false:
if metadata value "hunter" of player is false:
if indices of {ingui::} contain the uuid of player:
classgui(player)
else:
remove player's uuid from {ingui::}
else:
remove player's uuid from {ingui::}
else:
remove player's uuid from {ingui::}
else:
remove player's uuid from {ingui::}
else:
remove player's uuid from {ingui::}
else:
remove player's uuid from {ingui::*}

#

now i gotta add this monstrocity to it

graceful nova Jul 25, 2023, 4:34 AM

#

not necessarily. I only recommend metadata tags for things like guis

#

you don't need them for everything

cedar rapids Jul 25, 2023, 4:35 AM

#

@gleaming cosmos Guess what!!

#

all it took was waitatick

#

thank you both for dealing with my crap

#Attempt 5 of seeking proffesional help