install.php ➔ wpbo_single_activate() - Code Metrics - ThemeAvenue/BetterOptin - Measure and Improve Code Quality continuously with Scrutinizer

install.php ➔ wpbo_single_activate() B
last analyzed 2016-05-12 10:36 UTC

↳ Parent: Project

Complexity

Conditions	8
Paths	5

Size

Total Lines	54
Code Lines	20

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	1

Metric	Value
cc	8
eloc	20
c	3
b	0
f	1
nc	5
nop	0
dl	0
loc	54
rs	7.4119

How to fix Long Method

<?php
/**
 * BetterOptin Installer
 *
 * @package   BetterOptin/Install
 * @author    ThemeAvenue <[email protected]>
 * @license   GPL-2.0+
 * @link      http://themeavenue.net
 * @copyright 2015 ThemeAvenue
 */

// If this file is called directly, abort.
if ( ! defined( 'WPINC' ) ) {
	die;
}

register_activation_hook( WPBO_PLUGIN_FILE, 'wpbo_activate' );
add_action( 'wpmu_new_blog', 'wpbo_activate_new_site' );

/**
 * Fired when the plugin is activated.
 *
 * @since    1.0.0
 *
 * @param    boolean    $network_wide    True if WPMU superadmin uses
 *                                       "Network Activate" action, false if
 *                                       WPMU is disabled or plugin is
 *                                       activated on an individual blog.
 */
function wpbo_activate( $network_wide ) {

	if ( function_exists( 'is_multisite' ) && is_multisite() ) {

		if ( $network_wide  ) {

			// Get all blog ids
			$blog_ids = wpbo_get_blog_ids();

			foreach ( $blog_ids as $blog_id ) {
$collection = json_decode($data, true);
if ( ! is_array($collection)) {
    throw new \RuntimeException('$collection must be an array.');
}

foreach ($collection as $item) { /** ... */ }

				switch_to_blog( $blog_id );
				wpbo_single_activate();
			}

			restore_current_blog();

		} else {
			wpbo_single_activate();
		}

	} else {
		wpbo_single_activate();
	}

}

/**
 * Get all blog ids of blogs in the current network that are:
 * - not archived
 * - not spam
 * - not deleted
 *
 * @since    1.0.0
 *
 * @return   array|false    The blog ids, false if no matches.
 */
function wpbo_get_blog_ids() {

	global $wpdb;

	// get an array of blog ids
	$sql = "SELECT blog_id FROM $wpdb->blogs
			WHERE archived = '0' AND spam = '0'
			AND deleted = '0'";

	return $wpdb->get_col( $sql );

}

/**
 * Fired for each blog when the plugin is activated.
 *
 * @since    1.0.0
 */
function wpbo_single_activate() {

	/* Add new role */
	$subscriber = get_role( 'subscriber' );
	add_role( 'betteroptin', 'BetterOptin', $subscriber->capabilities );

	/* Create database table */
	wpbo_create_table();
	wpbo_failsafe_create_table();

	/* Write database version */
	update_option( 'wpbo_db_version', WPBO_DB_VERSION );

	$defaults = get_option( 'wpbo_options', array() );

	// Register default options
	if ( empty( $defaults ) ) {

		// Get default options
		if ( ! function_exists( 'wpbo_settings_general' ) ) {
			require( WPBO_PATH . 'includes/admin/settings/settings-general.php' );
		}

		$options = apply_filters( 'wpbo_plugin_settings', array() );

		foreach ( $options as $section_id => $section ) {

			foreach ( $section['options'] as $option ) {

				if ( ! isset( $option['id'] ) ) {
					continue;
				}

				$value                     = isset( $option['default'] ) ? $option['default'] : '';
				$defaults[ $option['id'] ] = $value;

			}

		}

		if ( ! empty( $defaults ) ) {
			update_option( 'wpbo_options', serialize( $defaults ) );
		}

	}

	/**
	 * Add an option in DB to know when the plugin has just been activated.
	 *
	 * @link http://stackoverflow.com/questions/7738953/is-there-a-way-to-determine-if-a-wordpress-plugin-is-just-installed/13927297#13927297
	 */
	add_option( 'wpbo_just_activated', true );

}

/**
 * Fired when a new site is activated with a WPMU environment.
 *
 * @since    1.0.0
 *
 * @param    int    $blog_id    ID of the new blog.
 */
function wpbo_activate_new_site( $blog_id ) {

	if ( 1 !== did_action( 'wpmu_new_blog' ) ) {
		return;
	}

	switch_to_blog( $blog_id );
	wpbo_single_activate();
	restore_current_blog();

}

/**
 * Redirect the user to the about page if the plugin was just installed
 *
 * @return void
 */
function wpbo_maybe_redirect_about() {

	if ( ! is_admin() ) {
		return;
	}

	$activated = get_option( 'wpbo_just_activated', false );

	/**
	 * First thing we check if the plugin has just been activated.
	 * If so, we take the user to the about page and delete the
	 * option we used for the check.
	 */
	if ( $activated ) {

		/* Delete the option */
		delete_option( 'wpbo_just_activated' );

		/* Redirect to about page */
		wp_redirect( add_query_arg( array(
			'post_type' => 'wpbo-popup',
			'page'      => 'wpbo-about'
		), admin_url( 'edit.php' ) ) );

		/* Don't do anything else */
		exit;

	}

}

1			<?php
2			/**
3			* BetterOptin Installer
4			*
5			* @package BetterOptin/Install
6			* @author ThemeAvenue <[email protected]>
7			* @license GPL-2.0+
8			* @link http://themeavenue.net
9			* @copyright 2015 ThemeAvenue
10			*/
11
12			// If this file is called directly, abort.
13			if ( ! defined( 'WPINC' ) ) {
14			die;
15			}
16
17			register_activation_hook( WPBO_PLUGIN_FILE, 'wpbo_activate' );
18			add_action( 'wpmu_new_blog', 'wpbo_activate_new_site' );
19
20			/**
21			* Fired when the plugin is activated.
22			*
23			* @since 1.0.0
24			*
25			* @param boolean $network_wide True if WPMU superadmin uses
26			* "Network Activate" action, false if
27			* WPMU is disabled or plugin is
28			* activated on an individual blog.
29			*/
30			function wpbo_activate( $network_wide ) {
31
32			if ( function_exists( 'is_multisite' ) && is_multisite() ) {
33
34			if ( $network_wide ) {
35
36			// Get all blog ids
37			$blog_ids = wpbo_get_blog_ids();
38
39			foreach ( $blog_ids as $blog_id ) {
			0 ignored issues – show Bug introduced 2015-11-30 08:11 UTC by Report Bug Copy Issue Report The expression `$blog_ids` of type `array\|false` is not guaranteed to be traversable. How about adding an additional type check? There are different options of fixing this problem. If you want to be on the safe side, you can add an additional type-check: $collection = json_decode($data, true); if ( ! is_array($collection)) { throw new \RuntimeException('$collection must be an array.'); } foreach ($collection as $item) { /** ... / } If you are sure that the expression is traversable, you might want to add a doc comment cast* to improve IDE auto-completion and static analysis: /** @var array $collection / $collection = json_decode($data, true); foreach ($collection as $item) { /* .. / } Mark the issue as a false-positive*: Just hover the remove button, in the top-right corner of this issue for more options. Loading history...
40
41			switch_to_blog( $blog_id );
42			wpbo_single_activate();
43			}
44
45			restore_current_blog();
46
47			} else {
48			wpbo_single_activate();
49			}
50
51			} else {
52			wpbo_single_activate();
53			}
54
55			}
56
57			/**
58			* Get all blog ids of blogs in the current network that are:
59			* - not archived
60			* - not spam
61			* - not deleted
62			*
63			* @since 1.0.0
64			*
65			* @return array\|false The blog ids, false if no matches.
66			*/
67			function wpbo_get_blog_ids() {
68
69			global $wpdb;
70
71			// get an array of blog ids
72			$sql = "SELECT blog_id FROM $wpdb->blogs
73			WHERE archived = '0' AND spam = '0'
74			AND deleted = '0'";
75
76			return $wpdb->get_col( $sql );
77
78			}
79
80			/**
81			* Fired for each blog when the plugin is activated.
82			*
83			* @since 1.0.0
84			*/
85			function wpbo_single_activate() {
86
87			/* Add new role */
88			$subscriber = get_role( 'subscriber' );
89			add_role( 'betteroptin', 'BetterOptin', $subscriber->capabilities );
90
91			/* Create database table */
92			wpbo_create_table();
93			wpbo_failsafe_create_table();
94
95			/* Write database version */
96			update_option( 'wpbo_db_version', WPBO_DB_VERSION );
97
98			$defaults = get_option( 'wpbo_options', array() );
99
100			// Register default options
101			if ( empty( $defaults ) ) {
102
103			// Get default options
104			if ( ! function_exists( 'wpbo_settings_general' ) ) {
105			require( WPBO_PATH . 'includes/admin/settings/settings-general.php' );
106			}
107
108			$options = apply_filters( 'wpbo_plugin_settings', array() );
109
110			foreach ( $options as $section_id => $section ) {
111
112			foreach ( $section['options'] as $option ) {
113
114			if ( ! isset( $option['id'] ) ) {
115			continue;
116			}
117
118			$value = isset( $option['default'] ) ? $option['default'] : '';
119			$defaults[ $option['id'] ] = $value;
120
121			}
122
123			}
124
125			if ( ! empty( $defaults ) ) {
126			update_option( 'wpbo_options', serialize( $defaults ) );
127			}
128
129			}
130
131			/**
132			* Add an option in DB to know when the plugin has just been activated.
133			*
134			* @link http://stackoverflow.com/questions/7738953/is-there-a-way-to-determine-if-a-wordpress-plugin-is-just-installed/13927297#13927297
135			*/
136			add_option( 'wpbo_just_activated', true );
137
138			}
139
140			/**
141			* Fired when a new site is activated with a WPMU environment.
142			*
143			* @since 1.0.0
144			*
145			* @param int $blog_id ID of the new blog.
146			*/
147			function wpbo_activate_new_site( $blog_id ) {
148
149			if ( 1 !== did_action( 'wpmu_new_blog' ) ) {
150			return;
151			}
152
153			switch_to_blog( $blog_id );
154			wpbo_single_activate();
155			restore_current_blog();
156
157			}
158
159			/**
160			* Redirect the user to the about page if the plugin was just installed
161			*
162			* @return void
163			*/
164			function wpbo_maybe_redirect_about() {
165
166			if ( ! is_admin() ) {
167			return;
168			}
169
170			$activated = get_option( 'wpbo_just_activated', false );
171
172			/**
173			* First thing we check if the plugin has just been activated.
174			* If so, we take the user to the about page and delete the
175			* option we used for the check.
176			*/
177			if ( $activated ) {
178
179			/* Delete the option */
180			delete_option( 'wpbo_just_activated' );
181
182			/* Redirect to about page */
183			wp_redirect( add_query_arg( array(
184			'post_type' => 'wpbo-popup',
185			'page' => 'wpbo-about'
186			), admin_url( 'edit.php' ) ) );
187
188			/* Don't do anything else */
189			exit;
190
191			}
192
193			}

ThemeAvenue / BetterOptin

install.php ➔ wpbo_single_activate() B last analyzed 2016-05-12 10:36 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like

install.php ➔ wpbo_single_activate() B
last analyzed 2016-05-12 10:36 UTC