Apply suggestions from code review

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Author: swissspidy

features/search-replace.feature

@@ -1582,8 +1582,12 @@ Feature: Do global search/replace
     And I run `wp db query "INSERT INTO wp_readonly_test (data) VALUES ('old-value');"`
     And I run `wp db query "CREATE TRIGGER prevent_update BEFORE UPDATE ON wp_readonly_test FOR EACH ROW SIGNAL SQLSTATE '45000' SET MESSAGE_TEXT = 'Table is read-only';"`
 
-    When I try `wp search-replace old-value new-value wp_readonly_test --all-tables-with-prefix`
+    When I try `wp search-replace old-value new-value wp_readonly_test`
     Then STDERR should contain:
       """
       Error updating column 'data' in table 'wp_readonly_test'
       """
+    And STDERR should contain:
+      """
+      Table is read-only
+      """

src/Search_Replace_Command.php

@@ -19,6 +19,14 @@ class Search_Replace_Command extends WP_CLI_Command {
 	 */
 	private $export_handle = false;
 
+	/**
+	 * Tracks table/column combinations that have encountered update errors,
+	 * so we can avoid repeated failing updates and noisy per-row warnings.
+	 *
+	 * @var array
+	 */
+	private $update_error_columns = array();
+
 	/**
 	 * @var int
 	 */
@@ -766,17 +774,36 @@ static function ( $key ) {
 					$replacer->clear_log_data();
 				}
 
-				++$count;
-				if ( ! $this->dry_run ) {
+				// If we've already seen an update error for this table/column and are not in dry-run,
+				// skip further attempts to avoid repeated failures and noisy warnings.
+				if ( ! $this->dry_run && ! empty( $this->update_error_columns[ $table ][ $col ] ) ) {
+					continue;
+				}
+
+				if ( $this->dry_run ) {
+					// In dry-run mode, count replacements once a change has been detected.
+					++$count;
+				} else {
 					$update_where = array();
 					foreach ( (array) $keys as $k => $v ) {
 						$update_where[ $k ] = $v;
 					}
 
-					$wpdb->update( $table, [ $col => $value ], $update_where );
-					if ( $wpdb->last_error ) {
-						WP_CLI::warning( sprintf( "Error updating column '%s' in table '%s': %s", $col, $table, $wpdb->last_error ) );
+					$result = $wpdb->update( $table, array( $col => $value ), $update_where );
+					if ( false === $result ) {
+						if ( empty( $this->update_error_columns[ $table ][ $col ] ) ) {
+							$this->update_error_columns[ $table ][ $col ] = true;
+							if ( $wpdb->last_error ) {
+								WP_CLI::warning( sprintf( "Error updating column '%s' in table '%s': %s", $col, $table, $wpdb->last_error ) );
+							} else {
+								WP_CLI::warning( sprintf( "Error updating column '%s' in table '%s'.", $col, $table ) );
+							}
+						}
+						continue;
 					}
+
+					// Only count successful updates.
+					++$count;
 				}
 			}