Twenty Fourteen: New PHP Coding Standards Brace Requirement

[rickalee]

Since 3.8 is new code I thought it would be appropriate to get in new brace requirement in trunk along with further feedback on how spacing after braces should be handled.

[nacin]

This was contributed during the WordCamp Orlando contributor day. Looks good!

Side note for lancewillett: Instead of get_the_tag_list(), it looks like the theme can just use the_tags(). This would eliminate the echo and the has_tag() conditional.

[lancewillett]

Replying to nacin:

This was contributed during the WordCamp Orlando contributor day. Looks good!

Very cool! Thanks for contributing.

Side note for lancewillett: Instead of get_the_tag_list(), it looks like the theme can just use the_tags(). This would eliminate the echo and the has_tag() conditional.

Good call.

[lancewillett]

In 26258:

Twenty Fourteen: simplify tags output in content templates, props nacin. See #26093.

[lancewillett]

Found a few more :)

[lancewillett]

Links for reference:

• ​http://make.wordpress.org/core/handbook/coding-standards/php/#brace-style
• ​http://make.wordpress.org/core/2013/11/13/proposed-coding-standards-change-always-require-braces/

[lancewillett]

In 26260:

Twenty Fourteen: updates all PHP files to meet new brace style in WP coding standards. Props rickalee for initial patch, fixes #26093.

[NikV]

I might be able to add a few more.

[nacin]

Hi NikV, attachments don't result in notifications, so no one saw [attchment:26093.3.patch]. Sorry about that!