Hey @orcalator
No, that’s not a bug. That bit of code has been around since the early days of OJS and we use it everywhere. Can you post a small snippet of your configuration file with the email settings, being careful to not include any passwords or sensitive information? You can PM it to me if you like.
What you’ve essentially done is modify the code so the bounce field is enabled only if they are turned off in your own configuration.
Cheers,
Jason