dev: fix reported PHPStan v2 errors

Author: Soare-Robert-Daniel

inc/admin.php

@@ -576,15 +576,11 @@ public function inline_bootstrap_script() {
 	/**
 	 * Add settings links in the plugin listing page.
 	 *
-	 * @param array $links Old plugin links.
+	 * @param string[] $links Old plugin links.
 	 *
-	 * @return array Altered links.
+	 * @return string[] Altered links.
 	 */
 	public function add_action_links( $links ) {
-		if ( ! is_array( $links ) ) {
-			return $links;
-		}
-
 		return array_merge(
 			$links,
 			[
@@ -599,7 +595,7 @@ public function add_action_links( $links ) {
 	 * @return bool Should show?
 	 */
 	public function should_show_notice() {
-		if ( ( defined( 'DOING_AJAX' ) && DOING_AJAX ) ) {
+		if ( wp_doing_ajax() ) {
 			return false;
 		}
 
@@ -676,7 +672,7 @@ public function add_notice_upgrade() {
 	 */
 	public function should_show_upgrade() {
 		$current_screen = get_current_screen();
-		if ( ( defined( 'DOING_AJAX' ) && DOING_AJAX ) ||
+		if ( wp_doing_ajax() ||
 			is_network_admin() ||
 			! current_user_can( 'manage_options' ) ||
 			! $this->settings->is_connected() ||
@@ -1003,7 +999,7 @@ public function maybe_redirect() {
 			return;
 		}
 
-		if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
+		if ( wp_doing_ajax() ) {
 			return;
 		}
 

inc/api.php

@@ -160,9 +160,7 @@ public function request( $path, $method = 'GET', $params = [], $extra_headers =
 		if ( ! empty( $this->api_key ) ) {
 			$headers['Authorization'] = 'Bearer ' . $this->api_key;
 		}
-		if ( is_array( $headers ) ) {
-			$headers = array_merge( $headers, $extra_headers );
-		}
+		$headers = array_merge( $headers, $extra_headers );
 		$url  = trailingslashit( $this->api_root ) . ltrim( $path, '/' );
 		// If there is a extra, add that as a url var.
 		if ( 'GET' === $method && ! empty( $params ) ) {

inc/app_replacer.php

@@ -490,7 +490,7 @@ public static function listen_to_sizes( $value, $orig_w, $orig_h, $dest_w, $dest
 	/**
 	 * Extract domains and use them as keys for fast processing.
 	 *
-	 * @param array $urls Input urls.
+	 * @param array|null $urls Input urls.
 	 *
 	 * @return array Array of domains as keys.
 	 */
@@ -529,7 +529,7 @@ function ( $value ) {
 	/**
 	 * Check if we can replace the url.
 	 *
-	 * @param string $url Url to change.
+	 * @param string|mixed $url Url to change.
 	 *
 	 * @return bool Either we can replace this url or not.
 	 */
@@ -668,7 +668,7 @@ protected function get_optimized_image_url( $url, $width, $height, $resize = []
 			->width( $width )
 			->height( $height );
 
-		if ( is_array( $resize ) && ! empty( $resize['type'] ) ) {
+		if ( ! empty( $resize['type'] ) ) {
 			$optimized_image->resize( $resize['type'], $resize['gravity'] ?? Position::CENTER, $resize['enlarge'] ?? false );
 
 		}

inc/compatibilities/pinterest.php

@@ -31,7 +31,7 @@ public function should_load() {
 			$selectors_array[] = '.heateorSssSharing.heateorSssPinterestBackground';
 			$load = true;
 		}
-		$this->selectors = implode( ', ', array_filter( $selectors_array ) );
+		$this->selectors = implode( ', ', $selectors_array );
 		return $load;
 	}
 

inc/conflicts/conflicting_plugins.php

@@ -158,7 +158,7 @@ public function dismiss_conflicting_plugins() {
 	 * @return bool Should show?
 	 */
 	public function should_show_notice() {
-		if ( ( defined( 'DOING_AJAX' ) && DOING_AJAX ) ) {
+		if ( wp_doing_ajax() ) {
 			return false;
 		}
 

inc/manager.php

@@ -435,7 +435,7 @@ public function replace_content( $html, $partial = false ) {
 					$hmac = wp_hash( $profile_id . $time, 'nonce' );
 					$js_optimizer = str_replace(
 						[ Profile::PLACEHOLDER, Profile::PLACEHOLDER_MISSING, Profile::PLACEHOLDER_TIME, Profile::PLACEHOLDER_HMAC ],
-						[ $profile_id, implode( ',', $missing ), $time, $hmac ],
+						[ $profile_id, implode( ',', $missing ), strval( $time ), $hmac ],
 						$js_optimizer
 					);
 					$html = str_replace( Optml_Admin::get_optimizer_script( true ), $js_optimizer, $html );

inc/media_offload.php

@@ -376,18 +376,15 @@ public static function get_image_size_from_width( $sizes, $width, $filename, $ju
 	/**
 	 * Replace image URLs in the srcset attributes.
 	 *
-	 * @param array  $sources Array of image sources.
-	 * @param array  $size_array Array of width and height values in pixels (in that order).
-	 * @param string $image_src The 'src' of the image.
-	 * @param array  $image_meta The image meta data as returned by 'wp_get_attachment_metadata()'.
-	 * @param int    $attachment_id Image attachment ID.
+	 * @param array<int, array{url: string, descriptor: string, value: int}> $sources Array of image sources.
+	 * @param array{0: int, 1: int}                                          $size_array Array of width and height values in pixels (in that order).
+	 * @param string                                                         $image_src The 'src' of the image.
+	 * @param array<string, mixed>                                           $image_meta The image meta data as returned by 'wp_get_attachment_metadata()'.
+	 * @param int                                                            $attachment_id Image attachment ID or 0.
 	 *
 	 * @return array
 	 */
 	public function calculate_image_srcset( $sources, $size_array, $image_src, $image_meta, $attachment_id ) {
-		if ( ! is_array( $sources ) ) {
-			return $sources;
-		}
 
 		if ( $this->is_legacy_offloaded_attachment( $attachment_id ) ) {
 			if ( ! Optml_Media_Offload::is_uploaded_image( $image_src ) || ! isset( $image_meta['file'] ) || ! Optml_Media_Offload::is_uploaded_image( $image_meta['file'] ) ) {

inc/media_rename/attachment_db_renamer.php

@@ -551,7 +551,7 @@ private function replace_in_data( $data, $old_url, $new_url ) {
 	/**
 	 * Check if a string is serialized
 	 *
-	 * @param string $data String to check.
+	 * @param string|mixed $data String to check.
 	 *
 	 * @return bool True if serialized
 	 */

inc/rest.php

@@ -631,7 +631,6 @@ private function fetch_sample_image() {
 	/**
 	 * Disconnect from optimole service.
 	 *
-	 * @SuppressWarnings(PHPMD.UnusedFormalParameter)
 	 * @param WP_REST_Request $request disconnect rest request.
 	 */
 	public function disconnect( WP_REST_Request $request ) {
@@ -784,7 +783,7 @@ public function check_redirects( WP_REST_Request $request ) {
 
 			if ( $processed_images > 0 ) {
 				$response = wp_remote_get( $value['src'][ rand( 0, $processed_images - 1 ) ], $args );
-				if ( ! is_wp_error( $response ) && is_array( $response ) ) {
+				if ( ! is_wp_error( $response ) ) {
 					$headers     = $response['headers']; // array of http header lines
 					$status_code = $response['response']['code'];
 					if ( $status_code === 301 ) {

inc/tag_replacer.php

@@ -430,20 +430,15 @@ public function regular_tag_replace( $new_tag, $original_url, $new_url, $optml_a
 	/**
 	 * Replace image URLs in the srcset attributes and in case there is a resize in action, also replace the sizes.
 	 *
-	 * @param array  $sources Array of image sources.
-	 * @param array  $size_array Array of width and height values in pixels (in that order).
-	 * @param string $image_src The 'src' of the image.
-	 * @param array  $image_meta The image meta data as returned by 'wp_get_attachment_metadata()'.
-	 * @param int    $attachment_id Image attachment ID.
+	 * @param array<int, array{url: string, descriptor: string, value: int}> $sources Array of image sources.
+	 * @param array{0: int, 1: int}|int[]                                    $size_array Array of width and height values in pixels (in that order).
+	 * @param string                                                         $image_src The 'src' of the image.
+	 * @param array<string, mixed>                                           $image_meta The image meta data as returned by 'wp_get_attachment_metadata()'.
+	 * @param int                                                            $attachment_id Image attachment ID or 0.
 	 *
 	 * @return array
 	 */
 	public function filter_srcset_attr( $sources = [], $size_array = [], $image_src = '', $image_meta = [], $attachment_id = 0 ) {
-
-		if ( ! is_array( $sources ) ) {
-			return $sources;
-		}
-
 		if ( Optml_Media_Offload::is_uploaded_image( $image_src ) ) {
 			return $sources;
 		}
@@ -513,10 +508,10 @@ public function filter_srcset_attr( $sources = [], $size_array = [], $image_src
 	/**
 	 * Filters sizes attribute of the images.
 	 *
-	 * @param array $sizes An array of media query breakpoints.
-	 * @param array $size Width and height of the image.
+	 * @param string       $sizes An array of media query breakpoints.
+	 * @param string|int[] $size Width and height of the image.
 	 *
-	 * @return mixed An array of media query breakpoints.
+	 * @return string An array of media query breakpoints.
 	 */
 	public function filter_sizes_attr( $sizes, $size ) {