Issues in debug-bar-cron.php - All Issues - Inspection of "Merge pull request #54 from tollmanz/feature/phpco..." - jrfnl/debug-bar-cron - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 02d327...35c1cd )

by Juliette

created 2016-08-26 03:15 UTC

debug-bar-cron.php (4 issues)

Labels

Severity

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php
/**
 * Debug Bar Cron, a WordPress plugin.
 *
 * @package     WordPress\Plugins\Debug Bar Cron
 * @author      Zack Tollman, Helen Hou-Sandi, Juliette Reinders Folmer
 * @link        https://github.com/tollmanz/debug-bar-cron
 * @version     0.1.2
 * @license     http://creativecommons.org/licenses/GPL/2.0/ GNU General Public License, version 2 or higher
 *
 * @wordpress-plugin
 * Plugin Name:	Debug Bar Cron
 * Plugin URI:	http://wordpress.org/extend/plugins/debug-bar-cron/
 * Description:	Debug Bar Cron adds information about WP scheduled events to the Debug Bar.
 * Version:		0.1.2
 * Author:		Zack Tollman, Helen Hou-Sandi
 * Author URI:	http://github.com/tollmanz/
 * Depends:     Debug Bar
 * Text Domain:	debug-bar-cron
 * Domain Path:	/languages/
 */

// Avoid direct calls to this file.
if ( ! function_exists( 'add_action' ) ) {
	header( 'Status: 403 Forbidden' );
	header( 'HTTP/1.1 403 Forbidden' );
	exit();
}

if ( ! function_exists( 'debug_bar_cron_has_parent_plugin' ) ) {
	/**
	 * Show admin notice & de-activate if debug-bar plugin not active.
	 */
	function debug_bar_cron_has_parent_plugin() {
		$file = plugin_basename( __FILE__ );

		if ( is_admin() && ( ! class_exists( 'Debug_Bar' ) && current_user_can( 'activate_plugins' ) ) && is_plugin_active( $file ) ) {
			add_action( 'admin_notices', create_function( null, 'echo \'<div class="error"><p>\', sprintf( __( \'Activation failed: Debug Bar must be activated to use the <strong>Debug Bar Cron</strong> Plugin. %sVisit your plugins page to activate.\', \'debug-bar-cron\' ), \'<a href="\' . esc_url( admin_url( \'plugins.php#debug-bar\' ) ) . \'">\' ), \'</a></p></div>\';' ) );
// Instead of
$function = create_function('$a, $b', 'return $a + $b');

// Better use
$function = function($a, $b) { return $a + $b; }

			deactivate_plugins( $file, false, is_network_admin() );

			// Add to recently active plugins list.
			if ( ! is_network_admin() ) {
				update_option( 'recently_activated', ( array( $file => time() ) + (array) get_option( 'recently_activated' ) ) );
			} else {
				update_site_option( 'recently_activated', ( array( $file => time() ) + (array) get_site_option( 'recently_activated' ) ) );
			}

			// Prevent trying again on page reload.
			if ( isset( $_GET['activate'] ) ) {

				unset( $_GET['activate'] );

			}
		}
	}
	add_action( 'admin_init', 'debug_bar_cron_has_parent_plugin' );
}


if ( ! function_exists( 'zt_add_debug_bar_cron_panel' ) ) {
	/**
	 * Adds panel, as defined in the included class, to Debug Bar.
	 *
	 * @param array $panels Existing debug bar panels.
	 *
	 * @return array
	 */
	function zt_add_debug_bar_cron_panel( $panels ) {
		if ( ! class_exists( 'ZT_Debug_Bar_Cron' ) ) {
			require_once 'class-debug-bar-cron.php';
			$panels[] = new ZT_Debug_Bar_Cron();
		}
		return $panels;
	}
	add_filter( 'debug_bar_panels', 'zt_add_debug_bar_cron_panel' );
}


1			<?php
2			/**
3			* Debug Bar Cron, a WordPress plugin.
4			*
5			* @package WordPress\Plugins\Debug Bar Cron
6			* @author Zack Tollman, Helen Hou-Sandi, Juliette Reinders Folmer
7			* @link https://github.com/tollmanz/debug-bar-cron
8			* @version 0.1.2
9			* @license http://creativecommons.org/licenses/GPL/2.0/ GNU General Public License, version 2 or higher
10			*
11			* @wordpress-plugin
12			* Plugin Name: Debug Bar Cron
13			* Plugin URI: http://wordpress.org/extend/plugins/debug-bar-cron/
14			* Description: Debug Bar Cron adds information about WP scheduled events to the Debug Bar.
15			* Version: 0.1.2
16			* Author: Zack Tollman, Helen Hou-Sandi
17			* Author URI: http://github.com/tollmanz/
18			* Depends: Debug Bar
19			* Text Domain: debug-bar-cron
20			* Domain Path: /languages/
21			*/
22
23			// Avoid direct calls to this file.
24			if ( ! function_exists( 'add_action' ) ) {
25			header( 'Status: 403 Forbidden' );
26			header( 'HTTP/1.1 403 Forbidden' );
27			exit();
28			}
29
30			if ( ! function_exists( 'debug_bar_cron_has_parent_plugin' ) ) {
31			/**
32			* Show admin notice & de-activate if debug-bar plugin not active.
33			*/
34			function debug_bar_cron_has_parent_plugin() {
35			$file = plugin_basename( __FILE__ );
36
37			if ( is_admin() && ( ! class_exists( 'Debug_Bar' ) && current_user_can( 'activate_plugins' ) ) && is_plugin_active( $file ) ) {
38			add_action( 'admin_notices', create_function( null, 'echo \'<div class="error"><p>\', sprintf( __( \'Activation failed: Debug Bar must be activated to use the <strong>Debug Bar Cron</strong> Plugin. %sVisit your plugins page to activate.\', \'debug-bar-cron\' ), \'<a href="\' . esc_url( admin_url( \'plugins.php#debug-bar\' ) ) . \'">\' ), \'</a></p></div>\';' ) );
			0 ignored issues – show introduced 2016-04-29 05:01 UTC by Report Bug Copy Issue Report Show Similar Issues like this create_function is discouraged, please use Anonymous functions instead. Loading history... Security Best Practice introduced 2015-12-13 02:53 UTC by Report Bug Copy Issue Report Show Similar Issues like this The use of `create_function` is highly discouraged, better use a closure. `create_function` can pose a great security vulnerability as it is similar to `eval`, and could be used for arbitrary code execution. We highly recommend to use a closure instead. // Instead of $function = create_function('$a, $b', 'return $a + $b'); // Better use $function = function($a, $b) { return $a + $b; } Loading history...
39
40			deactivate_plugins( $file, false, is_network_admin() );
41
42			// Add to recently active plugins list.
43			if ( ! is_network_admin() ) {
44			update_option( 'recently_activated', ( array( $file => time() ) + (array) get_option( 'recently_activated' ) ) );
45			} else {
46			update_site_option( 'recently_activated', ( array( $file => time() ) + (array) get_site_option( 'recently_activated' ) ) );
47			}
48
49			// Prevent trying again on page reload.
50			if ( isset( $_GET['activate'] ) ) {
			0 ignored issues – show introduced 2016-04-29 05:01 UTC by Report Bug Copy Issue Report Show Similar Issues like this Detected access of super global var $_GET, probably need manual inspection. Loading history...
51			unset( $_GET['activate'] );
			0 ignored issues – show introduced 2016-04-29 05:01 UTC by Report Bug Copy Issue Report Show Similar Issues like this Detected access of super global var $_GET, probably need manual inspection. Loading history...
52			}
53			}
54			}
55			add_action( 'admin_init', 'debug_bar_cron_has_parent_plugin' );
56			}
57
58
59			if ( ! function_exists( 'zt_add_debug_bar_cron_panel' ) ) {
60			/**
61			* Adds panel, as defined in the included class, to Debug Bar.
62			*
63			* @param array $panels Existing debug bar panels.
64			*
65			* @return array
66			*/
67			function zt_add_debug_bar_cron_panel( $panels ) {
68			if ( ! class_exists( 'ZT_Debug_Bar_Cron' ) ) {
69			require_once 'class-debug-bar-cron.php';
70			$panels[] = new ZT_Debug_Bar_Cron();
71			}
72			return $panels;
73			}
74			add_filter( 'debug_bar_panels', 'zt_add_debug_bar_cron_panel' );
75			}
76

jrfnl / debug-bar-cron

Push — master ( 02d327...35c1cd )

debug-bar-cron.php (4 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like