plugins.php ➔ give_recently_activated_addons() - Code Metrics - Inspection of "fix(admin-styles): load shortcode feature only if..." - ravinderk/Give - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — issues/2980 ( 6931f9 )

by Ravinder

created 2018-04-09 04:12 UTC

plugins.php ➔ give_recently_activated_addons() D

↳ Parent: Project

Complexity

Conditions	10
Paths	85

Size

Total Lines	31
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	10
eloc	18
nc	85
nop	0
dl	0
loc	31
rs	4.8196
c	0
b	0
f	0

How to fix Complexity

<?php
/**
 * Admin Plugins
 *
 * @package     Give
 * @subpackage  Admin/Plugins
 * @copyright   Copyright (c) 2016, WordImpress
 * @license     https://opensource.org/licenses/gpl-license GNU Public License
 * @since       1.4
 */

// Exit if accessed directly.
if ( ! defined( 'ABSPATH' ) ) {
	exit;
}

/**
 * Plugins row action links
 *
 * @since 1.4
 *
 * @param array $actions An array of plugin action links.
 *
 * @return array An array of updated action links.
 */
function give_plugin_action_links( $actions ) {
	$new_actions = array(
		'settings' => sprintf(
			'<a href="%1$s">%2$s</a>',
			admin_url( 'edit.php?post_type=give_forms&page=give-settings' ),
			__( 'Settings', 'give' )
		),
	);

	return array_merge( $new_actions, $actions );
}

add_filter( 'plugin_action_links_' . GIVE_PLUGIN_BASENAME, 'give_plugin_action_links' );


/**
 * Plugin row meta links
 *
 * @since 1.4
 *
 * @param array  $plugin_meta An array of the plugin's metadata.
 * @param string $plugin_file Path to the plugin file, relative to the plugins directory.
 *
 * @return array
 */
function give_plugin_row_meta( $plugin_meta, $plugin_file ) {
	if ( $plugin_file != GIVE_PLUGIN_BASENAME ) {
		return $plugin_meta;
	}

	$new_meta_links = array(
		sprintf(
			'<a href="%1$s" target="_blank">%2$s</a>',
			esc_url( add_query_arg( array(
					'utm_source'   => 'plugins-page',
					'utm_medium'   => 'plugin-row',
					'utm_campaign' => 'admin',
				), 'https://givewp.com/documentation/' )

			),
			__( 'Documentation', 'give' )
		),
		sprintf(
			'<a href="%1$s" target="_blank">%2$s</a>',
			esc_url( add_query_arg( array(
					'utm_source'   => 'plugins-page',
					'utm_medium'   => 'plugin-row',
					'utm_campaign' => 'admin',
				), 'https://givewp.com/addons/' )

			),
			__( 'Add-ons', 'give' )
		),
	);

	return array_merge( $plugin_meta, $new_meta_links );
}

add_filter( 'plugin_row_meta', 'give_plugin_row_meta', 10, 2 );


/**
 * Get the Parent Page Menu Title in admin section.
 * Based on get_admin_page_title WordPress Function.
 *
 * @since 1.8.17
 *
 * @global array  $submenu
 * @global string $plugin_page
 *
 * @return string $title Page title
 */
function give_get_admin_page_menu_title() {
	$title = '';
	global $submenu, $plugin_page;

	foreach ( array_keys( $submenu ) as $parent ) {
		if( 'edit.php?post_type=give_forms' !== $parent ) {

			continue;
		}

		foreach ( $submenu[ $parent ] as $submenu_array ) {
			if( $plugin_page !== $submenu_array[2] ){

				continue;
			}

			$title = isset( $submenu_array[0] ) ?
				$submenu_array[0] :
				$submenu_array[3];
		}
	}

	return $title;
}

/**
 * Store recently activated Give's addons to wp options.
 *
 * @since 2.1.0
 */
function give_recently_activated_addons() {
	// Check if action is set.
	if ( isset( $_REQUEST["action"] ) ) {
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
		$plugin_action = ( '-1' !== $_REQUEST['action'] ) ? $_REQUEST['action'] : ( isset( $_REQUEST['action2'] ) ? $_REQUEST['action2'] : '' );

		$plugins       = array();


		switch ( $plugin_action ) {
			case 'activate': // Single add-on activation.
				$plugins[] = $_REQUEST["plugin"];
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
				break;
			case 'activate-selected': // If multiple add-ons activated.
				$plugins = $_REQUEST["checked"];
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
				break;
		}

		if ( ! empty( $plugins ) ) {
			$give_addons = array();
			foreach ( $plugins as $plugin ) {
				// Get plugins which has 'Give-' as prefix.
				if ( stripos( $plugin, 'Give-' ) !== false ) {

					$give_addons[] = $plugin;
				}
			}

			if ( ! empty( $give_addons ) ) {
				// Update the Give's activated add-ons.
				update_option( 'give_recently_activated_addons', $give_addons );
			}
		}
	}
}

// Add add-on plugins to wp option table.
add_action( 'activated_plugin', 'give_recently_activated_addons', 10 );

/**
 * Create new menu in plugin section that include all the add-on
 *
 * @since 2.1.0
 *
 * @param $plugin_menu
 *
 * @return mixed
 */
function give_filter_addons_do_filter_addons( $plugin_menu ) {
	global $plugins;

	foreach ( $plugins['all'] as $plugin_slug => $plugin_data ) {

		if ( false !== strpos( $plugin_data['Name'], 'Give' ) && false !== strpos( $plugin_data['AuthorName'], 'WordImpress' ) ) {
			$plugins['give'][ $plugin_slug ]           = $plugins['all'][ $plugin_slug ];
			$plugins['give'][ $plugin_slug ]['plugin'] = $plugin_slug;
			// replicate the next step
			if ( current_user_can( 'update_plugins' ) ) {
				$current = get_site_transient( 'update_plugins' );
				if ( isset( $current->response[ $plugin_slug ] ) ) {
					$plugins['give'][ $plugin_slug ]['update'] = true;
				}
			}


		}
	}

	return $plugin_menu;

}

add_filter( 'show_advanced_plugins', 'give_filter_addons_do_filter_addons' );

/**
 * Make the Give Menu as an default menu and update the Menu Name
 *
 * @since 2.1.0
 *
 * @param $views
 *
 * @return mixed
 */
function give_filter_addons_filter_addons( $views ) {

	global $status, $plugins;

	if ( ! empty( $plugins['give'] ) ) {
		$class = "";
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

		if ( $status == 'give' ) {

			$class = 'current';
		}

		$views['give'] = sprintf(
			'<a class="%s" href="plugins.php?plugin_status=give"> %s <span class="count">(%s) </span></a>',
			$class,
			__( 'Give', 'give' ),
			count( $plugins['give'] )
		);
	}

	return $views;
}

add_filter( 'views_plugins', 'give_filter_addons_filter_addons' );

/**
 * Set the Give as the Main menu when admin click on the Give Menu in Plugin section.
 *
 * @since 2.1.0
 *
 * @param $plugins
 *
 * @return mixed
 */
function give_prepare_filter_addons( $plugins ) {
	global $status;

	if ( isset( $_REQUEST['plugin_status'] ) && $_REQUEST['plugin_status'] === 'give' ) {

		$status = 'give';

	}

	return $plugins;
}

add_filter( 'all_plugins', 'give_prepare_filter_addons' );


1			<?php
2			/**
3			* Admin Plugins
4			*
5			* @package Give
6			* @subpackage Admin/Plugins
7			* @copyright Copyright (c) 2016, WordImpress
8			* @license https://opensource.org/licenses/gpl-license GNU Public License
9			* @since 1.4
10			*/
11
12			// Exit if accessed directly.
13			if ( ! defined( 'ABSPATH' ) ) {
14			exit;
15			}
16
17			/**
18			* Plugins row action links
19			*
20			* @since 1.4
21			*
22			* @param array $actions An array of plugin action links.
23			*
24			* @return array An array of updated action links.
25			*/
26			function give_plugin_action_links( $actions ) {
27			$new_actions = array(
28			'settings' => sprintf(
29			'<a href="%1$s">%2$s</a>',
30			admin_url( 'edit.php?post_type=give_forms&page=give-settings' ),
31			__( 'Settings', 'give' )
32			),
33			);
34
35			return array_merge( $new_actions, $actions );
36			}
37
38			add_filter( 'plugin_action_links_' . GIVE_PLUGIN_BASENAME, 'give_plugin_action_links' );
39
40
41			/**
42			* Plugin row meta links
43			*
44			* @since 1.4
45			*
46			* @param array $plugin_meta An array of the plugin's metadata.
47			* @param string $plugin_file Path to the plugin file, relative to the plugins directory.
48			*
49			* @return array
50			*/
51			function give_plugin_row_meta( $plugin_meta, $plugin_file ) {
52			if ( $plugin_file != GIVE_PLUGIN_BASENAME ) {
53			return $plugin_meta;
54			}
55
56			$new_meta_links = array(
57			sprintf(
58			'<a href="%1$s" target="_blank">%2$s</a>',
59			esc_url( add_query_arg( array(
60			'utm_source' => 'plugins-page',
61			'utm_medium' => 'plugin-row',
62			'utm_campaign' => 'admin',
63			), 'https://givewp.com/documentation/' )
			0 ignored issues – show Coding Style introduced 2017-07-29 03:17 UTC by Report Bug Copy Issue Report This line of the multi-line function call does not seem to be indented correctly. Expected `12` spaces, but found `16`. Loading history...
64			),
65			__( 'Documentation', 'give' )
66			),
67			sprintf(
68			'<a href="%1$s" target="_blank">%2$s</a>',
69			esc_url( add_query_arg( array(
70			'utm_source' => 'plugins-page',
71			'utm_medium' => 'plugin-row',
72			'utm_campaign' => 'admin',
73			), 'https://givewp.com/addons/' )
			0 ignored issues – show Coding Style introduced 2017-07-29 03:17 UTC by Report Bug Copy Issue Report This line of the multi-line function call does not seem to be indented correctly. Expected `12` spaces, but found `16`. Loading history...
74			),
75			__( 'Add-ons', 'give' )
76			),
77			);
78
79			return array_merge( $plugin_meta, $new_meta_links );
80			}
81
82			add_filter( 'plugin_row_meta', 'give_plugin_row_meta', 10, 2 );
83
84
85			/**
86			* Get the Parent Page Menu Title in admin section.
87			* Based on get_admin_page_title WordPress Function.
88			*
89			* @since 1.8.17
90			*
91			* @global array $submenu
92			* @global string $plugin_page
93			*
94			* @return string $title Page title
95			*/
96			function give_get_admin_page_menu_title() {
97			$title = '';
98			global $submenu, $plugin_page;
99
100			foreach ( array_keys( $submenu ) as $parent ) {
101			if( 'edit.php?post_type=give_forms' !== $parent ) {
			0 ignored issues – show introduced 2017-11-07 05:35 UTC by Report Bug Copy Issue Report Space after opening control structure is required Loading history... introduced 2017-11-07 05:35 UTC by Report Bug Copy Issue Report No space before opening parenthesis is prohibited Loading history...
102			continue;
103			}
104
105			foreach ( $submenu[ $parent ] as $submenu_array ) {
106			if( $plugin_page !== $submenu_array[2] ){
			0 ignored issues – show introduced 2017-11-07 05:35 UTC by Report Bug Copy Issue Report Space after opening control structure is required Loading history... introduced 2017-11-07 05:35 UTC by Report Bug Copy Issue Report No space before opening parenthesis is prohibited Loading history...
107			continue;
108			}
109
110			$title = isset( $submenu_array[0] ) ?
111			$submenu_array[0] :
112			$submenu_array[3];
113			}
114			}
115
116			return $title;
117			}
118
119			/**
120			* Store recently activated Give's addons to wp options.
121			*
122			* @since 2.1.0
123			*/
124			function give_recently_activated_addons() {
125			// Check if action is set.
126			if ( isset( $_REQUEST["action"] ) ) {
			0 ignored issues – show Coding Style Comprehensibility introduced 2018-04-07 10:38 UTC by Report Bug Copy Issue Report The string literal `action` does not require double quotes, as per coding-style, please use single quotes. PHP provides two ways to mark string literals. Either with single quotes `'literal'` or with double quotes `"literal"`. The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences. String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (`\'`) and the backslash (`\\`). Every other character is displayed as is. Double quoted string literals may contain other variables or more complex escape sequences. <?php $singleQuoted = 'Value'; $doubleQuoted = "\tSingle is $singleQuoted"; print $doubleQuoted; will print an indented: `Single is Value` If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear. For more information on PHP string literals and available escape sequences see the PHP core documentation. Loading history...
127			$plugin_action = ( '-1' !== $_REQUEST['action'] ) ? $_REQUEST['action'] : ( isset( $_REQUEST['action2'] ) ? $_REQUEST['action2'] : '' );
			0 ignored issues – show introduced 2018-04-07 10:38 UTC by Report Bug Copy Issue Report Detected access of super global var $_REQUEST, probably need manual inspection. Loading history... introduced 2018-04-07 10:38 UTC by Report Bug Copy Issue Report Detected usage of a non-validated input variable: $_REQUEST Loading history... introduced 2018-04-07 10:38 UTC by Report Bug Copy Issue Report Detected usage of a non-sanitized input variable: $_REQUEST Loading history...
128			$plugins = array();
			0 ignored issues – show introduced 2018-04-07 10:38 UTC by Report Bug Copy Issue Report Overridding WordPress globals is prohibited Loading history...
129
130			switch ( $plugin_action ) {
131			case 'activate': // Single add-on activation.
132			$plugins[] = $_REQUEST["plugin"];
			0 ignored issues – show introduced 2018-04-07 10:38 UTC by Report Bug Copy Issue Report Detected access of super global var $_REQUEST, probably need manual inspection. Loading history... introduced 2018-04-07 10:38 UTC by Report Bug Copy Issue Report Detected usage of a non-sanitized input variable: $_REQUEST Loading history... Coding Style Comprehensibility introduced 2018-04-07 10:38 UTC by Report Bug Copy Issue Report The string literal `plugin` does not require double quotes, as per coding-style, please use single quotes. PHP provides two ways to mark string literals. Either with single quotes `'literal'` or with double quotes `"literal"`. The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences. String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (`\'`) and the backslash (`\\`). Every other character is displayed as is. Double quoted string literals may contain other variables or more complex escape sequences. <?php $singleQuoted = 'Value'; $doubleQuoted = "\tSingle is $singleQuoted"; print $doubleQuoted; will print an indented: `Single is Value` If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear. For more information on PHP string literals and available escape sequences see the PHP core documentation. Loading history...
133			break;
134			case 'activate-selected': // If multiple add-ons activated.
135			$plugins = $_REQUEST["checked"];
			0 ignored issues – show introduced 2018-04-07 10:38 UTC by Report Bug Copy Issue Report Overridding WordPress globals is prohibited Loading history... introduced 2018-04-07 10:38 UTC by Report Bug Copy Issue Report Detected access of super global var $_REQUEST, probably need manual inspection. Loading history... introduced 2018-04-07 10:38 UTC by Report Bug Copy Issue Report Detected usage of a non-sanitized input variable: $_REQUEST Loading history... Coding Style Comprehensibility introduced 2018-04-07 10:38 UTC by Report Bug Copy Issue Report The string literal `checked` does not require double quotes, as per coding-style, please use single quotes. PHP provides two ways to mark string literals. Either with single quotes `'literal'` or with double quotes `"literal"`. The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences. String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (`\'`) and the backslash (`\\`). Every other character is displayed as is. Double quoted string literals may contain other variables or more complex escape sequences. <?php $singleQuoted = 'Value'; $doubleQuoted = "\tSingle is $singleQuoted"; print $doubleQuoted; will print an indented: `Single is Value` If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear. For more information on PHP string literals and available escape sequences see the PHP core documentation. Loading history...
136			break;
137			}
138
139			if ( ! empty( $plugins ) ) {
140			$give_addons = array();
141			foreach ( $plugins as $plugin ) {
142			// Get plugins which has 'Give-' as prefix.
143			if ( stripos( $plugin, 'Give-' ) !== false ) {
			0 ignored issues – show introduced 2018-04-07 10:38 UTC by Report Bug Copy Issue Report Found "!== false". Use Yoda Condition checks, you must Loading history...
144			$give_addons[] = $plugin;
145			}
146			}
147
148			if ( ! empty( $give_addons ) ) {
149			// Update the Give's activated add-ons.
150			update_option( 'give_recently_activated_addons', $give_addons );
151			}
152			}
153			}
154			}
155
156			// Add add-on plugins to wp option table.
157			add_action( 'activated_plugin', 'give_recently_activated_addons', 10 );
158
159			/**
160			* Create new menu in plugin section that include all the add-on
161			*
162			* @since 2.1.0
163			*
164			* @param $plugin_menu
165			*
166			* @return mixed
167			*/
168			function give_filter_addons_do_filter_addons( $plugin_menu ) {
169			global $plugins;
170
171			foreach ( $plugins['all'] as $plugin_slug => $plugin_data ) {
172
173			if ( false !== strpos( $plugin_data['Name'], 'Give' ) && false !== strpos( $plugin_data['AuthorName'], 'WordImpress' ) ) {
174			$plugins['give'][ $plugin_slug ] = $plugins['all'][ $plugin_slug ];
175			$plugins['give'][ $plugin_slug ]['plugin'] = $plugin_slug;
176			// replicate the next step
177			if ( current_user_can( 'update_plugins' ) ) {
178			$current = get_site_transient( 'update_plugins' );
179			if ( isset( $current->response[ $plugin_slug ] ) ) {
180			$plugins['give'][ $plugin_slug ]['update'] = true;
181			}
182			}
			0 ignored issues – show introduced 2018-03-14 12:39 UTC by Report Bug Copy Issue Report Blank line found after control structure Loading history...
183
184			}
185			}
186
187			return $plugin_menu;
188
189			}
190
191			add_filter( 'show_advanced_plugins', 'give_filter_addons_do_filter_addons' );
192
193			/**
194			* Make the Give Menu as an default menu and update the Menu Name
195			*
196			* @since 2.1.0
197			*
198			* @param $views
199			*
200			* @return mixed
201			*/
202			function give_filter_addons_filter_addons( $views ) {
203
204			global $status, $plugins;
205
206			if ( ! empty( $plugins['give'] ) ) {
207			$class = "";
			0 ignored issues – show Coding Style Comprehensibility introduced 2018-03-14 12:39 UTC by Report Bug Copy Issue Report The string literal does not require double quotes, as per coding-style, please use single quotes. PHP provides two ways to mark string literals. Either with single quotes `'literal'` or with double quotes `"literal"`. The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences. String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (`\'`) and the backslash (`\\`). Every other character is displayed as is. Double quoted string literals may contain other variables or more complex escape sequences. <?php $singleQuoted = 'Value'; $doubleQuoted = "\tSingle is $singleQuoted"; print $doubleQuoted; will print an indented: `Single is Value` If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear. For more information on PHP string literals and available escape sequences see the PHP core documentation. Loading history...
208
209			if ( $status == 'give' ) {
			0 ignored issues – show introduced 2018-03-14 12:39 UTC by Report Bug Copy Issue Report Found "== '". Use Yoda Condition checks, you must Loading history...
210			$class = 'current';
211			}
212
213			$views['give'] = sprintf(
214			'<a class="%s" href="plugins.php?plugin_status=give"> %s <span class="count">(%s) </span></a>',
215			$class,
216			__( 'Give', 'give' ),
217			count( $plugins['give'] )
218			);
219			}
220
221			return $views;
222			}
223
224			add_filter( 'views_plugins', 'give_filter_addons_filter_addons' );
225
226			/**
227			* Set the Give as the Main menu when admin click on the Give Menu in Plugin section.
228			*
229			* @since 2.1.0
230			*
231			* @param $plugins
232			*
233			* @return mixed
234			*/
235			function give_prepare_filter_addons( $plugins ) {
236			global $status;
237
238			if ( isset( $_REQUEST['plugin_status'] ) && $_REQUEST['plugin_status'] === 'give' ) {
			0 ignored issues – show introduced 2018-03-14 12:39 UTC by Report Bug Copy Issue Report Found "=== '". Use Yoda Condition checks, you must Loading history...
239			$status = 'give';
			0 ignored issues – show introduced 2018-03-14 12:39 UTC by Report Bug Copy Issue Report Overridding WordPress globals is prohibited Loading history...
240			}
241
242			return $plugins;
243			}
244
245			add_filter( 'all_plugins', 'give_prepare_filter_addons' );
246

ravinderk / Give

Push — issues/2980 ( 6931f9 )

plugins.php ➔ give_recently_activated_addons() D

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like