Changeset 38040 for trunk/tests/phpunit/tests/meta/registerMeta.php

Timestamp:

07/13/2016 04:12:41 AM (10 years ago)

Author:

jeremyfelt

Message:

Meta: Remove filters when meta is unregistered.

If auth and/or sanitize callbacks are specified in the arguments for
register_meta(), filters are added to handle these callbacks. These
should be removed when calling unregister_meta_key() to avoid
unintentional filtering.

See #35658.

File:

• trunk/tests/phpunit/tests/meta/registerMeta.php (modified) (3 diffs)

trunk/tests/phpunit/tests/meta/registerMeta.php

@@ -33 +33 @@
         }
 
+        public function _new_auth_meta_cb( $allowed, $meta_key, $post_id, $user_id, $cap, $caps ) {
+                return $allowed;
+        }
+
         public function test_register_meta_with_valid_object_type_and_object_subtype_returns_true() {
                 $result = register_meta( 'post', 'flight_number', array( 'object_subtype' => 'post' ) );
@@ -201 +205 @@
                 unregister_meta_key( 'post', 'post', 'flight_number' );
 
-                remove_filter( 'sanitize_post_post_meta_flight_number', array( $this, '_new_sanitize_meta_cb') );
-                remove_filter( 'auth_post_post_meta_flight_number', '__return_true');
-
                 $this->assertTrue( $result );
         }
@@ -213 +214 @@
                 unregister_meta_key( 'post', 'post', 'new_sanitized_key' );
 
-                remove_filter( 'sanitize_post_post_meta_new_sanitized_key', array( $this, '_new_sanitize_meta_cb') );
-                remove_filter( 'auth_post_post_meta_new_sanitized_key', '__return_true');
-
                 $this->assertEquals( 'new_sanitized_key new sanitized', $meta );
+        }
+
+        public function test_register_meta_unregistered_meta_key_removes_sanitize_filter() {
+                register_meta( 'post', 'new_sanitized_key', array( 'object_subtype' => 'post', 'sanitize_callback' => array( $this, '_new_sanitize_meta_cb' ) ) );
+                unregister_meta_key( 'post', 'post', 'new_sanitized_key' );
+
+                $has_filter = has_filter( 'sanitize_post_post_meta_new_sanitized_key', array( $this, '_new_sanitize_meta_cb' ) );
+
+                $this->assertFalse( $has_filter );
+        }
+
+        public function test_register_meta_unregistered_meta_key_removes_auth_filter() {
+                register_meta( 'post', 'new_auth_key', array( 'object_subtype' => 'post', 'auth_callback' => array( $this, '_new_auth_meta_cb' ) ) );
+                unregister_meta_key( 'post', 'post', 'new_auth_key' );
+
+                $has_filter = has_filter( 'auth_post_post_meta_new_auth_key', array( $this, '_new_auth_meta_cb' ) );
+
+                $this->assertFalse( $has_filter );
         }