From 072ddbba0cde56d614583f265549e0323a8cb597 Mon Sep 17 00:00:00 2001
From: Faheedkhan <faheedk215@gmail.com>
Date: Sun, 1 Jun 2025 15:48:19 +0300
Subject: [PATCH] upto date on 1/6

---
 templates/inventory/car_detail.html | 47 ++++++++++++-----------------
 1 file changed, 20 insertions(+), 27 deletions(-)

diff --git a/templates/inventory/car_detail.html b/templates/inventory/car_detail.html
index da62e5fe..261dfcd9 100644
--- a/templates/inventory/car_detail.html
+++ b/templates/inventory/car_detail.html
@@ -17,24 +17,14 @@
 {% endblock customCSS %}                
 
 {% block content %}
-    
-    {% if not car.ready and not car.status == 'sold' %}
+    {% if not car.ready %}
         <div class="alert alert-outline-warning d-flex align-items-center"
              role="alert">
             <i class="fa-solid fa-circle-info fs-6"></i>
-            {%if not car.finances and not car.colors%}
             <p class="mb-0 flex-1">
-                {{ _("This car information is not complete , please add colors and finances both before making it ready for sale .") }}
+                {{ _("This car information is not complete , please add colors and finances before making it ready for sale .") }}<a class="ms-3 text-body-primary fs-9"
+   href="{% url 'add_color' car.slug %}">{{ _("Add Color") }}</a>
             </p>
-           {% elif car.finances and not car.colors %}
-             <p class="mb-0 flex-1">
-                {{ _("This car information is not complete , please add colors before making it ready for sale .") }}
-            </p>
-            {%else%}
-            <p class="mb-0 flex-1">
-                {{ _("This car information is not complete , please add finances before making it ready for sale .") }}
-            </p>
-            {%endif%}
             <button class="btn-close"
                     type="button"
                     data-bs-dismiss="alert"
@@ -247,7 +237,7 @@
                                     </tr>
                                     <tr>
                                         <th>{% trans "Discount Amount"|capfirst %}</th>
-                                        <td>{{ car.finances.discount_amount|floatformat:2 }}</td>
+                                        <td>{{ car.finances.discount_amount|floatformat:2 }} -</td>
                                     </tr>
                                     <tr>
                                         <th>{% trans "Additional Fee"|capfirst %}</th>
@@ -295,8 +285,6 @@
                     <div class="card-body">
                         <div class="table-responsive scrollbar mb-3">
                             <table class="table table-sm fs-9 mb-0 overflow-hidden">
-
-                                <!--test-->
                                 {% if car.colors %}
                                     <tr>
                                         <th>{% trans 'Exterior' %}</th>
@@ -321,27 +309,32 @@
                                     
                                     <tr>
                                         <td colspan="2">
-                                            {% if not car.get_transfer %}
-                                                <a href="{% url 'car_colors_update'   car.slug %}"
+                                            {% comment %} {% if not car.get_transfer %}
+                                                <a href="{% url 'car_finance_update' car.finances.pk %}"
                                                    class="btn btn-phoenix-warning btn-sm mb-3">{% trans "Edit" %}</a>
                                             {% else %}
                                                 <span class="badge bg-danger">{% trans "Cannot Edit, Car in Transfer." %}</span>
                                             {% endif %}
+                                        {% else %}
+                                            <p>{% trans "No finance details available." %}</p>
+                                            {% if perms.inventory.add_carfinance %}
+                                                <a href="{% url 'car_finance_create' car.slug %}"
+                                                   class="btn btn-phoenix-success btn-sm mb-3">{% trans "Add" %}</a>
+                                            {% endif %} {% endcomment %}
                                         </td>
                                     </tr>
-                                {% else %}
+                                    {% comment %} <tr>
+                                        <td colspan="2">{% trans "No colors available for this car." %}</td>
+                                    </tr>
                                     <tr>
-
-                                         <td colspan="2">
-                                            <p>{% trans "No color details available." %}</p>
-                                            {% if perms.inventory.add_carcolors %}
-                                                 <a class="btn btn-phoenix-success btn-sm mb-3" href="{% url 'add_color' car.slug %}">{{ _("Add Color") }}</a>
+                                        <td colspan="2">
+                                            {% if perms.inventory.change_carcolors %}
+                                                <a href="{% url 'add_color' car.slug %}"
+                                                   class="btn btn-phoenix-success btn-sm">{% trans "Add" %}</a>
                                             {% endif %}
                                         </td>
-                                    </tr>
+                                    </tr> {% endcomment %}
                                 {% endif %}
-                                <!--test-->
-                                
                             </table>
                         </div>
                     </div>