Making a package?

[tyler0]

Ok i am doing a little test but it seems there is something wrong. All i want to do is when you plant a brick it sends a message to all players saying "Hey".

Copied this code from CashmodII:

Code: [Select]

package CashPackage
}
function fxDTSbrick::Plant(%brick)
{
%e = Parent::Plant(%brick);
if(!%e)
{
%c = getbrickgroupfromobject(%brick).client;
if(isObject(%c) && isObject(%c.player) && isObject(%c.player.tempbrick))
{
%vol = %c.player.tempbrick.getVolume();
%cost = $Cash::BrickCostPerVolume / 10;
%amt = mFloatLength(%c.cashObj.cash,2);
if(%amt < %cost * %vol)
{
commandtoclient(%c,'centerprint',"\c6You need" SPC CashStr(%cost * %vol,"") SPC "\c6to plant this brick.",2);
%brick.schedule(0,"delete");
} else {
%c.cashObj.mod("Cash",0 - (%cost * %vol));
}
}
}
}
};
ActivatePackage(CashPackage);

This is my server.cs:

Code: [Select]

Package Test
{
function fxDTSbrick::Plant(%brick)
{
%e = Parent::Plant(%brick);
if(!%e)
{
MessageAll('', "Hey");
}
}
};
ActivatePackage(Test);

[Ad Bot]

[Makingblah]

Ok, I'm not that knowledgeable, but shouldn't your server.cs have:

Code: [Select]

exec("./otherfilenamehere.cs");

at the end or something?

[DrenDran]

Ok i am doing a little test but it seems there is something wrong. All i want to do is when you plant a brick it sends a message to all players saying "Hey".

Copied this code from CashmodII:
-snip-

This is my server.cs:

Code: [Select]

Package Test
{
function fxDTSbrick::Plant(%brick)
{
%e = Parent::Plant(%brick);
if(!%e)
{
MessageAll('', "Hey");
}
}
};
ActivatePackage(Test);

Take the %e out completly.
Like this:

Code: [Select]

package test
{
function fxDTSbrick::Plant(%brick)
{
%e = Parent::Plant(%brick);
MessageAll('', "Hey");
}
};
activatepackage(test);


[tyler0]

Take the %e out completly.
Like this:

Code: [Select]

package test
{
function fxDTSbrick::Plant(%brick)
{
%e = Parent::Plant(%brick);
MessageAll('', "Hey");
}
};
activatepackage(test);


Tried it but didn't work... :(

[Crysist]

Well... Now you have no use for %e, also, I think you have to return the parent. Like this

Code: [Select]

package test
{
function fxDTSbrick::Plant(%brick)
{
return Parent::Plant(%brick);
MessageAll('', "Hey");
}
};
activatepackage(test);

[Wicked]

Code: [Select]

Package Test
{
function fxDTSbrick::onPlant(%brick)
{
Parent::onPlant(%brick);

if(isObject(%brick))
{
MessageAll('', "Hey");
}
}
};
ActivatePackage(Test);

[tyler0]

Code: [Select]

Package Test
{
function fxDTSbrick::onPlant(%brick)
{
Parent::onPlant(%brick);

if(isObject(%brick))
{
MessageAll('', "Hey");
}
}
};
ActivatePackage(Test);

nope :(

[-Jetz-]

Is your add-on active in the add-ons list?

[Triple Nickels]

Take the %e out completly.
Like this:

Code: [Select]

package test
{
function fxDTSbrick::Plant(%brick)
{
%e = Parent::Plant(%brick);
MessageAll('', "Hey");
}
};
activatepackage(test);


Please don't post if you don't know what you're doing. This will cause the function to return "Hey" every time it is called. If you're going to take out the if statement you must do this:

Code: [Select]

package test
{
function fxDTSbrick::Plant(%brick)
{
%e = Parent::Plant(%brick);
MessageAll('', "Hey");
return %e;
}
};
activatepackage(test);

The reason I'm making a big deal of it is .plant(); returns a number value for the error, which many mods probably depend on. For example

if(%this.plant() == 3)
   messageClient(%client,'',"That brick's floating!");

Defaultly, if the brick was floating it would return 3, and that check would work. With your code, it would return "Hey". I don't believe "Hey" is equal to three, even if the brick was floating.

Also; Cry:

Also, I think you have to return the parent. Like this

Code: [Select]

package test
{
function fxDTSbrick::Plant(%brick)
{
return Parent::Plant(%brick);
MessageAll('', "Hey");
}
};
activatepackage(test);

return; skips all the rest of the function, so that wouldn't work at all. You're right about returning the parent, but you'd have to put it below everything like I noted earlier.

[lilboarder32]

Why not just modify fxDtsBrick::onPlant()? Is .plant() even overwritable?

[tyler0]

it must be... because the Cashmod script edits if you have the "Cash" to plant a that brick (the volume of the brick).

[otto-san]

the first { is a }