Simplifying code with multiple loops and logic, into queryset aggregates

My models:

class customer(models.Model):

cstid = models.AutoField(primary_key=True, unique=True)

insurance_number = models.CharField(max_length=100, blank=True, null=True)

name = models.CharField(max_length=35)

ageyrs = models.IntegerField(blank=True)

class Admission(models.Model):

id = models.AutoField(primary_key=True, unique=True)

clinic = models.ForeignKey(Clinic, on_delete=models.CASCADE)

customer = models.ForeignKey(customer, on_delete=models.CASCADE)

diagnosis = models.CharField(max_length=2000, default='', blank=True)

date_admission = models.DateTimeField(default=timezone.now)

ward = models.ForeignKey(Ward, on_delete=models.CASCADE)

bed = models.ForeignKey(Bed, on_delete=models.CASCADE)

discharged = models.BooleanField(default=False)

ip_number = models.IntegerField(blank=True)

ip_prefix = models.CharField(max_length=20, default='', blank=True)

My objective: Set a variable to a query filter, adding a property, 'is_admitted' to the queryset, so that I can pass this query set to the template and use the property while displaying data.

Code:

def is_admitted(cust):

admission = Admission.objects.filter(customer=cust, discharged=False)

admission_results = len(admission)

if admission_results > 0:

return True

return False

my_q = or_q_if_truthfull(

cstid=HospitalID,

name__lower__contains=name.lower() if name else None,

ageyrs=ageyrs if ageyrs.isdigit() else None,

agemnths=agemnths if agemnths.isdigit() else None,

mobile__contains=mobile if mobile else None,

alternate__contains=alternate if alternate else None,

email__lower__contains=email.lower() if email else None,

address__lower__contains=address.lower() if address else None,

city__lower__contains=city.lower() if city else None

)

ORSearchResult = customer.objects.filter(my_q, linkedclinic=clinicobj)

cust_set = []

cust_admission_status = []

for cust in ORSearchResult:

cust_set.append(cust)

cust_admission_status.append(is_admitted(cust))

print(f"Customer name: {cust.name} Admission status: {is_admitted(cust)}")

cust_templ_set = zip(cust_set, cust_admission_status)

And in template, I will do:

{% for cust, status in cust_templ_set %}

{{ cust.name }} {{ status }}

{% endfor %}

I want to understand how I can convert my above code by generating an aggregate over the queryset, so that I can use a property of the query, and change the template code to the following, and avoid the loop in the view, and the zip. So that the template code becomes:

{% for cust in customers %}

{{ cust.name }} {{ cust.status }}

{% endfor %}

I am not sure whether I am making complete sense, and can explain further.

Sincerely yours,

 Joel G Mathew

--
You received this message because you are subscribed to the Google Groups "Django users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to django-users+unsubscribe@googlegroups.com.
To post to this group, send email to django-users@googlegroups.com.
Visit this group at https://groups.google.com/group/django-users.
To view this discussion on the web visit https://groups.google.com/d/msgid/django-users/CAA%3Diw_-8OUE5cd%3DZBahkLMTSTD%3DOXJbYenHjiB3ORGfkY9mSrQ%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.