more clean and fixes

inventory/admin.py

@@ -24,6 +24,7 @@ admin.site.register(models.CarReservation)
 admin.site.register(models.Organization)
 admin.site.register(models.Representative)
 admin.site.register(models.CarTrim)
+admin.site.register(models.AdditionalServices)
 
 @admin.register(models.CarMake)
 class CarMakeAdmin(admin.ModelAdmin):

inventory/forms.py

@@ -17,7 +17,8 @@ from .models import (
     CarLocation,
     Organization,
     Representative,
-SaleQuotationCar
+SaleQuotationCar,
+AdditionalServices
 
 )
 from django.forms import ModelMultipleChoiceField
@@ -112,11 +113,24 @@ class CarUpdateForm(forms.ModelForm, AddClassMixin):
 
 
 class CarFinanceForm(AddClassMixin, forms.ModelForm):
-
+    additional_finances = forms.ModelMultipleChoiceField(
+    queryset=AdditionalServices.objects.all(),
+    widget=forms.CheckboxSelectMultiple(attrs={'class': 'form-check-input'}),
+    required=False
+)
     class Meta:
         model = CarFinance
-        exclude = ['car', 'profit_margin', 'vat_amount', 'total', 'vat_rate','additional_services']
+        exclude = ['car', 'additional_finances','profit_margin', 'vat_amount', 'total', 'vat_rate','additional_services']
 
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+        if self.instance.pk:
+            self.fields['additional_finances'].initial = self.instance.additional_services.all()
+    def save(self, commit=True):
+        instance = super().save()
+        instance.additional_services.set(self.cleaned_data['additional_finances'])        
+        instance.save()
+        return instance
 
 class CarLocationForm(forms.ModelForm):
     class Meta:

inventory/migrations/0037_alter_carfinance_additional_services.py

@@ -0,0 +1,18 @@
+# Generated by Django 4.2.17 on 2024-12-19 09:21
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('inventory', '0036_remove_car_additional_services_and_more'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='carfinance',
+            name='additional_services',
+            field=models.ManyToManyField(related_name='additional_finances', to='inventory.additionalservices'),
+        ),
+    ]

inventory/migrations/0038_remove_carfinance_administration_fee_and_more.py

@@ -0,0 +1,42 @@
+# Generated by Django 4.2.17 on 2024-12-19 09:33
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('inventory', '0037_alter_carfinance_additional_services'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='carfinance',
+            name='administration_fee',
+        ),
+        migrations.RemoveField(
+            model_name='carfinance',
+            name='custom_card_fee',
+        ),
+        migrations.RemoveField(
+            model_name='carfinance',
+            name='profit_margin',
+        ),
+        migrations.RemoveField(
+            model_name='carfinance',
+            name='registration_fee',
+        ),
+        migrations.RemoveField(
+            model_name='carfinance',
+            name='transportation_fee',
+        ),
+        migrations.RemoveField(
+            model_name='carfinance',
+            name='vat_amount',
+        ),
+        migrations.AlterField(
+            model_name='carfinance',
+            name='additional_services',
+            field=models.ManyToManyField(blank=True, null=True, related_name='additional_finances', to='inventory.additionalservices'),
+        ),
+    ]

inventory/migrations/0039_alter_carfinance_additional_services.py

@@ -0,0 +1,18 @@
+# Generated by Django 4.2.17 on 2024-12-19 09:33
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('inventory', '0038_remove_carfinance_administration_fee_and_more'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='carfinance',
+            name='additional_services',
+            field=models.ManyToManyField(blank=True, related_name='additional_finances', to='inventory.additionalservices'),
+        ),
+    ]

inventory/migrations/0040_additionalservices_display_name.py

@@ -0,0 +1,19 @@
+# Generated by Django 4.2.17 on 2024-12-19 12:01
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('inventory', '0039_alter_carfinance_additional_services'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='additionalservices',
+            name='display_name',
+            field=models.CharField(default=1, max_length=255, verbose_name='Display Name'),
+            preserve_default=False,
+        ),
+    ]

inventory/models.py

@@ -1,6 +1,7 @@
 from uuid import uuid4
 from django.conf import settings
 from django.db import models, transaction
+from django.db.models import Sum, F, Count
 from django.contrib.auth.models import User
 from django.db.models.signals import pre_save, post_save
 from django.dispatch import receiver
@@ -143,6 +144,7 @@ class DEALER_TYPES(models.TextChoices):
 
 class AdditionalServices(models.Model):
     name = models.CharField(max_length=255, verbose_name=_("Name"))
+    display_name = models.CharField(max_length=255, verbose_name=_("Display Name"))
     description = models.TextField(verbose_name=_("Description"))
     price = models.DecimalField(max_digits=14, decimal_places=2, verbose_name=_("Price"))
 
@@ -151,7 +153,7 @@ class AdditionalServices(models.Model):
         verbose_name_plural = _("Additional Services")
 
     def __str__(self):
-        return self.name
+        return self.name + " - " + str(self.price)
 
 class Car(models.Model):
     vin = models.CharField(max_length=17, unique=True, verbose_name=_("VIN"))
@@ -274,28 +276,28 @@ class CarReservation(models.Model):
 
 # Car Finance Model
 class CarFinance(models.Model):
-    additional_services = models.ManyToManyField(AdditionalServices, related_name="cars")
+    additional_services = models.ManyToManyField(AdditionalServices, related_name="additional_finances",blank=True)
     car = models.OneToOneField(Car, on_delete=models.CASCADE, related_name='finances')
     cost_price = models.DecimalField(max_digits=14, decimal_places=2, verbose_name=_("Cost Price"))
     selling_price = models.DecimalField(max_digits=14, decimal_places=2, verbose_name=_("Selling Price"))
-    profit_margin = models.DecimalField(max_digits=14,
-                                        decimal_places=2,
-                                        verbose_name=_("Profit Margin"),
-                                        editable=False)
-    vat_amount = models.DecimalField(max_digits=14,
-                                        decimal_places=2,
-                                        verbose_name=_("Vat Amount"),
-                                        editable=False,default=Decimal('0.00'))
     discount_amount = models.DecimalField(max_digits=14, decimal_places=2, verbose_name=_("Discount Amount"),
                                           default=Decimal('0.00'))
-    registration_fee = models.DecimalField(max_digits=14, decimal_places=2, verbose_name=_("Registration Fee"),
+    # profit_margin = models.DecimalField(max_digits=14,
-                                           default=Decimal('0.00'))
+    #                                     decimal_places=2,
-    administration_fee = models.DecimalField(max_digits=14, decimal_places=2, verbose_name=_("Administration Fee"),
+    #                                     verbose_name=_("Profit Margin"),
-                                             default=Decimal('0.00'))
+    #                                     editable=False)
-    transportation_fee = models.DecimalField(max_digits=14, decimal_places=2, verbose_name=_("Transportation Fee"),
+    # vat_amount = models.DecimalField(max_digits=14,
-                                             default=Decimal('0.00'))
+    #                                     decimal_places=2,
-    custom_card_fee = models.DecimalField(max_digits=14, decimal_places=2, verbose_name=_("Custom Card Fee"),
+    #                                     verbose_name=_("Vat Amount"),
-                                          default=Decimal('0.00'))
+    #                                     editable=False,default=Decimal('0.00'))
+    # registration_fee = models.DecimalField(max_digits=14, decimal_places=2, verbose_name=_("Registration Fee"),
+    #                                        default=Decimal('0.00'))
+    # administration_fee = models.DecimalField(max_digits=14, decimal_places=2, verbose_name=_("Administration Fee"),
+    #                                          default=Decimal('0.00'))
+    # transportation_fee = models.DecimalField(max_digits=14, decimal_places=2, verbose_name=_("Transportation Fee"),
+    #                                          default=Decimal('0.00'))
+    # custom_card_fee = models.DecimalField(max_digits=14, decimal_places=2, verbose_name=_("Custom Card Fee"),
+    #                                       default=Decimal('0.00'))
     @property
     def total(self):
         """Calculate the total amount including VAT."""
@@ -304,15 +306,15 @@ class CarFinance(models.Model):
     def __str__(self):
         return f"Car: {self.car}, Selling Price: {self.selling_price}"
 
-    def save(self, *args, **kwargs):        
+    # def save(self, *args, **kwargs):        
-        self.full_clean()
+    #     self.full_clean()
-        try:
+    #     try:
-            price_after_discount = self.selling_price - self.discount_amount
+    #         price_after_discount = self.selling_price - self.discount_amount
-            self.profit_margin = price_after_discount - self.cost_price
+    #         self.profit_margin = price_after_discount - self.cost_price
-            self.vat_amount = settings.VAT_RATE
+    #         self.vat_amount = settings.VAT_RATE
-        except InvalidOperation as e:
+    #     except InvalidOperation as e:
-            raise ValidationError(_("Invalid decimal operation: %s") % str(e))
+    #         raise ValidationError(_("Invalid decimal operation: %s") % str(e))
-        super().save(*args, **kwargs)
+    #     super().save(*args, **kwargs)
 
     class Meta:
         verbose_name = _("Car Financial Details")
@@ -669,6 +671,13 @@ class SaleQuotation(models.Model):
         total_quantity = self.quotation_cars.aggregate(total=Sum('quantity'))['total']
         return total_quantity or 0
 
+    # @property
+    # def total(self):
+    #     total = self.quotation_cars.aggregate(total_price=Sum(F('car__finances__selling_price') * F('quantity')))
+    #     if total:
+    #         return float(total["total_price"]) * 0.15 + float(total["total_price"])
+    #     return 0
+        
     def confirm(self):
         """Confirm the quotation and lock financial details."""
         if self.status != "DRAFT":
@@ -701,6 +710,9 @@ class SaleQuotationCar(models.Model):
     )
     quantity = models.PositiveIntegerField(default=1, verbose_name=_("Quantity"))
    
+    @property
+    def finance(self):
+        return self.car.finances
     @property
     def financial_details(self):
         """
@@ -721,14 +733,14 @@ class SaleQuotationCar(models.Model):
             # "total_amount": car_finance.total,
         }
 
-    @property
+    # @property
-    def total_price(self):
+    # def total(self):
-        """
+    #     """
-        Calculate total price dynamically based on quantity and selling price.
+    #     Calculate total price dynamically based on quantity and selling price.
-        """
+    #     """
-        if not self.car.finances:
+    #     if not self.car.finances:
-            return Decimal("0.00")
+    #         return Decimal("0.00")
-        return self.car.finances.selling_price * self.quantity
+    #     return self.car.finances.selling_price * self.quantity
 
     def __str__(self):
         return f"{self.car} - Quotation #{self.quotation.id}"

inventory/utilities/financials.py

@@ -1,26 +1,36 @@
 
 from decimal import Decimal
 from django.conf import settings
+from inventory import models
 
 def calculate_vat(value):
         """Helper to calculate VAT dynamically for a given value."""
         vat_rate = getattr(settings, 'VAT_RATE', Decimal('0.15'))  # Default VAT rate
         return (value * vat_rate).quantize(Decimal('0.01'))
-def get_financial_value(instance,attribute,vat=False):
+# def get_financial_value(instance,attribute,vat=False):
+#     if vat:
+#         return calculate_vat(getattr(instance, attribute, Decimal('0.00')) if instance else Decimal('0.00'))
+#     return getattr(instance, attribute, Decimal('0.00')) if instance else Decimal('0.00')
+
+def get_financial_value(name,vat=False):
+    val = models.AdditionalServices.objects.filter(name=name).first()
+    if not val:
+        return 0
     if vat:
-        return calculate_vat(getattr(instance, attribute, Decimal('0.00')) if instance else Decimal('0.00'))
+        return (val.price * settings.VAT_RATE).quantize(Decimal('0.01'))
-    return getattr(instance, attribute, Decimal('0.00')) if instance else Decimal('0.00')
+    return val.price
+    
     
 def get_total_financials(instance,vat=False):
-    price_after_discount = get_financial_value(instance,"selling_price",vat) - get_financial_value(instance,"discount_amount",vat)
+    # price_after_discount = get_financial_value(instance,"selling_price",vat) - get_financial_value(instance,"discount_amount",vat)
-    subtotal = (
+    # subtotal = (
-        price_after_discount +
+    #     price_after_discount +
-        get_financial_value(instance,"registration_fee") +
+    #     get_financial_value("registration_fee") +
-        get_financial_value(instance,"administration_fee",vat) +
+    #     get_financial_value("administration_fee",vat) +
-        get_financial_value(instance,"transportation_fee",vat) +
+    #     get_financial_value("transportation_fee",vat) +
-        get_financial_value(instance,"custom_card_fee",vat))
+    #     get_financial_value("custom_card_fee",vat))
 
-    return subtotal
+    return 1000
 
 def get_total(instance):
     total = get_total_financials(instance)

inventory/utils.py

@@ -1,7 +1,7 @@
 import requests
 from django.conf import settings
 from django.utils.translation import gettext_lazy as _
-
+from inventory import models
 from inventory.utilities.financials import get_financial_value
 
 
@@ -32,48 +32,20 @@ def localize_some_words():
     return None
 
 
-
+def get_calculations(quotation):
-def get_calculations(instance,quotation):
     context = {}
-    context['vat_rate'] = settings.VAT_RATE
+    qc_len = quotation.quotation_cars.count()
-    context['total_sales_before_vat'] = sum(item.car.finances.selling_price * item.quantity for item in quotation.quotation_cars.all())
+    cars = [x.car for x in quotation.quotation_cars.all()]
-    context['vat_amount'] = sum(item.car.finances.vat_amount * item.quantity for item in quotation.quotation_cars.all())
+    finances = models.CarFinance.objects.filter(car__in=cars)
-    context['total_sales_after_vat'] = context['total_sales_before_vat'] + context['vat_amount']
-
-    # Additional Costs
-    total_quantity = quotation.total_quantity
-    context['administration_fee'] = sum(item.car.finances.administration_fee for item in quotation.quotation_cars.all())
-    context['transportation_fee'] = sum(item.car.finances.transportation_fee for item in quotation.quotation_cars.all())
-    context['custom_card_fee'] = sum(item.car.finances.custom_card_fee for item in quotation.quotation_cars.all())
-    context['registration_fee'] = sum(item.car.finances.registration_fee for item in quotation.quotation_cars.all())
-    context['administration_fee_vat'] = sum(get_financial_value(item.car.finances,"administration_fee",True) for item in quotation.quotation_cars.all())
-    context['transportation_fee_vat'] = sum(get_financial_value(item.car.finances,"transportation_fee",True) for item in quotation.quotation_cars.all())
-    context['custom_card_fee_vat'] = sum(get_financial_value(item.car.finances,"custom_card_fee",True) for item in quotation.quotation_cars.all())
-    context['administration_fee_total'] = sum(item.car.finances.administration_fee + context['administration_fee_vat'] for item in quotation.quotation_cars.all())
-    context['transportation_fee_total'] = sum(item.car.finances.transportation_fee + context['transportation_fee_vat'] for item in quotation.quotation_cars.all())
-    context['custom_card_fee_total'] = sum(item.car.finances.custom_card_fee + context['custom_card_fee_vat'] for item in quotation.quotation_cars.all())
-    context['registration_fee_total'] = sum(item.car.finances.registration_fee * total_quantity for item in quotation.quotation_cars.all())
     
+    services = models.AdditionalServices.objects.filter(additional_finances__in=finances).all()
+    data = [{"name":x.name,"price":x.price,"total_price":x.price * qc_len,"vated":float(x.price) * 0.15 * float(qc_len),"total_price_vat":float(x.price) + (float(x.price) * 0.15 * float(qc_len))} for x in services]
+    context["services"] = data
+    context["total_cost"] = 0
+    context["total_vat"] = 0
+    context["total_cost_vat"] = 0
+    for k in context["services"]:
+        context["total_cost"] += k["total_price"]
+        context["total_vat"] += k["vated"]
+    context["total_cost_vat"] = float(context["total_cost"])+float(context["total_vat"])
     return context
-# def get_calculations(quotation):
-#     context = {}
-#     context['vat_rate'] = settings.VAT_RATE
-#     context['total_sales_before_vat'] = sum(item.car.selling_price * item.quantity for item in quotation.quotation_cars.all())
-#     context['vat_amount'] = sum(item.car.vat_amount * item.quantity for item in quotation.quotation_cars.all())
-#     context['total_sales_after_vat'] = context['total_sales_before_vat'] + context['vat_amount']
-
-#     # Additional Costs
-#     total_quantity = quotation.total_quantity
-#     context['administration_fee'] = sum(item.car.administration_fee for item in quotation.quotation_cars.all())
-#     context['transportation_fee'] = sum(item.car.transportation_fee for item in quotation.quotation_cars.all())
-#     context['custom_card_fee'] = sum(item.car.custom_card_fee for item in quotation.quotation_cars.all())
-#     context['registration_fee'] = sum(item.car.registration_fee for item in quotation.quotation_cars.all())
-#     context['administration_fee_vat'] = sum(item.car.administration_fee_vat for item in quotation.quotation_cars.all())
-#     context['transportation_fee_vat'] = sum(item.car.transportation_fee_vat for item in quotation.quotation_cars.all())
-#     context['custom_card_fee_vat'] = sum(item.car.custom_card_fee_vat for item in quotation.quotation_cars.all())
-#     context['administration_fee_total'] = sum(item.car.administration_fee + context['administration_fee_vat'] for item in quotation.quotation_cars.all())
-#     context['transportation_fee_total'] = sum(item.car.transportation_fee + context['transportation_fee_vat'] for item in quotation.quotation_cars.all())
-#     context['custom_card_fee_total'] = sum(item.car.custom_card_fee + context['custom_card_fee_vat'] for item in quotation.quotation_cars.all())
-#     context['registration_fee_total'] = sum(item.car.registration_fee * total_quantity for item in quotation.quotation_cars.all())
-    
-#     return context

inventory/views.py

@@ -28,7 +28,7 @@ from django.contrib import messages
 from django.db.models import Sum, F, Count
 
 from inventory.mixins import AddDealerInstanceMixin
-from inventory.utils import get_calculations
+
 from .services import elm, decodevin,get_make,get_model,normalize_name
 from .services import elm, decodevin, get_make, get_model, normalize_name, get_ledger_data
 from . import models, forms
@@ -38,7 +38,7 @@ from django.contrib.auth.decorators import user_passes_test
 from django.contrib.messages.views import SuccessMessageMixin
 from django.contrib.auth.models import Group
 from django.contrib.auth import get_user_model  
-
+from .utils import get_calculations
 User = get_user_model()
 
 logger = logging.getLogger(__name__)
@@ -406,6 +406,11 @@ class CarFinanceUpdateView(LoginRequiredMixin,SuccessMessageMixin, UpdateView):
     def get_success_url(self):
         return reverse('car_detail', kwargs={'pk': self.object.car.pk})
     
+    def get_form_kwargs(self):
+        kwargs = super().get_form_kwargs()
+        kwargs['instance'] = self.get_object()
+        return kwargs
+
 
 class CarUpdateView(LoginRequiredMixin, SuccessMessageMixin,UpdateView):
     model = models.Car
@@ -709,8 +714,7 @@ class QuotationDetailView(LoginRequiredMixin,PermissionRequiredMixin, DetailView
         context = super().get_context_data(**kwargs)
         quotation = self.object
                         
-        # context_result = get_calculations(quotation)
+        context_result = get_calculations(quotation)
-        context_result = get_calculations(self.object,quotation)
         context.update(context_result)       
 
         return context
@@ -767,7 +771,7 @@ class UserListView(LoginRequiredMixin,PermissionRequiredMixin, ListView):
 class UserDetailView(LoginRequiredMixin,PermissionRequiredMixin, DetailView):
     model = models.Dealer
     template_name = "users/user_detail.html"
-    context_object_name = "user"
+    context_object_name = "user_"
     permission_required = ('inventory.view_dealer',)      
        
 class UserCreateView(LoginRequiredMixin,PermissionRequiredMixin,SuccessMessageMixin,AddDealerInstanceMixin, CreateView):

templates/inventory/car_detail.html

@@ -228,26 +228,22 @@
                 <th>{% trans "Selling Price" %}</th>
                 <td>{{ car.finances.selling_price }}</td>
               </tr>                          
-              <tr>
-                <td><small class="ms-5">{% trans "Administration Fee" %}</small></td>
-                <td><small>{{ car.finances.administration_fee }}</small></td>
-              </tr>
-              <tr>
-                <td><small class="ms-5">{% trans "Registration Fee" %}</small></td>
-                <td><small>{{ car.finances.registration_fee }}</small></td>
-              </tr>
-              <tr>
-                <td><small class="ms-5">{% trans "Transportation Fee" %}</small></td>
-                <td><small>{{ car.finances.transportation_fee }}</small></td>
-              </tr>
-              <tr>
-                <td><small class="ms-5">{% trans "Custom Card Fee" %}</small></td>
-                <td><small>{{ car.finances.custom_card_fee }}</small></td>
-              </tr>
               <tr>
                 <th>{% trans "Discount Amount" %}</th>
                 <td>{{ car.finances.discount_amount }} -</td>
               </tr>              
+              <tr>
+                <th>{% trans "Additional Fee" %}</th>
+                <td></td>
+              </tr>
+              {% if car.finances.additional_services.first.pk %}
+                {% for service in car.finances.additional_services.all %}
+                <tr>
+                  <td><small class="ms-5">{{service.name}}</small></td>
+                  <td><small>{{ service.price }}</small></td>
+                </tr>
+                {% endfor %}
+              {% endif %}
               <tr>
                 <th>{% trans "VAT Amount" %}</th>
                 <td>{{ car.finances.vat_amount }}</td>

templates/sales/quotation_detail.html

@@ -74,8 +74,8 @@
                         <td>{{ item.car.year }}</td>
                         <td>{{ item.quantity }}</td>
                         <td>{{ item.car.finances.selling_price }}</td>
-                        <td>{{ item.car.vat_amount }}</td>
+                        <td>{{ 0.15 }}</td>
-                        <td>{{ item.car.total }}</td>
+                        <td>{{ item.total }}</td>
                     </tr>
                     {% endfor %}
                 </tbody>
@@ -96,35 +96,26 @@
                     <tr>
                         <th>{% trans "Additions" %}</th>
                         <th>{% trans "Cost" %}</th>
-                        <th>{% trans "VAT" %}</th>
+                        <th>{% trans "VAT %" %}</th>
                         <th>{% trans "Total Cost with VAT" %}</th>
                     </tr>
                 </thead>
                 <tbody>
+                    {% for service in services %}
                     <tr>
-                        <td>{% trans "Administration Fee" %}</td>
+                        <td>{{service.name}}</td>
-                       <td>{{ administration_fee }}</td>
+                        <td>{{ service.price }}</td>
-                        <td>{{ administration_fee_vat }}</td>
+                        <td>{{ service.vated }}</td>
-                        <td>{{ administration_fee_total }}</td>
+                        <td>{{ service.total_price_vat }}</td>
                     </tr>
+                    {% endfor %}
                     <tr>
-                        <td>{% trans "Transportation Fee" %}</td>
+                        <td></td>
-                        <td>{{ transportation_fee }}</td>
+                        <td>{{ total_cost }}</td>
-                        <td>{{ transportation_fee_vat }}</td>
+                        <td>{{ total_vat }}</td>
-                        <td>{{ transportation_fee_total }}</td>
+                        <td>{{ total_cost_vat }}</td>
-                    </tr>
-                    <tr>
-                        <td>{% trans "Custom Card Fee" %}</td>
-                        <td>{{ custom_card_fee }}</td>
-                        <td>{{ custom_card_fee_vat }}</td>
-                        <td>{{ custom_card_fee_total }}</td>
-                    </tr>
-                    <tr>
-                        <td>{% trans "Registration Fee (No VAT)" %}</td>
-                        <td>{{ registration_fee }}</td>
-                        <td>{% trans "N/A" %}</td>
-                        <td>{{ registration_fee_total }}</td>
                     </tr>
+                                  
                 </tbody>
             </table>
         </div>

templates/users/user_detail.html

@@ -1,4 +1,5 @@
 {% extends "base.html" %}
+{% load static %}
 {% load i18n %}
 
 {% block title %}{{ _("View Customer") }}{% endblock title %}
@@ -27,7 +28,7 @@
                     </button>
                     <a type="button"
                        class="btn btn-sm btn-danger"
-                       href="{% url 'user_delete' user.id %}">
+                       href="{% url 'user_delete' user_.id %}">
                         {% trans 'Yes' %}
                     </a>
                 </div>
@@ -43,18 +44,18 @@
         <div class="card-body">
             <div class="row">
                 <div class="col-md-6">                    
-                    <p><strong>{{ _("Name") }}:</strong> {{ user.name }}</p>
+                    <p><strong>{{ _("Name") }}:</strong> {{ user_.name }}</p>
-                    <p><strong>{{ _("Arabic Name") }}:</strong> {{ user.arabic_name }}</p>                    
+                    <p><strong>{{ _("Arabic Name") }}:</strong> {{ user_.arabic_name }}</p>                    
                 </div>
                 <div class="col-md-6">                                        
-                    <p><strong>{{ _("Phone Number") }}:</strong> {{ user.phone_number }}</p>
+                    <p><strong>{{ _("Phone Number") }}:</strong> {{ user_.phone_number }}</p>
-                    <p><strong>{{ _("Address") }}:</strong> {{ user.address }}</p>
+                    <p><strong>{{ _("Address") }}:</strong> {{ user_.address }}</p>
-                    <p><strong>{{ _("Role") }}:</strong> {{ user.dealer_type }}</p>
+                    <p><strong>{{ _("Role") }}:</strong> {{ user_.dealer_type }}</p>
                 </div>
             </div>
         </div>
         <div class="card-footer d-flex ">
-            <a class="btn btn-sm btn-primary me-1" href="{% url 'user_update' user.id %}">
+            <a class="btn btn-sm btn-primary me-1" href="{% url 'user_update' user_.id %}">
                 <!--<i class="bi bi-pencil-square"></i> -->
                 {{ _("Edit") }}
             </a>

templates/users/user_list.html

@@ -11,7 +11,6 @@
       <main class="d-grid gap-4 p-1">
         <div class="row g-4">
           <div class="col-lg-6 col-xl-12">            
-
         <div class="container-fluid p-2">
             <form method="get">
                 <div class="input-group input-group-sm">