fix products list urls

for #1

main/templates/main/product_list.html

@@ -6,7 +6,7 @@
 {% block content %}
 <h2 class="mb-3">Products</h2>
 
-{% for product in object_list %}
+{% for product in products %}
 <p>{{ product.name }}</p>
 <p>
   <a href="{% url 'main:product-detail' product.slug %}">See it here</a>

main/urls.py

@@ -9,6 +9,7 @@
     path('', views.home_view, name='home'),
     path('about/', views.about_view, name='about'),
     path('contact/', views.ContactView.as_view(), name='contact'),
+    path('products/', views.ProductListView.as_view(), name='product-list'),
     path('products/<slug:tag>/', views.ProductListView.as_view(), name='product-list'),
     path(
         'product/<slug:slug>/', views.ProductDetailView.as_view(), name='product-detail'

main/views.py

@@ -107,20 +107,19 @@ def form_valid(self, form):
 class ProductListView(generic.ListView):
     template_name = 'main/product_list.html'
     paginate_by = 6
+    context_object_name = 'products'
 
     def get_queryset(self):
-        tag = self.kwargs['tag']
-        self.tag = None
+        tag = self.kwargs.get('tag', None)
 
-        if tag != 'all':
-            self.tag = get_object_or_404(ProductTag, slug=tag)
+        if tag is None:
+            return Product.objects.active().order_by('name')
 
-        if self.tag:
-            products = Product.objects.active().filter(tags=self.tag)
+        if tag == 'all':
+            return Product.objects.active()
         else:
-            products = Product.objects.active()
-
-        return products.order_by('name')
+            tag: ProductTag = get_object_or_404(ProductTag, slug=tag)
+            return Product.objects.active().filter(tags=tag)
 
 
 class ProductDetailView(generic.DetailView):
@@ -130,8 +129,9 @@ class ProductDetailView(generic.DetailView):
 
 
 class TagDetailView(generic.ListView):
-    template_name = 'main/tag_detail.html'
+    template_name = 'main/product_list.html'
     paginate_by = 6
+    context_object_name = 'products'
 
     def get_queryset(self):
         tag = self.kwargs['tag']