/list skript (help)

[Antonio]

Skript Version: 2.2-Fixes-V8b
Skript Author: Dzikoysk
Minecraft Version: 1.8.x
---
The Problem:
The problem is if there is only one player it's working fine but if the are 2 players online the list will be broadcastet 2 times

Picture when there is only 1 player online
https://gyazo.com/f79150a2368c6f814dc10ed121b45a0d

Picture when there are 2 player in online
https://gyazo.com/d4bafd9129199c6ee1146530497c5e59

Full code:

options:
    list1: &7&m-------------&r &4&l»&cServer  List&4&l« &7&m-------------
    list2: &7&m----------------------------------------
 
command /list:
    trigger:
        loop all players:
            clear {owner::*}
            clear {admin::*}
            clear {mod::*}
            clear {helper::*}
            clear {staff::*}
            clear {kriger::*}
            clear {krigerlist::*}
            if loop-player has permission "list.owner":
                add loop-player to {staff::*}
                add loop-player to {owner::*}
            else if loop-player has permission "list.admin":
                add loop-player to {staff::*}
                add loop-player to {admin::*}
            else if loop-player has permission "list.mod":
                add loop-player to {staff::*}
                add loop-player to {mod::*}
            else if loop-player has permission "list.helper":
                add loop-player to {staff::*}
                add loop-player to {helper::*}
            else if loop-player has permission "list.kriger":
                add loop-player to {kriger::*}
                add 1 to {krigerlist::*}
            message "{@list1}"
            if size of {owner::*} is 0:
                message "&4Ejere:&7 (&40&7)&4:: &c&oIngen Ejere online..."
                message "{@list2}"
                wait 0 tick
            else:
                message "&4Ejere&8: &f%{owner::*}%"
                message "{@list2}"
            if size of {admin::*} is 0:
                message "&cAdministratorer&7 (&c0&7)&c: &c&oIngen Admins online..."
                message "{@list2}"
                wait 0 tick
            else:
                message ""
                message "&cAdministratorer(s)&8: &f%{admin::*}%"
                message "{@list2}"
            if size of {mod::*} is 0:
                message "&9Moderator&7 (&90&7)&9: &c&oIngen Moderators online..."
                message "{@list2}"
                wait 0 tick
            else:
                message ""
                message "&9Mderator(s)&8: &f%{mod::*}%"
                message "{@list2}"
            if size of {helper::*} is 0:
                message "&2Hjælpere&7 (&20&7)&2: &c&oIngen Hjælpere online..."
                message "{@list2}"
                wait 0 tick
            else:
                message ""
                message "&2Hjælpere&8: &f%{helper::*}%"
                message "{@list2}"
            if size of {kriger::*} is 0:
                message "&eKrigere&7 (&e0&7)&e: &c&oIngen Krigere online..."
                message "{@list1}"
                wait 0 tick
            else:
                message "&eKrigere&7 (&e%{krigerlist::*}%&7)&e: &f%{kriger::*}%"
                message "{@list1}"

Errors on Reload:

None

Console Errors: (if applicable)

None

Other Useful Info:

Addons using (including versions):
SkQuery,Skellett,WildSkript,PirateSK,RandomSK,Umbaska And SkRauFall all with newest update

Troubleshooting:

Have you tried searching the docs? Yes
Have you tried searching the forums? Yes and spigot and bukkit forum

 

[Deleted member 5254]

@Antonio
More efficient , less lines and removed useless list variables.

command /list:
    trigger:
        clear {staff::*}
        loop all players:
            if loop-player has permission "list.owner":
                add loop-player to {staff::owner::*}
            if loop-player has permission "list.admin":
                add loop-player to {staff::admin::*}
            if loop-player has permission "list.mod":
                add loop-player to {staff::mod::*}
            if loop-player has permission "list.helper":
                add loop-player to {staff::helper::*}
            if loop-player has permission "list.kriger":
                add loop-player to {staff::kriger::*}
        message "&7&m-------------&r &4&l»&cServer  List&4&l« &7&m-------------" to the player
        if size of {staff::owner::*} is 0:
            message "&cEjere:&7 (&40&7)&4:: &c&oIngen Ejere online..." to the player
        else if size of {staff::owner::*} is more than 0:
            message "&cEjere&8: &f%{staff::owner::*}%"  to the player
        if size of {staff::admin::*} is 0:
            message "&cAdmin:&7 (&40&7)&4:: &c&oIngen Admin online..." to the player
        else if size of {staff::admin::*} is more than 0:
            message "&cAdmin&8: &f%{staff::admin::*}%"  to the player
        if size of {staff::mod::*} is 0:
            message "&cMod:&7 (&40&7)&4:: &c&oIngen Mod online..." to the player
        else if size of {staff::mod::*} is more than 0:
            message "&cMod&8: &f%{staff::mod::*}%"  to the player
        if size of {staff::helper::*} is 0:
            message "&cHelper:&7 (&40&7)&4:: &c&oIngen Helper online..." to the player
        else if size of {staff::helper::*} is more than 0:
            message "&cHelper&8: &f%{staff::helper::*}%"  to the player
        if size of {staff::kriger::*} is 0:
            message "&cKriger:&7 (&40&7)&4:: &c&oIngen Kriger online..." to the player
        else if size of {staff::kriger::*} is more than 0:
            message "&cKriger&8: &f%{staff::kriger::*}%" to the player
        message "&7&m----------------------------------------" to the player

 

[Deleted member 5254]

@YoshYz
Thanks but if a player is up do just this and how to remove and and doing so for example, is Antonio0215, AntaryDK https://gyazo.com/9fff2d86183aaf32d5a89e1131e4e5a3

 

[Deleted member 5254]

@YoshYz
Thanks but if a player is up do just this and how to remove and and doing so for example, is Antonio0215, AntaryDK https://gyazo.com/9fff2d86183aaf32d5a89e1131e4e5a3

Use this trick for every group

loop {staff::owner::*}:
    set {_set.%loop-value%} to loop-value
    
loop {staff::admin::*}:
    if {_set.%loop-value%} is set:
        remove loop-value from {staff::admin::*}

 

[Deleted member 5254]

@YoshYz
Sorry but I'm new to the script where should I put it?

 

[Deleted member 5254]

@YoshYz
Sorry but I'm new to the script where should I put it?

Not more at home now , you can use single list variable for each group as first , at least your double message is fixed.

 

[Deleted member 5254]

@YoshYz
I get these two errors https://gyazo.com/80a9f83e5a8b572037ca6896d84d174e