Updating admin-bar.php for coding standards

[welcher]

This is my first ticket/patch so please let me know if I am not adhering to the correct process. I was looking into #28569 and noticed that there were some if statements that didn't have braces so I added them.

[welcher]

admin-bar.php with if statement braces added

[SergeyBiryukov]

Please note that formatting cleanup tickets are generally discouraged. Coding style fixes should be a result of more substantial changes: ​http://wpdevel.wordpress.com/2011/03/23/code-refactoring/. But thanks for the patch!

[welcher]

Replying to SergeyBiryukov:

Please note that formatting cleanup tickets are generally discouraged. Coding style fixes should be a result of more substantial changes: ​http://wpdevel.wordpress.com/2011/03/23/code-refactoring/. But thanks for the patch!

Thanks for letting me know - I'll do some actual work next time :)

[DrewAPicture]

Replying to welcher:

Replying to SergeyBiryukov:

Please note that formatting cleanup tickets are generally discouraged. Coding style fixes should be a result of more substantial changes: ​http://wpdevel.wordpress.com/2011/03/23/code-refactoring/. But thanks for the patch!

Thanks for letting me know - I'll do some actual work next time :)

It's definitely "actual work" to make updates like this, so thank you for the patch just the same. The reasoning behind only updating for coding standards when the lines would be changing anyway is to reduce undue churn in the codebase, that's all :)

If you're interested in contributing in ways similar to this, the docs team does accept patches that fix inline documentation in WordPress core. See #25412 for a recent example of this.