From 0ee526e8065e81a60e8eeacc4697118ddc559c4b Mon Sep 17 00:00:00 2001
From: Faheed <faheed@tenhal.sa>
Date: Thu, 18 Sep 2025 19:37:40 +0300
Subject: [PATCH] estimateforcarform queryset update

---
 inventory/forms.py                               | 2 +-
 inventory/views.py                               | 2 +-
 templates/inventory/car_detail.html              | 2 +-
 templates/purchase_orders/includes/card_po.html  | 2 +-
 templates/purchase_orders/po_confirm_delete.html | 6 ++++--
 templates/purchase_orders/po_delete.html         | 6 ++++--
 templates/sales/estimates/sale_order_form.html   | 3 ++-
 7 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/inventory/forms.py b/inventory/forms.py
index d5a34498..52c69ed6 100644
--- a/inventory/forms.py
+++ b/inventory/forms.py
@@ -2268,7 +2268,7 @@ class CarDealershipRegistrationForm(forms.ModelForm):
 
 class CarDetailsEstimateCreate(forms.Form):
     customer = forms.ModelChoiceField(
-        queryset=Customer.objects.all(),
+        queryset=Customer.objects.filter(active=True),
         required=True,
         label="Customer",
         widget=forms.Select(attrs={"class": "form-control"}),
diff --git a/inventory/views.py b/inventory/views.py
index 2a3326c4..1f3b1de4 100644
--- a/inventory/views.py
+++ b/inventory/views.py
@@ -5005,7 +5005,7 @@ def create_estimate(request, dealer_slug, slug=None):
         customer = opportunity.customer
         print(customer)
         form.fields["customer"].queryset = models.Customer.objects.filter(
-            pk=customer.pk
+            pk=customer.pk 
         )
         form.initial["customer"] = customer
 
diff --git a/templates/inventory/car_detail.html b/templates/inventory/car_detail.html
index e6c88c50..a0c300fd 100644
--- a/templates/inventory/car_detail.html
+++ b/templates/inventory/car_detail.html
@@ -75,7 +75,7 @@
         </div>
     {% endif %}
     <!-- Main row -->
-    {% if car.ready %}
+    {% if car.ready and  car.status != 'sold' %}
    <div class="d-flex align-items-center gap-2 mb-2">
     <button type="button"
             class="btn btn-sm btn-phoenix-success"
diff --git a/templates/purchase_orders/includes/card_po.html b/templates/purchase_orders/includes/card_po.html
index 9d963b2b..d5f057c4 100644
--- a/templates/purchase_orders/includes/card_po.html
+++ b/templates/purchase_orders/includes/card_po.html
@@ -19,7 +19,7 @@
             const modalBody = document.getElementById('POModalBody')
             modalBody.innerHTML = `
             <div class="d-flex justify-content-center gap-3 py-3">
-                <a hx-boost="true" class="btn btn-phoenix-primary px-4" href="${actionUrl}">
+                <a hx-boost="true" class="btn btn-phoenix-danger px-4" href="${actionUrl}">
                     <i class="fas fa-check-circle me-2"></i>${buttonText}
                 </a>
                 <button class="btn btn-phoenix-secondary" data-bs-dismiss="modal">
diff --git a/templates/purchase_orders/po_confirm_delete.html b/templates/purchase_orders/po_confirm_delete.html
index cd8b9196..e0e90738 100644
--- a/templates/purchase_orders/po_confirm_delete.html
+++ b/templates/purchase_orders/po_confirm_delete.html
@@ -1,17 +1,19 @@
 <!-- templates/purchase_orders/po_confirm_delete.html -->
 {% extends "base.html" %}
+{%load i18n %}
 {% block title %}{% trans "Confirm Delete"%} - {{ block.super }}{% endblock %}
 {% block content %}
     <div class="container mt-4">
         <h2>{% trans "Confirm Deletion" %}</h2>
         <p>
-            {% trans "Are you sure you want to delete the Purchase Order" %} <strong>"{{ po.po_number }}"</strong>?
+            {% trans "Are you sure you want to delete the Purchase Order" %} <strong>"{{ object.po_number }}"</strong>?
         </p>
         <form method="post">
             {% csrf_token %}
             <button type="submit" class="btn btn-phoenix-danger">{% trans "Yes, Delete" %}</button>
-            <a href="{% url 'purchase_order_detail' po.id %}"
+            <a href="{% url 'purchase_order_detail' request.dealer.slug request.dealer.entity.slug object.pk %}"
                class="btn btn-phoenix-secondary">{% trans "Cancel" %}</a>
         </form>
     </div>
 {% endblock %}
+
diff --git a/templates/purchase_orders/po_delete.html b/templates/purchase_orders/po_delete.html
index bfd337aa..28f22968 100644
--- a/templates/purchase_orders/po_delete.html
+++ b/templates/purchase_orders/po_delete.html
@@ -18,8 +18,8 @@
                             Purchase Order {{ po_model.po_number }}?{% endblocktrans %}
                         </h2>
                         <p class="card-text text-muted mb-4">
-                            {% trans "All transactions associated with this Purchase Order will be deleted.
-                            If you want to void the PO instead," %} <a href="{% url 'purchase_order_detail' dealer_slug=request.dealer.slug pk=po_model.uuid %}"
+                            {% trans "All transactions associated with this Purchase Order will be deleted.If you want to void the PO instead," %} 
+                            <a href="{% url 'purchase_order_detail' dealer_slug=request.dealer.slug entity_slug=view.kwargs.entity_slug pk=po_model.uuid %}"
                          class="text-decoration-none">{% trans "click here" %}</a>
                         </p>
                         <div class="d-flex justify-content-center gap-3 mt-4">
@@ -33,3 +33,5 @@
         </div>
     </div>
 {% endblock %}
+
+
diff --git a/templates/sales/estimates/sale_order_form.html b/templates/sales/estimates/sale_order_form.html
index 80d7cefe..0d834097 100644
--- a/templates/sales/estimates/sale_order_form.html
+++ b/templates/sales/estimates/sale_order_form.html
@@ -28,7 +28,7 @@
                                 <i class="fa-solid fa-floppy-disk me-1"></i>
                                 {% trans 'Save' %}
                             </button>
-                            <a href="{% url 'home' request.dealer.slug %}"
+                            <a href="{% url 'estimate_detail' request.dealer.slug  estimate.pk %}"
                                class="btn btn-phoenix-secondary btn-lg">
                                 <i class="fa-solid fa-ban me-1"></i>
                                 {% trans "Cancel" %}
@@ -40,3 +40,4 @@
         </div>
     </main>
 {% endblock content %}
+