- You can adopt {{ gold_user.num_supported_projects }} projects with your subscription. Select Projects
-
- Supporting Read the Docs lets us work more on features that people
- love. Your money will go directly to maintenance and development of
- the product.
-
- {% endif %}
-{% endblock %}
diff --git a/readthedocs/gold/templates/gold/subscription_confirm_delete.html b/readthedocs/gold/templates/gold/subscription_confirm_delete.html
new file mode 100644
index 00000000000..b341566c418
--- /dev/null
+++ b/readthedocs/gold/templates/gold/subscription_confirm_delete.html
@@ -0,0 +1,21 @@
+{% extends "profiles/base_profile_edit.html" %}
+{% load i18n %}
+
+{% block profile-admin-gold-edit %}active{% endblock %}
+
+{% block title %}Cancel Gold{% endblock %}
+
+{% block edit_content %}
+
+ {% blocktrans %}
+ Are you sure you want to cancel your subscription?
+ {% endblocktrans %}
+
+{% endblock %}
diff --git a/readthedocs/gold/templates/gold/subscription_detail.html b/readthedocs/gold/templates/gold/subscription_detail.html
new file mode 100644
index 00000000000..a25d3391a84
--- /dev/null
+++ b/readthedocs/gold/templates/gold/subscription_detail.html
@@ -0,0 +1,68 @@
+{% extends "profiles/base_profile_edit.html" %}
+{% load static %}
+{% load i18n %}
+
+{% block profile-admin-gold-edit %}active{% endblock %}
+
+{% block title %}{% trans "Gold Subscription" %}{% endblock %}
+
+{% block extra_scripts %}
+
+
+
+
+{% endblock %}
+
+{% block edit_content %}
+
+{% endblock %}
diff --git a/readthedocs/gold/templates/gold/subscription_form.html b/readthedocs/gold/templates/gold/subscription_form.html
new file mode 100644
index 00000000000..e04ac036cf0
--- /dev/null
+++ b/readthedocs/gold/templates/gold/subscription_form.html
@@ -0,0 +1,104 @@
+{% extends "profiles/base_profile_edit.html" %}
+{% load static %}
+{% load i18n %}
+
+{% block profile-admin-gold-edit %}active{% endblock %}
+
+{% block title %}
+Gold Status
+{% endblock %}
+
+{% block extra_links %}
+
+{% endblock %}
+
+{% block extra_scripts %}
+
+
+
+
+{% endblock %}
+
+{% block edit_content %}
+
+
Gold Status
+
+
+ {% blocktrans %}
+ Supporting Read the Docs lets us work more on features that people love.
+ Your money will go directly to maintenance and development of the
+ product.
+ {% endblocktrans %}
+
+
+ {% blocktrans %}
+ You can make one-time donations on our sustainability page.
+ {% endblocktrans %}
+
+
+ {% trans "Become a Gold Member" as subscription_title %}
+ {% if golduser %}
+ {% trans "Update Your Subscription" as subscription_title %}
+ {% endif %}
+
{{ subscription_title }}
+
+
+{% endblock %}
diff --git a/readthedocs/gold/templates/gold/thanks.html b/readthedocs/gold/templates/gold/thanks.html
deleted file mode 100644
index cd5b41614e5..00000000000
--- a/readthedocs/gold/templates/gold/thanks.html
+++ /dev/null
@@ -1,18 +0,0 @@
-{% extends "profiles/base_profile_edit.html" %}
-
-{% block profile-admin-gold-edit %}active{% endblock %}
-
-{% block title %}
-Thanks!
-{% endblock %}
-
-{% block edit_content %}
-
-
-
-Thanks for contributing to Read the Docs!
-
-
-{% endblock %}
diff --git a/readthedocs/gold/tests/__init__.py b/readthedocs/gold/tests/__init__.py
new file mode 100644
index 00000000000..e69de29bb2d
diff --git a/readthedocs/gold/tests/test_forms.py b/readthedocs/gold/tests/test_forms.py
new file mode 100644
index 00000000000..2c00a3bf050
--- /dev/null
+++ b/readthedocs/gold/tests/test_forms.py
@@ -0,0 +1,224 @@
+import mock
+import django_dynamic_fixture as fixture
+from django.test import TestCase
+from django.contrib.auth.models import User
+
+from readthedocs.projects.models import Project
+
+from ..models import GoldUser
+from ..forms import GoldSubscriptionForm
+
+
+class GoldSubscriptionFormTests(TestCase):
+
+ def setUp(self):
+ self.owner = fixture.get(User)
+ self.user = fixture.get(User)
+ self.project = fixture.get(Project, users=[self.user])
+
+ # Mocking
+ self.patches = {}
+ self.mocks = {}
+ self.patches['requestor'] = mock.patch('stripe.api_requestor.APIRequestor')
+
+ for patch in self.patches:
+ self.mocks[patch] = self.patches[patch].start()
+
+ self.mocks['request'] = self.mocks['requestor'].return_value
+ self.mock_request([({}, 'reskey')])
+
+ def mock_request(self, resp):
+ self.mocks['request'].request = mock.Mock(side_effect=resp)
+
+ def test_add_subscription(self):
+ """Valid subscription form"""
+ subscription_list = {
+ 'object': 'list',
+ 'data': [],
+ 'has_more': False,
+ 'total_count': 1,
+ 'url': '/v1/customers/cus_12345/subscriptions',
+ }
+ customer_obj = {
+ 'id': 'cus_12345',
+ 'description': self.user.get_full_name(),
+ 'email': self.user.email,
+ 'subscriptions': subscription_list
+ }
+ subscription_obj = {
+ 'id': 'sub_12345',
+ 'object': 'subscription',
+ 'customer': 'cus_12345',
+ 'plan': {
+ 'id': 'v1-org-5',
+ 'object': 'plan',
+ 'amount': 1000,
+ 'currency': 'usd',
+ 'name': 'Test',
+ }
+ }
+ self.mock_request([
+ (customer_obj, ''),
+ (subscription_list, ''),
+ (subscription_obj, ''),
+ ])
+
+ # Create user and subscription
+ subscription_form = GoldSubscriptionForm(
+ {'level': 'v1-org-5',
+ 'last_4_digits': '0000',
+ 'stripe_token': 'GARYBUSEY'},
+ customer=self.user
+ )
+ self.assertTrue(subscription_form.is_valid())
+ subscription = subscription_form.save()
+
+ self.assertEqual(subscription.level, 'v1-org-5')
+ self.assertEqual(subscription.stripe_id, 'cus_12345')
+ self.assertIsNotNone(self.user.gold)
+ self.assertEqual(self.user.gold.first().level, 'v1-org-5')
+
+ self.mocks['request'].request.assert_has_calls([
+ mock.call('post',
+ '/v1/customers',
+ {'description': mock.ANY, 'email': mock.ANY},
+ mock.ANY),
+ mock.call('get',
+ '/v1/customers/cus_12345/subscriptions',
+ mock.ANY,
+ mock.ANY),
+ mock.call('post',
+ '/v1/customers/cus_12345/subscriptions',
+ {'source': mock.ANY, 'plan': 'v1-org-5'},
+ mock.ANY),
+ ])
+
+ def test_add_subscription_update_user(self):
+ """Valid subscription form"""
+ subscription_list = {
+ 'object': 'list',
+ 'data': [],
+ 'has_more': False,
+ 'total_count': 1,
+ 'url': '/v1/customers/cus_12345/subscriptions',
+ }
+ customer_obj = {
+ 'id': 'cus_12345',
+ 'description': self.user.get_full_name(),
+ 'email': self.user.email,
+ 'subscriptions': subscription_list
+ }
+ subscription_obj = {
+ 'id': 'sub_12345',
+ 'object': 'subscription',
+ 'customer': 'cus_12345',
+ 'plan': {
+ 'id': 'v1-org-5',
+ 'object': 'plan',
+ 'amount': 1000,
+ 'currency': 'usd',
+ 'name': 'Test',
+ }
+ }
+ self.mock_request([
+ (customer_obj, ''),
+ (customer_obj, ''),
+ (subscription_list, ''),
+ (subscription_obj, ''),
+ ])
+
+ # Create user and update the current gold subscription
+ golduser = fixture.get(GoldUser, user=self.user, stripe_id='cus_12345')
+ subscription_form = GoldSubscriptionForm(
+ {'level': 'v1-org-5',
+ 'last_4_digits': '0000',
+ 'stripe_token': 'GARYBUSEY'},
+ customer=self.user,
+ instance=golduser
+ )
+ self.assertTrue(subscription_form.is_valid())
+ subscription = subscription_form.save()
+
+ self.assertEqual(subscription.level, 'v1-org-5')
+ self.assertEqual(subscription.stripe_id, 'cus_12345')
+ self.assertIsNotNone(self.user.gold)
+ self.assertEqual(self.user.gold.first().level, 'v1-org-5')
+
+ self.mocks['request'].request.assert_has_calls([
+ mock.call('get',
+ '/v1/customers/cus_12345',
+ {},
+ mock.ANY),
+ mock.call('post',
+ '/v1/customers/cus_12345',
+ {'description': mock.ANY, 'email': mock.ANY},
+ mock.ANY),
+ mock.call('get',
+ '/v1/customers/cus_12345/subscriptions',
+ mock.ANY,
+ mock.ANY),
+ mock.call('post',
+ '/v1/customers/cus_12345/subscriptions',
+ {'source': mock.ANY, 'plan': 'v1-org-5'},
+ mock.ANY),
+ ])
+
+ def test_update_subscription_plan(self):
+ """Update subcription plan"""
+ subscription_obj = {
+ 'id': 'sub_12345',
+ 'object': 'subscription',
+ 'customer': 'cus_12345',
+ 'plan': {
+ 'id': 'v1-org-5',
+ 'object': 'plan',
+ 'amount': 1000,
+ 'currency': 'usd',
+ 'name': 'Test',
+ }
+ }
+ subscription_list = {
+ 'object': 'list',
+ 'data': [subscription_obj],
+ 'has_more': False,
+ 'total_count': 1,
+ 'url': '/v1/customers/cus_12345/subscriptions',
+ }
+ customer_obj = {
+ 'id': 'cus_12345',
+ 'description': self.user.get_full_name(),
+ 'email': self.user.email,
+ 'subscriptions': subscription_list
+ }
+ self.mock_request([
+ (customer_obj, ''),
+ (subscription_list, ''),
+ (subscription_obj, ''),
+ ])
+ subscription_form = GoldSubscriptionForm(
+ {'level': 'v1-org-5',
+ 'last_4_digits': '0000',
+ 'stripe_token': 'GARYBUSEY'},
+ customer=self.user
+ )
+ self.assertTrue(subscription_form.is_valid())
+ subscription = subscription_form.save()
+
+ self.assertEqual(subscription.level, 'v1-org-5')
+ self.assertIsNotNone(self.user.gold)
+ self.assertEqual(self.user.gold.first().level, 'v1-org-5')
+
+ self.mocks['request'].request.assert_has_calls([
+ mock.call('post',
+ '/v1/customers',
+ {'description': mock.ANY, 'email': mock.ANY},
+ mock.ANY),
+ mock.call('get',
+ '/v1/customers/cus_12345/subscriptions',
+ mock.ANY,
+ mock.ANY),
+ mock.call('post',
+ '/v1/customers/cus_12345/subscriptions/sub_12345',
+ {'source': mock.ANY, 'plan': 'v1-org-5'},
+ mock.ANY),
+ ])
diff --git a/readthedocs/gold/tests/test_signals.py b/readthedocs/gold/tests/test_signals.py
new file mode 100644
index 00000000000..f6b10a0d477
--- /dev/null
+++ b/readthedocs/gold/tests/test_signals.py
@@ -0,0 +1,46 @@
+import mock
+import django_dynamic_fixture as fixture
+from django.test import TestCase
+from django.contrib.auth.models import User
+from django.db.models.signals import pre_delete
+
+from readthedocs.projects.models import Project
+
+from ..models import GoldUser
+from ..signals import delete_customer
+
+
+class GoldSignalTests(TestCase):
+
+ def setUp(self):
+ self.user = fixture.get(User)
+
+ # Mocking
+ self.patches = {}
+ self.mocks = {}
+ self.patches['requestor'] = mock.patch('stripe.api_requestor.APIRequestor')
+
+ for patch in self.patches:
+ self.mocks[patch] = self.patches[patch].start()
+
+ self.mocks['request'] = self.mocks['requestor'].return_value
+
+ def mock_request(self, resp=None):
+ if resp is None:
+ resp = ({}, '')
+ self.mocks['request'].request = mock.Mock(side_effect=resp)
+
+ def test_delete_subscription(self):
+ subscription = fixture.get(GoldUser, user=self.user, stripe_id='cus_123')
+ self.assertIsNotNone(subscription)
+ self.mock_request([
+ ({'id': 'cus_123', 'object': 'customer'}, ''),
+ ({'deleted': True, 'customer': 'cus_123'}, ''),
+ ])
+
+ subscription.delete()
+
+ self.mocks['request'].request.assert_has_calls([
+ mock.call('get', '/v1/customers/cus_123', {}, mock.ANY),
+ mock.call('delete', '/v1/customers/cus_123', {}, mock.ANY),
+ ])
diff --git a/readthedocs/gold/urls.py b/readthedocs/gold/urls.py
index ea87bd3b6ef..8f0ca1e5a7d 100644
--- a/readthedocs/gold/urls.py
+++ b/readthedocs/gold/urls.py
@@ -1,18 +1,20 @@
-from django.conf.urls import url, patterns, include
+"""Gold subscription URLs"""
+
+from django.conf.urls import url, patterns
from readthedocs.gold import views
from readthedocs.projects.constants import PROJECT_SLUG_REGEX
-urlpatterns = patterns('',
- url(r'^register/$', views.register, name='gold_register'),
- url(r'^edit/$', views.edit, name='gold_edit'),
- url(r'^cancel/$', views.cancel, name='gold_cancel'),
- url(r'^thanks/$', views.thanks, name='gold_thanks'),
- url(r'^projects/$', views.projects, name='gold_projects'),
- url(r'^projects/remove/(?P
{project_slug})/$'.format(
- project_slug=PROJECT_SLUG_REGEX
- ),
- views.projects_remove,
- name='gold_projects_remove'),
- )
+urlpatterns = patterns(
+ '',
+ url(r'^$', views.DetailGoldSubscription.as_view(), name='gold_detail'),
+ url(r'^subscription/$', views.UpdateGoldSubscription.as_view(),
+ name='gold_subscription'),
+ url(r'^cancel/$', views.DeleteGoldSubscription.as_view(), name='gold_cancel'),
+ url(r'^projects/$', views.projects, name='gold_projects'),
+ url((r'^projects/remove/(?P{project_slug})/$'
+ .format(project_slug=PROJECT_SLUG_REGEX)),
+ views.projects_remove,
+ name='gold_projects_remove'),
+)
diff --git a/readthedocs/gold/views.py b/readthedocs/gold/views.py
index c39b16c2710..314222abac3 100644
--- a/readthedocs/gold/views.py
+++ b/readthedocs/gold/views.py
@@ -1,146 +1,88 @@
-import datetime
+"""Gold suscription views"""
-from django.core.urlresolvers import reverse
+from django.core.urlresolvers import reverse, reverse_lazy
from django.conf import settings
-from django.db import IntegrityError
-from django.http import HttpResponseRedirect, Http404
+from django.contrib.messages.views import SuccessMessageMixin
+from django.contrib import messages
+from django.http import HttpResponseRedirect
from django.shortcuts import render_to_response, get_object_or_404
from django.template import RequestContext
from django.contrib.auth.decorators import login_required
+from django.utils.translation import ugettext_lazy as _
+from vanilla import DeleteView, UpdateView, DetailView
-import stripe
+from readthedocs.core.mixins import LoginRequiredMixin
+from readthedocs.projects.models import Project
+from readthedocs.payments.mixins import StripeMixin
+from readthedocs.payments.utils import stripe
-from .forms import CardForm, GoldProjectForm
+from .forms import GoldSubscriptionForm, GoldProjectForm
from .models import GoldUser
-from readthedocs.projects.models import Project
-stripe.api_key = settings.STRIPE_SECRET
+class GoldSubscriptionMixin(SuccessMessageMixin, StripeMixin, LoginRequiredMixin):
-def soon():
- soon = datetime.date.today() + datetime.timedelta(days=30)
- return {'month': soon.month, 'year': soon.year}
+ """Gold subscription mixin for view classes"""
+ model = GoldUser
+ form_class = GoldSubscriptionForm
-@login_required
-def register(request):
- user = request.user
- try:
- gold_user = GoldUser.objects.get(user=request.user)
- except GoldUser.DoesNotExist:
- gold_user = None
- if request.method == 'POST':
- form = CardForm(request.POST)
- if form.is_valid():
+ def get_object(self):
+ try:
+ return self.get_queryset().get(user=self.request.user)
+ except self.model.DoesNotExist:
+ return None
- customer = stripe.Customer.create(
- description=user.username,
- email=user.email,
- card=form.cleaned_data['stripe_token'],
- plan=form.cleaned_data['level'],
- )
-
- try:
- user = GoldUser.objects.get(user=user)
- except GoldUser.DoesNotExist:
- user = GoldUser(
- user=request.user,
- )
-
- user.level = form.cleaned_data['level']
- user.last_4_digits = form.cleaned_data['last_4_digits']
- user.stripe_id = customer.id
- user.subscribed = True
-
- try:
- user.save()
- except IntegrityError:
- form.add_error(None, user.user.username + ' is already a member')
- else:
- return HttpResponseRedirect(reverse('gold_thanks'))
+ def get_form(self, data=None, files=None, **kwargs):
+ """Pass in copy of POST data to avoid read only QueryDicts"""
+ kwargs['customer'] = self.request.user
+ return super(GoldSubscriptionMixin, self).get_form(data, files, **kwargs)
- else:
- form = CardForm()
+ def get_success_url(self, **__):
+ return reverse_lazy('gold_detail')
- return render_to_response(
- 'gold/register.html',
- {
- 'form': form,
- 'gold_user': gold_user,
- 'publishable': settings.STRIPE_PUBLISHABLE,
- 'soon': soon(),
- 'user': user,
- },
- context_instance=RequestContext(request)
- )
+ def get_template_names(self):
+ return ('gold/subscription{0}.html'
+ .format(self.template_name_suffix))
-@login_required
-def edit(request):
- user = get_object_or_404(GoldUser, user=request.user)
- if request.method == 'POST':
- form = CardForm(request.POST)
- if form.is_valid():
+# Subscription Views
+class DetailGoldSubscription(GoldSubscriptionMixin, DetailView):
- customer = stripe.Customer.retrieve(user.stripe_id)
- customer.card = form.cleaned_data['stripe_token']
- customer.plan = form.cleaned_data['level']
- customer.save()
+ def get(self, request, *args, **kwargs):
+ """GET handling for this view
- user.last_4_digits = form.cleaned_data['last_4_digits']
- user.stripe_id = customer.id
- user.level = form.cleaned_data['level']
- user.subscribed = True
- user.save()
+ If there is a gold subscription instance, then we show the normal detail
+ page, otherwise show the registration form
+ """
+ resp = super(DetailGoldSubscription, self).get(request, *args, **kwargs)
+ if self.object is None:
+ return HttpResponseRedirect(reverse('gold_subscription'))
+ return resp
- return HttpResponseRedirect(reverse('gold_thanks'))
- else:
- form = CardForm(initial={'level': user.level})
+class UpdateGoldSubscription(GoldSubscriptionMixin, UpdateView):
+ success_message = _('Your subscription has been updated')
- return render_to_response(
- 'gold/edit.html',
- {
- 'form': form,
- 'publishable': settings.STRIPE_PUBLISHABLE,
- 'soon': soon(),
- },
- context_instance=RequestContext(request)
- )
+class DeleteGoldSubscription(GoldSubscriptionMixin, DeleteView):
-@login_required
-def cancel(request):
- user = get_object_or_404(GoldUser, user=request.user)
- if request.method == 'POST':
- customer = stripe.Customer.retrieve(user.stripe_id)
- customer.delete()
- user.subscribed = False
- user.save()
- return HttpResponseRedirect(reverse('gold_register'))
- return render_to_response(
- 'gold/cancel.html',
- {
- 'publishable': settings.STRIPE_PUBLISHABLE,
- 'soon': soon(),
- 'months': range(1, 13),
- 'years': range(2011, 2036)
- },
- context_instance=RequestContext(request)
- )
+ """Delete Gold subscription view
+ On object deletion, the corresponding Stripe customer is deleted as well.
+ Deletion is triggered on subscription deletion using a signal, ensuring the
+ subscription is synced with Stripe.
+ """
-def thanks(request):
- return render_to_response(
- 'gold/thanks.html',
- {
- 'publishable': settings.STRIPE_PUBLISHABLE,
- 'soon': soon(),
- 'months': range(1, 13),
- 'years': range(2011, 2036)
- },
- context_instance=RequestContext(request)
- )
+ success_message = _('Your subscription has been cancelled')
+
+ def post(self, request, *args, **kwargs):
+ """Add success message to delete post"""
+ resp = super(DeleteGoldSubscription, self).post(request, *args, **kwargs)
+ success_message = self.get_success_message({})
+ if success_message:
+ messages.success(self.request, success_message)
+ return resp
@login_required
diff --git a/readthedocs/payments/__init__.py b/readthedocs/payments/__init__.py
new file mode 100644
index 00000000000..e69de29bb2d
diff --git a/readthedocs/payments/forms.py b/readthedocs/payments/forms.py
new file mode 100644
index 00000000000..a575b4c9fb2
--- /dev/null
+++ b/readthedocs/payments/forms.py
@@ -0,0 +1,192 @@
+"""Payment forms"""
+
+import logging
+
+from stripe.resource import Customer, Charge
+from stripe.error import InvalidRequestError
+from django import forms
+from django.conf import settings
+from django.utils.translation import ugettext_lazy as _
+
+from .utils import stripe
+
+log = logging.getLogger(__name__)
+
+
+class StripeResourceMixin(object):
+
+ """Stripe actions for resources, available as a Form mixin class"""
+
+ def ensure_stripe_resource(self, resource, attrs):
+ try:
+ instance = resource.retrieve(attrs['id'])
+ except (KeyError, InvalidRequestError):
+ try:
+ del attrs['id']
+ except KeyError:
+ pass
+ return resource.create(**attrs)
+ else:
+ for (key, val) in attrs.items():
+ setattr(instance, key, val)
+ instance.save()
+ return instance
+
+ def get_customer_kwargs(self):
+ raise NotImplementedError
+
+ def get_customer(self):
+ return self.ensure_stripe_resource(resource=Customer,
+ attrs=self.get_customer_kwargs())
+
+ def get_subscription_kwargs(self):
+ raise NotImplementedError
+
+ def get_subscription(self):
+ customer = self.get_customer()
+ return self.ensure_stripe_resource(resource=customer.subscriptions,
+ attrs=self.get_subscription_kwargs())
+
+ def get_charge_kwargs(self):
+ raise NotImplementedError
+
+ def get_charge(self):
+ return self.ensure_stripe_resource(resource=Charge,
+ attrs=self.get_charge_kwargs())
+
+
+class StripeModelForm(forms.ModelForm):
+
+ """Payment form base for Stripe interaction
+
+ Use this as a base class for payment forms. It includes the necessary fields
+ for card input and manipulates the Knockout field data bindings correctly.
+
+ :cvar stripe_token: Stripe token passed from Stripe.js
+ :cvar cc_number: Credit card number field, used only by Stripe.js
+ :cvar cc_expiry: Credit card expiry field, used only by Stripe.js
+ :cvar cc_cvv: Credit card security code field, used only by Stripe.js
+ """
+
+ # Stripe token input from Stripe.js
+ stripe_token = forms.CharField(
+ required=False,
+ widget=forms.HiddenInput(attrs={
+ 'data-bind': 'valueInit: stripe_token',
+ })
+ )
+
+ # Fields used for fetching token with javascript, listed as form fields so
+ # that data can survive validation errors
+ cc_number = forms.CharField(
+ label=_('Card number'),
+ widget=forms.TextInput(attrs={
+ 'data-bind': ('valueInit: cc_number, '
+ 'textInput: cc_number, '
+ '''css: {'field-error': error_cc_number() != null}''')
+ }),
+ max_length=25,
+ required=False)
+ cc_expiry = forms.CharField(
+ label=_('Card expiration'),
+ widget=forms.TextInput(attrs={
+ 'data-bind': ('valueInit: cc_expiry, '
+ 'textInput: cc_expiry, '
+ '''css: {'field-error': error_cc_expiry() != null}''')
+ }),
+ max_length=10,
+ required=False)
+ cc_cvv = forms.CharField(
+ label=_('Card CVV'),
+ widget=forms.TextInput(attrs={
+ 'data-bind': ('valueInit: cc_cvv, '
+ 'textInput: cc_cvv, '
+ '''css: {'field-error': error_cc_cvv() != null}'''),
+ 'autocomplete': 'off',
+ }),
+ max_length=8,
+ required=False)
+
+ def __init__(self, *args, **kwargs):
+ self.customer = kwargs.pop('customer', None)
+ super(StripeModelForm, self).__init__(*args, **kwargs)
+
+ def validate_stripe(self):
+ """Run validation against Stripe
+
+ This is what will create several objects using the Stripe API. We need
+ to actually create the objects, as that is what will provide us with
+ validation errors to throw back at the form.
+
+ Form fields can be accessed here via ``self.cleaned_data`` as this
+ method is triggered from the :py:meth:`clean` method. Cleaned form data
+ should already exist on the form at this point.
+ """
+ raise NotImplementedError
+
+ def clean_stripe_token(self):
+ data = self.cleaned_data['stripe_token']
+ if not data:
+ data = None
+ return data
+
+ def clean(self):
+ """Clean form to add Stripe objects via API during validation phase
+
+ This will handle ensuring a customer and subscription exist and will
+ raise any issues as validation errors. This is required because part
+ of Stripe's validation happens on the API call to establish a
+ subscription.
+ """
+ cleaned_data = super(StripeModelForm, self).clean()
+
+ # Form isn't valid, no need to try to associate a card now
+ if not self.is_valid():
+ self.clear_card_data()
+ return
+
+ try:
+ self.validate_stripe()
+ except stripe.error.CardError as e:
+ self.clear_card_data()
+ field_lookup = {
+ 'cvc': 'cc_cvv',
+ 'number': 'cc_number',
+ 'expiry': 'cc_expiry',
+ 'exp_month': 'cc_expiry',
+ 'exp_year': 'cc_expiry',
+ }
+ error_field = field_lookup.get(e.param, None)
+ self.add_error(
+ error_field,
+ forms.ValidationError(e.message),
+ )
+ except stripe.error.StripeError as e:
+ log.error('There was a problem communicating with Stripe: %s',
+ str(e), exc_info=True)
+ raise forms.ValidationError(
+ _('There was a problem communicating with Stripe'))
+ return cleaned_data
+
+ def clear_card_data(self):
+ """Clear card data on validation errors
+
+ This requires the form was created by passing in a mutable QueryDict
+ instance, see :py:cls:`readthedocs.payments.mixin.StripeMixin`
+ """
+ try:
+ self.data['stripe_token'] = None
+ except AttributeError:
+ raise AttributeError('Form was passed immutable QueryDict POST data')
+
+ def fields_with_cc_group(self):
+ group = {
+ 'is_cc_group': True,
+ 'fields': []
+ }
+ for field in self:
+ if field.name in ['cc_number', 'cc_expiry', 'cc_cvv']:
+ group['fields'].append(field)
+ else:
+ yield field
+ yield group
diff --git a/readthedocs/payments/mixins.py b/readthedocs/payments/mixins.py
new file mode 100644
index 00000000000..c9deb904008
--- /dev/null
+++ b/readthedocs/payments/mixins.py
@@ -0,0 +1,25 @@
+"""Payment view mixin classes"""
+
+from django.conf import settings
+
+
+class StripeMixin(object):
+
+ """Adds Stripe publishable key to the context data"""
+
+ def get_context_data(self, **kwargs):
+ context = super(StripeMixin, self).get_context_data(**kwargs)
+ context['stripe_publishable'] = settings.STRIPE_PUBLISHABLE
+ return context
+
+ def get_form(self, data=None, files=None, **kwargs):
+ """Pass in copy of POST data to avoid read only QueryDicts on form
+
+ This is used to be able to reset some important credit card fields if
+ card validation fails. In this case, the Stripe token was valid, but the
+ card was rejected during the charge or subscription instantiation.
+ """
+ if self.request.method == 'POST':
+ data = self.request.POST.copy()
+ cls = self.get_form_class()
+ return cls(data=data, files=files, **kwargs)
diff --git a/readthedocs/payments/static-src/payments/js/base.js b/readthedocs/payments/static-src/payments/js/base.js
new file mode 100644
index 00000000000..05c95f0fab4
--- /dev/null
+++ b/readthedocs/payments/static-src/payments/js/base.js
@@ -0,0 +1,150 @@
+// Stripe payment form views
+
+var ko = require('knockout'),
+ payment = require('jquery.payment'),
+ $ = require('jquery'),
+ stripe = null;
+
+// TODO stripe doesn't support loading locally very well, do they?
+if (typeof(window) != 'undefined' && typeof(window.Stripe) != 'undefined') {
+ stripe = window.Stripe || {};
+}
+
+/* Knockout binding to set initial observable values from HTML */
+ko.bindingHandlers.valueInit = {
+ init: function(element, accessor) {
+ var value = accessor();
+ if (ko.isWriteableObservable(value)) {
+ value(element.value);
+ }
+ }
+};
+
+function PaymentView (config) {
+ var self = this,
+ config = config || {};
+
+ // Config
+ stripe.publishableKey = self.stripe_key = config.key;
+ self.form = config.form;
+
+ // Credit card parameters
+ self.cc_number = ko.observable(null);
+ self.cc_expiry = ko.observable(null);
+ self.cc_cvv = ko.observable(null);
+ self.error_cc_number = ko.observable(null);
+ self.error_cc_expiry = ko.observable(null);
+ self.error_cc_cvv = ko.observable(null);
+
+ self.stripe_token = ko.observable(null);
+ self.card_digits = ko.observable(null);
+
+ // Form editing
+ self.is_editing_card = ko.observable(false);
+ self.show_card_form = ko.computed(function () {
+ return (self.is_editing_card() ||
+ !self.card_digits() ||
+ self.cc_number() ||
+ self.cc_expiry() ||
+ self.cc_cvv());
+ });
+
+ // Credit card validation
+ self.initialize_form();
+
+ // Outputs
+ self.error = ko.observable(null);
+
+ // Process form inputs and send API request to Stripe. Using jquery.payment
+ // for some validation, display field errors and some generic errors.
+ self.process_form = function () {
+ var expiry = $.payment.cardExpiryVal(self.cc_expiry()),
+ card = {
+ number: self.cc_number(),
+ exp_month: expiry.month,
+ exp_year: expiry.year,
+ cvc: self.cc_cvv()
+ };
+
+ self.error(null);
+ self.error_cc_number(null);
+ self.error_cc_expiry(null);
+ self.error_cc_cvv(null);
+
+ if (!$.payment.validateCardNumber(card.number)) {
+ self.error_cc_number('Invalid card number');
+ return false;
+ }
+ if (!$.payment.validateCardExpiry(card.exp_month, card.exp_year)) {
+ self.error_cc_expiry('Invalid expiration date');
+ return false;
+ }
+ if (!$.payment.validateCardCVC(card.cvc)) {
+ self.error_cc_cvv('Invalid security code');
+ return false;
+ }
+
+ stripe.createToken(card, function(status, response) {
+ if (response.error) {
+ if (response.error.type == 'card_error') {
+ var code_map = {
+ 'invalid_number': self.error_cc_number,
+ 'incorrect_number': self.error_cc_number,
+ 'expired_card': self.error_cc_number,
+ 'card_declined': self.error_cc_number,
+ 'invalid_expiry_month': self.error_cc_expiry,
+ 'invalid_expiry_year': self.error_cc_expiry,
+ 'invalid_cvc': self.error_cc_cvv,
+ 'incorrect_cvc': self.error_cc_cvv,
+ }
+ var fn = code_map[response.error.code] ||
+ self.error_cc_number;
+ fn(response.error.message);
+ }
+ else {
+ self.error_cc_number(response.error.message);
+ }
+ }
+ else {
+ self.submit_form(response.card.last4, response.id);
+ }
+ });
+ };
+
+ self.process_full_form = function () {
+ if (self.show_card_form()) {
+ self.process_form()
+ }
+ else {
+ return true;
+ }
+ };
+
+}
+
+PaymentView.prototype.submit_form = function (card_digits, token) {
+ this.form.find('#id_card_digits').val(card_digits);
+ this.form.find('#id_stripe_token').val(token);
+ this.form.submit();
+};
+
+PaymentView.prototype.initialize_form = function () {
+ var cc_number = $('input#id_cc_number'),
+ cc_cvv = $('input#id_cc_cvv'),
+ cc_expiry = $('input#id_cc_expiry');
+
+ cc_number.payment('formatCardNumber');
+ cc_expiry.payment('formatCardExpiry');
+ cc_cvv.payment('formatCardCVC');
+
+ cc_number.trigger('keyup');
+};
+
+PaymentView.init = function (config, obj) {
+ var view = new PaymentView(config),
+ obj = obj || $('#payment-form')[0];
+ ko.applyBindings(view, obj);
+ return view;
+}
+
+module.exports.PaymentView = PaymentView;
diff --git a/readthedocs/payments/static/payments/css/form.css b/readthedocs/payments/static/payments/css/form.css
new file mode 100644
index 00000000000..44c7f0828fc
--- /dev/null
+++ b/readthedocs/payments/static/payments/css/form.css
@@ -0,0 +1,26 @@
+/* Payment form CSS */
+
+form.payment input#id_cc_number {
+ height: 23px;
+ margin: 3px 0px 10px;
+ background: url('/static/donate/img/creditcard.png');
+ background-repeat: no-repeat;
+ background-position: -0px -128px;
+ padding-left: 40px;
+}
+
+form.payment input#id_cc_number.visa {
+ background-position: -40px -96px;
+}
+form.payment input#id_cc_number.mastercard {
+ background-position: -80px -64px;
+}
+form.payment input#id_cc_number.amex {
+ background-position: -120px -32px;
+}
+form.payment input#id_cc_number.discover {
+ background-position: -160px -0px;
+}
+
+form.payment input#id_cc_expiry { width: 150px; }
+form.payment input#id_cc_cvv { width: 100px; }
diff --git a/readthedocs/payments/utils.py b/readthedocs/payments/utils.py
new file mode 100644
index 00000000000..58dcc20ec8e
--- /dev/null
+++ b/readthedocs/payments/utils.py
@@ -0,0 +1,30 @@
+"""Payment utility functions
+
+These are mostly one-off functions. Define the bulk of Stripe operations on
+:py:cls:`readthedocs.payments.forms.StripeResourceMixin`.
+"""
+
+import stripe
+from django.conf import settings
+
+stripe.api_key = getattr(settings, 'STRIPE_SECRET', None)
+
+
+def delete_customer(customer_id):
+ """Delete customer from Stripe, cancelling subscriptions"""
+ try:
+ customer = stripe.Customer.retrieve(customer_id)
+ return customer.delete()
+ except stripe.error.InvalidRequestError:
+ pass
+
+
+def cancel_subscription(customer_id, subscription_id):
+ """Cancel Stripe subscription, if it exists"""
+ try:
+ customer = stripe.Customer.retrieve(customer_id)
+ if hasattr(customer, 'subscriptions'):
+ subscription = customer.subscriptions.retrieve(subscription_id)
+ return subscription.delete()
+ except stripe.error.StripeError:
+ pass
diff --git a/readthedocs/projects/views/private.py b/readthedocs/projects/views/private.py
index 207de770e22..3224e2e4db1 100644
--- a/readthedocs/projects/views/private.py
+++ b/readthedocs/projects/views/private.py
@@ -8,11 +8,9 @@
from django.core.urlresolvers import reverse
from django.http import (HttpResponseRedirect, HttpResponseNotAllowed,
Http404, HttpResponseBadRequest)
-from django.db.models import Q
from django.shortcuts import get_object_or_404, render_to_response, render
from django.template import RequestContext
from django.views.generic import View, TemplateView, ListView
-from django.utils.decorators import method_decorator
from django.utils.translation import ugettext_lazy as _
from formtools.wizard.views import SessionWizardView
from allauth.socialaccount.models import SocialAccount
@@ -26,7 +24,6 @@
from readthedocs.builds.models import VersionAlias
from readthedocs.core.utils import trigger_build
from readthedocs.core.mixins import ListViewWithForm
-from readthedocs.oauth import utils as oauth_utils
from readthedocs.projects.forms import (
ProjectBasicsForm, ProjectExtraForm,
ProjectAdvancedForm, UpdateProjectForm, SubprojectForm,
@@ -37,19 +34,12 @@
from readthedocs.projects import constants, tasks
from readthedocs.projects.tasks import remove_path_from_web
-
+from readthedocs.core.mixins import LoginRequiredMixin
from readthedocs.projects.signals import project_import
log = logging.getLogger(__name__)
-class LoginRequiredMixin(object):
-
- @method_decorator(login_required)
- def dispatch(self, *args, **kwargs):
- return super(LoginRequiredMixin, self).dispatch(*args, **kwargs)
-
-
class PrivateViewMixin(LoginRequiredMixin):
pass
diff --git a/readthedocs/settings/base.py b/readthedocs/settings/base.py
index 5bcc2251f3f..e740d813ccd 100644
--- a/readthedocs/settings/base.py
+++ b/readthedocs/settings/base.py
@@ -214,6 +214,7 @@
'readthedocs.privacy',
'readthedocs.gold',
'readthedocs.donate',
+ 'readthedocs.payments',
# allauth
'allauth',
diff --git a/readthedocs/settings/test.py b/readthedocs/settings/test.py
index 6ce85b79461..0b20fe0779e 100644
--- a/readthedocs/settings/test.py
+++ b/readthedocs/settings/test.py
@@ -9,7 +9,6 @@
PRODUCTION_DOMAIN = 'readthedocs.org'
GROK_API_HOST = 'http://localhost:8888'
-
if not os.environ.get('DJANGO_SETTINGS_SKIP_LOCAL', False):
try:
from local_settings import * # noqa
diff --git a/readthedocs/templates/profiles/base_profile_edit.html b/readthedocs/templates/profiles/base_profile_edit.html
index 634cedfc5c3..8cb11a086f0 100644
--- a/readthedocs/templates/profiles/base_profile_edit.html
+++ b/readthedocs/templates/profiles/base_profile_edit.html
@@ -48,7 +48,7 @@
{% trans "Social Accounts" %}
{% trans "Change Password" %}
{% trans "Change Email" %}
- {% trans "Gold" %}
+ {% trans "Gold" %}
{% block edit_content_header %}{% endblock %}