From 6d0e9118662053d78a90a302d7670df3b47528ae Mon Sep 17 00:00:00 2001
From: Oly <oliver.marks@influentialsoftware.com>
Date: Tue, 21 Feb 2017 09:00:54 +0000
Subject: [PATCH] remove duplicated success message

---
 mhackspace/base/management/commands/generate_test_data.py | 5 +++++
 mhackspace/subscriptions/payments.py                      | 2 +-
 mhackspace/subscriptions/views.py                         | 4 ----
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/mhackspace/base/management/commands/generate_test_data.py b/mhackspace/base/management/commands/generate_test_data.py
index da3d9c8..884e532 100644
--- a/mhackspace/base/management/commands/generate_test_data.py
+++ b/mhackspace/base/management/commands/generate_test_data.py
@@ -1,6 +1,7 @@
 from autofixture import AutoFixture
 from autofixture.generators import ImageGenerator
 from django.core.management.base import BaseCommand
+from django.core.management import call_command
 from mhackspace.base.models import BannerImages
 from mhackspace.feeds.models import Article, Feed
 from mhackspace.users.models import User
@@ -13,6 +14,10 @@ class Command(BaseCommand):
     help = 'Build test data for development environment'
 
     def handle(self, *args, **options):
+        # load known data
+        call_command('loaddata', 'mhackspace/users/fixtures/groups.json', verbose=0)
+
+        # random data
         users = AutoFixture(User)
         users.create(10)
 
diff --git a/mhackspace/subscriptions/payments.py b/mhackspace/subscriptions/payments.py
index 243493e..46e1844 100644
--- a/mhackspace/subscriptions/payments.py
+++ b/mhackspace/subscriptions/payments.py
@@ -113,7 +113,7 @@ class gocardless_provider:
             'amount': subscription.amount,
             'start_date': subscription.created_at,
             'reference': subscription.id,
-            'success': response.success
+            'success': response.get('success', False)
         }
 
     def create_subscription(self, amount, name, redirect_success, redirect_failure, interval_unit='month', interval_length='1'):
diff --git a/mhackspace/subscriptions/views.py b/mhackspace/subscriptions/views.py
index d2b1bba..09fba98 100644
--- a/mhackspace/subscriptions/views.py
+++ b/mhackspace/subscriptions/views.py
@@ -106,10 +106,6 @@ class MembershipJoinSuccessView(LoginRequiredMixin, RedirectView):
         member.date = result.get('start_date')
         member.status = Membership.lookup_status(name=result.get('status'))
         member.save()
-        messages.add_message(
-            self.request,
-            messages.SUCCESS,
-            'Success your membership should now be active')
         kwargs['username'] = self.request.user.get_username()
 
         # add user to group on success