Ticket #43559: 43559.diff

src/wp-admin/includes/ajax-actions.php

@@ -1433 +1433 @@
                 if ( '' == trim( $key ) ) {
                         wp_die( __( 'Please provide a custom field name.' ) );
                 }
-                if ( '' == trim( $value ) ) {
-                        wp_die( __( 'Please provide a custom field value.' ) );
-                }
                 if ( ! $meta = get_metadata_by_mid( 'post', $mid ) ) {
                         wp_die( 0 ); // if meta doesn't exist
                 }

src/wp-admin/includes/post.php

@@ -861 +861 @@
                 $metavalue = trim( $metavalue );
         }
 
-        if ( ( '0' === $metavalue || ! empty( $metavalue ) ) && ( ( ( '#NONE#' != $metakeyselect ) && ! empty( $metakeyselect ) ) || ! empty( $metakeyinput ) ) ) {
+        if ( ( ( '#NONE#' != $metakeyselect ) && ! empty( $metakeyselect ) ) || ! empty( $metakeyinput ) ) {
                 /*
                  * We have a key/value pair. If both the select and the input
                  * for the key have data, the input takes precedence.

tests/phpunit/tests/admin/includesPost.php

@@ -748 +748 @@
                 $this->assertSame( $p, post_exists( $title, $content, $date ) );
         }
 
+        /**
+         * @ticket 43559
+         */
+        public function test_post_add_meta_empty_is_allowed() {
+                $p     = self::factory()->post->create();
+
+                $_POST = array(
+                        'metakeyinput' => 'testkey',
+                        'metavalue'    => '',
+                );
+
+                wp_set_current_user( self::$admin_id );
+
+                $this->assertNotFalse( add_meta( $p ) );
+                $this->assertEquals( '', get_post_meta( $p, 'testkey', true ) );
+        }
 }

new file tests/phpunit/tests/ajax/AddMeta.php

@@ +1 @@
+<?php
+
+/**
+ * Admin ajax functions to be tested
+ */
+require_once( ABSPATH . 'wp-admin/includes/ajax-actions.php' );
+
+/**
+ * Testing Add Meta AJAX functionality.
+ *
+ * @group ajax
+ */
+class Tests_Ajax_AddMeta extends WP_Ajax_UnitTestCase {
+        /**
+         * @ticket 43559
+         */
+        public function test_post_add_meta_empty_is_allowed_ajax() {
+                $p = self::factory()->post->create();
+
+                // Become an administrator.
+                $this->_setRole( 'administrator' );
+
+                $_POST = array(
+                        'post_id'      => $p,
+                        'metakeyinput' => 'testkey',
+                        'metavalue'    => '',
+                        '_ajax_nonce-add-meta' => wp_create_nonce( 'add-meta' ),
+                );
+
+                // Make the request.
+                try {
+                        $this->_handleAjax( 'add-meta' );
+                } catch ( WPAjaxDieContinueException $e ) {
+                        unset( $e );
+                }
+
+                $this->assertEquals( '', get_post_meta( $p, 'testkey', true ) );
+        }
+
+        /**
+         * @ticket 43559
+         */
+        public function test_post_update_meta_empty_is_allowed_ajax() {
+                $p = self::factory()->post->create();
+
+                $m = add_post_meta( $p, 'testkey', 'hello' );
+
+                // Become an administrator.
+                $this->_setRole( 'administrator' );
+
+                $_POST = array(
+                        '_ajax_nonce-add-meta' => wp_create_nonce( 'add-meta' ),
+                        'post_id' => $p,
+                        'meta' => array(
+                                $m => array(
+                                        'key'   => 'testkey',
+                                        'value' => '',
+                                ),
+                        ),
+                );
+
+                // Make the request.
+                try {
+                        $this->_handleAjax( 'add-meta' );
+                } catch ( WPAjaxDieContinueException $e ) {
+                        unset( $e );
+                }
+
+                $this->assertEquals( '', get_post_meta( $p, 'testkey', true ) );
+        }
+}