From 4870e1cac9775b9994d5871bf3f7b2917d0493f5 Mon Sep 17 00:00:00 2001
From: ismail <ismail.mosa.ibrahim@gmail.com>
Date: Mon, 15 Sep 2025 17:45:39 +0300
Subject: [PATCH] bug fixes and updates

---
 inventory/utils.py    | 15 +++++----------
 templates/header.html |  1 -
 2 files changed, 5 insertions(+), 11 deletions(-)

diff --git a/inventory/utils.py b/inventory/utils.py
index d26c419d..ee70bdfe 100644
--- a/inventory/utils.py
+++ b/inventory/utils.py
@@ -1599,15 +1599,10 @@ def _post_sale_and_cogs(invoice, dealer):
     entity:EntityModel = invoice.ledger.entity
     # calc   = CarFinanceCalculator(invoice)
     data = get_finance_data(invoice, dealer)
-    
+
     car = data.get("car")
-    
+
     coa:ChartOfAccountModel = entity.get_default_coa()
-    # cash_acc = (
-    #     entity.get_default_coa_accounts()
-    #     .filter(role_default=True, role=roles.ASSET_CA_CASH)
-    #     .first()
-    # )
     cash_acc = invoice.cash_account or dealer.settings.invoice_cash_account
 
     vat_acc = dealer.settings.invoice_tax_payable_account or entity.get_default_coa_accounts().filter(role_default=True, role=roles.LIABILITY_CL_TAXES_PAYABLE).first()
@@ -1615,8 +1610,8 @@ def _post_sale_and_cogs(invoice, dealer):
     car_rev = dealer.settings.invoice_vehicle_sale_account or entity.get_default_coa_accounts().filter(role_default=True, role=roles.INCOME_OPERATIONAL).first()
 
     add_rev = dealer.settings.invoice_additional_services_account
-    
-    if not add_rev:        
+
+    if not add_rev:
         try:
             add_rev = entity.get_default_coa_accounts().filter(name="After-Sales Services", active=True).first()
             if not add_rev:
@@ -1696,7 +1691,7 @@ def _post_sale_and_cogs(invoice, dealer):
     if car.get_additional_services_amount > 0:
         # Cr Sales – Additional Services
         if not add_rev:
-            logger.warning(f"Additional Services account not set for dealer {dealer}. Skipping additional services revenue entry.")            
+            logger.warning(f"Additional Services account not set for dealer {dealer}. Skipping additional services revenue entry.")
         else:
             TransactionModel.objects.create(
                 journal_entry=je_sale,
diff --git a/templates/header.html b/templates/header.html
index 30c7c977..5a39dc54 100644
--- a/templates/header.html
+++ b/templates/header.html
@@ -572,7 +572,6 @@
                         <div class="avatar avatar-l text-center align-middle">
                             {% if request.is_dealer and user.dealer.logo %}
                                 <img class="rounded-circle" src="{{ user.dealer.logo.url }}" alt="" />
-                                <img src="{% if user.dealer.logo %}{{ user.dealer.logo }}{% else %}{% static 'default-image/user.jpg' %}{% endif %}" alt="" class="rounded-circle">
                             {% elif request.is_staff and request.staff.logo %}
                                 <img class="rounded-circle" src="{{ request.staff.thumbnail.url }}" alt="" />
                             {% else %}