Patch to fix a bug in KTemplateAbstract->loadString()

gistfile1.diff

Index: code/libraries/koowa/template/abstract.php
===================================================================
--- code/libraries/koowa/template/abstract.php	(revision 3467)
+++ code/libraries/koowa/template/abstract.php	(working copy)
@@ -10,74 +10,74 @@
 
  /**
   * Abstract Template class
-  * 
+  *
   * @author		Johan Janssens <[email protected]>
   * @category	Koowa
   * @package	Koowa_Template
   */
 abstract class KTemplateAbstract extends KObject implements KObjectIdentifiable
-{ 
-	/** 
+{
+	/**
 	 * The template path
-	 * 
+	 *
 	 * @var string
 	 */
 	protected $_path;
-	
+
 	/**
 	 * The template data
-	 * 
+	 *
 	 * @var array
 	 */
 	protected $_data = array();
-	
+
 	/**
 	 * The template contents
-	 * 
+	 *
 	 * @var string
 	 */
 	protected $_contents = '';
-	
+
    	/**
      * The set of template filters for templates
      *
      * @var array
      */
    	protected $_filters = array();
-   	
+
    	/**
 	 * View object or identifier (APP::com.COMPONENT.view.NAME.FORMAT)
 	 *
 	 * @var	string|object
 	 */
     protected $_view;
-    	
+
 	/**
 	 * Constructor
 	 *
 	 * Prevent creating instances of this class by making the contructor private
-	 * 
+	 *
 	 * @param 	object 	An optional KConfig object with configuration options
 	 */
 	public function __construct(KConfig $config)
 	{
 		parent::__construct($config);
-	
+
 		// Set the table indentifier
     	if(isset($config->view)) {
 			$this->_view = $config->view;
 		}
-			
+
 		//Register the template stream wrapper
 		KTemplateStream::register();
-		
+
 		//Set the template search paths
 		$this->_paths = KConfig::toData($config->paths);
-		
+
 		 // Mixin a command chain
         $this->mixin(new KMixinCommandchain($config->append(array('mixer' => $this))));
 	}
-	
+
  	/**
      * Initializes the options for the object
      *
@@ -95,31 +95,31 @@
     		'dispatch_events'   => false,
     		'enable_callbacks' 	=> false,
         ));
-        
+
         parent::_initialize($config);
     }
-    
+
 	/**
 	 * Get the object identifier
-	 * 
-	 * @return	KIdentifier	
+	 *
+	 * @return	KIdentifier
 	 * @see 	KObjectIdentifiable
 	 */
 	public function getIdentifier()
 	{
 		return $this->_identifier;
 	}
-	
+
 	/**
 	 * Get the template path
-	 * 
+	 *
 	 * @return	string
 	 */
 	public function getPath()
 	{
 		return $this->_path;
 	}
-	
+
 	/**
 	 * Get the view object attached to the template
 	 *
@@ -128,15 +128,15 @@
 	public function getView()
 	{
 	    if(!$this->_view instanceof KViewAbstract)
-		{	   
+		{
 		    //Make sure we have a view identifier
 		    if(!($this->_view instanceof KIdentifier)) {
 		        $this->setView($this->_view);
             }
-		    
+
 		    $this->_view = KFactory::tmp($this->_view, $config);
 		}
-		
+
 		return $this->_view;
 	}
 
@@ -152,29 +152,29 @@
 	{
 		if(!($view instanceof KViewAbstract))
 		{
-			if(is_string($view) && strpos($view, '.') === false ) 
+			if(is_string($view) && strpos($view, '.') === false )
 		    {
 			    $identifier			= clone $this->_identifier;
 			    $identifier->path	= array('view', $view);
 			    $identifier->name	= KRequest::format() ? KRequest::format() : 'html';
 			}
 			else $identifier = KFactory::identify($view);
-		    
+
 			if($identifier->path[0] != 'view') {
 				throw new KTemplateException('Identifier: '.$identifier.' is not a view identifier');
 			}
 
 			$view = $identifier;
 		}
-		
+
 		$this->_view = $view;
-		
+
 		return $this;
 	}
-	
+
 	/**
 	 * Load a template by identifier
-	 * 
+	 *
 	 * This functions only accepts full identifiers of the format
 	 * - application::com.component.view.[.path].name
 	 *
@@ -186,21 +186,21 @@
 	{
 	    //Identify the template
 	    $identifier = KFactory::identify($template);
-       
+
         // Load the identifier
-        $file = $identifier->name; 
-		
+        $file = $identifier->name;
+
         if($identifier->filepath) {
 	       $path = dirname($identifier->filepath);
         } else {
 	       $path = dirname(KLoader::path($identifier));
 	    }
-	   
+
 		$this->loadFile($path.'/'.$file.'.php', $data);
-		
+
 		return $this;
 	}
-	
+
 	/**
 	 * Load a template by path
 	 *
@@ -210,22 +210,22 @@
 	 */
 	public function loadFile($file, $data = array())
 	{
-		// find the template 
+		// find the template
 		$path = $this->findFile($file);
-	    
+
 		if ($path === false) {
 			throw new KTemplateException( 'Template "' . $file . '" not found' );
 		}
-		
+
 		// get the file contents
 		$contents = file_get_contents($path);
-		
+
 		// load the contents
 		$this->loadString($contents, $data, $file);
-		
+
 		return $this;
 	}
-	
+
 	/**
 	 * Load a template from a string
 	 *
@@ -238,42 +238,42 @@
 	{
 		$this->_contents = $string;
 		$this->_path     = empty($path) ? md5($string) : $path;
-		
+
 		// Merge the data
-	    $this->_data = array_merge($this->_data, $data);
-	
+	    $this->_data = array_merge((array) $this->_data, $data);
+
 		return $this;
 	}
-	
+
 	/**
 	 * Implement a sandbox to load and render a template
-	 * 
-	 * This function passes the template through the read filter chain and then include 
-	 * it in local scope buffers the result and passes it through the write filter chain 
+	 *
+	 * This function passes the template through the read filter chain and then include
+	 * it in local scope buffers the result and passes it through the write filter chain
 	 * before returning.
 	 *
 	 * @param  boolean 	If TRUE apply write filters. Default FALSE.
 	 * @return KTemplateAbstract
 	 */
 	public function render($filter = false)
-	{	
+	{
 		//Set the template in the template registry
        	KFactory::get('lib.koowa.template.registry')->set($this->_path, $this);
-       	
+
        	extract($this->_data, EXTR_SKIP); //extract the data in local scope
-       	
+
        	// Capturing output into a buffer
 		ob_start();
 		include 'tmpl://'.$this->_path;
 		$this->_contents = ob_get_clean();
-		
+
 		if($filter) {
 			$this->_contents = $this->filter(KTemplateFilter::MODE_WRITE);
 		}
-	
+
 		//Remove the template object from the template registry
        	KFactory::get('lib.koowa.template.registry')->del($this->_path);
-       
+
 		return $this->_contents;
 	}
 
@@ -284,18 +284,18 @@
 	 * @return string	The filtered data
 	 */
 	public function filter($mode = KTemplateFilter::MODE_READ)
-	{	
+	{
         $context = $this->getCommandContext();
 		$context->data = $this->_contents;
-				
+
         $result = $this->getCommandChain()->run($mode, $context);
- 		
+
  		return $context->data;
 	}
-	
+
 	/**
 	 * Adds one or multiple filters for template transformation
-	 * 
+	 *
 	 * @param array 	Array of one or more behaviors to add.
 	 * @return KTemplate
 	 */
@@ -303,23 +303,23 @@
  	{
  		foreach($filters as $filter)
 		{
-			if(!($filter instanceof KTemplateFilterInterface)) 
+			if(!($filter instanceof KTemplateFilterInterface))
 			{
 				$identifier = (string) $filter;
 				$filter     = KTemplateFilter::factory($filter);
 			}
 			else $identifier = (string) $filter->getIdentifier();
-				
+
 			//Enqueue the filter in the command chain
 			$this->getCommandChain()->enqueue($filter);
-			
+
 			//Store the filter
 			$this->_filters[$identifier] = $filter;
 		}
-		
+
 		return $this;
  	}
- 	
+
  	/**
 	 * Get the filters for the template
 	 *
@@ -329,7 +329,7 @@
  	{
  		return $this->_filters;
  	}
- 	
+
 	/**
 	 * Get a filter by identifier
 	 *
@@ -339,7 +339,7 @@
  	{
  		return isset($this->_filters[$identifier]) ? $this->_filters[$identifier] : null;
  	}
- 	
+
 	/**
 	 * Searches for the file
 	 *
@@ -348,10 +348,10 @@
 	 * 					if the file is not found
 	 */
 	public function findFile($file)
-	{    
+	{
 	    $result = false;
 	    $path   = dirname($file);
-	    
+
 	    // is the path based on a stream?
 		if (strpos($path, '://') === false)
 		{
@@ -362,7 +362,7 @@
 		}
 
 		// The substr() check added to make sure that the realpath()
-		// results in a directory registered so that non-registered directores 
+		// results in a directory registered so that non-registered directores
 		// are not accessible via directory traversal attempts.
 		if (file_exists($file) && substr($file, 0, strlen($path)) == $path) {
 			$result = $file;
@@ -371,10 +371,10 @@
 		// could not find the file in the set of paths
 		return $result;
 	}
-	
+
 	/**
 	 * Load a template helper
-	 * 
+	 *
 	 * This functions accepts a partial identifier, in the form of helper.function. If a partial
 	 * identifier is passed a full identifier will be created using the template identifier.
 	 *
@@ -387,27 +387,27 @@
 		//Get the function to call based on the $identifier
 		$parts    = explode('.', $identifier);
 		$function = array_pop($parts);
-		
+
 		//Create the complete identifier if a partial identifier was passed
-		if(is_string($identifier) && count($parts) == 1 ) 
+		if(is_string($identifier) && count($parts) == 1 )
 		{
             $identifier = clone $this->getIdentifier();
             $identifier->path = array('template','helper');
             $identifier->name = $parts[0];
-		} 
+		}
 		else $identifier = implode('.', $parts);
-		
+
 		//Create the template helper
 		$helper = KTemplateHelper::factory($identifier);
-		
+
 		//Call the helper function
 		if (!is_callable( array( $helper, $function ) )) {
 			throw new KTemplateHelperException( get_class($helper).'::'.$function.' not supported.' );
-		}	
-		
+		}
+
 		return $helper->$function($params);
 	}
-	
+
 	/**
 	 * Renders the template and returns the result
  	 *
@@ -419,8 +419,8 @@
 			$result = $this->render();
 		} catch (Exception $e) {
 			$result = $e->getMessage();
-		} 
-			
+		}
+
 		return $result;
 	}
 }
\ No newline at end of file