hightlight search result

[wanaka]

Hi,

I want to hightlight those word that match with the user specify search term for my search result.

//CCGetParam function will retrieve the value from URL parameter.
if (CCGetParam("s_keyword") != "" ) {
$string_to_split = CCGetParam("s_keyword");
$tok = split(" ", $string_to_split);
$size = sizeof($tok);
for ($i = 0; $i < $size; $i++) {
//Example of changing all matching keywords to a red font.
$ticket_result->ticket_desc->SetValue(str_replace($tok[$i], "<font color = red>". $tok[$i] ."</font>", $ticket_result->ticket_desc->GetValue()));
}
}

However I have noticed this doesn't work well as it is case senstive. For example

if the user key in "cpu threshold" in the search term.

When the result was returned, for example "PROBLEM CPU utilization exceeds configured threshold" only "threshold" was highlighted. Unless the user key in "CPU threshold", then "CPU" and "threshold" will be highlighted.

Apart from this issue, if the user key in the following string in the search box.

PROBLEM abc syslog: CVSDM; minor WARNING Event Code=265; One or more disks .

The hightlight totally not working at all. it will return
or = red>PROBLEM or = red>syslog: or = red>syslog: or = red>CVSDM; MAJOR warning or = red>Event Code=204; or = red>One or more disks have failed.; ; Hardware Address=0/

[rachel2004]

Maybe try using spliti() instead of split()...

[BuzzLY]

I would recommend using preg_replace to do this. And don't use <font> tags. Take the time to learn CSS, and use styles. You're going to need to soon anyway, so might as well do it now.

[Weedpacket]

Hmm, if you're using PHP5 you could use [man]str_ireplace[/man] instead of str_replace. If not, [man]preg_replace[/man] would do the job without too much trouble:

//CCGetParam function will retrieve the value from URL parameter.
if (CCGetParam("s_keyword") != "" ) {
	$string_to_split = CCGetParam("s_keyword");
	$toks = preg_split("/\\w+/", $string_to_split);
	$description = $ticket_result->ticket_desc->GetValue();
	foreach($toks as $token) {
		//Example of changing all matching keywords to a red font.
		$description = preg_replace('/\\b('.$token.')\\b/i', '<span style="color:red">$1</span>', $description);
	}
	$ticket_result->ticket_desc->SetValue($description);
} 

A few other changes:

• I used [man]preg_split/man instead of [man]split/man. What you wrote would have been better written with [man]expode/man (as noted on the split() manual page), but as written it didn't take into account the possibility that there might have been multiple consecutive spaces between words. Then again, there's no check against anything else that might possible be nasty - a "keyword" that is actually a bit of Javascript, for example. So I replaced the single space with "any positive number of whitespace characters".

• I used a foreach loop. Just tidier, I thought.

• I also used an intermediate string instead of setting and resetting the object property over and over.

• The replacement is only on complete words, so that "hold" doesn't result in threshold.

• I replaced the <font> tags (which have been deprecated from the standard for the past seven years).

Looking at the [man]preg_replace/man page, you'll note that it can take arrays as arguments, so the replacement can be taken out of the loop. This may be faster.

[BuzzLY]

yeah, what he said. Except that expode = [man]explode/man

[wanaka]

Hi I have tried but I think it highligh everything

$orig_string2 = "PROBLEM Free Space C: Current=21.77% Threshold=95.00%";
$string_to_split2 = "problem free space";
$toks = preg_split("/\w+/", $string_to_split2);
echo $string_to_split2;
$description = $orig_string2;
foreach($toks as $token) {
//Example of changing all matching keywords to a red font.
$description = preg_replace('/\b('.$token.')\b/i', '<span style="color:red">$1</span>', $description);
}
$orig_string2 = $description;
echo $orig_string2;

[Weedpacket]

Ooh yeah, my mistake. \s, not \w.

[TheDefender]

This may be a direction worth looking into...

Check out this thread here. I was trying to highlight my output in much the same manner, and got some help here. This may not work for you, but it is a route to look into.