Making WordPress.org

Changeset 8348


Ignore:
Timestamp:
02/25/2019 10:22:52 PM (6 years ago)
Author:
ocean90
Message:

Translate: Remove workarounds for locale variants.

GP_Locales returns the correct locale now.

See #4206.

Location:
sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-customizations/templates
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-customizations/templates/projects-wp-plugins.php

    r5524 r8348  
    7171            foreach ( $translation_locale_complete as $locale_slug => $total_complete ) :
    7272                $gp_locale = GP_Locales::by_slug( $locale_slug );
    73                 $set_slug  = 'default';
    7473
    75                 // Variants (de/formal for example) don't have GP_Locales in this context
    76                 if ( ! $gp_locale && ( list( $base_locale_slug, $set_slug ) = explode( '/', $locale_slug ) ) ) :
    77                     $_gp_locale = GP_Locales::by_slug( $base_locale_slug );
    78                     if ( $_gp_locale ) {
    79                         $gp_locale = clone $_gp_locale;
    80                         // Just append it for now..
    81                         $gp_locale->wp_locale .= '/' . $set_slug;
    82                         $gp_locale->english_name .= ' (' . ucfirst( $set_slug ) . ')';
    83                     }
    84                 endif;
     74                if ( ! $gp_locale || ! $gp_locale->wp_locale ) {
     75                    continue;
     76                }
    8577
    86                 if ( ! $gp_locale || ! $gp_locale->wp_locale ) :
    87                     continue;
    88                 endif;
    89             ?>
     78                list( $locale, $set_slug ) = array_merge( explode( '/', $locale_slug ), [ 'default' ] );
     79                ?>
    9080                <tr>
    9181                    <th title="<?php echo esc_attr( $gp_locale->wp_locale ); ?>">
    92                         <a href="<?php echo esc_url( gp_url( gp_url_join( 'locale', $gp_locale->slug, $set_slug, $project->path ) ) ); ?>">
     82                        <a href="<?php echo esc_url( gp_url( gp_url_join( 'locale', $locale, $set_slug, $project->path ) ) ); ?>">
    9383                            <?php echo esc_html( $gp_locale->english_name ); ?>
    9484                        </a>
     
    10999                                        }
    110100
    111                                         $link_url  = gp_url( gp_url_join( 'locale', $gp_locale->slug, $set_slug, $project->path ) );
     101                                        $link_url  = gp_url( gp_url_join( 'locale', $locale, $set_slug, $project->path ) );
    112102                                        $link_text = number_format( $percent );
    113103                                    else :
    114104                                        $percent_class = (int) ( $percent / 10 ) * 10;
    115                                         $link_url  = gp_url_project( $project->path, gp_url_join( $subproject_slug, $gp_locale->slug, $set_slug ) );
     105                                        $link_url  = gp_url_project( $project->path, gp_url_join( $subproject_slug, $locale, $set_slug ) );
    116106                                        $link_text = "$percent%";
    117107
  • sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-customizations/templates/projects-wp-themes.php

    r5502 r8348  
    6262            foreach ( $translation_locale_complete as $locale_slug => $total_complete ) :
    6363                $gp_locale = GP_Locales::by_slug( $locale_slug );
    64                 $set_slug  = 'default';
    6564
    66                 // Variants (de/formal for example) don't have GP_Locales in this context
    67                 if ( ! $gp_locale && ( list( $base_locale_slug, $set_slug ) = explode( '/', $locale_slug ) ) ) :
    68                     $_gp_locale = GP_Locales::by_slug( $base_locale_slug );
    69                     if ( $_gp_locale ) {
    70                         $gp_locale = clone $_gp_locale;
    71                         // Just append it for now..
    72                         $gp_locale->wp_locale .= '/' . $set_slug;
    73                         $gp_locale->english_name .= ' (' . ucfirst( $set_slug ) . ')';
    74                     }
    75                 endif;
     65                if ( ! $gp_locale || ! $gp_locale->wp_locale ) {
     66                    continue;
     67                }
    7668
    77                 if ( ! $gp_locale || ! $gp_locale->wp_locale ) :
    78                     continue;
    79                 endif;
    80             ?>
     69                list( $locale, $set_slug ) = array_merge( explode( '/', $locale_slug ), [ 'default' ] );
     70                ?>
    8171                <tr>
    8272                    <th title="<?php echo esc_attr( $gp_locale->wp_locale ); ?>">
    83                         <a href="<?php echo esc_url( gp_url( gp_url_join( 'locale', $gp_locale->slug, $set_slug, $project->path ) ) ); ?>">
     73                        <a href="<?php echo esc_url( gp_url( gp_url_join( 'locale', $locale, $set_slug, $project->path ) ) ); ?>">
    8474                            <?php echo esc_html( $gp_locale->english_name ); ?>
    8575                        </a>
     
    10090                                        }
    10191
    102                                         $link_url  = gp_url( gp_url_join( 'locale', $gp_locale->slug, $set_slug, $project->path ) );
     92                                        $link_url  = gp_url( gp_url_join( 'locale', $locale, $set_slug, $project->path ) );
    10393                                        $link_text = number_format( $percent );
    10494                                    else :
    10595                                        $percent_class = (int) ( $percent / 10 ) * 10;
    106                                         $link_url  = gp_url_project( $project->path, gp_url_join( $gp_locale->slug, $set_slug ) );
     96                                        $link_url  = gp_url_project( $project->path, gp_url_join( $locale, $set_slug ) );
    10797                                        $link_text = "$percent%";
    10898
  • sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-customizations/templates/stats-overview.php

    r8242 r8348  
    2727            foreach ( $translation_locale_complete as $locale_slug => $total_complete ) :
    2828                $gp_locale = GP_Locales::by_slug( $locale_slug );
    29                 $set_slug = 'default';
    30                 // Variants (de/formal for example) don't have GP_Locales in this context
    31                 if ( ! $gp_locale && strpos( $locale_slug, '/' ) && ( list( $base_locale_slug, $set_slug ) = explode( '/', $locale_slug ) ) ) {
    32                     $_gp_locale = GP_Locales::by_slug( $base_locale_slug );
    33                     if ( $_gp_locale ) {
    34                         $gp_locale = clone $_gp_locale;
    35                         // Just append it for now..
    36                         $gp_locale->wp_locale .= '/' . $set_slug;
    37                     }
    38                 }
     29
    3930                if ( ! $gp_locale || ! $gp_locale->wp_locale ) {
    4031                    continue;
    4132                }
    42             ?>
     33
     34                list( $locale, $set_slug ) = array_merge( explode( '/', $locale_slug ), [ 'default' ] );
     35                ?>
    4336                <tr>
    4437                    <th title="<?php echo esc_attr( $gp_locale->english_name ); ?>">
    45                         <a href="<?php echo esc_url( gp_url( gp_url_join( 'locale', $gp_locale->slug, $set_slug ) ) ); ?>">
     38                        <a href="<?php echo esc_url( gp_url( gp_url_join( 'locale', $locale, $set_slug ) ) ); ?>">
    4639                            <?php echo esc_html( $gp_locale->wp_locale ); ?>
    4740                        </a>
     
    4942                    <?php
    5043                    foreach ( $projects as $slug => $project ) {
    51                         $projecturl = gp_url( gp_url_join( 'locale', $gp_locale->slug, $set_slug, $project->path ) );
     44                        $projecturl = gp_url( gp_url_join( 'locale', $locale, $set_slug, $project->path ) );
    5245                        $project_name = str_replace( array( 'WordPress.org ', 'WordPress for ', 'WordPress ', 'ectory' ), '', $project->name );
    5346
Note: See TracChangeset for help on using the changeset viewer.