up vote 0 down vote favorite
share [fb]

I would like to append html to an item in my array before echoing it on my page and am unsure how to go about doing it.

My data is put into an array like so:

$query = $this->db->get();

        foreach ($query->result() as $row) {
            $data = array(
                'seo_title' => $row->seo_title,
                'seo_description' => $row->seo_description,
                'seo_keywords' => $row->seo_keywords,
                'category' => $row->category,
                'title' => $row->title,
                'intro' => $row->intro,
                'content' => $row->content,
                'tags' => $row->tags
            );
        }

        return $data;

I would like to perform the following on my 'tags' before returning the data to my view:

$all_tags = explode( ',' , $row->tags );
foreach ( $all_tags as $one_tag ){
    echo '<a href="/search/'. $one_tag .'">' . $one_tag . '</a>';

The reason for doing this is that the tags in my database contain no html and are simply separated by commas like so news,latest,sports and I want to convert them into <a href="/search/sports">sports</a> ...

My reason for doing this here rather than when I echo the data is that I don't want to repeat myself on every page.

link|improve this question
96% accept rate
feedback

2 Answers

up vote 2 down vote accepted

You could just create a function to be used everyhwere you are including tags in your output:

function formatTags($tags) {
    $tmp = explode(',', $tags);
    $result = "";
    foreach ($tmp as $t) {
       $result .= sprintf('<a href="/search/%s">%s</a>',
            urlencode(trim($t)), htmlentities(trim($t)));
    }
    return $result;
}

And whenever you do something like echo $tags; you do echo formatTags($tags); instead. View code should be separated from model code, which is why I would advise to not put HTML inside your array.

link|improve this answer
That is just amazing... Thank you!!!! – hairynuggets Nov 4 at 12:13
feedback
up vote 0 down vote

Well first of all you're overwriting $data with every run of the loop so only the final result row will be listed.

Once that's out of the way (fix with $data[] = ...), try this:

...
'tags' => preg_replace( "/(?:^|,)([^,]+)/", "<a href=\"/search/$1\">$1</a>", $row->tags);
...
link|improve this answer
feedback

Your Answer

 
or
required, but never shown

Not the answer you're looking for? Browse other questions tagged or ask your own question.