Changeset 56542

Timestamp:

09/07/2023 08:39:07 PM (13 months ago)

Author:

hellofromTonya

Message:

Code Modernization: Use wp_trigger_error() in WP_List_Table magic methods.

Replaces trigger_error() with wp_trigger_error() in each of the WP_List_Table magic methods.

[56349] added the dynamic properties deprecation messages to the __get(), __set(), __isset(), __unset() magic methods. Since that commit, wp_trigger_error() was introduced (see [56530]) as a wrapper for trigger_error().

Follow-up to [56349], [56356], [56530].

See #58896, #57686.

Location:

trunk

Files:

• src/wp-admin/includes/class-wp-list-table.php (modified) (4 diffs)
• tests/phpunit/tests/admin/wpListTable.php (modified) (4 diffs)

trunk/src/wp-admin/includes/class-wp-list-table.php

@@ -187 +187 @@
         }
 
-        trigger_error(
+        wp_trigger_error(
+            __METHOD__,
             "The property `{$name}` is not declared. Getting a dynamic property is " .
             'deprecated since version 6.4.0! Instead, declare the property on the class.',
@@ -210 +211 @@
         }
 
-        trigger_error(
+        wp_trigger_error(
+            __METHOD__,
             "The property `{$name}` is not declared. Setting a dynamic property is " .
             'deprecated since version 6.4.0! Instead, declare the property on the class.',
@@ -231 +233 @@
         }
 
-        trigger_error(
+        wp_trigger_error(
+            __METHOD__,
             "The property `{$name}` is not declared. Checking `isset()` on a dynamic property " .
             'is deprecated since version 6.4.0! Instead, declare the property on the class.',
@@ -253 +256 @@
         }
 
-        trigger_error(
+        wp_trigger_error(
+            __METHOD__,
             "A property `{$name}` is not declared. Unsetting a dynamic property is " .
             'deprecated since version 6.4.0! Instead, declare the property on the class.',

trunk/tests/phpunit/tests/admin/wpListTable.php

@@ -390 +390 @@
         $this->expectDeprecation();
         $this->expectDeprecationMessage(
+            'WP_List_Table::__get(): ' .
             'The property `undeclared_property` is not declared. Getting a dynamic property is ' .
             'deprecated since version 6.4.0! Instead, declare the property on the class.'
@@ -419 +420 @@
         $this->expectDeprecation();
         $this->expectDeprecationMessage(
+            'WP_List_Table::__set(): ' .
             'The property `undeclared_property` is not declared. Setting a dynamic property is ' .
             'deprecated since version 6.4.0! Instead, declare the property on the class.'
@@ -451 +453 @@
         $this->expectDeprecation();
         $this->expectDeprecationMessage(
+            'WP_List_Table::__isset(): ' .
             'The property `undeclared_property` is not declared. Checking `isset()` on a dynamic property ' .
             'is deprecated since version 6.4.0! Instead, declare the property on the class.'
@@ -478 +481 @@
         $this->expectDeprecation();
         $this->expectDeprecationMessage(
+            'WP_List_Table::__unset(): ' .
             'A property `undeclared_property` is not declared. Unsetting a dynamic property is ' .
             'deprecated since version 6.4.0! Instead, declare the property on the class.'