On Nov 14, 2007 4:39 PM, Rick DeNatale <[EMAIL PROTECTED]> wrote:

> It appears to me that he receiver and parameter of that merge message
> in the argument to the body message should be reversed.
>
> Am I missing something here?
>
> Because of this I had to do some fairly ugly code which temporarily
> set the sections cattr, and I had to act like the request insofar as
> providing an env method which returned a hash mapping "HTTP_HOST" to
> "localhost"
>
> If this is a bug rather than something I've overlooked, should I
> submit a patch to the rails trac or is there somewhere else?

For what it's worth, I switched this locally to
   body       { :controller => controller, :request => request,
                 :exception => exception, :host => request.env["HTTP_HOST"],
                 :backtrace => sanitize_backtrace(exception.backtrace),
                 :rails_root => rails_root, :data => data,
                 :sections => sections }.merge(data)

And $ rake test:plugins PLUGIN=exception_notification

runs with no failures/errors!
-- 
Rick DeNatale

My blog on Ruby
http://talklikeaduck.denhaven2.com/

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Ruby 
on Rails: Core" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/rubyonrails-core?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to