[Tefra]

I have created a custom terminology/glossary mod for vb, and i have also included a simple function to auto link to other terms. but i have two major issues. It breaks html tags.

Any help is appreciated, the preg_replace in line 15 is the vbulletin method to highlight strings, if that helps...

PHP Code:

function terms_build_relation($message, $intermid = 0)
{
    global $db;

    $words         = array();
    $keywords     = $db->escape_string(strip_tags($message));
    $sql         = $db->query_read("SELECT term_id, term_title FROM ".VBSED_PREFIX."terms WHERE MATCH(term_title) AGAINST('$keywords') ".iif($intermid, "AND term_id <> $intermid", '')." AND term_active = 1");
                    
    while($sim = $db->fetch_array($sql)) {    $words[$sim['term_id']] = $sim['term_title']; }
    
    $message = str_replace('\"', '"', $message);
    foreach ($words AS $key => $rep)
    {
        if(in_array(strtolower($rep), array('http', 'https', 'ftp'))) continue; //Not Good Enough
        $message = preg_replace('#(?<=[\s"\]>()]|^)(' . $rep . ')(([.,:;-?!()\s"<\[]|$))#siU', '[term='.$key.']\\1[/term]\\2', $message, 1);
    }
    $message = ereg_replace('\\[term=([0-9]+)\\]([^\\[]*)\\[/term\\]', '<em><a href="'.VBSED_TERMS_URL.'item=\\1">\\2</a></em>', $message);
    $message = ereg_replace('\\[term=([0-9]+)\\]([^\\[]*)\\[/term\\]', '\\2', $message); // BUG Step 1!!! Remove any open term bbcode
    return $message;    
} 


The second issue is that sometimes it breaks into other autolinked terms, that's why in the end i replace any open term bbcode.

For example, there are the terms "system" and "system registry". In a 3rd term definition there is the string "system registry". In the first pass it replaces the "system" term but in the second pass it tries to replace the whole string "system registry" Here comes in hand the removal of the empty term bbcode. This isn't a great issue as the first one as i believe if the first issue is fixed this will also be gone.

You can check it out here
Here is an example breaking img tag with the tag parameter.