Nsryjdtyk

I have 2 Django models Review and Item that I am working with. I want to see if the user has already reviewed the item. If yes he sees the review score. if no he sees the button to review the item

I have the below Review model

class Review (models.Model):

    review_from = models.ForeignKey(User, on_delete=models.SET_NULL, null=True, related_name='review_from')

    review_for = models.ForeignKey(User, on_delete=models.SET_NULL, null=True, related_name='review_for')

    item = models.ForeignKey(OrderItem, related_name='items') 

Defining the variables in the view context (pseudocode)

admin = User.objects.get(username="admin")

admins_reviews = Review.objects.filter(review_from__username = "admin")

Below is my template

 {% for item in buyers_items %}

    {% for review in buyers_review%}

         {% if review.item.id == item.id %}

              <button class="text-success">Your rating<br/><b>{{review.ratings}}/10</b></button>

         {% else %}

              <a href="{% url ... %}">

                <button>Leave Review</button>

              </a>

          {% endif %}

    {% endfor %}

{% endfor %}

If I do this I get a below error

How can I overcome this problem.

View

from django import template

register = template.Library()



class OrderHistory(LoginRequiredMixin, ListView):

    model = Order

    template_name = 'order/order_list.html'



    def get_context_data(self, **kwargs):

        context = super(OrderHistory, self).get_context_data()

        context['order_details'] = Order.objects.filter(emailAddress=self.request.user.email)

        context['order_items'] = OrderItem.objects.filter(order__emailAddress=self.request.user.email)

        context['buyers_review'] = Review.objects.filter(review_from=self.request.user)

        print(context['buyers_review'])

        return context

Custom Tag

    @register.filter()

    def review_bought_items(order_items, buyers_review):

        return buyers_review.filter(item__in=order_items).exists()

Based on what I see in your templates, you could do it simpler with a tag filter or in your view side. Let's go with a custom tag:

@register.filter

def review_bought_items(buyers_items,buyers_review):

     return buyers_review.filter(item__in=buyers_items).exists()

Now in the templates you could do

<!-- load the tag -->

{% load file %}



{% if buyers_items|review_bought_items:buyers_review %}

     <button class="text-success">Your rating<br/><b>{{review.ratings}}/10</b></button>

{% else %}

     <a href="{% url ... %}">Leave Review</a>

{% endif %}

The issue is that you are iterating over all buyers_reviews. In this particular case, you have 2 buyer reviews, one for the current item and one for a different one.

First iteration will evaluate to False the first condition and it will display all the Leave Review button and the 2nd iteration will evaluate it to True and display the "Your rating" block.

If you don't want to move all the logic on the backend, maybe make us of a template tag in order to filter the reviews based on item.id