41e3cb906b
All checks were successful
CI / changes (pull_request) Successful in 11s
CI Trade-In / changes (pull_request) Successful in 11s
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / backend-tests (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Has been skipped
CI Trade-In / backend-tests (pull_request) Successful in 2m38s
Замыкает петлю "город объявления -> геокодирование" (issue #2594, шаг 2/3). listings.city (миграция 196) заполняется скрапером из контекста развёртки, но три caller-места геокодера решали город по ТЕКСТУ адреса и игнорировали колонку - голый адрес без города в тексте ("ул. Победы, 30", тагильский) уходил в Екатеринбург. - app/tasks/geocode_missing.py: группировка по (address, city) вместо address, city_hint в geocode(), UPDATE/tried_at-пометка по паре через city IS NOT DISTINCT FROM :city (обычный `=` не поймал бы NULL-город и не даёт нужной симметрии между группами). - app/tasks/backfill_listings_coords_geoportal.py: гейт по колонке city ПЕРЕД матчем против EKB-only ekb_geoportal_buildings, ПЕРЕД текстовым гейтом _names_non_ekb_city (сохранён как fallback для city IS NULL). Это окно идёт раньше geocode_missing_listings, поэтому раньше успевало испортить координаты первым. - app/api/v1/admin.py: per-ID endpoint /geocode-missing читает city из SELECT (listings.city / deals.city) и передаёт как city_hint. geocoder.py не тронут (запрещено ТЗ). Тесты: falsification-прогон (stash impl, тесты остаются) - 7 failed / 36 passed на старом коде, все 7 - новые тесты на новое поведение; после stash pop - 43 passed / 0 failed. Полный pytest tradein-mvp/backend: 2970 passed, 1 failed (pre-existing tests/test_search_api.py::test_search_cache_hit, несвязан), 9 skipped. |
||
|---|---|---|
| .. | ||
| __init__.py | ||
| admin.py | ||
| audit.py | ||
| auth.py | ||
| brand.py | ||
| buildings.py | ||
| geocode.py | ||
| lead.py | ||
| me.py | ||
| search.py | ||
| support.py | ||
| team.py | ||
| trade_in.py | ||