Skip to content

fix typo in cbanmanager#4920

Open
WUBE1234 wants to merge 2 commits into
multitheftauto:masterfrom
WUBE1234:patch-4
Open

fix typo in cbanmanager#4920
WUBE1234 wants to merge 2 commits into
multitheftauto:masterfrom
WUBE1234:patch-4

Conversation

@WUBE1234
Copy link
Copy Markdown

usfixed a typo in GetBansWithBanner

Summary

comma operator was used instead of ==, fixing it now

Motivation

GetBansWithBanner lookup was broken because of the typo

Test plan

just syntax fix, safe to merge

Checklist

  • Your code should follow the coding guidelines.
  • Smaller pull requests are easier to review. If your pull request is beefy, your pull request should be reviewable commit-by-commit.

used == instead of comma in GetBansWithBanner
@Lpsd
Copy link
Copy Markdown
Member

Lpsd commented May 20, 2026

There are other changes here which you've not mentioned above. Why is the code removed?

@WUBE1234
Copy link
Copy Markdown
Author

my bad, messed up during copy-paste in the browser editor and didn't notice the file got truncated at the end. i'll revert the accidental deletions right away and push only the intended lookups fix. sorry about that.

reverted the unintended code removal from previous commit, file is complete now
@DmitriyColeman
Copy link
Copy Markdown
Contributor

Your last commit didn't change anything at all.

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.

3 participants