Ticket #3089: 3089.diff

File 3089.diff, 5.3 KB (added by DD32, 4 years ago)

Moves Translation and Display markup to Plugins page. Note that PHPDoc has not been updated to reflect; May break compat with Updater's web api due to markup changes, May not actually translate(do not have testing plugin)

  • wp-admin/includes/plugin.php

     
    112112 */ 
    113113function get_plugin_data( $plugin_file ) { 
    114114        $plugin_data = plugin_get_contents( $plugin_file ); 
    115         preg_match( '|Plugin Name:(.*)$|mi', $plugin_data, $plugin_name ); 
    116         preg_match( '|Plugin URI:(.*)$|mi', $plugin_data, $plugin_uri ); 
     115        preg_match( '|Plugin Name:(.*)$|mi', $plugin_data, $name ); 
     116        preg_match( '|Plugin URI:(.*)$|mi', $plugin_data, $uri ); 
     117        preg_match( '|Version:(.*)|i', $plugin_data, $version ); 
    117118        preg_match( '|Description:(.*)$|mi', $plugin_data, $description ); 
    118119        preg_match( '|Author:(.*)$|mi', $plugin_data, $author_name ); 
    119120        preg_match( '|Author URI:(.*)$|mi', $plugin_data, $author_uri ); 
     121        preg_match( '|Text Domain:(.*)$|mi', $plugin_data, $text_domain ); 
     122        preg_match( '|Domain Path:(.*)$|mi', $plugin_data, $domain_path ); 
    120123 
    121         if ( preg_match( "|Version:(.*)|i", $plugin_data, $version )) 
    122                 $version = trim( $version[1] ); 
    123         else 
    124                 $version = ''; 
    125  
    126         if( preg_match( '|Text Domain:(.*)$|mi', $plugin_data, $text_domain ) ) { 
    127                 if( preg_match( '|Domain Path:(.*)$|mi', $plugin_data, $domain_path ) ) 
    128                         $domain_path = trim( $domain_path[1] ); 
    129  
    130                 $text_domain = trim( $text_domain[1] ); 
    131  
    132                 if( !empty( $text_domain ) ) { 
    133                         if( !empty( $domain_path ) ) 
    134                                 load_plugin_textdomain($text_domain, dirname($plugin_file). $domain_path); 
    135                         else 
    136                                 load_plugin_textdomain($text_domain, dirname($plugin_file)); 
    137                 } 
    138  
    139                 $description[1] = translate(trim($description[1]), $text_domain); 
    140                 $plugin_name[1] = translate(trim($plugin_name[1]), $text_domain); 
    141                 $plugin_uri[1] = translate(trim($plugin_uri[1]), $text_domain); 
    142                 $author_name[1] = translate(trim($author_name[1]), $text_domain); 
    143                 $author_uri[1] = translate(trim($author_uri[1]), $text_domain); 
     124        foreach ( array( 'name', 'uri', 'version', 'description', 'author_name', 'author_uri', 'text_domain', 'domain_path' ) as $field ) { 
     125                if ( !empty( ${$field} ) ) 
     126                        ${$field} = trim(${$field}[1]); 
     127                else 
     128                        ${$field} = ''; 
    144129        } 
    145130 
    146         $description = wptexturize( trim( $description[1] )); 
    147  
    148         $name = $plugin_name[1]; 
    149         $name = trim( $name ); 
    150         $plugin = $name; 
    151         if ('' != trim($plugin_uri[1]) && '' != $name ) { 
    152                 $plugin = '<a href="' . trim( $plugin_uri[1] ) . '" title="'.__( 'Visit plugin homepage' ).'">'.$plugin.'</a>'; 
    153         } 
    154  
    155         if ('' == $author_uri[1] ) { 
    156                 $author = trim( $author_name[1] ); 
    157         } else { 
    158                 $author = '<a href="' . trim( $author_uri[1] ) . '" title="'.__( 'Visit author homepage' ).'">' . trim( $author_name[1] ) . '</a>'; 
    159         } 
    160  
    161         return array('Name' => $name, 'Title' => $plugin, 'Description' => $description, 'Author' => $author, 'Version' => $version); 
     131        return array( 
     132                                'Name' => $name, 'PluginURI' => $uri, 'Description' => $description,  
     133                                'Author' => $author_name, 'AuthorURI' => $author_uri, 'Version' => $version,  
     134                                'TextDomain' => $text_domain, 'DomainPath' => $domain_path 
     135                                ); 
    162136} 
    163137 
    164138function get_plugins($plugin_folder = '') { 

  • wp-admin/plugins.php

     
    216217 
    217218foreach( (array)$all_plugins as $plugin_file => $plugin_data) { 
    218219 
     220        //Translate fields 
     221        if( !empty($plugin_data['TextDomain']) ) { 
     222                if( !empty( $plugin_data['DomainPath'] ) ) 
     223                        load_plugin_textdomain($plugin_data['TextDomain'], dirname($plugin_file). $plugin_data['DomainPath']); 
     224                else 
     225                        load_plugin_textdomain($plugin_data['TextDomain'], dirname($plugin_file)); 
     226 
     227                foreach ( array('Name', 'PluginURI', 'Description', 'Author', 'AuthorURI', 'Version') as $field ) 
     228                        $plugin_data[ $field ] = translate($plugin_data[ $field ], $plugin_data['TextDomain']); 
     229        } 
     230 
     231        //Apply Markup 
     232        $plugin_data['Title'] = $plugin_data['Name']; 
     233        if ( !empty($plugin_data['PluginURI']) && !empty($plugin_data['Name']) ) 
     234                $plugin_data['Title'] = '<a href="' . $plugin_data['PluginURI'] . '" title="'.__( 'Visit plugin homepage' ).'">' . $plugin_data['Name'] . '</a>'; 
     235         
     236        if ( ! empty($plugin_data['AuthorURI']) ) 
     237                $plugin_data['Author'] = '<a href="' . $plugin_data['AuthorURI'] . '" title="'.__( 'Visit author homepage' ).'">' . $plugin_data['Author'] . '</a>'; 
     238         
     239        $plugin_data['Description'] = wptexturize( $plugin_data['Description'] ); 
     240 
    219241        // Sanitize all displayed data 
    220242        $plugin_data['Title']       = wp_kses($plugin_data['Title'], $plugins_allowedtags); 
    221243        $plugin_data['Version']     = wp_kses($plugin_data['Version'], $plugins_allowedtags); 
    … …  
    263285 
    264286                if( 'active' == $context ) 
    265287                        $action_links[] = '<a href="' . wp_nonce_url('plugins.php?action=deactivate&amp;plugin=' . $plugin_file, 'deactivate-plugin_' . $plugin_file) . '" title="' . __('Deactivate this plugin') . '" class="delete">' . __('Deactivate') . '</a>'; 
    266                 else //Available or Recently deactivated 
     288                else //Inactive or Recently deactivated 
    267289                        $action_links[] = '<a href="' . wp_nonce_url('plugins.php?action=activate&amp;plugin=' . $plugin_file, 'activate-plugin_' . $plugin_file) . '" title="' . __('Activate this plugin') . '" class="edit">' . __('Activate') . '</a>'; 
    268290 
    269291                if ( current_user_can('edit_plugins') && is_writable(WP_PLUGIN_DIR . '/' . $plugin_file) )