Clean code

expenses_manager/expenses/views.py

@@ -1,8 +1,7 @@
 import logging
-from operator import truediv
+import calendar
 from datetime import date, datetime
 from django.contrib import messages
-from django.template import context
 from .models import Account, Category, Expense, FuelEntry, Tag, Income, Goal
 from .forms import (
     ExpenseForm,
@@ -14,32 +13,13 @@ from .forms import (
     GoalForm,
 )
 
-# from dateutli.relativedelta import relativedelta
-
 from django.db.models import Sum
-from django.contrib.auth import login
 from django.core.paginator import Paginator
-from django.utils.ipv6 import is_valid_ipv6_address
 from django.db.models.functions import ExtractMonth, ExtractYear, ExtractDay
 
 from django.contrib.auth.decorators import login_required
 from django.shortcuts import get_object_or_404, render, redirect
 
-MONTHS = {
-    1: "ENERO",
-    2: "FEBRERO",
-    3: "MARZO",
-    4: "ABRIL",
-    5: "MAYO",
-    6: "JUNIO",
-    7: "JULIO",
-    8: "AGOSTO",
-    9: "SEPTIEMBRE",
-    10: "OCTUBRE",
-    11: "NOVIEMBRE",
-    12: "DICIEMBRE",
-}
-
 logger = logging.getLogger(__name__)
 
 def _get_int(value):
@@ -337,7 +317,7 @@ def dashboard(request):
     category_count = expenses_filtered.values("category").distinct().count()
     
     # Total by category
-    by_category_qs = by_category = (
+    by_category_qs = (
         expenses_filtered.values("category__name")
         .annotate(total=Sum("amount"))
         .order_by("-total")
@@ -368,8 +348,7 @@ def dashboard(request):
             .annotate(total=Sum("amount"))
         )
         day_totals = {row["day"]: float(row["total"]) for row in by_day_qs}
-        
-        import calendar
+
         num_days = calendar.monthrange(selected_year, selected_month)[1]
         if selected_year == today.year and selected_month == today.month:
             days_passed = today.day