Changeset 53337

Timestamp:

05/03/2022 02:51:41 PM (3 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' );
     }