Merge pull request 'Экспорт совета: убрать статус и переименовать колонку' (#61) from fix/council-xlsx-columns into feature/council-journal-counts

Reviewed-on: #61
Reviewed-by: agent <agent@vakanaut.ru>

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.14.3
+
+- Из XLSX диссертационного совета удалён столбец «Статус». Колонка «В БД» переименована в «Страница ВШЭ» на листах «Сотрудники» и «Публикации»; расчёт значений сохранён.
+
 ## 0.14.2
 
 - Исправлено сопоставление К1 для переименованного журнала: действующее и явно указанное в исходном PDF прежнее название распознаются как один журнал. Административное примечание хранится отдельно от названия; конфликтующие алиасы не присваиваются автоматически.

README.md

@@ -155,7 +155,7 @@ node --check app/static/admin.js
 
 Тесты с данными используют временную SQLite; отдельные API smoke-тесты запускают приложение с его текущей конфигурацией. Браузерная проверка: `pip install playwright`, `python -m playwright install chromium`, затем `python tests/browser_admin.py`. Она проверяет реальный рендеринг страниц, клавиатуру, диалог, фильтры и восстановление прогресса после ошибки. Скриншоты сохраняются во временную папку; путь выводится в конце.
 
-Версия сервиса: `0.14.2`. Админка всегда показывает версии backend и frontend в footer.
+Версия сервиса: `0.14.3`. Админка всегда показывает версии backend и frontend в footer.
 
 ## Разовый импорт публикаций
 

app/services/council_export.py

@@ -22,15 +22,15 @@ def council_xlsx(result: dict) -> bytes:
     employees.title = "Сотрудники"
     publications = workbook.create_sheet("Публикации")
     parameters = workbook.create_sheet("Параметры")
-    _append(employees, ["№", "ФИО", "Учёная степень", "Q1", "Q2", "К1", "ВСЕГО", "В БД", "Все публикации",
-                        "Статус", "Профиль ВШЭ", "Примечание"])
+    _append(employees, ["№", "ФИО", "Учёная степень", "Q1", "Q2", "К1", "ВСЕГО", "Страница ВШЭ", "Все публикации",
+                        "Профиль ВШЭ", "Примечание"])
     _append(publications, ["№ сотрудника", "ФИО", "Название публикации", "Год", "Тип материала", "Источники",
-                           "Критерии", "Q1", "Q2", "К1", "В ВСЕГО", "В БД", "DOI", "Ссылка на сайте", "Строки CSV"])
+                           "Критерии", "Q1", "Q2", "К1", "В ВСЕГО", "Страница ВШЭ", "DOI", "Ссылка на сайте", "Строки CSV"])
     for number, candidate in enumerate(result["candidates"], 1):
         note = "Совпадение ФИО: публикации CSV не присвоены автоматически" if candidate["ambiguous"] else ""
         _append(employees, [number, candidate["name"], candidate["degree"], candidate["q1_count"], candidate["q2_count"],
                             candidate["k1_count"], candidate["total_count"],
-                            candidate["db_count"], candidate["count"], "Уволен" if candidate["dismissed"] else "Не отмечен как уволенный",
+                            candidate["db_count"], candidate["count"],
                             candidate["profile_url"], note])
         for article in candidate["articles"]:
             _append(publications, [number, candidate["name"], article["title"], article["year"], article["type_label"],
@@ -41,7 +41,7 @@ def council_xlsx(result: dict) -> bytes:
     for values in [
         ["Начало периода", result["start"]], ["Конец периода", result["end"]], ["Поиск по ФИО", result["q"]],
         ["Сортировать по", {"full_name": "ФИО", "q1_count": "Q1", "q2_count": "Q2", "k1_count": "К1",
-                            "total_count": "ВСЕГО", "db_count": "В БД", "count": "Все публикации"}[result["sort"]]],
+                            "total_count": "ВСЕГО", "db_count": "Страница ВШЭ", "count": "Все публикации"}[result["sort"]]],
         ["Направление", "По возрастанию" if result["direction"] == "asc" else "По убыванию"],
         ["Сотрудников в выгрузке", result["matched"]], ["Всего сотрудников в списке", result["total"]],
         ["Отбор по степени", "Доктор наук в профиле; достаточно хотя бы одной публикации из CSV или БД"],
@@ -51,11 +51,12 @@ def council_xlsx(result: dict) -> bytes:
          "в справочнике из ВАК категории.pdf; историческая категория не подтверждается. "
          "Без названия журнала К1 не подтверждён."],
         ["ВСЕГО", "Уникальные публикации, входящие в Q1 ИЛИ Q2 ИЛИ К1; пересечения учитываются один раз."],
-        ["В БД / Все публикации", "Все годы и категории; общий перечень объединяет CSV и БД без дублей."],
+        ["Страница ВШЭ / Все публикации", "Все годы и категории; общий перечень объединяет CSV и БД без дублей. "
+         "Страница ВШЭ учитывает сохранённые публикации, включая импортированные из CSV; источник указан на листе Публикации."],
     ]:
         _append(parameters, values)
-    for sheet, widths in [(employees, [7, 34, 40, 10, 10, 10, 12, 12, 22, 30, 45, 55]),
-                          (publications, [16, 34, 65, 12, 40, 45, 65, 10, 10, 10, 12, 12, 45, 45, 22]),
+    for sheet, widths in [(employees, [7, 34, 40, 10, 10, 10, 12, 20, 22, 45, 55]),
+                          (publications, [16, 34, 65, 12, 40, 45, 65, 10, 10, 10, 12, 20, 45, 45, 22]),
                           (parameters, [32, 95])]:
         sheet.freeze_panes = "A2"
         sheet.auto_filter.ref = sheet.dimensions

app/version.py

@@ -1,3 +1,3 @@
-APP_VERSION = "0.14.2"
-FRONTEND_VERSION = "0.14.2"
-BACKEND_VERSION = "0.14.2"
+APP_VERSION = "0.14.3"
+FRONTEND_VERSION = "0.14.3"
+BACKEND_VERSION = "0.14.3"

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "miem-workers"
-version = "0.14.2"
+version = "0.14.3"
 description = "MIEM employees parser, admin API, and web admin"
 requires-python = ">=3.11"
 dependencies = [

tests/test_api.py

@@ -18,7 +18,7 @@ def test_health_returns_versions():
     response = TestClient(app).get("/api/health")
 
     assert response.status_code == 200
-    assert response.json()["backend_version"] == "0.14.2"
+    assert response.json()["backend_version"] == "0.14.3"
 
 
 def test_api_employees_and_stats_require_admin_session():

tests/test_council.py

@@ -211,7 +211,7 @@ def test_public_page_data_minimization_and_safe_links(db_session, export):
         assert "&lt;script&gt;" in response.text and "<script>" not in response.text
         assert "javascript:" not in response.text and "private@example.org" not in response.text
         assert "/admin/employees/" not in response.text and "/admin/logout" not in response.text
-        assert "Backend 0.14.2 · Frontend 0.14.2" in response.text
+        assert "Backend 0.14.3 · Frontend 0.14.3" in response.text
         assert "Вход в админку" in response.text
         assert client.get("/admin/directory", follow_redirects=False).status_code in (302, 303, 401, 403)
         assert "Сотрудники не найдены" in client.get("/admin/council?q=missing").text
@@ -254,6 +254,14 @@ def test_xlsx_exports_all_filtered_workers_in_order_and_deduplicated_evidence(db
         book = load_workbook(BytesIO(response.content))
         assert book.sheetnames == ["Сотрудники", "Публикации", "Параметры"]
         people, works = book["Сотрудники"], book["Публикации"]
+        assert [cell.value for cell in people[1]] == [
+            "№", "ФИО", "Учёная степень", "Q1", "Q2", "К1", "ВСЕГО", "Страница ВШЭ", "Все публикации",
+            "Профиль ВШЭ", "Примечание"
+        ]
+        assert people["J2"].value == "https://www.hse.ru/staff/test"
+        assert people["K2"].value is None
+        assert people.max_column == 11
+        assert works["L1"].value == "Страница ВШЭ"
         assert people.max_row == 29 and works.max_row == 29
         assert people["B2"].value == "Петров 27" and people["B29"].value == "Петров 00"
         assert [people.cell(2, n).value for n in [4, 5, 6, 7, 8, 9]] == [1, 1, 0, 1, 1, 1]