Solved list variable contains

[SoMuchWessel]

Hey guys, i thought i got a working code to mute people:

command /mute [<player>]:
    permission: mute.mute
    trigger:
        if arg-1 is set:
            if {mutes::*} contain arg-1:
                send "&b%arg-1% is already muted!" to player
                stop
            else:
                add arg-1 to {mutes::*}
                send "&b%arg-1% is now muted!" to player
                send "&bYou are muted by %player%!" to arg-1
                send "&b%player% muted %arg-1%!" to console
                stop
        else:
            send "&bPlease define a player." to player
            stop

This code doesn't give an error, but it doesn't work.
It a player is in {mutes::*}, but he can still talk.

Help?

 

[WiebeHero]

So you need to have an event that cancels you from chatting let me give you an example.
This is a raw sketch btw so it will not work with how ur code is right now.

on chat:
    if {Mutes.%player%} is set true:
        cancel event
        send "&7You are muted! You are not allowed to talk!"

 

[SoMuchWessel]

So you need to have an event that cancels you from chatting let me give you an example.
This is a raw sketch btw so it will not work with how ur code is right now.

on chat:
    if {Mutes.%player%} is set true:
        cancel event
        send "&7You are muted! You are not allowed to talk!"

i already got that piece of code working, but you can still chat when you are muted.
And i want to use a list variable so that i can unmute everyone with /unmute by deleting the list variable

 

[WiebeHero]

So unmuting everyone is simple enough you just do this.
However with the other thing i can not help sorry

command /unmute:
    trigger:

 

[Runakai]

i already got that piece of code working, but you can still chat when you are muted.
And i want to use a list variable so that i can unmute everyone with /unmute by deleting the list variable

command /mute [<player>]:
    permission: mute.mute
    trigger:
        if argument is set:
            if {mutes::*} contains argument:
                send "&b%arg-1% is already muted!" to player
            else:
                add argument to {mutes::*}
                send "&b%argument% is now muted!" to player
                send "&bYou are muted by %player%!" to arg-1
                send "&b%player% muted %arg-1%!" to console
        else:
            send "&bPlease define a player." to player
           
command /unmute {<player>]:
    trigger:
        if {mutes::*} contains argument:
            remove argument from {mutes::*}
        else:
            send "&7Please define a player"
           
on chat:
    if {mutes::*} contains player:
        cancel event
        send "&7You are muted"

 

[WiebeHero]

clear {mutes::*}

 

[FinalPlayer024]

I think 'contains' is buggy (?) in Skript. Try using

This function

function contains(list: objects, testo: object) :: boolean:
    loop {mutes::*}:
        if loop-value is {_testo}:
            return true
    return false

on chat:
  if contains({mutes::*}, "%player%") is true:
    cancel event
    send "You're muted" to player

or this

on chat:
  loop {mutes::*}:
    "%loop-value%" = "%player%"
    cancel event
    send "You're muted" to player

 

[Runakai]

I think 'contains' is buggy (?) in Skript. Try using

This function

function contains(list: objects, testo: object) :: boolean:
    loop {mutes::*}:
        if loop-value is {_testo}:
            return true
    return false

on chat:
  if contains({mutes::*}, "%player%") is true:
    cancel event
    send "You're muted" to player

or this

on chat:
  loop {mutes::*}:
    "%loop-value%" = "%player%"
    cancel event
    send "You're muted" to player

For me its not buggy. It's useless to loop it

 

[Donut]

you have to loop, contains for lists is broken. that being said the function that finalplayer gave you isnt quite correct

function contains(list: objects, testo: object) :: boolean:
    loop {_objects::*}:
        if loop-value is {_testo}:
            return true
    return false

on chat:
  if contains({mutes::*}, player) is true:
    cancel event
    send "You're muted" to player

 

[SoMuchWessel]

I used the %loop-player% = %player% code, and this worked fine!

 

[Donut]

no need to stringify it