Author Topic: [solved] using initcontainerboxsearch for lots in CityRPG (Read 2245 times)

Aoki� · August 17, 2014, 05:21:39 AM

This is supposed to change the ownership of the lot and all bricks on it, but atm it's doing nothing with the bricks on the lot; can someone tell me what I did wrong here?
solution:

Code: [Select]

function fxDTSBrick::transferLot(%brick, %targetBG)
{
	%ownerBG = getBrickGroupFromObject(%brick);

	if(isObject(%ownerBG))
		%ownerBG.remove(%brick);
	if(isObject(%targetBG))
		%targetBG.add(%brick);

	%boxSize = (getWord(%brick.trigger.scale, 0) / 2.5) * 10 SPC (getWord(%brick.trigger.scale, 1) / 2.5) * 10 SPC (getWord(%brick.trigger.scale, 2) / 2.5) * 10;
	initContainerBoxSearch(%brick.trigger.getWorldBoxCenter(), %boxSize, $typeMasks::fxBrickObjectType);

	while(isObject(%nBrick = containerSearchNext()))
	{
		if(!isObject(%nBrick) || !%nBrick.isPlanted)
			continue;

		%ownerBG = getBrickGroupFromObject(%nBrick);

		if(isObject(%ownerBG))
			%ownerBG.remove(%nBrick);

		if(isObject(%targetBG))
			%targetBG.add(%nBrick);
	}
}

Ad Bot · Advertisement

Cruxeis · August 17, 2014, 05:50:34 AM

Personally, I would use brick stacks, just like the duplorcator did. Try looking at that.

jes00 · August 17, 2014, 05:51:48 AM

You also have to change %brick.client and %brick.stackBL_ID.

Aoki� · August 17, 2014, 06:13:24 AM

hm for some reason the box size was .1 of what it was supposed to be so i fixed it by multiplying everything here by 10

Code: [Select]

%boxSize = (getWord(%brick.trigger.scale, 0) / 2.5) * 10 SPC (getWord(%brick.trigger.scale, 1) / 2.5) * 10 SPC (getWord(%brick.trigger.scale, 2) / 2.5) * 10;

i copied the code right from another place in the mod, i wonder if this was intentional or a bug

jes00 · August 17, 2014, 06:25:57 AM

Quote from: Aoki� on August 17, 2014, 06:13:24 AM

hm for some reason the box size was .1 of what it was supposed to be so i fixed it by multiplying everything here by 10
Code: [Select]
%boxSize = (getWord(%brick.trigger.scale, 0) / 2.5) * 10 SPC (getWord(%brick.trigger.scale, 1) / 2.5) * 10 SPC (getWord(%brick.trigger.scale, 2) / 2.5) * 10;i copied the code right from another place in the mod, i wonder if this was intentional or a bug

You could just not use a box search or raycast at all and still get all the bricks on top of it. Just use a for/while statement and %brick.getNumUpBricks(); + %brick.getUpBrick(%num);

Aoki� · August 17, 2014, 06:36:16 AM

Quote from: jes00 on August 17, 2014, 06:25:57 AM

You could just not use a box search or raycast at all and still get all the bricks on top of it. Just use a for/while statement and %brick.getNumUpBricks(); + %brick.getUpBrick(%num);

hanging bricks are left out

Zeblote · August 17, 2014, 07:39:05 AM

Quote from: Aoki� on August 17, 2014, 06:36:16 AM

hanging bricks are left out

Then use getNumDownBricks() and getDownBrick(num) too

Aoki� · August 17, 2014, 08:18:28 AM

Quote from: Zeblote on August 17, 2014, 07:39:05 AM

Then use getNumDownBricks() and getDownBrick(num) too

I would rather use initContainerBoxSearch because even with getDownBrick there could be some trouble with zones/floating bricks, besides I got it working already.