GV\Core

Complexity

Total Complexity

7

Size/Duplication

Total Lines	102
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	2

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	get()	0	5	2
A	__construct()	0	4	1
B	init()	0	38	2
A	__clone()	0	1	1
A	__wakeup()	0	1	1

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 15 and the first side effect is on line 5.

namespace GV;

/** If this file is called directly, abort. */
if ( ! defined( 'GRAVITYVIEW_DIR' ) )

It is generally a best practice to always use braces with control structures.

	die();

/**
 * The core GravityView API.
 *
 * Returned by the wrapper gravityview() global function, exposes
 * all the required public functionality and classes, sets up global
 * state depending on current request context, etc.
 */
final class Core {
	/**
	 * @var \GV\Core The \GV\Core static instance.
	 */
	private static $__instance = null;

	/**
	 * @var \GV\Plugin The WordPress plugin context.
	 *
	 * @api
	 * @since future
	 */
	public $plugin;

	/**
	 * @var \GV\Request The current request.
	 *
	 * @api
	 * @since future
	 */
	public $request;

	/**
	 * @var \GV\View_Collection The views attached to the current request.
	 *
	 * @see \GV\Request::$views A shortcut alias.
	 * @api
	 * @since future
	 */
	public $views;

	/**
	 * Get the global instance of \GV\Core.
	 *
	 * @return \GV\Core The global instance of GravityView Core.
	 */
	public static function get() {
		if ( ! self::$__instance instanceof self )

It is generally a best practice to always use braces with control structures.

			self::$__instance = new self;
		return self::$__instance;
	}

	/**
	 * Bootstrap.
	 *
	 * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

	 */
	private function __construct() {
		self::$__instance = $this;
		$this->init();
	}

	/**
	 * Early initialization.
	 *
	 * Loads dependencies, sets up the object, adds hooks, etc.
	 *
	 * @return void
	 */
	private function init() {
		require_once dirname( __FILE__ ) . '/class-gv-plugin.php';
		$this->plugin = Plugin::get();

		/**
		 * Stop all further functionality from loading if the WordPress
		 * plugin is incompatible with the current environment.
		 *
		 * @todo Output incompatibility notices.
		 */
		if ( ! $this->plugin->is_compatible() ) {
			return;
		}

		/** Register the gravityview post type upon WordPress core init. */
		require_once $this->plugin->dir( 'future/includes/class-gv-view.php' );
		add_action( 'init', array( '\GV\View', 'register_post_type' ) );

		/** Add rewrite endpoint for single-entry URLs. */
		require_once $this->plugin->dir( 'future/includes/class-gv-entry.php' );
		add_action( 'init', array( '\GV\Entry', 'add_rewrite_endpoint' ) );

		/** Generics */
		require_once $this->plugin->dir( 'future/includes/class-gv-collection.php' );
		require_once $this->plugin->dir( 'future/includes/class-gv-shortcode.php' );

		/** Shortcodes */
		require_once $this->plugin->dir( 'future/includes/class-gv-shortcode-gravityview.php' );
		// add_action( 'init', array( '\GV\Shortcodes\gravityview', 'add' ) ); // @todo uncomment when original is stubbed

56% of this comment could be valid code. Did you maybe forget this after debugging?

		/** Get the View_Collection ready. */
		require_once $this->plugin->dir( 'future/includes/class-gv-collection-view.php' );

		/** Initialize the current request. For now we assume a default WordPress frontent context. */
		require_once $this->plugin->dir( 'future/includes/class-gv-request.php' );
		$this->request = new Frontend_Request();
		$this->views = &$this->request->views;
	}

	private function __clone() { }

	private function __wakeup() { }
}