Changeset 58409 for trunk/src/wp-includes/functions.php

Timestamp:

06/13/2024 08:48:33 PM (6 months ago)

Author:

hellofromTonya

Message:

Code Modernization: Use wp_trigger_error() instead of trigger_error().

Replaces trigger_error() with wp_trigger_error().

The usage of wp_trigger_error() avoids generating E_USER family errors unless WP_DEBUG is on. In doing so, users should not see these messages in normal production.

Notes:

• Removes E_USER_NOTICE when passed as an argumnent, as it's the default error level.
• An empty string is passed for the function name when its name is already in the error message or does not add value to the error message.
• Externally maintained libraries are not included.

Follow-up to [55204], [25956], [29630], [38883], [52062], [52049], [54272], [38883], [55245], [51599], [14452], [38883], [24976].

Props prasadkarmalkar, rajinsharwar, thelovekesh, hellofromTonya, swissspidy.
Fixes #59652.

File:

• trunk/src/wp-includes/functions.php (modified) (3 diffs)

trunk/src/wp-includes/functions.php

@@ -4612 +4612 @@
 
     if ( ! $filename ) {
-        trigger_error(
+        wp_trigger_error(
+            __FUNCTION__,
             sprintf(
                 /* translators: %s: Path to the JSON file. */
@@ -4626 +4627 @@
 
     if ( JSON_ERROR_NONE !== json_last_error() ) {
-        trigger_error(
+        wp_trigger_error(
+            __FUNCTION__,
             sprintf(
                 /* translators: 1: Path to the JSON file, 2: Error message. */
@@ -8758 +8760 @@
 function clean_dirsize_cache( $path ) {
     if ( ! is_string( $path ) || empty( $path ) ) {
-        trigger_error(
+        wp_trigger_error(
+            '',
             sprintf(
                 /* translators: 1: Function name, 2: A variable type, like "boolean" or "integer". */