[deploy] Revert "serialize search score for articles (#7053)" (#7065)

This reverts commit e16a0f8c34.
This commit is contained in:
Molly Struve 2020-04-03 09:04:27 -05:00 committed by GitHub
parent 1d069d8ed1
commit 2f57952a1a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 7 additions and 7 deletions

View file

@ -427,11 +427,6 @@ class Article < ApplicationRecord
spaminess_rating: BlackBox.calculate_spaminess(self))
end
def search_score
calculated_score = hotness_score.to_i + ((comments_count * 3).to_i + positive_reactions_count.to_i * 300 * user.reputation_modifier * score.to_i)
calculated_score.to_i
end
private
def delete_related_objects
@ -439,6 +434,11 @@ class Article < ApplicationRecord
Search::RemoveFromIndexWorker.new.perform("ordered_articles_#{Rails.env}", index_id)
end
def search_score
calculated_score = hotness_score.to_i + ((comments_count * 3).to_i + positive_reactions_count.to_i * 300 * user.reputation_modifier * score.to_i)
calculated_score.to_i
end
def tag_keywords_for_search
tags.pluck(:keywords_for_search).join
end

View file

@ -8,7 +8,7 @@ module Search
:comments_count, :experience_level_rating, :experience_level_rating_distribution,
:featured, :featured_number, :hotness_score, :language,
:main_image, :path, :positive_reactions_count, :published,
:published_at, :reactions_count, :reading_time, :search_score, :score, :title
:published_at, :reactions_count, :reading_time, :score, :title
# video_duration_in_minutes in Elasticsearch is mapped as an integer
# however, it really is a string in the format 00:00 which is why we

View file

@ -12,7 +12,7 @@ RSpec.describe Search::ArticleSerializer do
expect(data_hash[:user]).to eq(user_data)
expect(data_hash.dig(:organization, :id)).to eq(organization.id)
expect(data_hash.dig(:flare_tag_hash, :name)).to eq(tag.name)
expect(data_hash.keys).to include(:id, :body_text, :hotness_score, :search_score)
expect(data_hash.keys).to include(:id, :body_text, :hotness_score)
end
it "creates valid json for Elasticsearch", elasticsearch: true do