Fixed airdrops sink in the ground

[shukari]

When merged this pull request will:

• fix helicopter airdrop bugs out of the world #6237

[commy2]

Wouldn't this mean the cargo is stuck inside the parachute model?

[shukari]

Yes... look here -> https://youtu.be/IYS29dWDe5s?t=1m20s

But the problem is... the driangle thing is the bottom of the parachute... and because the supply is attached is has no physics so... the parent model must be the bottom...

It now works for every supply box!

[dedmen]

It no works for every supply box!

How about only doing it for the boxes that actually don't have physx and need this "fix"?

[shukari]

The problem is the attachTo to the parachute.... then the supplybox is there but has no collision. So then the parachute then hit the ground and the box is in the ground if it is not attached a littlebit over the bottom of the parachute.

It no works for every supply box!

That was a fail... i mean NOW it works for everything.

Vanilla is doing the same... if you place a supplybox in zeus and then change the height to 100m (with ALT) it will automatic attach a parachute to it with attachTo [0,0,1] ^^

[Cuel]

Seems logical to attach it above the object instead of under

[kymckay]

I can confirm that this is how curator does it (and it works):

case "Object": {
	switch _objectType do {
		case "AmmoBox": {
			//--- Create parachute for vehicles in the air
			if ((position _object select 2) > 20 && alive _object) then {
				_para = createvehicle ["B_Parachute_02_F",_objectPos,[],0,"none"];
				_object attachto [_para,[0,0,1]];
			};
		};
	};
};

[kymckay]

Question though, why do we not just create the parachute where the object is? (Instead of at [0,0,0] then moving it)

[shukari]

Question though, why do we not just create the parachute where the object is? (Instead of at [0,0,0] then moving it)

I dont understand what you mean... with this PR it works for all container... do you mean dynamically?

[kymckay]

Not specific to this PR, I was just looking at the code we have above these changes:

    private _parachute = createVehicle ["B_Parachute_02_F", [0,0,0], [], 0, "CAN_COLLIDE"];
 
     // cannot use setPos on parachutes without them closing down
     _parachute attachTo [_item, [0,0,0]];
     detach _parachute;
 
     private _velocity = velocity _item;
 
    _item attachTo [_parachute, [0,0,1]];
     _parachute setVelocity _velocity;

[PabstMirror]

lower straps look a little funny but functionality over form

[Bosenator]

Before this fix, my solution was the following:

if (isServer) then {
	["ace_cargoUnloaded", {
		params ["_item", "_vehicle", ["_type", ""]];

		if (_type == "paradrop") then {
			[_item] spawn {
				params ["_crate"];
				
				if (isNull _crate) exitWith {};				
				waitUntil {(getPosVisual _crate select 2) < 1};
				_groundPosition = getPos _crate;
				detach _crate;
				_crate setPos ([_groundPosition select 0, _groundPosition select 1, 0]);
			};
		};
	}] call CBA_fnc_addEventHandler;
};

It's messy by all means, but perhaps the logic could be applied?