intraxia / jaxion

src/Core/Application.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 use Intraxia\Jaxion\Contract\Core\HasActions;
 use Intraxia\Jaxion\Contract\Core\HasFilters;
 use Intraxia\Jaxion\Contract\Core\Loader as LoaderContract;
-use WP_CLI;
 
 /**
  * Class Application

Indentation +128 added lines, -128 removed lines

@@ -11,132 +11,132 @@
  * @package Intraxia\Jaxion
  */
 class Application extends Container implements ApplicationContract {
-	/**
-	 * Singleton instance of the Application object
-	 *
-	 * @var Application
-	 */
-	protected static $instance = null;
-
-	/**
-	 * {@inheritdoc}
-	 *
-	 * @param string $file
-	 * @param array  $providers
-	 *
-	 * @throws ApplicationAlreadyBootedException
-	 */
-	public function __construct( $file, array $providers = array() ) {
-		if ( static::$instance !== null ) {
-			throw new ApplicationAlreadyBootedException;
-		}
-
-		static::$instance = $this;
-
-		parent::__construct( $providers );
-		$this->register_constants( $file );
-		$this->register_core_services();
-		$this->load_i18n();
-
-		register_activation_hook( $file, array( $this, 'activate' ) );
-		register_deactivation_hook( $file, array( $this, 'deactivate' ) );
-	}
-
-	/**
-	 * {@inheritDoc}
-	 */
-	public function boot() {
-		$loader = $this->fetch( 'loader' );
-
-		if ( ! $loader instanceof LoaderContract ) {
-			throw new \UnexpectedValueException;
-		}
-
-		foreach ( $this as $alias => $value ) {
-			if ( $value instanceof HasActions ) {
-				$loader->register_actions( $value );
-			}
-
-			if ( $value instanceof HasFilters ) {
-				$loader->register_filters( $value );
-			}
-		}
-
-		add_action( 'plugins_loaded', array( $loader, 'run' ) );
-	}
-
-	/**
-	 * {@inheritdoc}
-	 *
-	 * @codeCoverageIgnore
-	 */
-	public function activate() {
-		// no-op
-	}
-
-	/**
-	 * {@inheritdoc}
-	 *
-	 * @codeCoverageIgnore
-	 */
-	public function deactivate() {
-		// no-op
-	}
-
-	/**
-	 * {@inheritDoc}
-	 *
-	 * @return Application
-	 * @throws ApplicationNotBootedException
-	 */
-	public static function instance() {
-		if ( static::$instance === null ) {
-			throw new ApplicationNotBootedException;
-		}
-
-		return static::$instance;
-	}
-
-	/**
-	 * {@inheritDoc}
-	 */
-	public static function shutdown() {
-		if ( static::$instance !== null ) {
-			static::$instance = null;
-		}
-	}
-
-	/**
-	 * Sets the plugin's url, path, and basename.
-	 *
-	 * @param string $file
-	 */
-	private function register_constants( $file ) {
-		$this->share( 'url', plugin_dir_url( $file ) );
-		$this->share( 'path', plugin_dir_path( $file ) );
-		$this->share( 'basename', plugin_basename( $file ) );
-
-		$plugin_data = get_plugin_data( $file, false, false );
-		$this->share( 'version', isset( $plugin_data['Version'] ) ? $plugin_data['Version'] : null );
-	}
-
-	/**
-	 * Registers the built-in services with the Application container.
-	 */
-	private function register_core_services() {
-		$this->share( array( 'loader' => 'Intraxia\Jaxion\Contract\Core\Loader' ), function ( $app ) {
-			return new Loader( $app );
-		} );
-	}
-
-	/**
-	 * Load's the plugin's translation files.
-	 */
-	private function load_i18n() {
-		load_plugin_textdomain(
-			$this->fetch( 'basename' ),
-			false,
-			basename( $this->fetch( 'path' ) ) . '/languages/'
-		);
-	}
+    /**
+     * Singleton instance of the Application object
+     *
+     * @var Application
+     */
+    protected static $instance = null;
+
+    /**
+     * {@inheritdoc}
+     *
+     * @param string $file
+     * @param array  $providers
+     *
+     * @throws ApplicationAlreadyBootedException
+     */
+    public function __construct( $file, array $providers = array() ) {
+        if ( static::$instance !== null ) {
+            throw new ApplicationAlreadyBootedException;
+        }
+
+        static::$instance = $this;
+
+        parent::__construct( $providers );
+        $this->register_constants( $file );
+        $this->register_core_services();
+        $this->load_i18n();
+
+        register_activation_hook( $file, array( $this, 'activate' ) );
+        register_deactivation_hook( $file, array( $this, 'deactivate' ) );
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function boot() {
+        $loader = $this->fetch( 'loader' );
+
+        if ( ! $loader instanceof LoaderContract ) {
+            throw new \UnexpectedValueException;
+        }
+
+        foreach ( $this as $alias => $value ) {
+            if ( $value instanceof HasActions ) {
+                $loader->register_actions( $value );
+            }
+
+            if ( $value instanceof HasFilters ) {
+                $loader->register_filters( $value );
+            }
+        }
+
+        add_action( 'plugins_loaded', array( $loader, 'run' ) );
+    }
+
+    /**
+     * {@inheritdoc}
+     *
+     * @codeCoverageIgnore
+     */
+    public function activate() {
+        // no-op
+    }
+
+    /**
+     * {@inheritdoc}
+     *
+     * @codeCoverageIgnore
+     */
+    public function deactivate() {
+        // no-op
+    }
+
+    /**
+     * {@inheritDoc}
+     *
+     * @return Application
+     * @throws ApplicationNotBootedException
+     */
+    public static function instance() {
+        if ( static::$instance === null ) {
+            throw new ApplicationNotBootedException;
+        }
+
+        return static::$instance;
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public static function shutdown() {
+        if ( static::$instance !== null ) {
+            static::$instance = null;
+        }
+    }
+
+    /**
+     * Sets the plugin's url, path, and basename.
+     *
+     * @param string $file
+     */
+    private function register_constants( $file ) {
+        $this->share( 'url', plugin_dir_url( $file ) );
+        $this->share( 'path', plugin_dir_path( $file ) );
+        $this->share( 'basename', plugin_basename( $file ) );
+
+        $plugin_data = get_plugin_data( $file, false, false );
+        $this->share( 'version', isset( $plugin_data['Version'] ) ? $plugin_data['Version'] : null );
+    }
+
+    /**
+     * Registers the built-in services with the Application container.
+     */
+    private function register_core_services() {
+        $this->share( array( 'loader' => 'Intraxia\Jaxion\Contract\Core\Loader' ), function ( $app ) {
+            return new Loader( $app );
+        } );
+    }
+
+    /**
+     * Load's the plugin's translation files.
+     */
+    private function load_i18n() {
+        load_plugin_textdomain(
+            $this->fetch( 'basename' ),
+            false,
+            basename( $this->fetch( 'path' ) ) . '/languages/'
+        );
+    }
 }

Spacing +26 added lines, -26 removed lines

@@ -26,43 +26,43 @@ 
 	 *
 	 * @throws ApplicationAlreadyBootedException
 	 */
-	public function __construct( $file, array $providers = array() ) {
-		if ( static::$instance !== null ) {
+	public function __construct($file, array $providers = array()) {
+		if (static::$instance !== null) {
 			throw new ApplicationAlreadyBootedException;
 		}
 
 		static::$instance = $this;
 
-		parent::__construct( $providers );
-		$this->register_constants( $file );
+		parent::__construct($providers);
+		$this->register_constants($file);
 		$this->register_core_services();
 		$this->load_i18n();
 
-		register_activation_hook( $file, array( $this, 'activate' ) );
-		register_deactivation_hook( $file, array( $this, 'deactivate' ) );
+		register_activation_hook($file, array($this, 'activate'));
+		register_deactivation_hook($file, array($this, 'deactivate'));
 	}
 
 	/**
 	 * {@inheritDoc}
 	 */
 	public function boot() {
-		$loader = $this->fetch( 'loader' );
+		$loader = $this->fetch('loader');
 
-		if ( ! $loader instanceof LoaderContract ) {
+		if (!$loader instanceof LoaderContract) {
 			throw new \UnexpectedValueException;
 		}
 
-		foreach ( $this as $alias => $value ) {
-			if ( $value instanceof HasActions ) {
-				$loader->register_actions( $value );
+		foreach ($this as $alias => $value) {
+			if ($value instanceof HasActions) {
+				$loader->register_actions($value);
 			}
 
-			if ( $value instanceof HasFilters ) {
-				$loader->register_filters( $value );
+			if ($value instanceof HasFilters) {
+				$loader->register_filters($value);
 			}
 		}
 
-		add_action( 'plugins_loaded', array( $loader, 'run' ) );
+		add_action('plugins_loaded', array($loader, 'run'));
 	}
 
 	/**
@@ -90,7 +90,7 @@ 
 	 * @throws ApplicationNotBootedException
 	 */
 	public static function instance() {
-		if ( static::$instance === null ) {
+		if (static::$instance === null) {
 			throw new ApplicationNotBootedException;
 		}
 
@@ -101,7 +101,7 @@ 
 	 * {@inheritDoc}
 	 */
 	public static function shutdown() {
-		if ( static::$instance !== null ) {
+		if (static::$instance !== null) {
 			static::$instance = null;
 		}
 	}
@@ -111,21 +111,21 @@ 
 	 *
 	 * @param string $file
 	 */
-	private function register_constants( $file ) {
-		$this->share( 'url', plugin_dir_url( $file ) );
-		$this->share( 'path', plugin_dir_path( $file ) );
-		$this->share( 'basename', plugin_basename( $file ) );
+	private function register_constants($file) {
+		$this->share('url', plugin_dir_url($file));
+		$this->share('path', plugin_dir_path($file));
+		$this->share('basename', plugin_basename($file));
 
-		$plugin_data = get_plugin_data( $file, false, false );
-		$this->share( 'version', isset( $plugin_data['Version'] ) ? $plugin_data['Version'] : null );
+		$plugin_data = get_plugin_data($file, false, false);
+		$this->share('version', isset($plugin_data['Version']) ? $plugin_data['Version'] : null);
 	}
 
 	/**
 	 * Registers the built-in services with the Application container.
 	 */
 	private function register_core_services() {
-		$this->share( array( 'loader' => 'Intraxia\Jaxion\Contract\Core\Loader' ), function ( $app ) {
-			return new Loader( $app );
+		$this->share(array('loader' => 'Intraxia\Jaxion\Contract\Core\Loader'), function($app) {
+			return new Loader($app);
 		} );
 	}
 
@@ -134,9 +134,9 @@ 
 	 */
 	private function load_i18n() {
 		load_plugin_textdomain(
-			$this->fetch( 'basename' ),
+			$this->fetch('basename'),
 			false,
-			basename( $this->fetch( 'path' ) ) . '/languages/'
+			basename($this->fetch('path')) . '/languages/'
 		);
 	}
 }

src/Http/Guard.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace Intraxia\Jaxion\Http;
 
 use Intraxia\Jaxion\Contract\Http\Guard as GuardContract;
-use Intraxia\Jaxion\Utility\Str;
 use WP_Error;
 
 /**

Indentation +65 added lines, -65 removed lines

@@ -15,77 +15,77 @@
  * @subpackage Http
  */
 class Guard implements GuardContract {
-	/**
-	 * Default options.
-	 *
-	 * @var array
-	 */
-	protected $defaults = array(
-		'rule'     => 'public',
-		'callback' => false,
-	);
+    /**
+     * Default options.
+     *
+     * @var array
+     */
+    protected $defaults = array(
+        'rule'     => 'public',
+        'callback' => false,
+    );
 
-	/**
-	 * Guard options.
-	 *
-	 * @var array
-	 */
-	protected $options;
+    /**
+     * Guard options.
+     *
+     * @var array
+     */
+    protected $options;
 
-	/**
-	 * Instantiate a new Guard with provided options.
-	 *
-	 * @param array $options
-	 */
-	public function __construct( array $options = array() ) {
-		$this->options = $this->set_defaults( $options );
-	}
+    /**
+     * Instantiate a new Guard with provided options.
+     *
+     * @param array $options
+     */
+    public function __construct( array $options = array() ) {
+        $this->options = $this->set_defaults( $options );
+    }
 
-	/**
-	 * Validates whether the current user is authorized.
-	 *
-	 * @return true|WP_Error
-	 */
-	public function authorized() {
-		// if the rule is public, always authorized
-		if ( 'public' === $this->options['rule'] ) {
-			return true;
-		}
+    /**
+     * Validates whether the current user is authorized.
+     *
+     * @return true|WP_Error
+     */
+    public function authorized() {
+        // if the rule is public, always authorized
+        if ( 'public' === $this->options['rule'] ) {
+            return true;
+        }
 
-		// enable passing in callback
-		if ( 'callback' === $this->options['rule'] && is_callable( $this->options['callback'] ) ) {
-			return call_user_func( $this->options['callback'] );
-		}
+        // enable passing in callback
+        if ( 'callback' === $this->options['rule'] && is_callable( $this->options['callback'] ) ) {
+            return call_user_func( $this->options['callback'] );
+        }
 
-		// map rule to method
-		if ( method_exists( $this, $method = $this->options['rule'] ) ) {
-			return $this->{$method}();
-		}
+        // map rule to method
+        if ( method_exists( $this, $method = $this->options['rule'] ) ) {
+            return $this->{$method}();
+        }
 
-		// disable in rule is misconfigused
-		// @todo set up internal translations
-		// @todo also, this error message kinda sucks
-		return new WP_Error( '500', __( 'Guard failure', 'jaxion' ) );
-	}
+        // disable in rule is misconfigused
+        // @todo set up internal translations
+        // @todo also, this error message kinda sucks
+        return new WP_Error( '500', __( 'Guard failure', 'jaxion' ) );
+    }
 
-	/**
-	 * Checks whether the current user can edit other's posts.
-	 *
-	 * @return bool
-	 */
-	protected function can_edit_others_posts() {
-		return current_user_can( 'edit_others_posts' ) ?: new WP_Error( '401', __( 'Unauthorized user', 'jaxion' ) );
-	}
+    /**
+     * Checks whether the current user can edit other's posts.
+     *
+     * @return bool
+     */
+    protected function can_edit_others_posts() {
+        return current_user_can( 'edit_others_posts' ) ?: new WP_Error( '401', __( 'Unauthorized user', 'jaxion' ) );
+    }
 
-	/**
-	 * Sets the default params for the Guard options.
-	 *
-	 * @param array $options
-	 *
-	 * @return array
-	 */
-	protected function set_defaults( $options ) {
-		// these are the valid options
-		return wp_parse_args( $options, $this->defaults );
-	}
+    /**
+     * Sets the default params for the Guard options.
+     *
+     * @param array $options
+     *
+     * @return array
+     */
+    protected function set_defaults( $options ) {
+        // these are the valid options
+        return wp_parse_args( $options, $this->defaults );
+    }
 }

Spacing +10 added lines, -10 removed lines

@@ -37,8 +37,8 @@ 
 	 *
 	 * @param array $options
 	 */
-	public function __construct( array $options = array() ) {
-		$this->options = $this->set_defaults( $options );
+	public function __construct(array $options = array()) {
+		$this->options = $this->set_defaults($options);
 	}
 
 	/**
@@ -48,24 +48,24 @@ 
 	 */
 	public function authorized() {
 		// if the rule is public, always authorized
-		if ( 'public' === $this->options['rule'] ) {
+		if ('public' === $this->options['rule']) {
 			return true;
 		}
 
 		// enable passing in callback
-		if ( 'callback' === $this->options['rule'] && is_callable( $this->options['callback'] ) ) {
-			return call_user_func( $this->options['callback'] );
+		if ('callback' === $this->options['rule'] && is_callable($this->options['callback'])) {
+			return call_user_func($this->options['callback']);
 		}
 
 		// map rule to method
-		if ( method_exists( $this, $method = $this->options['rule'] ) ) {
+		if (method_exists($this, $method = $this->options['rule'])) {
 			return $this->{$method}();
 		}
 
 		// disable in rule is misconfigused
 		// @todo set up internal translations
 		// @todo also, this error message kinda sucks
-		return new WP_Error( '500', __( 'Guard failure', 'jaxion' ) );
+		return new WP_Error('500', __('Guard failure', 'jaxion'));
 	}
 
 	/**
@@ -74,7 +74,7 @@ 
 	 * @return bool
 	 */
 	protected function can_edit_others_posts() {
-		return current_user_can( 'edit_others_posts' ) ?: new WP_Error( '401', __( 'Unauthorized user', 'jaxion' ) );
+		return current_user_can('edit_others_posts') ?: new WP_Error('401', __('Unauthorized user', 'jaxion'));
 	}
 
 	/**
@@ -84,8 +84,8 @@ 
 	 *
 	 * @return array
 	 */
-	protected function set_defaults( $options ) {
+	protected function set_defaults($options) {
 		// these are the valid options
-		return wp_parse_args( $options, $this->defaults );
+		return wp_parse_args($options, $this->defaults);
 	}
 }

src/Model/Base.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 namespace Intraxia\Jaxion\Model;
 
-use Intraxia\Jaxion\Utility\Str;
 use stdClass;
 use WP_Post;
 

Indentation +254 added lines, -254 removed lines

@@ -17,258 +17,258 @@
  * @since 0.1.0
  */
 abstract class Base {
-	/**
-	 * Model attributes array.
-	 *
-	 * @var array
-	 */
-	private $attributes = array(
-		'table' => array(),
-		'post'  => null,
-	);
-
-	/**
-	 * Which custom table does this model uses.
-	 *
-	 * If false, model wil fall back to postmeta.
-	 *
-	 * @var bool|string
-	 */
-	protected $table = false;
-
-	/**
-	 * Whether to use WP_Post mappings.
-	 *
-	 * @var bool
-	 */
-	protected $post = true;
-
-	/**
-	 * Custom post type.
-	 *
-	 * @var bool|string
-	 */
-	protected $type = false;
-
-	/**
-	 * Properties which are allowed to be set on the model.
-	 *
-	 * If this array is empty, any attributes can be set on the model.
-	 *
-	 * @var string[]
-	 */
-	protected $fillable = array();
-
-	/**
-	 * Constructs a new model with provided attributes.
-	 *
-	 * If 'post' is passed as one of the attributes
-	 *
-	 * @param array <string, mixed> $attributes
-	 */
-	public function __construct( array $attributes = array() ) {
-		foreach ( $attributes as $name => $value ) {
-			$this->set_attribute( $name, $value );
-		}
-
-		if ( $this->post && ! isset( $this->attributes['post'] ) ) {
-			$this->create_default_post();
-		}
-	}
-
-	/**
-	 * Get the model's attributes.
-	 *
-	 * Returns the array of for the model that will either need to be
-	 * saved in postmeta or a separate table.
-	 *
-	 * @return array
-	 */
-	public function get_attributes() {
-		return $this->attributes['table'];
-	}
-
-	/**
-	 * Get the model's underlying post.
-	 *
-	 * Returns the underlying WP_Post object for the model, representing
-	 * the data that will be save in the wp_posts table.
-	 *
-	 * @return false|WP_Post
-	 */
-	public function get_underlying_post() {
-		if ( isset( $this->attributes['post'] ) ) {
-			return $this->attributes['post'];
-		}
-
-		return false;
-	}
-
-	/**
-	 * Magic __set method.
-	 *
-	 * Passes the name and value to set_attribute, which is where the magic happens.
-	 *
-	 * @param string $name
-	 * @param mixed  $value
-	 */
-	public function __set( $name, $value ) {
-		$this->set_attribute( $name, $value );
-	}
-
-	/**
-	 * Sets the model attributes.
-	 *
-	 * Checks whether the model attribute can be set, check if it
-	 * maps to the WP_Post property, otherwise, assigns it to the
-	 * table attribute array.
-	 *
-	 * @param string $name
-	 * @param mixed  $value
-	 */
-	private function set_attribute( $name, $value ) {
-		if ( 'post' === $name ) {
-			$this->override_post( $value );
-
-			return;
-		}
-
-		if ( ! $this->is_fillable( $name ) ) {
-			return;
-		}
-
-		if ( $method = $this->has_map_method( $name ) ) {
-			$this->attributes['post']->{$this->{$method}()} = $value;
-
-			return;
-		}
-
-		$this->attributes['table'][ $name ] = $value;
-	}
-
-	/**
-	 * Checks if a given attribute is mass-fillable.
-	 *
-	 * Returns true if the attribute can be filled, false if it can't.
-	 *
-	 * @param string $name
-	 *
-	 * @return bool
-	 */
-	private function is_fillable( $name ) {
-		// `type` is not fillable at all.
-		if ( 'type' === $name ) {
-			return false;
-		}
-
-		// If the `$fillable` array hasn't been defined, pass everything.
-		if ( ! $this->fillable ) {
-			return true;
-		}
-
-		return in_array( $name, $this->fillable );
-	}
-
-	/**
-	 * Overrides the current WP_Post with a provided one.
-	 *
-	 * Resets the post's default values and stores it in the attributes.
-	 *
-	 * @param WP_Post $value
-	 */
-	private function override_post( WP_Post $value ) {
-		$this->attributes['post'] = $this->enforce_post_defaults( $value );
-	}
-
-	/**
-	 * Create and set with a new blank post.
-	 *
-	 * Creates a new WP_Post object, assigns it the default attributes,
-	 * and stores it in the attributes.
-	 */
-	private function create_default_post() {
-		$this->attributes['post'] = $this->enforce_post_defaults( new WP_Post( new stdClass ) );
-	}
-
-	/**
-	 * Enforces values on the post that can't change.
-	 *
-	 * Primarily, this is used to make sure the post_type always maps
-	 * to the model's "$type" property, but this can all be overridden
-	 * by the developer to enforce other values in the model.
-	 *
-	 * @param WP_Post $post
-	 *
-	 * @return WP_Post
-	 */
-	protected function enforce_post_defaults( WP_Post $post ) {
-		if ( is_string( $this->type ) ) {
-			$post->post_type = $this->type;
-		}
-
-		return $post;
-	}
-
-	/**
-	 * Magic __get method.
-	 *
-	 * Passes the name and value to get_attribute, which is where the magic happens.
-	 *
-	 * @param string $name
-	 *
-	 * @return mixed
-	 */
-	public function __get( $name ) {
-		return $this->get_attribute( $name );
-	}
-
-	/**
-	 * Retrieves the model attribute.
-	 *
-	 * If the attribute maps to the WP_Post, retrieves it from there.
-	 * Otherwise, checks if it's in the attributes array
-	 *
-	 * @param string $name
-	 *
-	 * @return mixed
-	 *
-	 * @throws PropertyDoesNotExistException If property isn't found.
-	 */
-	protected function get_attribute( $name ) {
-		if ( 'type' === $name ) {
-			return $this->type;
-		}
-
-		if ( $method = $this->has_map_method( $name ) ) {
-			$value = $this->attributes['post']->{$this->{$method}()};
-		} else {
-			if ( ! isset( $this->attributes['table'][ $name ] ) ) {
-				throw new PropertyDoesNotExistException;
-			}
-
-			$value = $this->attributes['table'][ $name ];
-		}
-
-		return $value;
-
-	}
-
-	/**
-	 * Checks whether the attribute has a map method.
-	 *
-	 * This is used to determine whether the attribute maps to a
-	 * property on the underlying WP_Post object. Returns the
-	 * method if one exists, returns false if it doesn't.
-	 *
-	 * @param string $name
-	 *
-	 * @return false|string
-	 */
-	protected function has_map_method( $name ) {
-		if ( method_exists( $this, $method = "map_{$name}" ) ) {
-			return $method;
-		}
-
-		return false;
-	}
+    /**
+     * Model attributes array.
+     *
+     * @var array
+     */
+    private $attributes = array(
+        'table' => array(),
+        'post'  => null,
+    );
+
+    /**
+     * Which custom table does this model uses.
+     *
+     * If false, model wil fall back to postmeta.
+     *
+     * @var bool|string
+     */
+    protected $table = false;
+
+    /**
+     * Whether to use WP_Post mappings.
+     *
+     * @var bool
+     */
+    protected $post = true;
+
+    /**
+     * Custom post type.
+     *
+     * @var bool|string
+     */
+    protected $type = false;
+
+    /**
+     * Properties which are allowed to be set on the model.
+     *
+     * If this array is empty, any attributes can be set on the model.
+     *
+     * @var string[]
+     */
+    protected $fillable = array();
+
+    /**
+     * Constructs a new model with provided attributes.
+     *
+     * If 'post' is passed as one of the attributes
+     *
+     * @param array <string, mixed> $attributes
+     */
+    public function __construct( array $attributes = array() ) {
+        foreach ( $attributes as $name => $value ) {
+            $this->set_attribute( $name, $value );
+        }
+
+        if ( $this->post && ! isset( $this->attributes['post'] ) ) {
+            $this->create_default_post();
+        }
+    }
+
+    /**
+     * Get the model's attributes.
+     *
+     * Returns the array of for the model that will either need to be
+     * saved in postmeta or a separate table.
+     *
+     * @return array
+     */
+    public function get_attributes() {
+        return $this->attributes['table'];
+    }
+
+    /**
+     * Get the model's underlying post.
+     *
+     * Returns the underlying WP_Post object for the model, representing
+     * the data that will be save in the wp_posts table.
+     *
+     * @return false|WP_Post
+     */
+    public function get_underlying_post() {
+        if ( isset( $this->attributes['post'] ) ) {
+            return $this->attributes['post'];
+        }
+
+        return false;
+    }
+
+    /**
+     * Magic __set method.
+     *
+     * Passes the name and value to set_attribute, which is where the magic happens.
+     *
+     * @param string $name
+     * @param mixed  $value
+     */
+    public function __set( $name, $value ) {
+        $this->set_attribute( $name, $value );
+    }
+
+    /**
+     * Sets the model attributes.
+     *
+     * Checks whether the model attribute can be set, check if it
+     * maps to the WP_Post property, otherwise, assigns it to the
+     * table attribute array.
+     *
+     * @param string $name
+     * @param mixed  $value
+     */
+    private function set_attribute( $name, $value ) {
+        if ( 'post' === $name ) {
+            $this->override_post( $value );
+
+            return;
+        }
+
+        if ( ! $this->is_fillable( $name ) ) {
+            return;
+        }
+
+        if ( $method = $this->has_map_method( $name ) ) {
+            $this->attributes['post']->{$this->{$method}()} = $value;
+
+            return;
+        }
+
+        $this->attributes['table'][ $name ] = $value;
+    }
+
+    /**
+     * Checks if a given attribute is mass-fillable.
+     *
+     * Returns true if the attribute can be filled, false if it can't.
+     *
+     * @param string $name
+     *
+     * @return bool
+     */
+    private function is_fillable( $name ) {
+        // `type` is not fillable at all.
+        if ( 'type' === $name ) {
+            return false;
+        }
+
+        // If the `$fillable` array hasn't been defined, pass everything.
+        if ( ! $this->fillable ) {
+            return true;
+        }
+
+        return in_array( $name, $this->fillable );
+    }
+
+    /**
+     * Overrides the current WP_Post with a provided one.
+     *
+     * Resets the post's default values and stores it in the attributes.
+     *
+     * @param WP_Post $value
+     */
+    private function override_post( WP_Post $value ) {
+        $this->attributes['post'] = $this->enforce_post_defaults( $value );
+    }
+
+    /**
+     * Create and set with a new blank post.
+     *
+     * Creates a new WP_Post object, assigns it the default attributes,
+     * and stores it in the attributes.
+     */
+    private function create_default_post() {
+        $this->attributes['post'] = $this->enforce_post_defaults( new WP_Post( new stdClass ) );
+    }
+
+    /**
+     * Enforces values on the post that can't change.
+     *
+     * Primarily, this is used to make sure the post_type always maps
+     * to the model's "$type" property, but this can all be overridden
+     * by the developer to enforce other values in the model.
+     *
+     * @param WP_Post $post
+     *
+     * @return WP_Post
+     */
+    protected function enforce_post_defaults( WP_Post $post ) {
+        if ( is_string( $this->type ) ) {
+            $post->post_type = $this->type;
+        }
+
+        return $post;
+    }
+
+    /**
+     * Magic __get method.
+     *
+     * Passes the name and value to get_attribute, which is where the magic happens.
+     *
+     * @param string $name
+     *
+     * @return mixed
+     */
+    public function __get( $name ) {
+        return $this->get_attribute( $name );
+    }
+
+    /**
+     * Retrieves the model attribute.
+     *
+     * If the attribute maps to the WP_Post, retrieves it from there.
+     * Otherwise, checks if it's in the attributes array
+     *
+     * @param string $name
+     *
+     * @return mixed
+     *
+     * @throws PropertyDoesNotExistException If property isn't found.
+     */
+    protected function get_attribute( $name ) {
+        if ( 'type' === $name ) {
+            return $this->type;
+        }
+
+        if ( $method = $this->has_map_method( $name ) ) {
+            $value = $this->attributes['post']->{$this->{$method}()};
+        } else {
+            if ( ! isset( $this->attributes['table'][ $name ] ) ) {
+                throw new PropertyDoesNotExistException;
+            }
+
+            $value = $this->attributes['table'][ $name ];
+        }
+
+        return $value;
+
+    }
+
+    /**
+     * Checks whether the attribute has a map method.
+     *
+     * This is used to determine whether the attribute maps to a
+     * property on the underlying WP_Post object. Returns the
+     * method if one exists, returns false if it doesn't.
+     *
+     * @param string $name
+     *
+     * @return false|string
+     */
+    protected function has_map_method( $name ) {
+        if ( method_exists( $this, $method = "map_{$name}" ) ) {
+            return $method;
+        }
+
+        return false;
+    }
 }

Spacing +31 added lines, -31 removed lines

@@ -66,12 +66,12 @@ 
 	 *
 	 * @param array <string, mixed> $attributes
 	 */
-	public function __construct( array $attributes = array() ) {
-		foreach ( $attributes as $name => $value ) {
-			$this->set_attribute( $name, $value );
+	public function __construct(array $attributes = array()) {
+		foreach ($attributes as $name => $value) {
+			$this->set_attribute($name, $value);
 		}
 
-		if ( $this->post && ! isset( $this->attributes['post'] ) ) {
+		if ($this->post && !isset($this->attributes['post'])) {
 			$this->create_default_post();
 		}
 	}
@@ -97,7 +97,7 @@ 
 	 * @return false|WP_Post
 	 */
 	public function get_underlying_post() {
-		if ( isset( $this->attributes['post'] ) ) {
+		if (isset($this->attributes['post'])) {
 			return $this->attributes['post'];
 		}
 
@@ -112,8 +112,8 @@ 
 	 * @param string $name
 	 * @param mixed  $value
 	 */
-	public function __set( $name, $value ) {
-		$this->set_attribute( $name, $value );
+	public function __set($name, $value) {
+		$this->set_attribute($name, $value);
 	}
 
 	/**
@@ -126,24 +126,24 @@ 
 	 * @param string $name
 	 * @param mixed  $value
 	 */
-	private function set_attribute( $name, $value ) {
-		if ( 'post' === $name ) {
-			$this->override_post( $value );
+	private function set_attribute($name, $value) {
+		if ('post' === $name) {
+			$this->override_post($value);
 
 			return;
 		}
 
-		if ( ! $this->is_fillable( $name ) ) {
+		if (!$this->is_fillable($name)) {
 			return;
 		}
 
-		if ( $method = $this->has_map_method( $name ) ) {
+		if ($method = $this->has_map_method($name)) {
 			$this->attributes['post']->{$this->{$method}()} = $value;
 
 			return;
 		}
 
-		$this->attributes['table'][ $name ] = $value;
+		$this->attributes['table'][$name] = $value;
 	}
 
 	/**
@@ -155,18 +155,18 @@ 
 	 *
 	 * @return bool
 	 */
-	private function is_fillable( $name ) {
+	private function is_fillable($name) {
 		// `type` is not fillable at all.
-		if ( 'type' === $name ) {
+		if ('type' === $name) {
 			return false;
 		}
 
 		// If the `$fillable` array hasn't been defined, pass everything.
-		if ( ! $this->fillable ) {
+		if (!$this->fillable) {
 			return true;
 		}
 
-		return in_array( $name, $this->fillable );
+		return in_array($name, $this->fillable);
 	}
 
 	/**
@@ -176,8 +176,8 @@ 
 	 *
 	 * @param WP_Post $value
 	 */
-	private function override_post( WP_Post $value ) {
-		$this->attributes['post'] = $this->enforce_post_defaults( $value );
+	private function override_post(WP_Post $value) {
+		$this->attributes['post'] = $this->enforce_post_defaults($value);
 	}
 
 	/**
@@ -187,7 +187,7 @@ 
 	 * and stores it in the attributes.
 	 */
 	private function create_default_post() {
-		$this->attributes['post'] = $this->enforce_post_defaults( new WP_Post( new stdClass ) );
+		$this->attributes['post'] = $this->enforce_post_defaults(new WP_Post(new stdClass));
 	}
 
 	/**
@@ -201,8 +201,8 @@ 
 	 *
 	 * @return WP_Post
 	 */
-	protected function enforce_post_defaults( WP_Post $post ) {
-		if ( is_string( $this->type ) ) {
+	protected function enforce_post_defaults(WP_Post $post) {
+		if (is_string($this->type)) {
 			$post->post_type = $this->type;
 		}
 
@@ -218,8 +218,8 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function __get( $name ) {
-		return $this->get_attribute( $name );
+	public function __get($name) {
+		return $this->get_attribute($name);
 	}
 
 	/**
@@ -234,19 +234,19 @@ 
 	 *
 	 * @throws PropertyDoesNotExistException If property isn't found.
 	 */
-	protected function get_attribute( $name ) {
-		if ( 'type' === $name ) {
+	protected function get_attribute($name) {
+		if ('type' === $name) {
 			return $this->type;
 		}
 
-		if ( $method = $this->has_map_method( $name ) ) {
+		if ($method = $this->has_map_method($name)) {
 			$value = $this->attributes['post']->{$this->{$method}()};
 		} else {
-			if ( ! isset( $this->attributes['table'][ $name ] ) ) {
+			if (!isset($this->attributes['table'][$name])) {
 				throw new PropertyDoesNotExistException;
 			}
 
-			$value = $this->attributes['table'][ $name ];
+			$value = $this->attributes['table'][$name];
 		}
 
 		return $value;
@@ -264,8 +264,8 @@ 
 	 *
 	 * @return false|string
 	 */
-	protected function has_map_method( $name ) {
-		if ( method_exists( $this, $method = "map_{$name}" ) ) {
+	protected function has_map_method($name) {
+		if (method_exists($this, $method = "map_{$name}")) {
 			return $method;
 		}
 

src/Core/Loader.php

Indentation +89 added lines, -89 removed lines

@@ -14,100 +14,100 @@
  * @subpackage Core
  */
 class Loader implements LoaderContract {
-	/**
-	 * Array of action hooks to attach.
-	 *
-	 * @var array[]
-	 */
-	protected $actions = array();
+    /**
+     * Array of action hooks to attach.
+     *
+     * @var array[]
+     */
+    protected $actions = array();
 
-	/**
-	 * Array of filter hooks to attach.
-	 *
-	 * @var array[]
-	 */
-	protected $filters = array();
+    /**
+     * Array of filter hooks to attach.
+     *
+     * @var array[]
+     */
+    protected $filters = array();
 
-	/**
-	 * {@inheritDoc}
-	 */
-	public function run() {
-		foreach ( $this->actions as $action ) {
-			add_action(
-				$action['hook'],
-				array( $action['service'], $action['method'] ),
-				$action['priority'],
-				$action['args']
-			);
-		}
+    /**
+     * {@inheritDoc}
+     */
+    public function run() {
+        foreach ( $this->actions as $action ) {
+            add_action(
+                $action['hook'],
+                array( $action['service'], $action['method'] ),
+                $action['priority'],
+                $action['args']
+            );
+        }
 
-		foreach ( $this->filters as $filter ) {
-			add_filter(
-				$filter['hook'],
-				array( $filter['service'], $filter['method'] ),
-				$filter['priority'],
-				$filter['args']
-			);
-		}
-	}
+        foreach ( $this->filters as $filter ) {
+            add_filter(
+                $filter['hook'],
+                array( $filter['service'], $filter['method'] ),
+                $filter['priority'],
+                $filter['args']
+            );
+        }
+    }
 
-	/**
-	 * Registers a single Service's actions with WordPress.
-	 *
-	 * @param HasActions $service
-	 */
-	public function register_actions( HasActions $service ) {
-		foreach ( $service->action_hooks() as $action ) {
-			$this->actions = $this->add(
-				$this->actions,
-				$action['hook'],
-				$service,
-				$action['method'],
-				isset( $action['priority'] ) ? $action['priority'] : 10,
-				isset( $action['args'] ) ? $action['args'] : 1
-			);
-		}
-	}
+    /**
+     * Registers a single Service's actions with WordPress.
+     *
+     * @param HasActions $service
+     */
+    public function register_actions( HasActions $service ) {
+        foreach ( $service->action_hooks() as $action ) {
+            $this->actions = $this->add(
+                $this->actions,
+                $action['hook'],
+                $service,
+                $action['method'],
+                isset( $action['priority'] ) ? $action['priority'] : 10,
+                isset( $action['args'] ) ? $action['args'] : 1
+            );
+        }
+    }
 
-	/**
-	 * Registers a single Service's filters with WordPress.
-	 *
-	 * @param HasFilters $service
-	 */
-	public function register_filters( HasFilters $service ) {
-		foreach ( $service->filter_hooks() as $filter ) {
-			$this->filters = $this->add(
-				$this->filters,
-				$filter['hook'],
-				$service,
-				$filter['method'],
-				isset( $filter['priority'] ) ? $filter['priority'] : 10,
-				isset( $filter['args'] ) ? $filter['args'] : 1
-			);
-		}
-	}
+    /**
+     * Registers a single Service's filters with WordPress.
+     *
+     * @param HasFilters $service
+     */
+    public function register_filters( HasFilters $service ) {
+        foreach ( $service->filter_hooks() as $filter ) {
+            $this->filters = $this->add(
+                $this->filters,
+                $filter['hook'],
+                $service,
+                $filter['method'],
+                isset( $filter['priority'] ) ? $filter['priority'] : 10,
+                isset( $filter['args'] ) ? $filter['args'] : 1
+            );
+        }
+    }
 
-	/**
-	 * Utility to register the actions and hooks into a single collection.
-	 *
-	 * @param array  $hooks
-	 * @param string $hook
-	 * @param object $service
-	 * @param string $method
-	 * @param int    $priority
-	 * @param int    $accepted_args
-	 *
-	 * @return array
-	 */
-	protected function add( $hooks, $hook, $service, $method, $priority, $accepted_args ) {
-		$hooks[] = array(
-			'hook'     => $hook,
-			'service'  => $service,
-			'method'   => $method,
-			'priority' => $priority,
-			'args'     => $accepted_args,
-		);
+    /**
+     * Utility to register the actions and hooks into a single collection.
+     *
+     * @param array  $hooks
+     * @param string $hook
+     * @param object $service
+     * @param string $method
+     * @param int    $priority
+     * @param int    $accepted_args
+     *
+     * @return array
+     */
+    protected function add( $hooks, $hook, $service, $method, $priority, $accepted_args ) {
+        $hooks[] = array(
+            'hook'     => $hook,
+            'service'  => $service,
+            'method'   => $method,
+            'priority' => $priority,
+            'args'     => $accepted_args,
+        );
 
-		return $hooks;
-	}
+        return $hooks;
+    }
 }

Spacing +13 added lines, -13 removed lines

@@ -32,19 +32,19 @@ 
 	 * {@inheritDoc}
 	 */
 	public function run() {
-		foreach ( $this->actions as $action ) {
+		foreach ($this->actions as $action) {
 			add_action(
 				$action['hook'],
-				array( $action['service'], $action['method'] ),
+				array($action['service'], $action['method']),
 				$action['priority'],
 				$action['args']
 			);
 		}
 
-		foreach ( $this->filters as $filter ) {
+		foreach ($this->filters as $filter) {
 			add_filter(
 				$filter['hook'],
-				array( $filter['service'], $filter['method'] ),
+				array($filter['service'], $filter['method']),
 				$filter['priority'],
 				$filter['args']
 			);
@@ -56,15 +56,15 @@ 
 	 *
 	 * @param HasActions $service
 	 */
-	public function register_actions( HasActions $service ) {
-		foreach ( $service->action_hooks() as $action ) {
+	public function register_actions(HasActions $service) {
+		foreach ($service->action_hooks() as $action) {
 			$this->actions = $this->add(
 				$this->actions,
 				$action['hook'],
 				$service,
 				$action['method'],
-				isset( $action['priority'] ) ? $action['priority'] : 10,
-				isset( $action['args'] ) ? $action['args'] : 1
+				isset($action['priority']) ? $action['priority'] : 10,
+				isset($action['args']) ? $action['args'] : 1
 			);
 		}
 	}
@@ -74,15 +74,15 @@ 
 	 *
 	 * @param HasFilters $service
 	 */
-	public function register_filters( HasFilters $service ) {
-		foreach ( $service->filter_hooks() as $filter ) {
+	public function register_filters(HasFilters $service) {
+		foreach ($service->filter_hooks() as $filter) {
 			$this->filters = $this->add(
 				$this->filters,
 				$filter['hook'],
 				$service,
 				$filter['method'],
-				isset( $filter['priority'] ) ? $filter['priority'] : 10,
-				isset( $filter['args'] ) ? $filter['args'] : 1
+				isset($filter['priority']) ? $filter['priority'] : 10,
+				isset($filter['args']) ? $filter['args'] : 1
 			);
 		}
 	}
@@ -99,7 +99,7 @@ 
 	 *
 	 * @return array
 	 */
-	protected function add( $hooks, $hook, $service, $method, $priority, $accepted_args ) {
+	protected function add($hooks, $hook, $service, $method, $priority, $accepted_args) {
 		$hooks[] = array(
 			'hook'     => $hook,
 			'service'  => $service,

src/Core/Container.php

Indentation +305 added lines, -305 removed lines

@@ -13,309 +13,309 @@
  * @subpackage Core
  */
 class Container implements ContainerContract {
-	/**
-	 * Registered definitions.
-	 *
-	 * @var mixed[]
-	 */
-	private $definitions = array();
-
-	/**
-	 * Aliases to share between fetches.
-	 *
-	 * @var <string, true>[]
-	 */
-	private $shared = array();
-
-	/**
-	 * Aliases of all the registered services.
-	 *
-	 * @var <string, true>[]
-	 */
-	private $aliases = array();
-
-	/**
-	 * Array of classes registered on the container.
-	 *
-	 * @var <string, true>[]
-	 */
-	private $classes = array();
-
-	/**
-	 * Current position in the loop.
-	 *
-	 * @var int
-	 */
-	private $position;
-
-	/**
-	 * 0-indexed array of aliases for looping.
-	 *
-	 * @var string[]
-	 */
-	private $keys = array();
-
-	/**
-	 * Create a new container with the given providers.
-	 *
-	 * Providers can be instances or the class of the provider as a string.
-	 *
-	 * @param ServiceProvider[]|string[] $providers
-	 */
-	public function __construct( array $providers = array() ) {
-		foreach ( $providers as $provider ) {
-			if ( is_string( $provider ) && class_exists( $provider ) ) {
-				$provider = new $provider;
-			}
-
-			if ( $provider instanceof ServiceProvider ) {
-				$this->register( $provider );
-			}
-		}
-	}
-
-
-	/**
-	 * {@inheritdoc}
-	 *
-	 * @param string|array $alias
-	 * @param mixed        $definition
-	 *
-	 * @throws DefinedAliasException
-	 *
-	 * @return $this
-	 */
-	public function define( $alias, $definition ) {
-		if ( is_array( $alias ) ) {
-			$class = current( $alias );
-			$alias = key( $alias );
-		}
-
-		if ( isset( $this->aliases[ $alias ] ) ) {
-			throw new DefinedAliasException;
-		}
-
-		$this->aliases[ $alias ]     = true;
-		$this->definitions[ $alias ] = $definition;
-
-		// Closures are treated as factories unless
-		// defined via Container::share.
-		if ( ! $definition instanceof \Closure ) {
-			$this->shared[ $alias ] = true;
-		}
-
-		if ( isset( $class ) ) {
-			$this->classes[ $class ] = $alias;
-		}
-
-		return $this;
-	}
-
-	/**
-	 * {@inheritdoc}
-	 *
-	 * @param string|array $alias
-	 * @param mixed        $definition
-	 *
-	 * @throws DefinedAliasException
-	 *
-	 * @return $this
-	 */
-	public function share( $alias, $definition ) {
-		$this->define( $alias, $definition );
-
-		if ( is_array( $alias ) ) {
-			$alias = key( $alias );
-		}
-
-		$this->shared[ $alias ] = true;
-
-		return $this;
-	}
-
-	/**
-	 * {@inheritdoc}
-	 *
-	 * @param string $alias
-	 *
-	 * @throws UndefinedAliasException
-	 *
-	 * @return mixed
-	 */
-	public function fetch( $alias ) {
-		if ( isset( $this->classes[ $alias ] ) ) {
-			// If the alias is a class name,
-			// then retrieve its linked alias.
-			// This is only registered when
-			// registering using an array.
-			$alias = $this->classes[ $alias ];
-		}
-
-		if ( ! isset( $this->aliases[ $alias ] ) ) {
-			throw new UndefinedAliasException;
-		}
-
-		$value = $this->definitions[ $alias ];
-
-		// If the shared value is a closure,
-		// execute it and assign the result
-		// in place of the closure.
-		if ( $value instanceof \Closure ) {
-			$factory = $value;
-			$value   = $factory( $this );
-		}
-
-		// If the value is shared, save the shared value.
-		if ( isset( $this->shared[ $alias ] ) ) {
-			$this->definitions[ $alias ] = $value;
-		}
-
-		// Return the fetched value.
-		return $value;
-	}
-
-	/**
-	 * {@inheritdoc}
-	 *
-	 * @param  string $alias
-	 *
-	 * @return bool
-	 */
-	public function has( $alias ) {
-		return isset( $this->aliases[ $alias ] );
-	}
-
-	/**
-	 * {@inheritDoc}
-	 *
-	 * @param string $alias
-	 *
-	 * @return $this
-	 */
-	public function remove( $alias ) {
-		if ( isset( $this->aliases[ $alias ] ) ) {
-			/**
-			 * If there's no reference in the aliases array,
-			 * the service won't be found on fetching and
-			 * can be overwritten on setting.
-			 *
-			 * Pros: Quick setting/unsetting, faster
-			 * performance on those operations when doing
-			 * a lot of these.
-			 *
-			 * Cons: Objects and values set in the container
-			 * can't get garbage collected.
-			 *
-			 * If this is a problem, this may need to be revisited.
-			 */
-			unset( $this->aliases[ $alias ] );
-		}
-
-		return $this;
-	}
-
-	/**
-	 * {@inheritDoc}
-	 *
-	 * @param ServiceProvider $provider
-	 *
-	 * @return $this
-	 */
-	public function register( ServiceProvider $provider ) {
-		$provider->register( $this );
-
-		return $this;
-	}
-
-	/**
-	 * Set a value into the container.
-	 *
-	 * @param  string $id
-	 * @param  mixed  $value
-	 *
-	 * @see    define
-	 */
-	public function offsetSet( $id, $value ) {
-		$this->define( $id, $value );
-	}
-
-	/**
-	 * Get an value from the container.
-	 *
-	 * @param  string $id
-	 *
-	 * @return object
-	 * @throws UndefinedAliasException
-	 *
-	 * @see    fetch
-	 */
-	public function offsetGet( $id ) {
-		return $this->fetch( $id );
-	}
-
-	/**
-	 * Checks if a key is set on the container.
-	 *
-	 * @param  string $id
-	 *
-	 * @return bool
-	 *
-	 * @see    has
-	 */
-	public function offsetExists( $id ) {
-		return $this->has( $id );
-	}
-
-	/**
-	 * Remove a key from the container.
-	 *
-	 * @param string $id
-	 *
-	 * @see   remove
-	 */
-	public function offsetUnset( $id ) {
-		$this->remove( $id );
-	}
-
-	/**
-	 * Sets the object properties to prepare for the loop.
-	 */
-	public function rewind() {
-		$this->position = 0;
-		$this->keys     = array_keys( $this->aliases );
-	}
-
-	/**
-	 * Retrieves the service object for the current step in the loop.
-	 *
-	 * @return object
-	 */
-	public function current() {
-		return $this->fetch( $this->keys[ $this->position ] );
-	}
-
-	/**
-	 * Retrieves the key for the current step in the loop.
-	 *
-	 * @return string
-	 */
-	public function key() {
-		return $this->keys[ $this->position ];
-	}
-
-	/**
-	 * Increments to the next step in the loop.
-	 */
-	public function next() {
-		$this->position ++;
-	}
-
-	/**
-	 * Checks if the next step in the loop in valid.
-	 *
-	 * @return bool
-	 */
-	public function valid() {
-		return isset( $this->keys[ $this->position ] );
-	}
+    /**
+     * Registered definitions.
+     *
+     * @var mixed[]
+     */
+    private $definitions = array();
+
+    /**
+     * Aliases to share between fetches.
+     *
+     * @var <string, true>[]
+     */
+    private $shared = array();
+
+    /**
+     * Aliases of all the registered services.
+     *
+     * @var <string, true>[]
+     */
+    private $aliases = array();
+
+    /**
+     * Array of classes registered on the container.
+     *
+     * @var <string, true>[]
+     */
+    private $classes = array();
+
+    /**
+     * Current position in the loop.
+     *
+     * @var int
+     */
+    private $position;
+
+    /**
+     * 0-indexed array of aliases for looping.
+     *
+     * @var string[]
+     */
+    private $keys = array();
+
+    /**
+     * Create a new container with the given providers.
+     *
+     * Providers can be instances or the class of the provider as a string.
+     *
+     * @param ServiceProvider[]|string[] $providers
+     */
+    public function __construct( array $providers = array() ) {
+        foreach ( $providers as $provider ) {
+            if ( is_string( $provider ) && class_exists( $provider ) ) {
+                $provider = new $provider;
+            }
+
+            if ( $provider instanceof ServiceProvider ) {
+                $this->register( $provider );
+            }
+        }
+    }
+
+
+    /**
+     * {@inheritdoc}
+     *
+     * @param string|array $alias
+     * @param mixed        $definition
+     *
+     * @throws DefinedAliasException
+     *
+     * @return $this
+     */
+    public function define( $alias, $definition ) {
+        if ( is_array( $alias ) ) {
+            $class = current( $alias );
+            $alias = key( $alias );
+        }
+
+        if ( isset( $this->aliases[ $alias ] ) ) {
+            throw new DefinedAliasException;
+        }
+
+        $this->aliases[ $alias ]     = true;
+        $this->definitions[ $alias ] = $definition;
+
+        // Closures are treated as factories unless
+        // defined via Container::share.
+        if ( ! $definition instanceof \Closure ) {
+            $this->shared[ $alias ] = true;
+        }
+
+        if ( isset( $class ) ) {
+            $this->classes[ $class ] = $alias;
+        }
+
+        return $this;
+    }
+
+    /**
+     * {@inheritdoc}
+     *
+     * @param string|array $alias
+     * @param mixed        $definition
+     *
+     * @throws DefinedAliasException
+     *
+     * @return $this
+     */
+    public function share( $alias, $definition ) {
+        $this->define( $alias, $definition );
+
+        if ( is_array( $alias ) ) {
+            $alias = key( $alias );
+        }
+
+        $this->shared[ $alias ] = true;
+
+        return $this;
+    }
+
+    /**
+     * {@inheritdoc}
+     *
+     * @param string $alias
+     *
+     * @throws UndefinedAliasException
+     *
+     * @return mixed
+     */
+    public function fetch( $alias ) {
+        if ( isset( $this->classes[ $alias ] ) ) {
+            // If the alias is a class name,
+            // then retrieve its linked alias.
+            // This is only registered when
+            // registering using an array.
+            $alias = $this->classes[ $alias ];
+        }
+
+        if ( ! isset( $this->aliases[ $alias ] ) ) {
+            throw new UndefinedAliasException;
+        }
+
+        $value = $this->definitions[ $alias ];
+
+        // If the shared value is a closure,
+        // execute it and assign the result
+        // in place of the closure.
+        if ( $value instanceof \Closure ) {
+            $factory = $value;
+            $value   = $factory( $this );
+        }
+
+        // If the value is shared, save the shared value.
+        if ( isset( $this->shared[ $alias ] ) ) {
+            $this->definitions[ $alias ] = $value;
+        }
+
+        // Return the fetched value.
+        return $value;
+    }
+
+    /**
+     * {@inheritdoc}
+     *
+     * @param  string $alias
+     *
+     * @return bool
+     */
+    public function has( $alias ) {
+        return isset( $this->aliases[ $alias ] );
+    }
+
+    /**
+     * {@inheritDoc}
+     *
+     * @param string $alias
+     *
+     * @return $this
+     */
+    public function remove( $alias ) {
+        if ( isset( $this->aliases[ $alias ] ) ) {
+            /**
+             * If there's no reference in the aliases array,
+             * the service won't be found on fetching and
+             * can be overwritten on setting.
+             *
+             * Pros: Quick setting/unsetting, faster
+             * performance on those operations when doing
+             * a lot of these.
+             *
+             * Cons: Objects and values set in the container
+             * can't get garbage collected.
+             *
+             * If this is a problem, this may need to be revisited.
+             */
+            unset( $this->aliases[ $alias ] );
+        }
+
+        return $this;
+    }
+
+    /**
+     * {@inheritDoc}
+     *
+     * @param ServiceProvider $provider
+     *
+     * @return $this
+     */
+    public function register( ServiceProvider $provider ) {
+        $provider->register( $this );
+
+        return $this;
+    }
+
+    /**
+     * Set a value into the container.
+     *
+     * @param  string $id
+     * @param  mixed  $value
+     *
+     * @see    define
+     */
+    public function offsetSet( $id, $value ) {
+        $this->define( $id, $value );
+    }
+
+    /**
+     * Get an value from the container.
+     *
+     * @param  string $id
+     *
+     * @return object
+     * @throws UndefinedAliasException
+     *
+     * @see    fetch
+     */
+    public function offsetGet( $id ) {
+        return $this->fetch( $id );
+    }
+
+    /**
+     * Checks if a key is set on the container.
+     *
+     * @param  string $id
+     *
+     * @return bool
+     *
+     * @see    has
+     */
+    public function offsetExists( $id ) {
+        return $this->has( $id );
+    }
+
+    /**
+     * Remove a key from the container.
+     *
+     * @param string $id
+     *
+     * @see   remove
+     */
+    public function offsetUnset( $id ) {
+        $this->remove( $id );
+    }
+
+    /**
+     * Sets the object properties to prepare for the loop.
+     */
+    public function rewind() {
+        $this->position = 0;
+        $this->keys     = array_keys( $this->aliases );
+    }
+
+    /**
+     * Retrieves the service object for the current step in the loop.
+     *
+     * @return object
+     */
+    public function current() {
+        return $this->fetch( $this->keys[ $this->position ] );
+    }
+
+    /**
+     * Retrieves the key for the current step in the loop.
+     *
+     * @return string
+     */
+    public function key() {
+        return $this->keys[ $this->position ];
+    }
+
+    /**
+     * Increments to the next step in the loop.
+     */
+    public function next() {
+        $this->position ++;
+    }
+
+    /**
+     * Checks if the next step in the loop in valid.
+     *
+     * @return bool
+     */
+    public function valid() {
+        return isset( $this->keys[ $this->position ] );
+    }
 }

Spacing +50 added lines, -50 removed lines

@@ -62,14 +62,14 @@ 
 	 *
 	 * @param ServiceProvider[]|string[] $providers
 	 */
-	public function __construct( array $providers = array() ) {
-		foreach ( $providers as $provider ) {
-			if ( is_string( $provider ) && class_exists( $provider ) ) {
+	public function __construct(array $providers = array()) {
+		foreach ($providers as $provider) {
+			if (is_string($provider) && class_exists($provider)) {
 				$provider = new $provider;
 			}
 
-			if ( $provider instanceof ServiceProvider ) {
-				$this->register( $provider );
+			if ($provider instanceof ServiceProvider) {
+				$this->register($provider);
 			}
 		}
 	}
@@ -85,27 +85,27 @@ 
 	 *
 	 * @return $this
 	 */
-	public function define( $alias, $definition ) {
-		if ( is_array( $alias ) ) {
-			$class = current( $alias );
-			$alias = key( $alias );
+	public function define($alias, $definition) {
+		if (is_array($alias)) {
+			$class = current($alias);
+			$alias = key($alias);
 		}
 
-		if ( isset( $this->aliases[ $alias ] ) ) {
+		if (isset($this->aliases[$alias])) {
 			throw new DefinedAliasException;
 		}
 
-		$this->aliases[ $alias ]     = true;
-		$this->definitions[ $alias ] = $definition;
+		$this->aliases[$alias]     = true;
+		$this->definitions[$alias] = $definition;
 
 		// Closures are treated as factories unless
 		// defined via Container::share.
-		if ( ! $definition instanceof \Closure ) {
-			$this->shared[ $alias ] = true;
+		if (!$definition instanceof \Closure) {
+			$this->shared[$alias] = true;
 		}
 
-		if ( isset( $class ) ) {
-			$this->classes[ $class ] = $alias;
+		if (isset($class)) {
+			$this->classes[$class] = $alias;
 		}
 
 		return $this;
@@ -121,14 +121,14 @@ 
 	 *
 	 * @return $this
 	 */
-	public function share( $alias, $definition ) {
-		$this->define( $alias, $definition );
+	public function share($alias, $definition) {
+		$this->define($alias, $definition);
 
-		if ( is_array( $alias ) ) {
-			$alias = key( $alias );
+		if (is_array($alias)) {
+			$alias = key($alias);
 		}
 
-		$this->shared[ $alias ] = true;
+		$this->shared[$alias] = true;
 
 		return $this;
 	}
@@ -142,32 +142,32 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function fetch( $alias ) {
-		if ( isset( $this->classes[ $alias ] ) ) {
+	public function fetch($alias) {
+		if (isset($this->classes[$alias])) {
 			// If the alias is a class name,
 			// then retrieve its linked alias.
 			// This is only registered when
 			// registering using an array.
-			$alias = $this->classes[ $alias ];
+			$alias = $this->classes[$alias];
 		}
 
-		if ( ! isset( $this->aliases[ $alias ] ) ) {
+		if (!isset($this->aliases[$alias])) {
 			throw new UndefinedAliasException;
 		}
 
-		$value = $this->definitions[ $alias ];
+		$value = $this->definitions[$alias];
 
 		// If the shared value is a closure,
 		// execute it and assign the result
 		// in place of the closure.
-		if ( $value instanceof \Closure ) {
+		if ($value instanceof \Closure) {
 			$factory = $value;
-			$value   = $factory( $this );
+			$value   = $factory($this);
 		}
 
 		// If the value is shared, save the shared value.
-		if ( isset( $this->shared[ $alias ] ) ) {
-			$this->definitions[ $alias ] = $value;
+		if (isset($this->shared[$alias])) {
+			$this->definitions[$alias] = $value;
 		}
 
 		// Return the fetched value.
@@ -181,8 +181,8 @@ 
 	 *
 	 * @return bool
 	 */
-	public function has( $alias ) {
-		return isset( $this->aliases[ $alias ] );
+	public function has($alias) {
+		return isset($this->aliases[$alias]);
 	}
 
 	/**
@@ -192,8 +192,8 @@ 
 	 *
 	 * @return $this
 	 */
-	public function remove( $alias ) {
-		if ( isset( $this->aliases[ $alias ] ) ) {
+	public function remove($alias) {
+		if (isset($this->aliases[$alias])) {
 			/**
 			 * If there's no reference in the aliases array,
 			 * the service won't be found on fetching and
@@ -208,7 +208,7 @@ 
 			 *
 			 * If this is a problem, this may need to be revisited.
 			 */
-			unset( $this->aliases[ $alias ] );
+			unset($this->aliases[$alias]);
 		}
 
 		return $this;
@@ -221,8 +221,8 @@ 
 	 *
 	 * @return $this
 	 */
-	public function register( ServiceProvider $provider ) {
-		$provider->register( $this );
+	public function register(ServiceProvider $provider) {
+		$provider->register($this);
 
 		return $this;
 	}
@@ -235,8 +235,8 @@ 
 	 *
 	 * @see    define
 	 */
-	public function offsetSet( $id, $value ) {
-		$this->define( $id, $value );
+	public function offsetSet($id, $value) {
+		$this->define($id, $value);
 	}
 
 	/**
@@ -249,8 +249,8 @@ 
 	 *
 	 * @see    fetch
 	 */
-	public function offsetGet( $id ) {
-		return $this->fetch( $id );
+	public function offsetGet($id) {
+		return $this->fetch($id);
 	}
 
 	/**
@@ -262,8 +262,8 @@ 
 	 *
 	 * @see    has
 	 */
-	public function offsetExists( $id ) {
-		return $this->has( $id );
+	public function offsetExists($id) {
+		return $this->has($id);
 	}
 
 	/**
@@ -273,8 +273,8 @@ 
 	 *
 	 * @see   remove
 	 */
-	public function offsetUnset( $id ) {
-		$this->remove( $id );
+	public function offsetUnset($id) {
+		$this->remove($id);
 	}
 
 	/**
@@ -282,7 +282,7 @@ 
 	 */
 	public function rewind() {
 		$this->position = 0;
-		$this->keys     = array_keys( $this->aliases );
+		$this->keys     = array_keys($this->aliases);
 	}
 
 	/**
@@ -291,7 +291,7 @@ 
 	 * @return object
 	 */
 	public function current() {
-		return $this->fetch( $this->keys[ $this->position ] );
+		return $this->fetch($this->keys[$this->position]);
 	}
 
 	/**
@@ -300,14 +300,14 @@ 
 	 * @return string
 	 */
 	public function key() {
-		return $this->keys[ $this->position ];
+		return $this->keys[$this->position];
 	}
 
 	/**
 	 * Increments to the next step in the loop.
 	 */
 	public function next() {
-		$this->position ++;
+		$this->position++;
 	}
 
 	/**
@@ -316,6 +316,6 @@ 
 	 * @return bool
 	 */
 	public function valid() {
-		return isset( $this->keys[ $this->position ] );
+		return isset($this->keys[$this->position]);
 	}
 }

src/Model/Post.php

Indentation +64 added lines, -64 removed lines

@@ -18,80 +18,80 @@
  * @subpackage Model
  */
 class Post extends Base {
-	/**
-	 * Post type this model maps to.
-	 *
-	 * @var string
-	 */
-	protected $type = 'post';
+    /**
+     * Post type this model maps to.
+     *
+     * @var string
+     */
+    protected $type = 'post';
 
-	/**
-	 * Fillable attributes for the model.
-	 *
-	 * @var array
-	 */
-	protected $fillable = array( 'ID', 'author', 'slug', 'title', 'publish_date', 'content', 'excerpt' );
+    /**
+     * Fillable attributes for the model.
+     *
+     * @var array
+     */
+    protected $fillable = array( 'ID', 'author', 'slug', 'title', 'publish_date', 'content', 'excerpt' );
 
-	/**
-	 * ID property maps to ID.
-	 *
-	 * @return string
-	 */
-	protected function map_ID() {
-		return 'ID';
-	}
+    /**
+     * ID property maps to ID.
+     *
+     * @return string
+     */
+    protected function map_ID() {
+        return 'ID';
+    }
 
-	/**
-	 * Author property maps to post_author.
-	 *
-	 * @return string
-	 */
-	protected function map_author() {
-		return 'post_author';
-	}
+    /**
+     * Author property maps to post_author.
+     *
+     * @return string
+     */
+    protected function map_author() {
+        return 'post_author';
+    }
 
-	/**
-	 * Slug property maps to post_name.
-	 *
-	 * @return string
-	 */
-	protected function map_slug() {
-		return 'post_name';
-	}
+    /**
+     * Slug property maps to post_name.
+     *
+     * @return string
+     */
+    protected function map_slug() {
+        return 'post_name';
+    }
 
-	/**
-	 * Title property maps to post_title.
-	 *
-	 * @return string
-	 */
-	protected function map_title() {
-		return 'post_title';
-	}
+    /**
+     * Title property maps to post_title.
+     *
+     * @return string
+     */
+    protected function map_title() {
+        return 'post_title';
+    }
 
-	/**
-	 * Publish date property maps to post_date_gmt.
-	 *
-	 * @return string
-	 */
-	protected function map_publish_date() {
-		return 'post_date_gmt';
-	}
+    /**
+     * Publish date property maps to post_date_gmt.
+     *
+     * @return string
+     */
+    protected function map_publish_date() {
+        return 'post_date_gmt';
+    }
 
-	/**
-	 * Content property maps to post_content.
-	 *
-	 * @return string
-	 */
-	protected function map_content() {
-		return 'post_content';
-	}
+    /**
+     * Content property maps to post_content.
+     *
+     * @return string
+     */
+    protected function map_content() {
+        return 'post_content';
+    }
 
-	/**¡
+    /**¡
 	 * Excerpt property maps to post_excerpt.
 	 *
 	 * @return string
 	 */
-	protected function map_excerpt() {
-		return 'post_excerpt';
-	}
+    protected function map_excerpt() {
+        return 'post_excerpt';
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 	 *
 	 * @var array
 	 */
-	protected $fillable = array( 'ID', 'author', 'slug', 'title', 'publish_date', 'content', 'excerpt' );
+	protected $fillable = array('ID', 'author', 'slug', 'title', 'publish_date', 'content', 'excerpt');
 
 	/**
 	 * ID property maps to ID.

src/Contract/Core/HasFilters.php

Indentation +9 added lines, -9 removed lines

@@ -2,13 +2,13 @@
 namespace Intraxia\Jaxion\Contract\Core;
 
 interface HasFilters {
-	/**
-	 * Provides the array of filters the class wants to register with WordPress.
-	 *
-	 * These filters are retrieved by the Loader class and used to register the
-	 * correct service methods with WordPress.
-	 *
-	 * @return array[]
-	 */
-	public function filter_hooks();
+    /**
+     * Provides the array of filters the class wants to register with WordPress.
+     *
+     * These filters are retrieved by the Loader class and used to register the
+     * correct service methods with WordPress.
+     *
+     * @return array[]
+     */
+    public function filter_hooks();
 }
\ No newline at end of file

src/Contract/Core/Application.php

Indentation +55 added lines, -55 removed lines

@@ -4,64 +4,64 @@
 use Intraxia\Jaxion\Core\ApplicationNotBootedException;
 
 interface Application {
-	/**
-	 * Instantiates a new Application container.
-	 *
-	 * The Application constructor enforces the presence of of a single instance
-	 * of the Application. If an instance already exists, an Exception will be thrown.
-	 *
-	 * @param string $file
-	 *
-	 * @throws \Intraxia\Jaxion\Core\ApplicationAlreadyBootedException
-	 */
-	public function __construct( $file );
+    /**
+     * Instantiates a new Application container.
+     *
+     * The Application constructor enforces the presence of of a single instance
+     * of the Application. If an instance already exists, an Exception will be thrown.
+     *
+     * @param string $file
+     *
+     * @throws \Intraxia\Jaxion\Core\ApplicationAlreadyBootedException
+     */
+    public function __construct( $file );
 
-	/**
-	 * Starts up the Application.
-	 *
-	 * Retrieves the Application's loader instance, and registers all of the Application's
-	 * services with the loader. It loops through its own services, and any services that
-	 * implement the required interface are passed into the loader. It then attaches the
-	 * loader to the required WordPress hook.
-	 */
-	public function boot();
+    /**
+     * Starts up the Application.
+     *
+     * Retrieves the Application's loader instance, and registers all of the Application's
+     * services with the loader. It loops through its own services, and any services that
+     * implement the required interface are passed into the loader. It then attaches the
+     * loader to the required WordPress hook.
+     */
+    public function boot();
 
-	/**
-	 * Fired on plugin activation.
-	 *
-	 * This function is attached to `register_activation_hook` and is fired when the plugin is
-	 * activated by WordPress. This gives the developer a place to set up any options,
-	 * add any custom tables, or flush rewrite rules, as required.
-	 */
-	public function activate();
+    /**
+     * Fired on plugin activation.
+     *
+     * This function is attached to `register_activation_hook` and is fired when the plugin is
+     * activated by WordPress. This gives the developer a place to set up any options,
+     * add any custom tables, or flush rewrite rules, as required.
+     */
+    public function activate();
 
-	/**
-	 * Fired on plugin deactivation.
-	 *
-	 * This function is attached to `register_deactivation_hook` and is fired when the plugin
-	 * is deactivated by WordPress. This gives the developer a place to clean up anything left
-	 * behind by the plugin.
-	 */
-	public function deactivate();
+    /**
+     * Fired on plugin deactivation.
+     *
+     * This function is attached to `register_deactivation_hook` and is fired when the plugin
+     * is deactivated by WordPress. This gives the developer a place to clean up anything left
+     * behind by the plugin.
+     */
+    public function deactivate();
 
-	/**
-	 * Retrieves the booted Application instance.
-	 *
-	 * If the Application has not yet been booted, an Exception will be thrown.
-	 *
-	 * @return Application
-	 * @throws ApplicationNotBootedException
-	 */
-	public static function instance();
+    /**
+     * Retrieves the booted Application instance.
+     *
+     * If the Application has not yet been booted, an Exception will be thrown.
+     *
+     * @return Application
+     * @throws ApplicationNotBootedException
+     */
+    public static function instance();
 
-	/**
-	 * Shuts down the booted Application.
-	 *
-	 * If the Application has already been booted, the Application instance
-	 * will be destroyed by assigning it a null value, freeing it from memory.
-	 * However, the service objects will likely remain in memory if they've been
-	 * attached to hooks when this method is called. This function is primarily
-	 * for uniting testing to make sure you can boot a new instance for each test.
-	 */
-	public static function shutdown();
+    /**
+     * Shuts down the booted Application.
+     *
+     * If the Application has already been booted, the Application instance
+     * will be destroyed by assigning it a null value, freeing it from memory.
+     * However, the service objects will likely remain in memory if they've been
+     * attached to hooks when this method is called. This function is primarily
+     * for uniting testing to make sure you can boot a new instance for each test.
+     */
+    public static function shutdown();
 }

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 	 *
 	 * @throws \Intraxia\Jaxion\Core\ApplicationAlreadyBootedException
 	 */
-	public function __construct( $file );
+	public function __construct($file);
 
 	/**
 	 * Starts up the Application.

src/Contract/Core/ServiceProvider.php

Indentation +9 added lines, -9 removed lines

@@ -2,13 +2,13 @@
 namespace Intraxia\Jaxion\Contract\Core;
 
 interface ServiceProvider {
-	/**
-	 * Register the provider's services on the container.
-	 *
-	 * This method is passed the container to register on, giving the service provider
-	 * an opportunity to register its services on the container in an encapsulated way.
-	 *
-	 * @param Container $container
-	 */
-	public function register( Container $container );
+    /**
+     * Register the provider's services on the container.
+     *
+     * This method is passed the container to register on, giving the service provider
+     * an opportunity to register its services on the container in an encapsulated way.
+     *
+     * @param Container $container
+     */
+    public function register( Container $container );
 }
\ No newline at end of file

Spacing +1 added lines, -1 removed lines

@@ -10,5 +10,5 @@
 	 *
 	 * @param Container $container
 	 */
-	public function register( Container $container );
+	public function register(Container $container);
 }
\ No newline at end of file