Compare commits

..

4 Commits

Author SHA1 Message Date
mpeltriaux 87e4ee726c Merge pull request '# Extended OEK report' (#568) from 564_Show_deductions_on_EcoAccount_report into master
Reviewed-on: #568
2026-08-03 13:07:34 +02:00
mpeltriaux cbe309f114 # Extended OEK report
* adds available rest surface of eco account to public report
* adds amount of deducted surface per each deduction to public report
2026-08-03 13:06:51 +02:00
mpeltriaux 2aa625a6ac Merge pull request '# Quickfix' (#566) from 565_Unify_geometry_quality_checking into master
Reviewed-on: #566
2026-07-16 15:43:06 +02:00
mpeltriaux d39e758df4 # Quickfix
* fixes bug for better API response on geometry invalidities
2026-07-16 15:42:29 +02:00
5 changed files with 58 additions and 9 deletions
+2
View File
@@ -156,6 +156,8 @@ class AbstractModelAPISerializer:
if isinstance(geojson, dict):
geojson = json.dumps(geojson)
geometry = geos.fromstr(geojson)
if not geometry.valid:
raise ValueError(f"Invalid geometry: {geometry.valid_reason}")
is_4326 = Geometry.is_valid_4326(geometry)
if not is_4326:
raise ValueError("Geometry not in EPSG:4326 (WGS84). Unknown spatial reference system.")
@@ -30,17 +30,44 @@
{% endif %}
</td>
</tr>
<tr>
<th scope="row">{% trans 'Available' %}</th>
<td>
{{available_total|floatformat:2}} / {{obj.deductable_surface|default_if_none:0.00|floatformat:2}} m²
{% with available as value %}
{% include 'konova/widgets/progressbar.html' %}
{% endwith %}
</td>
</tr>
<tr>
<th scope="row">{% trans 'Deductions for' %}</th>
<td class="align-middle">
{% for deduction in deductions %}
<a href="{% url 'intervention:report' deduction.intervention__id %}">
{{deduction.intervention__identifier}} - {{deduction.intervention__title}}
</a>
<br>
{% empty %}
{% trans 'None' %}
{% endfor %}
<table class="table table-hover">
<th scope="col">
{% translate 'Intervention' %}
</th>
<th scope="col">
{% translate 'Amount' %}
</th>
{% for deduction in deductions %}
<tr>
<td>
<a href="{% url 'intervention:report' deduction.intervention__id %}">
{{deduction.intervention__identifier}} - {{deduction.intervention__title}}
</a>
</td>
<td>{{ deduction.surface|floatformat:2 }} m²</td>
{% empty %}
<td>
{% translate 'None' %}
</td>
<td>
/
</td>
</tr>
{% endfor %}
</table>
</td>
</tr>
<tr>
+11 -1
View File
@@ -66,7 +66,15 @@ class EcoAccountPublicReportView(AbstractPublicReportView):
deductions = acc.deductions.all() \
.distinct("intervention") \
.select_related("intervention") \
.values_list("intervention__id", "intervention__identifier", "intervention__title", named=True)
.values_list("intervention__id",
"intervention__identifier",
"intervention__title",
"surface",
named=True)
# Calculate rest of available surface for deductions
available_total = acc.deductable_rest
available_relative = acc.get_deductable_rest_relative()
context = {
"obj": acc,
@@ -86,6 +94,8 @@ class EcoAccountPublicReportView(AbstractPublicReportView):
"actions": actions,
"deductions": deductions,
"tables_scrollable": False,
"available": available_relative,
"available_total": available_total,
TAB_TITLE_IDENTIFIER: tab_title,
}
context = BaseContext(request, context).context
+5
View File
@@ -0,0 +1,5 @@
"""
Author: Michel Peltriaux
Created on: 16.07.26
"""
+5
View File
@@ -0,0 +1,5 @@
"""
Author: Michel Peltriaux
Created on: 16.07.26
"""