Changeset 61118 for trunk/tests/phpunit/tests/hooks/hasFilter.php

Timestamp:

11/03/2025 11:45:40 PM (2 months ago)

Author:

westonruter

Message:

Plugins: Add missing $priority parameter to has_filter() and has_action().

This brings has_filter()/has_action() in parity with add_filter()/add_action() and remove_filter()/remove_action(), all of which support a $priority parameter.

Props westonruter, swissspidy.
Fixes #64186.
See #64178.

File:

• trunk/tests/phpunit/tests/hooks/hasFilter.php (modified) (1 diff)

trunk/tests/phpunit/tests/hooks/hasFilter.php

@@ -9 +9 @@
 class Tests_Hooks_HasFilter extends WP_UnitTestCase {
 
+    /**
+     * @ticket 64186
+     */
     public function test_has_filter_with_function() {
         $callback      = '__return_null';
         $hook          = new WP_Hook();
         $hook_name     = __FUNCTION__;
-        $priority      = 1;
+        $priority_a    = 1;
+        $priority_b    = 10;
         $accepted_args = 2;
 
-        $hook->add_filter( $hook_name, $callback, $priority, $accepted_args );
+        $hook->add_filter( $hook_name, $callback, $priority_a, $accepted_args );
+        $hook->add_filter( $hook_name, $callback, $priority_b, $accepted_args );
 
-        $this->assertSame( $priority, $hook->has_filter( $hook_name, $callback ) );
+        $this->assertSame( $priority_a, $hook->has_filter( $hook_name, $callback ) );
+        $this->assertTrue( $hook->has_filter( $hook_name, $callback, $priority_a ) );
+        $this->assertTrue( $hook->has_filter( $hook_name, $callback, $priority_b ) );
+        $hook->remove_filter( $hook_name, $callback, $priority_a );
+        $this->assertSame( $priority_b, $hook->has_filter( $hook_name, $callback ) );
     }