Django Querying database

Question

I'm trying to implement search in django.

My view is as follows :

    search_term = request.GET['search_term']

    customers = Customer.objects.filter(
        Q(chassis__icontains=search_term) | Q(registration__icontains=search_term) |
        Q(email__icontains=search_term) | Q(firstname__icontains=search_term) |
        Q(lastname__icontains=search_term))
    calculations_data = []
    if customers:
        for customer in customers:

            try:
                calculation = Calculations.objects.get(customer=customer, user=request.user)
                calculations_data.append({
                    'calculation': calculation,
                    'price': price_incl_vat(calculation.purchase_price),
                    'customer_fullname': '{} {} '.format(customer.firstname, customer.lastname),
                    'car_chassis': customer.chassis,
                    'car_registration': customer.registration,

                })
            except Calculations.DoesNotExist:
                pass

    context = {'search_term': search_term, 'total_result': len(calculations_data), 'calculation_data': calculations_data}
    return render(request, 'master/search.html', context)

I have two models, calculations and customer. Inside calculation I have customer as ForeignKey, but it can be empty. Thus, every calculation doesn't need to have a customer.

In my example, if I have search term the result is good, but If there is not search term, then I get only the calculations which have a customer.

But what I need is, if there is no search_term, I want to get all calculations.

Is there maybe a better way to write the query?

Thanks.

narendra-choudhary · Accepted Answer · 2017-02-22 11:21:31Z

Since the results depend on availability of search_term, why aren't you using if-else on search_term.

search_term = request.GET.get('search_term', None)
if search_term:
    # when search term is not None
    # get relevant calculations
else:
    calculations = Calculations.objects.all()
    # rest of code

You can further simplify your code when search_term is not None by putting the Q objects directly in a Calculations.objects.filter() itself (instead of getting relevant customers and then finding the relevant calculations). In Django, you can query on attributes of foreign key in Q objects. You are first fetching Customers and then using those results to find Calculations. That will increase number of queries to database.

You can do something like following:

calculations = Calculations.objects.filter(
                   Q(customer__email__icontains=search_term) | 
                   Q(customer__chassis_icontains=search_term)| 
                   Q(....)).select_related('customer')

Related links:
1. Lookups that span relationships
2. select_related

amit_183 · Accepted Answer · 2017-02-22 11:03:10Z

0

try this:

if customers:
        try:
            calculations = Calculations.objects.filter(user=request.user)
            if customers:
                calculations=calculations.filter(customer__in=customers)

            for calculation in calculations:
                 calculations_data.append({
                    'calculation': calculation,
                    'price': price_incl_vat(calculation.purchase_price),
                    'customer_fullname': '{} {} '.format(customer.firstname, customer.lastname),
                    'car_chassis': customer.chassis,
                    'car_registration': customer.registration,
                 })
        except Calculations.DoesNotExist:
            pass

edited Feb 22, 2017 at 11:03

answered Feb 22, 2017 at 10:40

amit_183

9813 gold badges19 silver badges36 bronze badges

6 Comments

Boky Over a year ago

It doesn't work. I get the same result. Only the calculations which have a customer.

Boky Over a year ago

No. 'Customer' object does not support indexing.

Boky Over a year ago

Now if the search_term is empty, I get no results. No errors, but also no results.

amit_183 Over a year ago

can you tell whether this line gives any result or not "calculations = Calculations.objects.filter(user=request.user)" ?

Boky Over a year ago

Yes. It gives all results for current user.

|

Collectives™ on Stack Overflow

Django Querying database

2 Answers 2

Comments

6 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

6 Comments

Your Answer

Sign up or log in

Post as a guest

Related