Rails 3 errors #7

Open
danielb2 opened this Issue Oct 5, 2010 · 5 comments

Comments

Projects
None yet
2 participants

danielb2 commented Oct 5, 2010

Is this not working with rails 3?

[~/km/sio/trunk]$ bundle exec ar_sendmail --create-migration
Unhandled exception undefined method `classify' for "Email":String(NoMethodError):
        /Users/daniel/.rvm/gems/ruby-1.9.2-p0/gems/ar_mailer-1.5.1/lib/action_mailer/ar_sendmail.rb:125:in `create_migra
tion'
        /Users/daniel/.rvm/gems/ruby-1.9.2-p0/gems/ar_mailer-1.5.1/lib/action_mailer/ar_sendmail.rb:369:in `run'
        /Users/daniel/.rvm/gems/ruby-1.9.2-p0/gems/ar_mailer-1.5.1/bin/ar_sendmail:5:in `<top (required)>'
        /Users/daniel/.rvm/gems/ruby-1.9.2-p0/bin/ar_sendmail:19:in `load'
        /Users/daniel/.rvm/gems/ruby-1.9.2-p0/bin/ar_sendmail:19:in `<main>'
Owner

adzap commented Oct 5, 2010

No I haven't got to it yet. This is probably caused by not requiring the specific ActiveSupport component that defines the classify method. Feel free to have a go at fixing it and I'll pull in the changes.

danielb2 commented Oct 5, 2010

I ended up using this fork instead which works fine: http://github.com/yzhang/ar_mailer_rails3

Owner

adzap commented Oct 5, 2010

Thanks. Annoyingly he has not forked the project on github, but copied the source and made a new repo.

Oh well, I just do it myself soon.

danielb2 commented Oct 6, 2010

Hey I found an issue in the other code which will affect you too ( does already ) and is specifically for Rails 3 so I will mention it ( it uses the new ActionMailer stuffs ).

Should probably do something with Rails.env. It looks like during testing, this
gem actually tries to send mail but fails with a connection failure unless it's
configured.

Maybe also look into ActionMailer::Base.deliveries and see how that's used
during testing?

http://edgeguides.rubyonrails.org/action_mailer_basics.html#mailer-testing

Owner

adzap commented Oct 7, 2010

Thanks. I will take a look at it shortly to address the issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment