devise-onesignal 3.1.2 → 3.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b2232ec293e7d3a67295fe26055dbdc7db5ff24c4c86f5c0b9b78e43722985c6
-  data.tar.gz: 8446b547d9c23e92b25386c34b3d55ba753c557751bee00313e58dc086718478
+  metadata.gz: 707334af5eec5b6d9f438d18a43d64bbd8cc0b303eb13fc9375121ac6b25545b
+  data.tar.gz: 7262bab8d4bba70337b3149d177bb69c22c4cc03b025e5b652519247fc7ba05f
 SHA512:
-  metadata.gz: 00d052ac560e504a9dd337360afc01beabc1d5f0085a218c26725940432a1c33f46e06eecfa96729b20cedcab310e0db1ca81a1930a326fe480c76daca07b6af
-  data.tar.gz: a670f74484da03934eb58aa74c9799985724262234a0b92feb13b6aebf58f4cb9fb292f0f432bde9762db1d72aafb8820ada942dcdcb653fa001047a7a08738c
+  metadata.gz: 0a5239f4ec9f8c8e37aa3a6937eac1c881baed5d4a1b676715515cebfb344724f0f1c3b6daf25e9f928407f7f5ac3139799bf0ab9b53100e9c81077a3b5c1086
+  data.tar.gz: 1281729268836127af4c14d8fab344c4d7370278e3c8b03264eec7ecc04805d782bf9d9f4d3822fd6b93bfd225ff9079e5ced34e60d212a9746e5f86e0de619b

data/CHANGELOG.md

@@ -4,6 +4,11 @@
 
 * nothing yet
 
+### 3.1.3 - 2018-01-02
+
+* bugfixes
+    * fixed overriding `set_device_owner` method
+
 ### 3.1.2 - 2017-12-28
 
 * bugfixes

data/README.md

@@ -85,9 +85,11 @@ document.addEventListener( 'turbolinks:load', function() {
 
 You most likely want to associate your devise object (e.g. `current_user`) with your OneSignal integration. Now, if your Devise model is called `User` and the `current_user` method is available you don't have to worry about that.
 
-Let's say our Devise model is named `Admin`. Just add a `private` method to your `ApplicationController`:
+Let's say our Devise model is named `Admin`. Just add a `protected` method to your `ApplicationController`:
 
 ```ruby
+protected
+
 def set_device_owner
     current_admin if current_admin
 end

data/lib/devise-onesignal/device_concern.rb

@@ -17,13 +17,13 @@ module DeviseOnesignal
                 device = Device.find_or_create_by! onesignal_id: onesignal_player_id
                 device.onesignal_permission = cookies[:oneSignalPlayerPermission]
                 device.last_used = Time.now
-                device.owner = ApplicationController.try(:set_device_owner) || set_device_owner
+                device.owner = set_device_owner
                 device.save!
             end
         end
 
         def set_device_owner
-            current_user if current_user
+            super || current_user if current_user
         end
 
     end

data/lib/devise-onesignal/device_helper.rb

@@ -5,7 +5,7 @@ module DeviseOnesignal
             if cookies[:oneSignalPlayerId].nil?
                 device = Device.new
                 device.onesignal_permission = cookies[:oneSignalPlayerPermission]
-                device.owner = ApplicationController.try(:set_device_owner) || set_device_owner
+                device.owner = set_device_owner
                 device.last_used = Time.now
             else
                 device = Device.find_by onesignal_id: cookies[:oneSignalPlayerId]

data/lib/devise-onesignal/version.rb

@@ -1,5 +1,5 @@
 module DeviseOnesignal
 
-    VERSION = '3.1.2'
+    VERSION = '3.1.3'
 
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: devise-onesignal
 version: !ruby/object:Gem::Version
-  version: 3.1.2
+  version: 3.1.3
 platform: ruby
 authors:
 - Jonas Hübotter
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-12-28 00:00:00.000000000 Z
+date: 2018-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -130,7 +130,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.7.3
+rubygems_version: 2.7.4
 signing_key: 
 specification_version: 4
 summary: Implement user targeted cross-platform notifications in your Rails app