[static-analysis] Escape rules not following the logic of the code.

bahiirwa · bahiirwa · commit 6ee74066e662 · 2024-06-16T17:14:07.000+03:00
diff --git a/includes/class-freemius.php b/includes/class-freemius.php
@@ -1003,7 +1003,7 @@ private static function migrate_install_plan_to_plan_id( FS_Storage $storage, $b
 
             if ( isset( $install->plan ) && is_object( $install->plan ) ) {
                 if ( isset( $install->plan->id ) && ! empty( $install->plan->id ) ) {
-                    $install->plan_id = self::_decrypt( $install->plan->id ); // @phpstan-ignore-line
+                    $install->plan_id = self::_decrypt( $install->plan->id );
                 }
 
                 unset( $install->plan );
@@ -1564,8 +1564,8 @@ private function register_constructor_hooks() {
             );
             $this->add_filter( 'after_code_type_change', array( &$this, '_after_code_type_change' ) );
 
-            add_action( 'admin_init', array( &$this, '_add_trial_notice' ) ); // @phpstan-ignore-line
-            add_action( 'admin_init', array( &$this, '_add_affiliate_program_notice' ) ); // @phpstan-ignore-line
+            add_action( 'admin_init', array( &$this, '_add_trial_notice' ) );
+            add_action( 'admin_init', array( &$this, '_add_affiliate_program_notice' ) );
             add_action( 'admin_enqueue_scripts', array( &$this, '_enqueue_common_css' ) );
 
             /**
@@ -1882,7 +1882,7 @@ private function clear_module_main_file_cache( $store_prev_path = true ) {
                 $plugin_main_file = clone $this->_storage->plugin_main_file;
 
                 // Store cached path (2nd layer cache).
-                $plugin_main_file->prev_path = $plugin_main_file->path; // @phpstan-ignore-line
+                $plugin_main_file->prev_path = $plugin_main_file->path;
 
                 // Clear cached path.
                 unset( $plugin_main_file->path );
@@ -8016,7 +8016,6 @@ private function maybe_network_activate_addon_license( $license = null ) {
                     }
                 }
 
-                // @phpstan-ignore-next-line
                 if ( ! empty( $site_ids ) ) {
                     $this->activate_license_on_many_sites( $user, $license->secret_key, $site_ids );
                 }
@@ -8955,7 +8954,7 @@ private function update_plugin_version_event() {
          * @author Vova Feldman (@svovaf)
          * @since  2.0.0
          *
-         * @param string[] $plugins
+         * @param array<string, array> $plugins
          *
          * @return string
          */
@@ -9888,13 +9887,15 @@ private function sync_installs( $override = array(), $flush = false ) {
 
             $address_to_blog_map = $this->get_address_to_blog_map();
 
-            foreach ( $result->installs as $install ) {
-                $this->_site = new FS_Site( $install );
+            if ( ! empty( $result->installs ) ) {
+                foreach ( $result->installs as $install ) {
+                    $this->_site = new FS_Site( $install );
 
-                $address = trailingslashit( fs_strip_url_protocol( $install->url ) );
-                $blog_id = $address_to_blog_map[ $address ];
+                    $address = trailingslashit( fs_strip_url_protocol( $install->url ) );
+                    $blog_id = $address_to_blog_map[ $address ];
 
-                $this->_store_site( true, $blog_id );
+                    $this->_store_site( true, $blog_id );
+                }
             }
         }
 
@@ -14101,7 +14102,7 @@ private function activate_license(
                         $result = $fs->activate_license_on_many_installs( $user, $license_key, $blog_2_install_map );
                     }
 
-                    if ( true === $result && count( $site_ids ) > 0 ) {
+                    if ( true === $result && ! empty( $site_ids ) > 0 ) {
                         $result = $fs->activate_license_on_many_sites( $user, $license_key, $site_ids );
                     }
                 } else {
@@ -20057,7 +20058,7 @@ private function _store_licenses( $store = true, $module_id = false, $licenses =
                     }
                 }
 
-                if ( count( $new_user_licenses_map ) > 0 ) {
+                if ( ! empty( $new_user_licenses_map ) > 0 ) {
                     // Add new licenses.
                     $all_licenses[ $module_id ] = array_merge( array_values( $new_user_licenses_map ), $all_licenses[ $module_id ] );
                 }
@@ -21303,14 +21304,17 @@ private function _sync_plugin_license(
 
                     // Find the current context install.
                     $site = null;
-                    foreach ( $result->installs as $install ) {
-                        if ( $install->id == $this->_site->id ) {
-                            $site = new FS_Site( $install );
-                        } else {
-                            $address = trailingslashit( fs_strip_url_protocol( $install->url ) );
-                            $blog_id = $address_to_blog_map[ $address ];
 
-                            $this->_store_site( true, $blog_id, new FS_Site( $install ) );
+                    if ( ! empty( $result->installs ) ) {
+                        foreach ( $result->installs as $install ) {
+                            if ( $install->id == $this->_site->id ) {
+                                $site = new FS_Site( $install );
+                            } else {
+                                $address = trailingslashit( fs_strip_url_protocol( $install->url ) );
+                                $blog_id = $address_to_blog_map[ $address ];
+
+                                $this->_store_site( true, $blog_id, new FS_Site( $install ) );
+                            }
                         }
                     }
                 }
diff --git a/includes/class-fs-plugin-updater.php b/includes/class-fs-plugin-updater.php
@@ -1049,7 +1049,7 @@ function plugins_api_filter( $data, $action = '', $args = null ) {
                         false
                     );
 
-                    if ( is_array( $addon_plugin_data ) && isset( $addon_plugin_data['Version'] ) ) {
+                    if ( isset( $addon_plugin_data['Version'] ) ) {
                         $addon_version = $addon_plugin_data['Version'];
                     }
                 }
@@ -1070,9 +1070,9 @@ function plugins_api_filter( $data, $action = '', $args = null ) {
 
                 // Fetch as much as possible info from local files.
                 $plugin_local_data = $this->_fs->get_plugin_data();
-                $data->name        = $plugin_local_data['Name']; // @phpstan-ignore-line
-                $data->author      = $plugin_local_data['Author']; // @phpstan-ignore-line
-                $data->sections    = array( 'description' => 'Upgrade ' . $plugin_local_data['Name'] . ' to latest.' ); // @phpstan-ignore-line
+                $data->name        = $plugin_local_data['Name'];
+                $data->author      = $plugin_local_data['Author'];
+                $data->sections    = array( 'description' => 'Upgrade ' . $plugin_local_data['Name'] . ' to latest.' );
 
                 // @todo Store extra plugin info on Freemius or parse readme.txt markup.
                 /*$info = $this->_fs->get_api_site_scope()->call('/information.json');
@@ -1096,18 +1096,18 @@ function plugins_api_filter( $data, $action = '', $args = null ) {
                     $data->name    = $addon->title . ' ' . $this->_fs->get_text_inline( 'Add-On', 'addon' );
                     $data->slug    = $addon->slug;
                     $data->url     = WP_FS__ADDRESS;
-                    $data->package = $new_version->url; // @phpstan-ignore-line
+                    $data->package = $new_version->url;
                 }
 
                 if ( ! $plugin_in_repo ) {
-                    $data->last_updated = ! is_null( $new_version->updated ) ? $new_version->updated : $new_version->created; // @phpstan-ignore-line
-                    $data->requires     = $new_version->requires_platform_version; // @phpstan-ignore-line
-                    $data->requires_php = $new_version->requires_programming_language_version; // @phpstan-ignore-line
-                    $data->tested       = $new_version->tested_up_to_version; // @phpstan-ignore-line
+                    $data->last_updated = ! is_null( $new_version->updated ) ? $new_version->updated : $new_version->created;
+                    $data->requires     = $new_version->requires_platform_version;
+                    $data->requires_php = $new_version->requires_programming_language_version;
+                    $data->tested       = $new_version->tested_up_to_version;
                 }
 
                 $data->version       = $new_version->version;
-                $data->download_link = $new_version->url; // @phpstan-ignore-line
+                $data->download_link = $new_version->url;
 
                 if ( isset( $new_version->readme ) && is_object( $new_version->readme ) ) {
                     $new_version_readme_data = $new_version->readme;
diff --git a/includes/entities/class-fs-entity.php b/includes/entities/class-fs-entity.php
@@ -88,8 +88,8 @@ static function equals( $entity1, $entity2 ) {
 		 * @author Vova Feldman (@svovaf)
 		 * @since  1.0.9
 		 *
-		 * @param array|string[] $key
-		 * @param string|bool    $val
+		 * @param string|array<string, mixed> $key
+		 * @param string|bool                 $val
 		 *
 		 * @return bool
 		 */
@@ -156,4 +156,4 @@ static function is_valid_id($id){
         public static function get_class_name() {
             return get_called_class();
         }
-	}
+	}
diff --git a/includes/fs-core-functions.php b/includes/fs-core-functions.php
@@ -1389,8 +1389,8 @@ function fs_esc_html_echo_inline( $text, $key = '', $slug = 'freemius' ) {
          * @author Vova Feldman (@svovaf)
          * @since  1.1.6
          *
-         * @param array|string[] $key_value
-         * @param string         $slug
+         * @param array<string, string> $key_value
+         * @param string                $slug
          *
          * @global $fs_text_overrides
          */
diff --git a/includes/fs-essential-functions.php b/includes/fs-essential-functions.php
@@ -264,7 +264,7 @@ function fs_update_sdk_newest_version( $sdk_relative_path, $plugin_file = false
                 $in_activation = ( ! is_plugin_active( $plugin_file ) );
             } else {
                 $theme         = wp_get_theme();
-                $in_activation = ( $newest_sdk->plugin_path == $theme->stylesheet ); // @phpstan-ignore-line
+                $in_activation = ( $newest_sdk->plugin_path == $theme->stylesheet );
             }
 
             $fs_active_plugins->newest = (object) array(
@@ -415,4 +415,4 @@ function fs_fallback_to_newest_active_sdk() {
                 fs_update_sdk_newest_version( $newest_sdk_path, $newest_sdk_data->plugin_path );
             }
         }
-    }
+    }
diff --git a/includes/fs-plugin-info-dialog.php b/includes/fs-plugin-info-dialog.php
@@ -229,7 +229,7 @@ function _get_addon_info_filter( $data, $action = '', $args = null ) {
                         false
                     );
 
-                    if ( is_array( $addon_plugin_data ) && isset( $addon_plugin_data['Version'] ) ) {
+                    if ( isset( $addon_plugin_data['Version'] ) ) {
                         $current_addon_version = $addon_plugin_data['Version'];
                     }
                 }
@@ -527,7 +527,7 @@ private function get_checkout_cta( $api, $plan = null ) {
                 $plan->plugin_id,
                 $plan->pricing[0]->id,
                 $this->get_billing_cycle( $plan ),
-                $plan->has_trial(), // @phpstan-ignore-line
+                $plan->has_trial(),
                 ( $has_valid_blog_id ? false : null )
             );
 
@@ -536,7 +536,7 @@ private function get_checkout_cta( $api, $plan = null ) {
             }
 
             return '<a class="button button-primary fs-checkout-button right" href="' . $addon_checkout_url . '" target="_parent">' .
-                   esc_html( ! $plan->has_trial() ? // @phpstan-ignore-line
+                   esc_html( ! $plan->has_trial() ?
                        (
                            $api->has_purchased_license ?
                                fs_text_inline( 'Purchase More', 'purchase-more', $api->slug ) :
diff --git a/includes/managers/class-fs-cache-manager.php b/includes/managers/class-fs-cache-manager.php
@@ -166,8 +166,7 @@ function get( $key, $default = null ) {
                  isset( $cache_entry->timestamp ) &&
                  is_numeric( $cache_entry->timestamp )
             ) {
-                return $cache_entry->result; // @phpstan-ignore-line
-
+                return $cache_entry->result;
             }
 
             return is_object( $default ) ? clone $default : $default;
@@ -324,4 +323,4 @@ function migrate_to_network() {
         }
 
         #endregion
-    }
+    }
diff --git a/phpstan.neon b/phpstan.neon
@@ -18,14 +18,33 @@ parameters:
         -
             message: '#PHPDoc tag @var above a method has no effect.#'
             path: includes/class-freemius.php
+        -
+            message: '#Action callback returns bool but should not return anything.#'
+            path: includes/class-freemius.php
+        -
+            message: '#Variable \$site_ids in empty\(\) always exists and is always falsy.#'
+            path: includes/class-freemius.php
+        -
+            message: '#Variable \$new_user_licenses_map in empty\(\) always exists and is not falsy.#'
+            path: includes/class-freemius.php
         -
             message: '#Access to an undefined property#'
             paths:
+                - start.php
+                - templates/debug.php
+                - templates/add-ons.php
+                - templates/plugin-info/features.php
                 - includes/class-freemius.php
                 - includes/fs-plugin-info-dialog.php
+                - includes/fs-essential-functions.php
+                - includes/class-fs-plugin-updater.php
+                - includes/managers/class-fs-cache-manager.php
         -
             message: '#will always evaluate to true#'
             path: includes/class-fs-plugin-updater.php
+        -
+            message: '#Call to an undefined method object::has_trial\(\)#'
+            path: includes/fs-plugin-info-dialog.php
         -
             message: '#Variable \$VARS might not be defined#'
             paths:
diff --git a/templates/account.php b/templates/account.php
@@ -96,14 +96,13 @@
 		) );
 	}
 
+    $payments     = [];
     $show_billing = ( ! $is_whitelabeled && ! $fs->apply_filters( 'hide_billing_and_payments_info', false ) );
     if ( $show_billing ) {
-        $payments = $fs->_fetch_payments();
-
-        $show_billing = ( is_array( $payments ) && 0 < count( $payments ) );
+        $payments     = $fs->_fetch_payments();
+        $show_billing = ( 0 < count( $payments ) );
     }
 
-
     $has_tabs = $fs->_add_tabs_before_content();
 
 	// Aliases.
@@ -137,8 +136,7 @@
 
     $show_plan_row    = true;
     $show_license_row = is_object( $license );
-
-	$site_view_params        = array();
+	$site_view_params = array();
 
     if ( fs_is_network_admin() ) {
         $sites                   = Freemius::get_sites();
@@ -157,7 +155,7 @@
 
             $site_view_params[] = $view_params;
 
-            if ( is_object( $install ) ) {
+            if ( ! is_object( $install ) ) {
                 continue;
             }
 
@@ -871,7 +869,7 @@ class="fs-tag fs-<?php echo $fs->can_use_premium_code() ? 'success' : 'warn' ?>"
 
 						<?php
 							if ( $show_billing ) {
-                                $view_params = array( 'id' => $VARS['id'], 'payments' => $payments ); // @phpstan-ignore-line
+                                $view_params = array( 'id' => $VARS['id'], 'payments' => $payments );
                                 fs_require_once_template( 'account/billing.php', $view_params );
 								fs_require_once_template( 'account/payments.php', $view_params );
 							}
diff --git a/templates/account/partials/addon.php b/templates/account/partials/addon.php
@@ -67,6 +67,10 @@
     $show_upgrade           = false;
     $is_whitelabeled        = $VARS['is_whitelabeled'];
 
+    $version    = '';
+    $plan_name  = '';
+    $plan_title = '';
+
     if ( is_object( $fs_addon ) ) {
         $is_paying                  = $fs_addon->is_paying();
         $user                       = $fs_addon->get_user();
@@ -158,11 +162,11 @@
         <!--/ ID -->
 
         <!-- Version -->
-        <td><?php echo $version; // @phpstan-ignore-line ?></td>
+        <td><?php echo $version; ?></td>
         <!--/ Version -->
 
         <!-- Plan Title -->
-        <td><?php echo strtoupper( is_string( $plan_name ) ? $plan_title : $free_text ); // @phpstan-ignore-line ?></td>
+        <td><?php echo strtoupper( is_string( $plan_name ) ? $plan_title : $free_text ); ?></td>
         <!--/ Plan Title -->
 
         <!-- Expiration -->
diff --git a/templates/add-ons.php b/templates/add-ons.php
@@ -142,12 +142,8 @@
 									}
 
 									$has_paid_plan = true;
-
-                                    if ( isset( $plan->trial_period ) ) {
-									    $has_trial = ( is_numeric( $plan->trial_period ) && ( $plan->trial_period > 0 ) );
-                                    }
-
-									$min_price = 999999;
+                                    $has_trial     = isset( $plan->trial_period ) && is_numeric( $plan->trial_period ) && $plan->trial_period > 0;
+									$min_price     = 999999;
 									foreach ( $plan->pricing as $pricing ) {
                                         $pricing = new FS_Pricing( $pricing );
 
@@ -501,4 +497,4 @@ function toggleDropdown( $dropdown, state ) {
 		'module_slug'    => $slug,
 		'module_version' => $fs->get_plugin_version(),
 	);
-	fs_require_template( 'powered-by.php', $params );
+	fs_require_template( 'powered-by.php', $params );
diff --git a/templates/debug.php b/templates/debug.php
diff --git a/templates/plugin-info/features.php b/templates/plugin-info/features.php

Original file line number	Diff line number	Diff line change
`@@ -88,8 +88,8 @@ static function equals( $entity1, $entity2 ) {`
`88`	`88`	`* @author Vova Feldman (@svovaf)`
`89`	`89`	`* @since 1.0.9`
`90`	`90`	`*`
`91`		`- * @param array\|string[] $key`
`92`		`- * @param string\|bool $val`
	`91`	`+ * @param string\|array<string, mixed> $key`
	`92`	`+ * @param string\|bool $val`
`93`	`93`	`*`
`94`	`94`	`* @return bool`
`95`	`95`	`*/`
`@@ -156,4 +156,4 @@ static function is_valid_id($id){`
`156`	`156`	`public static function get_class_name() {`
`157`	`157`	`return get_called_class();`
`158`	`158`	`}`
`159`		`- }`
	`159`	`+ }`
Original file line number	Diff line number	Diff line change
`@@ -1389,8 +1389,8 @@ function fs_esc_html_echo_inline( $text, $key = '', $slug = 'freemius' ) {`
`1389`	`1389`	`* @author Vova Feldman (@svovaf)`
`1390`	`1390`	`* @since 1.1.6`
`1391`	`1391`	`*`
`1392`		`- * @param array\|string[] $key_value`
`1393`		`- * @param string $slug`
	`1392`	`+ * @param array<string, string> $key_value`
	`1393`	`+ * @param string $slug`
`1394`	`1394`	`*`
`1395`	`1395`	`* @global $fs_text_overrides`
`1396`	`1396`	`*/`
Original file line number	Diff line number	Diff line change
`@@ -264,7 +264,7 @@ function fs_update_sdk_newest_version( $sdk_relative_path, $plugin_file = false`
`264`	`264`	`$in_activation = ( ! is_plugin_active( $plugin_file ) );`
`265`	`265`	`} else {`
`266`	`266`	`$theme = wp_get_theme();`
`267`		`- $in_activation = ( $newest_sdk->plugin_path == $theme->stylesheet ); // @phpstan-ignore-line`
	`267`	`+ $in_activation = ( $newest_sdk->plugin_path == $theme->stylesheet );`
`268`	`268`	`}`
`269`	`269`
`270`	`270`	`$fs_active_plugins->newest = (object) array(`
`@@ -415,4 +415,4 @@ function fs_fallback_to_newest_active_sdk() {`
`415`	`415`	`fs_update_sdk_newest_version( $newest_sdk_path, $newest_sdk_data->plugin_path );`
`416`	`416`	`}`
`417`	`417`	`}`
`418`		`- }`
	`418`	`+ }`
Original file line number	Diff line number	Diff line change
`@@ -229,7 +229,7 @@ function _get_addon_info_filter( $data, $action = '', $args = null ) {`
`229`	`229`	`false`
`230`	`230`	`);`
`231`	`231`
`232`		`- if ( is_array( $addon_plugin_data ) && isset( $addon_plugin_data['Version'] ) ) {`
	`232`	`+ if ( isset( $addon_plugin_data['Version'] ) ) {`
`233`	`233`	`$current_addon_version = $addon_plugin_data['Version'];`
`234`	`234`	`}`
`235`	`235`	`}`
`@@ -527,7 +527,7 @@ private function get_checkout_cta( $api, $plan = null ) {`
`527`	`527`	`$plan->plugin_id,`
`528`	`528`	`$plan->pricing[0]->id,`
`529`	`529`	`$this->get_billing_cycle( $plan ),`
`530`		`- $plan->has_trial(), // @phpstan-ignore-line`
	`530`	`+ $plan->has_trial(),`
`531`	`531`	`( $has_valid_blog_id ? false : null )`
`532`	`532`	`);`
`533`	`533`
`@@ -536,7 +536,7 @@ private function get_checkout_cta( $api, $plan = null ) {`
`536`	`536`	`}`
`537`	`537`
`538`	`538`	`return '<a class="button button-primary fs-checkout-button right" href="' . $addon_checkout_url . '" target="_parent">' .`
`539`		`- esc_html( ! $plan->has_trial() ? // @phpstan-ignore-line`
	`539`	`+ esc_html( ! $plan->has_trial() ?`
`540`	`540`	`(`
`541`	`541`	`$api->has_purchased_license ?`
`542`	`542`	`fs_text_inline( 'Purchase More', 'purchase-more', $api->slug ) :`
Original file line number	Diff line number	Diff line change
`@@ -166,8 +166,7 @@ function get( $key, $default = null ) {`
`166`	`166`	`isset( $cache_entry->timestamp ) &&`
`167`	`167`	`is_numeric( $cache_entry->timestamp )`
`168`	`168`	`) {`
`169`		`- return $cache_entry->result; // @phpstan-ignore-line`
`170`		`-`
	`169`	`+ return $cache_entry->result;`
`171`	`170`	`}`
`172`	`171`
`173`	`172`	`return is_object( $default ) ? clone $default : $default;`
`@@ -324,4 +323,4 @@ function migrate_to_network() {`
`324`	`323`	`}`
`325`	`324`
`326`	`325`	`#endregion`
`327`		`- }`
	`326`	`+ }`