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

Switching to a lower-tier submarine makes you lose upgrades #9998

Closed
1 of 2 tasks
Regalis11 opened this issue Oct 6, 2022 · 4 comments
Closed
1 of 2 tasks

Switching to a lower-tier submarine makes you lose upgrades #9998

Regalis11 opened this issue Oct 6, 2022 · 4 comments
Assignees
Labels
Bug Something isn't working Code Programming task Severe An issue that causes severe problems and occurs consistently/frequently Unstable Tickets that are included and being tested in the current Unstable build.

Comments

@Regalis11
Copy link
Collaborator

Disclaimers

  • I have searched the issue tracker to check if the issue has already been reported.
  • My issue happened while using mods.

What happened?

When you switch to a lower-tier sub, the upgrades get clamped to the maximum of that sub. But when you switch back to the higher-tier one, they remain clamped, meaning you lose the upgrades.

Reproduction steps

  1. Select a tier 2 or 3 sub
  2. Max upgrades
  3. Switch to a lower tier sub
  4. Switch back to the original sub, notice the upgrades are no longer maxed

Bug prevalence

Happens every time I play

Version

0.19.11.0

-

No response

Which operating system did you encounter this bug on?

Windows

Relevant error messages and crash reports

No response

@Regalis11 Regalis11 added Bug Something isn't working Code Programming task Severe An issue that causes severe problems and occurs consistently/frequently labels Oct 6, 2022
@Regalis11 Regalis11 self-assigned this Oct 6, 2022
@Regalis11
Copy link
Collaborator Author

@Lithosagym
Copy link

Tested by maxing a Typhon2's upgrades, switching to a Dugong with clamped but maxed upgrades, and then switching back to Typhon2. Maximum upgrades were retained, issue is fixed.

@Regalis11 Regalis11 added the Unstable Tickets that are included and being tested in the current Unstable build. label Oct 16, 2022
@Rokvach Rokvach closed this as completed Oct 17, 2022
@Regalis11 Regalis11 reopened this Oct 19, 2022
@Regalis11
Copy link
Collaborator Author

Reopening: the fix causes the upgrades to get clamped to the default maximum for that specific upgrade, ignoring the tier/class bonuses.

Steps to repro:

  1. Start a campaign with Typhon 2
  2. Max turret upgrades (4 levels thanks to the Attack class bonus)
  3. Start a new round
  4. Notice the upgrades get reduced to level 3

@Rokvach Rokvach self-assigned this Oct 20, 2022
@Rokvach
Copy link
Collaborator

Rokvach commented Oct 20, 2022

Tested above repro steps, working correctly. Closing.

@Rokvach Rokvach closed this as completed Oct 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Code Programming task Severe An issue that causes severe problems and occurs consistently/frequently Unstable Tickets that are included and being tested in the current Unstable build.
Projects
None yet
Development

No branches or pull requests

3 participants