Ticket #30875: 30875.5.diff

src/wp-includes/plugin.php

@@ -81 +81 @@
 function add_filter( $tag, $function_to_add, $priority = 10, $accepted_args = 1 ) {
         global $wp_filter, $merged_filters;
 
-        $idx = _wp_filter_build_unique_id($tag, $function_to_add, $priority);
-        $wp_filter[$tag][$priority][$idx] = array('function' => $function_to_add, 'accepted_args' => $accepted_args);
+        if ( is_array( $priority ) || is_object( $priority ) ) {
+                return false;
+        }
+
+        $idx = _wp_filter_build_unique_id( $tag, $function_to_add, $priority );
+        $wp_filter[$tag][$priority][$idx] = array( 'function' => $function_to_add, 'accepted_args' => $accepted_args );
         unset( $merged_filters[ $tag ] );
         return true;
 }

tests/phpunit/tests/actions.php

@@ -143 +143 @@
                 $this->assertEquals($expected, $a->get_events());
         }
 
+        /**
+         * @ticket 30875
+         */
+        function test_add_filter_priority_value() {
+                $a = new MockAction();
+
+                // Test integers
+                add_action( '_test_int', array( &$a, 'action' ), 10 );
+                do_action( '_test_int' );
+                $this->assertEquals( 10, has_action( '_test_int', array( &$a, 'action' ) ) );
+
+                // Test integer as string
+                add_action( '_test_int_as_string', array( &$a, 'action' ), '9' );
+                do_action( '_test_int_as_string' );
+                $this->assertEquals( 9, has_action( '_test_int_as_string', array( &$a, 'action' ) ) );
+
+                // Test float
+                add_action( '_test_float', array( &$a, 'action' ), 8.1 );
+                do_action( '_test_float' );
+                $this->assertEquals( 8, has_action( '_test_float', array( &$a, 'action' ) ) );
+
+                // Test float as string
+                add_action( '_test_float_as_string', array( &$a, 'action' ), '7.2' );
+                do_action( '_test_float_as_string' );
+                $this->assertEquals( '7.2', has_action( '_test_float_as_string', array( &$a, 'action' ) ) );
+
+                // Test string
+                add_action( '_test_string', array( &$a, 'action' ), 'micky' );
+                do_action( '_test_string' );
+                $this->assertEquals( 'micky', has_action( '_test_string', array( &$a, 'action' ) ) );
+
+                // Test boolean true
+                add_action( '_test_bool_true', array( &$a, 'action' ), true );
+                do_action( '_test_bool_true' );
+                $this->assertEquals( 1, has_action( '_test_bool_true', array( &$a, 'action' ) ) );
+                
+                // Test boolean false
+                add_action( '_test_bool_false', array( &$a, 'action' ), false );
+                do_action( '_test_bool_false' );
+                $this->assertEquals( 0, has_action( '_test_bool_false', array( &$a, 'action' ) ) );
+
+                // Test boolean as string
+                add_action( '_test_bool_as_string', array( &$a, 'action' ), 'true' );
+                do_action( '_test_bool_as_string' );
+                $this->assertEquals( 'true', has_action( '_test_bool_as_string', array( &$a, 'action' ) ) );
+                
+                // Test array
+                add_action( '_test_array', array( &$a, 'action' ), array() );
+                do_action( '_test_array' );
+                $this->assertEquals( false, has_action( '_test_array', array( &$a, 'action' ) ) );
+                
+                // Test object
+                add_action( '_test_object', array( &$a, 'action' ), new stdClass() );
+                do_action( '_test_object' );
+                $this->assertEquals( false, has_action( '_test_object', array( &$a, 'action' ) ) );
+        }
+
         function test_did_action() {
                 $tag1 = rand_str();
                 $tag2 = rand_str();