Skip to content
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

Merged
merged 1 commit into from
Dec 16, 2024
Merged

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 <[email protected]>
Co-Authored-By: PiG13BR <[email protected]>
@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
1 check passed
@doxus doxus deleted the feat/fn_createCrew branch December 16, 2024 16:25
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