Hello, it's been a while. Noticed the issue got pushed back again. Regardless, changed the year to 2023 after @asterix 's suggestion :)

Updated a log error message

Performed refactoring according to @Stan's suggestions. Admittedly, I was feeling reluctant to use the conditional ternary operator in C++ (more used to seeing it in JS), though everything is much cleaner now.

Removed unnecessary check as suggested by @phosit.

Did some refactoring based on @phosit's great advice on cleaner code.
Also included some extra dependencies in header, so as to ensure compilation with nopch.

In D4623#205721, @Stan wrote:

Check your header to make sure it compiles with nopch

@gameboy As far as I'm concerned it's ready for release, unless there's anything else that needs fixing. I'm also pretty sure that the macOS build failure isn't related to my changeset (since it has happened before), but I can't confirm that as I don't have the permission to restart the build.

Thank you for the feedback everyone. It's been a while, what with the postponement of the issue to the Alpha 27 release (cheers for Alpha 26!).

Performed refactoring in accordance to basic design principles and added translation to user-facing messages.

Reverted translation, which led to build failure.

Added translation to user facing messages as suggested by @Stan .

Great find @Langbart . Turns out there was an issue with the metadata passed to the QuickSave function.
The time elapsed should display properly now.