+global\FS_Plugin_Updater::install_and_activate_plugin()

Complexity

Conditions	18
Paths	47

Size

Total Lines	141
Code Lines	83

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
		function install_and_activate_plugin( $plugin_id = false ) {

It is generally recommended to explicitly declare the visibility for methods.

			if ( ! empty( $plugin_id ) && ! FS_Plugin::is_valid_id( $plugin_id ) ) {
				// Invalid plugin ID.
				return array(
					'message' => $this->_fs->get_text( 'auto-install-error-invalid-id' ),
					'code'    => 'invalid_module_id',
				);
			}

			$is_addon = false;
			if ( FS_Plugin::is_valid_id( $plugin_id ) &&
			     $plugin_id != $this->_fs->get_id()
			) {
				$addon = $this->_fs->get_addon( $plugin_id );

It seems like $plugin_id defined by parameter $plugin_id on line 455 can also be of type boolean; however, Freemius::get_addon() does only seem to accept integer|double, maybe add an additional type check?

				if ( ! is_object( $addon ) ) {
					// Invalid add-on ID.
					return array(
						'message' => $this->_fs->get_text( 'auto-install-error-invalid-id' ),
						'code'    => 'invalid_module_id',
					);
				}

				$slug  = $addon->slug;
				$title = $addon->title . ' ' . $this->_fs->get_text( 'addon' );

				$is_addon = true;
			} else {
				$slug  = $this->_fs->get_slug();
				$title = $this->_fs->get_plugin_title() .
				         ( $this->_fs->is_addon() ? ' ' . $this->_fs->get_text( 'addon' ) : '' );
			}

			if ( $this->is_premium_plugin_active( $plugin_id ) ) {
				// Premium version already activated.
				return array(
					'message' => $this->_fs->get_text(
						$is_addon ?
							'auto-install-error-premium-addon-activated' :
							'auto-install-error-premium-activated'
					),
					'code'    => 'premium_installed',
				);
			}

			$latest_version = $this->get_latest_download_details( $plugin_id );
			$target_folder  = "{$slug}-premium";

			// Prep variables for Plugin_Installer_Skin class.
			$extra         = array();
			$extra['slug'] = $target_folder;
			$source        = $latest_version->url;
			$api           = null;

			$install_url = add_query_arg(
				array(
					'action' => 'install-plugin',
					'plugin' => urlencode( $slug ),
				),
				'update.php'
			);

			if ( ! class_exists( 'Plugin_Upgrader', false ) ) {
				// Include required resources for the installation.
				require_once ABSPATH . 'wp-admin/includes/class-wp-upgrader.php';
			}

			$skin_args = array(
				'type'   => 'web',
				'title'  => sprintf( fs_text( 'installing-plugin-x', $slug ), $title ),
				'url'    => esc_url_raw( $install_url ),
				'nonce'  => 'install-plugin_' . $slug,
				'plugin' => '',
				'api'    => $api,
				'extra'  => $extra,
			);

//			$skin = new Automatic_Upgrader_Skin( $skin_args );

42% of this comment could be valid code. Did you maybe forget this after debugging?

//			$skin = new Plugin_Installer_Skin( $skin_args );
			$skin = new WP_Ajax_Upgrader_Skin( $skin_args );

			// Create a new instance of Plugin_Upgrader.
			$upgrader = new Plugin_Upgrader( $skin );

			// Perform the action and install the plugin from the $source urldecode().
			add_filter( 'upgrader_source_selection', array( &$this, '_maybe_adjust_source_dir' ), 1, 3 );

			$install_result = $upgrader->install( $source );

			remove_filter( 'upgrader_source_selection', array( &$this, '_maybe_adjust_source_dir' ), 1 );

			if ( is_wp_error( $install_result ) ) {
				return array(
					'message' => $install_result->get_error_message(),
					'code'    => $install_result->get_error_code(),
				);
			} elseif ( is_wp_error( $skin->result ) ) {
				return array(
					'message' => $skin->result->get_error_message(),
					'code'    => $skin->result->get_error_code(),
				);
			} elseif ( $skin->get_errors()->get_error_code() ) {
				return array(
					'message' => $skin->get_error_messages(),
					'code'    => 'unknown',
				);
			} elseif ( is_null( $install_result ) ) {
				global $wp_filesystem;

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

				$error_code    = 'unable_to_connect_to_filesystem';
				$error_message = __( 'Unable to connect to the filesystem. Please confirm your credentials.' );

				// Pass through the error from WP_Filesystem if one was raised.
				if ( $wp_filesystem instanceof WP_Filesystem_Base &&

The class WP_Filesystem_Base does not exist. Did you forget a USE statement, or did you not list all dependencies?

				     is_wp_error( $wp_filesystem->errors ) &&
				     $wp_filesystem->errors->get_error_code()
				) {
					$error_message = $wp_filesystem->errors->get_error_message();
				}

				return array(
					'message' => $error_message,
					'code'    => $error_code,
				);
			}

			// Grab the full path to the main plugin's file.
			$plugin_activate = $upgrader->plugin_info();

			// Try to activate the plugin.
			$activation_result = $this->try_activate_plugin( $plugin_activate );

			if ( is_wp_error( $activation_result ) ) {
				return array(
					'message' => $activation_result->get_error_message(),
					'code'    => $activation_result->get_error_code(),
				);
			}

			return $skin->get_upgrade_messages();
		}

	}