Admin Message System

[Gunderak]

I know its fairly simple, but still useful.
Any feedback is appriciated.
PS: for players to broadcast there account needs to be in this.allowed.

PHP Code:

function onActionServerSide() {
  if (params[0] == "send") {
    for (temp.pl: allplayers) {
      temp.pl.triggerclient(this.name, "recieved", params[1]);
    }
  }
}

//#CLIENTSIDE
function onCreated() {
  this.allowed = {
    "Graal780374"
  };
}

function ChatBar.onAction() {
  if (ChatBar.text.starts("/adminmessage")) {
    if (player.account in this.allowed) {
      triggerserver("weapon", this.name, "send", ChatBar.text.substring(14));
      shared.chat("");
    }
  }
}

function onActionclientside() {
  if (params[0] == "recieved") {
    shared.adminMessage("<center><b><font size=15>Broadcast Message</b></font><br><font size=14>" @ params[1]);
  }
} 


[Twinny]

That account check really should be serverside...

[Gunderak]

Yeah so that haxors cant change it using memory editors, forgot to do that.

[cbk1994]

Besides the serverside check, onActionclientside should be onActionClientSide and recieved should be spelled correctly (received). Those are the only issues I see, nice work.

[Gunderak]

LOL i mispelt received. oh how dumb i feel.
and does the capitalization of onActionClientSide actually affect anything?

[cbk1994]

Originally Posted by Gunderak

LOL i mispelt received. oh how dumb i feel.
and does the capitalization of onActionClientSide actually affect anything?

No, it's only for consistency.