WordPress.org

Make WordPress Core

Changeset 26228


Ignore:
Timestamp:
11/15/13 21:24:43 (5 months ago)
Author:
nacin
Message:

Fix JSHint errors in shortcode.js.

props tommcfarlin.
fixes #25945.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/js/shortcode.js

    r22798 r26228  
    2020            match = re.exec( text ); 
    2121 
    22             if ( ! match ) 
     22            if ( ! match ) { 
    2323                return; 
     24            } 
    2425 
    2526            // If we matched an escaped shortcode, try again. 
    26             if ( match[1] === '[' && match[7] === ']' ) 
     27            if ( '[' === match[1] && ']' === match[7] ) { 
    2728                return wp.shortcode.next( tag, text, re.lastIndex ); 
     29            } 
    2830 
    2931            result = { 
     
    4143 
    4244            // If we matched a trailing `]`, strip it from the match. 
    43             if ( match[7] ) 
     45            if ( match[7] ) { 
    4446                result.match = result.match.slice( 0, -1 ); 
     47            } 
    4548 
    4649            return result; 
     
    5760        // and a boolean flag to indicate if the match was a `single` tag. 
    5861        replace: function( tag, text, callback ) { 
    59             return text.replace( wp.shortcode.regexp( tag ), function( match, left, tag, attrs, slash, content, closing, right, offset ) { 
     62            return text.replace( wp.shortcode.regexp( tag ), function( match, left, tag, attrs, slash, content, closing, right ) { 
    6063                // If both extra brackets exist, the shortcode has been 
    6164                // properly escaped. 
    62                 if ( left === '[' && right === ']' ) 
     65                if ( left === '[' && right === ']' ) { 
    6366                    return match; 
     67                } 
    6468 
    6569                // Create the match object and pass it through the callback. 
     
    165169            var type; 
    166170 
    167             if ( match[4] ) 
     171            if ( match[4] ) { 
    168172                type = 'self-closing'; 
    169             else if ( match[6] ) 
     173            } else if ( match[6] ) { 
    170174                type = 'closed'; 
    171             else 
     175            } else { 
    172176                type = 'single'; 
     177            } 
    173178 
    174179            return new wp.shortcode({ 
     
    203208        }; 
    204209 
    205         if ( ! attrs ) 
     210        if ( ! attrs ) { 
    206211            return; 
     212        } 
    207213 
    208214        // Parse a string of attributes. 
     
    245251 
    246252            _.each( this.attrs.numeric, function( value ) { 
    247                 if ( /\s/.test( value ) ) 
     253                if ( /\s/.test( value ) ) { 
    248254                    text += ' "' + value + '"'; 
    249                 else 
     255                } else { 
    250256                    text += ' ' + value; 
     257                } 
    251258            }); 
    252259 
     
    257264            // If the tag is marked as `single` or `self-closing`, close the 
    258265            // tag and ignore any additional content. 
    259             if ( 'single' === this.type ) 
     266            if ( 'single' === this.type ) { 
    260267                return text + ']'; 
    261             else if ( 'self-closing' === this.type ) 
     268            } else if ( 'self-closing' === this.type ) { 
    262269                return text + ' /]'; 
     270            } 
    263271 
    264272            // Complete the opening tag. 
    265273            text += ']'; 
    266274 
    267             if ( this.content ) 
     275            if ( this.content ) { 
    268276                text += this.content; 
     277            } 
    269278 
    270279            // Add the closing tag. 
     
    292301 
    293302            // If `content` ends in a slash, strip it. 
    294             if ( '/' === content[ content.length - 1 ] ) 
     303            if ( '/' === content[ content.length - 1 ] ) { 
    295304                content = content.slice( 0, -1 ); 
     305            } 
    296306 
    297307            result = wp.shortcode.attrs( content ); 
     
    299309 
    300310            _.each( result.numeric, function( key ) { 
    301                 if ( /\s/.test( key ) ) 
     311                if ( /\s/.test( key ) ) { 
    302312                    return; 
     313                } 
    303314 
    304315                attrs[ key ] = ''; 
     
    317328 
    318329                // Use empty attribute notation where possible. 
    319                 if ( '' === value ) 
     330                if ( '' === value ) { 
    320331                    return; 
     332                } 
    321333 
    322334                // Convert boolean values to strings. 
    323                 if ( _.isBoolean( value ) ) 
     335                if ( _.isBoolean( value ) ) { 
    324336                    value = value ? 'true' : 'false'; 
     337                } 
    325338 
    326339                text += '="' + value + '"'; 
     
    328341 
    329342            // Return the result if it is a self-closing tag. 
    330             if ( options.single ) 
     343            if ( options.single ) { 
    331344                return text + ' />'; 
     345            } 
    332346 
    333347            // Complete the opening tag. 
Note: See TracChangeset for help on using the changeset viewer.