WordPress.org

Make WordPress Core

Changeset 43595


Ignore:
Timestamp:
08/30/2018 12:13:53 PM (13 months ago)
Author:
jrf
Message:

I18n: Improve translators comments [1].

  • Add missing translators comments.
  • Fix placement of some translators comments. Translators comments should be on the line directly above the line containing the translation function call for optimal compatibility with various .pot file generation tools. The CS auto-fixing, which changed some inconsistent function calls to multi-line function calls, is part of the reason why this was no longer the case for a select group of translators comments.

Patch 44360-src.2.diff of the series.

Props garyj, alvarogois, michielatyoast
See #44360

Location:
trunk/src
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-login.php

    r43571 r43595  
    942942        if ( empty( $_COOKIE[ LOGGED_IN_COOKIE ] ) ) {
    943943            if ( headers_sent() ) {
    944                 /* translators: 1: Browser cookie documentation URL, 2: Support forums URL */
    945944                $user = new WP_Error(
    946945                    'test_cookie',
    947946                    sprintf(
     947                        /* translators: 1: Browser cookie documentation URL, 2: Support forums URL */
    948948                        __( '<strong>ERROR</strong>: Cookies are blocked due to unexpected output. For help, please see <a href="%1$s">this documentation</a> or try the <a href="%2$s">support forums</a>.' ),
    949949                        __( 'https://codex.wordpress.org/Cookies' ),
     
    953953            } elseif ( isset( $_POST['testcookie'] ) && empty( $_COOKIE[ TEST_COOKIE ] ) ) {
    954954                // If cookies are disabled we can't log in even with a valid user+pass
    955                 /* translators: %s: Browser cookie documentation URL */
    956955                $user = new WP_Error(
    957956                    'test_cookie',
    958957                    sprintf(
     958                        /* translators: %s: Browser cookie documentation URL */
    959959                        __( '<strong>ERROR</strong>: Cookies are blocked or not supported by your browser. You must <a href="%s">enable cookies</a> to use WordPress.' ),
    960960                        __( 'https://codex.wordpress.org/Cookies' )
  • trunk/src/wp-signup.php

    r43571 r43595  
    324324    $errors     = $filtered_results['errors'];
    325325
     326    /* translators: %s: Network's site name. */
    326327    echo '<h2>' . sprintf( __( 'Get <em>another</em> %s site in seconds' ), get_network()->site_name ) . '</h2>';
    327328
     
    330331    }
    331332    ?>
    332     <p><?php printf( __( 'Welcome back, %s. By filling out the form below, you can <strong>add another site to your account</strong>. There is no limit to the number of sites you can have, so create to your heart&#8217;s content, but write responsibly!' ), $current_user->display_name ); ?></p>
     333    <p>
     334        <?php
     335        /* translators: %s: Current user's display name. */
     336        printf( __( 'Welcome back, %s. By filling out the form below, you can <strong>add another site to your account</strong>. There is no limit to the number of sites you can have, so create to your heart&#8217;s content, but write responsibly!' ), $current_user->display_name );
     337        ?>
     338    </p>
    333339
    334340    <?php
     
    956962
    957963                if ( $active_signup == 'blog' || $active_signup == 'all' ) {
    958                     /* translators: %s: site address */
    959964                    printf(
     965                        /* translators: %s: site address */
    960966                        '<p><em>' . __( 'The site you were looking for, %s, does not exist, but you can create it now!' ) . '</em></p>',
    961967                        '<strong>' . $newblog . '</strong>'
    962968                    );
    963                 } else {                  /* translators: %s: site address */
     969                } else {
    964970                    printf(
     971                        /* translators: %s: site address */
    965972                        '<p><em>' . __( 'The site you were looking for, %s, does not exist.' ) . '</em></p>',
    966973                        '<strong>' . $newblog . '</strong>'
Note: See TracChangeset for help on using the changeset viewer.