Sfoglia il codice sorgente

use getattr instead of lazyattribute

master
Martin Winkel 3 anni fa
parent
commit
73567d8746
1 ha cambiato i file con 1 aggiunte e 4 eliminazioni
  1. 1
    4
      src/oscar/test/factories/order.py

+ 1
- 4
src/oscar/test/factories/order.py Vedi File

2
 
2
 
3
 import factory
3
 import factory
4
 from django.conf import settings
4
 from django.conf import settings
5
-from django.contrib.sites.models import Site
6
 
5
 
7
 from oscar.core.loading import get_class, get_model
6
 from oscar.core.loading import get_class, get_model
8
 from oscar.core.utils import slugify
7
 from oscar.core.utils import slugify
60
     if hasattr(settings, 'OSCAR_INITIAL_ORDER_STATUS'):
59
     if hasattr(settings, 'OSCAR_INITIAL_ORDER_STATUS'):
61
         status = settings.OSCAR_INITIAL_ORDER_STATUS
60
         status = settings.OSCAR_INITIAL_ORDER_STATUS
62
 
61
 
63
-    site_id = factory.LazyAttribute(
64
-        lambda o: settings.SITE_ID if hasattr(settings, "SITE_ID") else Site.objects.first()
65
-    )
62
+    site_id = getattr(settings, "SITE_ID", None)
66
     number = factory.LazyAttribute(lambda o: '%d' % (100000 + o.basket.pk))
63
     number = factory.LazyAttribute(lambda o: '%d' % (100000 + o.basket.pk))
67
     basket = factory.SubFactory(
64
     basket = factory.SubFactory(
68
         'oscar.test.factories.BasketFactory')
65
         'oscar.test.factories.BasketFactory')

Loading…
Annulla
Salva