[Rt-commit] rt branch 5.0/split-fulltext-searches updated. rt-5.0.5-148-gb0109dcd59

[BPS Git Server]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "rt".

The branch, 5.0/split-fulltext-searches has been updated
       via  b0109dcd596690c50e6bd5bbd1c28ee78a35ae39 (commit)
      from  8788a9923d69c0906b8b5aea6c2721031ef5dad7 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit b0109dcd596690c50e6bd5bbd1c28ee78a35ae39
Author: sunnavy <sunnavy at bestpractical.com>
Date:   Tue Jan 23 17:29:02 2024 -0500

    Revert "Only search Content, not Content and Subject, for better indexing"
    
    This reverts commit 8450f0a9f233d6a761ac22dbdf14926abc54d7fa.
    
    As we support to separate fulltext search terms from others in 8788a9923d,
    the performance penalty is gone for searches like
    
        Subject LIKE 'foo' OR Content LIKE 'foo'
    
    Via searching both Subject and Content, now we can cover cases where
    modified Subject is not included in any attachments.

diff --git a/lib/RT/Search/Simple.pm b/lib/RT/Search/Simple.pm
index f9702361a0..5da09c36d7 100644
--- a/lib/RT/Search/Simple.pm
+++ b/lib/RT/Search/Simple.pm
@@ -280,7 +280,7 @@ sub GuessType {
 sub HandleDefault   {
     my $fts = RT->Config->Get('FullTextSearch');
     if ($fts->{Enable} and $fts->{Indexed}) {
-        return default => "Content LIKE '$_[1]'";
+        return default => "(Subject LIKE '$_[1]' OR Content LIKE '$_[1]')";
     } else {
         return default => "Subject LIKE '$_[1]'";
     }

-----------------------------------------------------------------------

Summary of changes:
 lib/RT/Search/Simple.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


hooks/post-receive
-- 
rt