Skip to content

SERVER: No longer update perk machine angles on Soft_Restart. fixes nzp-team/nzportable#1193#143

Merged
MotoLegacy merged 3 commits into
nzp-team:mainfrom
sbeeds:main
Oct 22, 2025
Merged

SERVER: No longer update perk machine angles on Soft_Restart. fixes nzp-team/nzportable#1193#143
MotoLegacy merged 3 commits into
nzp-team:mainfrom
sbeeds:main

Conversation

@sbeeds
Copy link
Copy Markdown
Contributor

@sbeeds sbeeds commented Oct 10, 2025

Description of Changes


Changed GameRestart_TurnPerkOff to not update angles of perks, since they are already set when the machine is spawned in.
This change has been tested in co-op with players leaving/joining and on solo.

Visual Sample


Soft reloading the game while perk is still present:

2025-10-10.00-13-54.mp4

Soft reloading the game after perk has left (bought QR 3 times):

2025-10-10.00-13-32.mp4

Checklist


  • I have thoroughly tested my changes to the best of my ability
  • I confirm I have not contributed anything that would impact Nazi Zombies: Portable's licensing and usage
  • This Pull Request fixes a critical issue that should be reviewed and merged as soon as possible

This is done when the perk is initially spawned, so there is no need to update the angles again.
@sbeeds sbeeds changed the title SERVER: No longer update perk machine angles on Soft_Restart. closes #1193 SERVER: No longer update perk machine angles on Soft_Restart. fixes #1193 Oct 10, 2025
@sbeeds sbeeds changed the title SERVER: No longer update perk machine angles on Soft_Restart. fixes #1193 SERVER: No longer update perk machine angles on Soft_Restart. fixes nzp-team/nzportable#1193 Oct 10, 2025
@MotoLegacy
Copy link
Copy Markdown
Member

Can you write or adjust an existing unit test and confirm that the test fails before your fix, and passes after?

@sbeeds
Copy link
Copy Markdown
Contributor Author

sbeeds commented Oct 10, 2025

I have included a unit test and verified that this test fails prior to changes, and passes after changes.

@MotoLegacy MotoLegacy merged commit 321e91b into nzp-team:main Oct 22, 2025
2 checks passed
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.

2 participants