Use false === isset() instead of ! isset() for consistency with the rest of the sniff code

rodrigoprimo · rodrigoprimo · commit 4dbe72ada135 · 2025-03-12T15:00:07.000-03:00
diff --git a/WordPress/Sniffs/WP/OptionAutoloadSniff.php b/WordPress/Sniffs/WP/OptionAutoloadSniff.php
@@ -347,7 +347,7 @@ private function check_autoload_value( array $autoload_info, $function_name ) {
 		}
 
 		if ( $param_second_token
-			&& ! isset( Collections::arrayOpenTokensBC()[ $this->tokens[ $param_first_token ]['code'] ] )
+			&& false === isset( Collections::arrayOpenTokensBC()[ $this->tokens[ $param_first_token ]['code'] ] )
 		) {
 			// Bail early if the parameter has two or more non-empty tokens and the second token is
 			// not an array opener as this means an undetermined param value or a value that is not

Original file line number	Diff line number	Diff line change
`@@ -347,7 +347,7 @@ private function check_autoload_value( array $autoload_info, $function_name ) {`
`347`	`347`	`}`
`348`	`348`
`349`	`349`	`if ( $param_second_token`
`350`		`- && ! isset( Collections::arrayOpenTokensBC()[ $this->tokens[ $param_first_token ]['code'] ] )`
	`350`	`+ && false === isset( Collections::arrayOpenTokensBC()[ $this->tokens[ $param_first_token ]['code'] ] )`
`351`	`351`	`) {`
`352`	`352`	`// Bail early if the parameter has two or more non-empty tokens and the second token is`
`353`	`353`	`// not an array opener as this means an undetermined param value or a value that is not`