From 52349a39f6b6eb48cf15505437668f0f3fae6ea4 Mon Sep 17 00:00:00 2001 From: bot-backend Date: Wed, 19 Aug 2026 15:11:10 +0000 Subject: [PATCH] =?UTF-8?q?fix(ptica):=20=D1=81=D1=82=D1=80=D0=BE=D0=BA?= =?UTF-8?q?=D0=B0=20=D0=BE=20=D0=BF=D0=BE=D0=B4=D1=82=D0=BE=D0=BF=D0=BB?= =?UTF-8?q?=D0=B5=D0=BD=D0=B8=D0=B8=20=D0=B2=20PDF=20=D0=B8=20DOCX=20?= =?UTF-8?q?=D0=BF=D0=B5=D1=80=D0=B5=D1=81=D1=82=D0=B0=D1=91=D1=82=20=D1=87?= =?UTF-8?q?=D0=B8=D1=82=D0=B0=D1=82=D1=8C=D1=81=D1=8F=20=D0=BA=D0=B0=D0=BA?= =?UTF-8?q?=20=D0=B7=D0=B0=D0=BA=D0=BB=D1=8E=D1=87=D0=B5=D0=BD=D0=B8=D0=B5?= =?UTF-8?q?=20(#2934)=20(#2935)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- backend/app/api/v1/parcels.py | 27 +++++-- .../services/exporters/full_report_docx.py | 9 ++- .../services/exporters/full_report_html.py | 21 ++++- .../exporters/test_2934_flood_row_honesty.py | 80 +++++++++++++++++++ 4 files changed, 128 insertions(+), 9 deletions(-) create mode 100644 backend/tests/services/exporters/test_2934_flood_row_honesty.py diff --git a/backend/app/api/v1/parcels.py b/backend/app/api/v1/parcels.py index ac5a3245..88f8f648 100644 --- a/backend/app/api/v1/parcels.py +++ b/backend/app/api/v1/parcels.py @@ -2688,6 +2688,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": [ { @@ -2697,14 +2701,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: diff --git a/backend/app/services/exporters/full_report_docx.py b/backend/app/services/exporters/full_report_docx.py index 6d6f139e..68171ea2 100644 --- a/backend/app/services/exporters/full_report_docx.py +++ b/backend/app/services/exporters/full_report_docx.py @@ -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"))] diff --git a/backend/app/services/exporters/full_report_html.py b/backend/app/services/exporters/full_report_html.py index 6087e4a4..32ab2368 100644 --- a/backend/app/services/exporters/full_report_html.py +++ b/backend/app/services/exporters/full_report_html.py @@ -402,6 +402,19 @@ def _kv_row(label: str, value: Any) -> str: return f'{html.escape(label)}{_esc(value)}' +# #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'

{_esc(str(hydro_note))}

' water_rows = [ [w.get("name") or w.get("subtype"), _fmt_int_ru(w.get("distance_m"))] diff --git a/backend/tests/services/exporters/test_2934_flood_row_honesty.py b/backend/tests/services/exporters/test_2934_flood_row_honesty.py new file mode 100644 index 00000000..03fda10c --- /dev/null +++ b/backend/tests/services/exporters/test_2934_flood_row_honesty.py @@ -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 держит свою копию метки — форматы разъедутся при следующей правке"