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

Fixes inaccurate wait durations in the "Wait a while" menu #45418

Merged
merged 1 commit into from
Nov 15, 2020

Conversation

Squishums
Copy link
Contributor

@Squishums Squishums commented Nov 14, 2020

Summary

SUMMARY: Bugfixes "Fixes inaccurate wait duration in the Wait a while menu"

Purpose of change

The actual time waited for each of the entries in the Wait a while menu was 1 second shorter than it should have been. This applied to both relative (5 minutes) and absolute (until noon) waits. The issue was brought in over 4 years ago in #17116 and I don't really see why? Maybe the initial turn the activity was created wasn't counted back then.

Fixes #45352

Describe the solution

Remove - 1 from wait code.

Describe alternatives you've considered

It's impossible to unsee. There is no alternative.

Testing

  • Make character with watch
  • Wait 5 minutes. Sidebar time goes from 11:50 to 11:55
  • Wait until noon. Sidebar time goes from 11:50 to 12:00

@anothersimulacrum anothersimulacrum added <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` labels Nov 14, 2020
@ZhilkinSerg ZhilkinSerg merged commit a2c2a36 into CleverRaven:master Nov 15, 2020
@Squishums Squishums deleted the wait-time-fix branch November 15, 2020 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wait a while waits 1 second less than it should
3 participants