Changeset 56542

Timestamp:

09/07/2023 08:39:07 PM (3 years 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.'