fxDtsBrick::onPlant porblems

[pitfall]

It dose not work.

Code: [Select]

package atmbrickPackage
{
function fxDtsBrick::onPlant(%brick)
{
if(%brick.getDatablock().brickType == PitroRPATMBrickData)
{
if(PitroRPData.getData(%client.bl_id).valueMoney < mFloor($PitroRP::prices::ATM))
{
messageClient(%client, '', "\c6You need at least \c3$" @ mFloor($PitroRP::prices::ATM) SPC "\c6in order to plant ATM!");

%brick.schedule(0, "delete");

return;
}

messageClient(%client, '', "\c6You have paid \c3$" @ mFloor($PitroRP::prices::ATM) @ "\c6 to plant this ATM.");

PitroRPData.getData(%client.bl_id).valueMoney -= mFloor($PitroRP::prices::ATM);
}
}
};
activatePackage(atmbrickPackage);


[Ad Bot]

[Tom]

Try

Code: [Select]

if(%brick.getDatablock().getName() $= "PitroRPATMBrickData")


[pitfall]

Try

Code: [Select]

if(%brick.getDatablock().getName() $= "PitroRPATMBrickData")


No.

[Amade]

No.

Beggars can't be choosers

[pitfall]

Beggars can't be choosers

No i mean it dose not work. It deletes the brick when i place it.

[Wicked]

Show the brick code.

[pitfall]

Code: [Select]

datablock fxDTSBrickData(PitroRPATMBrickData : brick2x4FData)
{
category = "PitroRP";
subCategory = "PitroRPG Infoblocks";

uiName = "ATM Brick";

PitroRPBrickType = 2;
PitroRPBrickAdmin = false;

triggerDatablock = PitroRPInputTriggerData;
triggerSize = "2 4 1";
trigger = 0;
};

[Wicked]

Code: [Select]

if(%brick.getDatablock().PitroRPBrickType == 2)Or since Iban's CityRPG mod has other bricktypes that = 2, maybe add in

Code: [Select]

PitroRPATMBrickType = 2;and

Code: [Select]

if(%brick.getDatablock().PitroRPATMBrickType == 2)

[pitfall]

Great thanks, it dose not delete its self on plant, but now the bank functions don't work. It still dose not take money when its planted.
Edit:

Code: [Select]

$PitroRP::prices::ATM = 5000;So don't call me stupid.
Also i did NOT in any way mess with the original bank functions.

[M]

hey
hey
where did this %client variable come from, I don't see that being defined anywhere

[mctwist]

First, add a parent to it so it wont break anything else.

Code: [Select]

Parent::onPlant(%brick);Then, get the client that owns the brick. (Not sure if it works if it really isn't planted yet. May be fixed with a scheduled check.)

Code: [Select]

%client = getBrickGroupFromObject(%brick).client;