ingenerator / kohana-view

classes/Kohana/View/Model.php

Doc Comments +1 added lines, -1 removed lines

@@ -179,7 +179,7 @@
 	 *     $view->set_filename($file);
 	 *
 	 * @param   string  view filename
-	 * @return  View
+	 * @return  Kohana_View_Model
 	 * @throws  Kohana_View_Exception
 	 */
 	public function set_filename($file)

Braces +2 added lines, -4 removed lines

@@ -34,8 +34,7 @@ 
 		try
 		{
 			include 'kohana.view://'.$kohana_view_filename;
-		}
-		catch (Exception $e)
+		} catch (Exception $e)
 		{
 			// Delete the output buffer
 			ob_end_clean();
@@ -59,8 +58,7 @@ 
 		try
 		{
 			return $this->render();
-		}
-		catch (Exception $e)
+		} catch (Exception $e)
 		{
 			// Display the exception message
 			Kohana_Exception::handler($e);

classes/Ingenerator/KohanaView/TemplateSpecifyingViewModel.php

Indentation +4 added lines, -4 removed lines

@@ -16,8 +16,8 @@
 interface TemplateSpecifyingViewModel
 {
 
-    /**
-     * @return string
-     */
-    public function getTemplateName();
+	/**
+	 * @return string
+	 */
+	public function getTemplateName();
 }

classes/Ingenerator/KohanaView/ViewTemplateSelector.php

Indentation +51 added lines, -51 removed lines

@@ -18,61 +18,61 @@
 class ViewTemplateSelector
 {
 
-    /**
-     * @param ViewModel $view
-     *
-     * @return string
-     */
-    public function getTemplateName(ViewModel $view)
-    {
-        if ($view instanceof TemplateSpecifyingViewModel) {
-            return $this->validateSpecifiedTemplateName($view);
-        } else {
-            return $this->calculateTemplateFromClassName($view);
-        }
-    }
+	/**
+	 * @param ViewModel $view
+	 *
+	 * @return string
+	 */
+	public function getTemplateName(ViewModel $view)
+	{
+		if ($view instanceof TemplateSpecifyingViewModel) {
+			return $this->validateSpecifiedTemplateName($view);
+		} else {
+			return $this->calculateTemplateFromClassName($view);
+		}
+	}
 
-    /**
-     * @param TemplateSpecifyingViewModel $view
-     *
-     * @return mixed
-     * @throws \UnexpectedValueException if no template is provided
-     */
-    protected function validateSpecifiedTemplateName(TemplateSpecifyingViewModel $view)
-    {
-        $template   = $view->getTemplateName();
-        $view_class = get_class($view);
-        if ( ! $template) {
-            throw new \UnexpectedValueException(
-                "$view_class::getTemplateName() must return a template name, empty value returned"
-            );
-        }
+	/**
+	 * @param TemplateSpecifyingViewModel $view
+	 *
+	 * @return mixed
+	 * @throws \UnexpectedValueException if no template is provided
+	 */
+	protected function validateSpecifiedTemplateName(TemplateSpecifyingViewModel $view)
+	{
+		$template   = $view->getTemplateName();
+		$view_class = get_class($view);
+		if ( ! $template) {
+			throw new \UnexpectedValueException(
+				"$view_class::getTemplateName() must return a template name, empty value returned"
+			);
+		}
 
-        if ( ! is_string($template)) {
-            $type = is_object($template) ? get_class($template) : gettype($template);
-            throw new \UnexpectedValueException(
-                "$view_class::getTemplateName() must return a string template name, $type value returned"
-            );
-        }
+		if ( ! is_string($template)) {
+			$type = is_object($template) ? get_class($template) : gettype($template);
+			throw new \UnexpectedValueException(
+				"$view_class::getTemplateName() must return a string template name, $type value returned"
+			);
+		}
 
-        return $template;
-    }
+		return $template;
+	}
 
-    /**
-     * @param ViewModel $view
-     *
-     * @return string
-     */
-    protected function calculateTemplateFromClassName(ViewModel $view)
-    {
-        $template = preg_replace('/\\\\|_/', '/', get_class($view));
-        $template = preg_replace('/([a-z])([A-Z])/', '\1_\2', $template);
-        $template = strtolower($template);
-        if (substr($template, 0, 5) === 'view/') {
-            $template = substr($template, 5);
-        }
+	/**
+	 * @param ViewModel $view
+	 *
+	 * @return string
+	 */
+	protected function calculateTemplateFromClassName(ViewModel $view)
+	{
+		$template = preg_replace('/\\\\|_/', '/', get_class($view));
+		$template = preg_replace('/([a-z])([A-Z])/', '\1_\2', $template);
+		$template = strtolower($template);
+		if (substr($template, 0, 5) === 'view/') {
+			$template = substr($template, 5);
+		}
 
-        return $template;
-    }
+		return $template;
+	}
 
 }

classes/Ingenerator/KohanaView/ViewModel.php

Indentation +9 added lines, -9 removed lines

@@ -1,9 +1,9 @@ 
 <?php
 /**
- * @author     Andrew Coulton <andrew@ingenerator.com>
- * @copyright  2015 inGenerator Ltd
- * @license    http://kohanaframework.org/license
- */
+	 * @author     Andrew Coulton <andrew@ingenerator.com>
+	 * @copyright  2015 inGenerator Ltd
+	 * @license    http://kohanaframework.org/license
+	 */
 
 namespace Ingenerator\KohanaView;
 
@@ -17,10 +17,10 @@ 
 interface ViewModel
 {
 
-    /**
-     * @param  array $variables
-     * @return void
-     */
-    public function display(array $variables);
+	/**
+	 * @param  array $variables
+	 * @return void
+	 */
+	public function display(array $variables);
 
 }

classes/Ingenerator/KohanaView/ViewModel/AbstractViewModel.php

Indentation +68 added lines, -68 removed lines

@@ -37,81 +37,81 @@
  */
 abstract class AbstractViewModel implements ViewModel
 {
-    /**
-     * @var array The actual view data
-     */
-    protected $variables = [];
+	/**
+	 * @var array The actual view data
+	 */
+	protected $variables = [];
 
-    /**
-     * @var string[] The names of the valid set of fields that must be passed to the display() method
-     */
-    protected $expect_var_names = [];
+	/**
+	 * @var string[] The names of the valid set of fields that must be passed to the display() method
+	 */
+	protected $expect_var_names = [];
 
-    public function __construct()
-    {
-        // Assign the expect_var_names to ensure that we don't accidentally start requiring compiled fields
-        $this->expect_var_names = array_keys($this->variables);
-    }
+	public function __construct()
+	{
+		// Assign the expect_var_names to ensure that we don't accidentally start requiring compiled fields
+		$this->expect_var_names = array_keys($this->variables);
+	}
 
-    /**
-     * Get field values
-     *
-     * @param string $name
-     *
-     * @return mixed
-     */
-    public function __get($name)
-    {
-        if (array_key_exists($name, $this->variables)) {
-            return $this->variables[$name];
-        } elseif (method_exists($this, 'var_'.$name)) {
-            $method = 'var_'.$name;
+	/**
+	 * Get field values
+	 *
+	 * @param string $name
+	 *
+	 * @return mixed
+	 */
+	public function __get($name)
+	{
+		if (array_key_exists($name, $this->variables)) {
+			return $this->variables[$name];
+		} elseif (method_exists($this, 'var_'.$name)) {
+			$method = 'var_'.$name;
 
-            return $this->$method();
-        } else {
-            throw new \BadMethodCallException(static::class." does not define a '$name' field");
-        }
-    }
+			return $this->$method();
+		} else {
+			throw new \BadMethodCallException(static::class." does not define a '$name' field");
+		}
+	}
 
-    /**
-     * @param string $name
-     * @param mixed  $value
-     *
-     * @throws \BadMethodCallException values cannot be assigned except with the display method
-     */
-    public function __set($name, $value)
-    {
-        throw new \BadMethodCallException(static::class.' variables are read-only, cannot assign '.$name);
-    }
+	/**
+	 * @param string $name
+	 * @param mixed  $value
+	 *
+	 * @throws \BadMethodCallException values cannot be assigned except with the display method
+	 */
+	public function __set($name, $value)
+	{
+		throw new \BadMethodCallException(static::class.' variables are read-only, cannot assign '.$name);
+	}
 
-    /**
-     * Set the data to be rendered in the view - note this does not actually render the view.
-     *
-     * @param array $variables
-     */
-    public function display(array $variables)
-    {
-        $errors             = [];
-        $provided_variables = array_keys($variables);
-        foreach (array_diff($provided_variables, $this->expect_var_names) as $unexpected_var) {
-            if (method_exists($this, 'var_'.$unexpected_var)) {
-                $errors[] = "'$unexpected_var' conflicts with ::var_$unexpected_var()";
-            } else {
-                $errors[] = "'$unexpected_var' is not expected";
-            }
-        }
-        foreach (array_diff($this->expect_var_names, $provided_variables) as $missing_var) {
-            $errors[] = "'$missing_var' is missing";
-        }
+	/**
+	 * Set the data to be rendered in the view - note this does not actually render the view.
+	 *
+	 * @param array $variables
+	 */
+	public function display(array $variables)
+	{
+		$errors             = [];
+		$provided_variables = array_keys($variables);
+		foreach (array_diff($provided_variables, $this->expect_var_names) as $unexpected_var) {
+			if (method_exists($this, 'var_'.$unexpected_var)) {
+				$errors[] = "'$unexpected_var' conflicts with ::var_$unexpected_var()";
+			} else {
+				$errors[] = "'$unexpected_var' is not expected";
+			}
+		}
+		foreach (array_diff($this->expect_var_names, $provided_variables) as $missing_var) {
+			$errors[] = "'$missing_var' is missing";
+		}
 
-        if ( ! empty($errors)) {
-            throw new \InvalidArgumentException(
-                "Invalid variables provided to ".static::class."::display()"
-                ."\n - ".implode("\n - ", $errors)
-            );
-        }
+		if ( ! empty($errors)) {
+			throw new \InvalidArgumentException(
+				"Invalid variables provided to ".static::class."::display()"
+				."\n - ".implode("\n - ", $errors)
+			);
+		}
 
-        $this->variables = $variables;
-    }
+		$this->variables = $variables;
+	}
 
 }