fix(ptica): строка о подтоплении в PDF и DOCX перестаёт читаться как заключение (#2934)
All checks were successful
CI Trade-In / changes (pull_request) Successful in 12s
CI Trade-In / backend-tests (pull_request) Has been skipped
CI / changes (pull_request) Successful in 13s
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Successful in 2m3s
CI / backend-tests (pull_request) Successful in 17m1s
All checks were successful
CI Trade-In / changes (pull_request) Successful in 12s
CI Trade-In / backend-tests (pull_request) Has been skipped
CI / changes (pull_request) Successful in 13s
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Successful in 2m3s
CI / backend-tests (pull_request) Successful in 17m1s
В §2 «Окружение» экспортируемые документы печатали:
Риск подтопления | нет
Значение берётся из hydrology.flood_risk_flag — это близость реки или канала
ближе 200 м по OSM. Проверкой зон затопления оно не является. Замер на проде
19.08: таблица cad_risk_zones пуста (0 строк), слои risk_* НСПД дали 0 объектов
на 669 дампов. То есть ни один источник зон затопления в эту строку не входит, а
_fmt(False) превращает флаг в слово «нет» — читатель получал вывод, которого не
делали.
Опаснее прочих мест из #2934 по трём причинам: это экспортируемый документ, он
уходит наружу и живёт после сессии; печатается на основном пути сразу в двух
форматах; и честный текст УЖЕ был написан — hydrology["note"] — но терялся ровно
на границе экспортёра. Фронт его показывает (HydrologyBlock.tsx), документы нет.
Три правки:
1. Метка называет измеренное: «Река или канал ближе 200 м (OSM)».
2. Оговорка доезжает до HTML/PDF и DOCX.
3. Сама оговорка стала верной для ОБОИХ случаев. Раньше она была написана в
расчёте только на «пойма есть», и при flood_risk_flag=false фронт печатал
«повышенный риск подтопления» рядом со значением «нет» — текст противоречил
соседнему полю.
Метка вынесена общей константой: обе таблицы собираются одинаковыми списками пар,
DOCX уже импортирует хелперы из HTML, и правка в одном файле разошлась бы с
другим незаметно. На это есть отдельный тест.
Тесты: 3 красных на origin/main ПО ПОВЕДЕНИЮ (ассерт печатает саму строку из
документа), 1 контроль зелёный с обеих сторон. Первая версия импортировала новую
константу в шапке модуля — на origin/main это роняло сборку файла, и красное
означало бы «модуль не собрался», а не «документ утверждает не то»; импорт
перенесён внутрь единственного теста, которому он нужен.
pytest tests/services/exporters: 215 passed, rc=0 (перепрогнано после правок хука)
pytest tests/api/v1: 348 passed, 1 skipped, rc=0
Refs #2934
This commit is contained in:
parent
a9e9707018
commit
efd8d86d81
4 changed files with 128 additions and 9 deletions
|
|
@ -2670,6 +2670,10 @@ def analyze_parcel(
|
|||
.mappings()
|
||||
.all()
|
||||
)
|
||||
_flood_proximity = any(
|
||||
float(r["distance_m"]) < 200 and r["road_class"] in ("river", "canal")
|
||||
for r in hydro_rows
|
||||
)
|
||||
hydrology = {
|
||||
"nearest": [
|
||||
{
|
||||
|
|
@ -2679,14 +2683,23 @@ def analyze_parcel(
|
|||
}
|
||||
for r in hydro_rows[:5]
|
||||
],
|
||||
"flood_risk_flag": any(
|
||||
float(r["distance_m"]) < 200 and r["road_class"] in ("river", "canal")
|
||||
for r in hydro_rows
|
||||
),
|
||||
"flood_risk_flag": _flood_proximity,
|
||||
# #2934: оговорка была написана в расчёте ТОЛЬКО на случай «пойма есть» —
|
||||
# при flood_risk_flag=false фронт всё равно печатал «Пойма реки (<200м) —
|
||||
# повышенный риск подтопления», то есть текст противоречил значению рядом.
|
||||
# Вторая половина («официальные зоны — в Росреестре») верна всегда и
|
||||
# существенна: этот флаг — близость водного объекта по OSM, а НЕ проверка
|
||||
# зон затопления. Ни cad_risk_zones (пуста), ни слои risk_* НСПД в него
|
||||
# не входят.
|
||||
"note": (
|
||||
"Пойма реки (<200м) — повышенный риск подтопления. Точные данные о "
|
||||
"зонах затопления — в Росреестре (ЗОУИТ типа 33: 'Зона затопления, "
|
||||
"подтопления') через ФГИС ТП."
|
||||
(
|
||||
"Пойма реки или канала ближе 200 м — повышенный риск подтопления. "
|
||||
if _flood_proximity
|
||||
else "Рек и каналов ближе 200 м не найдено. "
|
||||
)
|
||||
+ "Это близость водного объекта по OSM, а НЕ проверка зон затопления: "
|
||||
"официальные зоны — ЗОУИТ типа 33 «Зона затопления, подтопления» "
|
||||
"(Росреестр, ФГИС ТП)."
|
||||
),
|
||||
}
|
||||
except Exception as e:
|
||||
|
|
|
|||
|
|
@ -41,6 +41,7 @@ from typing import TYPE_CHECKING, Any
|
|||
# `_fc_*`-хелперы (нормализация forecast-словаря) — реэкспорт из report_pdf через
|
||||
# full_report_html, тянем оттуда же (одна точка импорта).
|
||||
from app.services.exporters.full_report_html import (
|
||||
FLOOD_PROXIMITY_LABEL,
|
||||
_as_dict,
|
||||
_as_list,
|
||||
_development_type_ru,
|
||||
|
|
@ -404,10 +405,16 @@ def _build_geotech_hydro(doc: _DocxDocument, result: dict[str, Any]) -> None:
|
|||
("Балльность", geotech.get("seismic_intensity_balls")),
|
||||
("Многолетняя мерзлота", geotech.get("permafrost")),
|
||||
("Промобъектов в 500 м", geotech.get("industrial_within_500m")),
|
||||
("Риск подтопления", hydro.get("flood_risk_flag")),
|
||||
(FLOOD_PROXIMITY_LABEL, hydro.get("flood_risk_flag")),
|
||||
]
|
||||
pairs = [(k, v) for k, v in pairs if v not in (None, "")]
|
||||
_add_kv_table(doc, pairs)
|
||||
# #2934: та же оговорка, что в HTML-двойнике. Метка — общая константа оттуда же:
|
||||
# обе таблицы собираются одинаковыми списками пар, и правка в одном файле молча
|
||||
# разошлась бы с другим.
|
||||
_hydro_note = hydro.get("note")
|
||||
if _hydro_note:
|
||||
doc.add_paragraph(str(_hydro_note))
|
||||
|
||||
water_rows = [
|
||||
[w.get("name") or w.get("subtype"), _fmt_int_ru(w.get("distance_m"))]
|
||||
|
|
|
|||
|
|
@ -402,6 +402,19 @@ def _kv_row(label: str, value: Any) -> str:
|
|||
return f'<tr><td class="k">{html.escape(label)}</td><td class="v">{_esc(value)}</td></tr>'
|
||||
|
||||
|
||||
# #2934: метка строки о подтоплении. Прежняя — «Риск подтопления» — утверждала
|
||||
# результат проверки зон затопления, которой не было: значение берётся из
|
||||
# hydrology.flood_risk_flag, а это близость реки или канала ближе 200 м по OSM.
|
||||
# Ни cad_risk_zones (0 строк на проде), ни 11 слоёв risk_* НСПД (0 объектов на 669
|
||||
# дампов) в него не входят. `_fmt(False)` печатал «нет», и читатель экспортированного
|
||||
# документа получал «Риск подтопления — нет» как заключение.
|
||||
#
|
||||
# Константа общая с DOCX (`full_report_docx` импортирует хелперы отсюда): строка
|
||||
# собирается в двух файлах одинаковыми списками пар, и разъезд формулировок был бы
|
||||
# незаметен до чьей-нибудь жалобы.
|
||||
FLOOD_PROXIMITY_LABEL = "Река или канал ближе 200 м (OSM)"
|
||||
|
||||
|
||||
def _kv_table(pairs: list[tuple[str, Any]]) -> str:
|
||||
"""Таблица «метка → значение» из списка пар. Пустой список → «нет данных». PURE."""
|
||||
if not pairs:
|
||||
|
|
@ -643,10 +656,16 @@ def _build_geotech_hydro(result: dict[str, Any]) -> str:
|
|||
("Балльность", geotech.get("seismic_intensity_balls")),
|
||||
("Многолетняя мерзлота", geotech.get("permafrost")),
|
||||
("Промобъектов в 500 м", geotech.get("industrial_within_500m")),
|
||||
("Риск подтопления", hydro.get("flood_risk_flag")),
|
||||
(FLOOD_PROXIMITY_LABEL, hydro.get("flood_risk_flag")),
|
||||
]
|
||||
pairs = [(k, v) for k, v in pairs if v not in (None, "")]
|
||||
geotech_table = _kv_table(pairs)
|
||||
# Оговорка payload'а существовала и терялась ровно здесь, на границе экспортёра:
|
||||
# фронт её печатает (HydrologyBlock.tsx), а PDF и DOCX — нет. Именно она говорит,
|
||||
# что официальные зоны затопления живут в ЗОУИТ типа 33, а не в этой строке.
|
||||
hydro_note = hydro.get("note")
|
||||
if hydro_note:
|
||||
geotech_table += f'<p class="alt-meta">{_esc(str(hydro_note))}</p>'
|
||||
|
||||
water_rows = [
|
||||
[w.get("name") or w.get("subtype"), _fmt_int_ru(w.get("distance_m"))]
|
||||
|
|
|
|||
|
|
@ -0,0 +1,80 @@
|
|||
"""#2934: строка о подтоплении в экспортируемых документах не должна читаться
|
||||
как заключение о зонах затопления.
|
||||
|
||||
Что было: в §2 «Окружение» → «Геотехника и гидрология» PDF и DOCX печатали
|
||||
|
||||
Риск подтопления | нет
|
||||
|
||||
Значение берётся из ``hydrology.flood_risk_flag`` — это близость реки или канала
|
||||
ближе 200 м по OSM. Проверкой зон затопления оно не является. Замер на проде
|
||||
19.08.2026:
|
||||
|
||||
cad_risk_zones 0 строк
|
||||
слои risk_* НСПД 0 объектов на 669 дампов
|
||||
|
||||
То есть ни один источник зон затопления в эту строку не входит, а ``_fmt(False)``
|
||||
превращает флаг в слово «нет» — читатель экспортированного документа получает
|
||||
«Риск подтопления — нет» как вывод.
|
||||
|
||||
Оговорка при этом СУЩЕСТВОВАЛА: ``hydrology["note"]`` печатается фронтом
|
||||
(HydrologyBlock.tsx), но терялась на границе экспортёра.
|
||||
"""
|
||||
|
||||
from __future__ import annotations
|
||||
|
||||
from typing import Any
|
||||
|
||||
from app.services.exporters.full_report_html import build_full_report_html_part_a
|
||||
|
||||
# Константу НЕ импортируем на уровне модуля: на origin/main её ещё нет, и импорт
|
||||
# уронил бы СБОРКУ всего файла — тогда красное означало бы «модуль не собрался», а
|
||||
# не «документ утверждает не то». Проверки ниже смотрят на текст, который реально
|
||||
# попадает в документ.
|
||||
_NOTE = "Рек и каналов ближе 200 м не найдено. Это близость водного объекта по OSM"
|
||||
_OLD_LABEL = "Риск подтопления"
|
||||
|
||||
|
||||
def _result(flag: bool, note: str | None = _NOTE) -> dict[str, Any]:
|
||||
hydro: dict[str, Any] = {"flood_risk_flag": flag, "nearest": []}
|
||||
if note is not None:
|
||||
hydro["note"] = note
|
||||
return {
|
||||
"geotech_risk": {"seismic_label": "минимальная сейсмика", "permafrost": False},
|
||||
"hydrology": hydro,
|
||||
}
|
||||
|
||||
|
||||
def test_label_no_longer_claims_a_flood_verdict() -> None:
|
||||
"""Метка называет измеренное, а не вывод, которого не делали."""
|
||||
html = build_full_report_html_part_a(_result(flag=False), cad="00:00:0000000:0000")
|
||||
|
||||
assert (
|
||||
_OLD_LABEL not in html
|
||||
), "метка утверждает результат проверки зон затопления, которой не было"
|
||||
assert "ближе 200 м" in html, "метка должна называть измеренное — близость водотока"
|
||||
|
||||
|
||||
def test_note_reaches_the_exported_document() -> None:
|
||||
"""Оговорка была написана и молча выбрасывалась на границе экспортёра."""
|
||||
html = build_full_report_html_part_a(_result(flag=False), cad="X")
|
||||
assert "близость водного объекта по OSM" in html
|
||||
|
||||
|
||||
def test_missing_note_does_not_break_the_section() -> None:
|
||||
"""Контроль: payload без note по-прежнему рендерится (зелёный по обе стороны)."""
|
||||
html = build_full_report_html_part_a(_result(flag=True, note=None), cad="X")
|
||||
assert "Геотехника" in html or "минимальная сейсмика" in html
|
||||
|
||||
|
||||
def test_docx_twin_uses_the_same_label() -> None:
|
||||
"""Метка общая: две таблицы собираются одинаковыми списками пар, и правка в
|
||||
одном файле разошлась бы с другим незаметно.
|
||||
|
||||
Импорт внутри теста, а не в шапке — иначе на origin/main не собрался бы весь
|
||||
модуль и остальные проверки не исполнились бы вовсе.
|
||||
"""
|
||||
from app.services.exporters import full_report_docx, full_report_html
|
||||
|
||||
assert (
|
||||
full_report_docx.FLOOD_PROXIMITY_LABEL is full_report_html.FLOOD_PROXIMITY_LABEL
|
||||
), "DOCX держит свою копию метки — форматы разъедутся при следующей правке"
|
||||
Loading…
Add table
Reference in a new issue