I'm using the attachment_fu plugin to upload large files.  I'm using the
latest version and I'm running it under Rails 2.3.5.

My file upload doesn't work and I get this error in the log:

DEPRECATION WARNING: ActiveRecord::Errors.default_error_messages has
been deprecated. Please use
I18n.translate('activerecord.errors.messages').. (called from
default_error_messages at
C:/ruby/1.8.6/lib/ruby/gems/1.8/gems/activerecord-2.3.4/lib/active_record/validations.rb:131)

At first I guessed that there was an error in my code, and the system is
trying to display an error message, but failing.  In fact, it turns out
that there's nothing wrong with my code!

To fix the problem I edited
vendor/plugins/attachment_fu/lib/technoweenie/attachment_fu.rb around
line 385.  The original code was:

def attachment_attributes_valid?
    [:size, :content_type].each do |attr_name|
        enum = attachment_options[attr_name]
        errors.add attr_name,
            ActiveRecord::Errors.default_error_messages[:inclusion]
            unless enum.nil? || enum.include?(send(attr_name))
    end
end


I changed it to:

def attachment_attributes_valid?
    [:size, :content_type].each do |attr_name|
        enum = attachment_options[attr_name]
        errors.add attr_name,
            # ActiveRecord::Errors.default_error_messages[:inclusion]
            # unless enum.nil? || enum.include?(send(attr_name))
            # This is an internationalised version of "size is not
included in the list" etc
            inclusion_error_message =
                I18n.translate"activerecord.errors.messages.too_short"
            errors.add attr_name, inclusion_error_message unless
enum.nil? ||
                enum.include?(send(attr_name))
    end
end



And now my upload works.

If the locale speaks English, then
    ActiveRecord::Errors.default_error_messages[:inclusion]
becomes
    "is not included in the list"

The code seems to be building a list of error messages "size is not
included in the list", "content type is not included in the list" and so
on.  It will use them if any of those fields are missing.  (In my case,
they were all present, so the messages are not needed.

However, it's using a deprecated method to build the list, and the
result is a fatal error.  I think my replacement code does the same
thing, but using the modern style.

Interesting use of the term "deprecated" by the way.  In other systems,
it means "you can use this feature but in a future version it won't
work."  Here it seems to mean "Ha ha!  I'm going to waste hours of your
time by throwing a fatal error for no good reason."

Simon
-- 
Posted via http://www.ruby-forum.com/.

-- 
You received this message because you are subscribed to the Google Groups "Ruby 
on Rails: Talk" group.
To post to this group, send email to rubyonrails-t...@googlegroups.com.
To unsubscribe from this group, send email to 
rubyonrails-talk+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/rubyonrails-talk?hl=en.

Reply via email to