# Reduces db access
* reduces number of queries performed on detail views of intervention, compensation and eco_account * renders deductable_rest of eco account beneath progressbar on eco account index view * clarifies ordering logic of related column
This commit is contained in:
parent
cea40cd878
commit
8fccddf66f
@ -8,6 +8,7 @@ Created on: 18.08.22
|
|||||||
from django.http import HttpRequest
|
from django.http import HttpRequest
|
||||||
from django.template.loader import render_to_string
|
from django.template.loader import render_to_string
|
||||||
from django.urls import reverse
|
from django.urls import reverse
|
||||||
|
from django.utils.formats import number_format
|
||||||
from django.utils.html import format_html
|
from django.utils.html import format_html
|
||||||
from django.utils.translation import gettext_lazy as _
|
from django.utils.translation import gettext_lazy as _
|
||||||
|
|
||||||
@ -111,6 +112,7 @@ class EcoAccountTable(BaseTable, TableRenderMixin, TableOrderMixin):
|
|||||||
except ZeroDivisionError:
|
except ZeroDivisionError:
|
||||||
value_relative = 0
|
value_relative = 0
|
||||||
html = render_to_string("konova/widgets/progressbar.html", {"value": value_relative})
|
html = render_to_string("konova/widgets/progressbar.html", {"value": value_relative})
|
||||||
|
html += f"{number_format(record.deductable_rest, decimal_pos=2)} m²"
|
||||||
return format_html(html)
|
return format_html(html)
|
||||||
|
|
||||||
def render_r(self, value, record: EcoAccount):
|
def render_r(self, value, record: EcoAccount):
|
||||||
|
@ -204,7 +204,11 @@ def detail_view(request: HttpRequest, id: str):
|
|||||||
"""
|
"""
|
||||||
template = "compensation/detail/compensation/view.html"
|
template = "compensation/detail/compensation/view.html"
|
||||||
comp = get_object_or_404(
|
comp = get_object_or_404(
|
||||||
Compensation,
|
Compensation.objects.select_related(
|
||||||
|
"modified",
|
||||||
|
"created",
|
||||||
|
"geometry"
|
||||||
|
),
|
||||||
id=id,
|
id=id,
|
||||||
deleted=None,
|
deleted=None,
|
||||||
intervention__deleted=None,
|
intervention__deleted=None,
|
||||||
|
@ -142,6 +142,8 @@ def detail_view(request: HttpRequest, id: str):
|
|||||||
"geometry",
|
"geometry",
|
||||||
"legal",
|
"legal",
|
||||||
"responsible",
|
"responsible",
|
||||||
|
).prefetch_related(
|
||||||
|
"legal__revocations",
|
||||||
),
|
),
|
||||||
id=id,
|
id=id,
|
||||||
deleted=None
|
deleted=None
|
||||||
@ -158,7 +160,10 @@ def detail_view(request: HttpRequest, id: str):
|
|||||||
last_checked = intervention.get_last_checked_action()
|
last_checked = intervention.get_last_checked_action()
|
||||||
last_checked_tooltip = ""
|
last_checked_tooltip = ""
|
||||||
if last_checked:
|
if last_checked:
|
||||||
last_checked_tooltip = DATA_CHECKED_PREVIOUSLY_TEMPLATE.format(last_checked.get_timestamp_str_formatted(), last_checked.user)
|
last_checked_tooltip = DATA_CHECKED_PREVIOUSLY_TEMPLATE.format(
|
||||||
|
last_checked.get_timestamp_str_formatted(),
|
||||||
|
last_checked.user
|
||||||
|
)
|
||||||
|
|
||||||
has_payment_without_document = intervention.payments.exists() and not intervention.get_documents()[1].exists()
|
has_payment_without_document = intervention.payments.exists() and not intervention.get_documents()[1].exists()
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user