Skip to content

[Cleanup] Fix Issues with Strings::Commify and Mob::SendStatsWindow#4984

Merged
Kinglykrab merged 1 commit intomasterfrom
cleanup/strings_commify_sendstatswindow
Aug 17, 2025
Merged

[Cleanup] Fix Issues with Strings::Commify and Mob::SendStatsWindow#4984
Kinglykrab merged 1 commit intomasterfrom
cleanup/strings_commify_sendstatswindow

Conversation

@Kinglykrab
Copy link
Copy Markdown
Contributor

Description

  • Fixes an issue where a number like -100 results in a string of -,100 due to Strings::Commify using string length.
  • If the number has a length of 4 and begins with -, we return the string.
  • If the string has a length of 3, we return the string.
  • Removes an extra | spacing in Mob::SendStatsWindow().

Type of change

  • Bug fix

Checklist

  • I have tested my changes
  • I have performed a self-review of my code. Ensuring variables, functions and methods are named in a human-readable way, comments are added only where naming of variables, functions and methods can't give enough context.
  • I own the changes of my code and take responsibility for the potential issues that occur

@Kinglykrab Kinglykrab merged commit ed3f6c2 into master Aug 17, 2025
2 checks passed
@Kinglykrab Kinglykrab deleted the cleanup/strings_commify_sendstatswindow branch August 17, 2025 01:35
@joligario joligario mentioned this pull request Sep 15, 2025
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