Warning: preg_match() [function.preg-match]: Compilation failed

extrasolar

Hi Folkes,

I get the error Warning: preg_match() [function.preg-match]: Compilation failed, in detail I get this error in my namespace tree:

Warning: preg_match() [function.preg-match]: Compilation failed: missing terminating ] for character class at offset 17 in /var/kunden/webs/myuser/lfep/inc/search.php on line 122

search.php looks like this:

Lines 107 - 126

/**
 * Build the browsable index of pages
 *
 * $opts['ns'] is the currently viewed namespace
 *
 * @author  Andreas Gohr <andi@splitbrain.org>
 */
function search_index(&$data,$base,$file,$type,$lvl,$opts){
    global $conf;
    $opts = array(
        'pagesonly' => true,
        'listdirs' => true,
        'listfiles' => !$opts['nofiles'],
        'sneakyacl' => $conf['sneaky_index'],
        // Hacky, should rather use recmatch
        'depth' => preg_match('#^'.$file.'(/|$)#','/'.$opts['ns']) ? 0 : -1
    );

    return search_universal($data, $base, $file, $type, $lvl, $opts);
}

Any idea, where I have to close the bracket? What is the solution here? Thanks in advance!

Regards

lupo49

Since when do you get the error message? What have you done before?

extrasolar

I don't know. I never touched the search.php. You may check it on lfep.de in media manager, for instance.

turnermm

Try preg_quote($file). The $file name is part of your regular expression and it if has any regular expression markup you have to escape it. Though why a filename should have brackets is another question.

extrasolar

I'd like to say thank you to turnermm but also to all others helped with thinking about a solution. Last but not least only subsituting preg_match with preg_quote solved this issue! I couldn't find any disadvantages using preg_quote in search.php until now.

Regards!

michitux

I've applied the fix in commit 1c6c1c6c7, it's just a simple change in one line so you can easily manually apply this fix in your installation.

Global DokuWiki Links