DTE BreadCrumb Class

[eluser]ericsodt[/eluser]
[quote author="morozgrafix" date="1231984491"]Hello Dean,

Thank you for quick reply. I think you missing session_start(); call on line 2 of Breadcrumb.php in your latest version it should be this:

Code:

<?php  if (!defined('APPPATH')) exit('No direct script access allowed');
session_start();

/*
* @Author: Dean Ericson
* @Email: [email protected]
*
*/
//...

I also was looking around to be able to change delimiter within same project. For example to be able to specify individual delimiters for different pages. I have modified setBreadCrumb method in Breadcrumb.php (I know this may not be most efficient solution, but it works for me)

Code:

//...
function setBreadCrumb($bcObjArr){
    $this->title=$bcObjArr['title'];
    $this->url=$bcObjArr['url'];
    $this->isRoot=$bcObjArr['isRoot'];
    if(isset($bcObjArr['delimiter'])){
        $this->delimiter=$bcObjArr['delimiter'];    
    }
    
    $CI =& get_Instance();
    $CI->breadcrumblist->add($this);
    if(isset($this->delimiter)){
        $CI->breadcrumblist->setDelimiter($this->delimiter);
    }
    $_SESSION['breadCrumbList'] = $CI->breadcrumblist->getBreadCrumbs();        
}
//...

Now, delimiter could be specified within $bc array in my controller like this:

Code:

//...
$bc = array (
    'title' => 'Latest Reports',
    'url' => '/qareports',
    'delimiter' => ' any_new_delimiter ',
    'isRoot' => true
);
//...

With this change delimiter could be set from $bc array, if not set here it would be picked up from config.php (your latest change in 1.6 release) and then if not found in config.php would be defaulted to hardcoded setting.

Oh, and one more thing, I've noticed that word delimiter was misspelled in your code, it was spelled as "delimeter" - I ended up correcting spelling throughout the code.

Once again thank you for sharing your code and your quick replies.

Sergey.[/quote]

Sergey,

I realized I spelled it wrong on version 1.5, but didn't have the time to go through and fix it (being lazy really). I will make the changes you have brought to light. Yes I am missing the session_start() at the top of the page... I made the update quickly last night and left that out... no excuse really. I will also incorporate your changes into the next release (1.7) which I should be able to get out tonight or tomorrow depending on my work schedule.

Thank you for your feedback!!