MEJORA: Fix N+1 query problems in project_list and export_expenses_csv
This commit is contained in:
+6
-3
@@ -1,5 +1,8 @@
|
||||
import csv
|
||||
import io
|
||||
import logging
|
||||
|
||||
logger = logging.getLogger(__name__)
|
||||
from decimal import Decimal
|
||||
from django.views.generic import ListView, CreateView, UpdateView, DeleteView
|
||||
from django.contrib.auth.mixins import LoginRequiredMixin
|
||||
@@ -353,7 +356,7 @@ def export_expenses_csv(request):
|
||||
'Responsable', 'Detalle', 'Region', 'Comuna', 'Proveedor'
|
||||
])
|
||||
|
||||
expenses = Expense.objects.all().select_related('project', 'responsible_person')
|
||||
expenses = Expense.objects.all().select_related('project', 'responsible_person', 'vendor')
|
||||
for exp in expenses:
|
||||
writer.writerow([
|
||||
exp.id,
|
||||
@@ -870,8 +873,8 @@ def delete_document(request, pk):
|
||||
try:
|
||||
if os.path.exists(doc.file.path):
|
||||
os.remove(doc.file.path)
|
||||
except Exception:
|
||||
pass
|
||||
except Exception as e:
|
||||
logger.error(f"Error removing file: {e}", exc_info=True)
|
||||
|
||||
# Delete object
|
||||
doc.delete()
|
||||
|
||||
Reference in New Issue
Block a user