Merge pull request #314 from chubes4/fix/replace-function-false-positive

swissspidy · web-flow · commit 768a128641e7 · 2026-03-08T22:45:33.000+01:00
diff --git a/src/DB_Command.php b/src/DB_Command.php
@@ -538,7 +538,7 @@ public function query( $args, $assoc_args ) {
 			$assoc_args['execute'] = $this->get_sql_mode_query( $assoc_args ) . $assoc_args['execute'];
 		}
 
-		$is_row_modifying_query = isset( $assoc_args['execute'] ) && preg_match( '/\b(UPDATE|DELETE|INSERT|REPLACE|LOAD DATA)\b/i', $assoc_args['execute'] );
+		$is_row_modifying_query = isset( $assoc_args['execute'] ) && preg_match( '/\b(UPDATE|DELETE|INSERT|REPLACE(?!\s*\()|LOAD DATA)\b/i', $assoc_args['execute'] );
 
 		if ( $is_row_modifying_query ) {
 			$assoc_args['execute'] .= '; SELECT ROW_COUNT();';

Original file line number	Diff line number	Diff line change
`@@ -538,7 +538,7 @@ public function query( $args, $assoc_args ) {`
`538`	`538`	`$assoc_args['execute'] = $this->get_sql_mode_query( $assoc_args ) . $assoc_args['execute'];`
`539`	`539`	`}`
`540`	`540`
`541`		`- $is_row_modifying_query = isset( $assoc_args['execute'] ) && preg_match( '/\b(UPDATE\|DELETE\|INSERT\|REPLACE\|LOAD DATA)\b/i', $assoc_args['execute'] );`
	`541`	`+ $is_row_modifying_query = isset( $assoc_args['execute'] ) && preg_match( '/\b(UPDATE\|DELETE\|INSERT\|REPLACE(?!\s*\()\|LOAD DATA)\b/i', $assoc_args['execute'] );`
`542`	`542`
`543`	`543`	`if ( $is_row_modifying_query ) {`
`544`	`544`	`$assoc_args['execute'] .= '; SELECT ROW_COUNT();';`