update3

Reviewed-on: #75
Reviewed-by: ismail <ismail.mosa.ibrahim@gmail.com>

inventory/urls.py

@@ -279,6 +279,11 @@ urlpatterns = [
         views.CarFinanceUpdateView.as_view(),
         name="car_finance_update",
     ),
+    path(
+        "htmx/cars/bulk_update_car_price/",
+        views.bulk_update_car_price,
+        name="bulk_update_car_price",
+    ),
     path("ajax/", views.AjaxHandlerView.as_view(), name="ajax_handler"),
     path(
         "cars/<slug:slug>/add-color/", views.CarColorCreate.as_view(), name="add_color"

inventory/views.py

@@ -23,6 +23,7 @@ from inventory.models import Status as LeadStatus
 from django.db import IntegrityError
 from background_task.models import Task
 from django.views.generic import FormView
+from django.views.decorators.http import require_http_methods
 from django.db.models.deletion import RestrictedError
 from django.http.response import StreamingHttpResponse
 from django.core.exceptions import ImproperlyConfigured, ValidationError
@@ -9288,3 +9289,26 @@ def upload_cars(request,pk=None):
     return render(request, 'csv_upload.html',{"make_data":models.CarMake.objects.all(),"form":form,"item":item})
 ###############################################################
 ###############################################################
+
+@require_http_methods(["POST"])
+def bulk_update_car_price(request):
+    if request.method == "POST":
+        cars = request.POST.getlist('car')
+        price = request.POST.get('price')
+
+        if not price or int(price) <= 0:
+            messages.error(request, "Please enter a valid price")
+        elif not cars:
+            messages.error(request, "No cars selected for price update")
+        else:
+            for car_pk in cars:
+                car_finance , created = models.CarFinance.objects.get_or_create(car__pk=car_pk, cost_price=Decimal(price),selling_price=0)
+                if not created:
+                  car_finance.cost_price = Decimal(price)
+                  car_finance.selling_price = 0
+                  car_finance.save()
+            messages.success(request, "Price updated successfully")
+
+        response = HttpResponse()
+        response['HX-Redirect'] = reverse('car_list')
+        return response

templates/bill/bill_detail.html

@@ -152,7 +152,7 @@
                     <th class="sort white-space-nowrap align-middle " scope="col">{% trans 'PO' %}</th>
                   </tr>
                 </thead>
-               <tbody class="list" id="project-list-table-body">
+                <tbody class="list fs-9" id="project-list-table-body">
                   {% for bill_item in itemtxs_qs %}
                     <tr>
                       <td class="align-middle white-space-nowrap">
@@ -171,7 +171,7 @@
                       </td>
                       <td class="align-middle white-space-nowrap">
                         <span class="text-xs font-weight-bold">
-                      {% currency_symbol %}{{ bill_item.unit_cost | currency_format }}
+                          {{ bill_item.unit_cost | currency_format }}
                         </span>
                       </td>
                       <td class="align-middle white-space-nowrap">
@@ -179,7 +179,7 @@
                       </td>
                       <td class="align-middle white-space-nowrap">
                         <span class="text-xs font-weight-bold">
-                      {% currency_symbol %}{{ bill_item.total_amount | currency_format }}
+                          {{ bill_item.total_amount | currency_format }}
                         </span>
                       </td>
                       <td class="align-middle white-space-nowrap pe-2">

templates/bill/transactions/tags/txs_table.html

@@ -2,6 +2,7 @@
 {% load django_ledger %}
 
 {% if style == 'detail' %}
+
     <div class="table-responsive">
         <table class="table  table-hover">
             <thead >
@@ -15,24 +16,24 @@
                     <th class="sort white-space-nowrap align-middle"  scope="col">{% trans 'Description' %}</th>
                 </tr>
             </thead>
-            <tbody>
+            <tbody class="fs-9">
                 {% for transaction_model in transaction_model_qs %}
                     <tr>
-                        <td class="sort white-space-nowrap align-middle ps-1" scope="col" >{{ transaction_model.timestamp }}</td>
+                        <td class=" white-space-nowrap align-middle ps-2" scope="col" >{{ transaction_model.timestamp }}</td>
-                        <td class="sort white-space-nowrap align-middle"  scope="col">{{ transaction_model.account_code }}</td>
+                        <td class=" white-space-nowrap align-middle"  scope="col">{{ transaction_model.account_code }}</td>
-                        <td class="sort white-space-nowrap align-middle"  scope="col">{{ transaction_model.account_name }}</td>
+                        <td class=" white-space-nowrap align-middle" scope="col">{{ transaction_model.account_name }}</td>
-                        <td class="sort white-space-nowrap align-middle"  scope="col">{% if transaction_model.entity_unit_name %}{{ transaction_model.entity_unit_name }}{% endif %}</td>
+                        <td class=" white-space-nowrap align-middle"  scope="col">{% if transaction_model.entity_unit_name %}{{ transaction_model.entity_unit_name }}{% endif %}</td>
-                        <td class="sort white-space-nowrap align-middle"  scope="col">{% if transaction_model.is_credit %}${{ transaction_model.amount | currency_format }}{% endif %}</td>
+                        <td class=" white-space-nowrap align-middle" scope="col">{% if transaction_model.is_credit %}{{ transaction_model.amount | currency_format }}{% endif %}</td>
-                        <td class="sort white-space-nowrap align-middle"  scope="col">{% if transaction_model.is_debit %}${{ transaction_model.amount | currency_format }}{% endif %}</td>
+                        <td class=" white-space-nowrap align-middle"  scope="col">{% if transaction_model.is_debit %}{{ transaction_model.amount | currency_format }}{% endif %}</td>
-                        <td class="sort white-space-nowrap align-middle pe-1"  scope="col">{% if transaction_model.description %}{{ transaction_model.description }}{% endif %}</td>
+                        <td class=" white-space-nowrap align-middle pe-2"  scope="col">{% if transaction_model.description %}{{ transaction_model.description }}{% endif %}</td>
                     </tr>
                 {% endfor %}
                 <tr class="fw-bold">
-                    <td class="sort white-space-nowrap align-middle"  colspan="3"></td>
+                    <td class=" white-space-nowrap align-middle"  colspan="3"></td>
-                    <td  class="class="sort white-space-nowrap align-middle" " scope="col">{% trans 'Total' %}</td>
+                    <td  class=" white-space-nowrap align-middle" scope="col">{% trans 'Total' %}</td>
-                    <td class="sort white-space-nowrap align-middle"  scope="col">{% currency_symbol %}{{ total_credits | currency_format }}</td>
+                    <td class=" white-space-nowrap align-middle" scope="col">{% currency_symbol %}{{ total_credits | currency_format }}</td>
-                    <td class="sort white-space-nowrap align-middle"  scope="col">{% currency_symbol %}{{ total_debits | currency_format }}</td>
+                    <td class=" white-space-nowrap align-middle" scope="col">{% currency_symbol %}{{ total_debits | currency_format }}</td>
-                    <td class="sort white-space-nowrap align-middle"  scope="col"></td>
+                    <td class=" white-space-nowrap align-middle"  scope="col"></td>
                 </tr>
             </tbody>
         </table>
@@ -42,29 +43,28 @@
         <table class="table  table-hover">
             <thead>
                 <tr class="bg-body-highlight">
-                    <th>{% trans 'Account' %}</th>
+                    <th class="sort white-space-nowrap align-middle">{% trans 'Account Name' %}</th>
-                    <th>{% trans 'Account Name' %}</th>
+                    <th class="sort white-space-nowrap align-middle">{% trans 'Credit' %}</th>
-                    <th>{% trans 'Credit' %}</th>
+                    <th class="sort white-space-nowrap align-middle">{% trans 'Debit' %}</th>
-                    <th>{% trans 'Debit' %}</th>
+                    <th class="sort white-space-nowrap align-middle">{% trans 'Description' %}</th>
-                    <th>{% trans 'Description' %}</th>
                 </tr>
             </thead>
-            <tbody>
+            <tbody class="fs-9">
                 {% for transaction_model in transaction_model_qs %}
                     <tr>
-                        <td>{{ transaction_model.account_code }}</td>
+                        <td class=" white-space-nowrap align-middle ps-2">{{ transaction_model.account_code }}</td>
-                        <td>{{ transaction_model.account_name }}</td>
+                        <td class=" white-space-nowrap align-middle">{{ transaction_model.account_name }}</td>
-                        <td>{% if transaction_model.is_credit %}${{ transaction_model.amount | currency_format }}{% endif %}</td>
+                        <td class=" white-space-nowrap align-middle">{% if transaction_model.is_credit %}{{ transaction_model.amount | currency_format }}{% endif %}</td>
-                        <td>{% if transaction_model.is_debit %}${{ transaction_model.amount | currency_format }}{% endif %}</td>
+                        <td class=" white-space-nowrap align-middle">{% if transaction_model.is_debit %}{{ transaction_model.amount | currency_format }}{% endif %}</td>
-                        <td>{% if transaction_model.description %}{{ transaction_model.description }}{% endif %}</td>
+                        <td class=" white-space-nowrap align-middle pe-2">{% if transaction_model.description %}{{ transaction_model.description }}{% endif %}</td>
                     </tr>
                 {% endfor %}
                 <tr class="fw-bold">
-                    <td colspan="1"></td>
+                    <td class=" white-space-nowrap align-middle" colspan="1"></td>
-                    <td class="text-end">{% trans 'Total' %}</td>
+                    <td class=" white-space-nowrap align-middle text-end">{% trans 'Total' %}</td>
-                    <td>{% currency_symbol %}{{ total_credits | currency_format }}</td>
+                    <td class="white-space-nowrap align-middle">{% currency_symbol %}{{ total_credits | currency_format }}</td>
-                    <td>{% currency_symbol %}{{ total_debits | currency_format }}</td>
+                    <td class=" white-space-nowrap align-middle">{% currency_symbol %}{{ total_debits | currency_format }}</td>
-                    <td></td>
+                    <td class="white-space-nowrap align-middle"></td>
                 </tr>
             </tbody>
         </table>

templates/inventory/car_list_view.html

@@ -145,8 +145,14 @@
                         hx-on::after-request="filter_after_request()">{{ _("Search") }}</button>
             </div>
             <div class="row">
-              <form class="update-price-form d-flex flex-row align-items-center ms-auto w-25 d-none" action="" method="post" style="float:right;">
+                <form hx-boost='true' action="{% url 'bulk_update_car_price' %}" method="post"
-                <input class="form-control me-2" type="number" placeholder='{{ _("Search") }}' name="price" aria-label="Price">
+                      hx-include=".car-checkbox"
+                      class="update-price-form d-flex flex-row align-items-center ms-auto w-25 d-none" style="float:right;">
+                    {% csrf_token %}
+                    <div class="form-floating me-2">
+                        <input class="form-control" type="number" placeholder='{{ _("Search") }}' name="price" aria-label="Price" id="price">
+                        <label for="price">{{ _("Price") }}</label>
+                    </div>
                     <button class="btn btn-outline-primary" type="submit">{{ _("Search") }}</button>
                 </form>
                 <div class="table-responsive scrollbar transition">
@@ -186,7 +192,7 @@
                                     <tr class="position-static">
                                         <td class="align-middle white-space-nowrap">
                                             <div class="form-check">
-                                          <input class="form-check-input car-checkbox" type="checkbox" name="car" id="car-{{car.pk}}">
+                                                <input class="form-check-input car-checkbox" type="checkbox" name="car" value="{{ car.pk }}" id="car-{{car.pk}}">
                                             </div>
                                         </td>
                                         <td class="align-middle white-space-nowrap ps-1">
@@ -313,18 +319,31 @@
 
         document.getElementById('select-all').addEventListener('change', function() {
             const checkboxes = document.querySelectorAll('#project-list-table-body input[type="checkbox"]');
-          checkboxes.forEach(checkbox => {
-              checkbox.checked = this.checked;
-          });
-      });
-      document.getElementById('car-checkbox').addEventListener('change', function() {
-          const form = document.querySelector('.update-price-form');
             if (this.checked) {
+                checkboxes.forEach(checkbox => checkbox.checked = true);
+            } else {
+                checkboxes.forEach(checkbox => checkbox.checked = false);
+            }
+            updateFormVisibility();
+        });
+
+        const cbox = document.querySelectorAll('.car-checkbox');
+        cbox.forEach(checkbox => {
+            checkbox.addEventListener('change', function() {
+                updateFormVisibility();
+            });
+        });
+
+        function updateFormVisibility() {
+            const form = document.querySelector('.update-price-form');
+            const checkedCount = document.querySelectorAll('.car-checkbox:checked').length;
+            const submitButton = form.querySelector('button[type="submit"]');
+            if (checkedCount > 0) {
                 form.classList.remove('d-none');
+                submitButton.textContent = `Update Cost Price (${checkedCount})`;
             } else {
                 form.classList.add('d-none');
             }
-      });
+        }
-
     </script>
 {% endblock customJS %}

templates/purchase_orders/po_form.html

@@ -10,7 +10,7 @@
   {% endif %}
 {% endblock %}
 {% block content %}
-  <div class="row">
+  <div class="row mt-4">
     <div class="row">
       <div class="col-xl-9">
         <div class="d-sm-flex justify-content-between">