dwoo-project / dwoo

lib/Dwoo/Adapters/CodeIgniter/libraries/Dwootemplate.php

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 40 and the first side effect is on line 18.

/**
 * Copyright (c) 2013-2016
 *
 * @category  Library
 * @package   Dwoo\Adapters\CodeIgniter\libraries
 * @author    Jordi Boggiano <j.boggiano@seld.be>
 * @author    David Sanchez <david38sanchez@gmail.com>
 * @copyright 2008-2013 Jordi Boggiano
 * @copyright 2013-2016 David Sanchez
 * @license   http://dwoo.org/LICENSE Modified BSD License
 * @version   1.3.0
 * @date      2016-09-19
 * @link      http://dwoo.org/
 */

if (!defined('BASEPATH')) {
    exit('No direct script access allowed');
}

require 'dwoo/dwooAutoload.php';

/**
 * Creates an template interface from Codeigniter to DWOO.
 *
 * This software is provided 'as-is', without any express or implied warranty.
 * In no event will the authors be held liable for any damages arising from the use of this software.
 *
 * @author    Stefan Verstege <stefan.verstege@newmedia.nl>
 * @copyright Copyright (c) 2008, Stefan Verstege
 * @license   http://dwoo.org/LICENSE   Modified BSD License
 *
 * @link http://www.newmedia.nl/
 *
 * @version 1.1.0
 * @date    2009-07-18
 *
 * @uses the dwoo package from http://dwoo.org
 */
class Dwootemplate extends Dwoo_Core

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    protected $dwoo_data = array();

    /**
     * Constructor for the DwooTemplate engine.
     */
    public function __construct()
    {
        // Call parents constructor
        parent::__construct();

        // Set the config settings
        $this->initialize();

        // Assign some defaults to dwoo
        $CI = get_instance();
        $this->dwoo_data = new Dwoo_Data();

It seems like new \Dwoo_Data() of type object<Dwoo_Data> is incompatible with the declared type array of property $dwoo_data.

        $this->dwoo_data->js_files = array();
        $this->dwoo_data->css_files = array();
        $this->dwoo_data->CI = $CI;
        $this->dwoo_data->site_url = $CI->config->site_url(); // so we can get the full path to CI easily
        $this->dwoo_data->uniqid = uniqid();
        $this->dwoo_data->timestamp = mktime();

        log_message('debug', 'Dwoo Template Class Initialized');
    }

    /**
     * Assign data to dwoo data object.
     *
     * @param string $key
     * @param mixed  $value
     */
    public function assign($key, $value)
    {
        $this->dwoo_data->$key = $value;
    }

    /**
     * Add Javascript files to template.
     *
     * @param string $js
     */
    public function add_js($js)
    {
        $current = $this->dwoo_data->js_files;
        $current[] = $js;
        $this->dwoo_data->js_files = $current;
    }

    /**
     * Add Css stylesheets to template.
     *
     * @param string $css
     */
    public function add_css($css)
    {
        $current = $this->dwoo_data->css_files;
        $current[] = $css;
        $this->dwoo_data->css_files = $current;
    }

    /**
     * Display or return the compiled template
     * Since we assign the results to the standard CI output module
     * you can also use the helper from CI in your templates!!
     *
     * @param string $sTemplate
     * @param bool   $return
     *
     * @return mixed
     */
    public function display($sTemplate, $return = false)
    {
        // Start benchmark
        $CI = get_instance();
        $CI->benchmark->mark('dwoo_parse_start');

        // Check if file exists
        if (!file_exists($this->template_dir.$sTemplate)) {
            $message = sprintf('Template file \'%s\' not found.', $sTemplate);
            show_error($message);
            log_message('error', $message);
        }

        // Create new template
        $tpl = new Dwoo_Template_File($this->template_dir.$sTemplate);

        // render the template
        $template = $this->get($tpl, $this->dwoo_data);

        // Finish benchmark
        $CI->benchmark->mark('dwoo_parse_end');

        // Return results or not ?
        if ($return == false) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

            $CI->output->final_output = $template;
        } else {
            return $template;
        }
    }

    /**
     * Toggle Codeigniter profiler on/off.
     */
    public function enable_profiler($toggle = true)
    {
        $CI = get_instance();
        $CI->output->enable_profiler($toggle);
    }

    /**
     * Set http header.
     *
     * @example $this->output->set_header("HTTP/1.1 200 OK");
     * @example $this->output->set_header('Last-Modified: '.gmdate('D, d M Y H:i:s', $last_update).' GMT');
     *
     * @param string $header
     */
    public function set_header($header)
    {
        $CI = get_instance();
        $CI->output->set_header($header);
    }

    /**
     * Set status header.
     *
     * @example $this->output->set_status_header('401');
     * @example // Sets the header as: Unauthorized
     *
     * @param string $header
     */
    public function set_status_header($header)
    {
        $CI = get_instance();
        $CI->output->set_status_header($header);
    }

    /**
     * Assign the dwootemplate config items to the instance.
     */
    private function initialize()
    {
        $CI = get_instance();
        $CI->config->load('dwootemplate', true);
        $config = $CI->config->item('dwootemplate');
        foreach ($config as $key => $val) {
            $this->$key = $val;
        }
    }
}