RViewBase::loadTemplate()

Complexity

Conditions	15
Paths	385

Size

Total Lines	92
Code Lines	39

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
	public function loadTemplate($tpl = null)
	{
		// Uses the parent function if no framework suffix is present
		if (RHtmlMedia::$frameworkSuffix == '')
		{
			return parent::loadTemplate($tpl);
		}

		// Clear prior output
		$this->_output = null;

The property _output does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

		$template = JFactory::getApplication()->getTemplate();
		$layout = $this->getLayout();
		$layoutTemplate = $this->getLayoutTemplate();

		// Create the template file name based on the layout
		$file = isset($tpl) ? $layout . '_' . $tpl : $layout;

		// Clean the file name
		$file = preg_replace('/[^A-Z0-9_\.-]/i', '', $file);
		$tpl = isset($tpl) ? preg_replace('/[^A-Z0-9_\.-]/i', '', $tpl) : $tpl;

		// Load the language file for the template
		$lang = JFactory::getLanguage();
		$lang->load('tpl_' . $template, JPATH_BASE, null, false, true)
		|| $lang->load('tpl_' . $template, JPATH_THEMES . "/$template", null, false, true);

		// Change the template folder if alternative layout is in different template
		if (isset($layoutTemplate) && $layoutTemplate != '_' && $layoutTemplate != $template)
		{
			$this->_path['template'] = str_replace($template, $layoutTemplate, $this->_path['template']);

The property _path does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

		}

		// Load the template script
		jimport('joomla.filesystem.path');

		// Tries first with the framework suffix
		$filetofind = $this->_createFileName('template', array('name' => $file . '.' . RHtmlMedia::$frameworkSuffix));
		$this->_template = JPath::find($this->_path['template'], $filetofind);

The property _template does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

		// If no framework-speficic is found, tries with the normal alternate layout
		if ($this->_template == false)

It seems like you are loosely comparing $this->_template of type false|string against false; this is ambiguous if the string can be empty. Consider using a strict comparison === instead.

		{
			$filetofind      = $this->_createFileName('template', array('name' => $file));
			$this->_template = JPath::find($this->_path['template'], $filetofind);
		}

		// If alternate layout can't be found, fall back to default layout, first with framework suffix
		if ($this->_template == false)

It seems like you are loosely comparing $this->_template of type false|string against false; this is ambiguous if the string can be empty. Consider using a strict comparison === instead.

		{
			$filetofind = $this->_createFileName(
				'', array('name' => 'default' . (isset($tpl) ? '_' . $tpl : $tpl) . '.' . RHtmlMedia::$frameworkSuffix)
			);
			$this->_template = JPath::find($this->_path['template'], $filetofind);
		}

		// If no default with suffix is found, fall back to default layout without framework suffix
		if ($this->_template == false)

It seems like you are loosely comparing $this->_template of type false|string against false; this is ambiguous if the string can be empty. Consider using a strict comparison === instead.

		{
			$filetofind = $this->_createFileName('', array('name' => 'default' . (isset($tpl) ? '_' . $tpl : $tpl)));
			$this->_template = JPath::find($this->_path['template'], $filetofind);
		}

		if ($this->_template != false)

It seems like you are loosely comparing $this->_template of type false|string against false; this is ambiguous if the string can be empty. Consider using a strict comparison !== instead.

		{
			// Unset so as not to introduce into template scope
			unset($tpl);
			unset($file);

			// Never allow a 'this' property
			if (isset($this->this))
			{
				unset($this->this);
			}

			// Start capturing output into a buffer
			ob_start();

			// Include the requested template filename in the local scope
			// (this will execute the view logic).
			include $this->_template;

			// Done with the requested template; get the buffer and
			// clear it.
			$this->_output = ob_get_contents();
			ob_end_clean();

			return $this->_output;
		}
		else
		{
			throw new Exception(JText::sprintf('JLIB_APPLICATION_ERROR_LAYOUTFILE_NOT_FOUND', $file), 500);
		}