WPCS 3 changes

woocommerce · Mar 13, 2024 · 1c9df43 · 1c9df43
1 parent 2cf2a34
commit 1c9df43
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 5 deletions.
diff --git a/src/.phpcs.xml.dist b/src/.phpcs.xml.dist
@@ -55,6 +55,12 @@
         <exclude name="Generic.Arrays.DisallowShortArraySyntax"/>
         <exclude name="Generic.Commenting.DocComment.MissingShort"/>
 
+        <!-- WPCS 3 -->
+        <exclude name="Universal.Arrays.DisallowShortArraySyntax.Found"/>
+        <exclude name="WordPress.Security.EscapeOutput.ExceptionNotEscaped"/>
+        <exclude name="Universal.ControlStructures.DisallowLonelyIf.Found"/>
+        <exclude name="Universal.Operators.DisallowShortTernary.Found"/>
+
         <!-- Ignored until this is answered: https://github.com/squizlabs/PHP_CodeSniffer/issues/3570 -->
         <exclude name="Squiz.Commenting.FunctionComment.Missing"/>
 

diff --git a/src/src/Cache.php b/src/src/Cache.php
@@ -72,7 +72,7 @@ public function get( string $key, bool $ignore_expiration = false ) {
 			}
 
 			if ( $ignore_expiration === false && time() > $c['expire'] ) {
-				$deleted ++;
+				++$deleted;
 				unset( $this->cache[ $k ] );
 			}
 		}

diff --git a/src/src/RequestBuilder.php b/src/src/RequestBuilder.php
@@ -273,7 +273,7 @@ public function request(): string {
 
 			if ( $response_status_code === 429 ) {
 				if ( $this->retry_429 > 0 ) {
-					$this->retry_429 --;
+					--$this->retry_429;
 					$sleep_seconds = $this->wait_after_429( $headers );
 					App::make( Output::class )->writeln( sprintf( '<comment>Request failed... Waiting %d seconds and retrying (429 Too many Requests)</comment>', $sleep_seconds ) );
 
@@ -282,7 +282,7 @@ public function request(): string {
 				}
 			} else {
 				if ( $this->retry > 0 ) {
-					$this->retry --;
+					--$this->retry;
 					App::make( Output::class )->writeln( sprintf( '<comment>Request failed... Retrying (HTTP Status Code %s) %s</comment>', $response_status_code, $error_message ) );
 
 					// Between 1 and 5s.

diff --git a/src/src/Upload.php b/src/src/Upload.php
@@ -51,7 +51,7 @@ public function upload_build( int $woo_extension_id, string $extension_slug, str
 		$progress_bar->start();
 
 		while ( ! feof( $file ) ) {
-			$current_chunk ++;
+			++$current_chunk;
 
 			$r = $this->request_builder
 					->with_url( get_manager_url() . '/wp-json/cd/v1/upload-build' )

diff --git a/src/src/bootstrap.php b/src/src/bootstrap.php
@@ -81,7 +81,7 @@ public function getDefaultCommands() {
 if ( in_array( '--json', $GLOBALS['argv'], true ) ) {
 	class QIT_JSON_Filter extends \php_user_filter {
 		public function filter( $in, $out, &$consumed, $closing ): int {
-			while ( $bucket = stream_bucket_make_writeable( $in ) ) { // phpcs:ignore WordPress.CodeAnalysis.AssignmentInCondition.FoundInWhileCondition
+			while ( $bucket = stream_bucket_make_writeable( $in ) ) { // phpcs:ignore WordPress.CodeAnalysis.AssignmentInCondition.FoundInWhileCondition,Generic.CodeAnalysis.AssignmentInCondition.FoundInWhileCondition
 				if ( ! is_null( json_decode( $bucket->data ) ) ) {
 					$consumed += $bucket->datalen;
 					stream_bucket_append( $out, $bucket );