lsx\LSX_Logger

Complexity

Total Complexity

13

Size/Duplication

Total Lines	114
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	0

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	init()	0	8	2
A	log()	0	3	1
B	output_log()	0	19	6
A	loop_through_logs()	0	7	2
A	wp_admin_bar_menu()	0	19	1

<?php

namespace lsx;

if ( ! class_exists( '\lsx\LSX_Logger' ) ) {

	/**
	 * LSX_Logger Main Class
	 *
	 * @package   LSX_Logger
	 * @author    LightSpeed
	 * @license   GPL-3.0+
	 * @link
	 * @copyright 2017 LightSpeedDevelopment
	 */
	class LSX_Logger {

		/**
		 * Holds instance of the class
		 *
		 * @since   1.1.0
		 * @var     \lsx\Geo_Content
		 */
		private static $instance;

		/**
		 * Holds the Logs of what happened
		 */
		private $logs = array();

		/**
		 * Constructor.
		 */
		public function __construct() {
			add_action( 'wp_before_admin_bar_render', array( $this, 'wp_admin_bar_menu' ) );
		}

		/**
		 * Return an instance of this class.
		 *
		 * @return  LSX_Logger  A single instance
		 */
		public static function init() {

			// If the single instance hasn't been set, set it now.
			if ( ! isset( self::$instance ) ) {
				self::$instance = new self();

It seems like new self() of type object<lsx\LSX_Logger> is incompatible with the declared type object<lsx\Geo_Content> of property $instance.

			}
			return self::$instance;

The expression self::$instance; of type lsx\LSX_Logger|lsx\Geo_Content adds the type lsx\Geo_Content to the return on line 49 which is incompatible with the return type documented by lsx\LSX_Logger::init of type lsx\LSX_Logger.

		}

		/**
		 * Adds a field to the log
		 *
		 * @param $plugin string
		 * @param $key string
		 * @param $message string
		 * @return  void
		 */
		public function log( $plugin = '', $key = '', $message = '' ) {
			$this->logs[ $plugin ][ $key ] = $message;
		}

		/**
		 * Gets a field from the API lookup object
		 *
		 * @param $plugin string | boolean
		 * @return  string
		 */
		public function output_log( $plugin = false ) {
			$return = '';
			if ( ! empty( $this->logs ) ) {
				foreach ( $this->logs as $plugin_key => $log ) {

					$return = '<div style="padding: 15px;">';
					if ( false !== $plugin && $plugin_key !== $plugin ) {
						continue;
					} else if ( false === $plugin ) {
						$return .= '<h4>' . $plugin_key . '</h4>';
					}
					$return .= '<ul>';
					$return .= $this->loop_through_logs( $log );
					$return .= '</ul>';
					$return .= '</div>';
				}
			}
			return $return;
		}

		/**
		 * Gets a field from the API lookup object
		 *
		 * @param $log_array array
		 * @return  string
		 */
		public function loop_through_logs( $log_array = array() ) {
			$return = '';
			foreach ( $log_array as $key => $message ) {
				$return .= '<li>' . $message . ' <small>(' . $key . ')</small></li>';
			}
			return $return;
		}

		/**
		 * Registers an admin bar menu so you can see the output of the log messages
		 *
		 * @return  void
		 */
		function wp_admin_bar_menu() {

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

			global $wp_admin_bar;
			$wp_admin_bar->add_menu( array(
				'parent' => false,
				'id' => 'lsx_logger',
				'title' => __( 'LSX Log' ), // link title
				'href' => '',
			));
			$wp_admin_bar->add_menu( array(
				'parent' => 'lsx_logger',
				'id' => 'lsx_logger_output',
				'title' => '', // link title
				'href' => '',
				'meta' => array(
					'html' => $this->output_log(),
					'class' => '',
				),
			));
		}

	}
}