Проблемы SearchLogic из-за кеширования вызовов методов - PullRequest
Новая
       14

Проблемы SearchLogic из-за кеширования вызовов методов

1 голос
/ 03 ноября 2010

Я пытаюсь использовать searchlogic gem для поиска по нескольким таблицам Post has_many assets.Мне нужно, чтобы он выполнял левые внешние объединения, а не внутренние объединения в случае несуществующего актива.

Из того, что у меня ниже, запрос генерируется с необходимыми внешними соединениями и проходит первые три теста,но терпит неудачу на последнем.Однако, если я только запускаю последний тест, то он проходит.

Причина сбоя заключается в том, что @search_logic_filter var устанавливается только в первом тесте и используется для всех остальных тестов.

Причиной установки @search_logic_filter таким образом является то, что это единственный вызов method_missing, который передает параметр, переданный динамическому поисковому вызову метода Post.title_or_body_or...like("fun")

Есть лилучший способ установить параметр фильтра? 

test "find posts and assets by filter for user" do
  customer = users(:customer)

  create_post_for_user(customer, {:body => "Rails is fun", :tags => "rails ruby"})
  create_post_for_user(customer, {:body => "Fun is what Emacs is all about", :title => "emacs"})


  # File with post
  asset_post = create_post_for_user(customer, {:body => "Ruby is pretty fun too",
                                     :tags => "ruby"})
  asset_post.assets << Asset.new(:upload_file_name => "ruby_tips",
                                :upload_file_size => 100,
                                :upload_content_type => "text")
  asset_post.save

  # search post
  assert_equal 3, Post.find_for_user(customer.id, "fun").size
  assert_equal 2, Post.find_for_user(customer.id, "ruby").size
  assert_equal 1, Post.find_for_user(customer.id, "emacs").size

  # search asset
  puts "about to run last test"
  assert_equal 1, Post.find_for_user(customer.id, "ruby_tips").size
end

class Post < ActiveRecord::Base

  def self.find_for_user(user_id, filter, page=1)
    Post.
      user_id_equals(user_id).
      title_or_body_or_tags_or_assets_upload_file_name_like(filter).all
  end

  class << self
    def method_missing(name, *args, &block)
      if name.to_s =~ /\w+_or_\w+_like$/
        # ** only gets here once **
        @search_logic_filter = args.first
        super
      elsif name == :assets_upload_file_name_like
        # args is [] here which is the reason for the above setting of @search_logic_filter
        named_scope :assets_upload_file_name_like, lambda {
          {:joins => "left outer join assets on posts.id = assets.post_id",
            :conditions => "assets.upload_file_name like '%#{@search_logic_filter}%'"}
        }
        assets_upload_file_name_like
      else
        super
      end
    end
  end
end

** update Это запрос, выполняемый для окончательного теста.Обратите внимание, что параметром upload_file_name является «fun», а не «ruby_tips».Параметр 'fun' существует для всех тестов для столбца upload_file_name, но он имеет значение только для последнего теста. 

SELECT `posts`.* 
FROM `posts` 
  left outer join assets 
    on posts.id = assets.post_id 
WHERE (
  ((posts.title LIKE '%ruby_tips%') OR (posts.body LIKE '%ruby_tips%') OR (posts.tags LIKE '%ruby_tips%') OR (assets.upload_file_name like '%fun%')) 
  AND (posts.user_id = 20549131)
)

1 Ответ

1 голос
/ 03 ноября 2010

Вы не должны объявлять named_scope assets_upload_file_name_like таким образом. При первом вызове именованная область действия assets_upload_file_name_like определяется значением для :conditions, сгенерированным в соответствии со значением @search_logic_filter в то время. Вместо этого вы должны установить параметр на lambda.

Также нет необходимости использовать method_missing. Просто объявите named_scope в классе Post. В качестве бонуса, запрос должен быть отфильтрован для защиты от атак SQL-инъекций. 

class Post < ActiveRecord::Base
  named_scope :assets_upload_file_name_like, lambda { |file_name| {
    :joins => "left outer join assets on posts.id = assets.post_id",
    # Prevent SQL injection.
    :conditions => ["assets.upload_file_name like ?", "%#{file_name}%"]
  }}
end
...