brightnucleus / dependencies

src/DependencyManager.php

Spacing +84 added lines, -84 removed lines

@@ -50,7 +50,7 @@ 
 	 *
 	 * @var array;
 	 */
-	protected $dependencies = [ ];
+	protected $dependencies = [];
 
 	/**
 	 * Hold the handlers.
@@ -59,7 +59,7 @@ 
 	 *
 	 * @var DependencyHandlerInterface[]
 	 */
-	protected $handlers = [ ];
+	protected $handlers = [];
 
 	/**
 	 * Whether to enqueue immediately upon registration.
@@ -83,8 +83,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();
@@ -96,10 +96,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);
 			}
 		}
 	}
@@ -111,13 +111,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;
 			}
 		}
 	}
@@ -130,8 +130,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:
@@ -147,10 +147,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->init_dependency_type( $dependency_type );
+		array_walk($this->handlers,
+			function($handler, $dependency_type) {
+				if ($this->hasConfigKey($dependency_type)) {
+					$this->dependencies[$dependency_type] = $this->init_dependency_type($dependency_type);
 				}
 			} );
 	}
@@ -163,13 +163,13 @@ 
 	 * @param string $type The type of dependency to initialize.
 	 * @return array Array of dependency configurations.
 	 */
-	protected function init_dependency_type( $type ) {
-		$array = [ ];
-		$data  = $this->getConfigKey( $type );
-		foreach ( $data as $dependency ) {
-			$handle           = array_key_exists( 'handle',
-				$dependency ) ? $dependency['handle'] : '';
-			$array[ $handle ] = $dependency;
+	protected function init_dependency_type($type) {
+		$array = [];
+		$data  = $this->getConfigKey($type);
+		foreach ($data as $dependency) {
+			$handle           = array_key_exists('handle',
+				$dependency) ? $dependency['handle'] : '';
+			$array[$handle] = $dependency;
 		}
 		return $array;
 	}
@@ -181,10 +181,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);
 	}
 
 	/**
@@ -195,9 +195,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;
 	}
@@ -210,11 +210,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);
 	}
 
 	/**
@@ -229,9 +229,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 ) {
-		if ( ! $this->enqueue_internal_handle( $handle, $context ) ) {
-			return $this->enqueue_fallback_handle( $handle );
+	public function enqueue_handle($handle, $context = null, $fallback = false) {
+		if ( ! $this->enqueue_internal_handle($handle, $context)) {
+			return $this->enqueue_fallback_handle($handle);
 		}
 		return true;
 	}
@@ -247,11 +247,11 @@ 
 	 *                         dependencies.
 	 * @return bool Returns whether the handle was found or not.
 	 */
-	protected function enqueue_internal_handle( $handle, $context = null ) {
-		list( $dependency_type, $dependency ) = $this->get_dependency_array( $handle );
+	protected function enqueue_internal_handle($handle, $context = null) {
+		list($dependency_type, $dependency) = $this->get_dependency_array($handle);
 		$context['dependency_type'] = $dependency_type;
 
-		if ( ! $dependency ) {
+		if ( ! $dependency) {
 			return false;
 		}
 
@@ -261,7 +261,7 @@ 
 			$context
 		);
 
-		$this->maybe_localize( $dependency, $context );
+		$this->maybe_localize($dependency, $context);
 
 		return true;
 	}
@@ -275,14 +275,14 @@ 
 	 * @return array Array containing the dependency key as well as the
 	 *                       dependency array itself.
 	 */
-	protected function get_dependency_array( $handle ) {
-		foreach ( $this->dependencies as $type => $dependencies ) {
-			if ( array_key_exists( $handle, $dependencies ) ) {
-				return [ $type, $dependencies[ $handle ] ];
+	protected function get_dependency_array($handle) {
+		foreach ($this->dependencies as $type => $dependencies) {
+			if (array_key_exists($handle, $dependencies)) {
+				return [$type, $dependencies[$handle]];
 			}
 		}
 		// Handle not found, return an empty array.
-		return [ '', null ];
+		return ['', null];
 	}
 
 	/**
@@ -297,12 +297,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);
 	}
 
 	/**
@@ -316,16 +316,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);
 	}
 
 	/**
@@ -338,18 +338,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);
 	}
 
 	/**
@@ -361,10 +361,10 @@ 
 	 * @param string $handle The dependency handle to enqueue.
 	 * @return bool Returns whether the handle was found or not.
 	 */
-	protected function enqueue_fallback_handle( $handle ) {
+	protected function enqueue_fallback_handle($handle) {
 		$result = false;
-		foreach ( $this->handlers as $handler ) {
-			$result = $result || $handler->maybe_enqueue( $handle );
+		foreach ($this->handlers as $handler) {
+			$result = $result || $handler->maybe_enqueue($handle);
 		}
 		return $result;
 	}
@@ -379,9 +379,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);
 	}
 
 	/**
@@ -394,9 +394,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);
 	}
 
 	/**
@@ -411,12 +411,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);
 		}
 	}
 
@@ -430,14 +430,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);
 	}
 
 	/**
@@ -448,9 +448,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/StyleHandler.php

Spacing +2 added lines, -2 removed lines

@@ -41,8 +41,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, 'registered' );
+	protected function is_registered($handle) {
+		return wp_style_is($handle, 'registered');
 	}
 
 	/**

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(), (array) $args );
+	public function register($args = null) {
+		$this->invokeFunction($this->get_register_function(), (array) $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(), (array) $args );
+	public function enqueue($args = null) {
+		$this->invokeFunction($this->get_enqueue_function(), (array) $args);
 	}
 
 	/**
@@ -80,10 +80,10 @@ 
 	 * @param string $handle Handle of the dependency to enqueue.
 	 * @return bool Whether the handle was found or not.
 	 */
-	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);
 			return true;
 		}
 		return false;
@@ -97,5 +97,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/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, 'registered' );
+	protected function is_registered($handle) {
+		return wp_script_is($handle, 'registered');
 	}
 }