code duplication advice

[eluser]jwindhorst[/eluser]
I realize this thread has gotten pretty old, but I rewrote the above function as an extension to the loader class, and thought it might be useful, so here it is:

Code:

class MY_Loader extends CI_Loader {

    /**
     * attempting to overload this function to include our header and footer
     **/
    function view($views, $vars = array(), $return = FALSE)
    {

        // pass any header vars on to the header    
        $header_vars = (isset($vars['header_data'])) ? $vars['header_data'] : array();

        // pass any footer vars on to the header    
        $footer_vars = (isset($vars['header_data'])) ? $vars['header_data'] : array();

        $doc = $this->_ci_load(array('_ci_view' => 'layout/header', '_ci_vars' => $this->_ci_object_to_array($header_vars), '_ci_return' => $return));

        // if it's an array we need to loop through them
        if(is_array($views))
            foreach($views as $view)        
                $doc .= $this->_ci_load(array('_ci_view' => $view, '_ci_vars' => $this->_ci_object_to_array($vars), '_ci_return' => $return));
        else
            $doc .= $this->_ci_load(array('_ci_view' => $views, '_ci_vars' => $this->_ci_object_to_array($vars), '_ci_return' => $return));

        $doc .= $this->_ci_load(array('_ci_view' => 'layout/footer', '_ci_vars' => $this->_ci_object_to_array($footer_vars), '_ci_return' => $return));
        return $doc;
    }

}

Basic usage is that anything if you set an array inside your views data array called header_data, it will get passed to the header, same with footer_data getting passed to the footer.

I haven't found a use for footer_data yet, but header_data I usually pass a js_scripts array that sources in all of my javascripts. This way they are only included on the pages that use them, and they still get to be sourced in the head tag instead of hacked into the body somewhere.

Cheers!