Cargo - Add alternative unloading item from vehicle cargo

[smitt14ua]

When merged this pull request will:

• New button in cargo menu to select place for unloading

YouTube presentation:

Open/close

.

[severgun]

Duplicate?🤔 #8797

[smitt14ua]

Duplicate?🤔 #8797

If player cannot carry unloaded object, it will be placed randomly. My solution is for big (not-carried) objects and prevents collision bugs after unloading.

[johnb432]

What needs to be finished or looked at for this to be pulled?

[smitt14ua]

What needs to be finished or looked at for this to be pulled?

Nothing, I think. commy2 didn't respond me

[LinkIsGrim]

What needs to be finished or looked at for this to be pulled?

General testing, MP, and code review.

[johnb432]

I've taken the liberty to implement some fixes and improvements directly:

• Combined handle functions into a single function FUNC(handleDeployInterrupt).
• Implemented FUNC(handleDeployInterrupt) being called (on opening the interact menu, vehicle and weapon switching, being killed and captured, surrendering, going unconscious and feature cameras changing.)
• Made sure that setDir is called before any position setting commands. Wiki suggests this is necessary and I did not verify this.
• Added authors from functions that were used as basis for the newly written ones. I assume they came from sandbag, as the comments are exactly the same. If I'm wrong, we can easily correct that.
• Removed 1 second delay after placing. This, in my opinion, does not make any sense and I will update sandbag's code accordingly.

I have tested this both in SP and self-hosted MP, works in both cases.

My suggestion is to merge #9617 first (if it's accepted), then this PR could be updated to accommodate the changes.

[smitt14ua]

Thx, @johnb432 ❤️

[LinkIsGrim]

LGTM other than comment above

[LinkIsGrim]

I've removed hiding the object when deployment is blocked since it's less confusing that way IMO.

Tweaked object radius to use firegeometry bounding box as that's closer to the real thing.

You can place objects inside other objects up to a certain point. If we want to prevent that, the intersection checks need to be changed to lineIntersectsSurfaces and begPos needs to be moved further along the line between eye/vehiclePos and the model's position. Guesstimation says half of the object radius should be good.

[johnb432]

If we want to prevent that, the intersection checks need to be changed to lineIntersectsSurfaces and begPos needs to be moved further along the line between eye/vehiclePos and the model's position. Guesstimation says half of the object radius should be good.

You'll need to run that by me again chief, I don't get it.

[LinkIsGrim]

Math

[johnb432]

I went ahead and simplified the deployment method, by essentially copying what carrying does. I feel it's more robust and allows for greater flexibility.
We can improve the deployment method later, but imo it's better if we use something simple for now.

[LinkIsGrim]

LGTM

ArmA.3_2024.02.07-08.25.mp4