intraxia / jaxion

src/Core/Container.php

Indentation +318 added lines, -318 removed lines

@@ -13,322 +13,322 @@
  * @subpackage Core
  */
 class Container implements ContainerContract {
-	/**
-	 * ServiceProvider names to register with the container.
-	 *
-	 * Can be overwritten to include predefined providers.
-	 *
-	 * @var string[]
-	 */
-	protected $providers = array();
-
-	/**
-	 * 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() ) {
-		// array_unique ensures we only register each provider once.
-		$providers = array_unique( array_merge( $this->providers, $providers ) );
-
-		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( $alias );
-		}
-
-		$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( $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 ) {
-			$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 ) {
-		// @todo make sure provider is only registered once
-		$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 ] );
-	}
+    /**
+     * ServiceProvider names to register with the container.
+     *
+     * Can be overwritten to include predefined providers.
+     *
+     * @var string[]
+     */
+    protected $providers = array();
+
+    /**
+     * 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() ) {
+        // array_unique ensures we only register each provider once.
+        $providers = array_unique( array_merge( $this->providers, $providers ) );
+
+        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( $alias );
+        }
+
+        $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( $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 ) {
+            $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 ) {
+        // @todo make sure provider is only registered once
+        $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 +53 added lines, -53 removed lines

@@ -71,17 +71,17 @@ 
 	 *
 	 * @param ServiceProvider[]|string[] $providers
 	 */
-	public function __construct( array $providers = array() ) {
+	public function __construct(array $providers = array()) {
 		// array_unique ensures we only register each provider once.
-		$providers = array_unique( array_merge( $this->providers, $providers ) );
+		$providers = array_unique(array_merge($this->providers, $providers));
 
-		foreach ( $providers as $provider ) {
-			if ( is_string( $provider ) && class_exists( $provider ) ) {
+		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);
 			}
 		}
 	}
@@ -97,27 +97,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 ] ) ) {
-			throw new DefinedAliasException( $alias );
+		if (isset($this->aliases[$alias])) {
+			throw new DefinedAliasException($alias);
 		}
 
-		$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;
@@ -133,14 +133,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;
 	}
@@ -154,32 +154,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 ] ) ) {
-			throw new UndefinedAliasException( $alias );
+		if (!isset($this->aliases[$alias])) {
+			throw new UndefinedAliasException($alias);
 		}
 
-		$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.
@@ -193,8 +193,8 @@ 
 	 *
 	 * @return bool
 	 */
-	public function has( $alias ) {
-		return isset( $this->aliases[ $alias ] );
+	public function has($alias) {
+		return isset($this->aliases[$alias]);
 	}
 
 	/**
@@ -204,8 +204,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
@@ -220,7 +220,7 @@ 
 			 *
 			 * If this is a problem, this may need to be revisited.
 			 */
-			unset( $this->aliases[ $alias ] );
+			unset($this->aliases[$alias]);
 		}
 
 		return $this;
@@ -233,9 +233,9 @@ 
 	 *
 	 * @return $this
 	 */
-	public function register( ServiceProvider $provider ) {
+	public function register(ServiceProvider $provider) {
 		// @todo make sure provider is only registered once
-		$provider->register( $this );
+		$provider->register($this);
 
 		return $this;
 	}
@@ -248,8 +248,8 @@ 
 	 *
 	 * @see    define
 	 */
-	public function offsetSet( $id, $value ) {
-		$this->define( $id, $value );
+	public function offsetSet($id, $value) {
+		$this->define($id, $value);
 	}
 
 	/**
@@ -262,8 +262,8 @@ 
 	 *
 	 * @see    fetch
 	 */
-	public function offsetGet( $id ) {
-		return $this->fetch( $id );
+	public function offsetGet($id) {
+		return $this->fetch($id);
 	}
 
 	/**
@@ -275,8 +275,8 @@ 
 	 *
 	 * @see    has
 	 */
-	public function offsetExists( $id ) {
-		return $this->has( $id );
+	public function offsetExists($id) {
+		return $this->has($id);
 	}
 
 	/**
@@ -286,8 +286,8 @@ 
 	 *
 	 * @see   remove
 	 */
-	public function offsetUnset( $id ) {
-		$this->remove( $id );
+	public function offsetUnset($id) {
+		$this->remove($id);
 	}
 
 	/**
@@ -295,7 +295,7 @@ 
 	 */
 	public function rewind() {
 		$this->position = 0;
-		$this->keys     = array_keys( $this->aliases );
+		$this->keys     = array_keys($this->aliases);
 	}
 
 	/**
@@ -304,7 +304,7 @@ 
 	 * @return object
 	 */
 	public function current() {
-		return $this->fetch( $this->keys[ $this->position ] );
+		return $this->fetch($this->keys[$this->position]);
 	}
 
 	/**
@@ -313,14 +313,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++;
 	}
 
 	/**
@@ -329,6 +329,6 @@ 
 	 * @return bool
 	 */
 	public function valid() {
-		return isset( $this->keys[ $this->position ] );
+		return isset($this->keys[$this->position]);
 	}
 }

src/Utility/Str.php

Indentation +32 added lines, -32 removed lines

@@ -11,39 +11,39 @@
  * @subpackage Utility
  */
 class Str {
-	/**
-	 * Determine if a given string starts with a given substring.
-	 *
-	 * @param  string       $haystack
-	 * @param  string|array $needles
-	 *
-	 * @return bool
-	 */
-	public static function starts_with( $haystack, $needles ) {
-		foreach ( (array) $needles as $needle ) {
-			if ( '' !== $needle && 0 === strpos( $haystack, $needle ) ) {
-				return true;
-			}
-		}
+    /**
+     * Determine if a given string starts with a given substring.
+     *
+     * @param  string       $haystack
+     * @param  string|array $needles
+     *
+     * @return bool
+     */
+    public static function starts_with( $haystack, $needles ) {
+        foreach ( (array) $needles as $needle ) {
+            if ( '' !== $needle && 0 === strpos( $haystack, $needle ) ) {
+                return true;
+            }
+        }
 
-		return false;
-	}
+        return false;
+    }
 
-	/**
-	 * Determine if a given string ends with a given substring.
-	 *
-	 * @param  string       $haystack
-	 * @param  string|array $needles
-	 *
-	 * @return bool
-	 */
-	public static function ends_with( $haystack, $needles ) {
-		foreach ( (array) $needles as $needle ) {
-			if ( substr( $haystack, - strlen( $needle ) ) === (string) $needle ) {
-				return true;
-			}
-		}
+    /**
+     * Determine if a given string ends with a given substring.
+     *
+     * @param  string       $haystack
+     * @param  string|array $needles
+     *
+     * @return bool
+     */
+    public static function ends_with( $haystack, $needles ) {
+        foreach ( (array) $needles as $needle ) {
+            if ( substr( $haystack, - strlen( $needle ) ) === (string) $needle ) {
+                return true;
+            }
+        }
 
-		return false;
-	}
+        return false;
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -19,9 +19,9 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function starts_with( $haystack, $needles ) {
-		foreach ( (array) $needles as $needle ) {
-			if ( '' !== $needle && 0 === strpos( $haystack, $needle ) ) {
+	public static function starts_with($haystack, $needles) {
+		foreach ((array) $needles as $needle) {
+			if ('' !== $needle && 0 === strpos($haystack, $needle)) {
 				return true;
 			}
 		}
@@ -37,9 +37,9 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function ends_with( $haystack, $needles ) {
-		foreach ( (array) $needles as $needle ) {
-			if ( substr( $haystack, - strlen( $needle ) ) === (string) $needle ) {
+	public static function ends_with($haystack, $needles) {
+		foreach ((array) $needles as $needle) {
+			if (substr($haystack, - strlen($needle)) === (string) $needle) {
 				return true;
 			}
 		}

src/Contract/Core/I18n.php

Indentation +6 added lines, -6 removed lines

@@ -2,10 +2,10 @@
 namespace Intraxia\Jaxion\Contract\Core;
 
 interface I18n {
-	/**
-	 * Loads the plugin's textdomain.
-	 *
-	 * @return void
-	 */
-	public function load_plugin_textdomain();
+    /**
+     * Loads the plugin's textdomain.
+     *
+     * @return void
+     */
+    public function load_plugin_textdomain();
 }

src/Core/I18n.php

Indentation +45 added lines, -45 removed lines

@@ -11,53 +11,53 @@
  * @subpackage Core
  */
 class I18n implements I18nContract, HasActions {
-	/**
-	 * Plugin basename
-	 *
-	 * @var string
-	 */
-	private $basename;
+    /**
+     * Plugin basename
+     *
+     * @var string
+     */
+    private $basename;
 
-	/**
-	 * Plugin path.
-	 *
-	 * @var string
-	 */
-	private $path;
+    /**
+     * Plugin path.
+     *
+     * @var string
+     */
+    private $path;
 
-	/**
-	 * I18n constructor.
-	 *
-	 * @param string $basename Plugin basename.
-	 * @param string $path     Plugin path.
-	 */
-	public function __construct( $basename, $path ) {
-		$this->basename = $basename;
-		$this->path = $path;
-	}
+    /**
+     * I18n constructor.
+     *
+     * @param string $basename Plugin basename.
+     * @param string $path     Plugin path.
+     */
+    public function __construct( $basename, $path ) {
+        $this->basename = $basename;
+        $this->path = $path;
+    }
 
-	/**
-	 * {@inheritdoc}
-	 */
-	public function load_plugin_textdomain() {
-		load_plugin_textdomain(
-			$this->basename,
-			false,
-			basename( $this->path ) . '/languages/'
-		);
-	}
+    /**
+     * {@inheritdoc}
+     */
+    public function load_plugin_textdomain() {
+        load_plugin_textdomain(
+            $this->basename,
+            false,
+            basename( $this->path ) . '/languages/'
+        );
+    }
 
-	/**
-	 * {@inheritDoc}
-	 *
-	 * @return array
-	 */
-	public function action_hooks() {
-		return array(
-			array(
-				'hook'   => 'init',
-				'method' => 'load_plugin_textdomain',
-			),
-		);
-	}
+    /**
+     * {@inheritDoc}
+     *
+     * @return array
+     */
+    public function action_hooks() {
+        return array(
+            array(
+                'hook'   => 'init',
+                'method' => 'load_plugin_textdomain',
+            ),
+        );
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
 	 * @param string $basename Plugin basename.
 	 * @param string $path     Plugin path.
 	 */
-	public function __construct( $basename, $path ) {
+	public function __construct($basename, $path) {
 		$this->basename = $basename;
 		$this->path = $path;
 	}
@@ -43,7 +43,7 @@ 
 		load_plugin_textdomain(
 			$this->basename,
 			false,
-			basename( $this->path ) . '/languages/'
+			basename($this->path).'/languages/'
 		);
 	}
 

src/Contract/Axolotl/EntityManager.php

Indentation +46 added lines, -46 removed lines

@@ -6,54 +6,54 @@
 use WP_Error;
 
 interface EntityManager {
-	/**
-	 * Get a single model of the provided class with the given ID.
-	 *
-	 * @param string $class Fully qualified class name of model.
-	 * @param int    $id    ID of the model.
-	 *
-	 * @return Model|WP_Error
-	 */
-	public function find( $class, $id );
+    /**
+     * Get a single model of the provided class with the given ID.
+     *
+     * @param string $class Fully qualified class name of model.
+     * @param int    $id    ID of the model.
+     *
+     * @return Model|WP_Error
+     */
+    public function find( $class, $id );
 
-	/**
-	 * Finds all the models of the provided class for the given params.
-	 *
-	 * This method will return an empty Collection if the query returns no models.
-	 *
-	 * @param string $class  Fully qualified class name of models to find.
-	 * @param array  $params Params to constrain the find.
-	 *
-	 * @return Collection|WP_Error
-	 */
-	public function find_by( $class, array $params = array() );
+    /**
+     * Finds all the models of the provided class for the given params.
+     *
+     * This method will return an empty Collection if the query returns no models.
+     *
+     * @param string $class  Fully qualified class name of models to find.
+     * @param array  $params Params to constrain the find.
+     *
+     * @return Collection|WP_Error
+     */
+    public function find_by( $class, array $params = array() );
 
-	/**
-	 * Saves a new model of the provided class with the given data.
-	 *
-	 * @param string $class
-	 * @param array  $data
-	 *
-	 * @return Model|WP_Error
-	 */
-	public function create( $class, array $data = array() );
+    /**
+     * Saves a new model of the provided class with the given data.
+     *
+     * @param string $class
+     * @param array  $data
+     *
+     * @return Model|WP_Error
+     */
+    public function create( $class, array $data = array() );
 
-	/**
-	 * Updates a model with its latest dataE.
-	 *
-	 * @param Model $model
-	 *
-	 * @return Model|WP_Error
-	 */
-	public function persist( Model $model );
+    /**
+     * Updates a model with its latest dataE.
+     *
+     * @param Model $model
+     *
+     * @return Model|WP_Error
+     */
+    public function persist( Model $model );
 
-	/**
-	 * Delete the provide
-	 *
-	 * @param Model $model
-	 * @param bool  $force
-	 *
-	 * @return mixed
-	 */
-	public function delete( Model $model, $force = false );
+    /**
+     * Delete the provide
+     *
+     * @param Model $model
+     * @param bool  $force
+     *
+     * @return mixed
+     */
+    public function delete( Model $model, $force = false );
 }

Spacing +5 added lines, -5 removed lines

@@ -14,7 +14,7 @@ 
 	 *
 	 * @return Model|WP_Error
 	 */
-	public function find( $class, $id );
+	public function find($class, $id);
 
 	/**
 	 * Finds all the models of the provided class for the given params.
@@ -26,7 +26,7 @@ 
 	 *
 	 * @return Collection|WP_Error
 	 */
-	public function find_by( $class, array $params = array() );
+	public function find_by($class, array $params = array());
 
 	/**
 	 * Saves a new model of the provided class with the given data.
@@ -36,7 +36,7 @@ 
 	 *
 	 * @return Model|WP_Error
 	 */
-	public function create( $class, array $data = array() );
+	public function create($class, array $data = array());
 
 	/**
 	 * Updates a model with its latest dataE.
@@ -45,7 +45,7 @@ 
 	 *
 	 * @return Model|WP_Error
 	 */
-	public function persist( Model $model );
+	public function persist(Model $model);
 
 	/**
 	 * Delete the provide
@@ -55,5 +55,5 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function delete( Model $model, $force = false );
+	public function delete(Model $model, $force = false);
 }

src/Core/Application.php

Indentation +133 added lines, -133 removed lines

@@ -13,137 +13,137 @@
  * @package Intraxia\Jaxion
  */
 class Application extends Container implements ApplicationContract {
-	/**
-	 * Define plugin version on Application.
-	 */
-	const VERSION = '';
-
-	/**
-	 * Singleton instance of the Application object
-	 *
-	 * @var Application[]
-	 */
-	protected static $instances = array();
-
-	/**
-	 * 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
-	 * @param array  $providers
-	 *
-	 * @throws ApplicationAlreadyBootedException
-	 */
-	public function __construct( $file, array $providers = array() ) {
-		if ( isset( static::$instances[ get_called_class() ] ) ) {
-			throw new ApplicationAlreadyBootedException;
-		}
-
-		static::$instances[ get_called_class() ] = $this;
-
-		$this->register_constants( $file );
-		$this->register_core_services();
-
-		register_activation_hook( $file, array( $this, 'activate' ) );
-		register_deactivation_hook( $file, array( $this, 'deactivate' ) );
-
-		parent::__construct( $providers );
-	}
-
-	/**
-	 * {@inheritDoc}
-	 *
-	 * @throws UnexpectedValueException
-	 */
-	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 );
-			}
-
-			if ( $value instanceof HasShortcode ) {
-				$loader->register_shortcode( $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 ( ! isset( static::$instances[ get_called_class() ] ) ) {
-			throw new ApplicationNotBootedException;
-		}
-
-		return static::$instances[ get_called_class() ];
-	}
-
-	/**
-	 * {@inheritDoc}
-	 */
-	public static function shutdown() {
-		if ( isset( static::$instances[ get_called_class() ] ) ) {
-			unset( static::$instances[ get_called_class() ] );
-		}
-	}
-
-	/**
-	 * 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', $basename = plugin_basename( $file ) );
-		$this->share( 'slug', dirname( $basename ) );
-		$this->share( 'version', static::VERSION );
-	}
-
-	/**
-	 * 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( 'i18n' => 'Intaxia\Jaxion\Contract\Core\I18n' ), function ( $app ) {
-			return new I18n( $app->fetch( 'basename' ), $app->fetch( 'path' ) );
-		} );
-	}
+    /**
+     * Define plugin version on Application.
+     */
+    const VERSION = '';
+
+    /**
+     * Singleton instance of the Application object
+     *
+     * @var Application[]
+     */
+    protected static $instances = array();
+
+    /**
+     * 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
+     * @param array  $providers
+     *
+     * @throws ApplicationAlreadyBootedException
+     */
+    public function __construct( $file, array $providers = array() ) {
+        if ( isset( static::$instances[ get_called_class() ] ) ) {
+            throw new ApplicationAlreadyBootedException;
+        }
+
+        static::$instances[ get_called_class() ] = $this;
+
+        $this->register_constants( $file );
+        $this->register_core_services();
+
+        register_activation_hook( $file, array( $this, 'activate' ) );
+        register_deactivation_hook( $file, array( $this, 'deactivate' ) );
+
+        parent::__construct( $providers );
+    }
+
+    /**
+     * {@inheritDoc}
+     *
+     * @throws UnexpectedValueException
+     */
+    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 );
+            }
+
+            if ( $value instanceof HasShortcode ) {
+                $loader->register_shortcode( $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 ( ! isset( static::$instances[ get_called_class() ] ) ) {
+            throw new ApplicationNotBootedException;
+        }
+
+        return static::$instances[ get_called_class() ];
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public static function shutdown() {
+        if ( isset( static::$instances[ get_called_class() ] ) ) {
+            unset( static::$instances[ get_called_class() ] );
+        }
+    }
+
+    /**
+     * 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', $basename = plugin_basename( $file ) );
+        $this->share( 'slug', dirname( $basename ) );
+        $this->share( 'version', static::VERSION );
+    }
+
+    /**
+     * 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( 'i18n' => 'Intaxia\Jaxion\Contract\Core\I18n' ), function ( $app ) {
+            return new I18n( $app->fetch( 'basename' ), $app->fetch( 'path' ) );
+        } );
+    }
 }

Spacing +31 added lines, -31 removed lines

@@ -36,20 +36,20 @@ 
 	 *
 	 * @throws ApplicationAlreadyBootedException
 	 */
-	public function __construct( $file, array $providers = array() ) {
-		if ( isset( static::$instances[ get_called_class() ] ) ) {
+	public function __construct($file, array $providers = array()) {
+		if (isset(static::$instances[get_called_class()])) {
 			throw new ApplicationAlreadyBootedException;
 		}
 
-		static::$instances[ get_called_class() ] = $this;
+		static::$instances[get_called_class()] = $this;
 
-		$this->register_constants( $file );
+		$this->register_constants($file);
 		$this->register_core_services();
 
-		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'));
 
-		parent::__construct( $providers );
+		parent::__construct($providers);
 	}
 
 	/**
@@ -58,27 +58,27 @@ 
 	 * @throws UnexpectedValueException
 	 */
 	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);
 			}
 
-			if ( $value instanceof HasShortcode ) {
-				$loader->register_shortcode( $value );
+			if ($value instanceof HasShortcode) {
+				$loader->register_shortcode($value);
 			}
 		}
 
-		add_action( 'plugins_loaded', array( $loader, 'run' ) );
+		add_action('plugins_loaded', array($loader, 'run'));
 	}
 
 	/**
@@ -106,19 +106,19 @@ 
 	 * @throws ApplicationNotBootedException
 	 */
 	public static function instance() {
-		if ( ! isset( static::$instances[ get_called_class() ] ) ) {
+		if (!isset(static::$instances[get_called_class()])) {
 			throw new ApplicationNotBootedException;
 		}
 
-		return static::$instances[ get_called_class() ];
+		return static::$instances[get_called_class()];
 	}
 
 	/**
 	 * {@inheritDoc}
 	 */
 	public static function shutdown() {
-		if ( isset( static::$instances[ get_called_class() ] ) ) {
-			unset( static::$instances[ get_called_class() ] );
+		if (isset(static::$instances[get_called_class()])) {
+			unset(static::$instances[get_called_class()]);
 		}
 	}
 
@@ -127,23 +127,23 @@ 
 	 *
 	 * @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', $basename = plugin_basename( $file ) );
-		$this->share( 'slug', dirname( $basename ) );
-		$this->share( 'version', static::VERSION );
+	private function register_constants($file) {
+		$this->share('url', plugin_dir_url($file));
+		$this->share('path', plugin_dir_path($file));
+		$this->share('basename', $basename = plugin_basename($file));
+		$this->share('slug', dirname($basename));
+		$this->share('version', static::VERSION);
 	}
 
 	/**
 	 * 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);
 		} );
-		$this->share( array( 'i18n' => 'Intaxia\Jaxion\Contract\Core\I18n' ), function ( $app ) {
-			return new I18n( $app->fetch( 'basename' ), $app->fetch( 'path' ) );
+		$this->share(array('i18n' => 'Intaxia\Jaxion\Contract\Core\I18n'), function($app) {
+			return new I18n($app->fetch('basename'), $app->fetch('path'));
 		} );
 	}
 }

src/Assets/Register.php

Indentation +204 added lines, -204 removed lines

@@ -12,208 +12,208 @@
  * @subpackage Register
  */
 class Register implements RegisterContract {
-	/**
-	 * Minification string for enqueued assets.
-	 *
-	 * @var string
-	 */
-	private $min = '';
-
-	/**
-	 * Url to the plugin directory.
-	 *
-	 * @var string
-	 */
-	protected $url;
-
-	/**
-	 * Script/plugin version.
-	 *
-	 * @var string
-	 */
-	protected $version;
-
-	/**
-	 * Array of script definition arrays.
-	 *
-	 * @var array
-	 */
-	private $scripts = array();
-
-	/**
-	 * Array of style definition arrays.
-	 *
-	 * @var array
-	 */
-	private $styles = array();
-
-	/**
-	 * Instantiates a new instance of the Register class.
-	 *
-	 * The URL param should be relative to the plugin directory. The URL
-	 * form should always end with a '/'. All asset location definitions
-	 * should not begin with a slash and should be relative to the plugin's
-	 * root directory. The URL provided by default from the Application
-	 * class is compatible.
-	 *
-	 * @param string $url
-	 * @param string $version
-	 */
-	public function __construct( $url, $version = null ) {
-		$this->url     = $url;
-		$this->version = $version ?: null; // Empty string should remain null.
-	}
-
-	/**
-	 * {@inheritdoc}
-	 *
-	 * @param bool $debug
-	 */
-	public function set_debug( $debug ) {
-		if ( $debug ) {
-			$this->min = '.min';
-		} else {
-			$this->min = '';
-		}
-	}
-
-	/**
-	 * {@inheritdoc}
-	 *
-	 * @param array $script
-	 */
-	public function register_script( $script ) {
-		$this->scripts[] = $script;
-	}
-
-	/**
-	 * {@inheritdoc}
-	 *
-	 * @param array $style
-	 */
-	public function register_style( $style ) {
-		$this->styles[] = $style;
-	}
-
-	/**
-	 * {@inheritDoc}
-	 */
-	public function enqueue_web_scripts() {
-		foreach ( $this->scripts as $script ) {
-			if ( in_array( $script['type'], array( 'web', 'shared' ) ) ) {
-				$this->enqueue_script( $script );
-			}
-		}
-	}
-
-	/**
-	 * {@inheritDoc}
-	 */
-	public function enqueue_web_styles() {
-		foreach ( $this->styles as $style ) {
-			if ( in_array( $style['type'], array( 'web', 'shared' ) ) ) {
-				$this->enqueue_style( $style );
-			}
-		}
-	}
-
-	/**
-	 * {@inheritDoc}
-	 *
-	 * @param string $hook Passes a string representing the current page.
-	 */
-	public function enqueue_admin_scripts( $hook ) {
-		foreach ( $this->scripts as $script ) {
-			if ( in_array( $script['type'], array( 'admin', 'shared' ) ) ) {
-				$this->enqueue_script( $script, $hook );
-			}
-		}
-	}
-
-	/**
-	 * {@inheritDoc}
-	 *
-	 * @param string $hook Passes a string representing the current page.
-	 */
-	public function enqueue_admin_styles( $hook ) {
-		foreach ( $this->styles as $style ) {
-			if ( in_array( $style['type'], array( 'admin', 'shared' ) ) ) {
-				$this->enqueue_style( $style, $hook );
-			}
-		}
-	}
-
-	/**
-	 * {@inheritDoc}
-	 *
-	 * @return array[]
-	 */
-	public function action_hooks() {
-		return array(
-			array(
-				'hook'   => 'wp_enqueue_scripts',
-				'method' => 'enqueue_web_scripts',
-			),
-			array(
-				'hook'   => 'wp_enqueue_scripts',
-				'method' => 'enqueue_web_styles',
-			),
-			array(
-				'hook'   => 'admin_enqueue_scripts',
-				'method' => 'enqueue_admin_scripts',
-			),
-			array(
-				'hook'   => 'admin_enqueue_scripts',
-				'method' => 'enqueue_admin_styles',
-			),
-		);
-	}
-
-	/**
-	 * Enqueues an individual script if the style's condition is met.
-	 *
-	 * @param array  $script The script attachment callback.
-	 * @param string $hook   The location hook. Only passed on admin side.
-	 */
-	protected function enqueue_script( $script, $hook = null ) {
-		if ( $script['condition']( $hook ) ) {
-			wp_enqueue_script(
-				$script['handle'],
-				$this->url . $script['src'] . '.js',
-				isset( $script['deps'] ) ? $script['deps'] : array(),
-				$this->version,
-				isset( $script['footer'] ) ? $script['footer'] : false
-			);
-
-			if ( isset( $script['localize'] ) ) {
-				if ( is_callable( $script['localize'] ) ) { // @todo make all properties callables
-					$script['localize'] = call_user_func( $script['localize'] );
-				}
-
-				wp_localize_script(
-					$script['handle'],
-					$script['localize']['name'],
-					$script['localize']['data']
-				);
-			}
-		}
-	}
-
-	/**
-	 * Enqueues an individual stylesheet if the style's condition is met.
-	 *
-	 * @param array  $style The style attachment callback.
-	 * @param string $hook  The location hook.
-	 */
-	protected function enqueue_style( $style, $hook = null ) {
-		if ( $style['condition']( $hook ) ) {
-			wp_enqueue_style(
-				$style['handle'],
-				$this->url . $style['src'] . '.css',
-				isset( $style['deps'] ) ? $style['deps'] : array(),
-				$this->version,
-				isset( $style['media'] ) ? $style['media'] : 'all'
-			);
-		}
-	}
+    /**
+     * Minification string for enqueued assets.
+     *
+     * @var string
+     */
+    private $min = '';
+
+    /**
+     * Url to the plugin directory.
+     *
+     * @var string
+     */
+    protected $url;
+
+    /**
+     * Script/plugin version.
+     *
+     * @var string
+     */
+    protected $version;
+
+    /**
+     * Array of script definition arrays.
+     *
+     * @var array
+     */
+    private $scripts = array();
+
+    /**
+     * Array of style definition arrays.
+     *
+     * @var array
+     */
+    private $styles = array();
+
+    /**
+     * Instantiates a new instance of the Register class.
+     *
+     * The URL param should be relative to the plugin directory. The URL
+     * form should always end with a '/'. All asset location definitions
+     * should not begin with a slash and should be relative to the plugin's
+     * root directory. The URL provided by default from the Application
+     * class is compatible.
+     *
+     * @param string $url
+     * @param string $version
+     */
+    public function __construct( $url, $version = null ) {
+        $this->url     = $url;
+        $this->version = $version ?: null; // Empty string should remain null.
+    }
+
+    /**
+     * {@inheritdoc}
+     *
+     * @param bool $debug
+     */
+    public function set_debug( $debug ) {
+        if ( $debug ) {
+            $this->min = '.min';
+        } else {
+            $this->min = '';
+        }
+    }
+
+    /**
+     * {@inheritdoc}
+     *
+     * @param array $script
+     */
+    public function register_script( $script ) {
+        $this->scripts[] = $script;
+    }
+
+    /**
+     * {@inheritdoc}
+     *
+     * @param array $style
+     */
+    public function register_style( $style ) {
+        $this->styles[] = $style;
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function enqueue_web_scripts() {
+        foreach ( $this->scripts as $script ) {
+            if ( in_array( $script['type'], array( 'web', 'shared' ) ) ) {
+                $this->enqueue_script( $script );
+            }
+        }
+    }
+
+    /**
+     * {@inheritDoc}
+     */
+    public function enqueue_web_styles() {
+        foreach ( $this->styles as $style ) {
+            if ( in_array( $style['type'], array( 'web', 'shared' ) ) ) {
+                $this->enqueue_style( $style );
+            }
+        }
+    }
+
+    /**
+     * {@inheritDoc}
+     *
+     * @param string $hook Passes a string representing the current page.
+     */
+    public function enqueue_admin_scripts( $hook ) {
+        foreach ( $this->scripts as $script ) {
+            if ( in_array( $script['type'], array( 'admin', 'shared' ) ) ) {
+                $this->enqueue_script( $script, $hook );
+            }
+        }
+    }
+
+    /**
+     * {@inheritDoc}
+     *
+     * @param string $hook Passes a string representing the current page.
+     */
+    public function enqueue_admin_styles( $hook ) {
+        foreach ( $this->styles as $style ) {
+            if ( in_array( $style['type'], array( 'admin', 'shared' ) ) ) {
+                $this->enqueue_style( $style, $hook );
+            }
+        }
+    }
+
+    /**
+     * {@inheritDoc}
+     *
+     * @return array[]
+     */
+    public function action_hooks() {
+        return array(
+            array(
+                'hook'   => 'wp_enqueue_scripts',
+                'method' => 'enqueue_web_scripts',
+            ),
+            array(
+                'hook'   => 'wp_enqueue_scripts',
+                'method' => 'enqueue_web_styles',
+            ),
+            array(
+                'hook'   => 'admin_enqueue_scripts',
+                'method' => 'enqueue_admin_scripts',
+            ),
+            array(
+                'hook'   => 'admin_enqueue_scripts',
+                'method' => 'enqueue_admin_styles',
+            ),
+        );
+    }
+
+    /**
+     * Enqueues an individual script if the style's condition is met.
+     *
+     * @param array  $script The script attachment callback.
+     * @param string $hook   The location hook. Only passed on admin side.
+     */
+    protected function enqueue_script( $script, $hook = null ) {
+        if ( $script['condition']( $hook ) ) {
+            wp_enqueue_script(
+                $script['handle'],
+                $this->url . $script['src'] . '.js',
+                isset( $script['deps'] ) ? $script['deps'] : array(),
+                $this->version,
+                isset( $script['footer'] ) ? $script['footer'] : false
+            );
+
+            if ( isset( $script['localize'] ) ) {
+                if ( is_callable( $script['localize'] ) ) { // @todo make all properties callables
+                    $script['localize'] = call_user_func( $script['localize'] );
+                }
+
+                wp_localize_script(
+                    $script['handle'],
+                    $script['localize']['name'],
+                    $script['localize']['data']
+                );
+            }
+        }
+    }
+
+    /**
+     * Enqueues an individual stylesheet if the style's condition is met.
+     *
+     * @param array  $style The style attachment callback.
+     * @param string $hook  The location hook.
+     */
+    protected function enqueue_style( $style, $hook = null ) {
+        if ( $style['condition']( $hook ) ) {
+            wp_enqueue_style(
+                $style['handle'],
+                $this->url . $style['src'] . '.css',
+                isset( $style['deps'] ) ? $style['deps'] : array(),
+                $this->version,
+                isset( $style['media'] ) ? $style['media'] : 'all'
+            );
+        }
+    }
 }

Spacing +32 added lines, -32 removed lines

@@ -59,7 +59,7 @@ 
 	 * @param string $url
 	 * @param string $version
 	 */
-	public function __construct( $url, $version = null ) {
+	public function __construct($url, $version = null) {
 		$this->url     = $url;
 		$this->version = $version ?: null; // Empty string should remain null.
 	}
@@ -69,8 +69,8 @@ 
 	 *
 	 * @param bool $debug
 	 */
-	public function set_debug( $debug ) {
-		if ( $debug ) {
+	public function set_debug($debug) {
+		if ($debug) {
 			$this->min = '.min';
 		} else {
 			$this->min = '';
@@ -82,7 +82,7 @@ 
 	 *
 	 * @param array $script
 	 */
-	public function register_script( $script ) {
+	public function register_script($script) {
 		$this->scripts[] = $script;
 	}
 
@@ -91,7 +91,7 @@ 
 	 *
 	 * @param array $style
 	 */
-	public function register_style( $style ) {
+	public function register_style($style) {
 		$this->styles[] = $style;
 	}
 
@@ -99,9 +99,9 @@ 
 	 * {@inheritDoc}
 	 */
 	public function enqueue_web_scripts() {
-		foreach ( $this->scripts as $script ) {
-			if ( in_array( $script['type'], array( 'web', 'shared' ) ) ) {
-				$this->enqueue_script( $script );
+		foreach ($this->scripts as $script) {
+			if (in_array($script['type'], array('web', 'shared'))) {
+				$this->enqueue_script($script);
 			}
 		}
 	}
@@ -110,9 +110,9 @@ 
 	 * {@inheritDoc}
 	 */
 	public function enqueue_web_styles() {
-		foreach ( $this->styles as $style ) {
-			if ( in_array( $style['type'], array( 'web', 'shared' ) ) ) {
-				$this->enqueue_style( $style );
+		foreach ($this->styles as $style) {
+			if (in_array($style['type'], array('web', 'shared'))) {
+				$this->enqueue_style($style);
 			}
 		}
 	}
@@ -122,10 +122,10 @@ 
 	 *
 	 * @param string $hook Passes a string representing the current page.
 	 */
-	public function enqueue_admin_scripts( $hook ) {
-		foreach ( $this->scripts as $script ) {
-			if ( in_array( $script['type'], array( 'admin', 'shared' ) ) ) {
-				$this->enqueue_script( $script, $hook );
+	public function enqueue_admin_scripts($hook) {
+		foreach ($this->scripts as $script) {
+			if (in_array($script['type'], array('admin', 'shared'))) {
+				$this->enqueue_script($script, $hook);
 			}
 		}
 	}
@@ -135,10 +135,10 @@ 
 	 *
 	 * @param string $hook Passes a string representing the current page.
 	 */
-	public function enqueue_admin_styles( $hook ) {
-		foreach ( $this->styles as $style ) {
-			if ( in_array( $style['type'], array( 'admin', 'shared' ) ) ) {
-				$this->enqueue_style( $style, $hook );
+	public function enqueue_admin_styles($hook) {
+		foreach ($this->styles as $style) {
+			if (in_array($style['type'], array('admin', 'shared'))) {
+				$this->enqueue_style($style, $hook);
 			}
 		}
 	}
@@ -175,19 +175,19 @@ 
 	 * @param array  $script The script attachment callback.
 	 * @param string $hook   The location hook. Only passed on admin side.
 	 */
-	protected function enqueue_script( $script, $hook = null ) {
-		if ( $script['condition']( $hook ) ) {
+	protected function enqueue_script($script, $hook = null) {
+		if ($script['condition']($hook)) {
 			wp_enqueue_script(
 				$script['handle'],
-				$this->url . $script['src'] . '.js',
-				isset( $script['deps'] ) ? $script['deps'] : array(),
+				$this->url.$script['src'].'.js',
+				isset($script['deps']) ? $script['deps'] : array(),
 				$this->version,
-				isset( $script['footer'] ) ? $script['footer'] : false
+				isset($script['footer']) ? $script['footer'] : false
 			);
 
-			if ( isset( $script['localize'] ) ) {
-				if ( is_callable( $script['localize'] ) ) { // @todo make all properties callables
-					$script['localize'] = call_user_func( $script['localize'] );
+			if (isset($script['localize'])) {
+				if (is_callable($script['localize'])) { // @todo make all properties callables
+					$script['localize'] = call_user_func($script['localize']);
 				}
 
 				wp_localize_script(
@@ -205,14 +205,14 @@ 
 	 * @param array  $style The style attachment callback.
 	 * @param string $hook  The location hook.
 	 */
-	protected function enqueue_style( $style, $hook = null ) {
-		if ( $style['condition']( $hook ) ) {
+	protected function enqueue_style($style, $hook = null) {
+		if ($style['condition']($hook)) {
 			wp_enqueue_style(
 				$style['handle'],
-				$this->url . $style['src'] . '.css',
-				isset( $style['deps'] ) ? $style['deps'] : array(),
+				$this->url.$style['src'].'.css',
+				isset($style['deps']) ? $style['deps'] : array(),
 				$this->version,
-				isset( $style['media'] ) ? $style['media'] : 'all'
+				isset($style['media']) ? $style['media'] : 'all'
 			);
 		}
 	}

src/Contract/Assets/Register.php

Indentation +48 added lines, -48 removed lines

@@ -4,52 +4,52 @@
 use Intraxia\Jaxion\Contract\Core\HasActions;
 
 interface Register extends HasActions {
-	/**
-	 * Enable debug mode for the enqueued assets.
-	 *
-	 * Debug mode will enqueue unminified versions of the registered assets.
-	 * Primarily, this is intended to be used along with WordPress's `SCRIPT_DEBUG`
-	 * constant, which enables unminified core assets to be enqueued.
-	 *
-	 * @param bool $debug
-	 */
-	public function set_debug( $debug );
-
-	/**
-	 * Provides a method to register new scripts outside of the constructor.
-	 *
-	 * @param array $script
-	 */
-	public function register_script( $script );
-
-	/**
-	 * Provides a method to register new styles outside of the constructor.
-	 *
-	 * @param array $style
-	 */
-	public function register_style( $style );
-
-	/**
-	 * Enqueues the web & shared scripts on the Register.
-	 */
-	public function enqueue_web_scripts();
-
-	/**
-	 * Enqueues the web & shared styles on the Register.
-	 */
-	public function enqueue_web_styles();
-
-	/**
-	 * Enqueues the admin & shared scripts on the Register.
-	 *
-	 * @param string $hook Passes a string representing the current page.
-	 */
-	public function enqueue_admin_scripts( $hook );
-
-	/**
-	 * Enqueues the admin & shared styles on the Register.
-	 *
-	 * @param string $hook Passes a string representing the current page.
-	 */
-	public function enqueue_admin_styles( $hook );
+    /**
+     * Enable debug mode for the enqueued assets.
+     *
+     * Debug mode will enqueue unminified versions of the registered assets.
+     * Primarily, this is intended to be used along with WordPress's `SCRIPT_DEBUG`
+     * constant, which enables unminified core assets to be enqueued.
+     *
+     * @param bool $debug
+     */
+    public function set_debug( $debug );
+
+    /**
+     * Provides a method to register new scripts outside of the constructor.
+     *
+     * @param array $script
+     */
+    public function register_script( $script );
+
+    /**
+     * Provides a method to register new styles outside of the constructor.
+     *
+     * @param array $style
+     */
+    public function register_style( $style );
+
+    /**
+     * Enqueues the web & shared scripts on the Register.
+     */
+    public function enqueue_web_scripts();
+
+    /**
+     * Enqueues the web & shared styles on the Register.
+     */
+    public function enqueue_web_styles();
+
+    /**
+     * Enqueues the admin & shared scripts on the Register.
+     *
+     * @param string $hook Passes a string representing the current page.
+     */
+    public function enqueue_admin_scripts( $hook );
+
+    /**
+     * Enqueues the admin & shared styles on the Register.
+     *
+     * @param string $hook Passes a string representing the current page.
+     */
+    public function enqueue_admin_styles( $hook );
 }

Spacing +5 added lines, -5 removed lines

@@ -13,21 +13,21 @@ 
 	 *
 	 * @param bool $debug
 	 */
-	public function set_debug( $debug );
+	public function set_debug($debug);
 
 	/**
 	 * Provides a method to register new scripts outside of the constructor.
 	 *
 	 * @param array $script
 	 */
-	public function register_script( $script );
+	public function register_script($script);
 
 	/**
 	 * Provides a method to register new styles outside of the constructor.
 	 *
 	 * @param array $style
 	 */
-	public function register_style( $style );
+	public function register_style($style);
 
 	/**
 	 * Enqueues the web & shared scripts on the Register.
@@ -44,12 +44,12 @@ 
 	 *
 	 * @param string $hook Passes a string representing the current page.
 	 */
-	public function enqueue_admin_scripts( $hook );
+	public function enqueue_admin_scripts($hook);
 
 	/**
 	 * Enqueues the admin & shared styles on the Register.
 	 *
 	 * @param string $hook Passes a string representing the current page.
 	 */
-	public function enqueue_admin_styles( $hook );
+	public function enqueue_admin_styles($hook);
 }

src/Axolotl/Collection.php

Indentation +223 added lines, -223 removed lines

@@ -14,227 +14,227 @@
  * @subpackage Axolotl
  */
 class Collection implements Countable, Iterator, Serializes {
-	/**
-	 * Collection elements.
-	 *
-	 * @var array
-	 */
-	protected $elements = array();
-
-	/**
-	 * Collection configuration.
-	 *
-	 * @var array
-	 */
-	protected $config = array();
-
-	/**
-	 * Models registered to the collection.
-	 *
-	 * @var string
-	 */
-	protected $model;
-
-	/**
-	 * Where Collection is in loop.
-	 *
-	 * @var int
-	 */
-	protected $position = 0;
-
-	/**
-	 * Collection constructor.
-	 *
-	 * @param array $elements
-	 * @param array $config
-	 */
-	public function __construct( array $elements = array(), array $config = array() ) {
-		$this->parse_config( $this->config = $config );
-
-		foreach ( $elements as $element ) {
-			$this->add( $element );
-		}
-	}
-
-	/**
-	 * Adds a new element to the Collection.
-	 *
-	 * @param mixed $element
-	 *
-	 * @throws RuntimeException
-	 *
-	 * @return $this
-	 */
-	public function add( $element ) {
-		if ( $this->model && is_array( $element ) ) {
-			$element = new $this->model( $element );
-		}
-
-		if ( $this->model && ! ( $element instanceof $this->model ) ) {
-			throw new RuntimeException;
-		}
-
-		$this->elements[] = $element;
-
-		return $this;
-	}
-
-	/**
-	 * Removes an element by index or value.
-	 *
-	 * @param mixed $index
-	 *
-	 * @return $this
-	 */
-	public function remove( $index ) {
-		if ( ! is_string( $index ) || ! is_numeric( $index ) || ! isset( $this->elements[ $index ] ) ) {
-			foreach ( $this->elements as $key => $element ) {
-				if ( $element === $index ) {
-					$index = $key;
-					break;
-				}
-			}
-		}
-
-		if ( isset( $this->elements[ $index ] ) ) {
-			unset( $this->elements[ $index ] );
-			$this->elements = array_values( $this->elements );
-		}
-
-		return $this;
-	}
-
-	/**
-	 * Fetches the element at the provided index.
-	 *
-	 * @param int $index
-	 *
-	 * @return mixed|null
-	 */
-	public function at( $index ) {
-		return isset( $this->elements[ $index ] ) ? $this->elements[ $index ] : null;
-	}
-
-	/**
-	 * Maps over the Collection's elements,
-	 *
-	 * @param callable $callback
-	 *
-	 * @return Collection
-	 */
-	public function map( callable $callback ) {
-		return new Collection( array_map( $callback, $this->elements ) );
-	}
-
-	/**
-	 * Filters the Collection's elements.
-	 *
-	 * @param callable $callback
-	 *
-	 * @return Collection
-	 */
-	public function filter( callable $callback ) {
-		return new Collection( array_filter( $this->elements, $callback ) );
-	}
-
-	/**
-	 * Fetches a single element in the Collection by callback.
-	 *
-	 * @param callable $callback
-	 *
-	 * @return mixed|null
-	 */
-	public function find( callable $callback ) {
-		foreach ( $this->elements as $element ) {
-			if ( call_user_func( $callback, $element ) ) {
-				return $element;
-			}
-		}
-
-		return null;
-	}
-
-	/**
-	 * {@inheritDoc}
-	 *
-	 * @return array
-	 */
-	public function serialize() {
-		return array_map(function( $element ) {
-			if ( $element instanceof Serializes ) {
-				return $element->serialize();
-			}
-
-			return $element;
-		}, $this->elements);
-	}
-
-	/**
-	 * Return the current element.
-	 *
-	 * @return mixed
-	 */
-	public function current() {
-		return $this->at( $this->position );
-	}
-
-	/**
-	 * Move forward to next element.
-	 */
-	public function next() {
-		$this->position ++;
-	}
-
-	/**
-	 * Return the key of the current element.
-	 *
-	 * @return mixed
-	 */
-	public function key() {
-		return $this->position;
-	}
-
-	/**
-	 * Checks if current position is valid.
-	 *
-	 * @return bool
-	 */
-	public function valid() {
-		return isset( $this->elements[ $this->position ] );
-	}
-
-	/**
-	 * Rewind the Iterator to the first element.
-	 */
-	public function rewind() {
-		$this->position = 0;
-	}
-
-	/**
-	 * Count elements of an object.
-	 *
-	 * @return int
-	 */
-	public function count() {
-		return count( $this->elements );
-	}
-
-	/**
-	 * Parses the Collection config to set its properties.
-	 *
-	 * @param array $config
-	 *
-	 * @throws LogicException
-	 */
-	protected function parse_config( array $config ) {
-		if ( isset( $config['model'] ) ) {
-			$model = $config['model'];
-
-			if ( ! is_subclass_of( $model, 'Intraxia\Jaxion\Axolotl\Model' ) ) {
-				throw new LogicException;
-			}
-
-			$this->model = $model;
-		}
-	}
+    /**
+     * Collection elements.
+     *
+     * @var array
+     */
+    protected $elements = array();
+
+    /**
+     * Collection configuration.
+     *
+     * @var array
+     */
+    protected $config = array();
+
+    /**
+     * Models registered to the collection.
+     *
+     * @var string
+     */
+    protected $model;
+
+    /**
+     * Where Collection is in loop.
+     *
+     * @var int
+     */
+    protected $position = 0;
+
+    /**
+     * Collection constructor.
+     *
+     * @param array $elements
+     * @param array $config
+     */
+    public function __construct( array $elements = array(), array $config = array() ) {
+        $this->parse_config( $this->config = $config );
+
+        foreach ( $elements as $element ) {
+            $this->add( $element );
+        }
+    }
+
+    /**
+     * Adds a new element to the Collection.
+     *
+     * @param mixed $element
+     *
+     * @throws RuntimeException
+     *
+     * @return $this
+     */
+    public function add( $element ) {
+        if ( $this->model && is_array( $element ) ) {
+            $element = new $this->model( $element );
+        }
+
+        if ( $this->model && ! ( $element instanceof $this->model ) ) {
+            throw new RuntimeException;
+        }
+
+        $this->elements[] = $element;
+
+        return $this;
+    }
+
+    /**
+     * Removes an element by index or value.
+     *
+     * @param mixed $index
+     *
+     * @return $this
+     */
+    public function remove( $index ) {
+        if ( ! is_string( $index ) || ! is_numeric( $index ) || ! isset( $this->elements[ $index ] ) ) {
+            foreach ( $this->elements as $key => $element ) {
+                if ( $element === $index ) {
+                    $index = $key;
+                    break;
+                }
+            }
+        }
+
+        if ( isset( $this->elements[ $index ] ) ) {
+            unset( $this->elements[ $index ] );
+            $this->elements = array_values( $this->elements );
+        }
+
+        return $this;
+    }
+
+    /**
+     * Fetches the element at the provided index.
+     *
+     * @param int $index
+     *
+     * @return mixed|null
+     */
+    public function at( $index ) {
+        return isset( $this->elements[ $index ] ) ? $this->elements[ $index ] : null;
+    }
+
+    /**
+     * Maps over the Collection's elements,
+     *
+     * @param callable $callback
+     *
+     * @return Collection
+     */
+    public function map( callable $callback ) {
+        return new Collection( array_map( $callback, $this->elements ) );
+    }
+
+    /**
+     * Filters the Collection's elements.
+     *
+     * @param callable $callback
+     *
+     * @return Collection
+     */
+    public function filter( callable $callback ) {
+        return new Collection( array_filter( $this->elements, $callback ) );
+    }
+
+    /**
+     * Fetches a single element in the Collection by callback.
+     *
+     * @param callable $callback
+     *
+     * @return mixed|null
+     */
+    public function find( callable $callback ) {
+        foreach ( $this->elements as $element ) {
+            if ( call_user_func( $callback, $element ) ) {
+                return $element;
+            }
+        }
+
+        return null;
+    }
+
+    /**
+     * {@inheritDoc}
+     *
+     * @return array
+     */
+    public function serialize() {
+        return array_map(function( $element ) {
+            if ( $element instanceof Serializes ) {
+                return $element->serialize();
+            }
+
+            return $element;
+        }, $this->elements);
+    }
+
+    /**
+     * Return the current element.
+     *
+     * @return mixed
+     */
+    public function current() {
+        return $this->at( $this->position );
+    }
+
+    /**
+     * Move forward to next element.
+     */
+    public function next() {
+        $this->position ++;
+    }
+
+    /**
+     * Return the key of the current element.
+     *
+     * @return mixed
+     */
+    public function key() {
+        return $this->position;
+    }
+
+    /**
+     * Checks if current position is valid.
+     *
+     * @return bool
+     */
+    public function valid() {
+        return isset( $this->elements[ $this->position ] );
+    }
+
+    /**
+     * Rewind the Iterator to the first element.
+     */
+    public function rewind() {
+        $this->position = 0;
+    }
+
+    /**
+     * Count elements of an object.
+     *
+     * @return int
+     */
+    public function count() {
+        return count( $this->elements );
+    }
+
+    /**
+     * Parses the Collection config to set its properties.
+     *
+     * @param array $config
+     *
+     * @throws LogicException
+     */
+    protected function parse_config( array $config ) {
+        if ( isset( $config['model'] ) ) {
+            $model = $config['model'];
+
+            if ( ! is_subclass_of( $model, 'Intraxia\Jaxion\Axolotl\Model' ) ) {
+                throw new LogicException;
+            }
+
+            $this->model = $model;
+        }
+    }
 }

Spacing +33 added lines, -33 removed lines

@@ -48,11 +48,11 @@ 
 	 * @param array $elements
 	 * @param array $config
 	 */
-	public function __construct( array $elements = array(), array $config = array() ) {
-		$this->parse_config( $this->config = $config );
+	public function __construct(array $elements = array(), array $config = array()) {
+		$this->parse_config($this->config = $config);
 
-		foreach ( $elements as $element ) {
-			$this->add( $element );
+		foreach ($elements as $element) {
+			$this->add($element);
 		}
 	}
 
@@ -65,12 +65,12 @@ 
 	 *
 	 * @return $this
 	 */
-	public function add( $element ) {
-		if ( $this->model && is_array( $element ) ) {
-			$element = new $this->model( $element );
+	public function add($element) {
+		if ($this->model && is_array($element)) {
+			$element = new $this->model($element);
 		}
 
-		if ( $this->model && ! ( $element instanceof $this->model ) ) {
+		if ($this->model && !($element instanceof $this->model)) {
 			throw new RuntimeException;
 		}
 
@@ -86,19 +86,19 @@ 
 	 *
 	 * @return $this
 	 */
-	public function remove( $index ) {
-		if ( ! is_string( $index ) || ! is_numeric( $index ) || ! isset( $this->elements[ $index ] ) ) {
-			foreach ( $this->elements as $key => $element ) {
-				if ( $element === $index ) {
+	public function remove($index) {
+		if (!is_string($index) || !is_numeric($index) || !isset($this->elements[$index])) {
+			foreach ($this->elements as $key => $element) {
+				if ($element === $index) {
 					$index = $key;
 					break;
 				}
 			}
 		}
 
-		if ( isset( $this->elements[ $index ] ) ) {
-			unset( $this->elements[ $index ] );
-			$this->elements = array_values( $this->elements );
+		if (isset($this->elements[$index])) {
+			unset($this->elements[$index]);
+			$this->elements = array_values($this->elements);
 		}
 
 		return $this;
@@ -111,8 +111,8 @@ 
 	 *
 	 * @return mixed|null
 	 */
-	public function at( $index ) {
-		return isset( $this->elements[ $index ] ) ? $this->elements[ $index ] : null;
+	public function at($index) {
+		return isset($this->elements[$index]) ? $this->elements[$index] : null;
 	}
 
 	/**
@@ -122,8 +122,8 @@ 
 	 *
 	 * @return Collection
 	 */
-	public function map( callable $callback ) {
-		return new Collection( array_map( $callback, $this->elements ) );
+	public function map(callable $callback) {
+		return new Collection(array_map($callback, $this->elements));
 	}
 
 	/**
@@ -133,8 +133,8 @@ 
 	 *
 	 * @return Collection
 	 */
-	public function filter( callable $callback ) {
-		return new Collection( array_filter( $this->elements, $callback ) );
+	public function filter(callable $callback) {
+		return new Collection(array_filter($this->elements, $callback));
 	}
 
 	/**
@@ -144,9 +144,9 @@ 
 	 *
 	 * @return mixed|null
 	 */
-	public function find( callable $callback ) {
-		foreach ( $this->elements as $element ) {
-			if ( call_user_func( $callback, $element ) ) {
+	public function find(callable $callback) {
+		foreach ($this->elements as $element) {
+			if (call_user_func($callback, $element)) {
 				return $element;
 			}
 		}
@@ -160,8 +160,8 @@ 
 	 * @return array
 	 */
 	public function serialize() {
-		return array_map(function( $element ) {
-			if ( $element instanceof Serializes ) {
+		return array_map(function($element) {
+			if ($element instanceof Serializes) {
 				return $element->serialize();
 			}
 
@@ -175,14 +175,14 @@ 
 	 * @return mixed
 	 */
 	public function current() {
-		return $this->at( $this->position );
+		return $this->at($this->position);
 	}
 
 	/**
 	 * Move forward to next element.
 	 */
 	public function next() {
-		$this->position ++;
+		$this->position++;
 	}
 
 	/**
@@ -200,7 +200,7 @@ 
 	 * @return bool
 	 */
 	public function valid() {
-		return isset( $this->elements[ $this->position ] );
+		return isset($this->elements[$this->position]);
 	}
 
 	/**
@@ -216,7 +216,7 @@ 
 	 * @return int
 	 */
 	public function count() {
-		return count( $this->elements );
+		return count($this->elements);
 	}
 
 	/**
@@ -226,11 +226,11 @@ 
 	 *
 	 * @throws LogicException
 	 */
-	protected function parse_config( array $config ) {
-		if ( isset( $config['model'] ) ) {
+	protected function parse_config(array $config) {
+		if (isset($config['model'])) {
 			$model = $config['model'];
 
-			if ( ! is_subclass_of( $model, 'Intraxia\Jaxion\Axolotl\Model' ) ) {
+			if (!is_subclass_of($model, 'Intraxia\Jaxion\Axolotl\Model')) {
 				throw new LogicException;
 			}