/report Skript

[KngdomsGG]

Another Release but smaller now

 

command /report [<player>] [<text>]:
    trigger:
        if arg-2 is not set:
            send "&7Usage: &c/report [player] [reason]" to player
            send "&a&6/report duckbeary fly" to player
        else if arg-1 is not set:
            send "&7Usage: &c/report [player] [reason]" to player
            send "&a&6/report duckbeary fly" to player
        else:
            loop all players:
                if loop-player has permission "skript.reports":
                    send "&c&lREPORT!" to loop-player
                    send "&6%player% &7reported &6%arg-1%" to loop-player
                    send "&7Reason: &6%arg-2%" to loop-player

 

or

https://pastebin.com/raw/SsNLnETC

Make sure to credit me!

[Stargazers]

Instead of looping (which may cause lag) you can use:

send "msg" to all players where [input has the permission "perm"]

[Criticyl]

1 hour ago, Star said:

Instead of looping (which may cause lag) you can use:

send "msg" to all players where [input has the permission "perm"]

+1

[TheRustySpud]

18 hours ago, Criticyl said:

+1

+1

[noobie]

nice job. don't use "loop all players" tho, it could cause lag

[farwl]

Cool

[ThatOneTqnk]

Looking at the source, the 'where' filter loops anyway, it just is not evident to the Skripter. https://github.com/SkriptLang/Skript/blob/ec1e8d88bcd28f65c5a7613f63c0812332072256/src/main/java/ch/njol/skript/expressions/ExprFilter.java#L89-L109

 

So there shouldn't be a difference in performance actually. You have to iterate, a server cannot do multiple operations at once.

Edited December 3, 2019 by ThatOneTqnk

[pptjes]

ACCOUNT HACKED: https://youtube.com/ShadowSparkYT

ACCOUNT HACKED: https://youtube.com/ShadowSparkYT

ACCOUNT HACKED: https://youtube.com/ShadowSparkYT

ACCOUNT HACKED: https://youtube.com/ShadowSparkYT

ACCOUNT HACKED: https://youtube.com/ShadowSparkYT

ACCOUNT HACKED: https://youtube.com/ShadowSparkYT

ACCOUNT HACKED: https://youtube.com/ShadowSparkYT

ACCOUNT HACKED: https://youtube.com/ShadowSparkYT

ACCOUNT HACKED: https://youtube.com/ShadowSparkYT

ACCOUNT HACKED: https://youtube.com/ShadowSparkYT

ACCOUNT HACKED: https://youtube.com/ShadowSparkYT

Edited March 22, 2020 by pptjes

[zKoro]

On 12/30/2019 at 12:57 PM, pptjes said:

how do i see the reports

 

In the Skript, the reports aren't logged meaning that you cannot see previously made reports

[Swayl]

On 12/30/2019 at 8:57 PM, pptjes said:

how do i see the reports

 

If you want to see the previous reports, use this report system

[hacendj]

On 11/25/2019 at 9:19 PM, KngdomsGG said:

Another Release but smaller now

 

command /report [<player>] [<text>]:
    trigger:
        if arg-2 is not set:
            send "&7Usage: &c/report [player] [reason]" to player
            send "&a&6/report duckbeary fly" to player
        else if arg-1 is not set:
            send "&7Usage: &c/report [player] [reason]" to player
            send "&a&6/report duckbeary fly" to player
        else:
            loop all players:
                if loop-player has permission "skript.reports":
                    send "&c&lREPORT!" to loop-player
                    send "&6%player% &7reported &6%arg-1%" to loop-player
                    send "&7Reason: &6%arg-2%" to loop-player

 

or

https://pastebin.com/raw/SsNLnETC

Make sure to credit me! suck

 

[UntitledGoose]

On 12/2/2019 at 11:41 PM, ThatOneTqnk said:

Looking at the source, the 'where' filter loops anyway, it just is not evident to the Skripter. https://github.com/SkriptLang/Skript/blob/ec1e8d88bcd28f65c5a7613f63c0812332072256/src/main/java/ch/njol/skript/expressions/ExprFilter.java#L89-L109

 

So there shouldn't be a difference in performance actually. You have to iterate, a server cannot do multiple operations at once.

this is true but it does decrease parsing time (ever so slightly, that doesn't really matter) and it's more organized, organized as in fewer lines