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

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;
 
 /**

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;
 

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

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

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

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/Application.php

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

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

src/Contract/Core/Loader.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
 	 *
 	 * @param HasActions $service
 	 */
-	public function register_actions( HasActions $service );
+	public function register_actions(HasActions $service);
 
 	/**
 	 * Registers the service's filters with the loader.
@@ -28,5 +28,5 @@ 
 	 *
 	 * @param HasFilters $service
 	 */
-	public function register_filters( HasFilters $service );
+	public function register_filters(HasFilters $service);
 }