Fixed a bug caused by dict.pop which makes title and message not

available for notifications following the current one
rakanalh · Oct 4, 2017 · 084194b · 084194b
1 parent ba896c5
commit 084194b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/pushy/dispatchers.py b/pushy/dispatchers.py
@@ -1,3 +1,4 @@
+import copy
 from django.conf import settings
 from pushjack import (
     APNSClient,
@@ -75,7 +76,12 @@ def establish_connection(self):
             default_batch_size=100
         )
 
-    def _send(self, token, payload):
+    def _send(self, token, notification_payload):
+        # pop causes a bug in altering the original payload
+        # which causes title and message to be empty
+        # for notifications following the currrent one.
+        payload = copy.deepcopy(notification_payload)
+
         try:
             response = self._client.send(
                 [token],

diff --git a/setup.py b/setup.py
@@ -2,7 +2,7 @@
 
 setup(
     name='Django-Pushy',
-    version='1.0.0',
+    version='1.0.1',
     author='Rakan Alhneiti',
     author_email='[email protected]',