WPSiteMonitor\Settings_Menu

Complexity

Total Complexity

7

Size/Duplication

Total Lines	140
Duplicated Lines	0 %

Test Coverage

Coverage

76.74%

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	display_settings_page()	0	7	1
A	setting_input_html()	0	9	1
A	init_settings()	0	16	1
A	settings_page_html()	0	14	2
A	__construct()	0	10	1
A	setting_section_html()	0	3	1

<?php
/**
 * Register all settings and menus for the plugin.
 *
 * @package WPSiteMonitor
 * @link https://github.com/BWibrew/WP-Site-Monitor/
 * @author Benjamin Wibrew <benjamin.wibrew@gmail.com>
 * @since 1.0.0
 */

namespace WPSiteMonitor;

/**
 * Class Settings_Menu
 *
 * @package WPSiteMonitor
 */
class Settings_Menu {

	/**
	 * The setting option name.
	 *
	 * @var string
	 * @since 1.0.0
	 */
	const OPTION_NAME = 'wp_site_monitor_enable';

	/**
	 * The loader that's responsible for registering all hooks and filters.
	 *
	 * @var Hook_Loader
	 * @since 1.0.0
	 */
	protected $loader;

	/**
	 * Settings page slug.
	 *
	 * @var string
	 * @since 1.0.0
	 */
	protected $settings_page;

	/**
	 * Setting options group name.
	 *
	 * @var string
	 * @since 1.0.0
	 */
	protected $option_group;

	/**
	 * Initialise the plugin settings and menus.
	 *
	 * @since 1.0.0
	 */
	public function __construct() {
		$this->settings_page = 'wp_site_monitor';
		$this->option_group  = $this->settings_page;

		$this->loader = new Hook_Loader();

		$this->loader->add_action( 'admin_init', $this, 'init_settings' );
		$this->loader->add_action( 'admin_menu', $this, 'display_settings_page' );

		$this->loader->run();
	}

	/**
	 * Initialise plugin settings.
	 *
	 * @since 1.0.0
	 */
	public function init_settings() {
		register_setting( $this->option_group, self::OPTION_NAME );

		add_settings_section(
			self::OPTION_NAME . '_section',
			__( 'Enable/Disable WP Site Monitor', 'wp-site-monitor' ),
			array( $this, 'setting_section_html' ),
			$this->settings_page
		);

		add_settings_field(
			self::OPTION_NAME,
			__( 'Enable WP Site Monitor', 'wp-site-monitor' ),
			array( $this, 'setting_input_html' ),
			$this->settings_page,
			self::OPTION_NAME . '_section'
		);
	}

	/**
	 * Display the admin settings page.
	 *
	 * @since 1.0.0
	 */
	public function display_settings_page() {
		add_options_page(
			__( 'WP Site Monitor Settings', 'wp-site-monitor' ),
			__( 'WP Site Monitor', 'wp-site-monitor' ),
			'manage_options',
			$this->settings_page,
			array( $this, 'settings_page_html' )
		);
	}

	/**
	 * Settings page template.
	 *
	 * @since 1.0.0
	 */
	public function settings_page_html() {
		if ( ! current_user_can( 'manage_options' ) ) {
			return;
		}
		?>
		<div class="wrap">
			<h1><?php echo esc_html( get_admin_page_title() ); ?></h1>
			<form action="options.php" method="POST">
				<?php
				settings_fields( $this->option_group );
				do_settings_sections( $this->settings_page );
				submit_button();
				?>
			</form>
		</div>
		<?php
	}

	/**
	 * Form text.
	 *
	 * @param array $args Arguments passed into add_settings_section.
	 *
	 * @since 1.0.0
	 */
	public function setting_section_html( $args ) {
		?>
		<p id="<?php echo esc_attr( $args['id'] ); ?>"><?php esc_html_e( 'Use these options to control which extra data is available over the REST API.', 'wp-site-monitor' ); ?></p>

The call to esc_attr() has too many arguments starting with $args['id'].

Are you sure the usage of esc_attr($args['id']) is correct as it seems to always return null.

		<?php
	}

	/**
	 * Form inputs.
	 *
	 * @since 1.0.0
	 */
	public function setting_input_html() {
		?>
		<input type="checkbox"
			name="<?php esc_attr( self::OPTION_NAME ); ?>"

The call to esc_attr() has too many arguments starting with self::OPTION_NAME.

			id=""<?php checked( get_option( self::OPTION_NAME ), 1 ); ?> value="1">

		<p class="description">
			<?php esc_html_e( 'This checkbox enables/disables all plugin functionality.', 'wp-site-monitor' ); ?>
		</p>
		<?php
	}
}