WordPress.org

Make WordPress Core

Changes between Initial Version and Version 1 of Ticket #23216, comment 102


Ignore:
Timestamp:
07/17/13 17:00:55 (9 months ago)
Author:
adamsilverstein
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #23216, comment 102

    initial v1  
    44yes, its easy get mixed up, but posting to trac is a sure way to get corrected :) (i know from experience) 
    55 
    6 to azaozz's credit, i think the suggestion to always use curly brackets for conditionals in JS was added sometime after he wrote the bulk of that code. I might be wrong, but I'm pretty sure it used to read more like the PHP spec which  suggests omitting them for brevity, http://make.wordpress.org/core/handbook/coding-standards/php/#brace-style) 
     6to azaozz's credit, i think the suggestion to always use curly brackets for conditionals in JS was added sometime after he wrote the bulk of that code. I might be wrong, but I'm pretty sure it used to read more like the PHP spec which  suggests omitting them for brevity, http://make.wordpress.org/core/handbook/coding-standards/php/#brace-style 
    77 
    88I agree its better to use them (and Douglas Crockford explains why).