Avoid deeply nested control flow statements.

Open

                    if chkout.get_price() == Decimal(0) and\
                     len(addresses) == 0:
                        a = Wallet(user=request.user, label='default')
                        a.save()
                        chkout.wallet = a

Avoid deeply nested control flow statements.

Open

                        if p.product is None:
                            p.product = product
                            p.save()
                    except ObjectDoesNotExist:

Avoid deeply nested control flow statements.

Open

                    if country in dict(countries):
                        c = ShippingCountry(country=country, product=p)
                        c.save()
            p.save()

Avoid deeply nested control flow statements.

Open

                    if country not in dict(countries):
                        errors.append('%s is not a country!' % country)
                    else:
                        sc = ShippingCountry(product=product, country=country)
                        sc.save()

Avoid deeply nested control flow statements.

Open

                    if country not in dict(countries):
                        errors.append('%s is not a country!' % country)
            if 'free-shipping' in request.POST:

Avoid deeply nested control flow statements.

Open

                    if 0 < len(request.POST['zip'].strip()) <= 15 and\
                            0 < len(request.POST['name'].strip()) <= 200 and\
                            request.POST['address1'].strip() != '' and\
                            request.POST['country'].strip() in dict(countries)\
                            and request.POST['state'].strip() != '':

Function upload_file has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

def upload_file(request, id):
    product = get_object_or_404(Product, id=id, physical=False)
    if product.seller != request.user:
        e = {
            'status': 403,

• Read up
  Read up

Function shop has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

def shop(request, user):
    try:
        usershop = get_object_or_404(UserShop, user__username=user)
    except Http404:
        if User.objects.filter(username=user).count() > 0:

• Read up
  Read up

Avoid too many return statements within this function.

Open

        return self.outside_price

Avoid too many return statements within this function.

Open

            return render(request, 'shop/checkout2.html',

Avoid too many return statements within this function.

Open

        return redirect('shop:cart')

Avoid too many return statements within this function.

Open

            return -self.get_score()

Avoid too many return statements within this function.

Open

            return render(request, 'shop/checkout3.html', {'checkout': chkout})

Function get_shipping_price has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    def get_shipping_price(self):
        if not self.cached_shipping:
            s = 0
            for item in self.cart.cartentry_set.filter(
                    product__stock__gte=F('quantity'), product__physical=True):

• Read up
  Read up

Function get_key has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

def get_key(request):
    if request.method == 'POST' and 'purchaseid' in request.POST and \
       'keyid' in request.POST:
        try:
            purchase = PurchaseItem.objects.get(purchase__by=request.user,

• Read up
  Read up

Function api_newwallet has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

def api_newwallet(request):
    response = {'status': ""}
    errors = []
    if request.method == 'POST' and 'wallet_name' in request.POST:
        if len(request.POST['wallet_name']) > 30:

• Read up
  Read up