Removed uniqueness from receipt, added unique_together

django_clickbank/models.py

@@ -93,7 +93,7 @@ class Notification(ClickBankModel):
 	)
 
 	# Notification Fields
-	receipt = models.CharField(max_length=13, unique=True, db_index=True)
+	receipt = models.CharField(max_length=13, db_index=True)
 	role = models.CharField(max_length=9, choices=ROLE_CHOICES)
 	transaction_type = models.CharField(max_length=15, choices=TRANSACTION_TYPE_CHOICES)
 	transaction_vendor = models.CharField(max_length=10, blank=True)
@@ -155,6 +155,9 @@ class Meta:
 		verbose_name = u'notification'
 		verbose_name_plural = u'notifications'
 
+		# Receipts are the same for chargebacks and refunds
+		unique_together = ('receipt', 'transaction_type')
+
 	def initialize(self, request):
 		""" If notification is being added from an actual Post this will set sender IP and store query if neccesary """
 		if settings.CLICKBANK_DEBUG or settings.CLICKBANK_STORE_POSTS:

django_clickbank/views.py

@@ -79,15 +79,6 @@ def ipn(request, get=False):
 			except Exception, e:
 				raise NotificationFailedValidation(e)
 		else:
-			# If the form invalidates due to duplicate receipt, we go ahead and return 200 anyway.
-			if 'receipt' in form.errors:
-				if form.errors['receipt'][0] == u'Notification with this Receipt already exists.':
-					notification = Notification.objects.get(receipt=form.data['receipt'])
-					logger.info('Notification recognized as duplicate. Returning OK.')
-					if settings.CLICKBANK_DEBUG:
-						logger.debug('Resending Signals')
-						notification.send_signals()
-					return HttpResponse(notification.id)
 			raise NotificationFailedValidation('{0}\n{1}\n{2}'.format(form.errors, mapped_data, data))
 
 		notification.initialize(request)