[wp-trac] [WordPress Trac] #33073: Some strings need "no HTML entities" translator comments

[WordPress Trac]

#33073: Some strings need "no HTML entities" translator comments
-------------------------------------+------------------------------
 Reporter:  xibe                     |       Owner:
     Type:  enhancement              |      Status:  new
 Priority:  normal                   |   Milestone:  Awaiting Review
Component:  I18N                     |     Version:  trunk
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |     Focuses:
-------------------------------------+------------------------------

Comment (by xibe):

 Note: for some strings, I chose to expand the code block.

 For instance:


 {{{
 if ( EMPTY_TRASH_DAYS )
         $notify_message .= sprintf( __('Trash it: %s'),
 admin_url("comment.php?action=trash&c=$comment_id") ) . "\r\n";
 else
         $notify_message .= sprintf( __('Delete it: %s'),
 admin_url("comment.php?action=delete&c=$comment_id") ) . "\r\n";

 }}}

 ...is turned into:


 {{{
 if ( EMPTY_TRASH_DAYS ) {
         /* translators: Do not use HTML entities ( , etc.): this
 string is used in e-mails. */
         $notify_message .= sprintf( __('Trash it: %s'),
 admin_url("comment.php?action=trash&c=$comment_id") ) . "\r\n";
 }
 else {
         /* translators: Do not use HTML entities ( , etc.): this
 string is used in e-mails. */
         $notify_message .= sprintf( __('Delete it: %s'),
 admin_url("comment.php?action=delete&c=$comment_id") ) . "\r\n";
 }
 }}}

 The line I add is a comment, so I'm unsure if it's necessary to add curly
 braces to mark the code block, but I want to play it safe.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/33073#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform