This PR removes a redundant condition in the current code, but the redundant condition was due to a typo in #15208. I believe the correct fix is to replace banner is self.banner1 by banner is default_banner, which wouldn't be redundant anymore.

cc @tornaria

Or perhaps just banner == default_banner, since I believe default_banner is copied into self.banner1 instead of assigned.

Thanks for the suggestion! Just to confirm, should I update the if condition to check if banner == default_banner instead of removing it entirely? I can update this PR with that change.

I just tested and banner is default_banner seems to work, and it should be faster. So, yes, the suggestion I'm making is to replace the condition banner is self.banner1 by banner is default_banner, which is essentially what @tornaria suggested in #15208.

Wait, wait, I was testing banner == default_banner, which explains why it's working. Let me test the other variant.

Seems like banner is default_banner indeed works as well.