Help with Confusing Syntax Error!

[ScratchTehEPICSpaceDude]

Please help me with this syntax error! It's very confusing.

[Ad Bot]

[otto-san]

you forgot the closing " in the last messageClient parameter

[ScratchTehEPICSpaceDude]

Thank you so much!

[Treynolds416]

Don't lock any more of your threads

[jes00]

You know you could just not parent it and give them a message if they are not an admin instead of destroying the brick.

[Port]

This code is such a horrible mess which wouldn't even work if the missing " issue was solved. Here, let me point out a few of the flaws:

package NoBaseplates
{
function FxBTSBrick::OnPlant(%client, %this)
{
parent::OnPlant(%client, %this);
   %brick = %this; // Why would you even do this? The brick is available from %this, no need for an extra variable.
   if(%client.isAdmin)
      return;
       if(!(%brick.getType() & $TypeMasks::FxBrickAlwaysObjectType))
         if(!%brick.isPlanted) // What the hell is the point of the this? The brick will never not brick, and will never not placed. In fact, this will cause it to never define %brickData unless somebody is running this function on a namespace basis.

         %brickData = %brick.getDataBlock().getID();
         if(%brickData.category $= "Baseplates")
            {
               %brick.killBrick();
          MessageClient(%this, '', "\c6Non-admins aren't allowed to plant any bricks in the baseplate section.");
          }
}
};
ActivatePackage(NoBaseplates);

Here are some hints:

• The arguments for the fxDTSBrickData::onPlant callback are the brick datablock, %this, and the brick object, %obj.
• You can access the client of a brick by getting it's brick group using getBrickGroupFromObject( object ) and accessing it's .client field.
• The previously mentioned function is called when the brick is planted. There is no need to check whether it's planted.

[Greek2me]

Port is correct, aside from this:

• The previously mentioned function is called when the brick is planted. There is no need to check whether it's planted.

The fxDtsBrick::onPlant function is also called when a temp brick is placed. %brick.isPlanted checks whether the brick is a temp brick or not, so it is in fact something that should be included.

[Port]

The fxDtsBrick::onPlant function is also called when a temp brick is placed. %brick.isPlanted checks whether the brick is a temp brick or not, so it is in fact something that should be included.

What are you talking about? fxDTSBrick::plant() only calls fxDTSBrick::onPlant() upon successfully placing the brick. Try it yourself.

package test { function fxDTSBrick::onPlant( %this ) { announce ( "onPlant( " @ %this @ " )" ); parent::onPlant( %this ); } };
activatePackage( "test" );

Evaluate those two lines in the console and place a brick.

[ScratchTehEPICSpaceDude]

It seems to work.

[ScratchTehEPICSpaceDude]

Anyway, I don't know anything about this. Adam Savage coded this.

[Lugnut]

Anyway, I don't know anything about this. Adam Savage coded this.

Then why isn't he getting off his lazy ass and asking the question himself?

[ScratchTehEPICSpaceDude]

Then why isn't he getting off his lazy ass and asking the question himself?

I'm not sure his "lazy ass" is on the forums.

[otto-san]

I'm not sure his "lazy ass" is on the forums.

It is.

[Treynolds416]

I'm not sure his "lazy ass" is on the forums.

You are him.

?

[ScratchTehEPICSpaceDude]

This code is such a horrible mess which wouldn't even work if the missing " issue was solved. Here, let me point out a few of the flaws:

Here are some hints:

• The arguments for the fxDTSBrickData::onPlant callback are the brick datablock, %this, and the brick object, %obj.
• You can access the client of a brick by getting it's brick group using getBrickGroupFromObject( object ) and accessing it's .client field.
• The previously mentioned function is called when the brick is planted. There is no need to check whether it's planted.

Omg, you're right. It doesn't work. Sorry, can you send me a fixed server.cs? I'm not a good coder.