brightnucleus / view

src/View/ViewBuilder.php

Doc Comments +3 added lines, -3 removed lines

@@ -124,7 +124,7 @@ 
      *
      * @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.
+     * @param string|null           $type   Type of view to get.
      *
      * @return ViewInterface View that matches the given requirements.
      */
@@ -188,9 +188,9 @@ 
      *
      * @since 0.1.0
      *
-     * @param array $criteria Criteria to match.
+     * @param string[] $criteria Criteria to match.
      *
-     * @return string|bool URI of the requested view, or false if not found.
+     * @return string|false URI of the requested view, or false if not found.
      */
     public function scanLocations(array $criteria)
     {

Indentation +205 added lines, -205 removed lines

@@ -32,209 +32,209 @@
 class ViewBuilder
 {
 
-    use ConfigTrait;
-
-    const ENGINE_FINDER_KEY = 'EngineFinder';
-    const VIEW_FINDER_KEY   = 'ViewFinder';
-
-    /**
-     * ViewFinder instance.
-     *
-     * @since 0.1.0
-     *
-     * @var ViewFinderInterface
-     */
-    protected $viewFinder;
-
-    /**
-     * EngineFinder instance.
-     *
-     * @since 0.1.0
-     *
-     * @var EngineFinderInterface
-     */
-    protected $engineFinder;
-
-    /**
-     * 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;
-    }
-
-    /**
-     * 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 function create($view, $type = null)
-    {
-        $uri    = $this->scanLocations([$view]);
-        $engine = $this->getEngine($uri);
-
-        return $this->getView($uri, $engine, $type);
-    }
-
-    /**
-     * 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);
-        }
-
-        return $this->resolveType($type, $uri, $engine);
-    }
-
-    /**
-     * Get the ViewFinder instance.
-     *
-     * @since 0.1.0
-     *
-     * @return ViewFinderInterface Instance of a ViewFinder.
-     */
-    public function getViewFinder()
-    {
-        if (null === $this->viewFinder) {
-            $viewFinderClass  = $this->config->getKey(ViewBuilder::VIEW_FINDER_KEY, 'ClassName');
-            $this->viewFinder = new $viewFinderClass($this->config->getSubConfig(ViewBuilder::VIEW_FINDER_KEY));
-        }
-
-        return $this->viewFinder;
-    }
-
-    /**
-     * Get the EngineFinder instance.
-     *
-     * @since 0.1.0
-     *
-     * @return EngineFinderInterface Instance of a EngineFinder.
-     */
-    public function getEngineFinder()
-    {
-        if (null === $this->engineFinder) {
-            $engineFinderClass  = $this->config->getKey(ViewBuilder::ENGINE_FINDER_KEY, 'ClassName');
-            $this->engineFinder = new $engineFinderClass($this->config->getSubConfig(ViewBuilder::ENGINE_FINDER_KEY));
-        }
-
-        return $this->engineFinder;
-    }
-
-    /**
-     * 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|bool 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;
-    }
-
-    /**
-     * 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 $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)
-    {
-        if (is_string($type) && $this->config->hasKey(ViewBuilder::VIEW_FINDER_KEY)) {
-            $type = new $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)
-                )
-            );
-        }
-
-        return $type;
-    }
+	use ConfigTrait;
+
+	const ENGINE_FINDER_KEY = 'EngineFinder';
+	const VIEW_FINDER_KEY   = 'ViewFinder';
+
+	/**
+	 * ViewFinder instance.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @var ViewFinderInterface
+	 */
+	protected $viewFinder;
+
+	/**
+	 * EngineFinder instance.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @var EngineFinderInterface
+	 */
+	protected $engineFinder;
+
+	/**
+	 * 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;
+	}
+
+	/**
+	 * 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 function create($view, $type = null)
+	{
+		$uri    = $this->scanLocations([$view]);
+		$engine = $this->getEngine($uri);
+
+		return $this->getView($uri, $engine, $type);
+	}
+
+	/**
+	 * 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);
+		}
+
+		return $this->resolveType($type, $uri, $engine);
+	}
+
+	/**
+	 * Get the ViewFinder instance.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @return ViewFinderInterface Instance of a ViewFinder.
+	 */
+	public function getViewFinder()
+	{
+		if (null === $this->viewFinder) {
+			$viewFinderClass  = $this->config->getKey(ViewBuilder::VIEW_FINDER_KEY, 'ClassName');
+			$this->viewFinder = new $viewFinderClass($this->config->getSubConfig(ViewBuilder::VIEW_FINDER_KEY));
+		}
+
+		return $this->viewFinder;
+	}
+
+	/**
+	 * Get the EngineFinder instance.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @return EngineFinderInterface Instance of a EngineFinder.
+	 */
+	public function getEngineFinder()
+	{
+		if (null === $this->engineFinder) {
+			$engineFinderClass  = $this->config->getKey(ViewBuilder::ENGINE_FINDER_KEY, 'ClassName');
+			$this->engineFinder = new $engineFinderClass($this->config->getSubConfig(ViewBuilder::ENGINE_FINDER_KEY));
+		}
+
+		return $this->engineFinder;
+	}
+
+	/**
+	 * 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|bool 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;
+	}
+
+	/**
+	 * 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 $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)
+	{
+		if (is_string($type) && $this->config->hasKey(ViewBuilder::VIEW_FINDER_KEY)) {
+			$type = new $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)
+				)
+			);
+		}
+
+		return $type;
+	}
 }

src/View/Engine/PHPEngine.php

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

Indentation +73 added lines, -73 removed lines

@@ -25,86 +25,86 @@
 class EngineFinder extends AbstractFinder
 {
 
-    /**
-     * 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)
-    {
-        $this->initializeEngines();
+	/**
+	 * 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)
+	{
+		$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->nullObject;
-    }
+		return $this->nullObject;
+	}
 
-    /**
-     * 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);
+		}
 
-        $this->nullObject = $this->initializeEngine($this->nullObject);
-    }
+		$this->nullObject = $this->initializeEngine($this->nullObject);
+	}
 
-    /**
-     * 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/Engine/EngineInterface.php

Indentation +11 added lines, -11 removed lines

@@ -22,15 +22,15 @@
 interface EngineInterface
 {
 
-    /**
-     * 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.
-     */
-    public function render($uri, array $context = []);
+	/**
+	 * 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.
+	 */
+	public function render($uri, array $context = []);
 }

src/View/View/BaseView.php

Indentation +13 added lines, -13 removed lines

@@ -22,17 +22,17 @@
 class BaseView extends AbstractView
 {
 
-    /**
-     * 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 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)
+	{
+		return true;
+	}
 }

src/View/View/ViewFinder.php

Indentation +123 added lines, -123 removed lines

@@ -26,127 +26,127 @@
 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 $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);
-
-        foreach ($criteria as $entry) {
-            foreach ($this->findables as $viewObject) {
-                if ($viewObject->canHandle($entry)) {
-                    return $viewObject;
-                }
-            }
-        }
-
-        return $this->nullObject;
-    }
-
-    /**
-     * Initialize the views that can be iterated.
-     *
-     * @since 0.1.0
-     *
-     * @param string          $uri    URI to use for the view.
-     * @param EngineInterface $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);
-        }
-
-        $this->nullObject = $this->initializeView($this->nullObject, $uri);
-    }
-
-    /**
-     * 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 $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 (! $view instanceof ViewInterface) {
-            throw new FailedToInstantiateViewException(
-                sprintf(
-                    _('Could not instantiate view "%s".'),
-                    serialize($view)
-                )
-            );
-        }
-
-        return $view;
-    }
-
-    /**
-     * Instantiate a 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 $engine Optional. View to use with the view.
-     *
-     * @return ViewInterface Instantiated view.
-     * @throws FailedToInstantiateViewException If the view could not be instantiated.
-     */
-    protected function instantiateView($view, $uri, EngineInterface $engine = null)
-    {
-        if (is_string($view)) {
-            $view = new $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)
-                )
-            );
-        }
-
-        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';
-    }
+	/**
+	 * Find a result based on a specific criteria.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param array           $criteria Criteria to search for.
+	 * @param EngineInterface $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);
+
+		foreach ($criteria as $entry) {
+			foreach ($this->findables as $viewObject) {
+				if ($viewObject->canHandle($entry)) {
+					return $viewObject;
+				}
+			}
+		}
+
+		return $this->nullObject;
+	}
+
+	/**
+	 * Initialize the views that can be iterated.
+	 *
+	 * @since 0.1.0
+	 *
+	 * @param string          $uri    URI to use for the view.
+	 * @param EngineInterface $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);
+		}
+
+		$this->nullObject = $this->initializeView($this->nullObject, $uri);
+	}
+
+	/**
+	 * 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 $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 (! $view instanceof ViewInterface) {
+			throw new FailedToInstantiateViewException(
+				sprintf(
+					_('Could not instantiate view "%s".'),
+					serialize($view)
+				)
+			);
+		}
+
+		return $view;
+	}
+
+	/**
+	 * Instantiate a 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 $engine Optional. View to use with the view.
+	 *
+	 * @return ViewInterface Instantiated view.
+	 * @throws FailedToInstantiateViewException If the view could not be instantiated.
+	 */
+	protected function instantiateView($view, $uri, EngineInterface $engine = null)
+	{
+		if (is_string($view)) {
+			$view = new $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)
+				)
+			);
+		}
+
+		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';
+	}
 }

Spacing +2 added lines, -2 removed lines

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

src/View/Support/Findable.php

Indentation +10 added lines, -10 removed lines

@@ -22,14 +22,14 @@
 interface Findable
 {
 
-    /**
-     * 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);
+	/**
+	 * 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);
 }