hoptoad

Comments and changes to this ticket

• 

  Jon Yurek July 29th, 2008 @ 09:44 AM

  • → State changed from “new” to “open”
  • → Tag changed from “” to “bug hoptoad_notifier rake”
  • → Assigned user changed from “” to “Jon Yurek”

  Hi, thanks for the bug report. That's strange that ApplicationController would be defined but not prepend_before_fiter.

  What version of Rails are you running? We're only supporting 2.0 and up for our notifier plugin.

  What gems and plugins are you using with that app? Perhaps something is interfering.

• 

  Kenneth Kalmer July 30th, 2008 @ 08:57 AM

  I got the same issue (I assume the original reporter is also running 2.0.2, just like I have).

  I fixed the issue by editing the rake task and adding a "require 'application'" just before "require 'action_controller/test_process'".

  Shouldn't the stub application controller in the rake task also inherit from ActionController::Base?

• 

  Jon Yurek July 30th, 2008 @ 09:08 AM

  It shouldn't need to, since ApplicationController should already be defined and you don't need to re-specify superclasses when opening a class back up.

  I guess in 2.0.* putting a dependency on the :environment task isn't enough to get the ApplicationController included.

  Thanks. I'll get that in and verify it works with 2.0.2.

• 

  Jon Yurek July 30th, 2008 @ 05:13 PM

  • → State changed from “open” to “resolved”

  I've updated the notifier plugin and it should work out of the box on 2.0.2, as well as 1.2.6. On 1.2.6 the testing rake task won't work until you copy the hoptoad_notifier_tasks.rake into your app's lib/tasks directory, though.