Проблема с has_many :through и валидацией

[valodzka]

Модели:

Address: has_many :delivery_settings, :through => :delivery_address
DeliverySetting: has_many :addresses, :through => :delivery_addresses
DeliveryAddress: belongs_to :delivery_setting,  belongs_to :address,  validates_uniqueness_of :address_id, :scope => :delivery_setting_id

При попытке добавить вторую связь (которая не проходит валидацию):

can't convert ActiveRecord::Error into String
 vendor/plugins/russian/lib/russian/active_record_ext/custom_error_message.rb:47:in `+'

@errors выглядят так: {"address_id"=>[#<ActiveRecord::Error:0x26d39e8 @type=:taken, @message=:taken, @options={:value=>6}, @attribute=:address_id, @base=#<DeliveryAddress id: nil, address_id: 6, delivery_setting_id: 4>>]}

[valodzka]

добавление

elsif msg.kind_of? ActiveRecord::Error
    full_messages << msg.full_message

эту проблему решает

[yaroslav]

@valodzka а добавление куда? ты имеешь в виду что для Rails < 2.3.4 возникают ошибки (custom_error_message.rb L89 и ниже)?

Пришли патч плс если уже разобрался что к чему

[valodzka]

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

- vendor/plugins/russian/lib/russian/active_record_ext/custom_error_message.rb -
index 3b480c3..dbe1cd3 100644
@@ -41,13 +41,16 @@ module ActiveRecord
             full_messages << msg
           elsif msg =~ /^\^/
             full_messages << msg[1..-1]
+     elsif msg.kind_of? ActiveRecord::Error
+       full_messages << msg.full_message
      else
        full_messages << @base.class.human_attribute_name(attr) + " " + msg
      end
    end
  end

[yaroslav]

Спасибо, пушнул в мастер

Проверь что все ок, отпиши — если да, сделаю релиз

[valodzka]

Всё ок, только это не для Rails < 2.3.4, это именно 2.3.4 offtop: Я делал локализацию стандартного рэйлсвого time_zone_select, возможно стоит добавить в Russian: http://gist.github.com/219923

[yaroslav]

Штука полезная, как раз нужна :) Попробуй оформить плагин для перевода отдельно? Можно будет его пропихнуть или в Rails Core, или в плагин на github.com/rails или вроде того.

[valodzka]

Сделаю когда-нибудь, хотя кода там на полтора метода, IMHO логичнее с общей локализацией