WordPress.org

Plugin Directory

Changeset 627268


Ignore:
Timestamp:
11/19/12 20:24:10 (17 months ago)
Author:
mdawaffe
Message:

Shortcodes: WS cleanup and case insensitivity for audio file extensions.

Location:
jetpack/trunk/modules/shortcodes
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • jetpack/trunk/modules/shortcodes/audio.php

    r588731 r627268  
    77 
    88    static $add_script = false; 
    9      
     9 
    1010    /** 
    1111     * Add all the actions & resgister the shortcode 
     
    162162        // generate default titles 
    163163        $file_titles = array(); 
    164         for ( $i = 0; $i < $num_files; $i++ ) {  
    165             $file_titles[] = 'Track #' . ($i+1);  
     164        for ( $i = 0; $i < $num_files; $i++ ) { 
     165            $file_titles[] = 'Track #' . ($i+1); 
    166166        } 
    167167 
     
    191191        foreach ( $sound_files as $i => $sfile ) { 
    192192            $file_extension = pathinfo( $sfile, PATHINFO_EXTENSION ); 
    193             if ( ! preg_match( '/^(mp3|wav|ogg|oga|m4a|aac|webm)$/', $file_extension ) ) { 
     193            if ( ! preg_match( '/^(mp3|wav|ogg|oga|m4a|aac|webm)$/i', $file_extension ) ) { 
    194194                $html5_audio .= '<!-- Audio shortcode unsupported audio format -->'; 
    195195                if ( 1 == $num_files ) { 
     
    200200                $all_mp3 = false; 
    201201                continue; 
    202             } elseif ( ! preg_match( '/^mp3$/', $file_extension ) ) { 
     202            } elseif ( ! preg_match( '/^mp3$/i', $file_extension ) ) { 
    203203                $all_mp3 = false; 
    204204            } 
     
    241241        if ( is_feed() ) { 
    242242            $html5_audio = $not_supported; 
    243         }  
    244  
    245         if ( $all_mp3 ) {  
     243        } 
     244 
     245        if ( $all_mp3 ) { 
    246246            // process regular flash player, inserting HTML5 tags into object as fallback 
    247247            $audio_tags = <<<FLASH 
     
    269269        // mashup the artist/titles for the script 
    270270        $script_titles = array(); 
    271         for ( $i = 0; $i < $num_files; $i++ ) {  
     271        for ( $i = 0; $i < $num_files; $i++ ) { 
    272272            $script_titles[] = $file_artists[$i] . $file_titles[$i]; 
    273273 
     
    301301                } 
    302302            })(); 
    303             //]]>  
     303            //]]> 
    304304            </script> 
    305305SCRIPT; 
     
    309309            $audio_tags .= $script; 
    310310        } 
    311          
     311 
    312312        return "<span style='text-align:left;display:block;'><p>$audio_tags</p></span>"; 
    313313    } 
     
    343343                    wp_as_js.src = $script_url; 
    344344                    wp_as_js.async = true; 
    345                     wp_as_js.onload = function() {  
    346                         jQuery( document.body ).trigger( 'as-script-load' );  
     345                    wp_as_js.onload = function() { 
     346                        jQuery( document.body ).trigger( 'as-script-load' ); 
    347347                    }; 
    348348                    document.getElementsByTagName( 'head' )[0].appendChild( wp_as_js ); 
  • jetpack/trunk/modules/shortcodes/vimeo.php

    r535805 r627268  
    4040    if ( isset( $args['w'] ) ) { 
    4141        $width = (int) $args['w']; 
    42          
     42 
    4343        if ( ! isset( $args['h'] ) ) { 
    4444            // The case where w=300 is specified without h=200, otherwise $height 
     
    4747        } 
    4848    } 
    49      
     49 
    5050    if ( isset( $args['h'] ) ) { 
    5151        $height = (int) $args['h']; 
     
    5555        } 
    5656    } 
    57      
     57 
    5858    if ( ! $width ) 
    5959        $width = absint( $content_width ); 
     
    7272 
    7373function vimeo_embed_to_shortcode( $content ) { 
    74     if ( false === stripos( $content, 'player.vimeo.com/video/' ) )  
     74    if ( false === stripos( $content, 'player.vimeo.com/video/' ) ) 
    7575        return $content; 
    7676 
    7777    $regexp = '!<iframe\s+src=[\'"]http://player\.vimeo\.com/video/(\d+)[\'"]((?:\s+\w+=[\'"][^\'"]*[\'"])*)></iframe>!i'; 
    78     $regexp_ent = str_replace( '&amp;#0*58;', '&amp;#0*58;|&#0*58;', htmlspecialchars( $regexp, ENT_NOQUOTES ) );  
    79    
    80     foreach ( array( 'regexp', 'regexp_ent' ) as $reg ) {  
     78    $regexp_ent = str_replace( '&amp;#0*58;', '&amp;#0*58;|&#0*58;', htmlspecialchars( $regexp, ENT_NOQUOTES ) ); 
     79 
     80    foreach ( array( 'regexp', 'regexp_ent' ) as $reg ) { 
    8181        if ( !preg_match_all( $$reg, $content, $matches, PREG_SET_ORDER ) ) 
    8282            continue; 
     
    8787            $params = $match[2]; 
    8888 
    89             if ( 'regexp_ent' == $reg )  
     89            if ( 'regexp_ent' == $reg ) 
    9090                $params = html_entity_decode( $params ); 
    9191 
     
    9696 
    9797            $wh = ''; 
    98             if ( $width && $height )  
    99                 $wh = ' w=' . $width . ' h=' . $height;  
     98            if ( $width && $height ) 
     99                $wh = ' w=' . $width . ' h=' . $height; 
    100100 
    101101            $shortcode = '[vimeo ' . $id . $wh . ']'; 
Note: See TracChangeset for help on using the changeset viewer.