Skript not working

[Hyper611]

I have a server that you can dupe stuff with /dupe and was trying to make a few things not able to dupe.
The pink netherite still dupes while the blue netherite doesn't. Can anyone help?

 

command /dupe:
	cooldown: 2 seconds
	cooldown message: This command is on cooldown
	trigger:
		if name of player's held item is "&dPink Netherite":
			send "&cYou can not dupe this item!" to player
		else:
		if name of player's held item is "&1Blue Netherite":
			send "&cYou can not dupe this item!" to player
		else:
			give player player's held item

 

[_Tarna_]

Its probably because its running both if statements. When you are holding the pink netherite, it runs the if player's held item is blue netherite line, and since the pink netherite is not that, it still dupes it. Do something like this

set {_blocked::*} to stick, stone, bedrock
if {_blocked::*} contains player's tool:
	send "You may not dupe this item"
else:
	give player player's held item

And then just set the local blocked list variable to the items you don't want players to be able to dupe.

[Hyper611]

6 minutes ago, _Tarna_ said:

Its probably because its running both if statements. When you are holding the pink netherite, it runs the if player's held item is blue netherite line, and since the pink netherite is not that, it still dupes it. Do something like this

set {_blocked::*} to stick, stone, bedrock
if {_blocked::*} contains player's tool:
	send "You may not dupe this item"
else:
	give player player's held item

And then just set the local blocked list variable to the items you don't want players to be able to dupe.

Ok thanks!

[Hyper611]

11 hours ago, _Tarna_ said:

Its probably because its running both if statements. When you are holding the pink netherite, it runs the if player's held item is blue netherite line, and since the pink netherite is not that, it still dupes it. Do something like this

set {_blocked::*} to stick, stone, bedrock
if {_blocked::*} contains player's tool:
	send "You may not dupe this item"
else:
	give player player's held item

And then just set the local blocked list variable to the items you don't want players to be able to dupe.

 didn't work

[MrTibo]

It's caused by your indentation being wrong, but instead you can use the word stop to stop the script.

I've changed your original script in the code block below

command /dupe:
	cooldown: 2 seconds
	cooldown message: This command is on cooldown
	trigger:
		if name of player's held item is "&dPink Netherite":
			send "&cYou can not dupe this item!" to player
			stop
		if name of player's held item is "&1Blue Netherite":
			send "&cYou can not dupe this item!" to player
			stop
		give player player's held item

 

[Hyper611]

6 minutes ago, MrTibo said:

It's caused by your indentation being wrong, but instead you can use the word stop to stop the script.

I've changed your original script in the code block below

command /dupe:
	cooldown: 2 seconds
	cooldown message: This command is on cooldown
	trigger:
		if name of player's held item is "&dPink Netherite":
			send "&cYou can not dupe this item!" to player
			stop
		if name of player's held item is "&1Blue Netherite":
			send "&cYou can not dupe this item!" to player
			stop
		give player player's held item

 

Thank you!

[_Tarna_]

Moved to Skript

Please post in the correct category next time