Safemode - add public setWeaponSafety function

[AndreasBrostrom]

When merged this pull request will:

• Added safeWeapon function allowing the weapon to be set on safe if it is not.

A tiny function i use to set people's weapons on safe at spawn and respawn. So here is a little donation.

[Dystopian]

maybe this function should get true/false parameter and lock/unlock accordingly?

[AndreasBrostrom]

Like that idea two
I make sure to add that to the function as well.

[AndreasBrostrom]

Note: Not tested cause I'm on traveling foot.

Can be checked code wise anyways. if the solution for the bool param is agreeable.

[AndreasBrostrom]

All done

[commy2]

• Should the function be named setWeaponSafety?
• There is no reason for the user not to set muzzle to currentMuzzle _unit, therefore I think this should not be argument of the function, but just a variable set to currentMuzzle _unit.

[AndreasBrostrom]

• Should the function be named setWeaponSafety?

Yes agree.

• There is no reason for the user not to set muzzle to currentMuzzle _unit, therefore I think this should not be argument of the function, but just a variable set to currentMuzzle _unit.

Agree with this two.
Added it cause it was in the other function.

[commy2]

For the internal function it is used to switch back to the last muzzle. It is needed, because you can turn off safety with F. Otherwise it would switch to the primary muzzle from the secondary muzzle for example in addition to unlocking the weapon, which feels weird. Not needed for this public function though.

[commy2]

_weapon should be kept as argument. _muzzle cannot be converted to config case like this, and it is not needed, because currentMuzzle reports config case already.

[AndreasBrostrom]

Test in progress

[AndreasBrostrom]

Tested and did some final fixes to the script.

Testprotocol:

• Line used [player, currentWeapon player, true] call ace_safemode_fnc_setWeaponSafety & [player, currentWeapon player, false] call ace_safemode_fnc_setWeaponSafety
• Tested manually applied function via debug menu
• Tested trigger set and unset safemode.

[veteran29]

If this function is used for something like eg. "safestart" (locking the weapon on game start) and the unit does not have weapon it will try to lock "empty" weapon.

Maybe it would be wise to add:
if (_weapon == "") exitWith {};

Unsafing the "empty" weapon results in an error .