Fatal error in wp_insert_term() when the term name is not a string

src/wp-includes/class-wp-xmlrpc-server.php

@@ -374,6 +374,10 @@ public function login_pass_ok(
 	 * @param string|array $data Escape single string or array of strings.
 	 * @return string|null Returns with string if passed, alters by-reference
 	 *                     when array is passed.
+	 *
+	 * @phpstan-template T of string|array
+	 * @phpstan-param T $data
+	 * @phpstan-param-out T $data
 	 */
 	public function escape( &$data ) {
 		if ( ! is_array( $data ) ) {
@@ -387,6 +391,8 @@ public function escape( &$data ) {
 				$v = wp_slash( $v );
 			}
 		}
+
+		// @phpstan-ignore paramOut.type (The loop only slashes string values in place, so the shape of T is preserved.)
 		return null;
 	}
 
@@ -1658,7 +1664,11 @@ protected function _insert_post( $user, $content_struct ) {
 						return new IXR_Error( 401, __( 'Sorry, you are not allowed to assign a term to one of the given taxonomies.' ) );
 					}
 
-					$term_ids           = $post_data['terms'][ $taxonomy ];
+					$term_ids = $post_data['terms'][ $taxonomy ];
+					if ( ! is_array( $term_ids ) ) {
+						return new IXR_Error( 403, __( 'Invalid term ID.' ) );
+					}
+
 					$terms[ $taxonomy ] = array();
 					foreach ( $term_ids as $term_id ) {
 						$term = get_term_by( 'id', $term_id, $taxonomy );
@@ -1709,12 +1719,21 @@ protected function _insert_post( $user, $content_struct ) {
 					}
 
 					$term_names = $post_data['terms_names'][ $taxonomy ];
+					if ( ! is_array( $term_names ) ) {
+						return new IXR_Error( 401, __( 'Invalid term name.' ) );
+					}
+
 					foreach ( $term_names as $term_name ) {
+						// Numeric names may arrive as <int> or <double> values.
+						if ( ! ( is_string( $term_name ) || is_int( $term_name ) || is_float( $term_name ) ) ) {
+							return new IXR_Error( 401, __( 'Invalid term name.' ) );
+						}
+
 						if ( in_array( $term_name, $ambiguous_terms, true ) ) {
 							return new IXR_Error( 401, __( 'Ambiguous term name used in a hierarchical taxonomy. Please use term ID instead.' ) );
 						}
 
-						$term = get_term_by( 'name', $term_name, $taxonomy );
+						$term = get_term_by( 'name', (string) $term_name, $taxonomy );
 
 						if ( ! $term ) {
 							// Term doesn't exist, so check that the user is allowed to create new terms.
@@ -2198,6 +2217,10 @@ public function wp_newTerm( $args ) {
 		// Hold the data of the term.
 		$term_data = array();
 
+		if ( isset( $content_struct['name'] ) && ! ( is_string( $content_struct['name'] ) || is_int( $content_struct['name'] ) || is_float( $content_struct['name'] ) ) ) {
+			return new IXR_Error( 403, __( 'Invalid term name.' ) );
+		}
+
 		$term_data['name'] = trim( $content_struct['name'] );
 		if ( empty( $term_data['name'] ) ) {
 			return new IXR_Error( 403, __( 'The term name cannot be empty.' ) );
@@ -2316,6 +2339,10 @@ public function wp_editTerm( $args ) {
 		}
 
 		if ( isset( $content_struct['name'] ) ) {
+			if ( ! ( is_string( $content_struct['name'] ) || is_int( $content_struct['name'] ) || is_float( $content_struct['name'] ) ) ) {
+				return new IXR_Error( 403, __( 'Invalid term name.' ) );
+			}
+
 			$term_data['name'] = trim( $content_struct['name'] );
 
 			if ( empty( $term_data['name'] ) ) {
@@ -3541,6 +3568,10 @@ public function wp_newCategory( $args ) {
 			return new IXR_Error( 401, __( 'Sorry, you are not allowed to add a category.' ) );
 		}
 
+		if ( isset( $category['name'] ) && ! ( is_string( $category['name'] ) || is_int( $category['name'] ) || is_float( $category['name'] ) ) ) {
+			return new IXR_Error( 403, __( 'Invalid term name.' ) );
+		}
+
 		/*
 		 * If no slug was provided, make it empty
 		 * so that WordPress will generate one.

src/wp-includes/taxonomy.php

@@ -2545,10 +2545,11 @@ function wp_get_object_terms( $object_ids, $taxonomies, $args = array() ) {
  * @global wpdb $wpdb WordPress database abstraction object.
  *
  * @since 2.3.0
+ * @since 7.2.0 A `$term` that is not a string or number now returns a WP_Error instead of causing a fatal error.
  *
- * @param string       $term     The term name to add.
- * @param string       $taxonomy The taxonomy to which to add the term.
- * @param array|string $args {
+ * @param string|int|float $term     The term name to add. A number is stored as its string form.
+ * @param string           $taxonomy The taxonomy to which to add the term.
+ * @param array|string     $args {
  *     Optional. Array or query string of arguments for inserting a term.
  *
  *     @type string $alias_of    Slug of the term to make this term an alias of.
@@ -2602,7 +2603,14 @@ function wp_insert_term( $term, $taxonomy, $args = array() ) {
 		return new WP_Error( 'invalid_term_id', __( 'Invalid term ID.' ) );
 	}
 
-	if ( '' === trim( $term ) ) {
+	// Null and false are treated as a missing name rather than an invalid one, as they were before.
+	if ( null === $term || false === $term ) {
+		$term = '';
+	} elseif ( ! ( is_string( $term ) || is_int( $term ) || is_float( $term ) ) ) {
+		return new WP_Error( 'invalid_term_name', __( 'Invalid term name.' ) );
+	}
+
+	if ( '' === trim( (string) $term ) ) {
 		return new WP_Error( 'empty_term_name', __( 'A name is required for this term.' ) );
 	}
 
@@ -2957,6 +2965,8 @@ function wp_insert_term( $term, $taxonomy, $args = array() ) {
  * exists under.
  *
  * @since 2.3.0
+ * @since 7.2.0 A term that is an array, or an object without a `__toString()` method, is now skipped
+ *              instead of causing a fatal error.
  *
  * @global wpdb $wpdb WordPress database abstraction object.
  *
@@ -3001,7 +3011,17 @@ function wp_set_object_terms( $object_id, $terms, $taxonomy, $append = false ) {
 	$new_tt_ids = array();
 
 	foreach ( (array) $terms as $term ) {
-		if ( '' === trim( $term ) ) {
+		// Skip values that cannot be a term name or ID, such as arrays.
+		if ( ! is_scalar( $term ) && ! ( is_object( $term ) && method_exists( $term, '__toString' ) ) ) {
+			continue;
+		}
+
+		// Anything other than an integer ID is a name or slug, including objects such as SimpleXMLElement.
+		if ( ! is_int( $term ) ) {
+			$term = (string) $term;
+		}
+
+		if ( '' === trim( (string) $term ) ) {
 			continue;
 		}
 

tests/phpunit/tests/term/wpInsertTerm.php

@@ -86,6 +86,68 @@ public function test_wp_insert_term_term_trims_to_empty_string() {
 		$this->assertSame( 'empty_term_name', $found->get_error_code() );
 	}
 
+	/**
+	 * Tests that a term name that is not a string or number returns a WP_Error instead of causing a fatal error.
+	 *
+	 * @ticket 66108
+	 *
+	 * @dataProvider data_wp_insert_term_invalid_term_name_type
+	 *
+	 * @param mixed            $name                The term name to insert.
+	 * @param non-empty-string $expected_error_code The expected error code.
+	 */
+	public function test_wp_insert_term_invalid_term_name_type( $name, string $expected_error_code ): void {
+		$found = wp_insert_term( $name, 'post_tag' ); // @phpstan-ignore argument.type (Intentionally passing invalid types.)
+
+		$this->assertWPError( $found );
+		$this->assertSame( $expected_error_code, $found->get_error_code() );
+	}
+
+	/**
+	 * Data provider for {@see self::test_wp_insert_term_invalid_term_name_type()}.
+	 *
+	 * @return array<truthy-string, array{ mixed, non-empty-string }>
+	 */
+	public function data_wp_insert_term_invalid_term_name_type(): array {
+		return array(
+			'array'  => array( array( 'foo', 'bar' ), 'invalid_term_name' ),
+			'object' => array( new stdClass(), 'invalid_term_name' ),
+			'true'   => array( true, 'invalid_term_name' ),
+			'false'  => array( false, 'empty_term_name' ),
+			'null'   => array( null, 'empty_term_name' ),
+		);
+	}
+
+	/**
+	 * Tests that a numeric term name is inserted and stored as its string form.
+	 *
+	 * @ticket 66108
+	 *
+	 * @dataProvider data_wp_insert_term_numeric_term_name
+	 *
+	 * @param int|float $name The term name to insert.
+	 */
+	public function test_wp_insert_term_numeric_term_name( $name ): void {
+		$found = wp_insert_term( $name, 'post_tag' );
+
+		$this->assertNotWPError( $found );
+		$term = get_term( $found['term_id'], 'post_tag' );
+		$this->assertInstanceOf( WP_Term::class, $term );
+		$this->assertSame( (string) $name, $term->name );
+	}
+
+	/**
+	 * Data provider for {@see self::test_wp_insert_term_numeric_term_name()}.
+	 *
+	 * @return array<truthy-string, array{ int|float }>
+	 */
+	public function data_wp_insert_term_numeric_term_name(): array {
+		return array(
+			'integer' => array( 2024 ),
+			'float'   => array( 1.5 ),
+		);
+	}
+
 	public function test_wp_insert_term_parent_does_not_exist() {
 		$found = wp_insert_term(
 			'foo',

tests/phpunit/tests/term/wpSetObjectTerms.php

@@ -7,6 +7,10 @@
  */
 class Tests_Term_WpSetObjectTerms extends WP_UnitTestCase {
 	protected static $taxonomy = 'category';
+
+	/**
+	 * @var int[]
+	 */
 	protected static $post_ids = array();
 	protected static $term_ids = array();
 
@@ -15,6 +19,75 @@ public static function wpSetUpBeforeClass( WP_UnitTest_Factory $factory ) {
 		self::$term_ids = $factory->term->create_many( 5, array( 'taxonomy' => self::$taxonomy ) );
 	}
 
+	/**
+	 * Tests that a term that cannot be a term name or ID is skipped and the other terms are still set.
+	 *
+	 * @ticket 66108
+	 *
+	 * @dataProvider data_invalid_term_type_is_skipped
+	 *
+	 * @param mixed $term The invalid term to set ahead of a valid one.
+	 */
+	public function test_invalid_term_type_is_skipped( $term ): void {
+		$post_id = self::$post_ids[0];
+
+		$result = wp_set_object_terms( $post_id, array( $term, 'new' ), 'post_tag' );
+
+		$this->assertIsArray( $result );
+		$this->assertSame( array( 'new' ), wp_get_object_terms( $post_id, 'post_tag', array( 'fields' => 'names' ) ) );
+	}
+
+	/**
+	 * Data provider for {@see self::test_invalid_term_type_is_skipped()}.
+	 *
+	 * @return array<truthy-string, array{ mixed }>
+	 */
+	public static function data_invalid_term_type_is_skipped(): array {
+		return array(
+			'array'  => array( array( 'foo' ) ),
+			'object' => array( new stdClass() ),
+		);
+	}
+
+	/**
+	 * Tests that an object with a `__toString()` method, such as a SimpleXMLElement, is used by its string value.
+	 *
+	 * @ticket 66108
+	 */
+	public function test_object_with_to_string_method_is_used_as_term_name(): void {
+		$post_id = self::$post_ids[0];
+		$term_id = self::factory()->term->create(
+			array(
+				'taxonomy' => 'post_tag',
+				'name'     => 'existing',
+			)
+		);
+		$term    = new class() {
+			public function __toString(): string {
+				return 'existing';
+			}
+		};
+
+		$result = wp_set_object_terms( $post_id, array( $term ), 'post_tag' );
+
+		$this->assertIsArray( $result );
+		$this->assertSame( array( $term_id ), wp_get_object_terms( $post_id, 'post_tag', array( 'fields' => 'ids' ) ) );
+	}
+
+	/**
+	 * Tests that null and false terms are skipped rather than rejected.
+	 *
+	 * @ticket 66108
+	 */
+	public function test_null_and_false_terms_are_skipped(): void {
+		$post_id = self::$post_ids[0];
+
+		$result = wp_set_object_terms( $post_id, array( 'foo', null, false ), 'post_tag' );
+
+		$this->assertNotWPError( $result );
+		$this->assertSame( array( 'foo' ), wp_get_object_terms( $post_id, 'post_tag', array( 'fields' => 'names' ) ) );
+	}
+
 	/**
 	 * @ticket 26570
 	 */

tests/phpunit/tests/xmlrpc/mw/editPost.php

@@ -30,6 +30,26 @@ public function test_edit_own_post() {
 		$this->assertSame( $new_title, $out->post_title );
 	}
 
+	/**
+	 * Tests that a keyword that is not a string or number is skipped and the other keywords replace the existing tags.
+	 *
+	 * @ticket 66108
+	 */
+	public function test_invalid_keyword(): void {
+		$author_id = $this->make_user_by_role( 'author' );
+		$post_id   = self::factory()->post->create( array( 'post_author' => $author_id ) );
+		$this->assertIsArray( wp_set_post_tags( $post_id, array( 'existing' ) ) );
+
+		$post   = array(
+			'title'       => 'Test',
+			'mt_keywords' => array( 'foo', array( 'bar' ) ),
+		);
+		$result = $this->myxmlrpcserver->mw_editPost( array( $post_id, 'author', 'author', $post ) );
+
+		$this->assertNotIXRError( $result );
+		$this->assertSame( array( 'foo' ), wp_get_post_tags( $post_id, array( 'fields' => 'names' ) ) );
+	}
+
 	public function test_capable_edit_others_post() {
 		$this->make_user_by_role( 'editor' );
 		$contributor_id = $this->make_user_by_role( 'contributor' );

tests/phpunit/tests/xmlrpc/mw/newPost.php

@@ -21,6 +21,24 @@ public function test_incapable_user() {
 		$this->assertSame( 401, $result->code );
 	}
 
+	/**
+	 * Tests that a keyword that is not a string or number is skipped and the other keywords are kept.
+	 *
+	 * @ticket 66108
+	 */
+	public function test_invalid_keyword(): void {
+		$this->make_user_by_role( 'author' );
+
+		$post   = array(
+			'title'       => 'Test',
+			'mt_keywords' => array( 'foo', array( 'bar' ) ),
+		);
+		$result = $this->myxmlrpcserver->mw_newPost( array( 1, 'author', 'author', $post ) );
+
+		$this->assertNotIXRError( $result );
+		$this->assertSame( array( 'foo' ), wp_get_post_tags( (int) $result, array( 'fields' => 'names' ) ) );
+	}
+
 	public function test_no_content() {
 		$this->make_user_by_role( 'author' );