Help with GUI

[xTarheel]

So I made this

command /Crystals [<player>]:
    aliases: crystal,c
    trigger:
        if {Crystals::%player%} is not set:
            set {Crystals::%player%} to 0
        if {Crystals::%player%} is -1:
            set {Crystals::%player%} to 0
        if arg 1 is not set:
            message "&8You have &b%{Crystals::%player%}% Crystals."
        else:
            message "&8%arg 1% has &b%{Crystals::%player%}% Crystals."

command /cshop:
    trigger:
        open chest with 3 rows named "&9Crystals&bShop" to player
        format slot 4 of player with arg 1's skull named "%player%" with lore "&cThats you!" to be unstealable
        format slot 11 of player with nether star named "&eAccess to /feed" to run [pex user %Player% add essentials.feed]

I need it to be like when you click the nether star it takes 50 crystals away from you and gives you access to /feed, but im lost. Very lost

 

[ShaneBee]

Well first line 17 is not correct. It goes like this:

 format slot 11 of player with nether star named "&eAccess to /feed" to run "pex user %Player% add essentials.feed"

Also you would have to make it run a command that runs the pex command and removes 50 from your crystals variable. Like this:

command /crystalshoprem <player> <text>:
    trigger:
        if arg 2 is "feed":
            if {Crystals::%arg 1%} is not less than 50:
                remove 50 from {Crystals::%arg 1%}
                execute console command "pex user %arg 1% add essentials.feed"
                send "&aYou have bought the /feed command for 50 crystals!" to arg 1
            else:
                send "&cYou don't have enough crystals!" to arg 1

 

[ShaneBee]

Well first line 17 is not correct. It goes like this:

 format slot 11 of player with nether star named "&eAccess to /feed" to run "pex user %Player% add essentials.feed"

Also you would have to make it run a command that runs the pex command and removes 50 from your crystals variable. Like this:

command /crystalshoprem <player> <text>:
    trigger:
        if arg 2 is "feed":
            if {Crystals::%arg 1%} is not less than 50:
                remove 50 from {Crystals::%arg 1%}
                execute console command "pex user %arg 1% add essentials.feed"
                send "&aYou have bought the /feed command for 50 crystals!" to arg 1
            else:
                send "&cYou don't have enough crystals!" to arg 1

Ok, thanks

 

[ShaneBee]

Ok, thanks

I understand that you have found a solution to your problem, but that isn't the best way to do it, as that allows for unnecessary commands and the use of format slot, which isn't the best option for GUIs.

I would suggest using set slot to format your GUIs. For example, something like:

set slot 5 of player's current inventory to stone named "&7Hey" with lore "Heyyyy"

Then, you would use the on inventory click event to trigger certain functions.
For example:

on inventory click:
    if inventory name of player's current inventory is " "
    # Check the name of the player's inventory against a value.
    # Replace the spaces with the name of your inventory. For example, "&9Shop."
        if clicked slot is 44:
        # Check the slot the player clicked against a value.
            doStuff(player)

Will this, you can also check against and grab the name and lore of the item that the player clicked. You could also build your function directly into the inventory click event.

 

[ShaneBee]

snip

Or even better, use the TuSKe's GUI Manager which is very similar to the format slot but a lot more customizable and less buggy.

 

[ShaneBee]

Or even better, use the TuSKe's GUI Manager which is very similar to the format slot but a lot more customizable and less buggy.

I've always found set slot easier to manipulate and use, but hey, we all have preferences.