Completed deleting products functionality and resolve #62

JoyZadan · Dec 7, 2022 · c0c5c4f · c0c5c4f
1 parent f5611aa
commit c0c5c4f
Show file tree

Hide file tree

Showing 8 changed files with 25 additions and 7 deletions.
diff --git a/home/templates/home/index.html b/home/templates/home/index.html
@@ -37,7 +37,7 @@ <h2 class="display-4 brand-font-alt text-shadow text-center pt-2 my-5">
         {% for brand in brands %}
             {% if brand.is_featured %}
             <div class="col mb-4">
-                <div class="card">
+                <div class="card h-100">
                     <img class="card-img-top" src="{{ brand.image.url }}" alt="">
                     <div class="card-body">
                         <h3 class="card-title brand-font-alt h6 text-center">{{ brand.friendly_name }}</h3>

diff --git a/media/Illiyoon-europe-logo.jpg b/media/Illiyoon-europe-logo.jpg
diff --git a/media/illiyoon-ceramide-ato-body-lotion.png b/media/illiyoon-ceramide-ato-body-lotion.png
diff --git a/media/tonymoly-cherry-blossom-hand-cream.png b/media/tonymoly-cherry-blossom-hand-cream.png
diff --git a/products/templates/products/product_detail.html b/products/templates/products/product_detail.html
@@ -51,6 +51,12 @@
                     {% else %}
                     <small class="text-muted float-right"><i class="fas fa-star mr-1"></i>No Rating</small>
                     {% endif %} -->
+                    {% if request.user.is_superuser %}
+                    <small class="ml-3 float-right h6">
+                        <a href="{% url 'edit_product' product.id %}" class="text-purple">Edit</a> |
+                        <a class="text-success" href="{% url 'delete_product' product.id %}">Delete</a>
+                    </small>
+                    {% endif %}
                     <div class="price-box">
                         <p class="mb-0 pt-3 pb-1"><span class="h3 text-left font-weight-bold price-font">£{{ product.price }}</span></p>
                         {% if product.discount > 0.01 %}

diff --git a/products/templates/products/products.html b/products/templates/products/products.html
@@ -97,10 +97,12 @@ <h2 class="display-4 brand-font-alt text-shadow text-center my-5">Products</h2>
                         </a>
                     </p>
 
-                    <!-- <span class="float-right">
-                        {{ product.subcategory.friendly_name }}
-                        <i class="fas fa-tag text-warning"></i>
-                    </span> -->
+                    {% if request.user.is_superuser %}
+                    <small class="float-right h6 mt-3">
+                        <a href="{% url 'edit_product' product.id %}" class="text-purple">Edit</a> |
+                        <a class="text-success" href="{% url 'delete_product' product.id %}">Delete</a>
+                    </small>
+                    {% endif %}
                     {% endif %}
 
                 </div>

diff --git a/products/urls.py b/products/urls.py
@@ -7,4 +7,6 @@
     path('<int:product_id>/', views.product_detail, name='product_detail'),
     path('add/', views.add_product, name='add_product'),
     path('edit/<int:product_id>/', views.edit_product, name='edit_product'),
+    path('delete/<int:product_id>/', views.delete_product,
+         name='delete_product'),
 ]
diff --git a/products/views.py b/products/views.py
@@ -85,9 +85,9 @@ def add_product(request):
     if request.method == 'POST':
         form = ProductForm(request.POST, request.FILES)
         if form.is_valid():
-            form.save()
+            product = form.save()
             messages.success(request, 'Successfully added product!')
-            return redirect(reverse('add_product'))
+            return redirect(reverse('product_detail', args=[product.id]))
         else:
             messages.error(request, 'Failed to add product. \
                            Please ensure the form is valid.')
@@ -125,3 +125,11 @@ def edit_product(request, product_id):
     }
 
     return render(request, template, context)
+
+
+def delete_product(request, product_id):
+    """ Delete a product from the store """
+    product = get_object_or_404(Product, pk=product_id)
+    product.delete()
+    messages.success(request, 'Product deleted!')
+    return redirect(reverse('products'))