Sfoglia il codice sorgente

Fixed circular reference problem from CC project. Caused by import of

model from within forms.validators
master
David Winterbottom 13 anni fa
parent
commit
102904261c
2 ha cambiato i file con 4 aggiunte e 6 eliminazioni
  1. 2
    2
      oscar/apps/checkout/views.py
  2. 2
    4
      oscar/core/validators.py

+ 2
- 2
oscar/apps/checkout/views.py Vedi File

@@ -653,11 +653,11 @@ class PaymentDetailsView(OrderPlacementMixin, TemplateView):
653 653
         """
654 654
         # Check that shipping address has been completed
655 655
         if not self.checkout_session.is_shipping_address_set():
656
-            messages.error(request, _("Please choose a shipping address"))
656
+            messages.error(self.request, _("Please choose a shipping address"))
657 657
             return HttpResponseRedirect(reverse('checkout:shipping-address'))
658 658
         # Check that shipping method has been set
659 659
         if not self.checkout_session.is_shipping_method_set():
660
-            messages.error(request, _("Please choose a shipping method"))
660
+            messages.error(self.request, _("Please choose a shipping method"))
661 661
             return HttpResponseRedirect(reverse('checkout:shipping-method'))
662 662
         # Next, check that basket isn't empty
663 663
         if basket.is_empty:

+ 2
- 4
oscar/core/validators.py Vedi File

@@ -8,9 +8,6 @@ from django.utils.translation import ugettext_lazy as _
8 8
 
9 9
 from django.db.models import get_model
10 10
 
11
-# FlatPages is None if not installed
12
-FlatPage = get_model('flatpages', 'FlatPage')
13
-
14 11
 
15 12
 class ExtendedURLValidator(validators.URLValidator):
16 13
     def __call__(self, value):
@@ -32,7 +29,8 @@ class ExtendedURLValidator(validators.URLValidator):
32 29
                 resolve(value)
33 30
             self.is_local_url = True
34 31
         except Http404:
35
-            # check for existing urls of flatpages if package installed
32
+            # FlatPages is None if not installed
33
+            FlatPage = get_model('flatpages', 'FlatPage')
36 34
             if FlatPage is not None:
37 35
                 for page in FlatPage.objects.all().only(('url')):
38 36
                     if value == page.url:

Loading…
Annulla
Salva