Ошибка при объединении областей поиска

Question

У меня есть веб-сервис, который позволяет клиентам искать статьи с параметрами запроса. Он работает нормально, если включен только один параметр, но не работает, если я объединяю search_query а также category, Это основано на Comfortable_Mexican_Sofa, где находится for_category. Даже если я удалю order Заявление, я получаю эту ошибку.

ошибка

PG::InvalidColumnReference: ERROR: для SELECT DISTINCT выражения ORDER BY должны появляться в списке выбора LINE 1: ...ms_categories"." Label" = 'Новости компании' ORDER BY pg_search_... ^: SELECT DISTINCT "comfy_cms_pages".* ИЗ "comfy_cms_pages" ВНУТРЕННЕЕ СОЕДИНЕНИЕ "comfy_cms_categorizations" ВКЛ. "Comfy_cms_categorizations". "Ategorized_id "=" comfy_cms_pages "." Id "И" comfy_cms_categorizations "." "comfy_cms_categorizations". "category_id" INNER JOIN (ВЫБЕРИТЕ "comfy_cms_pages"."id" AS pg_search_id, (ts_rank((to_tsvector('simple', coalesce("comfy_cms_pages"."content_cache"::text, '), content_cache "||,') to_tsvector ('simple', coalesce ("comfy_cms_pages". "label":: text, ''))), (to_tsquery ('simple', '' '' || 'austin' || '' '' || ': ')), 0)) AS rank ОТ "comfy_cms_pages" WHERE (((to_tsvector('simple', coalesce("comfy_cms_pages"."Content_cache":: text,' ')) || to_tsvector (' simple ', coalesce ("comfy_cms_pages". "label":: text, ''))) @@ (to_t squery ('simple', '' '' || 'Остин' || '' '' || ': '))))) pg_search_comfy_cms_pages ON "comfy_cms_pages". "id" = pg_search_comfy_cms_pages.pg_search_id ГДЕ (layout_id = '1' И is_published = 't') И "comfy_cms_category" pg_search_comfy_cms_pages.rank DESC, "comfy_cms_pages"."id" ASC, "comfy_cms_pages"."creation_at" DESC

приложение / модели /article.rb

class Article < Comfy::Cms::Page
  cms_is_categorized
  include PgSearch


  pg_search_scope :search_by_keywords, against: [:content_cache, :label], using: { tsearch: { any_word: true, prefix: true } }

Приложение / Команды /search_articles_command.rb

class SearchArticlesCommand
  def initialize(params = {})
    @since = params[:since_date]
    @keys = params[:search_query]
    @category = params[:category]
  end

  def execute
    Article.unscoped do
      query = if @since.present?
                Article.article.since_date(@since)
              else
                Article.published_article
              end
      query = query.for_category(@category) if @category.present?
      query = query.search_by_keywords(@keys) if @keys.present?
      query.where('').order(created_at: :desc)
    end
  end
end

удобный-мексиканский-диван / Библиотека / comfortable_mexican_sofa / расширение / is_categorized.rb

module ComfortableMexicanSofa::IsCategorized

  def self.included(base)
    base.send :extend, ClassMethods
  end

  module ClassMethods
    def cms_is_categorized
      include ComfortableMexicanSofa::IsCategorized::InstanceMethods

      has_many :categorizations,
        :as         => :categorized,
        :class_name => 'Comfy::Cms::Categorization',
        :dependent  => :destroy
      has_many :categories,
        :through    => :categorizations,
        :class_name => 'Comfy::Cms::Category'

      attr_accessor :category_ids

      after_save :sync_categories

      scope :for_category, lambda { |*categories|
        if (categories = [categories].flatten.compact).present?
          self.distinct.
            joins(:categorizations => :category).
            where('comfy_cms_categories.label' => categories)
        end
      }
    end
  end

  module InstanceMethods
    def sync_categories
      (self.category_ids || {}).each do |category_id, flag|
        case flag.to_i
        when 1
          if category = Comfy::Cms::Category.find_by_id(category_id)
            category.categorizations.create(:categorized => self)
          end
        when 0
          self.categorizations.where(:category_id => category_id).destroy_all
        end
      end
    end
  end
end

ActiveRecord:: Base.send: включите, ComfortableMexicanSofa::IsCategorized

Обновленная ошибка

PG::SyntaxError: ERROR:  syntax error at or near "."
LINE 4: ...e = 'Class' AND categorized_id = 'comfy_cms_pages'.'id' AND ...
                                                             ^
: SELECT "comfy_cms_pages".* FROM "comfy_cms_pages" INNER JOIN (SELECT "comfy_cms_pages"."id" AS pg_search_id, (ts_rank((to_tsvector('simple', coalesce("comfy_cms_pages"."content_cache"::text, '')) || to_tsvector('simple', coalesce("comfy_cms_pages"."label"::text, ''))), (to_tsquery('simple', ''' ' || 'austin' || ' ''' || ':*')), 0)) AS rank FROM "comfy_cms_pages" WHERE (((to_tsvector('simple', coalesce("comfy_cms_pages"."content_cache"::text, '')) || to_tsvector('simple', coalesce("comfy_cms_pages"."label"::text, ''))) @@ (to_tsquery('simple', ''' ' || 'austin' || ' ''' || ':*'))))) pg_search_comfy_cms_pages ON "comfy_cms_pages"."id" = pg_search_comfy_cms_pages.pg_search_id WHERE "comfy_cms_pages"."layout_id" = $1 AND "comfy_cms_pages"."is_published" = $2 AND (
        EXISTS (
          SELECT 1 FROM categorizations
          WHERE categorized_type = 'Class' AND categorized_id = 'comfy_cms_pages'.'id' AND category_id IN (2)
        ))  ORDER BY pg_search_comfy_cms_pages.rank DESC, "comfy_cms_pages"."id" ASC

рабочее решение, но не область и должны быть осторожны, чтобы его называли

  def self.for_category(_category)
    Comfy::Cms::Categorization.includes(:category).references(:category).select(:categorized).pluck(:categorized_id)
    find(ids)
  end

Answer 1

Я смог решить эту проблему, применив reorder к результату результата pg_search.

search_command.rb

class SearchArticlesCommand
  def initialize(params = {})
    @since = params['since_date']
    @keys = params['search_query']
    @category = params['category']
  end

  def execute
    Article.unscoped do
      query = Article.article
      query = if @since.present?
                query.since_date(@since)
              else
                query.published
              end
      query = query.for_category(@category) if @category.present?
      query = query.search_by_keywords(@keys).reorder('updated_at DESC') if @keys.present?
      query
    end
  end
end

Я также переопределил for_category (не требуется)

article.rb

  scope :for_category, (lambda do |category|
    published
      .joins(:categories)
      .group(:id)
      .where('comfy_cms_categories.label' => category)
      .select('comfy_cms_pages.*')
  end)

Answer 2

Я думаю, что лучше переопределить for_category встроенный фильтр вашей CMS. Слишком много соединений в этом запросе.

Override for_category как это:

scope :for_category, lambda { |*categories|
  if (categories = [categories].flatten.compact).present?
    self_ids = "{connection.quote_table_name(self.table_name)}.#{connection.quote_column_name(self.primary_key)}"
    self.where(
      "EXISTS (" +
        Comfy::Cms::Categorization.select('1').
          where(categorized_type: self.name).
          where('categorized_id' => self_ids).
          where(category_id: Comfy::Cms::Category.where(label: categories).pluck(:id)).to_sql +
      ")"
    )
  end
}

Подробнее о SQL EXISTS Использование в Rails вы можете прочитать в моем Rails: SQL EXISTS краткое руководство.

Подробнее о том, почему вы столкнулись с этой ошибкой, вы можете прочитать в вопросе и ответить здесь.

В частности, pg_search хочет упорядочить ваши результаты по рангу. А также for_category хочет выбрать только отдельные поля статьи и не заботится о поисковом рейтинге. Изменение его кода для использования простого EXISTS вместо сложного JOIN запрос это исправит.