Add skipping expressions in arguments.

Author: kagg-design

WPForms/Sniffs/Comments/ParamTagHooksSniff.php

@@ -119,9 +119,17 @@ private function countArguments( $phpcsFile, $stackPtr ) { // phpcs:ignore Gener
 
 		$currentPosition = $commaPtr + 1;
 		$quantity        = 0;
+		$lookingForComma = false;
 
 		while ( $currentPosition < $lastPosition ) {
-			if ( in_array( $tokens[ $currentPosition ]['code'], [ T_WHITESPACE, T_COMMA ], true ) ) {
+			if ( $tokens[ $currentPosition ]['code'] === T_WHITESPACE ) {
+				$currentPosition ++;
+				continue;
+			}
+
+			if ( $tokens[ $currentPosition ]['code'] === T_COMMA ) {
+				$lookingForComma = false;
+
 				$currentPosition ++;
 				continue;
 			}
@@ -134,13 +142,15 @@ private function countArguments( $phpcsFile, $stackPtr ) { // phpcs:ignore Gener
 				continue;
 			}
 
-			$quantity ++;
-
-			$currentPosition = $phpcsFile->findNext( [ T_COMMA, T_ARRAY, T_OPEN_SHORT_ARRAY, T_OPEN_PARENTHESIS, T_OPEN_SQUARE_BRACKET, T_OPEN_CURLY_BRACKET ], $currentPosition + 1 );
+			$currentPosition ++;
 
-			if ( ! $currentPosition ) {
-				break;
+			if ( $lookingForComma ) {
+				continue;
 			}
+
+			$quantity ++;
+
+			$lookingForComma = true;
 		}
 
 		return $quantity;

WPForms/Tests/TestFiles/Comments/ParamTagHooks.php

@@ -212,6 +212,8 @@ function () {
 /** This filter is documented in wp-includes/post-template.php */
 $content = apply_filters( 'the_content', $content );
 
+// This filter counted args improperly due to presence of the function with parenthesis.
+
 /**
  * Allow modifying the text or url for the full page on the AMP pages.
  *
@@ -235,3 +237,14 @@ function () {
 	$full_page_url,
 	$form_data
 );
+
+// This filter counted args improperly due to expression in the argument.
+
+/**
+ * Allow filtering Rich Text field media cleanup window time.
+ *
+ * @since 1.7.0
+ *
+ * @param int $time Time.
+ */
+$time = (int) apply_filters( 'wpforms_richtext_override_auth_for_ajax_media_calls_time', time() + 1 * DAY_IN_SECONDS );