brightnucleus / view

config/defaults.php

Indentation +16 added lines, -16 removed lines

@@ -12,26 +12,26 @@
 namespace BrightNucleus\View;
 
 $engineFinder = [
-    'ClassName'  => 'BrightNucleus\View\Engine\EngineFinder',
-    'Engines'    => [
-        'PHPEngine' => 'BrightNucleus\View\Engine\PHPEngine',
-    ],
-    'NullObject' => 'BrightNucleus\View\Engine\NullEngine',
+	'ClassName'  => 'BrightNucleus\View\Engine\EngineFinder',
+	'Engines'    => [
+		'PHPEngine' => 'BrightNucleus\View\Engine\PHPEngine',
+	],
+	'NullObject' => 'BrightNucleus\View\Engine\NullEngine',
 ];
 
 $viewFinder = [
-    'ClassName'  => 'BrightNucleus\View\View\ViewFinder',
-    'Views'      => [
-        'BaseView' => 'BrightNucleus\View\View\BaseView',
-    ],
-    'NullObject' => 'BrightNucleus\View\View\NullView',
+	'ClassName'  => 'BrightNucleus\View\View\ViewFinder',
+	'Views'      => [
+		'BaseView' => 'BrightNucleus\View\View\BaseView',
+	],
+	'NullObject' => 'BrightNucleus\View\View\NullView',
 ];
 
 return [
-    'BrightNucleus' => [
-        'View' => [
-            'EngineFinder' => $engineFinder,
-            'ViewFinder'   => $viewFinder,
-        ],
-    ],
+	'BrightNucleus' => [
+		'View' => [
+			'EngineFinder' => $engineFinder,
+			'ViewFinder'   => $viewFinder,
+		],
+	],
 ];

src/View/View/ViewFinder.php

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
             $view = $view($uri, $engine);
         }
 
-        if (! $view instanceof ViewInterface) {
+        if ( ! $view instanceof ViewInterface) {
             throw new FailedToInstantiateViewException(
                 sprintf(
                     _('Could not instantiate view "%s".'),

Doc Comments +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
      *
      * @since 0.1.0
      *
-     * @param mixed                $view   View to instantiate.
+     * @param \BrightNucleus\View\Support\Findable                $view   View to instantiate.
      * @param string               $uri    URI to use for the view.
      * @param EngineInterface|null $engine Optional. Engine to use with the view.
      *

Indentation +78 added lines, -78 removed lines

@@ -26,91 +26,91 @@
 class ViewFinder extends AbstractFinder
 {
 
-    /**
-     * Find a result based on a specific criteria.
-     *
-     * @since 0.1.0
-     *
-     * @param array                $criteria Criteria to search for.
-     * @param EngineInterface|null $engine   Optional. Engine to use with the view.
-     *
-     * @return ViewInterface View that was found.
-     */
-    public function find(array $criteria, EngineInterface $engine = null)
-    {
-        $uri = $criteria[0];
+	/**
+	 * Find a result based on a specific criteria.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param array                $criteria Criteria to search for.
+	 * @param EngineInterface|null $engine   Optional. Engine to use with the view.
+	 *
+	 * @return ViewInterface View that was found.
+	 */
+	public function find(array $criteria, EngineInterface $engine = null)
+	{
+		$uri = $criteria[0];
 
-        $this->initializeViews($uri, $engine);
+		$this->initializeViews($uri, $engine);
 
-        foreach ($criteria as $entry) {
-            foreach ($this->findables as $viewObject) {
-                if ($viewObject->canHandle($entry)) {
-                    return $viewObject;
-                }
-            }
-        }
+		foreach ($criteria as $entry) {
+			foreach ($this->findables as $viewObject) {
+				if ($viewObject->canHandle($entry)) {
+					return $viewObject;
+				}
+			}
+		}
 
-        return $this->getNullObject();
-    }
+		return $this->getNullObject();
+	}
 
-    /**
-     * Initialize the views that can be iterated.
-     *
-     * @since 0.1.0
-     *
-     * @param string               $uri    URI to use for the view.
-     * @param EngineInterface|null $engine Optional. Engine to use with the view.
-     */
-    protected function initializeViews($uri, EngineInterface $engine = null)
-    {
-        foreach ($this->findables as &$view) {
-            $view = $this->initializeView($view, $uri, $engine);
-        }
-    }
+	/**
+	 * Initialize the views that can be iterated.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param string               $uri    URI to use for the view.
+	 * @param EngineInterface|null $engine Optional. Engine to use with the view.
+	 */
+	protected function initializeViews($uri, EngineInterface $engine = null)
+	{
+		foreach ($this->findables as &$view) {
+			$view = $this->initializeView($view, $uri, $engine);
+		}
+	}
 
-    /**
-     * Initialize a single view by instantiating class name strings and calling closures.
-     *
-     * @since 0.1.0
-     *
-     * @param mixed                $view   View to instantiate.
-     * @param string               $uri    URI to use for the view.
-     * @param EngineInterface|null $engine Optional. Engine to use with the view.
-     *
-     * @return ViewInterface Instantiated view.
-     * @throws FailedToInstantiateViewException If the view could not be instantiated.
-     */
-    protected function initializeView($view, $uri, EngineInterface $engine = null)
-    {
-        if (is_string($view)) {
-            $view = new $view($uri, $engine);
-        }
+	/**
+	 * Initialize a single view by instantiating class name strings and calling closures.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param mixed                $view   View to instantiate.
+	 * @param string               $uri    URI to use for the view.
+	 * @param EngineInterface|null $engine Optional. Engine to use with the view.
+	 *
+	 * @return ViewInterface Instantiated view.
+	 * @throws FailedToInstantiateViewException If the view could not be instantiated.
+	 */
+	protected function initializeView($view, $uri, EngineInterface $engine = null)
+	{
+		if (is_string($view)) {
+			$view = new $view($uri, $engine);
+		}
 
-        if (is_callable($view)) {
-            $view = $view($uri, $engine);
-        }
+		if (is_callable($view)) {
+			$view = $view($uri, $engine);
+		}
 
-        if (! $view instanceof ViewInterface) {
-            throw new FailedToInstantiateViewException(
-                sprintf(
-                    _('Could not instantiate view "%s".'),
-                    serialize($view)
-                )
-            );
-        }
+		if (! $view instanceof ViewInterface) {
+			throw new FailedToInstantiateViewException(
+				sprintf(
+					_('Could not instantiate view "%s".'),
+					serialize($view)
+				)
+			);
+		}
 
-        return $view;
-    }
+		return $view;
+	}
 
-    /**
-     * Get the config key for the Findables definitions.
-     *
-     * @since 0.1.0
-     *
-     * @return string Config key use to define the Findables.
-     */
-    protected function getFindablesConfigKey()
-    {
-        return 'Views';
-    }
+	/**
+	 * Get the config key for the Findables definitions.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @return string Config key use to define the Findables.
+	 */
+	protected function getFindablesConfigKey()
+	{
+		return 'Views';
+	}
 }

src/View/View/AbstractView.php

Indentation +48 added lines, -48 removed lines

@@ -25,55 +25,55 @@
 abstract class AbstractView implements ViewInterface, Findable
 {
 
-    /**
-     * URI of the view.
-     *
-     * @since 0.1.0
-     *
-     * @var string
-     */
-    protected $uri;
+	/**
+	 * URI of the view.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @var string
+	 */
+	protected $uri;
 
-    /**
-     * Engine to use for the view.
-     *
-     * @since 0.1.0
-     *
-     * @var EngineInterface
-     */
-    protected $engine;
+	/**
+	 * Engine to use for the view.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @var EngineInterface
+	 */
+	protected $engine;
 
-    /**
-     * Instantiate an AbstractView object.
-     *
-     * @since 0.1.0
-     *
-     * @param string          $uri    URI for the view.
-     * @param EngineInterface $engine Engine to use for the view.
-     */
-    public function __construct($uri, EngineInterface $engine)
-    {
-        $this->uri    = $uri;
-        $this->engine = $engine;
-    }
+	/**
+	 * Instantiate an AbstractView object.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param string          $uri    URI for the view.
+	 * @param EngineInterface $engine Engine to use for the view.
+	 */
+	public function __construct($uri, EngineInterface $engine)
+	{
+		$this->uri    = $uri;
+		$this->engine = $engine;
+	}
 
-    /**
-     * Render the view.
-     *
-     * @since 0.1.0
-     *
-     * @param array $context Optional. The context in which to render the view.
-     * @param bool  $echo    Optional. Whether to echo the output immediately. Defaults to false.
-     *
-     * @return string|void Rendered HTML or nothing, depending on $echo argument.
-     */
-    public function render(array $context = [], $echo = false)
-    {
-        $output = $this->engine->render($this->uri, $context);
-        if ($echo) {
-            echo $output;
-        } else {
-            return $output;
-        }
-    }
+	/**
+	 * Render the view.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param array $context Optional. The context in which to render the view.
+	 * @param bool  $echo    Optional. Whether to echo the output immediately. Defaults to false.
+	 *
+	 * @return string|void Rendered HTML or nothing, depending on $echo argument.
+	 */
+	public function render(array $context = [], $echo = false)
+	{
+		$output = $this->engine->render($this->uri, $context);
+		if ($echo) {
+			echo $output;
+		} else {
+			return $output;
+		}
+	}
 }

src/View/View/ViewInterface.php

Indentation +10 added lines, -10 removed lines

@@ -22,14 +22,14 @@
 interface ViewInterface
 {
 
-    /**
-     * Render the view.
-     *
-     * @since 0.1.0
-     *
-     * @param array $context Optional. The context in which to render the view.
-     *
-     * @return string Rendered HTML.
-     */
-    public function render(array $context = []);
+	/**
+	 * Render the view.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param array $context Optional. The context in which to render the view.
+	 *
+	 * @return string Rendered HTML.
+	 */
+	public function render(array $context = []);
 }

src/View/Engine/AbstractEngine.php

Indentation +19 added lines, -19 removed lines

@@ -24,24 +24,24 @@
 abstract class AbstractEngine implements EngineInterface, Findable
 {
 
-    /**
-     * Check whether a given URI has a specific extension.
-     *
-     * @since 0.1.0
-     *
-     * @param string $uri       URI to check the extension of.
-     * @param string $extension Extension to check for.
-     *
-     * @return bool
-     */
-    protected function hasExtension($uri, $extension)
-    {
-        $uriLength       = mb_strlen($uri);
-        $extensionLength = mb_strlen($extension);
-        if ($extensionLength > $uriLength) {
-            return false;
-        }
+	/**
+	 * Check whether a given URI has a specific extension.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param string $uri       URI to check the extension of.
+	 * @param string $extension Extension to check for.
+	 *
+	 * @return bool
+	 */
+	protected function hasExtension($uri, $extension)
+	{
+		$uriLength       = mb_strlen($uri);
+		$extensionLength = mb_strlen($extension);
+		if ($extensionLength > $uriLength) {
+			return false;
+		}
 
-        return substr_compare($uri, $extension, $uriLength - $extensionLength, $extensionLength) === 0;
-    }
+		return substr_compare($uri, $extension, $uriLength - $extensionLength, $extensionLength) === 0;
+	}
 }

src/View/Engine/PHPEngine.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
      */
     public function render($uri, array $context = [])
     {
-        if (! is_readable($uri)) {
+        if ( ! is_readable($uri)) {
             throw new FailedToLoadViewException(
                 sprintf(
                     _('The View URI "%1$s" is not accessible or readable.'),

Indentation +61 added lines, -61 removed lines

@@ -25,73 +25,73 @@
 class PHPEngine extends AbstractEngine
 {
 
-    const PHP_EXTENSION = '.php';
+	const PHP_EXTENSION = '.php';
 
-    /**
-     * Check whether the Findable can handle an individual criterion.
-     *
-     * @since 0.1.0
-     *
-     * @param mixed $criterion Criterion to check.
-     *
-     * @return bool Whether the Findable can handle the criterion.
-     */
-    public function canHandle($criterion)
-    {
-        return $this->hasExtension($criterion, static::PHP_EXTENSION)
-               && is_readable($criterion);
-    }
+	/**
+	 * Check whether the Findable can handle an individual criterion.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param mixed $criterion Criterion to check.
+	 *
+	 * @return bool Whether the Findable can handle the criterion.
+	 */
+	public function canHandle($criterion)
+	{
+		return $this->hasExtension($criterion, static::PHP_EXTENSION)
+			   && is_readable($criterion);
+	}
 
-    /**
-     * Render a given URI.
-     *
-     * @since 0.1.0
-     *
-     * @param string $uri     URI to render.
-     * @param array  $context Context in which to render.
-     *
-     * @return string Rendered HTML.
-     * @throws FailedToLoadViewException If the View URI is not accessible or readable.
-     * @throws FailedToLoadViewException If the View URI could not be loaded.
-     */
-    public function render($uri, array $context = [])
-    {
-        if (! is_readable($uri)) {
-            throw new FailedToLoadViewException(
-                sprintf(
-                    _('The View URI "%1$s" is not accessible or readable.'),
-                    $uri
-                )
-            );
-        }
+	/**
+	 * Render a given URI.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param string $uri     URI to render.
+	 * @param array  $context Context in which to render.
+	 *
+	 * @return string Rendered HTML.
+	 * @throws FailedToLoadViewException If the View URI is not accessible or readable.
+	 * @throws FailedToLoadViewException If the View URI could not be loaded.
+	 */
+	public function render($uri, array $context = [])
+	{
+		if (! is_readable($uri)) {
+			throw new FailedToLoadViewException(
+				sprintf(
+					_('The View URI "%1$s" is not accessible or readable.'),
+					$uri
+				)
+			);
+		}
 
-        extract($context, EXTR_SKIP);
+		extract($context, EXTR_SKIP);
 
-        // Save current buffering level so we can backtrack in case of an error.
-        // This is needed because the view itself might also add an unknown number of output buffering levels.
-        $bufferLevel = ob_get_level();
-        ob_start();
+		// Save current buffering level so we can backtrack in case of an error.
+		// This is needed because the view itself might also add an unknown number of output buffering levels.
+		$bufferLevel = ob_get_level();
+		ob_start();
 
-        try {
-            include($uri);
-        } catch (Exception $exception) {
+		try {
+			include($uri);
+		} catch (Exception $exception) {
 
-            // Remove whatever levels were added up until now.
-            while (ob_get_level() > $bufferLevel) {
-                ob_end_clean();
-            }
+			// Remove whatever levels were added up until now.
+			while (ob_get_level() > $bufferLevel) {
+				ob_end_clean();
+			}
 
-            throw new FailedToLoadViewException(
-                sprintf(
-                    _('Could not load the View URI "%1$s". Reason: "%2$s".'),
-                    $uri,
-                    $exception->getMessage()
-                ),
-                $exception->getCode(),
-                $exception
-            );
-        }
+			throw new FailedToLoadViewException(
+				sprintf(
+					_('Could not load the View URI "%1$s". Reason: "%2$s".'),
+					$uri,
+					$exception->getMessage()
+				),
+				$exception->getCode(),
+				$exception
+			);
+		}
 
-        return ob_get_clean();
-    }
+		return ob_get_clean();
+	}
 }

src/View/Engine/EngineFinder.php

Spacing +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
             $engine = $engine();
         }
 
-        if (! $engine instanceof EngineInterface) {
+        if ( ! $engine instanceof EngineInterface) {
             throw new FailedToInstantiateEngineException(
                 sprintf(
                     _('Could not instantiate engine "%s".'),

Doc Comments +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
      *
      * @since 0.1.0
      *
-     * @param mixed $engine Engine to instantiate.
+     * @param \BrightNucleus\View\Support\Findable $engine Engine to instantiate.
      *
      * @return EngineInterface Instantiated engine.
      * @throws FailedToInstantiateEngineException If the engine could not be instantiated.

Indentation +72 added lines, -72 removed lines

@@ -25,84 +25,84 @@
 class EngineFinder extends AbstractFinder
 {
 
-    /**
-     * Find a result based on a specific criteria.
-     *
-     * @since 0.1.0
-     *
-     * @param array $criteria Criteria to search for.
-     *
-     * @return EngineInterface Result of the search.
-     */
-    public function find(array $criteria)
-    {
-        $this->initializeEngines();
+	/**
+	 * Find a result based on a specific criteria.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param array $criteria Criteria to search for.
+	 *
+	 * @return EngineInterface Result of the search.
+	 */
+	public function find(array $criteria)
+	{
+		$this->initializeEngines();
 
-        foreach ($criteria as $entry) {
-            foreach ($this->findables as $engine) {
-                if ($engine->canHandle($entry)) {
-                    return $engine;
-                }
-            }
-        }
+		foreach ($criteria as $entry) {
+			foreach ($this->findables as $engine) {
+				if ($engine->canHandle($entry)) {
+					return $engine;
+				}
+			}
+		}
 
-        return $this->getNullObject();
-    }
+		return $this->getNullObject();
+	}
 
-    /**
-     * Initialize the engines that can be iterated.
-     *
-     * @since 0.1.0
-     *
-     */
-    protected function initializeEngines()
-    {
-        foreach ($this->findables as &$engine) {
-            $engine = $this->initializeEngine($engine);
-        }
-    }
+	/**
+	 * Initialize the engines that can be iterated.
+	 *
+	 * @since 0.1.0
+	 *
+	 */
+	protected function initializeEngines()
+	{
+		foreach ($this->findables as &$engine) {
+			$engine = $this->initializeEngine($engine);
+		}
+	}
 
-    /**
-     * Initialize a single engine by instantiating class name strings and calling closures.
-     *
-     * @since 0.1.0
-     *
-     * @param mixed $engine Engine to instantiate.
-     *
-     * @return EngineInterface Instantiated engine.
-     * @throws FailedToInstantiateEngineException If the engine could not be instantiated.
-     */
-    protected function initializeEngine($engine)
-    {
-        if (is_string($engine)) {
-            $engine = new $engine();
-        }
+	/**
+	 * Initialize a single engine by instantiating class name strings and calling closures.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param mixed $engine Engine to instantiate.
+	 *
+	 * @return EngineInterface Instantiated engine.
+	 * @throws FailedToInstantiateEngineException If the engine could not be instantiated.
+	 */
+	protected function initializeEngine($engine)
+	{
+		if (is_string($engine)) {
+			$engine = new $engine();
+		}
 
-        if (is_callable($engine)) {
-            $engine = $engine();
-        }
+		if (is_callable($engine)) {
+			$engine = $engine();
+		}
 
-        if (! $engine instanceof EngineInterface) {
-            throw new FailedToInstantiateEngineException(
-                sprintf(
-                    _('Could not instantiate engine "%s".'),
-                    serialize($engine)
-                )
-            );
-        }
+		if (! $engine instanceof EngineInterface) {
+			throw new FailedToInstantiateEngineException(
+				sprintf(
+					_('Could not instantiate engine "%s".'),
+					serialize($engine)
+				)
+			);
+		}
 
-        return $engine;
-    }
+		return $engine;
+	}
 
-    /**
-     * Get the config key for the Findables definitions.
-     *
-     * @since 0.1.0
-     *
-     * @return string Config key use to define the Findables.
-     */
-    protected function getFindablesConfigKey()
-    {
-        return 'Engines';
-    }
+	/**
+	 * Get the config key for the Findables definitions.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @return string Config key use to define the Findables.
+	 */
+	protected function getFindablesConfigKey()
+	{
+		return 'Engines';
+	}
 }

src/View/ViewBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -226,7 +226,7 @@
             $type = $type($uri, $engine);
         }
 
-        if (! $type instanceof ViewInterface) {
+        if ( ! $type instanceof ViewInterface) {
             throw new FailedToInstantiateViewException(
                 sprintf(
                     _('Could not instantiate view "%s".'),

Doc Comments +2 added lines, -2 removed lines

@@ -91,7 +91,7 @@ 
      * @since 0.1.0
      *
      * @param string $view View identifier to create a view for.
-     * @param mixed  $type Type of view to create.
+     * @param string|null  $type Type of view to create.
      *
      * @return ViewInterface Instance of the requested view.
      */
@@ -190,7 +190,7 @@ 
      *
      * @since 0.1.0
      *
-     * @param array $criteria Criteria to match.
+     * @param string[] $criteria Criteria to match.
      *
      * @return string|false URI of the requested view, or false if not found.
      */

Indentation +201 added lines, -201 removed lines

@@ -33,228 +33,228 @@
 class ViewBuilder
 {
 
-    use ConfigTrait;
+	use ConfigTrait;
 
-    const ENGINE_FINDER_KEY = 'EngineFinder';
-    const VIEW_FINDER_KEY   = 'ViewFinder';
+	const ENGINE_FINDER_KEY = 'EngineFinder';
+	const VIEW_FINDER_KEY   = 'ViewFinder';
 
-    /**
-     * ViewFinder instance.
-     *
-     * @since 0.1.0
-     *
-     * @var ViewFinderInterface
-     */
-    protected $viewFinder;
+	/**
+	 * ViewFinder instance.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @var ViewFinderInterface
+	 */
+	protected $viewFinder;
 
-    /**
-     * EngineFinder instance.
-     *
-     * @since 0.1.0
-     *
-     * @var EngineFinderInterface
-     */
-    protected $engineFinder;
+	/**
+	 * EngineFinder instance.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @var EngineFinderInterface
+	 */
+	protected $engineFinder;
 
-    /**
-     * Locations to scan for views.
-     *
-     * @since 0.1.0
-     *
-     * @var LocationInterface[]
-     */
-    protected $locations;
+	/**
+	 * Locations to scan for views.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @var LocationInterface[]
+	 */
+	protected $locations;
 
-    /**
-     * Instantiate a ViewBuilder object.
-     *
-     * @since 0.1.0
-     *
-     * @param ConfigInterface            $config       Configuration settings.
-     * @param ViewFinderInterface|null   $viewFinder   ViewFinder instance.
-     * @param EngineFinderInterface|null $engineFinder EngineFinder instance.
-     *
-     * @throws FailedToProcessConfigException If the config could not be processed.
-     */
-    public function __construct(
-        ConfigInterface $config,
-        ViewFinderInterface $viewFinder = null,
-        EngineFinderInterface $engineFinder = null
-    ) {
-        $this->processConfig($config);
-        $this->viewFinder   = $viewFinder;
-        $this->engineFinder = $engineFinder;
-    }
+	/**
+	 * Instantiate a ViewBuilder object.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param ConfigInterface            $config       Configuration settings.
+	 * @param ViewFinderInterface|null   $viewFinder   ViewFinder instance.
+	 * @param EngineFinderInterface|null $engineFinder EngineFinder instance.
+	 *
+	 * @throws FailedToProcessConfigException If the config could not be processed.
+	 */
+	public function __construct(
+		ConfigInterface $config,
+		ViewFinderInterface $viewFinder = null,
+		EngineFinderInterface $engineFinder = null
+	) {
+		$this->processConfig($config);
+		$this->viewFinder   = $viewFinder;
+		$this->engineFinder = $engineFinder;
+	}
 
-    /**
-     * Create a new view for a given URI.
-     *
-     * @since 0.1.0
-     *
-     * @param string $view View identifier to create a view for.
-     * @param mixed  $type Type of view to create.
-     *
-     * @return ViewInterface Instance of the requested view.
-     */
-    public function create($view, $type = null)
-    {
-        $uri    = $this->scanLocations([$view]);
-        $engine = $this->getEngine($uri);
+	/**
+	 * Create a new view for a given URI.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param string $view View identifier to create a view for.
+	 * @param mixed  $type Type of view to create.
+	 *
+	 * @return ViewInterface Instance of the requested view.
+	 */
+	public function create($view, $type = null)
+	{
+		$uri    = $this->scanLocations([$view]);
+		$engine = $this->getEngine($uri);
 
-        return $uri
-            ? $this->getView($uri, $engine, $type)
-            : $this->getViewFinder()->getNullObject();
-    }
+		return $uri
+			? $this->getView($uri, $engine, $type)
+			: $this->getViewFinder()->getNullObject();
+	}
 
-    /**
-     * Get an Engine that can deal with the given URI.
-     *
-     * @since 0.1.0
-     *
-     * @param string|false $uri URI to get an engine for.
-     *
-     * @return EngineInterface Instance of an engine that can deal with the given URI.
-     */
-    public function getEngine($uri)
-    {
-        return $this->getEngineFinder()->find([$uri]);
-    }
+	/**
+	 * Get an Engine that can deal with the given URI.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param string|false $uri URI to get an engine for.
+	 *
+	 * @return EngineInterface Instance of an engine that can deal with the given URI.
+	 */
+	public function getEngine($uri)
+	{
+		return $this->getEngineFinder()->find([$uri]);
+	}
 
-    /**
-     * Get a view for a given URI, engine and type.
-     *
-     * @since 0.1.0
-     *
-     * @param string          $uri    URI to get a view for.
-     * @param EngineInterface $engine Engine to use for the view.
-     * @param mixed           $type   Type of view to get.
-     *
-     * @return ViewInterface View that matches the given requirements.
-     */
-    public function getView($uri, EngineInterface $engine, $type = null)
-    {
-        if (null === $type) {
-            return $this->getViewFinder()->find([$uri], $engine);
-        }
+	/**
+	 * Get a view for a given URI, engine and type.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param string          $uri    URI to get a view for.
+	 * @param EngineInterface $engine Engine to use for the view.
+	 * @param mixed           $type   Type of view to get.
+	 *
+	 * @return ViewInterface View that matches the given requirements.
+	 */
+	public function getView($uri, EngineInterface $engine, $type = null)
+	{
+		if (null === $type) {
+			return $this->getViewFinder()->find([$uri], $engine);
+		}
 
-        return $this->resolveType($type, $uri, $engine);
-    }
+		return $this->resolveType($type, $uri, $engine);
+	}
 
-    /**
-     * Get the ViewFinder instance.
-     *
-     * @since 0.1.0
-     *
-     * @return ViewFinderInterface Instance of a ViewFinder.
-     */
-    public function getViewFinder()
-    {
-        return $this->getFinder($this->viewFinder, static::VIEW_FINDER_KEY);
-    }
+	/**
+	 * Get the ViewFinder instance.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @return ViewFinderInterface Instance of a ViewFinder.
+	 */
+	public function getViewFinder()
+	{
+		return $this->getFinder($this->viewFinder, static::VIEW_FINDER_KEY);
+	}
 
-    /**
-     * Get the EngineFinder instance.
-     *
-     * @since 0.1.0
-     *
-     * @return EngineFinderInterface Instance of a EngineFinder.
-     */
-    public function getEngineFinder()
-    {
-        return $this->getFinder($this->engineFinder, static::ENGINE_FINDER_KEY);
-    }
+	/**
+	 * Get the EngineFinder instance.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @return EngineFinderInterface Instance of a EngineFinder.
+	 */
+	public function getEngineFinder()
+	{
+		return $this->getFinder($this->engineFinder, static::ENGINE_FINDER_KEY);
+	}
 
-    /**
-     * Add a location to scan with the ViewFinder.
-     *
-     * @since 0.1.0
-     *
-     * @param LocationInterface $location Location to scan with the ViewFinder.
-     */
-    public function addLocation(LocationInterface $location)
-    {
-        $this->locations[] = $location;
-    }
+	/**
+	 * Add a location to scan with the ViewFinder.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param LocationInterface $location Location to scan with the ViewFinder.
+	 */
+	public function addLocation(LocationInterface $location)
+	{
+		$this->locations[] = $location;
+	}
 
-    /**
-     * Scan Locations for an URI that matches the specified criteria.
-     *
-     * @since 0.1.0
-     *
-     * @param array $criteria Criteria to match.
-     *
-     * @return string|false URI of the requested view, or false if not found.
-     */
-    public function scanLocations(array $criteria)
-    {
-        /** @var LocationInterface $location */
-        foreach ($this->locations as $location) {
-            if ($uri = $location->getURI($criteria)) {
-                return $uri;
-            }
-        }
+	/**
+	 * Scan Locations for an URI that matches the specified criteria.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param array $criteria Criteria to match.
+	 *
+	 * @return string|false URI of the requested view, or false if not found.
+	 */
+	public function scanLocations(array $criteria)
+	{
+		/** @var LocationInterface $location */
+		foreach ($this->locations as $location) {
+			if ($uri = $location->getURI($criteria)) {
+				return $uri;
+			}
+		}
 
-        return false;
-    }
+		return false;
+	}
 
-    /**
-     * Get a finder instance.
-     *
-     * @since 0.1.1
-     *
-     * @param mixed  $property Property to use.
-     * @param string $key      Configuration key to use.
-     *
-     * @return FinderInterface The requested finder instance.
-     */
-    protected function getFinder(&$property, $key)
-    {
-        if (null === $property) {
-            $engineFinderClass = $this->config->getKey($key, 'ClassName');
-            $property          = new $engineFinderClass($this->config->getSubConfig($key));
-        }
+	/**
+	 * Get a finder instance.
+	 *
+	 * @since 0.1.1
+	 *
+	 * @param mixed  $property Property to use.
+	 * @param string $key      Configuration key to use.
+	 *
+	 * @return FinderInterface The requested finder instance.
+	 */
+	protected function getFinder(&$property, $key)
+	{
+		if (null === $property) {
+			$engineFinderClass = $this->config->getKey($key, 'ClassName');
+			$property          = new $engineFinderClass($this->config->getSubConfig($key));
+		}
 
-        return $property;
-    }
+		return $property;
+	}
 
-    /**
-     * Resolve the view type.
-     *
-     * @since 0.1.0
-     *
-     * @param mixed                $type   Type of view that was requested.
-     * @param string               $uri    URI to get a view for.
-     * @param EngineInterface|null $engine Engine to use for the view.
-     *
-     * @return ViewInterface Resolved View object.
-     * @throws FailedToInstantiateViewException If the view type could not be resolved.
-     */
-    protected function resolveType($type, $uri, EngineInterface $engine = null)
-    {
-        $configKey = [static::VIEW_FINDER_KEY, 'Views', $type];
+	/**
+	 * Resolve the view type.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param mixed                $type   Type of view that was requested.
+	 * @param string               $uri    URI to get a view for.
+	 * @param EngineInterface|null $engine Engine to use for the view.
+	 *
+	 * @return ViewInterface Resolved View object.
+	 * @throws FailedToInstantiateViewException If the view type could not be resolved.
+	 */
+	protected function resolveType($type, $uri, EngineInterface $engine = null)
+	{
+		$configKey = [static::VIEW_FINDER_KEY, 'Views', $type];
 
-        if (is_string($type) && $this->config->hasKey($configKey)) {
-            $className = $this->config->getKey($configKey);
-            $type      = new $className($uri, $engine);
-        }
+		if (is_string($type) && $this->config->hasKey($configKey)) {
+			$className = $this->config->getKey($configKey);
+			$type      = new $className($uri, $engine);
+		}
 
-        if (is_string($type)) {
-            $type = new $type($uri, $engine);
-        }
+		if (is_string($type)) {
+			$type = new $type($uri, $engine);
+		}
 
-        if (is_callable($type)) {
-            $type = $type($uri, $engine);
-        }
+		if (is_callable($type)) {
+			$type = $type($uri, $engine);
+		}
 
-        if (! $type instanceof ViewInterface) {
-            throw new FailedToInstantiateViewException(
-                sprintf(
-                    _('Could not instantiate view "%s".'),
-                    serialize($type)
-                )
-            );
-        }
+		if (! $type instanceof ViewInterface) {
+			throw new FailedToInstantiateViewException(
+				sprintf(
+					_('Could not instantiate view "%s".'),
+					serialize($type)
+				)
+			);
+		}
 
-        return $type;
-    }
+		return $type;
+	}
 }

src/View.php

Spacing +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
      */
     public static function getDefaultConfig()
     {
-        return ConfigFactory::create(__DIR__ . '/../config/defaults.php')
+        return ConfigFactory::create(__DIR__.'/../config/defaults.php')
                             ->getSubConfig('BrightNucleus\View');
     }
 

Indentation +91 added lines, -91 removed lines

@@ -28,104 +28,104 @@
 class View
 {
 
-    /**
-     * ViewBuilder Instance.
-     *
-     * @since 0.1.0
-     *
-     * @var ViewBuilder
-     */
-    protected static $viewBuilder;
+	/**
+	 * ViewBuilder Instance.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @var ViewBuilder
+	 */
+	protected static $viewBuilder;
 
-    /**
-     * Add a location to the ViewBuilder.
-     *
-     * @since 0.1.0
-     *
-     * @param LocationInterface $location Location to add.
-     */
-    public static function addLocation(LocationInterface $location)
-    {
-        $viewBuilder = static::getViewBuilder();
-        $viewBuilder->addLocation($location);
-    }
+	/**
+	 * Add a location to the ViewBuilder.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param LocationInterface $location Location to add.
+	 */
+	public static function addLocation(LocationInterface $location)
+	{
+		$viewBuilder = static::getViewBuilder();
+		$viewBuilder->addLocation($location);
+	}
 
-    /**
-     * Get the ViewBuilder instance.
-     *
-     * @since 0.1.0
-     *
-     * @return ViewBuilder
-     */
-    public static function getViewBuilder()
-    {
-        if (null === static::$viewBuilder) {
-            static::$viewBuilder = static::instantiateViewBuilder();
-        }
+	/**
+	 * Get the ViewBuilder instance.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @return ViewBuilder
+	 */
+	public static function getViewBuilder()
+	{
+		if (null === static::$viewBuilder) {
+			static::$viewBuilder = static::instantiateViewBuilder();
+		}
 
-        return static::$viewBuilder;
-    }
+		return static::$viewBuilder;
+	}
 
-    /**
-     * Instantiate the ViewBuilder.
-     *
-     * @since 0.1.0
-     *
-     * @param ConfigInterface|null $config Optional. Configuration to pass into the ViewBuilder.
-     *
-     * @return ViewBuilder Instance of the ViewBuilder.
-     */
-    public static function instantiateViewBuilder(ConfigInterface $config = null)
-    {
-        return static::$viewBuilder = new ViewBuilder($config ?: static::getDefaultConfig());
-    }
+	/**
+	 * Instantiate the ViewBuilder.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param ConfigInterface|null $config Optional. Configuration to pass into the ViewBuilder.
+	 *
+	 * @return ViewBuilder Instance of the ViewBuilder.
+	 */
+	public static function instantiateViewBuilder(ConfigInterface $config = null)
+	{
+		return static::$viewBuilder = new ViewBuilder($config ?: static::getDefaultConfig());
+	}
 
-    /**
-     * Get the default configuration to inject into the ViewBuilder.
-     *
-     * @since 0.1.0
-     *
-     * @return ConfigInterface Default configuration.
-     */
-    public static function getDefaultConfig()
-    {
-        return ConfigFactory::create(__DIR__ . '/../config/defaults.php')
-                            ->getSubConfig('BrightNucleus\View');
-    }
+	/**
+	 * Get the default configuration to inject into the ViewBuilder.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @return ConfigInterface Default configuration.
+	 */
+	public static function getDefaultConfig()
+	{
+		return ConfigFactory::create(__DIR__ . '/../config/defaults.php')
+							->getSubConfig('BrightNucleus\View');
+	}
 
-    /**
-     * Create a new view for a given URI.
-     *
-     * @since 0.1.0
-     *
-     * @param string      $view View identifier to create a view for.
-     * @param string|null $type Type of view to create.
-     *
-     * @return ViewInterface Instance of the requested view.
-     */
-    public static function create($view, $type = null)
-    {
-        $viewBuilder = static::getViewBuilder();
+	/**
+	 * Create a new view for a given URI.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param string      $view View identifier to create a view for.
+	 * @param string|null $type Type of view to create.
+	 *
+	 * @return ViewInterface Instance of the requested view.
+	 */
+	public static function create($view, $type = null)
+	{
+		$viewBuilder = static::getViewBuilder();
 
-        return $viewBuilder->create($view, $type);
-    }
+		return $viewBuilder->create($view, $type);
+	}
 
-    /**
-     * Render a view for a given URI.
-     *
-     * @since 0.1.0
-     *
-     * @param string      $view    View identifier to create a view for.
-     * @param array       $context Optional. The context in which to render the view.
-     * @param string|null $type    Type of view to create.
-     *
-     * @return string Rendered HTML content.
-     */
-    public static function render($view, array $context = [], $type = null)
-    {
-        $viewBuilder = static::getViewBuilder();
-        $viewObject  = $viewBuilder->create($view, $type);
+	/**
+	 * Render a view for a given URI.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param string      $view    View identifier to create a view for.
+	 * @param array       $context Optional. The context in which to render the view.
+	 * @param string|null $type    Type of view to create.
+	 *
+	 * @return string Rendered HTML content.
+	 */
+	public static function render($view, array $context = [], $type = null)
+	{
+		$viewBuilder = static::getViewBuilder();
+		$viewObject  = $viewBuilder->create($view, $type);
 
-        return $viewObject->render($context);
-    }
+		return $viewObject->render($context);
+	}
 }