Something as simple as a Bomb

[Lance]

Only have time for a small comment right now, but please remember that triggeractions are trusted implicitly; you have just created some functions where a malicious person could make any other player pick up or throw your bombs. Don't send the account as a parameter like that.

Also I assume it was just a typo but your onActionPlaced has an unfinished line.

[MrOmega]

Ok, how would I talk to the serverside of a class script? Where do you see a typo in onActionPlaced.

[Lance]

Originally Posted by MrOmega

Ok, how would I talk to the serverside of a class script?

The problem occurs when you are including an arbitrary account as a parameter, so a malicious player could send the wrong account to do some screwy stuff. I guess the simplest way around that would be to stick with what you're doing but use 'actionplayer' serverside to get the account of the person doing the action instead of passing it as a parameter.

Where do you see a typo in onActionPlaced.

I was referring to:

PHP Code:

  temp.pl.client.carry =
  temp.pl.client.freeze = false; 


where your code implies you are trying to do

PHP Code:

  temp.pl.client.carry = temp.pl.client.freeze = false; 


while your formatting implies that these are supposed to be two distinct lines with one being unfinished. I'm not trying to do any stylistic criticism here but it just seemed a bit ambiguous to me.

[MrOmega]

Actionplayer it is.

Oh and that's just my styling, it can be chaotic to others but I can read it fast and good.

[salesman]

your styling annoys me almost as much as people who put spaces before punctuation !

[Fulg0reSama]

Originally Posted by salesman

your styling annoys me almost as much as people who put spaces before punctuation !

zomg ! I'm doing what annoys salesman !!!