Skip to content

Replace createVehicleCrew with fn_createCrew#30

Merged
doxus merged 1 commit intomainfrom
feat/fn_createCrew
Dec 16, 2024
Merged

Replace createVehicleCrew with fn_createCrew#30
doxus merged 1 commit intomainfrom
feat/fn_createCrew

Conversation

@carlmylo
Copy link

@carlmylo carlmylo commented Nov 17, 2024

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

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 <4293906+veteran29@users.noreply.github.com>
Co-Authored-By: PiG13BR <170875982+pig13br@users.noreply.github.com>
@carlmylo carlmylo added scripts any pending tweaks to scripts presets anything relating to arsenal, loadouts, and whatever else in the "presets" folder gameplay anything that's gameflow related. includes balancing, gameloop, etc labels Nov 17, 2024
Copy link

@doxus doxus left a 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;
Copy link

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?

Comment on lines +39 to +65
/*
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;
}
}
}
};
*/
Copy link

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?

Copy link

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.

@doxus doxus merged commit 6ed4806 into main Dec 16, 2024
@doxus doxus deleted the feat/fn_createCrew branch December 16, 2024 16:25
jishnukarri added a commit to phantom24201/KP-Liberation-APR that referenced this pull request Apr 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gameplay anything that's gameflow related. includes balancing, gameloop, etc presets anything relating to arsenal, loadouts, and whatever else in the "presets" folder scripts any pending tweaks to scripts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants