From 172f8306452c4022c1cc002b0319f418310b45c4 Mon Sep 17 00:00:00 2001
From: Faheedkhan <faheedk215@gmail.com>
Date: Sun, 1 Jun 2025 15:25:48 +0300
Subject: [PATCH] car detail colors edit page

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

diff --git a/templates/inventory/car_detail.html b/templates/inventory/car_detail.html
index 261dfcd9..da62e5fe 100644
--- a/templates/inventory/car_detail.html
+++ b/templates/inventory/car_detail.html
@@ -17,14 +17,24 @@
 {% endblock customCSS %}                
 
 {% block content %}
-    {% if not car.ready %}
+    
+    {% if not car.ready and not car.status == 'sold' %}
         <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 before making it ready for sale .") }}<a class="ms-3 text-body-primary fs-9"
-   href="{% url 'add_color' car.slug %}">{{ _("Add Color") }}</a>
+                {{ _("This car information is not complete , please add colors and finances both before making it ready for sale .") }}
             </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"
@@ -237,7 +247,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>
@@ -285,6 +295,8 @@
                     <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>
@@ -309,32 +321,27 @@
                                     
                                     <tr>
                                         <td colspan="2">
-                                            {% comment %} {% if not car.get_transfer %}
-                                                <a href="{% url 'car_finance_update' car.finances.pk %}"
+                                            {% if not car.get_transfer %}
+                                                <a href="{% url 'car_colors_update'   car.slug %}"
                                                    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>
-                                    {% comment %} <tr>
-                                        <td colspan="2">{% trans "No colors available for this car." %}</td>
-                                    </tr>
+                                {% else %}
                                     <tr>
-                                        <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>
+
+                                         <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>
                                             {% endif %}
                                         </td>
-                                    </tr> {% endcomment %}
+                                    </tr>
                                 {% endif %}
+                                <!--test-->
+                                
                             </table>
                         </div>
                     </div>