Fix Cancel (RMB) Action

[jonpas]

When merged this pull request will:

• FIxes "Cancel" action which is right mouse button when placing things

Untested (no access to Arma-capable machine at the moment)

[jonpas]

Tested, seems to work fine. Tagging @commy2 for review.

[commy2]

Okay this works as far as I can tell.
But.
We should add simple sanity checks after the != -1 check

previously these actions would only be called when the current action was active (e.g. carrying a ladder), but now all of them are executed when you RMB every time.

e.g.:
_magClassname in (magazines _unit)

[PabstMirror]

We don't want to remove this entire if statement, just the RMB part.
All of the other checks need to be run each frame in the PFEH so it will cancel if any of them happen.
e.g. if canInteractWith is false, we need to cancel right away instead of waiting for RMB

[PabstMirror]

I wonder if we could simplify the event handler code to a single common
EFUNC(common,rmbPressed) = true;

and then in all our placement blocks we do

EFUNC(common,rmbPressed) = false
PFEH: if (EFUNC(common,rmbPressed)) then {cancle};

[commy2]

there is no RMB release event though.

[nicolasbadano]

I still think installing only one of this EH and launching an ACE event on RMB would be more efficient and easier to use.

[jonpas]

@esteldunedain I believe this is supposed to be solved with CBA_fnc_addDisplayHandler once new CBA is released. @commy2 can confirm?

[commy2]

Yes.