Interactivity API: fix data-wp-bind skipping valid directives after encountering an invalid one

Change `return` to `continue` in `data_wp_bind_processor` so that when an element has multiple `data-wp-bind` directives and one is invalid (empty suffix or unique ID), the invalid entry is skipped instead of causing the entire function to exit, allowing valid directives on the same element to still be processed.

Props DAreRodz, ozgursar, alexodiy.
Fixes #64518.

git-svn-id: https://develop.svn.wordpress.org/trunk@62070 602fd350-edb4-49c9-b593-d223f7449a82

Author: luisherranz

src/wp-includes/interactivity-api/class-wp-interactivity-api.php

@@ -1026,7 +1026,7 @@ private function data_wp_bind_processor( WP_Interactivity_API_Directives_Process
 			$entries = $this->get_directive_entries( $p, 'bind' );
 			foreach ( $entries as $entry ) {
 				if ( empty( $entry['suffix'] ) || null !== $entry['unique_id'] ) {
-						return;
+						continue;
 				}
 
 				// Skip if the suffix is an event handler.

tests/phpunit/tests/interactivity-api/wpInteractivityAPI-wp-bind.php

@@ -134,6 +134,20 @@ public function test_wp_bind_ignores_empty_bound_attribute() {
 		$this->assertSame( $html, $new_html );
 	}
 
+	/**
+	 * Tests that `data-wp-bind` ignores directives with no suffix but still
+	 * processes valid bind directives on the same element.
+	 *
+	 * @ticket 64518
+	 *
+	 * @covers ::process_directives
+	 */
+	public function test_wp_bind_ignores_empty_suffix_but_processes_valid_binds() {
+		$html    = '<div data-wp-bind="myPlugin::state.id" data-wp-bind--id="myPlugin::state.id">Text</div>';
+		list($p) = $this->process_directives( $html );
+		$this->assertSame( 'some-id', $p->get_attribute( 'id' ) );
+	}
+
 	/**
 	 * Tests that `data-wp-bind` does nothing when referencing non-existent
 	 * references.
@@ -416,4 +430,18 @@ public function test_wp_bind_ignores_unique_ids() {
 		$this->assertNull( $p->get_attribute( 'id' ) );
 		$this->assertNull( $p->get_attribute( 'id---unique-id' ) );
 	}
+
+	/**
+	 * Tests that `data-wp-bind` ignores directives with unique IDs but still
+	 * processes valid bind directives on the same element.
+	 *
+	 * @ticket 64518
+	 *
+	 * @covers ::process_directives
+	 */
+	public function test_wp_bind_ignores_unique_id_but_processes_valid_binds() {
+		$html    = '<div data-wp-bind--id---unique-id="myPlugin::state.id" data-wp-bind--id="myPlugin::state.id">Text</div>';
+		list($p) = $this->process_directives( $html );
+		$this->assertSame( 'some-id', $p->get_attribute( 'id' ) );
+	}
 }