Changeset 53337

Timestamp:

05/03/2022 02:51:41 PM (4 years ago)

Author:

audrasjb

Message:

General: Add missing strong tag to some error messages.

This changeset adds missing strong tag to some error messages and updates some AJAX messages, for better consistency.

Follow-up to [53118].

Props NekoJonez, audrasjb, ocean90.
Fixes #54437.

Location:

trunk

Files:

• src/wp-admin/includes/ajax-actions.php (modified) (3 diffs)
• src/wp-admin/options-permalink.php (modified) (3 diffs)
• tests/phpunit/tests/ajax/ReplytoComment.php (modified) (1 diff)

trunk/src/wp-admin/includes/ajax-actions.php

@@ -1292 +1292 @@
                 wp_die( 1 );
         } elseif ( in_array( $post->post_status, array( 'draft', 'pending', 'trash' ), true ) ) {
-                wp_die( __( 'Error: You cannot reply to a comment on a draft post.' ) );
+                wp_die( __( 'You cannot reply to a comment on a draft post.' ) );
         }
 
@@ -1322 +1322 @@
 
         if ( '' === $comment_content ) {
-                wp_die( __( 'Error: Please type your comment text.' ) );
+                wp_die( __( 'Please type your comment text.' ) );
         }
 
@@ -1424 +1424 @@
 
         if ( '' === $_POST['content'] ) {
-                wp_die( __( 'Error: Please type your comment text.' ) );
+                wp_die( __( 'Please type your comment text.' ) );
         }
 

trunk/src/wp-admin/options-permalink.php

@@ -369 +369 @@
                                 printf(
                                         /* translators: 1: web.config, 2: Documentation URL, 3: Ctrl + A, 4: ⌘ + A, 5: Element code. */
-                                        __( 'Error: Your %1$s file is not <a href="%2$s">writable</a>, so updating it automatically was not possible. This is the URL rewrite rule you should have in your %1$s file. Click in the field and press %3$s (or %4$s on Mac) to select all. Then insert this rule inside of the %5$s element in %1$s file.' ),
+                                        __( '<strong>Error:</strong> Your %1$s file is not <a href="%2$s">writable</a>, so updating it automatically was not possible. This is the URL rewrite rule you should have in your %1$s file. Click in the field and press %3$s (or %4$s on Mac) to select all. Then insert this rule inside of the %5$s element in %1$s file.' ),
                                         '<code>web.config</code>',
                                         __( 'https://wordpress.org/support/article/changing-file-permissions/' ),
@@ -396 +396 @@
                         printf(
                                 /* translators: 1: Documentation URL, 2: web.config, 3: Ctrl + A, 4: ⌘ + A */
-                                __( 'Error: The root directory of your site is not <a href="%1$s">writable</a>, so creating a file automatically was not possible. This is the URL rewrite rule you should have in your %2$s file. Create a new file called %2$s in the root directory of your site. Click in the field and press %3$s (or %4$s on Mac) to select all. Then insert this code into the %2$s file.' ),
+                                __( '<strong>Error:</strong> The root directory of your site is not <a href="%1$s">writable</a>, so creating a file automatically was not possible. This is the URL rewrite rule you should have in your %2$s file. Create a new file called %2$s in the root directory of your site. Click in the field and press %3$s (or %4$s on Mac) to select all. Then insert this code into the %2$s file.' ),
                                 __( 'https://wordpress.org/support/article/changing-file-permissions/' ),
                                 '<code>web.config</code>',
@@ -427 +427 @@
                 printf(
                         /* translators: 1: .htaccess, 2: Documentation URL, 3: Ctrl + A, 4: ⌘ + A */
-                        __( 'Error: Your %1$s file is not <a href="%2$s">writable</a>, so updating it automatically was not possible. These are the mod_rewrite rules you should have in your %1$s file. Click in the field and press %3$s (or %4$s on Mac) to select all.' ),
+                        __( '<strong>Error:</strong> Your %1$s file is not <a href="%2$s">writable</a>, so updating it automatically was not possible. These are the mod_rewrite rules you should have in your %1$s file. Click in the field and press %3$s (or %4$s on Mac) to select all.' ),
                         '<code>.htaccess</code>',
                         __( 'https://wordpress.org/support/article/changing-file-permissions/' ),

trunk/tests/phpunit/tests/ajax/ReplytoComment.php

@@ -187 +187 @@
                 // Make the request.
                 $this->expectException( 'WPAjaxDieStopException' );
-                $this->expectExceptionMessage( 'Error: You cannot reply to a comment on a draft post.' );
+                $this->expectExceptionMessage( 'You cannot reply to a comment on a draft post.' );
                 $this->_handleAjax( 'replyto-comment' );
         }