Clientr.PROBLEM

[xXziroXx]

Originally Posted by zokemon

You don't have to put it in the Control-NPC (You can use onPlayerLogin from any WNPC) but if you use WNPCs, just know that the player doesn't have to have the npc for the function to be called. It's very annoying when there something being done to the player when he logs in and you can't find the reason in the control-npc.

I've seen cases where onPlayerLogin and onPlayerLogout fail to trigger - especially on heavy populated servers. It's better to use onActionPlayerOnline in the Control-NPC.

[xAndrewx]

(clientr.mud_nation == null? "Neutral":clientr.mud_nation);

[Angel_Light]

Originally Posted by xAndrewx

(clientr.mud_nation == null? "Neutral":clientr.mud_nation);

Explain to me how all the operators work in this please.

[xXziroXx]

PHP Code:

clientr.mud_nation = (clientr.mud_nation == null? "Neutral":clientr.mud_nation); 


Is the same like doing:

PHP Code:

if (clientr.mud_nation == null) clientr.mud_nation = "Neutral";
else clientr.mud_nation = clientr.mud_nation; 


[napo_p2p]

Originally Posted by Angel_Light

Explain to me how all the operators work in this please.

Consider

PHP Code:

something = (a > b) ? a : b 


If the condition before the ? is true, then 'a' is assigned to 'something', otherwise 'b' is assigned.

So in this case, xAndrewx's expression is pretty much the equivalent to:

PHP Code:

if (clientr.mud_nation == null) {
  clientr.mud_nation = "Neutral";
}
else {
  clientr.mud_nation = clientr.mud_nation;
} 


(You can see that this isn't the best situation to use a ternary operation because there is no need to set clientr.mud_nation to itself).

EDIT: Ziro beat me to it, but hopefully now you get the point :P.

[xXziroXx]

Originally Posted by napo_p2p

Consider

PHP Code:

something = (a > b) ? a : b 


If the condition before the ? is true, then 'a' is assigned to 'something', otherwise 'b' is assigned.

So in this case, xAndrewx's expression is pretty much the equivalent to:

PHP Code:

if (clientr.mud_nation == null) {
  clientr.mud_nation = "Neutral";
}
else {
  clientr.mud_nation = clientr.mud_nation;
} 


(You can see that this isn't the best situation to use a ternary operation because there is no need to set clientr.mud_nation to itself).

EDIT: Ziro beat me to it, but hopefully now you get the point :P.

Bah, way to steal points from me by explaining everything.

[napo_p2p]

Originally Posted by xXziroXx

Bah, way to steal points from me by explaining everything.

*Gives Ziro back his points*

Seriously though, when I pushed reply your post appeared. I wouldn't have replied if I knew you had already .

[xXziroXx]

Originally Posted by napo_p2p

*Gives Ziro back his points*

Seriously though, when I pushed reply your post appeared. I wouldn't have replied if I knew you had already .

[xAndrewx]

it's an operator. beats an if statement for the same result.

[TalonShriner]

I tried to put in that script that coreys suggested:

PHP Code:

function onActionPlayerOnline() {
  if (clientr.mud_nation == NULL)
    clientr.mud_nation = "Nuetral";
} 


But my NPC server already had the ActionPlayerOnline function and a bunch of other scripts under it. So what I did was take out the "function onActionPlayerOnline() {" part from coreys code and stuck in the rest, but I couldn't get it to work. Any suggestions?

[xXziroXx]

Originally Posted by TalonShriner

I tried to put in that script that coreys suggested:

PHP Code:

function onActionPlayerOnline() {
  if (clientr.mud_nation == NULL)
    clientr.mud_nation = "Nuetral";
} 


But my NPC server already had the ActionPlayerOnline function and a bunch of other scripts under it. So what I did was take out the "function onActionPlayerOnline() {" part from coreys code and stuck in the rest, but I couldn't get it to work. Any suggestions?

Add this some where in your onActionPlayerOnline() function (prefferably at the bottom, since there's a chance you have a script in it that removes/loads your clients strings.. or.. something..):

PHP Code:

 if (clientr.mud_nation == NULL) clientr.mud_nation = "Neutral"; 


[zokemon]

A more simple way to put it:

foo = bool ? a : b;

If bool is true, foo would be assigned a.
If bool is false, foo would be assigned b.

You just then put a condition statement in place of the bool such as:

foo = (a < b) ? a : b;

Because condition statements are bools

[Skyld]

Originally Posted by zokemon

Because condition statements are bools

Because conditional statements produce resulting bools.

[TalonShriner]

Thanks everyone. I got it to work ^_^

Now I have a new problem, but its doesn't have to do with the clientr. flags or w/e. I'm trying to script something so that when this weapons is added to someone, it will also add a bunch of other weapons to the player. For instance, if I add the weapon "-StaffPackage" to the account "Wookieman", then it will also add the weapons "-Jump", "-Staffcontrol" and "Staff/Adder" to "Wookieman". Can anyone help me out here?

[Angel_Light]

PHP Code:

function onCreated()
{

  this.weapons = { "weapons1", "weapons2", ...};

  for ( i = 0; i < this.weapons.size(); i++) {
    findPlayer( player.account).addWeapon( this.weapons[ i]);
  }
}