diff --git a/core/__pycache__/models.cpython-311.pyc b/core/__pycache__/models.cpython-311.pyc index 933112c..a1d345b 100644 Binary files a/core/__pycache__/models.cpython-311.pyc and b/core/__pycache__/models.cpython-311.pyc differ diff --git a/core/__pycache__/urls.cpython-311.pyc b/core/__pycache__/urls.cpython-311.pyc index cd848e1..a855790 100644 Binary files a/core/__pycache__/urls.cpython-311.pyc and b/core/__pycache__/urls.cpython-311.pyc differ diff --git a/core/__pycache__/views.cpython-311.pyc b/core/__pycache__/views.cpython-311.pyc index 0c44493..6fb6114 100644 Binary files a/core/__pycache__/views.cpython-311.pyc and b/core/__pycache__/views.cpython-311.pyc differ diff --git a/core/migrations/0016_expensecategory_expense.py b/core/migrations/0016_expensecategory_expense.py new file mode 100644 index 0000000..98ccf52 --- /dev/null +++ b/core/migrations/0016_expensecategory_expense.py @@ -0,0 +1,43 @@ +# Generated by Django 5.2.7 on 2026-02-02 17:15 + +import django.db.models.deletion +import django.utils.timezone +from django.conf import settings +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('core', '0015_userprofile'), + migrations.swappable_dependency(settings.AUTH_USER_MODEL), + ] + + operations = [ + migrations.CreateModel( + name='ExpenseCategory', + fields=[ + ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('name_en', models.CharField(max_length=100, verbose_name='Name (English)')), + ('name_ar', models.CharField(max_length=100, verbose_name='Name (Arabic)')), + ('description', models.TextField(blank=True, verbose_name='Description')), + ], + options={ + 'verbose_name_plural': 'Expense Categories', + }, + ), + migrations.CreateModel( + name='Expense', + fields=[ + ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')), + ('amount', models.DecimalField(decimal_places=3, max_digits=15, verbose_name='Amount')), + ('date', models.DateField(default=django.utils.timezone.now, verbose_name='Date')), + ('description', models.TextField(blank=True, verbose_name='Description')), + ('attachment', models.FileField(blank=True, null=True, upload_to='expense_attachments/', verbose_name='Attachment')), + ('created_at', models.DateTimeField(auto_now_add=True)), + ('created_by', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='expenses', to=settings.AUTH_USER_MODEL)), + ('payment_method', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='expenses', to='core.paymentmethod')), + ('category', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='expenses', to='core.expensecategory')), + ], + ), + ] diff --git a/core/migrations/__pycache__/0016_expensecategory_expense.cpython-311.pyc b/core/migrations/__pycache__/0016_expensecategory_expense.cpython-311.pyc new file mode 100644 index 0000000..4a3e0c7 Binary files /dev/null and b/core/migrations/__pycache__/0016_expensecategory_expense.cpython-311.pyc differ diff --git a/core/models.py b/core/models.py index 6d25482..4c5b59e 100644 --- a/core/models.py +++ b/core/models.py @@ -104,6 +104,30 @@ class PaymentMethod(models.Model): def __str__(self): return f"{self.name_en} / {self.name_ar}" +class ExpenseCategory(models.Model): + name_en = models.CharField(_("Name (English)"), max_length=100) + name_ar = models.CharField(_("Name (Arabic)"), max_length=100) + description = models.TextField(_("Description"), blank=True) + + class Meta: + verbose_name_plural = _("Expense Categories") + + def __str__(self): + return f"{self.name_en} / {self.name_ar}" + +class Expense(models.Model): + category = models.ForeignKey(ExpenseCategory, on_delete=models.CASCADE, related_name="expenses") + amount = models.DecimalField(_("Amount"), max_digits=15, decimal_places=3) + date = models.DateField(_("Date"), default=timezone.now) + description = models.TextField(_("Description"), blank=True) + payment_method = models.ForeignKey(PaymentMethod, on_delete=models.SET_NULL, null=True, blank=True, related_name="expenses") + created_by = models.ForeignKey(User, on_delete=models.SET_NULL, null=True, blank=True, related_name="expenses") + attachment = models.FileField(_("Attachment"), upload_to="expense_attachments/", blank=True, null=True) + created_at = models.DateTimeField(auto_now_add=True) + + def __str__(self): + return f"Expense {self.id} - {self.amount} ({self.category.name_en})" + class Sale(models.Model): PAYMENT_TYPE_CHOICES = [ ('cash', _('Cash')), @@ -357,4 +381,4 @@ def create_user_profile(sender, instance, created, **kwargs): def save_user_profile(sender, instance, **kwargs): if not hasattr(instance, 'profile'): UserProfile.objects.create(user=instance) - instance.profile.save() + instance.profile.save() \ No newline at end of file diff --git a/core/templates/base.html b/core/templates/base.html index 96b77d5..dd1e563 100644 --- a/core/templates/base.html +++ b/core/templates/base.html @@ -121,6 +121,26 @@ + +
  • + + {% trans "Expenses" %} + + + +
    • +
  • @@ -276,4 +296,4 @@ {% block scripts %}{% endblock %} - \ No newline at end of file + diff --git a/core/templates/core/customers.html b/core/templates/core/customers.html index f32865c..e6c1f75 100644 --- a/core/templates/core/customers.html +++ b/core/templates/core/customers.html @@ -69,6 +69,7 @@ + {% include "core/pagination.html" with page_obj=customers %} diff --git a/core/templates/core/expense_categories.html b/core/templates/core/expense_categories.html new file mode 100644 index 0000000..f5c7c14 --- /dev/null +++ b/core/templates/core/expense_categories.html @@ -0,0 +1,202 @@ +{% extends 'base.html' %} +{% load i18n static %} + +{% block title %}{% trans "Expense Categories" %} | {{ site_settings.business_name }}{% endblock %} + +{% block content %} +
    +
    +
    +

    {% trans "Expense Categories" %}

    +     +
    + +
    + + {% if messages %} +
    + {% for message in messages %} + + {% endfor %} +
    + {% endif %} + +
    +
    + + + + + + + + + + + {% for category in categories %} + + + + + + + + + + {% empty %} + + + + {% endfor %} + +
    {% trans "Category Name" %}{% trans "Description" %}{% trans "Expenses Count" %}{% trans "Actions" %}
    +
    {{ category.name_en }}
    +
    {{ category.name_ar }}
    +     		{{ category.description|default:"-" }} + + {{ category.expenses.count }} + + + + +
    +
    + +

    {% trans "No categories found." %}

    +
    +
    +
    +
    +
    + + + + + + + + +{% endblock %} diff --git a/core/templates/core/expenses.html b/core/templates/core/expenses.html new file mode 100644 index 0000000..21558ca --- /dev/null +++ b/core/templates/core/expenses.html @@ -0,0 +1,220 @@ +{% extends 'base.html' %} +{% load i18n static %} + +{% block title %}{% trans "Expenses" %} | {{ site_settings.business_name }}{% endblock %} + +{% block content %} +
    +
    +
    +

    {% trans "Expenses" %}

    + +
    +
    + + + {% trans "Expense Categories" %} + +
    +
    + + {% if messages %} +
    + {% for message in messages %} + + {% endfor %} +
    + {% endif %} + + +
    +
    +
    +
    +
    {% trans "Total Expenses" %}
    +

    {{ site_settings.currency_symbol }} {{ total_expenses|floatformat:site_settings.decimal_places }}

    +
    +
    +
    +
    + + +
    +
    +
    +
    + + +
    +
    + + +
    +
    + + +
    +
    + + {% trans "Reset" %} +
    +
    +
    +
    + + +
    +
    + + + + + + + + + + + + + {% for expense in expenses %} + + + + + + + + + + + + {% empty %} + + + + {% endfor %} + +
    {% trans "Date" %}{% trans "Category" %}{% trans "Description" %}{% trans "Payment Method" %}{% trans "Amount" %}{% trans "Actions" %}
    + {{ expense.date|date:"Y-m-d" }} + + + {{ expense.category.name_en }} / {{ expense.category.name_ar }} + + +
    + {{ expense.description|default:"-" }} +
    +     		{{ expense.payment_method.name_en|default:"-" }}{{ site_settings.currency_symbol }} {{ expense.amount|floatformat:site_settings.decimal_places }} + {% if expense.attachment %} + + + + {% endif %} + +
    +
    + +

    {% trans "No expense records found." %}

    +
    +
    +
    + {% include "core/pagination.html" with page_obj=expenses %} +
    +
    + + + +{% endblock %} diff --git a/core/templates/core/inventory.html b/core/templates/core/inventory.html index 265af33..19f84ef 100644 --- a/core/templates/core/inventory.html +++ b/core/templates/core/inventory.html @@ -165,6 +165,7 @@ + {% include "core/pagination.html" with page_obj=products %} diff --git a/core/templates/core/invoice_edit.html b/core/templates/core/invoice_edit.html new file mode 100644 index 0000000..b67a66d --- /dev/null +++ b/core/templates/core/invoice_edit.html @@ -0,0 +1,306 @@ +{% extends 'base.html' %} +{% load i18n %} + +{% block title %}{% trans "Edit Invoice" %} | {{ site_settings.business_name }}{% endblock %} + +{% block content %} +
    +
    + +
    +
    +
    +
    +
    {% trans "Edit Sales Invoice" %} #[[ invoiceNumber || '{{ sale.id }}' ]]
    + + {% trans "Back to List" %} + +
    +
    +
    + +
    +
    + + +
    +
    + + +
    +
    + + +
    + +
    + + +
    + +
    +
    + +
    +
    +
    + + +
    + + + + + + + + + + + + + + + + + + + + + + +
    {% trans "Product" %}{% trans "Unit Price" %}{% trans "Quantity" %}{% trans "Total" %}
    +
    [[ item.name_en ]]
    +
    [[ item.sku ]]
    +     		+ + + + [[ currencySymbol ]][[ (item.price * item.quantity).toFixed(3) ]] + +
    + {% trans "Search and add products to this invoice." %} +
    +
    +
    +
    +
    + + +
    +
    +
    +
    {% trans "Invoice Summary" %}
    + +
    + {% trans "Subtotal" %} + [[ currencySymbol ]][[ subtotal.toFixed(3) ]] +
    + +
    + {% trans "Discount" %} +
    + +
    +
    + +
    + +
    +

    {% trans "Grand Total" %}

    +

    [[ currencySymbol ]][[ grandTotal.toFixed(3) ]]

    +
    + + +
    + + +
    + +
    + + +
    + +
    + + +
    + +
    + + +
    + +
    + + +
    + +
    + +
    +
    +
    +
    +
    +
    + + + + +{% endblock %} \ No newline at end of file diff --git a/core/templates/core/invoices.html b/core/templates/core/invoices.html index a72ff3b..648c046 100644 --- a/core/templates/core/invoices.html +++ b/core/templates/core/invoices.html @@ -26,6 +26,46 @@ {% endif %} + +
    +
    +
    +
    + + +
    +
    + + +
    +
    + + +
    +
    + + +
    +
    + + {% trans "Reset" %} +
    +
    +
    +
    +
    @@ -69,6 +109,9 @@ + + + {% if sale.balance_due > 0 %}
    + {% include "core/pagination.html" with page_obj=sales %}
    diff --git a/core/templates/core/pagination.html b/core/templates/core/pagination.html new file mode 100644 index 0000000..cb80d7f --- /dev/null +++ b/core/templates/core/pagination.html @@ -0,0 +1,47 @@ +{% if page_obj.has_other_pages %} + +{% endif %} diff --git a/core/templates/core/purchase_returns.html b/core/templates/core/purchase_returns.html index 5829777..ab1b764 100644 --- a/core/templates/core/purchase_returns.html +++ b/core/templates/core/purchase_returns.html @@ -95,6 +95,7 @@ + {% include "core/pagination.html" with page_obj=returns %} diff --git a/core/templates/core/purchases.html b/core/templates/core/purchases.html index 4612be4..2dd1e00 100644 --- a/core/templates/core/purchases.html +++ b/core/templates/core/purchases.html @@ -150,6 +150,7 @@ + {% include "core/pagination.html" with page_obj=purchases %} diff --git a/core/templates/core/quotations.html b/core/templates/core/quotations.html index e8d5a1e..9ce1e37 100644 --- a/core/templates/core/quotations.html +++ b/core/templates/core/quotations.html @@ -135,6 +135,7 @@ + {% include "core/pagination.html" with page_obj=quotations %} diff --git a/core/templates/core/sales_returns.html b/core/templates/core/sales_returns.html index 38da94d..bed1343 100644 --- a/core/templates/core/sales_returns.html +++ b/core/templates/core/sales_returns.html @@ -95,6 +95,7 @@ + {% include "core/pagination.html" with page_obj=returns %} diff --git a/core/templates/core/suppliers.html b/core/templates/core/suppliers.html index e74c9d6..5fa1e18 100644 --- a/core/templates/core/suppliers.html +++ b/core/templates/core/suppliers.html @@ -64,6 +64,7 @@ + {% include "core/pagination.html" with page_obj=suppliers %} diff --git a/core/templates/core/users.html b/core/templates/core/users.html index ffd5525..9bbcc96 100644 --- a/core/templates/core/users.html +++ b/core/templates/core/users.html @@ -103,6 +103,7 @@ + {% include "core/pagination.html" with page_obj=users %} diff --git a/core/urls.py b/core/urls.py index 733c743..b8392f9 100644 --- a/core/urls.py +++ b/core/urls.py @@ -20,6 +20,7 @@ urlpatterns = [ path('invoices//', views.invoice_detail, name='invoice_detail'), path('invoices/payment//', views.add_sale_payment, name='add_sale_payment'), path('invoices/delete//', views.delete_sale, name='delete_sale'), + path("invoices/edit//", views.edit_invoice, name="edit_invoice"), # Quotations path('quotations/', views.quotations, name='quotations'), @@ -49,8 +50,16 @@ urlpatterns = [ path('purchases/returns/delete//', views.delete_purchase_return, name='delete_purchase_return'), path('api/create-purchase-return/', views.create_purchase_return_api, name='create_purchase_return_api'), + # Expenses + path('expenses/', views.expenses_view, name='expenses'), + path('expenses/create/', views.expense_create_view, name='expense_create'), + path('expenses/delete//', views.expense_delete_view, name='expense_delete'), + path('expenses/categories/', views.expense_categories_view, name='expense_categories'), + path('expenses/categories/delete//', views.expense_category_delete_view, name='expense_category_delete'), + # API / Actions path('api/create-sale/', views.create_sale_api, name='create_sale_api'), + path('api/update-sale//', views.update_sale_api, name='update_sale_api'), path('api/create-purchase/', views.create_purchase_api, name='create_purchase_api'), # POS Held Sales diff --git a/core/views.py b/core/views.py index 42dcb7a..38fa5a5 100644 --- a/core/views.py +++ b/core/views.py @@ -1,3 +1,4 @@ +from django.core.paginator import Paginator import decimal from django.contrib.auth.models import User, Group, Permission from django.urls import reverse @@ -9,7 +10,7 @@ from django.db.models.functions import TruncDate, TruncMonth from django.http import JsonResponse from django.views.decorators.csrf import csrf_exempt from django.contrib.auth.decorators import login_required -from .models import ( +from .models import ( Expense, ExpenseCategory, Product, Sale, Category, Unit, Customer, Supplier, Purchase, PurchaseItem, PurchasePayment, SaleItem, SalePayment, SystemSetting, @@ -73,7 +74,10 @@ def index(request): @login_required def inventory(request): - products = Product.objects.all().select_related('category', 'unit', 'supplier') + products_list = Product.objects.all().select_related('category', 'unit', 'supplier').order_by('-created_at') + paginator = Paginator(products_list, 25) + page_number = request.GET.get('page') + products = paginator.get_page(page_number) categories = Category.objects.all() suppliers = Supplier.objects.all() units = Unit.objects.all() @@ -109,13 +113,19 @@ def pos(request): @login_required def customers(request): - customers_list = Customer.objects.all().annotate(total_sales=Sum('sales__total_amount')) + customers_qs = Customer.objects.all().annotate(total_sales=Sum('sales__total_amount')).order_by('name') + paginator = Paginator(customers_qs, 25) + page_number = request.GET.get('page') + customers_list = paginator.get_page(page_number) context = {'customers': customers_list} return render(request, 'core/customers.html', context) @login_required def suppliers(request): - suppliers_list = Supplier.objects.all() + suppliers_qs = Supplier.objects.all().order_by('name') + paginator = Paginator(suppliers_qs, 25) + page_number = request.GET.get('page') + suppliers_list = paginator.get_page(page_number) context = {'suppliers': suppliers_list} return render(request, 'core/suppliers.html', context) @@ -123,8 +133,14 @@ def suppliers(request): @login_required def purchases(request): - purchases_list = Purchase.objects.all().select_related('supplier', 'created_by').order_by('-created_at') - suppliers_list = Supplier.objects.all() + purchases_qs = Purchase.objects.all().select_related('supplier', 'created_by').order_by('-created_at') + paginator = Paginator(purchases_qs, 25) + page_number = request.GET.get('page') + purchases_list = paginator.get_page(page_number) + suppliers_qs = Supplier.objects.all().order_by('name') + paginator = Paginator(suppliers_qs, 25) + page_number = request.GET.get('page') + suppliers_list = paginator.get_page(page_number) payment_methods = PaymentMethod.objects.filter(is_active=True) context = { 'purchases': purchases_list, @@ -266,17 +282,38 @@ def delete_purchase(request, pk): @login_required def invoice_list(request): - sales = Sale.objects.all().select_related('customer', 'created_by').order_by('-created_at') + sales = Sale.objects.all().select_related("customer", "created_by") + + # Filtering + start_date = request.GET.get("start_date") + end_date = request.GET.get("end_date") + customer_id = request.GET.get("customer") + status = request.GET.get("status") + + if start_date: + sales = sales.filter(created_at__date__gte=start_date) + if end_date: + sales = sales.filter(created_at__date__lte=end_date) + if customer_id: + sales = sales.filter(customer_id=customer_id) + if status: + sales = sales.filter(status=status) + + sales = sales.order_by("-created_at") + paginator = Paginator(sales, 25) + page_number = request.GET.get("page") + sales = paginator.get_page(page_number) + customers = Customer.objects.all() payment_methods = PaymentMethod.objects.filter(is_active=True) - return render(request, 'core/invoices.html', { - 'sales': sales, - 'customers': customers, - 'payment_methods': payment_methods + return render(request, "core/invoices.html", { + "sales": sales, + "customers": customers, + "payment_methods": payment_methods }) - @login_required def invoice_create(request): + products = Product.objects.filter(is_active=True) customers = Customer.objects.all() payment_methods = PaymentMethod.objects.filter(is_active=True) @@ -292,8 +329,37 @@ def invoice_detail(request, pk): settings = SystemSetting.objects.first() return render(request, 'core/invoice_detail.html', {'sale': sale, 'settings': settings}) -@csrf_exempt @login_required +def edit_invoice(request, pk): + sale = get_object_or_404(Sale, pk=pk) + # Prepare cart items for JSON + cart_items = [] + for item in sale.items.all(): + cart_items.append({ + 'id': item.product.id, + 'name_en': item.product.name_en, + 'sku': item.product.sku, + 'price': float(item.unit_price), + 'quantity': item.quantity + }) + + customers = Customer.objects.all() + products = Product.objects.filter(is_active=True) + payment_methods = PaymentMethod.objects.filter(is_active=True) + + # Find initial payment method + initial_payment = sale.payments.filter(notes='Initial payment').first() + payment_method_id = initial_payment.payment_method_id if initial_payment else '' + + return render(request, 'core/invoice_edit.html', { + 'sale': sale, + 'customers': customers, + 'products': products, + 'payment_methods': payment_methods, + 'cart_json': json.dumps(cart_items), + 'payment_method_id': payment_method_id + }) + def create_sale_api(request): if request.method == 'POST': try: @@ -482,7 +548,10 @@ def delete_sale(request, pk): @login_required def quotations(request): - quotations_list = Quotation.objects.all().select_related('customer', 'created_by').order_by('-created_at') + quotations_qs = Quotation.objects.all().select_related('customer', 'created_by').order_by('-created_at') + paginator = Paginator(quotations_qs, 25) + page_number = request.GET.get('page') + quotations_list = paginator.get_page(page_number) customers = Customer.objects.all() return render(request, 'core/quotations.html', {'quotations': quotations_list, 'customers': customers}) @@ -594,7 +663,10 @@ def delete_quotation(request, pk): @login_required def sales_returns(request): - returns = SaleReturn.objects.all().select_related('customer', 'created_by').order_by('-created_at') + returns_qs = SaleReturn.objects.all().select_related('customer', 'created_by').order_by('-created_at') + paginator = Paginator(returns_qs, 25) + page_number = request.GET.get('page') + returns = paginator.get_page(page_number) return render(request, 'core/sales_returns.html', {'returns': returns}) @login_required @@ -677,7 +749,10 @@ def delete_sale_return(request, pk): @login_required def purchase_returns(request): - returns = PurchaseReturn.objects.all().select_related('supplier', 'created_by').order_by('-created_at') + returns_qs = PurchaseReturn.objects.all().select_related('supplier', 'created_by').order_by('-created_at') + paginator = Paginator(returns_qs, 25) + page_number = request.GET.get('page') + returns = paginator.get_page(page_number) return render(request, 'core/purchase_returns.html', {'returns': returns}) @login_required @@ -1221,7 +1296,10 @@ def user_management(request): messages.error(request, "Access denied.") return redirect('index') - users = User.objects.all().prefetch_related('groups') + users_qs = User.objects.all().prefetch_related('groups').order_by('username') + paginator = Paginator(users_qs, 25) + page_number = request.GET.get('page') + users = paginator.get_page(page_number) groups = Group.objects.all().prefetch_related('permissions') # Filter for relevant permissions (core and auth) permissions = Permission.objects.select_related('content_type').all().order_by('content_type__app_label', 'codename') @@ -1529,3 +1607,255 @@ def profile_view(request): return redirect('profile') return render(request, 'core/profile.html') + +# --- Expenses Views --- + +@login_required +def expenses_view(request): + """ + List and filter expenses + """ + expenses = Expense.objects.all().order_by('-date', '-created_at') + + # Filtering + start_date = request.GET.get('start_date') + end_date = request.GET.get('end_date') + category_id = request.GET.get('category') + + if start_date: + expenses = expenses.filter(date__gte=start_date) + if end_date: + expenses = expenses.filter(date__lte=end_date) + if category_id: + expenses = expenses.filter(category_id=category_id) + + paginator = Paginator(expenses, 25) + page_number = request.GET.get('page') + expenses = paginator.get_page(page_number) + categories = ExpenseCategory.objects.all() + payment_methods = PaymentMethod.objects.filter(is_active=True) + + context = { + 'expenses': expenses, + 'categories': categories, + 'payment_methods': payment_methods, + 'start_date': start_date, + 'end_date': end_date, + 'category_id': category_id, + } + return render(request, 'core/expenses.html', context) + +@login_required +def expense_create_view(request): + """ + Create a new expense + """ + if request.method == 'POST': + category_id = request.POST.get('category') + amount = request.POST.get('amount') + date = request.POST.get('date') or timezone.now().date() + description = request.POST.get('description', '') + payment_method_id = request.POST.get('payment_method') + attachment = request.FILES.get('attachment') + + category = get_object_or_404(ExpenseCategory, id=category_id) + pm = None + if payment_method_id: + pm = get_object_or_404(PaymentMethod, id=payment_method_id) + + Expense.objects.create( + category=category, + amount=amount, + date=date, + description=description, + payment_method=pm, + attachment=attachment, + created_by=request.user + ) + messages.success(request, "Expense recorded successfully!") + + return redirect('expenses') + +@login_required +def expense_delete_view(request, pk): + """ + Delete an expense + """ + expense = get_object_or_404(Expense, pk=pk) + expense.delete() + messages.success(request, "Expense deleted successfully!") + return redirect('expenses') + +@login_required +def expense_categories_view(request): + """ + Manage expense categories + """ + if request.method == 'POST': + name_en = request.POST.get('name_en') + name_ar = request.POST.get('name_ar') + description = request.POST.get('description', '') + + ExpenseCategory.objects.create( + name_en=name_en, + name_ar=name_ar, + description=description + ) + messages.success(request, "Expense category created successfully!") + return redirect('expense_categories') + + paginator = Paginator(expenses, 25) + page_number = request.GET.get('page') + expenses = paginator.get_page(page_number) + categories = ExpenseCategory.objects.all() + return render(request, 'core/expense_categories.html', {'categories': categories}) + +@login_required +def expense_category_delete_view(request, pk): + """ + Delete an expense category + """ + category = get_object_or_404(ExpenseCategory, pk=pk) + category.delete() + messages.success(request, "Expense category deleted successfully!") + return redirect('expense_categories') +@csrf_exempt +@login_required +def update_sale_api(request, pk): + if request.method == 'POST': + try: + sale = get_object_or_404(Sale, pk=pk) + data = json.loads(request.body) + customer_id = data.get('customer_id') + invoice_number = data.get('invoice_number', '') + items = data.get('items', []) + total_amount = data.get('total_amount', 0) + paid_amount = data.get('paid_amount', 0) + discount = data.get('discount', 0) + payment_type = data.get('payment_type', 'cash') + payment_method_id = data.get('payment_method_id') + due_date = data.get('due_date') + notes = data.get('notes', '') + points_to_redeem = data.get('loyalty_points_redeemed', 0) + + settings = SystemSetting.objects.first() + if not settings: + settings = SystemSetting.objects.create() + + # 1. Restore Stock + for item in sale.items.all(): + item.product.stock_quantity += item.quantity + item.product.save() + + # 2. Reverse Loyalty Points for the old customer + if sale.customer and settings.loyalty_enabled: + for lt in sale.loyalty_transactions.all(): + sale.customer.loyalty_points -= decimal.Decimal(str(lt.points)) + lt.delete() + sale.customer.update_tier() + sale.customer.save() + + # 3. Update Sale Metadata + customer = None + if customer_id: + customer = Customer.objects.get(id=customer_id) + + sale.customer = customer + sale.invoice_number = invoice_number + sale.total_amount = total_amount + sale.discount = discount + sale.payment_type = payment_type + sale.due_date = due_date if due_date else None + sale.notes = notes + + # Loyalty discount recalculation + loyalty_discount = 0 + if settings.loyalty_enabled and customer and points_to_redeem > 0: + if float(customer.loyalty_points) >= float(points_to_redeem): + loyalty_discount = float(points_to_redeem) * float(settings.currency_per_point) + + sale.loyalty_points_redeemed = points_to_redeem + sale.loyalty_discount_amount = loyalty_discount + sale.save() + + # 4. Handle Items (Delete old, Create new) + sale.items.all().delete() + for item in items: + product = Product.objects.get(id=item['id']) + SaleItem.objects.create( + sale=sale, + product=product, + quantity=item['quantity'], + unit_price=item['price'], + line_total=item['line_total'] + ) + product.stock_quantity -= int(item['quantity']) + product.save() + + # 5. Handle Payments + sale.paid_amount = paid_amount + sale.balance_due = float(total_amount) - float(paid_amount) + + if float(paid_amount) >= float(total_amount): + sale.status = 'paid' + elif float(paid_amount) > 0: + sale.status = 'partial' + else: + sale.status = 'unpaid' + sale.save() + + pm = None + if payment_method_id: + pm = PaymentMethod.objects.filter(id=payment_method_id).first() + + initial_payment = sale.payments.filter(notes="Initial payment").first() + if initial_payment: + initial_payment.amount = paid_amount + initial_payment.payment_method = pm + initial_payment.payment_method_name = pm.name_en if pm else payment_type.capitalize() + initial_payment.save() + elif float(paid_amount) > 0: + SalePayment.objects.create( + sale=sale, + amount=paid_amount, + payment_method=pm, + payment_method_name=pm.name_en if pm else payment_type.capitalize(), + notes="Initial payment", + created_by=request.user + ) + + # 6. Re-apply Loyalty for the (possibly new) customer + if settings.loyalty_enabled and customer: + points_earned = float(total_amount) * float(settings.points_per_currency) + if customer.loyalty_tier: + points_earned *= float(customer.loyalty_tier.point_multiplier) + + if points_earned > 0: + customer.loyalty_points += decimal.Decimal(str(points_earned)) + LoyaltyTransaction.objects.create( + customer=customer, + sale=sale, + transaction_type='earned', + points=points_earned, + notes=f"Points earned from Updated Sale #{sale.id}" + ) + + if points_to_redeem > 0: + customer.loyalty_points -= decimal.Decimal(str(points_to_redeem)) + LoyaltyTransaction.objects.create( + customer=customer, + sale=sale, + transaction_type='redeemed', + points=-points_to_redeem, + notes=f"Points redeemed for Updated Sale #{sale.id}" + ) + + customer.update_tier() + customer.save() + + return JsonResponse({'success': True}) + except Exception as e: + import traceback + traceback.print_exc() + return JsonResponse({'success': False, 'error': str(e)}, status=400) + return JsonResponse({'success': False, 'error': 'Invalid request'}, status=405)