ingenerator / kohana-view

koharness.php

Indentation +7 added lines, -7 removed lines

@@ -1,13 +1,13 @@
 <?php
 /**
- * Configuration for the koharness module testing environment.
- * 
- * @author    Andrew Coulton <andrew@ingenerator.com>
- * @copyright 2013 inGenerator Ltd
- * @link      https://github.com/ingenerator/koharness
- */
+	 * Configuration for the koharness module testing environment.
+	 * 
+	 * @author    Andrew Coulton <andrew@ingenerator.com>
+	 * @copyright 2013 inGenerator Ltd
+	 * @link      https://github.com/ingenerator/koharness
+	 */
 return array(
   'modules' => array(
-    'kohana-view' => __DIR__,
+	'kohana-view' => __DIR__,
   )
 );

classes/Ingenerator/KohanaView/TemplateCompiler.php

Indentation +71 added lines, -71 removed lines

@@ -31,86 +31,86 @@
 class TemplateCompiler
 {
 
-    /**
-     * @var array
-     */
-    protected $options = [
-        'raw_output_prefix' => '!',
-        'escape_method'     => 'HTML::chars',
-    ];
+	/**
+	 * @var array
+	 */
+	protected $options = [
+		'raw_output_prefix' => '!',
+		'escape_method'     => 'HTML::chars',
+	];
 
-    /**
-     * @param array $options
-     */
-    public function __construct(array $options = [])
-    {
-        $this->options = array_merge($this->options, $options);
-    }
+	/**
+	 * @param array $options
+	 */
+	public function __construct(array $options = [])
+	{
+		$this->options = array_merge($this->options, $options);
+	}
 
-    /**
-     * Compile a string containing a PHP template, automatically escaping variables that are echoed in PHP short tags,
-     * and return the compiled PHP string.
-     *
-     * @param string $source
-     *
-     * @return string
-     * @throws \InvalidArgumentException if the template is empty or invalid
-     */
-    public function compile($source)
-    {
-        if ( ! $source) {
-            throw new \InvalidArgumentException('Cannot compile empty template');
-        }
+	/**
+	 * Compile a string containing a PHP template, automatically escaping variables that are echoed in PHP short tags,
+	 * and return the compiled PHP string.
+	 *
+	 * @param string $source
+	 *
+	 * @return string
+	 * @throws \InvalidArgumentException if the template is empty or invalid
+	 */
+	public function compile($source)
+	{
+		if ( ! $source) {
+			throw new \InvalidArgumentException('Cannot compile empty template');
+		}
 
-        return preg_replace_callback('/<\?=(.+?)(;|\?>)/s', [$this, 'compilePhpShortTag'], $source);
-    }
+		return preg_replace_callback('/<\?=(.+?)(;|\?>)/s', [$this, 'compilePhpShortTag'], $source);
+	}
 
-    /**
-     * @param string[] $matches
-     *
-     * @return string
-     */
-    protected function compilePhpShortTag($matches)
-    {
-        $var               = trim($matches[1]);
-        $terminator        = $matches[2];
-        $escape_method     = $this->options['escape_method'];
-        $raw_output_prefix = $this->options['raw_output_prefix'];
+	/**
+	 * @param string[] $matches
+	 *
+	 * @return string
+	 */
+	protected function compilePhpShortTag($matches)
+	{
+		$var               = trim($matches[1]);
+		$terminator        = $matches[2];
+		$escape_method     = $this->options['escape_method'];
+		$raw_output_prefix = $this->options['raw_output_prefix'];
 
-        if ($this->startsWith($var, $raw_output_prefix)) {
-            // Remove prefix and echo unescaped
-            $compiled = '<?='.trim(substr($var, strlen($raw_output_prefix))).';';
-        } elseif ($this->startsWith($var, '//')) {
-            // Echo an empty string to prevent the comment causing a parse error
-            $compiled = "<?='';$var;";
+		if ($this->startsWith($var, $raw_output_prefix)) {
+			// Remove prefix and echo unescaped
+			$compiled = '<?='.trim(substr($var, strlen($raw_output_prefix))).';';
+		} elseif ($this->startsWith($var, '//')) {
+			// Echo an empty string to prevent the comment causing a parse error
+			$compiled = "<?='';$var;";
 
-        } elseif ($this->startsWith($var, $escape_method)) {
-            // Try to help user to avoid accidental double-escaping
-            throw new \InvalidArgumentException(
-                "Invalid implicit double-escape in template - remove $escape_method from {$matches[0]} or mark as raw"
-            );
+		} elseif ($this->startsWith($var, $escape_method)) {
+			// Try to help user to avoid accidental double-escaping
+			throw new \InvalidArgumentException(
+				"Invalid implicit double-escape in template - remove $escape_method from {$matches[0]} or mark as raw"
+			);
 
-        } else {
-            // Escape the value before echoing
-            $compiled = "<?={$escape_method}($var);";
-        }
+		} else {
+			// Escape the value before echoing
+			$compiled = "<?={$escape_method}($var);";
+		}
 
-        if ($terminator === '?>') {
-            $compiled .= '?>';
-        }
+		if ($terminator === '?>') {
+			$compiled .= '?>';
+		}
 
-        return $compiled;
-    }
+		return $compiled;
+	}
 
-    /**
-     * @param string $string
-     * @param string $prefix
-     *
-     * @return bool
-     */
-    protected function startsWith($string, $prefix)
-    {
-        return (strncmp($string, $prefix, strlen($prefix)) === 0);
-    }
+	/**
+	 * @param string $string
+	 * @param string $prefix
+	 *
+	 * @return bool
+	 */
+	protected function startsWith($string, $prefix)
+	{
+		return (strncmp($string, $prefix, strlen($prefix)) === 0);
+	}
 
 }

classes/Ingenerator/KohanaView/TemplateManager.php

Indentation +6 added lines, -6 removed lines

@@ -17,11 +17,11 @@
 interface TemplateManager
 {
 
-    /**
-     * @param string $template_name Name of the template to use (can include path separators)
-     *
-     * @return string Path to the compiled template file
-     */
-    public function getPath($template_name);
+	/**
+	 * @param string $template_name Name of the template to use (can include path separators)
+	 *
+	 * @return string Path to the compiled template file
+	 */
+	public function getPath($template_name);
 
 }

classes/Ingenerator/KohanaView/TemplateManager/CFSWrapper.php

Indentation +7 added lines, -7 removed lines

@@ -15,11 +15,11 @@
 class CFSWrapper
 {
 
-    /**
-     * @see \Kohana::find_file
-     */
-    public function find_file($dir, $file)
-    {
-        return \Kohana::find_file($dir, $file, NULL, FALSE);
-    }
+	/**
+	 * @see \Kohana::find_file
+	 */
+	public function find_file($dir, $file)
+	{
+		return \Kohana::find_file($dir, $file, NULL, FALSE);
+	}
 }

classes/Ingenerator/KohanaView/TemplateSpecifyingViewModel.php

Indentation +8 added lines, -8 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;
 
@@ -16,8 +16,8 @@ 
 interface TemplateSpecifyingViewModel
 {
 
-    /**
-     * @return string
-     */
-    public function getTemplateName();
+	/**
+	 * @return string
+	 */
+	public function getTemplateName();
 }

classes/Ingenerator/KohanaView/Renderer.php

Indentation +6 added lines, -6 removed lines

@@ -17,11 +17,11 @@
 interface Renderer
 {
 
-    /**
-     * @param ViewModel $view
-     *
-     * @return string
-     */
-    public function render(ViewModel $view);
+	/**
+	 * @param ViewModel $view
+	 *
+	 * @return string
+	 */
+	public function render(ViewModel $view);
 
 }

classes/Ingenerator/KohanaView/Renderer/HTMLRenderer.php

Indentation +67 added lines, -67 removed lines

@@ -22,72 +22,72 @@
 class HTMLRenderer implements Renderer
 {
 
-    /**
-     * @var TemplateManager
-     */
-    protected $template_manager;
-
-    /**
-     * @var ViewTemplateSelector
-     */
-    protected $template_selector;
-
-    public function __construct(ViewTemplateSelector $template_selector, TemplateManager $template_manager)
-    {
-        $this->template_selector = $template_selector;
-        $this->template_manager  = $template_manager;
-    }
-
-
-    /**
-     * {@inheritdoc}
-     */
-    public function render(ViewModel $view)
-    {
-        $template_path = $this->getTemplatePath($view);
-
-        ob_start();
-        try {
-            $this->includeWithAnonymousScope($view, $template_path);
-        } finally {
-            $output = ob_get_clean();
-        }
-
-        return $output;
-    }
-
-    /**
-     * @param ViewModel $view
-     *
-     * @return string
-     */
-    protected function getTemplatePath(ViewModel $view)
-    {
-        $template_name = $this->template_selector->getTemplateName($view);
-        $template      = $this->template_manager->getPath($template_name);
-
-        return $template;
-    }
-
-    /**
-     * @param ViewModel $view
-     * @param string    $template_path
-     */
-    protected function includeWithAnonymousScope(ViewModel $view, $template_path)
-    {
-        /** @noinspection PhpUnusedParameterInspection */
-        /** @noinspection PhpDocSignatureInspection */
-        $bound_capture = function (ViewModel $view, Renderer $renderer, $template) {
-            /** @noinspection PhpIncludeInspection */
-            return include $template;
-        };
-        $anon_capture  = $bound_capture->bindTo(NULL);
-
-        // A user's own error handler may throw an exception here if the include fails - which we will bubble as-is.
-        // If they have not configured an error handler, we need to throw an exception of our own.
-        if ($anon_capture($view, $this, $template_path) === FALSE) {
-            throw new \UnexpectedValueException('Failed to include template '.$template_path);
-        }
-    }
+	/**
+	 * @var TemplateManager
+	 */
+	protected $template_manager;
+
+	/**
+	 * @var ViewTemplateSelector
+	 */
+	protected $template_selector;
+
+	public function __construct(ViewTemplateSelector $template_selector, TemplateManager $template_manager)
+	{
+		$this->template_selector = $template_selector;
+		$this->template_manager  = $template_manager;
+	}
+
+
+	/**
+	 * {@inheritdoc}
+	 */
+	public function render(ViewModel $view)
+	{
+		$template_path = $this->getTemplatePath($view);
+
+		ob_start();
+		try {
+			$this->includeWithAnonymousScope($view, $template_path);
+		} finally {
+			$output = ob_get_clean();
+		}
+
+		return $output;
+	}
+
+	/**
+	 * @param ViewModel $view
+	 *
+	 * @return string
+	 */
+	protected function getTemplatePath(ViewModel $view)
+	{
+		$template_name = $this->template_selector->getTemplateName($view);
+		$template      = $this->template_manager->getPath($template_name);
+
+		return $template;
+	}
+
+	/**
+	 * @param ViewModel $view
+	 * @param string    $template_path
+	 */
+	protected function includeWithAnonymousScope(ViewModel $view, $template_path)
+	{
+		/** @noinspection PhpUnusedParameterInspection */
+		/** @noinspection PhpDocSignatureInspection */
+		$bound_capture = function (ViewModel $view, Renderer $renderer, $template) {
+			/** @noinspection PhpIncludeInspection */
+			return include $template;
+		};
+		$anon_capture  = $bound_capture->bindTo(NULL);
+
+		// A user's own error handler may throw an exception here if the include fails - which we will bubble as-is.
+		// If they have not configured an error handler, we need to throw an exception of our own.
+		if ($anon_capture($view, $this, $template_path) === FALSE) {
+			throw new \UnexpectedValueException('Failed to include template '.$template_path);
+		}
+	}
 
 }

Spacing +2 added lines, -2 removed lines

@@ -77,11 +77,11 @@
     {
         /** @noinspection PhpUnusedParameterInspection */
         /** @noinspection PhpDocSignatureInspection */
-        $bound_capture = function (ViewModel $view, Renderer $renderer, $template) {
+        $bound_capture = function(ViewModel $view, Renderer $renderer, $template) {
             /** @noinspection PhpIncludeInspection */
             return include $template;
         };
-        $anon_capture  = $bound_capture->bindTo(NULL);
+        $anon_capture = $bound_capture->bindTo(NULL);
 
         // A user's own error handler may throw an exception here if the include fails - which we will bubble as-is.
         // If they have not configured an error handler, we need to throw an exception of our own.

classes/Ingenerator/KohanaView/ViewModel/PageContentView.php

Indentation +6 added lines, -6 removed lines

@@ -21,10 +21,10 @@
 interface PageContentView extends ViewModel
 {
 
-    /**
-     * The page layout that this content view will be rendered into
-     *
-     * @return PageLayoutView
-     */
-    public function var_page();
+	/**
+	 * The page layout that this content view will be rendered into
+	 *
+	 * @return PageLayoutView
+	 */
+	public function var_page();
 }

classes/Ingenerator/KohanaView/ViewModel/PageLayout/AbstractPageContentView.php

Indentation +22 added lines, -22 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\ViewModel\PageLayout;
 
@@ -27,23 +27,23 @@ 
 abstract class AbstractPageContentView extends AbstractViewModel implements PageContentView
 {
 
-    /**
-     * @var PageLayoutView
-     */
-    protected $page_view;
-
-    public function __construct(PageLayoutView $page)
-    {
-        $this->page_view = $page;
-        parent::__construct();
-    }
-
-    /**
-     * @return PageLayoutView
-     */
-    public function var_page()
-    {
-        return $this->page_view;
-    }
+	/**
+	 * @var PageLayoutView
+	 */
+	protected $page_view;
+
+	public function __construct(PageLayoutView $page)
+	{
+		$this->page_view = $page;
+		parent::__construct();
+	}
+
+	/**
+	 * @return PageLayoutView
+	 */
+	public function var_page()
+	{
+		return $this->page_view;
+	}
 
 }