[SCHAT] - Zarządzanie Chatem

Nazwa: SCHAT
Autor: Szymuś
Opis: Zarządzanie Chatem

if player has permission "szymus.chat":
else:
	...

po co jest to?
Wystarczy to zastąpić:
 

if player doesn't have permission "szymus.chat":
	...

taki protip: Nie rób broadcasta, bo to zaśmieca konsolę. Lepiej zrób

send "..." to all players

Moim zdaniem to jest skrypt z tego filmu: https://www.youtube.com/watch?v=lCHSQhn_3B4

32 minuty temu, lie_pan1 napisał:

if player has permission "szymus.chat":
else:
	...

po co jest to?
Wystarczy to zastąpić:
 

if player doesn't have permission "szymus.chat":
	...

taki protip: Nie rób broadcasta, bo to zaśmieca konsolę. Lepiej zrób

send "..." to all players

Moim zdaniem to jest skrypt z tego filmu: https://www.youtube.com/watch?v=lCHSQhn_3B4

Dzięki za protipy mordeczko

Skrypt nie jest z filmiku mimo ze wygląda tak

Zacznijmy od tego, że skrypt jest źle wytabowany,

druga sprawa po co else używać gdy nie ma takiej potrzeby, zamiast tego zrób

on chat:
    if player has permission "szymus.chat":
        stop
    if {chat::status} is true:
        stop
    cancel event 
    send "&8» &7Chat jest &cwylaczony"

 uniknie to robienia wielkiego drzewa if / else,

ewentualnie inline conditional 

on chat:
    if player has permission "szymus.chat":
        stop
    {chat::status} is false
    cancel event 
    send "&8» &7Chat jest &cwylaczony"

Dodatkowo, w komendzie brakuje "poprawnego użycia", wystarczy dodać po permission message (przed trigger):

command /chat [<text>]:
    permission: szymus.chat
    permission message: &cNie masz permisji!
    usage: Poprawne uzycie /chat on/off

lub na końcu kodu dodać, wedle uznania

Tak jak ktoś wspomniał, zamiast broadcast zrób send to all players, nie zaspamuj konsoli

i co do zarzutu

16 godzin temu, lie_pan1 napisał:

Moim zdaniem to jest skrypt z tego filmu: https://www.youtube.com/watch?v=lCHSQhn_3B4

nie przypomina w ogóle tego