Lipsiasoft-exception-notifier 1.0 → 1.1

data/Rakefile

@@ -4,7 +4,7 @@ require 'rake/rdoctask'
 require 'rake/gempackagetask'
 
 PKG_NAME      = 'exception-notifier'
-PKG_VERSION   = "1.0"
+PKG_VERSION   = "1.1"
 PKG_FILE_NAME = "#{PKG_NAME}-#{PKG_VERSION}"
 
 $LOAD_PATH << File.join(File.dirname(__FILE__), 'lib')

data/generators/exception_notifier/templates/views/404.html.erb

@@ -1,4 +1,4 @@
 <div class="exception">
-  <h1>The page you were looking for doesn't exist.</h1>
+  <h1>The page you were looking for doesn't exist</h1>
   <p>You may have mistyped the address or the page may have moved.</p>
 </div>

data/generators/exception_notifier/templates/views/404.html.haml

@@ -1,5 +1,5 @@
 .exception
-  %h1 The page you were looking for doesn't exist.
-  %p You may have mistyped the address or the page may have moved. 
+  %h1 The page you were looking for doesn't exist
+  %p You may have mistyped the address or the page may have moved.
     
     

data/generators/exception_notifier/templates/views/422.html.erb

@@ -1,4 +1,4 @@
 <div class="exception">
-  <h1>The change you wanted was rejected.</h1>
-  <p>Maybe you tried to change something you didn't have access to.</p>
+  <h1>The change you wanted was rejected</h1>
+  <p>Maybe you tried to change something you didn't have access to</p>
 </div>

data/generators/exception_notifier/templates/views/422.html.haml

@@ -1,3 +1,3 @@
 .exception
-  %h1 The change you wanted was rejected.
+  %h1 The change you wanted was rejected
   %p Maybe you tried to change something you didn't have access to.

data/generators/exception_notifier/templates/views/500.html.erb

@@ -1,5 +1,5 @@
 <div class="exception">
-  <h1>We're sorry, but something went wrong.</h1>
+  <h1>We're sorry, but something went wrong</h1>
   <p>We've been notified about this issue and we'll take a look at it shortly.</p>
   <p><small>(If you're the administrator of this website, then please read
   the log file "<%=h RAILS_ENV %>.log"

data/generators/exception_notifier/templates/views/500.html.haml

@@ -1,5 +1,5 @@
 .exception
-  %h1 We're sorry, but something went wrong.
+  %h1 We're sorry, but something went wrong
   %p We've been notified about this issue and we'll take a look at it shortly.
   %p 
     %small

data/lib/controller/rescue.rb

@@ -25,14 +25,19 @@ module Lipsiadmin
       def rescue_action_in_public_with_notifier(exception) #:doc:
         response_code = response_code_for_rescue(exception)
         status        = interpret_status(response_code)[0,3]
-        render :template => "/exceptions/#{status}", :status => status
-        if response_code != :not_found
-          Lipsiadmin::Mailer::ExceptionNotifier.deliver_exception(exception, self, request)
+        respond_to do |format|
+          format.html { render :template => "/exceptions/#{status}", :status => status }
+          format.all  { render :nothing => true, :status => true }
+          #format.js   { render(:update) { |page| page.call "alert", interpret_status(response_code)  } }
         end
       rescue Exception => e
         logger.error e.message
         erase_results
         rescue_action_in_public_without_notifier(exception)
+      ensure
+        if response_code != :not_found
+          Lipsiadmin::Mailer::ExceptionNotifier.deliver_exception(exception, self, request)
+        end
       end
       
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: Lipsiasoft-exception-notifier
 version: !ruby/object:Gem::Version 
-  version: "1.0"
+  version: "1.1"
 platform: ruby
 authors: 
 - Davide D'Agostino