Cookoff - Mini-Rewrite

addons/cookoff/ACE_Settings.hpp

@@ -1,12 +1,11 @@
-
 class ACE_Settings {
     class GVAR(enable) {
         movedToSqf = 1;
     };
     class GVAR(enableAmmobox) {
         movedToSQF = 1;
     };
-    class GVAR(enableAmmoCookoff) { // For CBA Setting Switch: we can eliminate and just use (ammoCookoffDuration == 0)
+    class GVAR(enableAmmoCookoff) {
         movedToSQF = 1;
     };
     class GVAR(ammoCookoffDuration) {

addons/cookoff/CfgCloudlets.hpp

@@ -1,4 +1,3 @@
-
 class CfgCloudlets {
     class GVAR(CookOff) {
         interval = 0.004;

addons/cookoff/CfgEden.hpp

@@ -1,4 +1,3 @@
-
 class Cfg3DEN {
     class Object {
         class AttributeCategories {
@@ -7,9 +6,9 @@ class Cfg3DEN {
                     class GVAR(enable) {
                         property = QGVAR(enable);
                         control = "Checkbox";
-                        displayName = CSTRING(enable_hd_name);
-                        tooltip = CSTRING(enable_hd_tooltip);
-                        expression = QUOTE(if !(_value) then {_this setVariable [ARR_3('%s',_value,true)];};);
+                        displayName = CSTRING(enable_name);
+                        tooltip = CSTRING(enable_tooltip);
+                        expression = QUOTE(if (!_value) then {_this setVariable [ARR_3('%s',_value,true)]});
                         typeName = "BOOL";
                         condition = "objectVehicle";
                         defaultValue = QUOTE((GETMVAR(QGVAR(enable),0)) in [ARR_2(1,2)]);
@@ -19,10 +18,10 @@ class Cfg3DEN {
                         control = "Checkbox";
                         displayName = CSTRING(enableAmmoCookoff_name);
                         tooltip = CSTRING(enableAmmoCookoff_tooltip);
-                        expression = QUOTE(if !(_value) then {_this setVariable [ARR_3('%s',_value,true)];};);
+                        expression = QUOTE(if (!_value) then {_this setVariable [ARR_3('%s',_value,true)]});
                         typeName = "BOOL";
                         condition = "objectHasInventoryCargo";
-                        defaultValue = QUOTE(if (_this isKindOf 'ReammoBox_F') then { GETMVAR(QGVAR(enableAmmobox),true) } else { GETMVAR(QGVAR(enableAmmoCookoff),true) };);
+                        defaultValue = QUOTE(if (_this isKindOf 'ReammoBox_F') then {GETMVAR(QGVAR(enableAmmobox),true)} else {GETMVAR(QGVAR(enableAmmoCookoff),true)});
                     };
                 };
             };

addons/cookoff/CfgEventHandlers.hpp

@@ -1,4 +1,3 @@
-
 class Extended_PreStart_EventHandlers {
     class ADDON {
         init = QUOTE(call COMPILE_SCRIPT(XEH_preStart));

addons/cookoff/CfgSFX.hpp

@@ -1,4 +1,3 @@
-
 class CfgSFX {
     class GVAR(CookOff_low) {
         name = QGVAR(cookoff_low);

addons/cookoff/CfgVehicles.hpp

@@ -1,4 +1,3 @@
-
 class CfgVehicles {
     class Sound;
     class GVAR(Sound_low): Sound {
@@ -7,7 +6,6 @@ class CfgVehicles {
         scope = 1;
         sound = QGVAR(CookOff_low);
     };
-
     class GVAR(Sound_mid): GVAR(Sound_low) {
         sound = QGVAR(CookOff_mid);
     };
@@ -17,47 +15,14 @@ class CfgVehicles {
 
     class Tank;
     class Tank_F: Tank {
-        GVAR(ammoLocation) = "HitHull";
         GVAR(cookoffSelections)[] = {"poklop_gunner","poklop_commander"};
-        GVAR(probability) = 0.5;
-    };
-    class MBT_02_base_F: Tank_F {
-        GVAR(ammoLocation) = "HitTurret";
     };
 
     class Car_F;
     class Wheeled_APC_F: Car_F {
-        GVAR(ammoLocation) = "HitHull";
         GVAR(cookoffSelections)[] = {"poklop_gunner","poklop_commander"};
-        GVAR(probability) = 0.8;
 
-        // big explosions for wheeled APCs (same as for tanks)
+        // Big explosions for wheeled APCs (same as for tanks)
         explosionEffect = "FuelExplosionBig";
     };
-
-
-    class MRAP_01_base_F: Car_F {
-        GVAR(engineSmokeOffset)[] = {0,-2,0};
-    };
-
-    class MRAP_02_base_F: Car_F {
-        GVAR(engineSmokeOffset)[] = {0,-2,0};
-    };
-
-    class MRAP_03_base_F: Car_F {
-        GVAR(engineSmokeOffset)[] = {0,-2,0};
-    };
-
-    class Quadbike_01_base_F: Car_F {
-        GVAR(engineSmokeOffset)[] = {0,1,0};
-    };
-
-    class Truck_F;
-    class Truck_02_base_F: Truck_F {
-        GVAR(engineSmokeOffset)[] = {0,-2.6,-0.1};
-    };
-
-    class Truck_02_MRL_base_F: Truck_02_base_F {
-        GVAR(engineSmokeOffset)[] = {0,0.3,-0.1};
-    };
 };

addons/cookoff/XEH_PREP.hpp

@@ -1,10 +1,11 @@
-
-PREP(handleDamageBox);
-PREP(engineFire);
 PREP(cookOff);
-PREP(smoke);
-PREP(cookOffEffect);
 PREP(cookOffBox);
+PREP(cookOffBoxLocal);
+PREP(cookOffEffect);
 PREP(detonateAmmunition);
+PREP(engineFire);
+PREP(engineFireLocal);
 PREP(getVehicleAmmo);
+PREP(handleDamageBox);
 PREP(isMagazineFlare);
+PREP(smoke);

addons/cookoff/XEH_postInit.sqf

@@ -1,63 +1,69 @@
 #include "script_component.hpp"
 
-[QGVAR(engineFire), FUNC(engineFire)] call CBA_fnc_addEventHandler;
-[QGVAR(cookOff), {
-    params ["_vehicle"];
-    if (local _vehicle) then {
-        _this call FUNC(cookOff);
-    };
-}] call CBA_fnc_addEventHandler;
-[QGVAR(cookOffEffect), FUNC(cookOffEffect)] call CBA_fnc_addEventHandler;
-[QGVAR(smoke), FUNC(smoke)] call CBA_fnc_addEventHandler;
-[QGVAR(cookOffBox), FUNC(cookOffBox)] call CBA_fnc_addEventHandler;
+[QGVAR(cookOffBoxLocal), LINKFUNC(cookOffBoxLocal)] call CBA_fnc_addEventHandler;
+[QGVAR(cookOffEffect), LINKFUNC(cookOffEffect)] call CBA_fnc_addEventHandler;
+[QGVAR(engineFireLocal), LINKFUNC(engineFireLocal)] call CBA_fnc_addEventHandler;
+[QGVAR(smoke), LINKFUNC(smoke)] call CBA_fnc_addEventHandler;
+
+if (isServer) then {
+    [QGVAR(cookOff), LINKFUNC(cookOff)] call CBA_fnc_addEventHandler;
+    [QGVAR(cookOffBox), LINKFUNC(cookOffBox)] call CBA_fnc_addEventHandler;
+    [QGVAR(engineFire), LINKFUNC(engineFire)] call CBA_fnc_addEventHandler;
+    [QGVAR(detonateAmmunition), LINKFUNC(detonateAmmunition)] call CBA_fnc_addEventHandler;
+};
 
-// handle cleaning up effects when vehicle is deleted mid-cookoff
+// Handle cleaning up effects when vehicle is deleted mid cook-off
 [QGVAR(addCleanupHandlers), {
-    params ["_vehicle"];
+    params ["_object"];
 
-    // Don't add a new EH if cookoff is run multiple times
-    if ((_vehicle getVariable [QGVAR(deletedEH), -1]) == -1) then {
-        private _deletedEH = _vehicle addEventHandler ["Deleted", {
-            params ["_vehicle"];
+    // Don't add a new EH if cook-off is run multiple times
+    if (!isNil {_object getVariable QGVAR(deletedEH)}) exitWith {};
 
-            [QGVAR(cleanupEffects), [_vehicle]] call CBA_fnc_localEvent;
-        }];
-
-        _vehicle setVariable [QGVAR(deletedEH), _deletedEH];
-    };
+    _object setVariable [QGVAR(deletedEH),
+        _object addEventHandler ["Deleted", {
+            [QGVAR(cleanupEffects), _this select 0] call CBA_fnc_localEvent;
+        }]
+    ];
 }] call CBA_fnc_addEventHandler;
 
 [QGVAR(cleanupEffects), {
-    params ["_vehicle", ["_effects", []]];
+    params ["_object"];
+
+    if (isServer) then {
+        // Reset, so that the object can cook-off again
+        _object setVariable [QGVAR(isCookingOff), nil, true];
 
-    _effects = _effects + (_vehicle getVariable [QGVAR(effects), []]);
-    if (_effects isNotEqualTo []) then {
-         { deleteVehicle _x } count _effects;
+        // Remove effects from JIP
+        {
+            _x call CBA_fnc_removeGlobalEventJIP;
+        } forEach (_object getVariable [QGVAR(jipIDs), []]);
+
+        _object setVariable [QGVAR(jipIDs), nil];
     };
+
+    // All effects are local (apart from sound, which is global, but is handled by server)
+    {
+        deleteVehicle _x;
+    } forEach (_object getVariable [QGVAR(effects), []]);
+
+    _object setVariable [QGVAR(effects), nil];
 }] call CBA_fnc_addEventHandler;
 
 ["ReammoBox_F", "init", {
-    (_this select 0) addEventHandler ["HandleDamage", {
-        if ((_this select 0) getVariable [QGVAR(enableAmmoCookoff), GVAR(enableAmmobox)]) then {
-            _this call FUNC(handleDamageBox);
-        };
-    }];
+    // Calling this function inside curly brackets allows the usage of "exitWith", which would be broken with "HandleDamage" otherwise
+    (_this select 0) addEventHandler ["HandleDamage", {_this call FUNC(handleDamageBox)}];
 }, nil, nil, true] call CBA_fnc_addClassEventHandler;
 
-// secondary explosions
-["AllVehicles", "killed", {
+// Secondary explosions
+["AllVehicles", "Killed", {
+    if (!GVAR(enableAmmoCookoff) || {GVAR(ammoCookoffDuration) == 0}) exitWith {};
+
     params ["_vehicle", "", "", "_useEffects"];
-    if (
-        _useEffects &&
-        _vehicle getVariable [QGVAR(enableAmmoCookoff), GVAR(enableAmmoCookoff)]
-    ) then {
-        if (GVAR(ammoCookoffDuration) == 0) exitWith {};
-        ([_vehicle] call FUNC(getVehicleAmmo)) params ["_mags", "_total"];
-
-        private _delay = (random MAX_AMMO_DETONATION_START_DELAY) max MIN_AMMO_DETONATION_START_DELAY;
-        [FUNC(detonateAmmunition), [_vehicle, _mags, _total], _delay] call CBA_fnc_waitAndExecute;
+
+    if (_useEffects && {_vehicle getVariable [QGVAR(enableAmmoCookoff), true]}) then {
+        [QGVAR(detonateAmmunition), [_vehicle, false, objNull, objNull, (random MAX_AMMO_DETONATION_START_DELAY) max MIN_AMMO_DETONATION_START_DELAY]] call CBA_fnc_serverEvent;
     };
-}, nil, ["Man","StaticWeapon"]] call CBA_fnc_addClassEventHandler;
+}, nil, ["CAManBase", "StaticWeapon"]] call CBA_fnc_addClassEventHandler;
 
 if (hasInterface) then {
     // Plays a sound locally, so that different sounds can be used for various distances
@@ -68,7 +74,7 @@ if (hasInterface) then {
 
         private _distance = _object distance (positionCameraToWorld [0, 0, 0]);
 
-        TRACE_3("",_object,_sound,_maxDistance);
+        TRACE_2("",_object,_sound);
 
         // 3 classes of distances: close, mid and far, each having different sound files
         private _classDistance = switch (true) do {