Added JS locking of forms to prevent multiple items being created.

oscar/apps/checkout/utils.py

@@ -1,5 +1,3 @@
-from django.core.urlresolvers import resolve
-
 from oscar.core.loading import get_class
 Repository = get_class('shipping.repository', 'Repository')
 

oscar/apps/order/models.py

@@ -1,6 +1,3 @@
-u"""Vanilla implementation of order models"""
-from django.db import models
-
 from oscar.apps.order.abstract_models import *
 from oscar.apps.address.abstract_models import AbstractShippingAddress, AbstractBillingAddress
 

oscar/apps/payment/forms.py

@@ -1,4 +1,4 @@
-from datetime import date, datetime
+from datetime import date
 from calendar import monthrange
 import re
 
@@ -219,7 +219,7 @@ class BillingAddressForm(forms.ModelForm):
         super(BillingAddressForm,self ).__init__(*args, **kwargs)
         self.set_country_queryset() 
         
-    def set_country_queryset(self):    
+    def set_country_queryset(self):
         self.fields['country'].queryset = address_models.Country._default_manager.all()
     
     class Meta:

oscar/static/oscar/js/oscar/ui.js

@@ -12,6 +12,21 @@ oscar.messages = {
     warning: function(msg) { oscar.messages.addMessage('warning', msg); },
     error: function(msg) { oscar.messages.addMessage('error:', msg); }
 };
+oscar.forms = {
+    init: function() {
+        // Forms with this behaviour are 'locked' once they are submitted to 
+        // prevent multiple submissions
+        $('form[data-behaviours~="lock"]').submit(oscar.forms.submitIfNotLocked);
+    },
+    submitIfNotLocked: function(event) {
+        $form = $(this);
+        if ($form.data('locked')) {
+            return false;
+        }
+        $form.data('locked', true);
+    }
+};
+$(function(){oscar.forms.init();});
 
 $(document).ready(function()
 {   

oscar/templates/customer/address-form.html

@@ -30,11 +30,11 @@
 {% endblock header %}
 
 {% block content %}
-<form action="." method="post" class="form-horizontal">
+<form action="." method="post" class="form-horizontal" data-behaviours="lock">
     {% csrf_token %}
     {% include "partials/form_fields.html" with form=form %}
     <div class="form-actions">
-        <button type="submit" value="Save address" class="btn btn-primary">Save address</button>
+        <button type="submit" value="Save" class="btn btn-primary btn-large">Save</button>
         or
         <a href="{% url customer:address-list %}">cancel</a>
     </div>