Ahh great. I mixed up my definition of preg_match's output and thought it was the...
authorDaniel Friesen <dantman@users.mediawiki.org>
Thu, 8 May 2008 09:29:48 +0000 (09:29 +0000)
committerDaniel Friesen <dantman@users.mediawiki.org>
Thu, 8 May 2008 09:29:48 +0000 (09:29 +0000)
includes/Database.php

index c871137..f2bb4ba 100644 (file)
@@ -1380,7 +1380,7 @@ class Database {
                # Note that we use a whitespace test rather than a \b test to avoid
                # any remote case where a word like on may be inside of a table name
                # surrounded by symbols which may be considered word breaks.
-               if( preg_match( '/(^|\s)(JOIN|ON)(\s|$)/i', $name ) !== false ) return $name;
+               if( preg_match( '/(^|\s)(JOIN|ON)(\s|$)/i', $name ) !== 0 ) return $name;
                
                # Split database and table into proper variables.
                # We reverse the explode so that database.table and table both output