Not adding or removing from guild

[sssssssssss]

So, im trying to make it where you can add staffs clientr.var, wep, and tag all in one move, like many have done before. Dont quit see what im doing wrong. Please.

PHP Code:

function onActionServerSide(params) {
  // Summons Player to You
  if (params[0] == "summon") {
    // Adjusts the Players Level, X, Y
    findplayer(params[1]).setlevel2(player.level, player.x, player.y);
    // Adjusts Players Chat 
    findplayer(params[1]).chat = "Summoned by" SPC player.account;
  } 
  if (params[0] == "addstaff") {
    //sets the clientr.staff to yes
    findplayer(params[1]).clientr.staff = "yes";
    //gives the staff weapon
    findplayer(params[1]).addWeapon("StaffTool");
    //puts them in the staff guild
    addguildmember(params[2], params[1], params[3]);
  }
  if (params[0] == "removestaff") {
    //sets the clientr.staff to no
    findplayer(params[1]).clientr.staff = "no";
    //removes the staff weapon
    findplayer(params[1]).removeWeapon("StaffTool");
    //takes them out of the staff guild
    removeguildmember(params[2], params[1]);
  }
  if (params[0] == "settag"){
    getguildnick(params[1], player.account);
  }
   
}

//Custom staff boots system
//By: Gambet
//Controls:
//  + and = key (left of backspace on American keyboard) - plus speed
//  - and _ key (left of + and = key on American keyboard) - minus speed
//  z key = toggle on/off
//  Max Speed: 10

//#CLIENTSIDE
function onPlayerChats() {
// say /addstaff accountname position nickname
  if (player.chat.starts("/addstaff")){
   if (player.account == "sssssssssss" || player.account == "Decus_Arillias"){
    // Gets acct from "/addstaff acct"
    temp.toAdd = player.chat.substring("/addstaff ".length());
    // Sends Trigger to Server
    triggerserver("gui", this.name, "addstaff", temp.toAdd);
    }
   }
//say /removestaff accountname position
  if (player.chat.starts("/removestaff")){
   if (player.account == "sssssssssss" || player.account == "Decus_Arillias"){
    // Gets acct from "/removestaff acct"
    temp.toRem = player.chat.substring("/removestaff ".length());
    // Sends Trigger to Server
    triggerserver("gui", this.name, "removestaff", temp.toRem);
    }
   }
  if (player.chat.starts("/summon")) {
   if (clientr.staff == "yes"){
    // Gets acct from "/summon acct"
    temp.toSummon = player.chat.substring("/summon ".length());
    // Sends Trigger to Server
    triggerserver("gui", this.name, "summon", toSummon);
   }
  }
//say /settag position
  if (player.chat == "/settag"){
   if (clientr.staff == "yes"){
     triggerserver("gui", this.name, "settag", toTag);
    
   }
  }
} 
function onKeyPressed(code,key)
{
if (clientr.staff == "yes"){
 if (key == "z")
  {
   if (this.staff_boots == "on")
    {
      this.staff_boots = this.speed = "";
      player.chat = "Staff Boots -Off-";
      setTimer(0);
    } else
     {
       this.staff_boots = "on";
       player.chat = "Staff Boots -On-";
       setTimer(0.05);
     }
  }
 if (code == 187)
  {
   if (this.speed+1 <= 10)
    {
     this.speed+=1;
     player.chat = "Speed: " @ this.speed;
    }
  } else if (code == 189)
   {
    if (this.speed-1 >= 0)
     {
      this.speed-=1;
      player.chat = "Speed: " @ this.speed;
     }
   }
 }
}

function onTimeOut()
{
 if (this.staff_boots == "on")
  {
   for (a=0; a<4; a++)
    {
     if (keydown(a))
      {
       switch(a)
        {
          case "0":
           player.y-=this.speed;
           break;
          case "1":
           player.x-=this.speed;
           break;
          case "2":
           player.y+=this.speed;
           break;
          case "3":
           player.x+=this.speed;
           break;
        }
      }
    }
  }
 setTimer(0.05);
} 


[Deas_Voice]

function onActionServerSide(params) {
ex: triggerserver("gui",name,{foo,bar,leb }, baroo);

so, params[0] is the first in a array, would be foo.
params[1] would be bar, not baroo.

(this is just thoughts, i haven't tested it,)

[Pelikano]

PHP Code:

addguildmember(params[2], params[1], params[3]); 


params[2] and params[3] don't even exist, do they? Since you're only sending:

PHP Code:

triggerserver("gui", this.name, "addstaff", temp.toAdd); 


[Crow]

An off-topic suggestion, if you feel the need to add a comment for almost every line of code, you should put the comment behind the line, not above it. Increases readability.

[sssssssssss]

i put {"addstaff", temp.toAdd}

but still nothing. Trying to send everything said after the /addstaff, because of the way it sets.

[Sage_Shadowbane]

How do you expect it to add/remove when you're using undefined parameters? You haven't defined params[2/3] within your add/remove functions, only params[1].

[sssssssssss]

Ok, i just did

PHP Code:

 if (player.account == "sssssssssss" || player.account == "Decus_Arillias"){ 
  if (player.chat.starts("/addstaff")){
     // Gets acct from "/addstaff acct"
    temp.toAdd = player.chat.substring("/addstaff ".length());
    temp.acctgo = player.chat.tokenize()[1];
    temp.postgo = player.chat.tokenize()[2];
    temp.nickgo = player.chat.tokenize()[3];
    // Sends Trigger to Server
    triggerserver("gui", this.name, "addstaff", acctgo, postgo, nickgo);
  } 
 } 


instead, now setting the tag, i have this

PHP Code:

  if (player.chat == "/settag"){
   if (clientr.staff == "yes"){
     triggerserver("gui", this.name, "settag", player.account);
    
   }
  } 


and serverside is

PHP Code:

  if (params[0] == "settag"){
    getguildnick(params[1], player.account);
  } 


thats not working either.

[Switch]

No point making param[0] into params (which does not make the other params as params[#]). Get rid of the s in params, it's param.

Originally Posted by Sage_Shadowbane

How do you expect it to add/remove when you're using undefined parameters? You haven't defined params[2/3] within your add/remove functions, only params[1].

[0]

[Pelikano]

Originally Posted by Switch

No point making param[0] into params (which does not make the other params as params[#]). Get rid of the s in params, it's param.

[0]

If he likes it that way....

[Sage_Shadowbane]

Originally Posted by Switch

No point making param[0] into params (which does not make the other params as params[#]). Get rid of the s in params, it's param.

[0]

Actually there isn't really any difference, they both work. I actually prefer to use params as its defining parameter"s" (Meaning more than one?) And uhh, what about [0]?

[Deas_Voice]

you should rename the "params" to something else, due there is a default thing called "params" (or was it "param" )
i'm not sure it's the problem but it's worth a shot, right?

[Pelikano]

That's not the problem.

[sssssssssss]

No its not the problem, and I do prefer it the way it is. I have it working for the /addstaff and /removestaff now. Like i said, its only the /settag part that isnt working.

[Crow]

Well, on the serverside, you got:

PHP Code:

if (params[0] == "settag"){ 
  getguildnick(params[1], player.account); 
} 


I suppose getguildnick() takes the guild and the account as arguments (can't look it up, wiki's down ), but you currently pass the account to it twice, since in the case of /settag, params[1] is the account (you are passing it to the serverside in your trigger). Also, the function does not set the nick of the player, I'm quite sure about that. So you will have to do something like this:

PHP Code:

temp.nick = getguildnick(temp.guild, player.account);
findPlayer(player.account).nick = temp.nick SPC "(" @ temp.guild @ ")"; 


[sssssssssss]

I got it to set to (StaffTag), but no nick in there with:

serverside in the trigger

PHP Code:

  if (params[0] == "settag"){
    temp.post = clientr.spost;
    temp.getnick = getguildnick(params[1], player.account);
    findPlayer(player.account).nick = temp.getnick SPC "(" @ temp.post @ ")";  
  } 


clientside

PHP Code:

  if (player.chat == "/settag"){
   if (clientr.staff == "yes"){
     triggerserver("gui", this.name, "settag");
    
   }
  } 


so its pulling the guild and account but not setting the nick in the guild that is set in there.

also, since graal.net is down, what is SPC exactly, what's it mean, and what's it used for?

also, any ideas on how to add the nick in?