fixed bug 603-4

KillahPotatoes · May 12, 2019 · 8f195ab · 8f195ab
1 parent fca8db5
commit 8f195ab
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 14 deletions.
diff --git a/Missionframework/modules/11_permission/fnc/fn_permission_checkVehiclePermission.sqf b/Missionframework/modules/11_permission/fnc/fn_permission_checkVehiclePermission.sqf
@@ -5,7 +5,7 @@
     File: fn_permission_checkVehiclePermission.sqf
     Author: KP Liberation Dev Team - https://github.com/KillahPotatoes
     Date: 2018-12-21
-    Last Update: 2019-05-04
+    Last Update: 2019-05-12
     License: GNU General Public License v3.0 - https://www.gnu.org/licenses/gpl-3.0.html
     Public: No
 
@@ -14,7 +14,6 @@
 
     Parameter(s):
         _unit       - Unit for the Check    [OBJECT, defaults to player]
-        _role       - Player role           [STRING, defaults to ""]
         _vehicle    - Player vehicle        [OBJECT, defaults to objNull]
 
     Returns:
@@ -23,7 +22,6 @@
 
 params [
     ["_unit", player, [objNull]],
-    ["_role", "", [""]],
     ["_vehicle", objNull, [objNull]]
 ];
 
@@ -35,14 +33,14 @@ if !(KPLIB_param_permission) exitWith {
 private _types = PGVAR("permissionTypes", []);
 private _data = [];
 private _permission = "";
-private _classNames = [];
-
-if (
-    _role isEqualTo "cargo" ||
-    typeOf _vehicle isEqualTo KPLIB_preset_potatoF ||
-    typeOf _vehicle isEqualTo KPLIB_preset_addHeliF ||
-    typeOf _vehicle isEqualTo KPLIB_preset_addBoatF
-) exitWith {
+private _isCargo = (_vehicle getCargoIndex _unit) != -1;
+private _whitelist = [
+    KPLIB_preset_potatoF,
+    KPLIB_preset_addHeliF,
+    KPLIB_preset_addBoatF
+];
+
+if (_isCargo || typeOf _vehicle in _whitelist) exitWith {
     true
 };
 

diff --git a/Missionframework/modules/11_permission/fnc/fn_permission_postInit.sqf b/Missionframework/modules/11_permission/fnc/fn_permission_postInit.sqf
@@ -4,7 +4,7 @@
     File: fn_permission_postInit.sqf
     Author: KP Liberation Dev Team - https://github.com/KillahPotatoes
     Date: 2018-12-07
-    Last Update: 2019-05-04
+    Last Update: 2019-05-12
     License: GNU General Public License v3.0 - https://www.gnu.org/licenses/gpl-3.0.html
     Public: No
 
@@ -147,8 +147,8 @@ if (hasInterface) then {
     };
 
     // Add vehicle event handlers
-    player addEventHandler ["GetInMan", {if !([_this select 0, _this select 1, _this select 2] call KPLIB_fnc_permission_checkVehiclePermission) then {[] call KPLIB_fnc_permission_ejectPlayer};}];
-    player addEventHandler ["SeatSwitchedMan", {if !([_this select 0, "", _this select 2] call KPLIB_fnc_permission_checkVehiclePermission) then {[] call KPLIB_fnc_permission_ejectPlayer};}];
+    player addEventHandler ["GetInMan", {if !([_this select 0, _this select 2] call KPLIB_fnc_permission_checkVehiclePermission) then {[] call KPLIB_fnc_permission_ejectPlayer};}];
+    player addEventHandler ["SeatSwitchedMan", {if !([_this select 0, _this select 2] call KPLIB_fnc_permission_checkVehiclePermission) then {[] call KPLIB_fnc_permission_ejectPlayer};}];
 };
 
 if (isServer) then {["Module initialized", "POST] [PERMISSION", true] call KPLIB_fnc_common_log;};