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

Fix control point progress recovery NPE #886

Merged
merged 1 commit into from
Jun 4, 2021

Conversation

Pugzy
Copy link
Contributor

@Pugzy Pugzy commented Jun 4, 2021

Fixes issue described in #dev Discord chat. The capture progress is retained by a different team once they take over the hill.

Video example: https://youtu.be/R5EZBafHEx4

Adds/fixes null check which was previously mistakenly checking for nullness of team not subtractCaptureTime (which does return null). This also fixes up a few formatting issues from the control point PR (new lines, method commends and some ZERO use.

Signed-off-by: Pugzy pugzy@mail.com

Signed-off-by: Pugzy <pugzy@mail.com>
@Pugzy Pugzy requested a review from Electroid as a code owner June 4, 2021 20:41
@Electroid Electroid merged commit 66455cd into PGMDev:dev Jun 4, 2021
@Electroid Electroid added the bug Something isn't working label Jun 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

Successfully merging this pull request may close these issues.

2 participants