urbanairship 3.0.1 → 3.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fcee0f1f419c973ed8f480899359df8205ef23f6
-  data.tar.gz: 70e3a0e2e5f3527d25e02afb522b65dd2dcf4e16
+  metadata.gz: f00fd9af99e731c4f1b2b1d05d9663b0ecf24f94
+  data.tar.gz: 50e049c31aa5c1f467982b6b72741b92e156a639
 SHA512:
-  metadata.gz: 7dd348a203f1f9cfaf1a4e796a4c44b8b303180a91b1e0a43fa00ca6daa6f746c97cbca7ed063bb1ce35e625b2022d4d5427321dec3282ad502d9f19d9855f7f
-  data.tar.gz: d64ec3eafef734228058582157ad9abbb13e425bbfc10cfdd05b724ab64d565e3a22c5777fd53e2cefea0d289d50d6afe471eb63617ea6180ed0da995e2b609c
+  metadata.gz: 7d80f09d4cb0a219645dad8c003e468bfd31ee2381f66b2903a9ac7736d5f1fa0bde038f8e42ce59af05ac2f7318ec0178cd8bbb6f29eb3e394d1a5bb4710eb6
+  data.tar.gz: 262f4082d982ba83d242ed0d0803d41b9384eb0a9bc8f52ae9bf51d47d6f6816c3ef3c2489de02e8b4a978fa9ba321b6b45a88741498a011eec5fd69501e5b6d

data/CHANGELOG

@@ -1,3 +1,10 @@
+--------------------
+3.0.2
+--------------------
+- Resolve compatibility issues with Rails ActiveSupport
+
+
+
 --------------------
 3.0.1
 --------------------

data/lib/urbanairship/common.rb

@@ -16,7 +16,7 @@ module Urbanairship
     TAGS_URL = BASE_URL + '/tags/'
     SEGMENTS_URL = BASE_URL + '/segments/'
 
-    # Helper method for required keyword args in 2.0 that is compatible with 2.1+
+    # Helper method for required keyword args in Ruby 2.0 that is compatible with 2.1+
     # @example
     #   def say(greeting: required('greeting'))
     #     puts greeting
@@ -32,6 +32,26 @@ module Urbanairship
       raise ArgumentError.new("required parameter #{arg.to_sym.inspect + ' ' if arg}not passed to method #{method}")
     end
 
+    # Helper method that sends the indicated method to the indicated object, if the object responds to the method
+    # @example
+    #   try_helper(:first, [1,2,3])
+    #
+    #   >> 1
+    def try_helper(method, obj)
+      if obj.respond_to?(method)
+        obj.send(method)
+      end
+    end
+
+    # Helper method that deletes every key-value pair from a hash for which the value is nil
+    # @example
+    #   compact_helper({"a" => 1, "b" => nil})
+    #
+    #   >> {"a" => 1}
+    def compact_helper(a_hash)
+      a_hash.keep_if {|_, value| !value.nil?}
+    end
+
     class Unauthorized < StandardError
       # raised when we get a 401 from server
     end

data/lib/urbanairship/push/payload.rb

@@ -1,14 +1,15 @@
 module Urbanairship
   module Push
     module Payload
-      require 'ext/hash'
+      require 'urbanairship/common'
+
       include Urbanairship::Common
 
       # Notification Object for a Push Payload
       def notification(alert: nil, ios: nil, android: nil, amazon: nil,
                        blackberry: nil, wns: nil, mpns: nil, actions: nil,
                        interactive: nil)
-        payload = {
+        payload = compact_helper({
           alert: alert,
           actions: actions,
           ios: ios,
@@ -18,7 +19,7 @@ module Urbanairship
           wns: wns,
           mpns: mpns,
           interactive: interactive
-        }.compact
+        })
         fail ArgumentError, 'Notification body is empty' if payload.empty?
         payload
       end
@@ -26,7 +27,7 @@ module Urbanairship
       # iOS specific portion of Push Notification Object
       def ios(alert: nil, badge: nil, sound: nil, extra: nil, expiry: nil,
               category: nil, interactive: nil, content_available: nil)
-        {
+        compact_helper({
           alert: alert,
           badge: badge,
           sound: sound,
@@ -35,13 +36,13 @@ module Urbanairship
           category: category,
           interactive: interactive,
           'content-available' => content_available
-        }.compact
+        })
       end
 
       # Amazon specific portion of Push Notification Object
       def amazon(alert: nil, consolidation_key: nil, expires_after: nil,
                  extra: nil, title: nil, summary: nil, interactive: nil)
-        {
+        compact_helper({
           alert: alert,
           consolidation_key: consolidation_key,
           expires_after: expires_after,
@@ -49,20 +50,20 @@ module Urbanairship
           title: title,
           summary: summary,
           interactive: interactive
-        }.compact
+        })
       end
 
       # Android specific portion of Push Notification Object
       def android(alert: nil, collapse_key: nil, time_to_live: nil,
                   extra: nil, delay_while_idle: nil, interactive: nil)
-        {
+        compact_helper({
           alert: alert,
           collapse_key: collapse_key,
           time_to_live: time_to_live,
           extra: extra,
           delay_while_idle: delay_while_idle,
           interactive: interactive
-        }.compact
+        })
       end
 
       # BlackBerry specific portion of Push Notification Object
@@ -72,23 +73,23 @@ module Urbanairship
 
       # WNS specific portion of Push Notification Object
       def wns_payload(alert: nil, toast: nil, tile: nil, badge: nil)
-        payload = {
+        payload = compact_helper({
           alert: alert,
           toast: toast,
           tile: tile,
           badge: badge
-        }.compact
+        })
         fail ArgumentError, 'Must specify one message type' if payload.size != 1
         payload
       end
 
       # MPNS specific portion of Push Notification Object
       def mpns_payload(alert: nil, toast: nil, tile: nil)
-        payload = {
+        payload = compact_helper({
           alert: alert,
           toast: toast,
           tile: tile
-        }.compact
+        })
         fail ArgumentError, 'Must specify one message type' if payload.size != 1
         payload
       end
@@ -96,7 +97,7 @@ module Urbanairship
       # Rich Message specific portion of Push Notification Object
       def message(title: required('title'), body: required('body'), content_type: nil, content_encoding: nil,
                   extra: nil, expiry: nil, icons: nil, options: nil)
-        {
+        compact_helper({
           title: title,
           body: body,
           content_type: content_type,
@@ -105,13 +106,13 @@ module Urbanairship
           expiry: expiry,
           icons: icons,
           options: options
-        }.compact
+        })
       end
 
       # Interactive Notification portion of Push Notification Object
       def interactive(type: required('type'), button_actions: nil)
         fail ArgumentError, 'type must not be nil' if type.nil?
-        { type: type, button_actions: button_actions }.compact
+        compact_helper({ type: type, button_actions: button_actions })
       end
 
       def all
@@ -131,13 +132,13 @@ module Urbanairship
       # Actions for a Push Notification Object
       def actions(add_tag: nil, remove_tag: nil, open_: nil, share: nil,
                   app_defined: nil)
-        {
+        compact_helper({
           add_tag: add_tag,
           remove_tag: remove_tag,
           open: open_,
           share: share,
           app_defined: app_defined
-        }.compact
+        })
       end
     end
   end

data/lib/urbanairship/push/push.rb

@@ -1,6 +1,5 @@
 require 'json'
 
-require 'ext/object'
 require 'urbanairship/common'
 require 'urbanairship/loggable'
 
@@ -23,13 +22,13 @@ module Urbanairship
       end
 
       def payload
-        {
+        compact_helper({
           audience: @audience,
           notification: @notification,
           options: @options,
           device_types: @device_types,
           message: @message
-        }.compact
+        })
       end
 
       # Send the Push Object
@@ -67,11 +66,11 @@ module Urbanairship
       end
 
       def payload
-        {
+        compact_helper({
           name: @name,
           schedule: @schedule,
           push: @push.payload
-        }.compact
+        })
       end
 
       # Schedule the Push Notification
@@ -165,12 +164,13 @@ module Urbanairship
     # Response to a successful push notification send or schedule.
     class PushResponse
       attr_reader :ok, :push_ids, :schedule_url, :operation_id, :payload, :status_code
+      include Urbanairship::Common
 
       def initialize(http_response_body: nil, http_response_code: nil)
         @payload = http_response_body || "No Content"
         @ok = @payload['ok'] || "None"
         @push_ids = @payload['push_ids'] || "None"
-        @schedule_url = @payload['schedule_urls'].try(:first) || "None"
+        @schedule_url = try_helper(:first, @payload['schedule_urls']) || "None"
         @operation_id = @payload['operation_id'] || "None"
         @status_code = http_response_code
       end

data/lib/urbanairship/version.rb

@@ -1,3 +1,3 @@
 module Urbanairship
-  VERSION = '3.0.1'
+  VERSION = '3.0.2'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: urbanairship
 version: !ruby/object:Gem::Version
-  version: 3.0.1
+  version: 3.0.2
 platform: ruby
 authors:
 - Urban Airship
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2015-07-10 00:00:00.000000000 Z
+date: 2015-07-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: unirest
@@ -133,8 +133,6 @@ files:
 - Rakefile
 - bin/console
 - bin/setup
-- lib/ext/hash.rb
-- lib/ext/object.rb
 - lib/urbanairship.rb
 - lib/urbanairship/client.rb
 - lib/urbanairship/common.rb

data/lib/ext/hash.rb

@@ -1,5 +0,0 @@
-class Hash
-  def compact
-    keep_if { |_, value| !value.nil? }
-  end
-end

data/lib/ext/object.rb

@@ -1,5 +0,0 @@
-class Object
-  def try(method)
-    send method if respond_to? method
-  end
-end