Alrighty, now we properly remove old blocks before inserting the new one. (Bug 10080...

[lhc/web/wiklou.git] / includes / SearchOracle.php

diff --git a/includes/SearchOracle.php b/includes/SearchOracle.php
index de3ed55..bf9368d 100644 (file)
--- a/includes/SearchOracle.php
+++ b/includes/SearchOracle.php
@@ -18,13 +18,15 @@
 # http://www.gnu.org/copyleft/gpl.html
 
 /**
- * Search engine hook base class for Oracle (ConText).
- * @addtogroup Search
+ * @file
+ * @ingroup Search
  */
 
+/**
+ * Search engine hook base class for Oracle (ConText).
+ * @ingroup Search
+ */
 class SearchOracle extends SearchEngine {
-       var $strictMatching = false;
-
        function __construct($db) {
                $this->db = $db;
        }
@@ -73,6 +75,8 @@ class SearchOracle extends SearchEngine {
         * @private
         */
        function queryNamespaces() {
+               if( is_null($this->namespaces) )
+                       return '';
                $namespaces = implode(',', $this->namespaces);
                if ($namespaces == '') {
                        $namespaces = '0';
@@ -96,7 +100,7 @@ class SearchOracle extends SearchEngine {
         * @private
         */
        function queryRanking($filteredTerm, $fulltext) {
-               return '';
+               return ' ORDER BY score(1)';
        }
 
        /**
@@ -141,38 +145,33 @@ class SearchOracle extends SearchEngine {
        }
 
        /** @todo document */
-       function parseQuery( $filteredText, $fulltext ) {
+       function parseQuery($filteredText, $fulltext) {
                global $wgContLang;
                $lc = SearchEngine::legalSearchChars();
-               $searchon = '';
                $this->searchTerms = array();
 
                # FIXME: This doesn't handle parenthetical expressions.
                $m = array();
-               if( preg_match_all( '/([-+<>~]?)(([' . $lc . ']+)(\*?)|"[^"]*")/',
-                         $filteredText, $m, PREG_SET_ORDER ) ) {
-                       foreach( $m as $terms ) {
-                               if( $searchon !== '' ) $searchon .= ' ';
-                               if( $this->strictMatching && ($terms[1] == '') ) {
-                                       $terms[1] = '+';
-                               }
-                               $searchon .= $terms[1] . $wgContLang->stripForSearch( $terms[2] );
-                               if( !empty( $terms[3] ) ) {
+               $q = array();
+
+               if (preg_match_all('/([-+<>~]?)(([' . $lc . ']+)(\*?)|"[^"]*")/',
+                         $filteredText, $m, PREG_SET_ORDER)) {
+                       foreach($m as $terms) {
+                               $q[] = $terms[1] . $wgContLang->stripForSearch($terms[2]);
+
+                               if (!empty($terms[3])) {
                                        $regexp = preg_quote( $terms[3], '/' );
-                                       if( $terms[4] ) $regexp .= "[0-9A-Za-z_]+";
+                                       if ($terms[4])
+                                               $regexp .= "[0-9A-Za-z_]+";
                                } else {
-                                       $regexp = preg_quote( str_replace( '"', '', $terms[2] ), '/' );
+                                       $regexp = preg_quote(str_replace('"', '', $terms[2]), '/');
                                }
                                $this->searchTerms[] = $regexp;
                        }
-                       wfDebug( "Would search with '$searchon'\n" );
-                       wfDebug( 'Match with /\b' . implode( '\b|\b', $this->searchTerms ) . "\b/\n" );
-               } else {
-                       wfDebug( "Can't understand search query '{$filteredText}'\n" );
                }
 
-               $searchon = $this->db->strencode($searchon);
-               $field = $this->getIndexField( $fulltext );
+               $searchon = $this->db->strencode(join(',', $q));
+               $field = $this->getIndexField($fulltext);
                return " CONTAINS($field, '$searchon', 1) > 0 ";
        }
 
@@ -193,6 +192,8 @@ class SearchOracle extends SearchEngine {
                                'si_title' => $title,
                                'si_text' => $text
                        ), 'SearchOracle::update' );
+               $dbw->query("CALL ctx_ddl.sync_index('si_text_idx')");
+               $dbw->query("CALL ctx_ddl.sync_index('si_title_idx')");
        }
 
        /**
@@ -202,7 +203,7 @@ class SearchOracle extends SearchEngine {
         * @param int $id
         * @param string $title
         */
-    function updateTitle( $id, $title ) {
+       function updateTitle($id, $title) {
                $dbw = wfGetDB(DB_MASTER);
 
                $dbw->update('searchindex',
@@ -213,6 +214,9 @@ class SearchOracle extends SearchEngine {
        }
 }
 
+/**
+ * @ingroup Search
+ */
 class OracleSearchResultSet extends SearchResultSet {
        function __construct($resultSet, $terms) {
                $this->mResultSet = $resultSet;
@@ -229,12 +233,8 @@ class OracleSearchResultSet extends SearchResultSet {
 
        function next() {
                $row = $this->mResultSet->fetchObject();
-               if( $row === false) {
+               if ($row === false)
                        return false;
-               } else {
-                       return new SearchResult($row);
-               }
+               return new SearchResult($row);
        }
 }
-
-?>