Changeset 56276

Timestamp:

07/21/2023 01:31:17 PM (18 months ago)

Author:

SergeyBiryukov

Message:

I18N: Add missing translator comment in WP_Upgrader::generic_strings().

This resolves a WPCS warning:

A gettext call containing placeholders was found, but was not accompanied by a "translators:" comment on the line above to clarify the meaning of the placeholders.

Includes moving wp-content out of the translatable string in a similar message in _wp_delete_all_temp_backups().

Follow-up to [55720], [56117].

Props jrf.
See #58831.

Location:

trunk/src

Files:

• wp-admin/includes/class-wp-upgrader.php (modified) (1 diff)
• wp-includes/update.php (modified) (1 diff)

trunk/src/wp-admin/includes/class-wp-upgrader.php

@@ -190 +190 @@
         $this->strings['fs_error']          = __( 'Filesystem error.' );
         $this->strings['fs_no_root_dir']    = __( 'Unable to locate WordPress root directory.' );
+        /* translators: %s: Directory name. */
         $this->strings['fs_no_content_dir'] = sprintf( __( 'Unable to locate WordPress content directory (%s).' ), 'wp-content' );
         $this->strings['fs_no_plugins_dir'] = __( 'Unable to locate WordPress plugin directory.' );

trunk/src/wp-includes/update.php

@@ -1120 +1120 @@
 
     if ( ! $wp_filesystem->wp_content_dir() ) {
-        return new WP_Error( 'fs_no_content_dir', __( 'Unable to locate WordPress content directory (wp-content).' ) );
+        return new WP_Error(
+            'fs_no_content_dir',
+            /* translators: %s: Directory name. */
+            sprintf( __( 'Unable to locate WordPress content directory (%s).' ), 'wp-content' )
+        );
     }