Merge pull request #1720 from codidact/0valt/387/post-title-search

Author: ArtOfCode-

app/models/application_record.rb

@@ -2,12 +2,12 @@ class ApplicationRecord < ActiveRecord::Base
   self.abstract_class = true
 
   def self.fuzzy_search(term, **cols)
-    sanitized = sanitize_for_search term, **cols
+    sanitized = sanitize_for_search(term, **cols)
     select(Arel.sql("`#{table_name}`.*, #{sanitized} AS search_score"))
   end
 
   def self.match_search(term, **cols)
-    sanitized = sanitize_for_search term, **cols
+    sanitized = sanitize_for_search(term, **cols)
     select(Arel.sql("`#{table_name}`.*, #{sanitized} AS search_score")).where(sanitized)
   end
 

app/models/post.rb

@@ -76,7 +76,7 @@ def self.accessible_to(user)
   # @param term [String] the search term
   # @return [ActiveRecord::Relation<Post>]
   def self.search(term)
-    match_search term, posts: :body_markdown
+    match_search term, posts: [:body_markdown, :title]
   end
 
   def self.by_slug(slug, user)

db/migrate/20250801124236_add_post_titles_to_posts_fts.rb

@@ -0,0 +1,7 @@
+# initial post FTS did not include titles
+class AddPostTitlesToPostsFts < ActiveRecord::Migration[7.2]
+  def change
+    remove_index :posts, :body_markdown, if_exists: true, type: :fulltext
+    add_index :posts, [:body_markdown, :title], if_not_exists: true, type: :fulltext
+  end
+end

db/schema.rb

@@ -507,7 +507,7 @@
     t.datetime "locked_at", precision: nil
     t.datetime "locked_until", precision: nil
     t.index ["att_source"], name: "index_posts_on_att_source"
-    t.index ["body_markdown"], name: "index_posts_on_body_markdown", type: :fulltext
+    t.index ["body_markdown", "title"], name: "index_posts_on_body_markdown_and_title", type: :fulltext
     t.index ["category_id"], name: "index_posts_on_category_id"
     t.index ["close_reason_id"], name: "index_posts_on_close_reason_id"
     t.index ["community_id"], name: "index_posts_on_community_id"

test/models/application_record_test.rb

@@ -0,0 +1,9 @@
+require 'test_helper'
+
+class ApplicationRecordTest < ActiveSupport::TestCase
+  test 'fuzzy_search should correctly search records' do
+    results = Post.fuzzy_search('Q2', posts: [:body_markdown, :title]).map(&:id)
+    post_ids = posts.select { |p| /^Q\d+/.match?(p.title) }.map(&:id)
+    assert(post_ids.all? { |id| results.include?(id) })
+  end
+end