I18n: Improve translators comments [1].

jrfnl · jrfnl · commit 7839cf651f32 · 2018-08-30T12:13:53.000Z
* 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 git-svn-id: https://develop.svn.wordpress.org/trunk@43595 602fd350-edb4-49c9-b593-d223f7449a82
diff --git a/src/wp-login.php b/src/wp-login.php
@@ -941,21 +941,21 @@ function retrieve_password() {
 
 		if ( empty( $_COOKIE[ LOGGED_IN_COOKIE ] ) ) {
 			if ( headers_sent() ) {
-				/* translators: 1: Browser cookie documentation URL, 2: Support forums URL */
 				$user = new WP_Error(
 					'test_cookie',
 					sprintf(
+						/* translators: 1: Browser cookie documentation URL, 2: Support forums URL */
 						__( '<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>.' ),
 						__( 'https://codex.wordpress.org/Cookies' ),
 						__( 'https://wordpress.org/support/' )
 					)
 				);
 			} elseif ( isset( $_POST['testcookie'] ) && empty( $_COOKIE[ TEST_COOKIE ] ) ) {
 				// If cookies are disabled we can't log in even with a valid user+pass
-				/* translators: %s: Browser cookie documentation URL */
 				$user = new WP_Error(
 					'test_cookie',
 					sprintf(
+						/* translators: %s: Browser cookie documentation URL */
 						__( '<strong>ERROR</strong>: Cookies are blocked or not supported by your browser. You must <a href="%s">enable cookies</a> to use WordPress.' ),
 						__( 'https://codex.wordpress.org/Cookies' )
 					)
diff --git a/src/wp-signup.php b/src/wp-signup.php
@@ -323,13 +323,19 @@ function signup_another_blog( $blogname = '', $blog_title = '', $errors = '' ) {
 	$blog_title = $filtered_results['blog_title'];
 	$errors     = $filtered_results['errors'];
 
+	/* translators: %s: Network's site name. */
 	echo '<h2>' . sprintf( __( 'Get <em>another</em> %s site in seconds' ), get_network()->site_name ) . '</h2>';
 
 	if ( $errors->has_errors() ) {
 		echo '<p>' . __( 'There was a problem, please correct the form below and try again.' ) . '</p>';
 	}
 	?>
-	<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>
+	<p>
+		<?php
+		/* translators: %s: Current user's display name. */
+		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>
 
 	<?php
 	$blogs = get_blogs_of_user( $current_user->ID );
@@ -955,13 +961,14 @@ function signup_get_available_languages() {
 				$newblog = get_blogaddress_by_name( $newblogname );
 
 				if ( $active_signup == 'blog' || $active_signup == 'all' ) {
-					/* translators: %s: site address */
 					printf(
+						/* translators: %s: site address */
 						'<p><em>' . __( 'The site you were looking for, %s, does not exist, but you can create it now!' ) . '</em></p>',
 						'<strong>' . $newblog . '</strong>'
 					);
-				} else {                  /* translators: %s: site address */
+				} else {
 					printf(
+						/* translators: %s: site address */
 						'<p><em>' . __( 'The site you were looking for, %s, does not exist.' ) . '</em></p>',
 						'<strong>' . $newblog . '</strong>'
 					);
