Skip to content

Fix missing string on can't remove vehicle error#3603

Merged
duncanspumpkin merged 3 commits intoOpenLoco:masterfrom
ethan-xd:cant-remove-string-error
Jan 7, 2026
Merged

Fix missing string on can't remove vehicle error#3603
duncanspumpkin merged 3 commits intoOpenLoco:masterfrom
ethan-xd:cant-remove-string-error

Conversation

@ethan-xd
Copy link
Copy Markdown
Contributor

@ethan-xd ethan-xd commented Jan 5, 2026

Format arguments were skipping the wrong amount.

Before
image
After
image

@ethan-xd
Copy link
Copy Markdown
Contributor Author

ethan-xd commented Jan 5, 2026

Additionally I think #2627 should already be closed as fixed as I didn't encounter that bug. With disable breakdowns on, the vehicle eventually repaired.

Copy link
Copy Markdown
Member

@AaronVanGeffen AaronVanGeffen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Could you add a changelog entry?

@AaronVanGeffen AaronVanGeffen added this to the v25.12+ milestone Jan 5, 2026
@ethan-xd
Copy link
Copy Markdown
Contributor Author

ethan-xd commented Jan 5, 2026

@AaronVanGeffen Do you know which issue I should be linking to in the changelog or should I just reference this PR? #2627 is a different bug, and #2803 kind of references it but was closed as a duplicate of the previously mentioned one.

@ethan-xd
Copy link
Copy Markdown
Contributor Author

ethan-xd commented Jan 5, 2026

I'll just link to this PR for now, let me know if you want it different

@LeftofZen LeftofZen added the changelog Requires a changelog entry label Jan 5, 2026
@LeftofZen
Copy link
Copy Markdown
Contributor

@ethan-xd You link the PR that had the code change being merged. So you'd link this one, 3603

Copy link
Copy Markdown
Contributor

@duncanspumpkin duncanspumpkin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checked against the assembly.

@duncanspumpkin duncanspumpkin merged commit 0a007a9 into OpenLoco:master Jan 7, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog Requires a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants