brightnucleus / view

src/View/Engine/NullEngine.php

Doc Comments +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
      * @param string $uri     URI to render.
      * @param array  $context Context in which to render.
      *
-     * @return callable Rendering callback.
+     * @return \Closure Rendering callback.
      */
     public function getRenderCallback(string $uri, array $context = []): callable
     {

Indentation +27 added lines, -27 removed lines

@@ -24,32 +24,32 @@
 class NullEngine implements Engine, NullFindable
 {
 
-    /**
-     * 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): bool
-    {
-        return true;
-    }
+	/**
+	 * 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): bool
+	{
+		return true;
+	}
 
-    /**
-     * Get the rendering callback for a given URI.
-     *
-     * @since 0.1.0
-     *
-     * @param string $uri     URI to render.
-     * @param array  $context Context in which to render.
-     *
-     * @return callable Rendering callback.
-     */
-    public function getRenderCallback(string $uri, array $context = []): callable
-    {
-        return function () { return ''; };
-    }
+	/**
+	 * Get the rendering callback for a given URI.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param string $uri     URI to render.
+	 * @param array  $context Context in which to render.
+	 *
+	 * @return callable Rendering callback.
+	 */
+	public function getRenderCallback(string $uri, array $context = []): callable
+	{
+		return function () { return ''; };
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -50,6 +50,6 @@
      */
     public function getRenderCallback(string $uri, array $context = []): callable
     {
-        return function () { return ''; };
+        return function() { return ''; };
     }
 }

src/View/ViewBuilder.php

Indentation +242 added lines, -242 removed lines

@@ -33,274 +33,274 @@
 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';
 
-    /**
-     * BaseViewFinder instance.
-     *
-     * @since 0.1.0
-     *
-     * @var ViewFinder
-     */
-    protected $viewFinder;
+	/**
+	 * BaseViewFinder instance.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @var ViewFinder
+	 */
+	protected $viewFinder;
 
-    /**
-     * BaseEngineFinder instance.
-     *
-     * @since 0.1.0
-     *
-     * @var EngineFinder
-     */
-    protected $engineFinder;
+	/**
+	 * BaseEngineFinder instance.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @var EngineFinder
+	 */
+	protected $engineFinder;
 
-    /**
-     * Locations to scan for views.
-     *
-     * @since 0.1.0
-     *
-     * @var Locations
-     */
-    protected $locations;
+	/**
+	 * Locations to scan for views.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @var Locations
+	 */
+	protected $locations;
 
-    /**
-     * Instantiate a ViewBuilder object.
-     *
-     * @since 0.1.0
-     *
-     * @param ConfigInterface   $config       Optional. Configuration settings.
-     * @param ViewFinder|null   $viewFinder   Optional. ViewFinder instance.
-     * @param EngineFinder|null $engineFinder Optional. EngineFinder instance.
-     *
-     * @throws FailedToProcessConfigException If the config could not be processed.
-     */
-    public function __construct(
-        ConfigInterface $config = null,
-        ViewFinder $viewFinder = null,
-        EngineFinder $engineFinder = null
-    ) {
-        $this->processConfig($this->getConfig($config));
-        $this->viewFinder   = $viewFinder ?? $this->getViewFinder();
-        $this->engineFinder = $engineFinder ?? $this->getEngineFinder();
-        $this->locations    = new Locations();
-    }
+	/**
+	 * Instantiate a ViewBuilder object.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param ConfigInterface   $config       Optional. Configuration settings.
+	 * @param ViewFinder|null   $viewFinder   Optional. ViewFinder instance.
+	 * @param EngineFinder|null $engineFinder Optional. EngineFinder instance.
+	 *
+	 * @throws FailedToProcessConfigException If the config could not be processed.
+	 */
+	public function __construct(
+		ConfigInterface $config = null,
+		ViewFinder $viewFinder = null,
+		EngineFinder $engineFinder = null
+	) {
+		$this->processConfig($this->getConfig($config));
+		$this->viewFinder   = $viewFinder ?? $this->getViewFinder();
+		$this->engineFinder = $engineFinder ?? $this->getEngineFinder();
+		$this->locations    = new Locations();
+	}
 
-    /**
-     * 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 View Instance of the requested view.
-     * @throws FailedToInstantiateView If the view could not be instantiated.
-     */
-    public function create(string $view, $type = null): View
-    {
-        $uri    = $this->scanLocations([$view]);
+	/**
+	 * 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 View Instance of the requested view.
+	 * @throws FailedToInstantiateView If the view could not be instantiated.
+	 */
+	public function create(string $view, $type = null): View
+	{
+		$uri    = $this->scanLocations([$view]);
 
-        $engine = $uri
-            ? $this->getEngine($uri)
-            : false;
+		$engine = $uri
+			? $this->getEngine($uri)
+			: false;
 
-        return ($uri && $engine)
-            ? $this->getView($uri, $engine, $type)
-            : $this->getViewFinder()->getNullObject();
-    }
+		return ($uri && $engine)
+			? $this->getView($uri, $engine, $type)
+			: $this->getViewFinder()->getNullObject();
+	}
 
-    /**
-     * Get an Engine that can deal with the given URI.
-     *
-     * @since 0.1.0
-     *
-     * @param string $uri URI to get an engine for.
-     *
-     * @return Engine Instance of an engine that can deal with the given URI.
-     */
-    public function getEngine(string $uri): Engine
-    {
-        return $this->getEngineFinder()->find([$uri]);
-    }
+	/**
+	 * Get an Engine that can deal with the given URI.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param string $uri URI to get an engine for.
+	 *
+	 * @return Engine Instance of an engine that can deal with the given URI.
+	 */
+	public function getEngine(string $uri): Engine
+	{
+		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 Engine $engine Engine to use for the view.
-     * @param mixed  $type   Type of view to get.
-     *
-     * @return View View that matches the given requirements.
-     * @throws FailedToInstantiateView If the view could not be instantiated.
-     */
-    public function getView(string $uri, Engine $engine, $type = null): View
-    {
-        if (null === $type) {
-            $view = $this->getViewFinder()->find([$uri], $engine);
-            return $view->setBuilder( $this );
-        }
+	/**
+	 * Get a view for a given URI, engine and type.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param string $uri    URI to get a view for.
+	 * @param Engine $engine Engine to use for the view.
+	 * @param mixed  $type   Type of view to get.
+	 *
+	 * @return View View that matches the given requirements.
+	 * @throws FailedToInstantiateView If the view could not be instantiated.
+	 */
+	public function getView(string $uri, Engine $engine, $type = null): View
+	{
+		if (null === $type) {
+			$view = $this->getViewFinder()->find([$uri], $engine);
+			return $view->setBuilder( $this );
+		}
 
-        return $this->resolveType($type, $uri, $engine);
-    }
+		return $this->resolveType($type, $uri, $engine);
+	}
 
-    /**
-     * Get the ViewFinder instance.
-     *
-     * @since 0.1.0
-     *
-     * @return ViewFinder Instance of a BaseViewFinder.
-     */
-    public function getViewFinder(): ViewFinder
-    {
-        return $this->viewFinder ?? $this->getFinder(static::VIEW_FINDER_KEY);
-    }
+	/**
+	 * Get the ViewFinder instance.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @return ViewFinder Instance of a BaseViewFinder.
+	 */
+	public function getViewFinder(): ViewFinder
+	{
+		return $this->viewFinder ?? $this->getFinder(static::VIEW_FINDER_KEY);
+	}
 
-    /**
-     * Get the EngineFinder instance.
-     *
-     * @since 0.1.0
-     *
-     * @return EngineFinder Instance of a BaseEngineFinder.
-     */
-    public function getEngineFinder(): EngineFinder
-    {
-        return $this->engineFinder ?? $this->getFinder(static::ENGINE_FINDER_KEY);
-    }
+	/**
+	 * Get the EngineFinder instance.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @return EngineFinder Instance of a BaseEngineFinder.
+	 */
+	public function getEngineFinder(): EngineFinder
+	{
+		return $this->engineFinder ?? $this->getFinder(static::ENGINE_FINDER_KEY);
+	}
 
-    /**
-     * Add a location to scan with the BaseViewFinder.
-     *
-     * @since 0.1.0
-     *
-     * @param Location $location Location to scan with the BaseViewFinder.
-     *
-     * @return static
-     */
-    public function addLocation(Location $location)
-    {
-        $this->locations->add($location);
+	/**
+	 * Add a location to scan with the BaseViewFinder.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param Location $location Location to scan with the BaseViewFinder.
+	 *
+	 * @return static
+	 */
+	public function addLocation(Location $location)
+	{
+		$this->locations->add($location);
 
-        return $this;
-    }
+		return $this;
+	}
 
-    /**
-     * Get the collection of locations registered with this ViewBuilder.
-     *
-     * @since 0.1.3
-     *
-     * @return Locations Collection of locations.
-     */
-    public function getLocations()
-    {
-        return $this->locations;
-    }
+	/**
+	 * Get the collection of locations registered with this ViewBuilder.
+	 *
+	 * @since 0.1.3
+	 *
+	 * @return Locations Collection of locations.
+	 */
+	public function getLocations()
+	{
+		return $this->locations;
+	}
 
-    /**
-     * 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)
-    {
-        $uris = $this->locations->map(function ($location) use ($criteria) {
-            /** @var Location $location */
-            return $location->getURI($criteria);
-        })->filter(function ($uri) {
-            return false !== $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)
+	{
+		$uris = $this->locations->map(function ($location) use ($criteria) {
+			/** @var Location $location */
+			return $location->getURI($criteria);
+		})->filter(function ($uri) {
+			return false !== $uri;
+		});
         
-        // Fall back for absolute paths on current filesystem.
-        if ($uris->isEmpty()) {
-            foreach ($criteria as $criterion) {
-                if (file_exists($criterion)) {
-                    return $criterion;
-                }
-            }
-        }
+		// Fall back for absolute paths on current filesystem.
+		if ($uris->isEmpty()) {
+			foreach ($criteria as $criterion) {
+				if (file_exists($criterion)) {
+					return $criterion;
+				}
+			}
+		}
 
-        return $uris->isEmpty() ? false : $uris->first();
-    }
+		return $uris->isEmpty() ? false : $uris->first();
+	}
 
-    /**
-     * Get a finder instance.
-     *
-     * @since 0.1.1
-     *
-     * @param string $key Configuration key to use.
-     *
-     * @return ViewFinder|EngineFinder The requested finder instance.
-     */
-    protected function getFinder($key)
-    {
-        $finderClass = $this->config->getKey($key, 'ClassName');
-        return new $finderClass($this->config->getSubConfig($key));
-    }
+	/**
+	 * Get a finder instance.
+	 *
+	 * @since 0.1.1
+	 *
+	 * @param string $key Configuration key to use.
+	 *
+	 * @return ViewFinder|EngineFinder The requested finder instance.
+	 */
+	protected function getFinder($key)
+	{
+		$finderClass = $this->config->getKey($key, 'ClassName');
+		return new $finderClass($this->config->getSubConfig($key));
+	}
 
-    /**
-     * 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 Engine|null $engine Engine to use for the view.
-     *
-     * @return View Resolved View object.
-     * @throws FailedToInstantiateView If the view type could not be resolved.
-     */
-    protected function resolveType($type, string $uri, Engine $engine = null): View
-    {
-        $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 Engine|null $engine Engine to use for the view.
+	 *
+	 * @return View Resolved View object.
+	 * @throws FailedToInstantiateView If the view type could not be resolved.
+	 */
+	protected function resolveType($type, string $uri, Engine $engine = null): View
+	{
+		$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, $this);
-        }
+		if (is_string($type) && $this->config->hasKey($configKey)) {
+			$className = $this->config->getKey($configKey);
+			$type      = new $className($uri, $engine, $this);
+		}
 
-        if (is_string($type)) {
-            $type = new $type($uri, $engine, $this);
-        }
+		if (is_string($type)) {
+			$type = new $type($uri, $engine, $this);
+		}
 
-        if (is_callable($type)) {
-            $type = $type($uri, $engine, $this);
-        }
+		if (is_callable($type)) {
+			$type = $type($uri, $engine, $this);
+		}
 
-        if (! $type instanceof View) {
-            throw new FailedToInstantiateView(
-                sprintf(
-                    _('Could not instantiate view "%s".'),
-                    serialize($type)
-                )
-            );
-        }
+		if (! $type instanceof View) {
+			throw new FailedToInstantiateView(
+				sprintf(
+					_('Could not instantiate view "%s".'),
+					serialize($type)
+				)
+			);
+		}
 
-        return $type;
-    }
+		return $type;
+	}
 
-    /**
-     * Get the configuration to use in the ViewBuilder.
-     *
-     * @since 0.2.0
-     *
-     * @param ConfigInterface|array $config Config to merge with defaults.
-     *
-     * @return ConfigInterface Configuration passed in through the constructor.
-     */
-    protected function getConfig($config = []): ConfigInterface
-    {
-        $defaults = ConfigFactory::create(dirname(__DIR__, 2) . '/config/defaults.php', $config);
-        $config   = $config
-            ? ConfigFactory::createFromArray(array_merge_recursive($defaults->getArrayCopy(), $config->getArrayCopy()))
-            : $defaults;
+	/**
+	 * Get the configuration to use in the ViewBuilder.
+	 *
+	 * @since 0.2.0
+	 *
+	 * @param ConfigInterface|array $config Config to merge with defaults.
+	 *
+	 * @return ConfigInterface Configuration passed in through the constructor.
+	 */
+	protected function getConfig($config = []): ConfigInterface
+	{
+		$defaults = ConfigFactory::create(dirname(__DIR__, 2) . '/config/defaults.php', $config);
+		$config   = $config
+			? ConfigFactory::createFromArray(array_merge_recursive($defaults->getArrayCopy(), $config->getArrayCopy()))
+			: $defaults;
 
-        return $config->getSubConfig('BrightNucleus\View');
-    }
+		return $config->getSubConfig('BrightNucleus\View');
+	}
 }

Spacing +5 added lines, -5 removed lines

@@ -141,7 +141,7 @@ 
     {
         if (null === $type) {
             $view = $this->getViewFinder()->find([$uri], $engine);
-            return $view->setBuilder( $this );
+            return $view->setBuilder($this);
         }
 
         return $this->resolveType($type, $uri, $engine);
@@ -210,10 +210,10 @@ 
      */
     public function scanLocations(array $criteria)
     {
-        $uris = $this->locations->map(function ($location) use ($criteria) {
+        $uris = $this->locations->map(function($location) use ($criteria) {
             /** @var Location $location */
             return $location->getURI($criteria);
-        })->filter(function ($uri) {
+        })->filter(function($uri) {
             return false !== $uri;
         });
         
@@ -273,7 +273,7 @@ 
             $type = $type($uri, $engine, $this);
         }
 
-        if (! $type instanceof View) {
+        if ( ! $type instanceof View) {
             throw new FailedToInstantiateView(
                 sprintf(
                     _('Could not instantiate view "%s".'),
@@ -296,7 +296,7 @@ 
      */
     protected function getConfig($config = []): ConfigInterface
     {
-        $defaults = ConfigFactory::create(dirname(__DIR__, 2) . '/config/defaults.php', $config);
+        $defaults = ConfigFactory::create(dirname(__DIR__, 2).'/config/defaults.php', $config);
         $config   = $config
             ? ConfigFactory::createFromArray(array_merge_recursive($defaults->getArrayCopy(), $config->getArrayCopy()))
             : $defaults;

src/View/View/ViewFinder.php

Indentation +23 added lines, -23 removed lines

@@ -26,29 +26,29 @@
 interface ViewFinder extends Finder
 {
 
-    // Constants to be used for the Config file sections.
-    const CLASS_NAME_KEY = 'ClassName';
-    const VIEWS_KEY = 'Views';
-    const NULL_OBJECT = 'NullObject';
+	// Constants to be used for the Config file sections.
+	const CLASS_NAME_KEY = 'ClassName';
+	const VIEWS_KEY = 'Views';
+	const NULL_OBJECT = 'NullObject';
 
-    /**
-     * Find a result based on a specific criteria.
-     *
-     * @since 0.1.0
-     *
-     * @param array       $criteria Criteria to search for.
-     * @param Engine|null $engine   Optional. Engine to use with the view.
-     *
-     * @return View View that was found.
-     */
-    public function find(array $criteria, Engine $engine = null): View;
+	/**
+	 * Find a result based on a specific criteria.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param array       $criteria Criteria to search for.
+	 * @param Engine|null $engine   Optional. Engine to use with the view.
+	 *
+	 * @return View View that was found.
+	 */
+	public function find(array $criteria, Engine $engine = null): View;
 
-    /**
-     * Get the NullObject.
-     *
-     * @since 0.1.1
-     *
-     * @return NullView NullObject for the current Finder.
-     */
-    public function getNullObject(): NullView;
+	/**
+	 * Get the NullObject.
+	 *
+	 * @since 0.1.1
+	 *
+	 * @return NullView NullObject for the current Finder.
+	 */
+	public function getNullObject(): NullView;
 }

src/View/View/BaseViewFinder.php

Indentation +47 added lines, -47 removed lines

@@ -27,56 +27,56 @@
 class BaseViewFinder extends AbstractFinder implements ViewFinder
 {
 
-    /**
-     * Find a result based on a specific criteria.
-     *
-     * @since 0.1.0
-     *
-     * @param array       $criteria Criteria to search for.
-     * @param Engine|null $engine   Optional. Engine to use with the view.
-     *
-     * @return View View that was found.
-     * @throws FailedToInstantiateFindable If the Findable could not be instantiated.
-     */
-    public function find(array $criteria, Engine $engine = null): View
-    {
-        $uri = $criteria[0];
+	/**
+	 * Find a result based on a specific criteria.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param array       $criteria Criteria to search for.
+	 * @param Engine|null $engine   Optional. Engine to use with the view.
+	 *
+	 * @return View View that was found.
+	 * @throws FailedToInstantiateFindable If the Findable could not be instantiated.
+	 */
+	public function find(array $criteria, Engine $engine = null): View
+	{
+		$uri = $criteria[0];
 
-        $views = $this->initializeFindables([$uri, $engine]);
+		$views = $this->initializeFindables([$uri, $engine]);
 
-        foreach ($criteria as $entry) {
-            foreach ($views as $viewObject) {
-                if ($viewObject->canHandle($entry)) {
-                    return $viewObject;
-                }
-            }
-        }
+		foreach ($criteria as $entry) {
+			foreach ($views as $viewObject) {
+				if ($viewObject->canHandle($entry)) {
+					return $viewObject;
+				}
+			}
+		}
 
-        return $this->getNullObject();
-    }
+		return $this->getNullObject();
+	}
 
-    /**
-     * Get the config key for the Findables definitions.
-     *
-     * @since 0.1.0
-     *
-     * @return string Config key use to define the Findables.
-     */
-    protected function getFindablesConfigKey(): string
-    {
-        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(): string
+	{
+		return 'Views';
+	}
 
-    /**
-     * Get the NullObject.
-     *
-     * @since 0.1.1
-     *
-     * @return NullView NullObject for the current Finder.
-     * @throws FailedToInstantiateFindable If the Findable could not be instantiated.
-     */
-    public function getNullObject(): NullView
-    {
-        return parent::getNullObject();
-    }
+	/**
+	 * Get the NullObject.
+	 *
+	 * @since 0.1.1
+	 *
+	 * @return NullView NullObject for the current Finder.
+	 * @throws FailedToInstantiateFindable If the Findable could not be instantiated.
+	 */
+	public function getNullObject(): NullView
+	{
+		return parent::getNullObject();
+	}
 }

src/View/View/AbstractView.php

Indentation +177 added lines, -177 removed lines

@@ -30,181 +30,181 @@
 abstract class AbstractView implements View
 {
 
-    /**
-     * URI of the view.
-     *
-     * The underscores are used to prevent accidental use of these properties from within the rendering closure.
-     *
-     * @since 0.1.0
-     *
-     * @var string
-     */
-    protected $_uri_;
-
-    /**
-     * Engine to use for the view.
-     *
-     * The underscores are used to prevent accidental use of these properties from within the rendering closure.
-     *
-     * @since 0.1.0
-     *
-     * @var Engine
-     */
-    protected $_engine_;
-
-    /**
-     * ViewBuilder instance.
-     *
-     * The underscores are used to prevent accidental use of these properties from within the rendering closure.
-     *
-     * @since 0.2.0
-     *
-     * @var ViewBuilder
-     */
-    protected $_builder_;
-
-    /**
-     * The context with which the view will be rendered.
-     *
-     * The underscores are used to prevent accidental use of these properties from within the rendering closure.
-     *
-     * @since 0.4.0
-     *
-     * @var array
-     */
-    protected $_context_ = [];
-
-    /**
-     * Instantiate an AbstractView object.
-     *
-     * @since 0.1.0
-     *
-     * @param string      $uri         URI for the view.
-     * @param Engine      $engine      Engine to use for the view.
-     * @param ViewBuilder $viewBuilder View builder instance to use.
-     */
-    public function __construct(string $uri, Engine $engine, ViewBuilder $viewBuilder = null)
-    {
-        $this->_uri_     = $uri;
-        $this->_engine_  = $engine;
-        $this->_builder_ = $viewBuilder ?? Views::getViewBuilder();
-    }
-
-    /**
-     * 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 Rendered HTML.
-     * @throws FailedToProcessConfigException If the Config could not be processed.
-     */
-    public function render(array $context = [], bool $echo = false): string
-    {
-        $this->assimilateContext($context);
-
-        $closure = Closure::bind(
-            $this->_engine_->getRenderCallback($this->_uri_, $context),
-            $this,
-            static::class
-        );
-
-        $output = $closure();
-
-        if ($echo) {
-            echo $output;
-        }
-
-        return $output;
-    }
-
-    /**
-     * Render a partial view (or section) for a given URI.
-     *
-     * @since 0.2.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.
-     * @throws FailedToProcessConfigException If the Config could not be processed.
-     * @throws FailedToInstantiateView If the View could not be instantiated.
-     */
-    public function section(string $view, array $context = null, $type = null): string
-    {
-        if (null === $context) {
-            $context = $this->_context_;
-        }
-
-        $viewObject = $this->_builder_->create($view, $type);
-
-        return $viewObject->render($context);
-    }
-
-    /**
-     * Get the entire array of contextual data.
-     *
-     * @since 0.4.0
-     *
-     * @return array Array of contextual data.
-     */
-    public function getContext(): array
-    {
-        return $this->_context_;
-    }
-
-    /**
-     * Associate a view builder with this view.
-     *
-     * @since 0.2.0
-     *
-     * @param ViewBuilder $builder
-     *
-     * @return View
-     */
-    public function setBuilder(ViewBuilder $builder): View
-    {
-        $this->_builder_ = $builder;
-
-        return $this;
-    }
-
-    /**
-     * Assimilate the context to make it available as properties.
-     *
-     * @since 0.2.0
-     *
-     * @param array $context Context to assimilate.
-     */
-    protected function assimilateContext(array $context = [])
-    {
-        $this->_context_ = $context;
-        foreach ($context as $key => $value) {
-            $this->$key = $value;
-        }
-    }
-
-    /**
-     * Turn invokable objects as properties into methods of the view.
-     *
-     * @param string $method    Method that was called on the view.
-     * @param array  $arguments Array of arguments that were used.
-     * @return mixed Return value of the invokable object.
-     */
-    public function __call($method, $arguments) {
-        if ( ! property_exists($this, $method)
-             || ! is_callable($this->$method)) {
-            trigger_error(
-                "Call to undefined method {$method} on a view.",
-                E_USER_ERROR
-            );
-        }
-
-        $callable = $this->$method;
-
-        return $callable(...$arguments);
-    }
+	/**
+	 * URI of the view.
+	 *
+	 * The underscores are used to prevent accidental use of these properties from within the rendering closure.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @var string
+	 */
+	protected $_uri_;
+
+	/**
+	 * Engine to use for the view.
+	 *
+	 * The underscores are used to prevent accidental use of these properties from within the rendering closure.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @var Engine
+	 */
+	protected $_engine_;
+
+	/**
+	 * ViewBuilder instance.
+	 *
+	 * The underscores are used to prevent accidental use of these properties from within the rendering closure.
+	 *
+	 * @since 0.2.0
+	 *
+	 * @var ViewBuilder
+	 */
+	protected $_builder_;
+
+	/**
+	 * The context with which the view will be rendered.
+	 *
+	 * The underscores are used to prevent accidental use of these properties from within the rendering closure.
+	 *
+	 * @since 0.4.0
+	 *
+	 * @var array
+	 */
+	protected $_context_ = [];
+
+	/**
+	 * Instantiate an AbstractView object.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param string      $uri         URI for the view.
+	 * @param Engine      $engine      Engine to use for the view.
+	 * @param ViewBuilder $viewBuilder View builder instance to use.
+	 */
+	public function __construct(string $uri, Engine $engine, ViewBuilder $viewBuilder = null)
+	{
+		$this->_uri_     = $uri;
+		$this->_engine_  = $engine;
+		$this->_builder_ = $viewBuilder ?? Views::getViewBuilder();
+	}
+
+	/**
+	 * 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 Rendered HTML.
+	 * @throws FailedToProcessConfigException If the Config could not be processed.
+	 */
+	public function render(array $context = [], bool $echo = false): string
+	{
+		$this->assimilateContext($context);
+
+		$closure = Closure::bind(
+			$this->_engine_->getRenderCallback($this->_uri_, $context),
+			$this,
+			static::class
+		);
+
+		$output = $closure();
+
+		if ($echo) {
+			echo $output;
+		}
+
+		return $output;
+	}
+
+	/**
+	 * Render a partial view (or section) for a given URI.
+	 *
+	 * @since 0.2.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.
+	 * @throws FailedToProcessConfigException If the Config could not be processed.
+	 * @throws FailedToInstantiateView If the View could not be instantiated.
+	 */
+	public function section(string $view, array $context = null, $type = null): string
+	{
+		if (null === $context) {
+			$context = $this->_context_;
+		}
+
+		$viewObject = $this->_builder_->create($view, $type);
+
+		return $viewObject->render($context);
+	}
+
+	/**
+	 * Get the entire array of contextual data.
+	 *
+	 * @since 0.4.0
+	 *
+	 * @return array Array of contextual data.
+	 */
+	public function getContext(): array
+	{
+		return $this->_context_;
+	}
+
+	/**
+	 * Associate a view builder with this view.
+	 *
+	 * @since 0.2.0
+	 *
+	 * @param ViewBuilder $builder
+	 *
+	 * @return View
+	 */
+	public function setBuilder(ViewBuilder $builder): View
+	{
+		$this->_builder_ = $builder;
+
+		return $this;
+	}
+
+	/**
+	 * Assimilate the context to make it available as properties.
+	 *
+	 * @since 0.2.0
+	 *
+	 * @param array $context Context to assimilate.
+	 */
+	protected function assimilateContext(array $context = [])
+	{
+		$this->_context_ = $context;
+		foreach ($context as $key => $value) {
+			$this->$key = $value;
+		}
+	}
+
+	/**
+	 * Turn invokable objects as properties into methods of the view.
+	 *
+	 * @param string $method    Method that was called on the view.
+	 * @param array  $arguments Array of arguments that were used.
+	 * @return mixed Return value of the invokable object.
+	 */
+	public function __call($method, $arguments) {
+		if ( ! property_exists($this, $method)
+			 || ! is_callable($this->$method)) {
+			trigger_error(
+				"Call to undefined method {$method} on a view.",
+				E_USER_ERROR
+			);
+		}
+
+		$callable = $this->$method;
+
+		return $callable(...$arguments);
+	}
 }

src/View/Location/FilesystemLocation.php

Indentation +154 added lines, -154 removed lines

@@ -28,158 +28,158 @@
 class FilesystemLocation implements Location
 {
 
-    /**
-     * Path that this location points to.
-     *
-     * @since 0.1.0
-     *
-     * @var string
-     */
-    protected $path;
-
-    /**
-     * Extensions that this location can accept.
-     *
-     * @since 0.1.0
-     *
-     * @var Extensions
-     */
-    protected $extensions;
-
-    /**
-     * Instantiate a FilesystemLocation object.
-     *
-     * @since 0.1.0
-     *
-     * @param string                       $path       Path that this location points to.
-     * @param Extensions|array|string|null $extensions Optional. Extensions that this location can accept.
-     */
-    public function __construct(string $path, $extensions = null)
-    {
-        $this->path       = $path;
-        $this->extensions = $this->validateExtensions($extensions);
-    }
-
-    /**
-     * Get the first URI that matches the given criteria.
-     *
-     * @since 0.1.0
-     *
-     * @param array $criteria Criteria to match.
-     *
-     * @return string|false URI that matches the criteria or false if none found.
-     */
-    public function getURI(array $criteria)
-    {
-        $uris = $this->getURIs($criteria);
-
-        return $uris->count() > 0
-            ? $uris->first()
-            : false;
-    }
-
-    /**
-     * Get all URIs that match the given criteria.
-     *
-     * @since 0.1.1
-     *
-     * @param array $criteria Criteria to match.
-     *
-     * @return URIs URIs that match the criteria or an empty collection if none found.
-     */
-    public function getURIs(array $criteria): URIs
-    {
-        $uris = new URIs();
-
-        foreach ($this->extensions as $extension) {
-            $finder = new Finder();
-
-            try {
-                $finder->files()
-                    ->name($this->getNamePattern($criteria, $extension))
-                    ->in($this->getPathPattern());
-                foreach ($finder as $file) {
-                    /** @var SplFileInfo $file */
-                    $uris->add($file->getPathname());
-                }
-            } catch (Exception $exception) {
-                // Fail silently;
-            }
-        }
-
-        return $uris;
-    }
-
-    /**
-     * Get the name pattern to pass to the file finder.
-     *
-     * @since 0.1.3
-     *
-     * @param array  $criteria  Criteria to match.
-     * @param string $extension Extension to match.
-     *
-     * @return string Name pattern to pass to the file finder.
-     */
-    protected function getNamePattern(array $criteria, string $extension): string
-    {
-        $names = [];
-
-        $names[] = array_map(function ($criterion) use ($extension) {
-            $uriExtension = URIHelper::containsExtension($criterion);
-            if (! empty($extension)) {
-                $extension = ltrim($extension, '.');
-
-                if ($uriExtension === $extension) {
-                    $criterion = substr($criterion,0,-strlen(".{$extension}"));
-                }
-            } else {
-                $extension = URIHelper::containsExtension($criterion);
-                if (!empty($extension)) {
-                    $criterion = substr($criterion,0,-strlen(".{$extension}"));
-                }
-            }
-
-            $criterion = preg_quote(URIHelper::getFilename($criterion), chr(1));
-
-            return (empty($extension) || URIHelper::hasExtension($criterion, $extension))
-                ? "{$criterion}(?:\..*?)$"
-                : "{$criterion}\.{$extension}$";
-        }, $criteria)[0];
-
-        return chr(1) . implode('|', array_unique($names)) . chr(1);
-    }
-
-    /**
-     * Get the path pattern to pass to the file finder.
-     *
-     * @since 0.1.3
-     *
-     * @return string Path pattern to pass to the file finder.
-     */
-    protected function getPathPattern(): string
-    {
-        return $this->path;
-    }
-
-    /**
-     * Validate the extensions and return a collection.
-     *
-     * @since 0.1.1
-     *
-     * @param Extensions|array|string|null $extensions Extensions to validate.
-     *
-     * @return Extensions Validated extensions collection.
-     */
-    protected function validateExtensions($extensions): Extensions
-    {
-        if (empty($extensions)) {
-            $extensions = new Extensions(['']);
-        }
-
-        if (! $extensions instanceof Extensions) {
-            $extensions = new Extensions((array)$extensions);
-        }
-
-        return $extensions;
-    }
+	/**
+	 * Path that this location points to.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @var string
+	 */
+	protected $path;
+
+	/**
+	 * Extensions that this location can accept.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @var Extensions
+	 */
+	protected $extensions;
+
+	/**
+	 * Instantiate a FilesystemLocation object.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param string                       $path       Path that this location points to.
+	 * @param Extensions|array|string|null $extensions Optional. Extensions that this location can accept.
+	 */
+	public function __construct(string $path, $extensions = null)
+	{
+		$this->path       = $path;
+		$this->extensions = $this->validateExtensions($extensions);
+	}
+
+	/**
+	 * Get the first URI that matches the given criteria.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param array $criteria Criteria to match.
+	 *
+	 * @return string|false URI that matches the criteria or false if none found.
+	 */
+	public function getURI(array $criteria)
+	{
+		$uris = $this->getURIs($criteria);
+
+		return $uris->count() > 0
+			? $uris->first()
+			: false;
+	}
+
+	/**
+	 * Get all URIs that match the given criteria.
+	 *
+	 * @since 0.1.1
+	 *
+	 * @param array $criteria Criteria to match.
+	 *
+	 * @return URIs URIs that match the criteria or an empty collection if none found.
+	 */
+	public function getURIs(array $criteria): URIs
+	{
+		$uris = new URIs();
+
+		foreach ($this->extensions as $extension) {
+			$finder = new Finder();
+
+			try {
+				$finder->files()
+					->name($this->getNamePattern($criteria, $extension))
+					->in($this->getPathPattern());
+				foreach ($finder as $file) {
+					/** @var SplFileInfo $file */
+					$uris->add($file->getPathname());
+				}
+			} catch (Exception $exception) {
+				// Fail silently;
+			}
+		}
+
+		return $uris;
+	}
+
+	/**
+	 * Get the name pattern to pass to the file finder.
+	 *
+	 * @since 0.1.3
+	 *
+	 * @param array  $criteria  Criteria to match.
+	 * @param string $extension Extension to match.
+	 *
+	 * @return string Name pattern to pass to the file finder.
+	 */
+	protected function getNamePattern(array $criteria, string $extension): string
+	{
+		$names = [];
+
+		$names[] = array_map(function ($criterion) use ($extension) {
+			$uriExtension = URIHelper::containsExtension($criterion);
+			if (! empty($extension)) {
+				$extension = ltrim($extension, '.');
+
+				if ($uriExtension === $extension) {
+					$criterion = substr($criterion,0,-strlen(".{$extension}"));
+				}
+			} else {
+				$extension = URIHelper::containsExtension($criterion);
+				if (!empty($extension)) {
+					$criterion = substr($criterion,0,-strlen(".{$extension}"));
+				}
+			}
+
+			$criterion = preg_quote(URIHelper::getFilename($criterion), chr(1));
+
+			return (empty($extension) || URIHelper::hasExtension($criterion, $extension))
+				? "{$criterion}(?:\..*?)$"
+				: "{$criterion}\.{$extension}$";
+		}, $criteria)[0];
+
+		return chr(1) . implode('|', array_unique($names)) . chr(1);
+	}
+
+	/**
+	 * Get the path pattern to pass to the file finder.
+	 *
+	 * @since 0.1.3
+	 *
+	 * @return string Path pattern to pass to the file finder.
+	 */
+	protected function getPathPattern(): string
+	{
+		return $this->path;
+	}
+
+	/**
+	 * Validate the extensions and return a collection.
+	 *
+	 * @since 0.1.1
+	 *
+	 * @param Extensions|array|string|null $extensions Extensions to validate.
+	 *
+	 * @return Extensions Validated extensions collection.
+	 */
+	protected function validateExtensions($extensions): Extensions
+	{
+		if (empty($extensions)) {
+			$extensions = new Extensions(['']);
+		}
+
+		if (! $extensions instanceof Extensions) {
+			$extensions = new Extensions((array)$extensions);
+		}
+
+		return $extensions;
+	}
 }

Spacing +8 added lines, -8 removed lines

@@ -124,18 +124,18 @@ 
     {
         $names = [];
 
-        $names[] = array_map(function ($criterion) use ($extension) {
+        $names[] = array_map(function($criterion) use ($extension) {
             $uriExtension = URIHelper::containsExtension($criterion);
-            if (! empty($extension)) {
+            if ( ! empty($extension)) {
                 $extension = ltrim($extension, '.');
 
                 if ($uriExtension === $extension) {
-                    $criterion = substr($criterion,0,-strlen(".{$extension}"));
+                    $criterion = substr($criterion, 0, -strlen(".{$extension}"));
                 }
             } else {
                 $extension = URIHelper::containsExtension($criterion);
-                if (!empty($extension)) {
-                    $criterion = substr($criterion,0,-strlen(".{$extension}"));
+                if ( ! empty($extension)) {
+                    $criterion = substr($criterion, 0, -strlen(".{$extension}"));
                 }
             }
 
@@ -146,7 +146,7 @@ 
                 : "{$criterion}\.{$extension}$";
         }, $criteria)[0];
 
-        return chr(1) . implode('|', array_unique($names)) . chr(1);
+        return chr(1).implode('|', array_unique($names)).chr(1);
     }
 
     /**
@@ -176,8 +176,8 @@ 
             $extensions = new Extensions(['']);
         }
 
-        if (! $extensions instanceof Extensions) {
-            $extensions = new Extensions((array)$extensions);
+        if ( ! $extensions instanceof Extensions) {
+            $extensions = new Extensions((array) $extensions);
         }
 
         return $extensions;

src/View/Engine/EngineFinder.php

Indentation +12 added lines, -12 removed lines

@@ -23,17 +23,17 @@
  */
 interface EngineFinder extends Finder
 {
-    // Constants to be used for the Config file sections.
-    const CLASS_NAME_KEY = 'ClassName';
-    const ENGINES_KEY = 'Engines';
-    const NULL_OBJECT = 'NullObject';
+	// Constants to be used for the Config file sections.
+	const CLASS_NAME_KEY = 'ClassName';
+	const ENGINES_KEY = 'Engines';
+	const NULL_OBJECT = 'NullObject';
 
-    /**
-     * Get the NullObject.
-     *
-     * @since 0.1.1
-     *
-     * @return NullEngine NullObject for the current Finder.
-     */
-    public function getNullObject(): NullEngine;
+	/**
+	 * Get the NullObject.
+	 *
+	 * @since 0.1.1
+	 *
+	 * @return NullEngine NullObject for the current Finder.
+	 */
+	public function getNullObject(): NullEngine;
 }

src/View/Engine/BaseEngineFinder.php

Indentation +45 added lines, -45 removed lines

@@ -25,53 +25,53 @@
 class BaseEngineFinder extends AbstractFinder implements EngineFinder
 {
 
-    /**
-     * Find a result based on a specific criteria.
-     *
-     * @since 0.1.0
-     *
-     * @param array $criteria Criteria to search for.
-     *
-     * @return Engine Result of the search.
-     * @throws FailedToInstantiateFindable If the Findable could not be instantiated.
-     */
-    public function find(array $criteria): Engine
-    {
-        $this->findables = $this->initializeFindables();
+	/**
+	 * Find a result based on a specific criteria.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param array $criteria Criteria to search for.
+	 *
+	 * @return Engine Result of the search.
+	 * @throws FailedToInstantiateFindable If the Findable could not be instantiated.
+	 */
+	public function find(array $criteria): Engine
+	{
+		$this->findables = $this->initializeFindables();
 
-        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();
+	}
 
-    /**
-     * Get the config key for the Findables definitions.
-     *
-     * @since 0.1.0
-     *
-     * @return string Config key use to define the Findables.
-     */
-    protected function getFindablesConfigKey(): string
-    {
-        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(): string
+	{
+		return 'Engines';
+	}
 
-    /**
-     * Get the NullObject.
-     *
-     * @since 0.1.1
-     *
-     * @return NullEngine NullObject for the current Finder.
-     * @throws FailedToInstantiateFindable If the Findable could not be instantiated.
-     */
-    public function getNullObject(): NullEngine
-    {
-        return parent::getNullObject();
-    }
+	/**
+	 * Get the NullObject.
+	 *
+	 * @since 0.1.1
+	 *
+	 * @return NullEngine NullObject for the current Finder.
+	 * @throws FailedToInstantiateFindable If the Findable could not be instantiated.
+	 */
+	public function getNullObject(): NullEngine
+	{
+		return parent::getNullObject();
+	}
 }

src/View/Support/Finder.php

Indentation +18 added lines, -18 removed lines

@@ -22,23 +22,23 @@
 interface Finder
 {
 
-    /**
-     * Find a result based on a specific criteria.
-     *
-     * @since 0.1.0
-     *
-     * @param array $criteria Criteria to search for.
-     *
-     * @return mixed Result of the search.
-     */
-    public function find(array $criteria);
+	/**
+	 * Find a result based on a specific criteria.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param array $criteria Criteria to search for.
+	 *
+	 * @return mixed Result of the search.
+	 */
+	public function find(array $criteria);
 
-    /**
-     * Get the NullObject.
-     *
-     * @since 0.1.1
-     *
-     * @return NullFindable NullObject for the current Finder.
-     */
-    public function getNullObject();
+	/**
+	 * Get the NullObject.
+	 *
+	 * @since 0.1.1
+	 *
+	 * @return NullFindable NullObject for the current Finder.
+	 */
+	public function getNullObject();
 }