brightnucleus / dependencies

src/DependencyHandlerInterface.php

Doc Comments +1 added lines

@@ -30,6 +30,7 @@
 	 * @since 0.2.3
 	 *
 	 * @param string $handle Handle of the dependency to enqueue.
+	 * @return void
 	 */
 	public function maybe_enqueue( $handle );
 }

Spacing +1 added lines, -1 removed lines

@@ -31,5 +31,5 @@
 	 *
 	 * @param string $handle Handle of the dependency to enqueue.
 	 */
-	public function maybe_enqueue( $handle );
+	public function maybe_enqueue($handle);
 }

src/AbstractDependencyHandler.php

Spacing +8 added lines, -8 removed lines

@@ -35,8 +35,8 @@ 
 	 * @throws InvalidArgumentException If the register function could not be
 	 *                                  called.
 	 */
-	public function register( $args = null ) {
-		$this->invokeFunction( $this->get_register_function(), $args );
+	public function register($args = null) {
+		$this->invokeFunction($this->get_register_function(), $args);
 	}
 
 	/**
@@ -58,8 +58,8 @@ 
 	 * @throws InvalidArgumentException If the register function could not be
 	 *                                  called.
 	 */
-	public function enqueue( $args = null ) {
-		$this->invokeFunction( $this->get_enqueue_function(), $args );
+	public function enqueue($args = null) {
+		$this->invokeFunction($this->get_enqueue_function(), $args);
 	}
 
 	/**
@@ -79,10 +79,10 @@ 
 	 *
 	 * @param string $handle Handle of the dependency to enqueue.
 	 */
-	public function maybe_enqueue( $handle ) {
-		if ( $this->is_registered( $handle ) ) {
+	public function maybe_enqueue($handle) {
+		if ($this->is_registered($handle)) {
 			$enqueue = $this->get_enqueue_function();
-			$enqueue( $handle );
+			$enqueue($handle);
 		}
 	}
 
@@ -94,5 +94,5 @@ 
 	 * @param string $handle The handle to check
 	 * @return bool Whether it is registered or not.
 	 */
-	abstract protected function is_registered( $handle );
+	abstract protected function is_registered($handle);
 }

src/DependencyManager.php

Indentation +8 added lines, -8 removed lines

@@ -1,13 +1,13 @@
 <?php
 /**
- * DependencyManager Class.
- *
- * @package   BrightNucleus\Dependency
- * @author    Alain Schlesser <alain.schlesser@gmail.com>
- * @license   GPL-2.0+
- * @link      http://www.brightnucleus.com/
- * @copyright 2015 Alain Schlesser, Bright NucleusInterface
- */
+	 * DependencyManager Class.
+	 *
+	 * @package   BrightNucleus\Dependency
+	 * @author    Alain Schlesser <alain.schlesser@gmail.com>
+	 * @license   GPL-2.0+
+	 * @link      http://www.brightnucleus.com/
+	 * @copyright 2015 Alain Schlesser, Bright NucleusInterface
+	 */
 
 namespace BrightNucleus\Dependency;
 

Spacing +75 added lines, -75 removed lines

@@ -48,7 +48,7 @@ 
 	 *
 	 * @var array;
 	 */
-	protected $dependencies = [ ];
+	protected $dependencies = [];
 
 	/**
 	 * Hold the handlers.
@@ -57,7 +57,7 @@ 
 	 *
 	 * @var DependencyHandlerInterface[]
 	 */
-	protected $handlers = [ ];
+	protected $handlers = [];
 
 	/**
 	 * Whether to enqueue immediately upon registration.
@@ -81,8 +81,8 @@ 
 	 * @throws InvalidArgumentException If no dependency handlers were
 	 *                                  specified.
 	 */
-	public function __construct( ConfigInterface $config, $enqueue = true ) {
-		$this->processConfig( $config );
+	public function __construct(ConfigInterface $config, $enqueue = true) {
+		$this->processConfig($config);
 		$this->enqueue_immediately = $enqueue;
 		$this->init_handlers();
 		$this->init_dependencies();
@@ -94,10 +94,10 @@ 
 	 * @since 0.1.0
 	 */
 	protected function init_handlers() {
-		$keys = [ self::KEY_SCRIPTS, self::KEY_STYLES ];
-		foreach ( $keys as $key ) {
-			if ( $this->hasConfigKey( $key ) ) {
-				$this->add_handler( $key );
+		$keys = [self::KEY_SCRIPTS, self::KEY_STYLES];
+		foreach ($keys as $key) {
+			if ($this->hasConfigKey($key)) {
+				$this->add_handler($key);
 			}
 		}
 	}
@@ -109,13 +109,13 @@ 
 	 *
 	 * @param string $dependency The dependency type for which to add a handler.
 	 */
-	protected function add_handler( $dependency ) {
-		if ( $this->hasConfigKey( $dependency ) ) {
-			$handler = $this->hasConfigKey( self::KEY_HANDLERS, $dependency )
-				? $this->getConfigKey( self::KEY_HANDLERS, $dependency )
-				: $this->get_default_handler( $dependency );
-			if ( $handler ) {
-				$this->handlers[ $dependency ] = new $handler;
+	protected function add_handler($dependency) {
+		if ($this->hasConfigKey($dependency)) {
+			$handler = $this->hasConfigKey(self::KEY_HANDLERS, $dependency)
+				? $this->getConfigKey(self::KEY_HANDLERS, $dependency)
+				: $this->get_default_handler($dependency);
+			if ($handler) {
+				$this->handlers[$dependency] = new $handler;
 			}
 		}
 	}
@@ -128,8 +128,8 @@ 
 	 * @param string $dependency The dependency that needs a handler.
 	 * @return string|null Class name of the handler. Null if none.
 	 */
-	protected function get_default_handler( $dependency ) {
-		switch ( $dependency ) {
+	protected function get_default_handler($dependency) {
+		switch ($dependency) {
 			case self::KEY_STYLES:
 				return self::DEFAULT_STYLE_HANDLER;
 			case self::KEY_SCRIPTS:
@@ -145,10 +145,10 @@ 
 	 * @since 0.1.0
 	 */
 	protected function init_dependencies() {
-		array_walk( $this->handlers,
-			function ( $handler, $dependency_type ) {
-				if ( $this->hasConfigKey( $dependency_type ) ) {
-					$this->dependencies[ $dependency_type ] = $this->getConfigKey( $dependency_type );
+		array_walk($this->handlers,
+			function($handler, $dependency_type) {
+				if ($this->hasConfigKey($dependency_type)) {
+					$this->dependencies[$dependency_type] = $this->getConfigKey($dependency_type);
 				}
 			} );
 	}
@@ -160,10 +160,10 @@ 
 	 *
 	 * @param mixed $context Optional. The context to pass to the dependencies.
 	 */
-	public function register( $context = null ) {
-		$context = $this->validate_context( $context );
-		array_walk( $this->dependencies,
-			[ $this, 'register_dependency_type' ], $context );
+	public function register($context = null) {
+		$context = $this->validate_context($context);
+		array_walk($this->dependencies,
+			[$this, 'register_dependency_type'], $context);
 	}
 
 	/**
@@ -174,9 +174,9 @@ 
 	 * @param mixed $context The context as passed in by WordPress.
 	 * @return array Validated context.
 	 */
-	protected function validate_context( $context ) {
-		if ( is_string( $context ) ) {
-			return [ 'wp_context' => $context ];
+	protected function validate_context($context) {
+		if (is_string($context)) {
+			return ['wp_context' => $context];
 		}
 		return (array) $context;
 	}
@@ -189,11 +189,11 @@ 
 	 * @param mixed $context  Optional. The context to pass to the
 	 *                        dependencies.
 	 */
-	public function enqueue( $context = null ) {
-		$context = $this->validate_context( $context );
+	public function enqueue($context = null) {
+		$context = $this->validate_context($context);
 
-		array_walk( $this->dependencies,
-			[ $this, 'enqueue_dependency_type' ], $context );
+		array_walk($this->dependencies,
+			[$this, 'enqueue_dependency_type'], $context);
 	}
 
 	/**
@@ -208,9 +208,9 @@ 
 	 *                         outside of DependencyManager. Defaults to false.
 	 * @return bool Returns whether the handle was found or not.
 	 */
-	public function enqueue_handle( $handle, $context = null, $fallback = false ) {
-		list( $dependency_key, $dependency ) = $this->get_dependency_array( $handle );
-		if ( $dependency ) {
+	public function enqueue_handle($handle, $context = null, $fallback = false) {
+		list($dependency_key, $dependency) = $this->get_dependency_array($handle);
+		if ($dependency) {
 
 			$this->enqueue_dependency(
 				$dependency,
@@ -218,14 +218,14 @@ 
 				$context
 			);
 
-			$this->maybe_localize( $dependency, $context );
+			$this->maybe_localize($dependency, $context);
 
 			return true;
 		}
 
-		if ( $fallback ) {
-			foreach ( $this->handlers as $handler ) {
-				$handler->maybe_enqueue( $handle );
+		if ($fallback) {
+			foreach ($this->handlers as $handler) {
+				$handler->maybe_enqueue($handle);
 			}
 		}
 		return false;
@@ -240,15 +240,15 @@ 
 	 * @return array Array containing the dependency key as well as the
 	 *                       dependency array itself.
 	 */
-	protected function get_dependency_array( $handle ) {
-		foreach ( $this->dependencies as $dependency_type ) {
-			$dependency_key = array_search( $handle, $dependency_type, true );
-			if ( $dependency_key ) {
-				return [ $dependency_key, $dependency_type[ $dependency_key ] ];
+	protected function get_dependency_array($handle) {
+		foreach ($this->dependencies as $dependency_type) {
+			$dependency_key = array_search($handle, $dependency_type, true);
+			if ($dependency_key) {
+				return [$dependency_key, $dependency_type[$dependency_key]];
 			}
 		}
 		// Handle not found, return an empty array.
-		return [ '', null ];
+		return ['', null];
 	}
 
 	/**
@@ -263,12 +263,12 @@ 
 	 *                               dependency at key
 	 *                               'dependency_type'.
 	 */
-	protected function enqueue_dependency( $dependency, $dependency_key, $context = null ) {
-		if ( ! $this->is_needed( $dependency, $context ) ) {
+	protected function enqueue_dependency($dependency, $dependency_key, $context = null) {
+		if ( ! $this->is_needed($dependency, $context)) {
 			return;
 		}
-		$handler = $this->handlers[ $context['dependency_type'] ];
-		$handler->enqueue( $dependency );
+		$handler = $this->handlers[$context['dependency_type']];
+		$handler->enqueue($dependency);
 	}
 
 	/**
@@ -282,16 +282,16 @@ 
 	 *                          'dependency_type'.
 	 * @return bool Whether it is needed or not.
 	 */
-	protected function is_needed( $dependency, $context ) {
-		$is_needed = array_key_exists( 'is_needed', $dependency )
+	protected function is_needed($dependency, $context) {
+		$is_needed = array_key_exists('is_needed', $dependency)
 			? $dependency['is_needed']
 			: null;
 
-		if ( null === $is_needed ) {
+		if (null === $is_needed) {
 			return true;
 		}
 
-		return is_callable( $is_needed ) && $is_needed( $context );
+		return is_callable($is_needed) && $is_needed($context);
 	}
 
 	/**
@@ -304,18 +304,18 @@ 
 	 *                          Contains the type of the dependency at key
 	 *                          'dependency_type'.
 	 */
-	protected function maybe_localize( $dependency, $context ) {
-		if ( ! array_key_exists( 'localize', $dependency ) ) {
+	protected function maybe_localize($dependency, $context) {
+		if ( ! array_key_exists('localize', $dependency)) {
 			return;
 		}
 
 		$localize = $dependency['localize'];
 		$data     = $localize['data'];
-		if ( is_callable( $data ) ) {
-			$data = $data( $context );
+		if (is_callable($data)) {
+			$data = $data($context);
 		}
 
-		\wp_localize_script( $dependency['handle'], $localize['name'], $data );
+		\wp_localize_script($dependency['handle'], $localize['name'], $data);
 	}
 
 	/**
@@ -328,9 +328,9 @@ 
 	 * @param mixed  $context         Optional. The context to pass to the
 	 *                                dependencies.
 	 */
-	protected function enqueue_dependency_type( $dependencies, $dependency_type, $context = null ) {
+	protected function enqueue_dependency_type($dependencies, $dependency_type, $context = null) {
 		$context['dependency_type'] = $dependency_type;
-		array_walk( $dependencies, [ $this, 'enqueue_dependency' ], $context );
+		array_walk($dependencies, [$this, 'enqueue_dependency'], $context);
 	}
 
 	/**
@@ -343,9 +343,9 @@ 
 	 * @param mixed  $context         Optional. The context to pass to the
 	 *                                dependencies.
 	 */
-	protected function register_dependency_type( $dependencies, $dependency_type, $context = null ) {
+	protected function register_dependency_type($dependencies, $dependency_type, $context = null) {
 		$context['dependency_type'] = $dependency_type;
-		array_walk( $dependencies, [ $this, 'register_dependency' ], $context );
+		array_walk($dependencies, [$this, 'register_dependency'], $context);
 	}
 
 	/**
@@ -360,12 +360,12 @@ 
 	 *                               dependency at key
 	 *                               'dependency_type'.
 	 */
-	protected function register_dependency( $dependency, $dependency_key, $context = null ) {
-		$handler = $this->handlers[ $context['dependency_type'] ];
-		$handler->register( $dependency );
+	protected function register_dependency($dependency, $dependency_key, $context = null) {
+		$handler = $this->handlers[$context['dependency_type']];
+		$handler->register($dependency);
 
-		if ( $this->enqueue_immediately ) {
-			$this->register_enqueue_hooks( $dependency, $context );
+		if ($this->enqueue_immediately) {
+			$this->register_enqueue_hooks($dependency, $context);
 		}
 	}
 
@@ -379,14 +379,14 @@ 
 	 *                          Contains the type of the dependency at key
 	 *                          'dependency_type'.
 	 */
-	protected function register_enqueue_hooks( $dependency, $context = null ) {
-		$priority = $this->get_priority( $dependency );
+	protected function register_enqueue_hooks($dependency, $context = null) {
+		$priority = $this->get_priority($dependency);
 
-		foreach ( [ 'wp_enqueue_scripts', 'admin_enqueue_scripts' ] as $hook ) {
-			\add_action( $hook, [ $this, 'enqueue' ], $priority, 1 );
+		foreach (['wp_enqueue_scripts', 'admin_enqueue_scripts'] as $hook) {
+			\add_action($hook, [$this, 'enqueue'], $priority, 1);
 		}
 
-		$this->maybe_localize( $dependency, $context );
+		$this->maybe_localize($dependency, $context);
 	}
 
 	/**
@@ -397,9 +397,9 @@ 
 	 * @param array $dependency Configuration data of the dependency.
 	 * @return int Priority to use.
 	 */
-	protected function get_priority( $dependency ) {
-		if ( array_key_exists( 'priority', $dependency ) ) {
-			return intval( $dependency['priority'] );
+	protected function get_priority($dependency) {
+		if (array_key_exists('priority', $dependency)) {
+			return intval($dependency['priority']);
 		}
 		return 10;
 	}

src/DependencyManagerInterface.php

Spacing +1 added lines, -1 removed lines

@@ -36,5 +36,5 @@
 	 *                         outside of DependencyManager. Defaults to false.
 	 * @return bool Returns whether the handle was found or not.
 	 */
-	public function enqueue_handle( $handle, $context = null, $fallback = false );
+	public function enqueue_handle($handle, $context = null, $fallback = false);
 }

src/ScriptHandler.php

Spacing +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@
 	 * @param string $handle The handle to check
 	 * @return bool Whether it is registered or not.
 	 */
-	protected function is_registered( $handle ) {
-		return \wp_script_is( $handle );
+	protected function is_registered($handle) {
+		return \wp_script_is($handle);
 	}
 }

src/StyleHandler.php

Spacing +2 added lines, -2 removed lines

@@ -39,8 +39,8 @@
 	 * @param string $handle The handle to check
 	 * @return bool Whether it is registered or not.
 	 */
-	protected function is_registered( $handle ) {
-		return \wp_style_is( $handle );
+	protected function is_registered($handle) {
+		return \wp_style_is($handle);
 	}
 
 	/**