Brion Vibber schrieb:
Consider using wfEmptyMsg() for these checks.
This seems a bit roundabout... it may be clearer to just check each regex in turn if it's set, then return true if it doesn't match any of the filters.
Done with r23136.
$regexes .= $regexStart .
str_replace( '/', '\/', preg_replace('|\\\*/|', '/', $build) ) .
Use preg_quote() for these rather than str_replace(); it'll be clearer.
Hmmm, it seems, that preg_quote() is doing too much:
with preg_quote which does not work: "/http://+[a-z0-9_-.]*(wiener-gasometer\.at/index\.html|dispatch\.opac\.d-nb.de|wikipedia\.org)/Si"
instead of
with str_replace which works already in SpamBlacklist: "/http://+[a-z0-9_-.]*(wiener-gasometer.at/index.html|dispatch.opac.d-nb.de|wikipedia.org)/Si"
But I am no regex expert, maybe I missed a parameter/point :-(
Raymond.