ingenerator / kohana-view

classes/Kohana/View/Model.php

Doc Comments +2 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)
@@ -260,6 +260,7 @@ 
 	 * overwritten by the local variable.
 	 *
 	 * @param    string  view filename
+	 * @param string $file
 	 * @return   string
 	 * @throws   Kohana_View_Exception
 	 * @uses     View::capture

Spacing +1 added lines, -1 removed lines

@@ -245,7 +245,7 @@
 	 */
 	public function bind($key, & $value)
 	{
-		$this->{$key} =& $value;
+		$this->{$key} = & $value;
 
 		return $this;
 	}

Indentation +241 added lines, -241 removed lines

@@ -12,272 +12,272 @@
  */
 class Kohana_View_Model {
 
-	// View filename
-	protected $_file;
+    // View filename
+    protected $_file;
 
-	/**
-	 * Returns a new raw View object. If you do not define the "file" parameter,
-	 * you must call [View::set_filename].
-	 *
-	 *     $view = View::factory($file);
-	 *
-	 * @param   string  view filename
-	 * @param   array   array of values
-	 * @return  View
-	 */
-	public static function factory($file = NULL, array $data = NULL)
-	{
-		// Return a raw view object if no template is specified.
-		if ($file === FALSE)
-			return new View(FALSE, $data);
+    /**
+     * Returns a new raw View object. If you do not define the "file" parameter,
+     * you must call [View::set_filename].
+     *
+     *     $view = View::factory($file);
+     *
+     * @param   string  view filename
+     * @param   array   array of values
+     * @return  View
+     */
+    public static function factory($file = NULL, array $data = NULL)
+    {
+        // Return a raw view object if no template is specified.
+        if ($file === FALSE)
+            return new View(FALSE, $data);
 
-		$class = 'View_'.strtr($file, '/', '_');
+        $class = 'View_'.strtr($file, '/', '_');
 
-		if ( ! class_exists($class))
-		{
-			$class = 'View';
-		}
+        if ( ! class_exists($class))
+        {
+            $class = 'View';
+        }
 
-		return new $class($file, $data);
-	}
+        return new $class($file, $data);
+    }
 
-	/**
-	 * Captures the output that is generated when a view is included.
-	 * The view data will be extracted to make local variables.
-	 *
-	 *     $output = $this->capture($file);
-	 *
-	 * @param   string  filename
-	 * @return  string
-	 */
-	protected function capture($kohana_view_filename)
-	{
-		if ( ! in_array('kohana.view', stream_get_wrappers()))
-		{
-			stream_wrapper_register('kohana.view', 'View_Stream_Wrapper');
-		}
+    /**
+     * Captures the output that is generated when a view is included.
+     * The view data will be extracted to make local variables.
+     *
+     *     $output = $this->capture($file);
+     *
+     * @param   string  filename
+     * @return  string
+     */
+    protected function capture($kohana_view_filename)
+    {
+        if ( ! in_array('kohana.view', stream_get_wrappers()))
+        {
+            stream_wrapper_register('kohana.view', 'View_Stream_Wrapper');
+        }
 
-		// Capture the view output
-		ob_start();
+        // Capture the view output
+        ob_start();
 
-		try
-		{
-			include 'kohana.view://'.$kohana_view_filename;
-		}
-		catch (Exception $e)
-		{
-			// Delete the output buffer
-			ob_end_clean();
+        try
+        {
+            include 'kohana.view://'.$kohana_view_filename;
+        }
+        catch (Exception $e)
+        {
+            // Delete the output buffer
+            ob_end_clean();
 
-			// Re-throw the exception
-			throw $e;
-		}
+            // Re-throw the exception
+            throw $e;
+        }
 
-		// Get the captured output and close the buffer
-		return ob_get_clean();
-	}
+        // Get the captured output and close the buffer
+        return ob_get_clean();
+    }
 
-	/**
-	 * Sets the initial view filename and local data. Views should almost
-	 * always only be created using [View::factory].
-	 *
-	 *     $view = new View($file);
-	 *
-	 * @param   string  view filename
-	 * @param   array   array of values
-	 * @return  void
-	 * @uses    View::set_filename
-	 */
-	public function __construct($file = NULL, array $data = NULL)
-	{
-		if ($file === NULL)
-		{
-			$foo = explode('_', get_class($this));
-			array_shift($foo);
-			$file = strtolower(implode('/', $foo));
-		}
+    /**
+     * Sets the initial view filename and local data. Views should almost
+     * always only be created using [View::factory].
+     *
+     *     $view = new View($file);
+     *
+     * @param   string  view filename
+     * @param   array   array of values
+     * @return  void
+     * @uses    View::set_filename
+     */
+    public function __construct($file = NULL, array $data = NULL)
+    {
+        if ($file === NULL)
+        {
+            $foo = explode('_', get_class($this));
+            array_shift($foo);
+            $file = strtolower(implode('/', $foo));
+        }
 		
-		if ($file !== FALSE)
-		{
-			$this->set_filename($file);
-		}
+        if ($file !== FALSE)
+        {
+            $this->set_filename($file);
+        }
 
-		if ($data !== NULL)
-		{
-			// Add the values to the current data
-			$this->set($data);
-		}
-	}
+        if ($data !== NULL)
+        {
+            // Add the values to the current data
+            $this->set($data);
+        }
+    }
 
-	/**
-	 * Magic method, searches for the given variable and returns its value.
-	 * Local variables will be returned before global variables.
-	 *
-	 *     $value = $view->foo;
-	 *
-	 * [!!] If the variable has not yet been set, an exception will be thrown.
-	 *
-	 * @param   string  variable name
-	 * @return  mixed
-	 * @throws  Kohana_Exception
-	 */
-	public function __get($key)
-	{
-		if (method_exists($this, $key))
-		{
-			return $this->$key();
-		}
-		elseif (property_exists($this, $key))
-		{
-			return $this->$key;
-		}
-		else
-		{
-			throw new Kohana_Exception('View variable is not set: :var',
-				array(':var' => $key));
-		}
-	}
+    /**
+     * Magic method, searches for the given variable and returns its value.
+     * Local variables will be returned before global variables.
+     *
+     *     $value = $view->foo;
+     *
+     * [!!] If the variable has not yet been set, an exception will be thrown.
+     *
+     * @param   string  variable name
+     * @return  mixed
+     * @throws  Kohana_Exception
+     */
+    public function __get($key)
+    {
+        if (method_exists($this, $key))
+        {
+            return $this->$key();
+        }
+        elseif (property_exists($this, $key))
+        {
+            return $this->$key;
+        }
+        else
+        {
+            throw new Kohana_Exception('View variable is not set: :var',
+                array(':var' => $key));
+        }
+    }
 
-	/**
-	 * Magic method, calls [View::set] with the same parameters.
-	 *
-	 *     $view->foo = 'something';
-	 *
-	 * @param   string  variable name
-	 * @param   mixed   value
-	 * @return  void
-	 */
-	public function __set($key, $value)
-	{
-		$this->set($key, $value);
-	}
+    /**
+     * Magic method, calls [View::set] with the same parameters.
+     *
+     *     $view->foo = 'something';
+     *
+     * @param   string  variable name
+     * @param   mixed   value
+     * @return  void
+     */
+    public function __set($key, $value)
+    {
+        $this->set($key, $value);
+    }
 
 
-	/**
-	 * Magic method, returns the output of [View::render].
-	 *
-	 * @return  string
-	 * @uses    View::render
-	 */
-	public function __toString()
-	{
-		try
-		{
-			return $this->render();
-		}
-		catch (Exception $e)
-		{
-			// Display the exception message
-			Kohana_Exception::handler($e);
+    /**
+     * Magic method, returns the output of [View::render].
+     *
+     * @return  string
+     * @uses    View::render
+     */
+    public function __toString()
+    {
+        try
+        {
+            return $this->render();
+        }
+        catch (Exception $e)
+        {
+            // Display the exception message
+            Kohana_Exception::handler($e);
 
-			return '';
-		}
-	}
+            return '';
+        }
+    }
 
-	/**
-	 * Sets the view filename.
-	 *
-	 *     $view->set_filename($file);
-	 *
-	 * @param   string  view filename
-	 * @return  View
-	 * @throws  Kohana_View_Exception
-	 */
-	public function set_filename($file)
-	{
-		if (($path = Kohana::find_file('views', $file)) === FALSE)
-		{
-			throw new Kohana_View_Exception('The requested view :file could not be found', array(
-				':file' => $file,
-			));
-		}
+    /**
+     * Sets the view filename.
+     *
+     *     $view->set_filename($file);
+     *
+     * @param   string  view filename
+     * @return  View
+     * @throws  Kohana_View_Exception
+     */
+    public function set_filename($file)
+    {
+        if (($path = Kohana::find_file('views', $file)) === FALSE)
+        {
+            throw new Kohana_View_Exception('The requested view :file could not be found', array(
+                ':file' => $file,
+            ));
+        }
 
-		// Store the file path locally
-		$this->_file = $path;
+        // Store the file path locally
+        $this->_file = $path;
 
-		return $this;
-	}
+        return $this;
+    }
 
-	/**
-	 * Assigns a variable by name. Assigned values will be available as a
-	 * variable within the view file:
-	 *
-	 *     // This value can be accessed as $foo within the view
-	 *     $view->set('foo', 'my value');
-	 *
-	 * You can also use an array to set several values at once:
-	 *
-	 *     // Create the values $food and $beverage in the view
-	 *     $view->set(array('food' => 'bread', 'beverage' => 'water'));
-	 *
-	 * @param   string   variable name or an array of variables
-	 * @param   mixed    value
-	 * @return  $this
-	 */
-	public function set($key, $value = NULL)
-	{
-		if (is_array($key))
-		{
-			foreach ($key as $name => $value)
-			{
-				$this->{$name} = $value;
-			}
-		}
-		else
-		{
-			$this->{$key} = $value;
-		}
+    /**
+     * Assigns a variable by name. Assigned values will be available as a
+     * variable within the view file:
+     *
+     *     // This value can be accessed as $foo within the view
+     *     $view->set('foo', 'my value');
+     *
+     * You can also use an array to set several values at once:
+     *
+     *     // Create the values $food and $beverage in the view
+     *     $view->set(array('food' => 'bread', 'beverage' => 'water'));
+     *
+     * @param   string   variable name or an array of variables
+     * @param   mixed    value
+     * @return  $this
+     */
+    public function set($key, $value = NULL)
+    {
+        if (is_array($key))
+        {
+            foreach ($key as $name => $value)
+            {
+                $this->{$name} = $value;
+            }
+        }
+        else
+        {
+            $this->{$key} = $value;
+        }
 
-		return $this;
-	}
+        return $this;
+    }
 
-	/**
-	 * Assigns a value by reference. The benefit of binding is that values can
-	 * be altered without re-setting them. It is also possible to bind variables
-	 * before they have values. Assigned values will be available as a
-	 * variable within the view file:
-	 *
-	 *     // This reference can be accessed as $ref within the view
-	 *     $view->bind('ref', $bar);
-	 *
-	 * @param   string   variable name
-	 * @param   mixed    referenced variable
-	 * @return  $this
-	 */
-	public function bind($key, & $value)
-	{
-		$this->{$key} =& $value;
+    /**
+     * Assigns a value by reference. The benefit of binding is that values can
+     * be altered without re-setting them. It is also possible to bind variables
+     * before they have values. Assigned values will be available as a
+     * variable within the view file:
+     *
+     *     // This reference can be accessed as $ref within the view
+     *     $view->bind('ref', $bar);
+     *
+     * @param   string   variable name
+     * @param   mixed    referenced variable
+     * @return  $this
+     */
+    public function bind($key, & $value)
+    {
+        $this->{$key} =& $value;
 
-		return $this;
-	}
+        return $this;
+    }
 
-	/**
-	 * Renders the view object to a string. Global and local data are merged
-	 * and extracted to create local variables within the view file.
-	 *
-	 *     $output = View::render();
-	 *
-	 * [!!] Global variables with the same key name as local variables will be
-	 * overwritten by the local variable.
-	 *
-	 * @param    string  view filename
-	 * @return   string
-	 * @throws   Kohana_View_Exception
-	 * @uses     View::capture
-	 */
-	public function render($file = NULL)
-	{
-		if ($file !== NULL)
-		{
-			$this->set_filename($file);
-		}
+    /**
+     * Renders the view object to a string. Global and local data are merged
+     * and extracted to create local variables within the view file.
+     *
+     *     $output = View::render();
+     *
+     * [!!] Global variables with the same key name as local variables will be
+     * overwritten by the local variable.
+     *
+     * @param    string  view filename
+     * @return   string
+     * @throws   Kohana_View_Exception
+     * @uses     View::capture
+     */
+    public function render($file = NULL)
+    {
+        if ($file !== NULL)
+        {
+            $this->set_filename($file);
+        }
 
-		if (empty($this->_file))
-		{
-			throw new Kohana_View_Exception('You must set the file to use within your view before rendering');
-		}
+        if (empty($this->_file))
+        {
+            throw new Kohana_View_Exception('You must set the file to use within your view before rendering');
+        }
 
-		// Combine local and global data and capture the output
-		return $this->capture($this->_file);
-	}
+        // Combine local and global data and capture the output
+        return $this->capture($this->_file);
+    }
 } // End View
\ No newline at end of file

Braces +23 added lines, -44 removed lines

@@ -10,7 +10,8 @@ 
  * @copyright  (c) 2008-2010 Kohana Team
  * @license    http://kohanaphp.com/license
  */
-class Kohana_View_Model {
+class Kohana_View_Model
+{
 
 	// View filename
 	protected $_file;
@@ -28,13 +29,13 @@ 
 	public static function factory($file = NULL, array $data = NULL)
 	{
 		// Return a raw view object if no template is specified.
-		if ($file === FALSE)
-			return new View(FALSE, $data);
+		if ($file === FALSE) {
+					return new View(FALSE, $data);
+		}
 
 		$class = 'View_'.strtr($file, '/', '_');
 
-		if ( ! class_exists($class))
-		{
+		if ( ! class_exists($class)) {
 			$class = 'View';
 		}
 
@@ -52,20 +53,16 @@ 
 	 */
 	protected function capture($kohana_view_filename)
 	{
-		if ( ! in_array('kohana.view', stream_get_wrappers()))
-		{
+		if ( ! in_array('kohana.view', stream_get_wrappers())) {
 			stream_wrapper_register('kohana.view', 'View_Stream_Wrapper');
 		}
 
 		// Capture the view output
 		ob_start();
 
-		try
-		{
+		try {
 			include 'kohana.view://'.$kohana_view_filename;
-		}
-		catch (Exception $e)
-		{
+		} catch (Exception $e) {
 			// Delete the output buffer
 			ob_end_clean();
 
@@ -90,20 +87,17 @@ 
 	 */
 	public function __construct($file = NULL, array $data = NULL)
 	{
-		if ($file === NULL)
-		{
+		if ($file === NULL) {
 			$foo = explode('_', get_class($this));
 			array_shift($foo);
 			$file = strtolower(implode('/', $foo));
 		}
 		
-		if ($file !== FALSE)
-		{
+		if ($file !== FALSE) {
 			$this->set_filename($file);
 		}
 
-		if ($data !== NULL)
-		{
+		if ($data !== NULL) {
 			// Add the values to the current data
 			$this->set($data);
 		}
@@ -123,16 +117,11 @@ 
 	 */
 	public function __get($key)
 	{
-		if (method_exists($this, $key))
-		{
+		if (method_exists($this, $key)) {
 			return $this->$key();
-		}
-		elseif (property_exists($this, $key))
-		{
+		} elseif (property_exists($this, $key)) {
 			return $this->$key;
-		}
-		else
-		{
+		} else {
 			throw new Kohana_Exception('View variable is not set: :var',
 				array(':var' => $key));
 		}
@@ -161,12 +150,9 @@ 
 	 */
 	public function __toString()
 	{
-		try
-		{
+		try {
 			return $this->render();
-		}
-		catch (Exception $e)
-		{
+		} catch (Exception $e) {
 			// Display the exception message
 			Kohana_Exception::handler($e);
 
@@ -185,8 +171,7 @@ 
 	 */
 	public function set_filename($file)
 	{
-		if (($path = Kohana::find_file('views', $file)) === FALSE)
-		{
+		if (($path = Kohana::find_file('views', $file)) === FALSE) {
 			throw new Kohana_View_Exception('The requested view :file could not be found', array(
 				':file' => $file,
 			));
@@ -216,15 +201,11 @@ 
 	 */
 	public function set($key, $value = NULL)
 	{
-		if (is_array($key))
-		{
-			foreach ($key as $name => $value)
-			{
+		if (is_array($key)) {
+			foreach ($key as $name => $value) {
 				$this->{$name} = $value;
 			}
-		}
-		else
-		{
+		} else {
 			$this->{$key} = $value;
 		}
 
@@ -267,13 +248,11 @@ 
 	 */
 	public function render($file = NULL)
 	{
-		if ($file !== NULL)
-		{
+		if ($file !== NULL) {
 			$this->set_filename($file);
 		}
 
-		if (empty($this->_file))
-		{
+		if (empty($this->_file)) {
 			throw new Kohana_View_Exception('You must set the file to use within your view before rendering');
 		}
 

classes/Ingenerator/View/Layout.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
      */
     public function var_page()
     {
-        if ( is_string($this->template))
+        if (is_string($this->template))
         {
             $class = 'View_Template_'.$this->template;
             if ( ! class_exists($class))

Braces +8 added lines, -17 removed lines

@@ -47,11 +47,9 @@ 
      */
     public function var_page()
     {
-        if ( is_string($this->template))
-        {
+        if ( is_string($this->template)) {
             $class = 'View_Template_'.$this->template;
-            if ( ! class_exists($class))
-            {
+            if ( ! class_exists($class)) {
                 $class = 'View';
             }
             $this->template = new $class();
@@ -109,8 +107,7 @@ 
         $this->post_render();
 
         // Render the template if required
-        if ($this->use_template())
-        {
+        if ($this->use_template()) {
             $template = $this->var_page();
             $template->set('var_body', $body);
             return $template->render();
@@ -138,27 +135,21 @@ 
     public function use_template($use_template = null)
     {
         // If we're being called as a setter
-        if ($use_template !== null)
-        {
+        if ($use_template !== null) {
             $this->_use_template = $use_template;
             return $this;
         }
 
         // If an explicit value was set
-        if ($this->_use_template !== null)
-        {
+        if ($this->_use_template !== null) {
             return $this->_use_template;
         }
 
         // Guess based on is_ajax
-        if (Request::current())
-        {
-            if (Request::current()->is_ajax())
-            {
+        if (Request::current()) {
+            if (Request::current()->is_ajax()) {
                 return false;
-            }
-            else
-            {
+            } else {
                 return true;
             }
         }

Upper-Lower-Casing +7 added lines, -7 removed lines

@@ -26,7 +26,7 @@ 
     public $template = 'Global';
 
     // Whether to render the view within the template or not, null for auto
-    protected $_use_template = null;
+    protected $_use_template = NULL;
 
     /**
      * Sets a new template to use for this view model. By default template names
@@ -99,7 +99,7 @@ 
      * @param string $file If wishing to generate a view to a file
      * @return string The formatted view content
      */
-    public function render($file = null)
+    public function render($file = NULL)
     {
         $this->pre_render();
 
@@ -135,17 +135,17 @@ 
      * @return boolean If called as getter
      * @return View_Layout If called as setter
      */
-    public function use_template($use_template = null)
+    public function use_template($use_template = NULL)
     {
         // If we're being called as a setter
-        if ($use_template !== null)
+        if ($use_template !== NULL)
         {
             $this->_use_template = $use_template;
             return $this;
         }
 
         // If an explicit value was set
-        if ($this->_use_template !== null)
+        if ($this->_use_template !== NULL)
         {
             return $this->_use_template;
         }
@@ -155,11 +155,11 @@ 
         {
             if (Request::current()->is_ajax())
             {
-                return false;
+                return FALSE;
             }
             else
             {
-                return true;
+                return TRUE;
             }
         }
     }

classes/Kohana/View/Stream/Wrapper.php

Spacing +2 added lines, -2 removed lines

@@ -125,7 +125,7 @@ 
             // If this block is not the special <?= block return
             if ($matches[1] !== '=')
             {
-                return '<?php '. $code . '?>';
+                return '<?php '.$code.'?>';
             }
 
             // Remove trailing ; characters
@@ -138,7 +138,7 @@ 
             else
             {
                 // Remove the "turn off escape" character
-                return '<?php echo '.substr($var, strlen($this->_raw_output_char), strlen($var)-1).'; ?>';
+                return '<?php echo '.substr($var, strlen($this->_raw_output_char), strlen($var) - 1).'; ?>';
             }
 	}
 

Indentation +171 added lines, -171 removed lines

@@ -39,85 +39,85 @@ 
  */
 class Kohana_View_Stream_Wrapper
 {
-	/**
-	 * Current stream position.
-	 *
-	 * @var int
-	 */
-	protected $_pos = 0;
-
-	/**
-	 * Data for streaming.
-	 *
-	 * @var string
-	 */
-	protected $_data;
-
-	/**
-	 * Stream stats.
-	 *
-	 * @var array
-	 */
-	protected $_stat;
-
-	/**
-	 * Raw output character. Prepend this on any echo variables to
-	 * turn off auto encoding of the output
-	 */
-	protected $_raw_output_char = '!';
-
-	/**
-	 * The encoding method to use on view output. Only use the method name
-	 */
-	protected $_encode_method = 'HTML::chars';
-
-	/**
-	 * Opens the script file and converts markup.
-	 */
-	public function stream_open($path, $mode, $options, &$opened_path)
-	{
-		// get the view script source
-		$path        = str_replace('kohana.view://', '', $path);
-		$this->_data = file_get_contents($path);
-
-		/**
-		 * If reading the file failed, update our local stat store
-		 * to reflect the real stat of the file, then return on failure
-		 */
-		if ($this->_data === false)
-		{
-			$this->_stat = stat($path);
-			return false;
-		}
+    /**
+     * Current stream position.
+     *
+     * @var int
+     */
+    protected $_pos = 0;
+
+    /**
+     * Data for streaming.
+     *
+     * @var string
+     */
+    protected $_data;
+
+    /**
+     * Stream stats.
+     *
+     * @var array
+     */
+    protected $_stat;
+
+    /**
+     * Raw output character. Prepend this on any echo variables to
+     * turn off auto encoding of the output
+     */
+    protected $_raw_output_char = '!';
+
+    /**
+     * The encoding method to use on view output. Only use the method name
+     */
+    protected $_encode_method = 'HTML::chars';
+
+    /**
+     * Opens the script file and converts markup.
+     */
+    public function stream_open($path, $mode, $options, &$opened_path)
+    {
+        // get the view script source
+        $path        = str_replace('kohana.view://', '', $path);
+        $this->_data = file_get_contents($path);
+
+        /**
+         * If reading the file failed, update our local stat store
+         * to reflect the real stat of the file, then return on failure
+         */
+        if ($this->_data === false)
+        {
+            $this->_stat = stat($path);
+            return false;
+        }
                 
-		/**
-                 * Escape all variables and convert <?= ?> to long-form <?php echo ?>
-		 *
-		 */
-		$regex = '/<\?(\=|php)?(.+?)\?>/s';
-		$this->_data = preg_replace_callback($regex, array($this, '_escape_val'), $this->_data);
-
-		/**
-		 * file_get_contents() won't update PHP's stat cache, so we grab a stat
-		 * of the file to prevent additional reads should the script be
-		 * requested again, which will make include() happy.
-		 */
-		$this->_stat = stat($path);
-
-		// PHP7.4 requires that we return the accurate value for the filesize, otherwise it does not read the full string
+        /**
+         * Escape all variables and convert <?= ?> to long-form <?php echo ?>
+         *
+         */
+        $regex = '/<\?(\=|php)?(.+?)\?>/s';
+        $this->_data = preg_replace_callback($regex, array($this, '_escape_val'), $this->_data);
+
+        /**
+         * file_get_contents() won't update PHP's stat cache, so we grab a stat
+         * of the file to prevent additional reads should the script be
+         * requested again, which will make include() happy.
+         */
+        $this->_stat = stat($path);
+
+        // PHP7.4 requires that we return the accurate value for the filesize, otherwise it does not read the full string
         // and then causes a cryptic "unexpected end of file" error
         $this->_stat['size'] = strlen($this->_data);
-		return true;
-	}
-
-	/**
-	 * Escapes a variable from template matching
-	 *
-	 * @param   array   matches
-	 * @return  string
-	 */
-	protected function _escape_val($matches)
-	{
+        return true;
+    }
+
+    /**
+     * Escapes a variable from template matching
+     *
+     * @param   array   matches
+     * @return  string
+     */
+    protected function _escape_val($matches)
+    {
             // Start by trimming the echoed string
             $code = trim($matches[2]);
 
@@ -143,101 +143,101 @@ 
                 // Remove the "turn off escape" character
                 return '<?php echo '.substr($var, strlen($this->_raw_output_char), strlen($var)-1).'; ?>';
             }
-	}
-
-	/**
-	 * Included so that __FILE__ returns the appropriate info
-	 *
-	 * @return array
-	 */
-	public function url_stat()
-	{
-		return $this->_stat;
-	}
-
-	/**
-	 * Reads from the stream.
-	 */
-	public function stream_read($count)
-	{
-		$ret = substr($this->_data, $this->_pos, $count);
-		$this->_pos += strlen($ret);
-		return $ret;
-	}
-
-	/**
-	 * Tells the current position in the stream.
-	 */
-	public function stream_tell()
-	{
-		return $this->_pos;
-	}
-
-	/**
-	 * Tells if we are at the end of the stream.
-	 */
-	public function stream_eof()
-	{
-		return $this->_pos >= strlen($this->_data);
-	}
-
-	/**
-	 * Stream statistics.
-	 */
-	public function stream_stat()
-	{
-		return $this->_stat;
-	}
-
-	/**
-	 * Seek to a specific point in the stream.
-	 */
-	public function stream_seek($offset, $whence)
-	{
-		switch ($whence)
-		{
-			case SEEK_SET:
-				if ($offset < strlen($this->_data) && $offset >= 0)
-				{
-					$this->_pos = $offset;
-					return true;
-				}
-				else
-				{
-					return false;
-				}
-				break;
-
-			case SEEK_CUR:
-				if ($offset >= 0)
-				{
-					$this->_pos += $offset;
-					return true;
-				}
-				else
-				{
-					return false;
-				}
-				break;
-
-			case SEEK_END:
-				if (strlen($this->_data) + $offset >= 0)
-				{
-					$this->_pos = strlen($this->_data) + $offset;
-					return true;
-				}
-				else
-				{
-					return false;
-				}
-				break;
-
-			default:
-				return false;
-		}
-	}
-
-	public function stream_set_option($option, $arg1, $arg2)
+    }
+
+    /**
+     * Included so that __FILE__ returns the appropriate info
+     *
+     * @return array
+     */
+    public function url_stat()
+    {
+        return $this->_stat;
+    }
+
+    /**
+     * Reads from the stream.
+     */
+    public function stream_read($count)
+    {
+        $ret = substr($this->_data, $this->_pos, $count);
+        $this->_pos += strlen($ret);
+        return $ret;
+    }
+
+    /**
+     * Tells the current position in the stream.
+     */
+    public function stream_tell()
+    {
+        return $this->_pos;
+    }
+
+    /**
+     * Tells if we are at the end of the stream.
+     */
+    public function stream_eof()
+    {
+        return $this->_pos >= strlen($this->_data);
+    }
+
+    /**
+     * Stream statistics.
+     */
+    public function stream_stat()
+    {
+        return $this->_stat;
+    }
+
+    /**
+     * Seek to a specific point in the stream.
+     */
+    public function stream_seek($offset, $whence)
+    {
+        switch ($whence)
+        {
+            case SEEK_SET:
+                if ($offset < strlen($this->_data) && $offset >= 0)
+                {
+                    $this->_pos = $offset;
+                    return true;
+                }
+                else
+                {
+                    return false;
+                }
+                break;
+
+            case SEEK_CUR:
+                if ($offset >= 0)
+                {
+                    $this->_pos += $offset;
+                    return true;
+                }
+                else
+                {
+                    return false;
+                }
+                break;
+
+            case SEEK_END:
+                if (strlen($this->_data) + $offset >= 0)
+                {
+                    $this->_pos = strlen($this->_data) + $offset;
+                    return true;
+                }
+                else
+                {
+                    return false;
+                }
+                break;
+
+            default:
+                return false;
+        }
+    }
+
+    public function stream_set_option($option, $arg1, $arg2)
     {
         // PHP7.4 requires this be implemented, but we can return FALSE that we don't support any of the options
         // (blocking mode, buffering, etc)

Braces +12 added lines, -27 removed lines

@@ -84,8 +84,7 @@ 
 		 * If reading the file failed, update our local stat store
 		 * to reflect the real stat of the file, then return on failure
 		 */
-		if ($this->_data === false)
-		{
+		if ($this->_data === false) {
 			$this->_stat = stat($path);
 			return false;
 		}
@@ -126,20 +125,16 @@ 
                     '$this->var_$1', $code);
 
             // If this block is not the special <?= block return
-            if ($matches[1] !== '=')
-            {
+            if ($matches[1] !== '=') {
                 return '<?php '. $code . '?>';
             }
 
             // Remove trailing ; characters
             $var = trim($code, ';');
 
-            if (substr($var, 0, 1) != $this->_raw_output_char)
-            {
+            if (substr($var, 0, 1) != $this->_raw_output_char) {
                 return '<?php echo '.$this->_encode_method.'('.$var.'); ?>';
-            }
-            else
-            {
+            } else {
                 // Remove the "turn off escape" character
                 return '<?php echo '.substr($var, strlen($this->_raw_output_char), strlen($var)-1).'; ?>';
             }
@@ -194,40 +189,30 @@ 
 	 */
 	public function stream_seek($offset, $whence)
 	{
-		switch ($whence)
-		{
+		switch ($whence) {
 			case SEEK_SET:
-				if ($offset < strlen($this->_data) && $offset >= 0)
-				{
+				if ($offset < strlen($this->_data) && $offset >= 0) {
 					$this->_pos = $offset;
 					return true;
-				}
-				else
-				{
+				} else {
 					return false;
 				}
 				break;
 
 			case SEEK_CUR:
-				if ($offset >= 0)
-				{
+				if ($offset >= 0) {
 					$this->_pos += $offset;
 					return true;
-				}
-				else
-				{
+				} else {
 					return false;
 				}
 				break;
 
 			case SEEK_END:
-				if (strlen($this->_data) + $offset >= 0)
-				{
+				if (strlen($this->_data) + $offset >= 0) {
 					$this->_pos = strlen($this->_data) + $offset;
 					return true;
-				}
-				else
-				{
+				} else {
 					return false;
 				}
 				break;
@@ -238,7 +223,7 @@ 
 	}
 
 	public function stream_set_option($option, $arg1, $arg2)
-    {
+	{
         // PHP7.4 requires this be implemented, but we can return FALSE that we don't support any of the options
         // (blocking mode, buffering, etc)
         return FALSE;

Upper-Lower-Casing +10 added lines, -10 removed lines

@@ -84,10 +84,10 @@ 
 		 * If reading the file failed, update our local stat store
 		 * to reflect the real stat of the file, then return on failure
 		 */
-		if ($this->_data === false)
+		if ($this->_data === FALSE)
 		{
 			$this->_stat = stat($path);
-			return false;
+			return FALSE;
 		}
                 
 		/**
@@ -107,7 +107,7 @@ 
 		// PHP7.4 requires that we return the accurate value for the filesize, otherwise it does not read the full string
         // and then causes a cryptic "unexpected end of file" error
         $this->_stat['size'] = strlen($this->_data);
-		return true;
+		return TRUE;
 	}
 
 	/**
@@ -200,11 +200,11 @@ 
 				if ($offset < strlen($this->_data) && $offset >= 0)
 				{
 					$this->_pos = $offset;
-					return true;
+					return TRUE;
 				}
 				else
 				{
-					return false;
+					return FALSE;
 				}
 				break;
 
@@ -212,11 +212,11 @@ 
 				if ($offset >= 0)
 				{
 					$this->_pos += $offset;
-					return true;
+					return TRUE;
 				}
 				else
 				{
-					return false;
+					return FALSE;
 				}
 				break;
 
@@ -224,16 +224,16 @@ 
 				if (strlen($this->_data) + $offset >= 0)
 				{
 					$this->_pos = strlen($this->_data) + $offset;
-					return true;
+					return TRUE;
 				}
 				else
 				{
-					return false;
+					return FALSE;
 				}
 				break;
 
 			default:
-				return false;
+				return FALSE;
 		}
 	}
 

views/template/global.php

Spacing +2 added lines, -2 removed lines

@@ -10,9 +10,9 @@
 <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">
 <html>
     <head>
-        <title><?=$title;?></title>
+        <title><?=$title; ?></title>
     </head>
     <body>
-        <?=$body;?>
+        <?=$body; ?>
     </body>
 </html>

koharness.php

Indentation +2 added lines, -2 removed lines

@@ -7,7 +7,7 @@
  * @link      https://github.com/ingenerator/koharness
  */
 return array(
-  'modules' => array(
+    'modules' => array(
     'kohana-view' => __DIR__,
-  )
+    )
 );

classes/ViewFactory.php

Spacing +1 added lines, -1 removed lines

@@ -14,4 +14,4 @@
  * The ViewFactory creates view classes, to allow stubbing of views for testing and easier extension in applications.
  *
  */
-class ViewFactory extends Ingenerator\ViewFactory{}
\ No newline at end of file
+class ViewFactory extends Ingenerator\ViewFactory {}
\ No newline at end of file

Braces +3 added lines, -1 removed lines

@@ -14,4 +14,6 @@
  * The ViewFactory creates view classes, to allow stubbing of views for testing and easier extension in applications.
  *
  */
-class ViewFactory extends Ingenerator\ViewFactory{}
\ No newline at end of file
+class ViewFactory extends Ingenerator\ViewFactory
+{
+}
\ No newline at end of file

classes/View/Stream/Wrapper.php

Braces +3 added lines, -1 removed lines

@@ -1,3 +1,5 @@
 <?php defined('SYSPATH') or die('No direct script access.');
 
-class View_Stream_Wrapper extends Kohana_View_Stream_Wrapper {}
+class View_Stream_Wrapper extends Kohana_View_Stream_Wrapper
+{
+}

classes/View/Model.php

Braces +3 added lines, -1 removed lines

@@ -1,3 +1,5 @@
 <?php defined('SYSPATH') or die('No direct script access.');
 
-class View_Model extends Kohana_View_Model {}
+class View_Model extends Kohana_View_Model
+{
+}

classes/Ingenerator/ViewFactory.php

Indentation +37 added lines, -37 removed lines

@@ -18,42 +18,42 @@
  */
 class ViewFactory {
 
-	/**
-	 * Create and return a view of the specified type
-	 *
-	 * @param string $class name of the view class to create
-	 * @param array  $data  array of data to pass to the view
-	 *
-	 * @return \View
-	 * @throws \View_Exception if the class is not defined
-	 */
-	public function create($class, $data = array())
-	{
-		// Check the view class exists
-		if ( ! class_exists($class))
-		{
-			throw new \View_Exception("View class $class does not exist");
-		}
-
-		// Temporary fix to map template names for namespaced classes
-		//@todo: Refactor the existing factory methods and classes to support namespaces
-		if ('\\View\\' === substr($class, 0, 6))
-		{
-			// Strip the \View\ prefix
-			$template = substr($class, 6);
-
-			// Exchange namespace separators for directory separators
-			$template = str_replace('\\', DIRECTORY_SEPARATOR, $template);
-
-			// Lowercase the View file name
-			$template = strtolower($template);
-		}
-		else
-		{
-			$template = NULL;
-		}
-
-		return new $class($template, $data);
-	}
+    /**
+     * Create and return a view of the specified type
+     *
+     * @param string $class name of the view class to create
+     * @param array  $data  array of data to pass to the view
+     *
+     * @return \View
+     * @throws \View_Exception if the class is not defined
+     */
+    public function create($class, $data = array())
+    {
+        // Check the view class exists
+        if ( ! class_exists($class))
+        {
+            throw new \View_Exception("View class $class does not exist");
+        }
+
+        // Temporary fix to map template names for namespaced classes
+        //@todo: Refactor the existing factory methods and classes to support namespaces
+        if ('\\View\\' === substr($class, 0, 6))
+        {
+            // Strip the \View\ prefix
+            $template = substr($class, 6);
+
+            // Exchange namespace separators for directory separators
+            $template = str_replace('\\', DIRECTORY_SEPARATOR, $template);
+
+            // Lowercase the View file name
+            $template = strtolower($template);
+        }
+        else
+        {
+            $template = NULL;
+        }
+
+        return new $class($template, $data);
+    }
 
 }
\ No newline at end of file

Braces +5 added lines, -8 removed lines

@@ -16,7 +16,8 @@ 
  *
  * @package Ingenerator
  */
-class ViewFactory {
+class ViewFactory
+{
 
 	/**
 	 * Create and return a view of the specified type
@@ -30,15 +31,13 @@ 
 	public function create($class, $data = array())
 	{
 		// Check the view class exists
-		if ( ! class_exists($class))
-		{
+		if ( ! class_exists($class)) {
 			throw new \View_Exception("View class $class does not exist");
 		}
 
 		// Temporary fix to map template names for namespaced classes
 		//@todo: Refactor the existing factory methods and classes to support namespaces
-		if ('\\View\\' === substr($class, 0, 6))
-		{
+		if ('\\View\\' === substr($class, 0, 6)) {
 			// Strip the \View\ prefix
 			$template = substr($class, 6);
 
@@ -47,9 +46,7 @@ 
 
 			// Lowercase the View file name
 			$template = strtolower($template);
-		}
-		else
-		{
+		} else {
 			$template = NULL;
 		}