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

Make it so the [!] icon in the PCS tab appears only if a PCS slot is empty. #1345

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Larryturbo
Copy link

@Larryturbo Larryturbo commented May 11, 2024

Fixes #1348
For the weapon mods the little [!] icon appears in a soldier's list only if there's an empty slot for a mod.
The same isn't true for the PCS tab. After researching "Interchangeable PCSs" the orange [!] will remain there permanently for every soldier that can equip PCSs.
This isn't exactly a bug but it's not useful to have the game constantly tell the player that they can swap a soldier's equipped PCS with one that might even be worse, just because there is an unused PCS in the player's inventory.

@RustyDios
Copy link
Contributor

surely this should be;
PCSAvailabilityData.bNoCombatSimsEquipped = ( EquippedImplants.Length <= 0 );

@chrishayesmu
Copy link

surely this should be;
PCSAvailabilityData.bNoCombatSimsEquipped = ( EquippedImplants.Length <= 0 );

I think the logic is right, but the name is wrong. It should be something like bHasEmptyCombatSimSlots.

@RustyDios
Copy link
Contributor

which is what PCSAvailabilityData.bHasCombatSimsSlotsAvailable does?

@RustyDios
Copy link
Contributor

oh! see this is where the issue is because of the || XComHQ.bReusePCS; right ?

@Larryturbo
Copy link
Author

Larryturbo commented May 11, 2024

oh! see this is where the issue is because of the || XComHQ.bReusePCS; right ?

Yes because of that, once you finish the research, bHasNeurochipImplantsInInventory becomes the only condition for the [!] to show for every single soldier, which is practically always, unless you go and sell every extra PCS you can't immediately equip. I didn't want to alter the original variable since it has other uses, bHasEmptyCombatSimSlots is a better name for what it checks but then it'll have the same name as bHasCombatSimsSlotsAvailable. The logic is correct, the same as bHasCombatSimsSlotsAvailable without the OR, in case some mod adds more slots or something.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NeedsAttention icon in the PCS tab ignores if there's a PCS already equipped.
3 participants