-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace createVehicleCrew with fn_createCrew #30
Conversation
Adapted from main KP-Liberation PR. Only edits were to adjust the player custom.sqf file as we use our own array setup. As the original PR says, we'll need to add the new variables in presets (KPLIB_o_crewman, KPLIB_o_pilot, KPLIB_b_crewStatic). More info here: KillahPotatoes#963 Co-Authored-By: Filip Maciejewski <[email protected]> Co-Authored-By: PiG13BR <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
huh? looks like you commented out some of it? why
// Placeholder types | ||
switch (_side) do { | ||
case KPLIB_side_enemy : { | ||
_typeCrew = KPLIB_o_rifleman; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All opfor crew is riflemen?
/* | ||
Save it for later changes | ||
switch (_side) do { | ||
case KPLIB_side_enemy : { | ||
if (_vehicle isKindOf "Air") then { | ||
_typeCrew = KPLIB_o_pilot; | ||
} else { | ||
if ((_vehicle isKindOf "Tank") || {_vehicle isKindOf "Wheeled_APC_F"}) then { | ||
_typeCrew = KPLIB_o_crewman; | ||
} else { | ||
_typeCrew = KPLIB_o_rifleman; | ||
} | ||
} | ||
}; | ||
case KPLIB_side_player : { | ||
if (_vehicle isKindOf "Air") then { | ||
_typeCrew = KPLIB_b_heliPilotUnit; | ||
} else { | ||
if ((_vehicle isKindOf "Tank") || {_vehicle isKindOf "Wheeled_APC_F"}) then { | ||
_typeCrew = KPLIB_b_crewUnit; | ||
} else { | ||
_typeCrew = KPLIB_b_crewStatic; | ||
} | ||
} | ||
} | ||
}; | ||
*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this supposed to be commented out?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this supposed to be commented out?
If you want to add those new variables in presets, then yes. The original version uses them. I tested a lot and works just fine.
Adapted from a v0.96.8 branch KP-Liberation PR.
Only edits were to adjust the player custom.sqf file as we use our own array setup. As the original PR says, we'll need to add the new variables into presets (KPLIB_o_crewman, KPLIB_o_pilot, KPLIB_b_crewStatic).
More info here: KillahPotatoes#963