wireframe-apn_on_rails 0.3.0.20110210134602 → 0.3.0.20110214171409

data/lib/apn_on_rails/app/models/apn/notification.rb

@@ -79,15 +79,17 @@ class APN::Notification < APN::Base
     # This can be run from the following Rake task:
     #   $ rake apn:notifications:deliver
     def send_notifications(notifications = APN::Notification.all(:conditions => {:sent_at => nil}))
-      unless notifications.nil? || notifications.empty?
-
+      return if notifications.nil? || notifications.empty?
+      sent_ids = []
+      begin
         APN::Connection.open_for_delivery do |conn, sock|
           notifications.each do |noty|
+            sent_ids << noty.id
             conn.write(noty.message_for_sending)
           end
         end
-
-        APN::Notification.update_all ['sent_at = ?', Time.now.utc], ['id in (?)', notifications.collect(&:id)]
+      ensure
+        APN::Notification.update_all(['sent_at = ?', Time.now.utc], ['id in (?)', sent_ids]) if sent_ids.any?
       end
     end
     

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification 
 name: wireframe-apn_on_rails
 version: !ruby/object:Gem::Version 
-  hash: 40220420269251
+  hash: 40220428342901
   prerelease: 
   segments: 
   - 0
   - 3
   - 0
-  - 20110210134602
-  version: 0.3.0.20110210134602
+  - 20110214171409
+  version: 0.3.0.20110214171409
 platform: ruby
 authors: 
 - ryan
@@ -16,7 +16,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-02-10 00:00:00 -06:00
+date: 2011-02-14 00:00:00 -06:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -104,7 +104,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: magrathea
-rubygems_version: 1.5.0
+rubygems_version: 1.5.2
 signing_key: 
 specification_version: 3
 summary: apn_on_rails