geokit-rails的未定义方法'merge_conditions'

时间:2011-06-20 09:45:17

标签: ruby-on-rails ruby-on-rails-3 geolocation geokit

我正在玩Geokit-Rails(https://github.com/andre/geokit-rails),我需要查找距离原点10英里范围内的所有记录。这是我的错误:

Company.find(:all, :origin=>'New York, NY', :within=>10)
NoMethodError: undefined method `merge_conditions' for #<Class:0x0000010832f740>
from /Users/san/.rvm/gems/ruby-1.9.2-p0/gems/activerecord-3.0.1/lib/active_record/base.rb:1016:in 'method_missing'
from /Users/san/Documents/sanj/app/vendor/plugins/geokit-rails/lib/geokit-rails/acts_as_mappable.rb:349:in `apply_bounds_conditions'
from /Users/san/Documents/sanj/app/vendor/plugins/geokit-rails/lib/geokit-rails/acts_as_mappable.rb:260:in `prepare_for_find_or_count'
from /Users/san/Documents/sanj/app/vendor/plugins/geokit-rails/lib/geokit-rails/acts_as_mappable.rb:152:in `find'
from (irb):4
from /Users/san/.rvm/gems/ruby-1.9.2-p0/gems/railties-3.0.1/lib/rails/commands/console.rb:44:in `start'
from /Users/san/.rvm/gems/ruby-1.9.2-p0/gems/railties-3.0.1/lib/rails/commands/console.rb:8:in `start'
from /Users/san/.rvm/gems/ruby-1.9.2-p0/gems/railties-3.0.1/lib/rails/commands.rb:23:in `<top (required)>'
from script/rails:6:in `require'
from script/rails:6:in `<main>'

我在Company.rb中有acts_as_mappable,我已将列latlng迁移到公司。我哪里错了?

1 个答案:

答案 0 :(得分:5)

我自己偶然发现了这件事。

这完全是一个1分钟的黑客修复,但由于它是一个相对无害的方法插入,不会影响它的使用范围,我只是将方法直接添加到插件中。

merge_conditions是rails 2.3.8中的deprecated ActiveRecord :: Base方法。

插入app / vendor / plugins / geokit-rails / lib / geokit-rails / acts_as_mappable.rb

  private
    def merge_conditions(*conditions)
    segments = []

    conditions.each do |condition|
      unless condition.blank?
        sql = sanitize_sql(condition)
        segments << sql unless sql.blank?
      end
    end

    "(#{segments.join(') AND (')})" unless segments.empty?
    end

我不确定是什么替换了这种手动SQL条件链接(可能是:范围?),所以geokit代码有点旧,但有时只需要1分钟的黑客修复,尤其是当黑客攻击时包含在较旧的外部库中......

我想我会发现原始代码是否在其他地方调用merge_conditions。也许我会花时间找出更新代码的“正确”方法,如果它仍在积极管理中。

我们not the only ones遇到了这个问题。

必须像第一条评论一样尝试myModel.geo_scope,bblzzz ......