From fff36156c20183bbd3d32be51514b7700b5d577c Mon Sep 17 00:00:00 2001
From: Sergey Biryukov <sergeybiryukov@git.wordpress.org>
Date: Fri, 30 Oct 2015 10:39:02 +0000
Subject: [PATCH] Don't use `<a>` in translatable strings in
 `theme-compat/sidebar.php`.

Add translator comments.

Props ramiy.
Fixes #34503.

git-svn-id: https://develop.svn.wordpress.org/trunk@35458 602fd350-edb4-49c9-b593-d223f7449a82
---
 src/wp-includes/theme-compat/sidebar.php | 73 ++++++++++++++++++------
 1 file changed, 54 insertions(+), 19 deletions(-)

diff --git a/src/wp-includes/theme-compat/sidebar.php b/src/wp-includes/theme-compat/sidebar.php
index fc529c9a4b..f879ec4d41 100644
--- a/src/wp-includes/theme-compat/sidebar.php
+++ b/src/wp-includes/theme-compat/sidebar.php
@@ -7,7 +7,14 @@
  * This file is here for Backwards compatibility with old themes and will be removed in a future version
  *
  */
-_deprecated_file( sprintf( __( 'Theme without %1$s' ), basename(__FILE__) ), '3.0', null, sprintf( __('Please include a %1$s template in your theme.'), basename(__FILE__) ) );
+_deprecated_file(
+	/* translators: %s: template name */
+	sprintf( __( 'Theme without %s' ), basename( __FILE__ ) ),
+	'3.0',
+	null,
+	/* translators: %s: template name */
+	sprintf( __( 'Please include a %s template in your theme.' ), basename( __FILE__ ) )
+);
 ?>
 	<div id="sidebar" role="complementary">
 		<ul>
@@ -24,32 +31,60 @@ _deprecated_file( sprintf( __( 'Theme without %1$s' ), basename(__FILE__) ), '3.
 			-->
 
 			<?php if ( is_404() || is_category() || is_day() || is_month() ||
-						is_year() || is_search() || is_paged() ) {
+						is_year() || is_search() || is_paged() ) :
 			?> <li>
 
-			<?php /* If this is a 404 page */ if (is_404()) { ?>
-			<?php /* If this is a category archive */ } elseif (is_category()) { ?>
-			<p><?php printf(__('You are currently browsing the archives for the %s category.'), single_cat_title('', false)); ?></p>
+			<?php if ( is_404() ) : /* If this is a 404 page */ ?>
+			<?php elseif ( is_category() ) : /* If this is a category archive */ ?>
+				<p><?php /* translators: %s: category name */
+					printf( __( 'You are currently browsing the archives for the %s category.' ),
+						single_cat_title( '', false )
+					);
+				?></p>
 
-			<?php /* If this is a daily archive */ } elseif (is_day()) { ?>
-			<p><?php printf(__('You are currently browsing the <a href="%1$s/">%2$s</a> blog archives for the day %3$s.'), get_bloginfo('url'), get_bloginfo('name'), get_the_time(__('l, F jS, Y'))); ?></p>
+			<?php elseif ( is_day() ) : /* If this is a daily archive */ ?>
+				<p><?php /* translators: 1: site link, 2: archive date */
+					printf( __( 'You are currently browsing the %1$s blog archives for the day %2$s.' ),
+						sprintf( '<a href="%1$s/">%2$s</a>', get_bloginfo( 'url' ), get_bloginfo( 'name' ) ),
+						get_the_time( __( 'l, F jS, Y' ) )
+					);
+				?></p>
 
-			<?php /* If this is a monthly archive */ } elseif (is_month()) { ?>
-			<p><?php printf(__('You are currently browsing the <a href="%1$s/">%2$s</a> blog archives for %3$s.'), get_bloginfo('url'), get_bloginfo('name'), get_the_time(__('F, Y'))); ?></p>
+			<?php elseif ( is_month() ) : /* If this is a monthly archive */ ?>
+				<p><?php /* translators: 1: site link, 2: archive month */
+					printf( __( 'You are currently browsing the %1$s blog archives for %2$s.' ),
+						sprintf( '<a href="%1$s/">%2$s</a>', get_bloginfo( 'url' ), get_bloginfo( 'name' ) ),
+						get_the_time( __( 'F, Y' ) )
+					);
+				?></p>
 
-			<?php /* If this is a yearly archive */ } elseif (is_year()) { ?>
-			<p><?php printf(__('You are currently browsing the <a href="%1$s/">%2$s</a> blog archives for the year %3$s.'), get_bloginfo('url'), get_bloginfo('name'), get_the_time('Y')); ?></p>
+			<?php elseif ( is_year() ) : /* If this is a yearly archive */ ?>
+				<p><?php /* translators: 1: site link, 2: archive year */
+					printf( __( 'You are currently browsing the %1$s blog archives for the year %2$s.' ),
+						sprintf( '<a href="%1$s/">%2$s</a>', get_bloginfo( 'url' ), get_bloginfo( 'name' ) ),
+						get_the_time( 'Y' )
+					);
+				?></p>
 
-			<?php /* If this is a search result */ } elseif (is_search()) { ?>
-			<p><?php printf(__('You have searched the <a href="%1$s/">%2$s</a> blog archives for <strong>&#8216;%3$s&#8217;</strong>. If you are unable to find anything in these search results, you can try one of these links.'), get_bloginfo('url'), get_bloginfo('name'), esc_html( get_search_query() ) ); ?></p>
+			<?php elseif ( is_search() ) : /* If this is a search result */ ?>
+				<p><?php /* translators: 1: site link, 2: search query */
+					printf( __( 'You have searched the %1$s blog archives for <strong>&#8216;%2$s&#8217;</strong>. If you are unable to find anything in these search results, you can try one of these links.' ),
+						sprintf( '<a href="%1$s/">%2$s</a>', get_bloginfo( 'url' ), get_bloginfo( 'name' ) ),
+						esc_html( get_search_query() )
+					); 
+				?></p>
 
-			<?php /* If this set is paginated */ } elseif (isset($_GET['paged']) && !empty($_GET['paged'])) { ?>
-			<p><?php printf(__('You are currently browsing the <a href="%1$s/">%2$s</a> blog archives.'), get_bloginfo('url'), get_bloginfo('name')); ?></p>
+			<?php elseif ( isset( $_GET['paged'] ) && ! empty( $_GET['paged'] ) ) : /* If this set is paginated */ ?>
+				<p><?php /* translators: %s: site link */
+					printf( __( 'You are currently browsing the %s blog archives.' ),
+						sprintf( '<a href="%1$s/">%2$s</a>', get_bloginfo( 'url' ), get_bloginfo( 'name' ) )
+					);
+				?></p>
 
-			<?php } ?>
+			<?php endif; ?>
 
 			</li>
-		<?php }?>
+		<?php endif; ?>
 		</ul>
 		<ul role="navigation">
 			<?php wp_list_pages('title_li=<h2>' . __('Pages') . '</h2>' ); ?>
@@ -63,7 +98,7 @@ _deprecated_file( sprintf( __( 'Theme without %1$s' ), basename(__FILE__) ), '3.
 			<?php wp_list_categories(array('show_count' => 1, 'title_li' => '<h2>' . __('Categories') . '</h2>')); ?>
 		</ul>
 		<ul>
-			<?php /* If this is the frontpage */ if ( is_home() || is_page() ) { ?>
+			<?php if ( is_home() || is_page() ) { /* If this is the frontpage */ ?>
 				<?php wp_list_bookmarks(); ?>
 
 				<li><h2><?php _e('Meta'); ?></h2>
@@ -78,6 +113,6 @@ _deprecated_file( sprintf( __( 'Theme without %1$s' ), basename(__FILE__) ), '3.
 				</li>
 			<?php } ?>
 
-			<?php endif; ?>
+			<?php endif; /* ! dynamic_sidebar() */ ?>
 		</ul>
 	</div>