From 556fdf3b5ea5179ed67d9016f9d9b5cf86914517 Mon Sep 17 00:00:00 2001 From: Vladislav Kopnin Date: Fri, 2 Oct 2026 15:41:16 +0300 Subject: [PATCH] Redesign B3 bordeaux + finish lab-page rework - Lab page is now read-only with a settings screen for levels/variants; variant issuance moved to a large modal on the group page (assign-fragment). - Header: bordeaux logo and 2px accent underline, bordeaux login/logout/theme buttons; smaller radii across the UI; filled secondary/danger buttons. - Includes prior agent work (login backoff, no-cache pages, API timeouts, content tweaks) and fixes (variant usage Map.values, import order). - Bump static asset version to v=9. --- design/style-a-university.html | 267 ++++++++++++ design/style-b-minimal.html | 254 +++++++++++ design/style-b2-minimal.html | 400 ++++++++++++++++++ design/style-b3-bordeaux.html | 348 +++++++++++++++ design/style-c-institutional.html | 264 ++++++++++++ libs/common-http/build.gradle.kts | 2 + .../kotlin/ru/ysu/lms/common/LmsException.kt | 5 + .../ru/ysu/lms/common/LmsExceptionHandler.kt | 35 ++ .../lms/academic/content/ContentImporter.kt | 44 +- .../controller/ContentOrderController.kt | 5 +- .../controller/ContentPublicController.kt | 21 +- .../ysu/lms/academic/lab/LabImportParser.kt | 44 +- .../security/JwtAuthenticationFilter.kt | 18 +- .../ru/ysu/lms/identity/auth/AuthService.kt | 13 +- .../ru/ysu/lms/identity/auth/LoginBackoff.kt | 74 ++++ .../ysu/lms/identity/domain/LoginAttempt.kt | 20 + .../identity/repo/LoginAttemptRepository.kt | 6 + .../db/migration/V3__login_attempts.sql | 7 + .../identity/AuthServiceLoginBackoffTest.kt | 142 +++++++ .../ru/ysu/lms/progress/AcademicClient.kt | 24 +- .../ru/ysu/lms/progress/DefenseService.kt | 3 +- .../main/kotlin/ru/ysu/lms/progress/Dtos.kt | 15 + .../lms/progress/GroupLabDataController.kt | 24 ++ .../ysu/lms/progress/GroupLabDataService.kt | 46 ++ .../ru/ysu/lms/progress/SubmissionService.kt | 20 +- .../ru/ysu/lms/progress/SubmissionStorage.kt | 22 +- .../ProgressDistributionIntegrationTest.kt | 42 ++ .../ysu/lms/progress/SubmissionServiceTest.kt | 14 +- .../ysu/lms/web/admin/AdminGroupController.kt | 65 ++- .../ysu/lms/web/admin/AdminLabController.kt | 98 +---- .../ru/ysu/lms/web/admin/AdminLabView.kt | 26 -- .../lms/web/admin/AdminStudentController.kt | 15 +- .../ysu/lms/web/admin/AdminUsersController.kt | 24 +- .../kotlin/ru/ysu/lms/web/api/ApiClient.kt | 17 +- .../ru/ysu/lms/web/api/ApiClientGroup.kt | 25 ++ .../main/kotlin/ru/ysu/lms/web/api/ApiDtos.kt | 16 + .../kotlin/ru/ysu/lms/web/config/WebConfig.kt | 18 + .../ru/ysu/lms/web/learn/ContentAssetProxy.kt | 14 +- web/src/main/resources/application.yml | 3 + web/src/main/resources/static/css/app.css | 299 ++++++++----- web/src/main/resources/static/favicon.svg | 7 +- .../main/resources/static/js/admin-group.js | 68 ++- web/src/main/resources/static/js/admin-lab.js | 36 -- .../main/resources/static/js/content-sort.js | 3 + web/src/main/resources/static/js/htmx.min.js | 1 - web/src/main/resources/static/js/mdeditor.js | 3 +- .../main/resources/static/js/submit-guard.js | 32 ++ web/src/main/resources/static/js/tablekit.js | 17 +- .../templates/admin/assign-fragment.html | 76 ++++ .../resources/templates/admin/dashboard.html | 4 +- .../main/resources/templates/admin/group.html | 148 ++++--- .../resources/templates/admin/groups.html | 67 ++- .../resources/templates/admin/lab-edit.html | 14 +- .../main/resources/templates/admin/lab.html | 163 +------ .../main/resources/templates/admin/labs.html | 33 +- .../main/resources/templates/admin/users.html | 8 +- .../resources/templates/change_password.html | 6 +- .../resources/templates/fragments/base.html | 41 +- .../resources/templates/learn/article.html | 18 +- .../resources/templates/learn/chapter.html | 33 +- .../main/resources/templates/learn/index.html | 42 +- .../resources/templates/learn/section.html | 27 +- web/src/main/resources/templates/login.html | 4 +- web/src/main/resources/templates/setup.html | 6 +- .../templates/student/dashboard.html | 2 +- .../main/resources/templates/student/lab.html | 12 +- .../resources/templates/student/labs.html | 2 +- 67 files changed, 3019 insertions(+), 653 deletions(-) create mode 100644 design/style-a-university.html create mode 100644 design/style-b-minimal.html create mode 100644 design/style-b2-minimal.html create mode 100644 design/style-b3-bordeaux.html create mode 100644 design/style-c-institutional.html create mode 100644 services/identity-service/src/main/kotlin/ru/ysu/lms/identity/auth/LoginBackoff.kt create mode 100644 services/identity-service/src/main/kotlin/ru/ysu/lms/identity/domain/LoginAttempt.kt create mode 100644 services/identity-service/src/main/kotlin/ru/ysu/lms/identity/repo/LoginAttemptRepository.kt create mode 100644 services/identity-service/src/main/resources/db/migration/V3__login_attempts.sql create mode 100644 services/identity-service/src/test/kotlin/ru/ysu/lms/identity/AuthServiceLoginBackoffTest.kt create mode 100644 services/progress-service/src/main/kotlin/ru/ysu/lms/progress/GroupLabDataController.kt create mode 100644 services/progress-service/src/main/kotlin/ru/ysu/lms/progress/GroupLabDataService.kt create mode 100644 web/src/main/kotlin/ru/ysu/lms/web/api/ApiClientGroup.kt delete mode 100644 web/src/main/resources/static/js/admin-lab.js delete mode 100644 web/src/main/resources/static/js/htmx.min.js create mode 100644 web/src/main/resources/static/js/submit-guard.js create mode 100644 web/src/main/resources/templates/admin/assign-fragment.html diff --git a/design/style-a-university.html b/design/style-a-university.html new file mode 100644 index 0000000..2a0f134 --- /dev/null +++ b/design/style-a-university.html @@ -0,0 +1,267 @@ + + + + + +LMS — демо стиля А: университетская классика + + + + + + +
Демо-шаблон «А. Университетская классика» · данные условные · тёмная тема будет в реализации
+ +
+
+ + +
+ Копнин В. + +
+
+
+ +
+
+
+

Ведомость группы

+

М202-1 · Первый курс

+

24 студента · 3 опубликованные лабораторные · длительность защиты 30 мин

+
+
+ + +
+
+ +
+ + +
+ +
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
СтудентЛР 1Анализ данныхЛР 2СортировкиЛР 3Деревья поискаПарольДействия
Иванова Анна
2025001
 +
В-74.5◉
+
сдача ↓ 2защита: готов
+ 		+
В-23.0◉
+
сдача ↓ 1защита: код 552314
+ 		+
—
+
не выдан
+ 		временный
Петров Павел
2025002
 +
В-15.0◉
+
сдача ↓ 3защита: готов
+ 		+
В-4—◉
+
сдача ↓ 1защита: идёт · 12:34
+ 		+
В-3—○
+
не сдано
+ 		изменён
Сидоров Алексей
2025017
 +
В-5—◉
+
не сдано
+ 		+
—
+
не выдан
+ 		+
—
+
не выдан
+ 		временный
Кузнецова Мария
2025021
 +
В-34.0○
+
сдача ↓ 1защита: готов
+ 		+
В-6—○
+
сдача ↓ 1защита: идёт · 03:07
+ 		+
—
+
не выдан
+ 		изменён
+
+ +
+

Элементы интерфейса

+
+ + + + published + draft +
+
+ + +
+
Оценки опубликованы всей группе — студенты видят баллы и комментарии.
+
+
+ + + + diff --git a/design/style-b-minimal.html b/design/style-b-minimal.html new file mode 100644 index 0000000..b68dd6f --- /dev/null +++ b/design/style-b-minimal.html @@ -0,0 +1,254 @@ + + + + + +LMS — демо стиля Б: строгий минимализм + + + + + +
Демо-шаблон «Б. Строгий минимализм» · данные условные · тёмная тема будет в реализации
+ +
+
+ + +
+ Копнин В. + +
+
+
+ +
+
+
+

Ведомость группы

+

М202-1 · Первый курс

+

24 студента · 3 опубликованные лабораторные · длительность защиты 30 мин

+
+
+ + +
+
+ +
+ + +
+ +
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
СтудентЛР 1Анализ данныхЛР 2СортировкиЛР 3Деревья поискаПарольДействия
Иванова Анна
2025001
 +
В-74.5◉
+
сдача ↓ 2защита: готов
+ 		+
В-23.0◉
+
сдача ↓ 1защита: код 552314
+ 		+
—
+
не выдан
+ 		временный
Петров Павел
2025002
 +
В-15.0◉
+
сдача ↓ 3защита: готов
+ 		+
В-4—◉
+
сдача ↓ 1защита: идёт · 12:34
+ 		+
В-3—○
+
не сдано
+ 		изменён
Сидоров Алексей
2025017
 +
В-5—◉
+
не сдано
+ 		+
—
+
не выдан
+ 		+
—
+
не выдан
+ 		временный
Кузнецова Мария
2025021
 +
В-34.0○
+
сдача ↓ 1защита: готов
+ 		+
В-6—○
+
сдача ↓ 1защита: идёт · 03:07
+ 		+
—
+
не выдан
+ 		изменён
+
+ +
+

Элементы интерфейса

+
+ + + + published + draft +
+
+ + +
+
Оценки опубликованы всей группе — студенты видят баллы и комментарии.
+
+
+ + + + diff --git a/design/style-b2-minimal.html b/design/style-b2-minimal.html new file mode 100644 index 0000000..bfde7d9 --- /dev/null +++ b/design/style-b2-minimal.html @@ -0,0 +1,400 @@ + + + + + +LMS — демо стиля Б2: минимализм с живой палитрой + + + + + +
Демо-шаблон «Б2» · развитый минимализм: живая палитра, адаптивные таблицы, бейджи-индикаторы · сузьте окно, чтобы увидеть все три режима таблицы
+ +
+ Акцентный цвет: + + + + + — красный остаётся только на действиях «Удалить/Отчислить» +
+ +
+
+ + +
+ Копнин В. + +
+
+
+ +
+
+
+

Ведомость группы

+

М202-1 · Первый курс

+

24 студента · 3 опубликованные лабораторные · длительность защиты 30 мин

+
+
+ + +
+
+ +
+ + +
+ +
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
СтудентЛР 1Анализ данныхЛР 2СортировкиЛР 3Деревья поискаПарольДействия
Иванова Анна
2025001
 +
В-74.5◉
+
сдача ↓ 2защита: готова
+ 		+
В-23.0◉
+
сдача ↓ 1защита: код 552314
+ 		+
—
+
вариант не выдан
+ 		временный
Петров Павел
2025002
 +
В-15.0◉
+
сдача ↓ 3защита: готова
+ 		+
В-4—◉
+
сдача ↓ 1защита: идёт · 12:34
+ 		+
В-3—○
+
не сдано
+ 		изменён
Сидоров Алексей
2025017
 +
В-5—◉
+
не сдано
+ 		+
—
+
вариант не выдан
+ 		+
—
+
вариант не выдан
+ 		временный
Кузнецова Мария
2025021
 +
В-34.0○
+
сдача ↓ 1защита: готова
+ 		+
В-6—○
+
сдача ↓ 1защита: идёт · 03:07
+ 		+
—
+
вариант не выдан
+ 		изменён
+
+
+ +
+

Элементы интерфейса

+ +
+ + + +
+ +
+ + + +
+ +

Статусы: бейдж — только индикатор, действие — отдельной кнопкой

+
+ ЛР 1 · Анализ данных + Опубликована + +
+
+ ЛР 2 · Сортировки + Черновик + +
+
+ Глава 3 · Жадные алгоритмы + Скрыто + +
+ +

Уведомления

+
Оценки опубликованы всей группе — студенты видят баллы и комментарии.
+
Срок сдачи ЛР 2 истекает завтра в 23:59 МСК.
+
Не удалось сохранить файлы решения — повторите попытку.
+
Новый студент добавлен: логин 2025031, пароль показан один раз.
+
+
+ + + + + + diff --git a/design/style-b3-bordeaux.html b/design/style-b3-bordeaux.html new file mode 100644 index 0000000..d7ede35 --- /dev/null +++ b/design/style-b3-bordeaux.html @@ -0,0 +1,348 @@ + + + + + +LMS — демо стиля Б3: минимализм, акцент бордо + + + + + +
Демо-шаблон «Б3» · сдержанность Б1 + бордовый акцент · цвет только там, где он несёт смысл · сузьте окно: таблица скроллится, колонка «Студент» прилипает
+ +
+ Акцентный цвет: + + + +
+ +
+
+ + +
+ Копнин В. + +
+
+
+ +
+
+
+

Ведомость группы

+

М202-1 · Первый курс

+

24 студента · 3 опубликованные лабораторные · длительность защиты 30 мин

+
+
+ + +
+
+ +
+ + +
+ +
+
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
СтудентЛР 1Анализ данныхЛР 2СортировкиЛР 3Деревья поискаПарольДействия
Иванова Анна
2025001
 +
В-74.5◉
+
сдача ↓ 2защита: готова
+ 		+
В-23.0◉
+
сдача ↓ 1защита: код 552314
+ 		+
—
+
вариант не выдан
+ 		временный
Петров Павел
2025002
 +
В-15.0◉
+
сдача ↓ 3защита: готова
+ 		+
В-4—◉
+
сдача ↓ 1защита: идёт · 12:34
+ 		+
В-3—○
+
не сдано
+ 		изменён
Сидоров Алексей
2025017
 +
В-5—◉
+
не сдано
+ 		+
—
+
вариант не выдан
+ 		+
—
+
вариант не выдан
+ 		временный
Кузнецова Мария
2025021
 +
В-34.0○
+
сдача ↓ 1защита: готова
+ 		+
В-6—○
+
сдача ↓ 1защита: идёт · 03:07
+ 		+
—
+
вариант не выдан
+ 		изменён
+
+
+ +
+

Элементы интерфейса

+ +
+ + + +
+ +
+ + +
+ +

Статусы: бейдж — только индикатор, действие — отдельной кнопкой

+
+ ЛР 1 · Анализ данных + Опубликована + +
+
+ ЛР 2 · Сортировки + Черновик + +
+
+ Глава 3 · Жадные алгоритмы + Скрыто + +
+ +

Уведомление

+
Оценки опубликованы всей группе — студенты видят баллы и комментарии.
+
+
+ + + + + + diff --git a/design/style-c-institutional.html b/design/style-c-institutional.html new file mode 100644 index 0000000..e448b67 --- /dev/null +++ b/design/style-c-institutional.html @@ -0,0 +1,264 @@ + + + + + +LMS — демо стиля В: институциональный + + + + + +
Демо-шаблон «В. Институциональный» · данные условные · тёмная тема будет в реализации
+ +
+
+ + +
+ Копнин В. + +
+
+
+ +
+
+
+

Ведомость группы

+

М202-1 · Первый курс

+

24 студента · 3 опубликованные лабораторные · длительность защиты 30 мин

+
+
+ + +
+
+ +
+ + +
+ +
+ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
СтудентЛР 1Анализ данныхЛР 2СортировкиЛР 3Деревья поискаПарольДействия
Иванова Анна
2025001
 +
В-74.5◉
+
сдача ↓ 2защита: готов
+ 		+
В-23.0◉
+
сдача ↓ 1защита: код 552314
+ 		+
—
+
не выдан
+ 		временный
Петров Павел
2025002
 +
В-15.0◉
+
сдача ↓ 3защита: готов
+ 		+
В-4—◉
+
сдача ↓ 1защита: идёт · 12:34
+ 		+
В-3—○
+
не сдано
+ 		изменён
Сидоров Алексей
2025017
 +
В-5—◉
+
не сдано
+ 		+
—
+
не выдан
+ 		+
—
+
не выдан
+ 		временный
Кузнецова Мария
2025021
 +
В-34.0○
+
сдача ↓ 1защита: готов
+ 		+
В-6—○
+
сдача ↓ 1защита: идёт · 03:07
+ 		+
—
+
не выдан
+ 		изменён
+
+ +
+

Элементы интерфейса

+
+ + + + published + draft +
+
+ + +
+
Оценки опубликованы всей группе — студенты видят баллы и комментарии.
+
+
+ + + + diff --git a/libs/common-http/build.gradle.kts b/libs/common-http/build.gradle.kts index 10e74be..15dfc4c 100644 --- a/libs/common-http/build.gradle.kts +++ b/libs/common-http/build.gradle.kts @@ -7,6 +7,8 @@ dependencies { api(libs.springWeb) api(libs.springBootAutoconfigure) + // spring-tx нужен для обработчика DataIntegrityViolationException в LmsExceptionHandler. + api("org.springframework:spring-tx") // Kotlin-модуль Jackson: корректная десериализация data-классов со значениями по умолчанию. api(libs.jacksonModuleKotlin) } diff --git a/libs/common-http/src/main/kotlin/ru/ysu/lms/common/LmsException.kt b/libs/common-http/src/main/kotlin/ru/ysu/lms/common/LmsException.kt index ba2eaa0..9ed0391 100644 --- a/libs/common-http/src/main/kotlin/ru/ysu/lms/common/LmsException.kt +++ b/libs/common-http/src/main/kotlin/ru/ysu/lms/common/LmsException.kt @@ -37,3 +37,8 @@ class UnauthorizedException( message: String, code: String? = null, ) : LmsException(HttpStatus.UNAUTHORIZED, message, code) + +class TooManyRequestsException( + message: String, + code: String? = null, +) : LmsException(HttpStatus.TOO_MANY_REQUESTS, message, code) diff --git a/libs/common-http/src/main/kotlin/ru/ysu/lms/common/LmsExceptionHandler.kt b/libs/common-http/src/main/kotlin/ru/ysu/lms/common/LmsExceptionHandler.kt index b6a8d0e..b2aff1b 100644 --- a/libs/common-http/src/main/kotlin/ru/ysu/lms/common/LmsExceptionHandler.kt +++ b/libs/common-http/src/main/kotlin/ru/ysu/lms/common/LmsExceptionHandler.kt @@ -1,9 +1,11 @@ package ru.ysu.lms.common +import org.springframework.dao.DataIntegrityViolationException import org.springframework.http.ResponseEntity import org.springframework.web.bind.MethodArgumentNotValidException import org.springframework.web.bind.annotation.ExceptionHandler import org.springframework.web.bind.annotation.RestControllerAdvice +import org.springframework.web.multipart.MaxUploadSizeExceededException @RestControllerAdvice class LmsExceptionHandler { @@ -34,4 +36,37 @@ class LmsExceptionHandler { ), ) } + + /** Некорректный UUID/аргумент в пути или теле — клиентская ошибка, а не 500. */ + @ExceptionHandler(IllegalArgumentException::class) + fun handleIllegalArgument(): ResponseEntity = + ResponseEntity.badRequest().body( + ApiError( + title = "Bad Request", + status = 400, + detail = "Некорректный параметр запроса", + ), + ) + + /** Нарушение уникальности/ссылок (гонки двойной отправки) — конфликт, а не 500. */ + @ExceptionHandler(DataIntegrityViolationException::class) + fun handleDataIntegrity(): ResponseEntity = + ResponseEntity.status(409).body( + ApiError( + title = "Conflict", + status = 409, + detail = "Действие конфликтует с уже существующими данными", + ), + ) + + /** Превышение лимита загрузки — вместо 500 понятный 413. */ + @ExceptionHandler(MaxUploadSizeExceededException::class) + fun handleMaxUpload(): ResponseEntity = + ResponseEntity.status(413).body( + ApiError( + title = "Payload Too Large", + status = 413, + detail = "Файл слишком большой", + ), + ) } diff --git a/services/academic-service/src/main/kotlin/ru/ysu/lms/academic/content/ContentImporter.kt b/services/academic-service/src/main/kotlin/ru/ysu/lms/academic/content/ContentImporter.kt index cf8c28c..21a460c 100644 --- a/services/academic-service/src/main/kotlin/ru/ysu/lms/academic/content/ContentImporter.kt +++ b/services/academic-service/src/main/kotlin/ru/ysu/lms/academic/content/ContentImporter.kt @@ -10,6 +10,7 @@ import ru.ysu.lms.academic.repo.ContentChapterRepository import ru.ysu.lms.academic.repo.ContentSectionRepository import ru.ysu.lms.common.BadRequestException import java.io.ByteArrayInputStream +import java.io.ByteArrayOutputStream import java.util.UUID import java.util.zip.ZipInputStream @@ -233,16 +234,45 @@ class ContentImporter( private fun readZip(bytes: ByteArray): Map { val result = mutableMapOf() + var totalUncompressed = 0L ZipInputStream(ByteArrayInputStream(bytes)).use { zis -> - var entry = zis.nextEntry - while (entry != null) { - if (!entry.isDirectory) result[entry.name.removePrefix("/")] = zis.readAllBytes() - entry = zis.nextEntry + while (true) { + val entry = zis.nextEntry ?: break + if (entry.isDirectory) continue + val data = readBounded(zis, MAX_ENTRY_BYTES, "запись «${entry.name}»") + totalUncompressed += data.size + if (totalUncompressed > MAX_TOTAL_BYTES) { + throw BadRequestException( + "Архив слишком большой: после распаковки суммарно более ${MAX_TOTAL_BYTES / MB} МБ", + ) + } + result[entry.name.removePrefix("/")] = data } } return result } + /** Читает запись архива с ограничением размера — защита от zip-бомб. */ + private fun readBounded( + zis: ZipInputStream, + maxEntryBytes: Long, + what: String, + ): ByteArray { + val buffer = ByteArrayOutputStream() + val chunk = ByteArray(64 * 1024) + var total = 0L + while (true) { + val n = zis.read(chunk) + if (n < 0) break + total += n + if (total > maxEntryBytes) { + throw BadRequestException("$what распаковывается более чем в ${maxEntryBytes / MB} МБ") + } + buffer.write(chunk, 0, n) + } + return buffer.toByteArray() + } + private fun splitFrontMatter(content: String): Pair, String> { if (!content.startsWith("---")) return emptyMap() to content val lines = content.lines() @@ -285,4 +315,10 @@ class ContentImporter( val seenSections: MutableSet = mutableSetOf() val seenArticles: MutableSet = mutableSetOf() } + + private companion object { + const val MB = 1024L * 1024L + const val MAX_ENTRY_BYTES = 128L * MB + const val MAX_TOTAL_BYTES = 256L * MB + } } diff --git a/services/academic-service/src/main/kotlin/ru/ysu/lms/academic/controller/ContentOrderController.kt b/services/academic-service/src/main/kotlin/ru/ysu/lms/academic/controller/ContentOrderController.kt index 056040b..a741909 100644 --- a/services/academic-service/src/main/kotlin/ru/ysu/lms/academic/controller/ContentOrderController.kt +++ b/services/academic-service/src/main/kotlin/ru/ysu/lms/academic/controller/ContentOrderController.kt @@ -1,5 +1,6 @@ package ru.ysu.lms.academic.controller +import org.springframework.security.access.prepost.PreAuthorize import org.springframework.web.bind.annotation.PathVariable import org.springframework.web.bind.annotation.PostMapping import org.springframework.web.bind.annotation.RequestBody @@ -11,8 +12,10 @@ import ru.ysu.lms.academic.content.ReorderSections import ru.ysu.lms.academic.content.ReorderTree import java.util.UUID +/** Реордер материалов: меняет публичный порядок глав/разделов/статей — только админ. */ @RestController -@RequestMapping("/api/v1") +@RequestMapping("/api/v1/content") +@PreAuthorize("hasAuthority('ROLE_ADMIN')") class ContentOrderController( private val order: ContentOrderService, ) { diff --git a/services/academic-service/src/main/kotlin/ru/ysu/lms/academic/controller/ContentPublicController.kt b/services/academic-service/src/main/kotlin/ru/ysu/lms/academic/controller/ContentPublicController.kt index e0322d1..2b3749c 100644 --- a/services/academic-service/src/main/kotlin/ru/ysu/lms/academic/controller/ContentPublicController.kt +++ b/services/academic-service/src/main/kotlin/ru/ysu/lms/academic/controller/ContentPublicController.kt @@ -60,12 +60,19 @@ class ContentPublicController( assetStorage.read(name) ?: return ResponseEntity.notFound().build() val media = runCatching { MediaType.parseMediaType(mime) }.getOrElse { MediaType.APPLICATION_OCTET_STREAM } - return ResponseEntity - .ok() - .contentType(media) - .cacheControl( - org.springframework.http.CacheControl - .maxAge(java.time.Duration.ofDays(7)), - ).body(FileSystemResource(path)) + val builder = + ResponseEntity + .ok() + .contentType(media) + .cacheControl( + org.springframework.http.CacheControl + .maxAge(java.time.Duration.ofDays(7)), + ) + // SVG может содержать скрипты: при открытии напрямую как документа запрещаем их исполнение. + if (media.isCompatibleWith(MediaType.parseMediaType("image/svg+xml"))) { + builder.header("Content-Security-Policy", "script-src 'none'; object-src 'none'") + } + builder.header("X-Content-Type-Options", "nosniff") + return builder.body(FileSystemResource(path)) } } diff --git a/services/academic-service/src/main/kotlin/ru/ysu/lms/academic/lab/LabImportParser.kt b/services/academic-service/src/main/kotlin/ru/ysu/lms/academic/lab/LabImportParser.kt index 1bb3056..185a62e 100644 --- a/services/academic-service/src/main/kotlin/ru/ysu/lms/academic/lab/LabImportParser.kt +++ b/services/academic-service/src/main/kotlin/ru/ysu/lms/academic/lab/LabImportParser.kt @@ -7,6 +7,7 @@ import com.fasterxml.jackson.module.kotlin.KotlinModule import org.springframework.stereotype.Component import ru.ysu.lms.common.BadRequestException import java.io.ByteArrayInputStream +import java.io.ByteArrayOutputStream import java.util.zip.ZipInputStream /** @@ -55,16 +56,45 @@ class LabImportParser { private fun readEntries(bytes: ByteArray): Map { val entries = mutableMapOf() + var totalUncompressed = 0L ZipInputStream(ByteArrayInputStream(bytes)).use { zis -> - var entry = zis.nextEntry - while (entry != null) { - if (!entry.isDirectory) entries[entry.name] = zis.readAllBytes() - entry = zis.nextEntry + while (true) { + val entry = zis.nextEntry ?: break + if (entry.isDirectory) continue + val data = readBounded(zis, MAX_ENTRY_BYTES, "запись «${entry.name}»") + totalUncompressed += data.size + if (totalUncompressed > MAX_TOTAL_BYTES) { + throw BadRequestException( + "Архив слишком большой: после распаковки суммарно более ${MAX_TOTAL_BYTES / MB} МБ", + ) + } + entries[entry.name] = data } } return entries } + /** Читает запись архива с ограничением размера — защита от zip-бомб. */ + private fun readBounded( + zis: ZipInputStream, + maxEntryBytes: Long, + what: String, + ): ByteArray { + val buffer = ByteArrayOutputStream() + val chunk = ByteArray(64 * 1024) + var total = 0L + while (true) { + val n = zis.read(chunk) + if (n < 0) break + total += n + if (total > maxEntryBytes) { + throw BadRequestException("$what распаковывается более чем в ${maxEntryBytes / MB} МБ") + } + buffer.write(chunk, 0, n) + } + return buffer.toByteArray() + } + private fun readMeta(entries: Map): LabMeta { val metaEntry = entries.entries.firstOrNull { it.key.endsWith("lab.yaml") || it.key.endsWith("lab.yml") } @@ -146,4 +176,10 @@ class LabImportParser { "advanced", "повышенная", "повышенной", "hard" -> "advanced" else -> s.lowercase().trim() } + + private companion object { + const val MB = 1024L * 1024L + const val MAX_ENTRY_BYTES = 128L * MB + const val MAX_TOTAL_BYTES = 256L * MB + } } diff --git a/services/gateway/src/main/kotlin/ru/ysu/lms/gateway/security/JwtAuthenticationFilter.kt b/services/gateway/src/main/kotlin/ru/ysu/lms/gateway/security/JwtAuthenticationFilter.kt index b12ce54..489de6c 100644 --- a/services/gateway/src/main/kotlin/ru/ysu/lms/gateway/security/JwtAuthenticationFilter.kt +++ b/services/gateway/src/main/kotlin/ru/ysu/lms/gateway/security/JwtAuthenticationFilter.kt @@ -26,15 +26,18 @@ class JwtAuthenticationFilter( ): Mono { val path = exchange.request.path.value() if (isPublic(path)) { - return chain.filter(exchange) + // Клиентские X-Account-* не должны доходить до сервисов: эти заголовки ставит только gateway. + val stripped = stripAccountHeaders(exchange) + return chain.filter(stripped) } val token = bearerToken(exchange) ?: return unauthorized(exchange, "Требуется вход") return try { val jwt = jwtDecoder.decode(token) + val stripped = stripAccountHeaders(exchange) val request = - exchange.request + stripped.request .mutate() .header("X-Account-Id", jwt.subject.orEmpty()) .header("X-Account-Login", jwt.getClaimAsString("login").orEmpty()) @@ -48,6 +51,17 @@ class JwtAuthenticationFilter( override fun getOrder(): Int = -100 + private fun stripAccountHeaders(exchange: ServerWebExchange): ServerWebExchange = + exchange + .mutate() + .request { req -> + req.headers { headers -> + headers.remove("X-Account-Id") + headers.remove("X-Account-Login") + headers.remove("X-Account-Role") + } + }.build() + private fun isPublic(path: String): Boolean = path.startsWith("/actuator") || path == "/api/v1/auth/login" || diff --git a/services/identity-service/src/main/kotlin/ru/ysu/lms/identity/auth/AuthService.kt b/services/identity-service/src/main/kotlin/ru/ysu/lms/identity/auth/AuthService.kt index 577c736..24d01f6 100644 --- a/services/identity-service/src/main/kotlin/ru/ysu/lms/identity/auth/AuthService.kt +++ b/services/identity-service/src/main/kotlin/ru/ysu/lms/identity/auth/AuthService.kt @@ -27,6 +27,7 @@ class AuthService( private val passwordEncoder: PasswordEncoder, private val jwtEncoder: JwtEncoder, private val jwtProperties: JwtProperties, + private val backoff: LoginBackoff, ) { fun adminExists(): Boolean = accounts.existsByRole(LmsRole.ADMIN) @@ -51,14 +52,16 @@ class AuthService( return toResponse(account, account.login) } - @Transactional(readOnly = true) + /** Без @Transactional на методе: задержка backoff не должна держать JDBC-соединение. */ fun login(request: LoginRequest): AuthResponse { - val account = - accounts.findByLogin(request.login.trim()) - ?: throw UnauthorizedException("Неверный логин или пароль") - if (!passwordEncoder.matches(request.password, account.passwordHash)) { + val login = request.login.trim() + backoff.delayBeforeAttempt(login) + val account = accounts.findByLogin(login) + if (account == null || !passwordEncoder.matches(request.password, account.passwordHash)) { + backoff.recordFailure(login) throw UnauthorizedException("Неверный логин или пароль") } + backoff.reset(login) return toResponse(account, displayName(account)) } diff --git a/services/identity-service/src/main/kotlin/ru/ysu/lms/identity/auth/LoginBackoff.kt b/services/identity-service/src/main/kotlin/ru/ysu/lms/identity/auth/LoginBackoff.kt new file mode 100644 index 0000000..c834c47 --- /dev/null +++ b/services/identity-service/src/main/kotlin/ru/ysu/lms/identity/auth/LoginBackoff.kt @@ -0,0 +1,74 @@ +package ru.ysu.lms.identity.auth + +import org.springframework.stereotype.Service +import org.springframework.transaction.annotation.Propagation +import org.springframework.transaction.annotation.Transactional +import ru.ysu.lms.common.TooManyRequestsException +import ru.ysu.lms.identity.domain.LoginAttempt +import ru.ysu.lms.identity.repo.LoginAttemptRepository +import java.time.Duration +import java.time.Instant +import kotlin.math.min +import kotlin.math.pow + +/** + * Экспоненциальная задержка между неудачными попытками входа (1с, 2с, 4с…). + * Задержка растёт по числу неудач, а не по времени ожидания ответа, поэтому + * перебор пароля становится бессмысленно долгим, а легитимный пользователь, + * знающий пароль, блокировок не видит. Счётчик живёт в БД (переживает рестарт) + * и обнуляется по успеху либо по истечении окна [WINDOW]. + */ +@Service +class LoginBackoff( + private val attempts: LoginAttemptRepository, +) { + /** Короткие задержки выдерживаем в потоке запроса; длиннее — сразу 429 с паузой для клиента. */ + fun delayBeforeAttempt(login: String) { + val failures = effectiveFailures(login) + if (failures <= 0) return + val delaySec = delaySecondsFor(failures) + if (delaySec > INLINE_SLEEP_SECONDS) { + throw TooManyRequestsException( + "Слишком много неудачных попыток входа. Повторите через $delaySec с.", + ) + } + Thread.sleep(delaySec * 1000) + } + + @Transactional(propagation = Propagation.REQUIRES_NEW) + fun recordFailure(login: String) { + val now = Instant.now() + val row = attempts.findById(login).orElse(null) + if (row == null) { + attempts.save(LoginAttempt(login = login, failures = 1, lastFailureAt = now)) + } else { + row.failures += 1 + row.lastFailureAt = now + attempts.save(row) + } + } + + @Transactional(propagation = Propagation.REQUIRES_NEW) + fun reset(login: String) { + attempts.findById(login).orElse(null)?.let(attempts::delete) + } + + /** Неудачи внутри окна [WINDOW]; устаревшую строку сразу гасим. */ + private fun effectiveFailures(login: String): Int { + val row = attempts.findById(login).orElse(null) ?: return 0 + if (row.lastFailureAt.isBefore(Instant.now().minus(WINDOW))) { + attempts.delete(row) + return 0 + } + return row.failures + } + + companion object { + private val WINDOW: Duration = Duration.ofMinutes(15) + private const val MAX_DELAY_SECONDS = 60L + private const val INLINE_SLEEP_SECONDS = 5L + + /** 1, 2, 4, 8 … секунд, с потолком [MAX_DELAY_SECONDS]. */ + fun delaySecondsFor(failures: Int): Long = min(2.0.pow((failures - 1).coerceAtMost(6)), MAX_DELAY_SECONDS.toDouble()).toLong() + } +} diff --git a/services/identity-service/src/main/kotlin/ru/ysu/lms/identity/domain/LoginAttempt.kt b/services/identity-service/src/main/kotlin/ru/ysu/lms/identity/domain/LoginAttempt.kt new file mode 100644 index 0000000..2c9542e --- /dev/null +++ b/services/identity-service/src/main/kotlin/ru/ysu/lms/identity/domain/LoginAttempt.kt @@ -0,0 +1,20 @@ +package ru.ysu.lms.identity.domain + +import jakarta.persistence.Column +import jakarta.persistence.Entity +import jakarta.persistence.Id +import jakarta.persistence.Table +import java.time.Instant + +/** Счётчик неудачных попыток входа для конкретного логина (см. LoginBackoff). */ +@Entity +@Table(name = "login_attempts") +class LoginAttempt( + @Id + @Column(nullable = false, length = 64) + var login: String = "", + @Column(nullable = false) + var failures: Int = 0, + @Column(name = "last_failure_at", nullable = false) + var lastFailureAt: Instant = Instant.now(), +) diff --git a/services/identity-service/src/main/kotlin/ru/ysu/lms/identity/repo/LoginAttemptRepository.kt b/services/identity-service/src/main/kotlin/ru/ysu/lms/identity/repo/LoginAttemptRepository.kt new file mode 100644 index 0000000..b6e2700 --- /dev/null +++ b/services/identity-service/src/main/kotlin/ru/ysu/lms/identity/repo/LoginAttemptRepository.kt @@ -0,0 +1,6 @@ +package ru.ysu.lms.identity.repo + +import org.springframework.data.jpa.repository.JpaRepository +import ru.ysu.lms.identity.domain.LoginAttempt + +interface LoginAttemptRepository : JpaRepository diff --git a/services/identity-service/src/main/resources/db/migration/V3__login_attempts.sql b/services/identity-service/src/main/resources/db/migration/V3__login_attempts.sql new file mode 100644 index 0000000..f48f890 --- /dev/null +++ b/services/identity-service/src/main/resources/db/migration/V3__login_attempts.sql @@ -0,0 +1,7 @@ +-- Счётчик неудачных входов для экспоненциальной задержки (защита от перебора паролей). + +CREATE TABLE login_attempts ( + login varchar(64) PRIMARY KEY, + failures integer NOT NULL DEFAULT 0, + last_failure_at timestamptz NOT NULL DEFAULT now() +); diff --git a/services/identity-service/src/test/kotlin/ru/ysu/lms/identity/AuthServiceLoginBackoffTest.kt b/services/identity-service/src/test/kotlin/ru/ysu/lms/identity/AuthServiceLoginBackoffTest.kt new file mode 100644 index 0000000..ef4ceeb --- /dev/null +++ b/services/identity-service/src/test/kotlin/ru/ysu/lms/identity/AuthServiceLoginBackoffTest.kt @@ -0,0 +1,142 @@ +package ru.ysu.lms.identity + +import io.mockk.Runs +import io.mockk.every +import io.mockk.just +import io.mockk.mockk +import io.mockk.verify +import org.assertj.core.api.Assertions.assertThat +import org.assertj.core.api.Assertions.assertThatThrownBy +import org.junit.jupiter.api.Test +import org.springframework.security.crypto.password.PasswordEncoder +import org.springframework.security.oauth2.jwt.Jwt +import org.springframework.security.oauth2.jwt.JwtEncoder +import org.springframework.security.oauth2.jwt.JwtEncoderParameters +import ru.ysu.lms.common.TooManyRequestsException +import ru.ysu.lms.common.UnauthorizedException +import ru.ysu.lms.identity.auth.AuthService +import ru.ysu.lms.identity.auth.LoginBackoff +import ru.ysu.lms.identity.auth.LoginRequest +import ru.ysu.lms.identity.domain.Account +import ru.ysu.lms.identity.domain.LoginAttempt +import ru.ysu.lms.identity.repo.AccountRepository +import ru.ysu.lms.identity.repo.LoginAttemptRepository +import ru.ysu.lms.identity.repo.StudentRepository +import ru.ysu.lms.security.JwtProperties +import ru.ysu.lms.security.LmsRole +import java.time.Duration +import java.time.Instant +import java.util.Optional + +class AuthServiceLoginBackoffTest { + private val login = "2025001" + private val account = Account(login = login, passwordHash = "hash", role = LmsRole.STUDENT) + + /** Не relaxed: дженерик save у JpaRepository в relaxed-режиме ломает тип возвращаемого значения. */ + private fun attemptsRepo(): LoginAttemptRepository { + val repo = mockk() + every { repo.save(any()) } answers { firstArg() } + every { repo.delete(any()) } just Runs + return repo + } + + private fun service( + accounts: AccountRepository, + attempts: LoginAttemptRepository, + matches: Boolean, + ): AuthService { + val students = mockk() + every { students.findById(account.id) } returns Optional.empty() + val passwordEncoder = mockk() + every { passwordEncoder.matches(any(), any()) } returns matches + val jwtEncoder = mockk() + every { jwtEncoder.encode(any()) } returns + mockk { every { tokenValue } returns "token" } + val backoff = LoginBackoff(attempts) + return AuthService( + accounts, + students, + passwordEncoder, + jwtEncoder, + JwtProperties(accessTokenTtl = Duration.ofMinutes(10)), + backoff, + ) + } + + @Test + fun `failed login records failure`() { + val attempts = attemptsRepo() + val accounts = mockk() + every { accounts.findByLogin(login) } returns account + every { attempts.findById(login) } returns Optional.empty() + val svc = service(accounts, attempts, matches = false) + + assertThatThrownBy { svc.login(LoginRequest(login = login, password = "wrong")) } + .isInstanceOf(UnauthorizedException::class.java) + + verify { + attempts.save( + match { + it.login == login && + it.failures == 1 && + it.lastFailureAt.isAfter(Instant.now().minusSeconds(5)) + }, + ) + } + } + + @Test + fun `many failures rejected with 429 before password check`() { + val attempts = attemptsRepo() + val accounts = mockk() + every { attempts.findById(login) } returns + Optional.of(LoginAttempt(login = login, failures = 6, lastFailureAt = Instant.now())) + val svc = service(accounts, attempts, matches = true) + + assertThatThrownBy { svc.login(LoginRequest(login = login, password = "secret")) } + .isInstanceOf(TooManyRequestsException::class.java) + .hasMessageContaining("32") + verify(exactly = 0) { accounts.findByLogin(any()) } + } + + @Test + fun `successful login resets counter`() { + val attempts = attemptsRepo() + val accounts = mockk() + val stale = LoginAttempt(login = login, failures = 2, lastFailureAt = Instant.now()) + every { accounts.findByLogin(login) } returns account + every { attempts.findById(login) } returnsMany + listOf(Optional.empty(), Optional.of(stale)) + val svc = service(accounts, attempts, matches = true) + + val response = svc.login(LoginRequest(login = login, password = "secret")) + + assertThat(response.token).isEqualTo("token") + verify { attempts.delete(stale) } + } + + @Test + fun `backoff grows exponentially and caps at 60s`() { + assertThat(LoginBackoff.delaySecondsFor(1)).isEqualTo(1) + assertThat(LoginBackoff.delaySecondsFor(2)).isEqualTo(2) + assertThat(LoginBackoff.delaySecondsFor(3)).isEqualTo(4) + assertThat(LoginBackoff.delaySecondsFor(4)).isEqualTo(8) + assertThat(LoginBackoff.delaySecondsFor(7)).isEqualTo(60) + assertThat(LoginBackoff.delaySecondsFor(50)).isEqualTo(60) + } + + @Test + fun `stale failures outside window are ignored and cleared`() { + val attempts = attemptsRepo() + val accounts = mockk() + val stale = LoginAttempt(login = login, failures = 9, lastFailureAt = Instant.now().minusSeconds(16 * 60)) + every { accounts.findByLogin(login) } returns account + every { attempts.findById(login) } returns Optional.of(stale) + val svc = service(accounts, attempts, matches = false) + + // failures=9 должны были дать 429; из-за устаревания строки — обычная неудача входа. + assertThatThrownBy { svc.login(LoginRequest(login = login, password = "wrong")) } + .isInstanceOf(UnauthorizedException::class.java) + verify { attempts.delete(stale) } + } +} diff --git a/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/AcademicClient.kt b/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/AcademicClient.kt index 0f3c9ea..51f5391 100644 --- a/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/AcademicClient.kt +++ b/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/AcademicClient.kt @@ -2,15 +2,18 @@ package ru.ysu.lms.progress import org.springframework.beans.factory.annotation.Value import org.springframework.core.ParameterizedTypeReference +import org.springframework.http.client.JdkClientHttpRequestFactory import org.springframework.stereotype.Service import org.springframework.web.client.RestClient import ru.ysu.lms.common.NotFoundException +import java.net.http.HttpClient +import java.time.Duration @Service class AcademicClient( @Value("\${lms.academic.base-url}") baseUrl: String, ) { - private val client = RestClient.builder().baseUrl(baseUrl).build() + private val client = restClient(baseUrl) fun getLab( labId: String, @@ -41,7 +44,7 @@ class AcademicClient( class IdentityClient( @Value("\${lms.identity.base-url}") baseUrl: String, ) { - private val client = RestClient.builder().baseUrl(baseUrl).build() + private val client = restClient(baseUrl) fun getGroup( groupId: String, @@ -65,3 +68,20 @@ class IdentityClient( .body(type) ?: emptyList() } } + +/** Таймауты обязательны: зависший downstream держит JDBC-соединение из пула и вешает сервис. */ +private const val CONNECT_TIMEOUT_SECONDS = 3L +private const val READ_TIMEOUT_SECONDS = 30L + +private fun restClient(baseUrl: String): RestClient = + RestClient + .builder() + .baseUrl(baseUrl) + .requestFactory( + JdkClientHttpRequestFactory( + HttpClient + .newBuilder() + .connectTimeout(Duration.ofSeconds(CONNECT_TIMEOUT_SECONDS)) + .build(), + ).apply { setReadTimeout(Duration.ofSeconds(READ_TIMEOUT_SECONDS)) }, + ).build() diff --git a/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/DefenseService.kt b/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/DefenseService.kt index 94866cd..2ee1469 100644 --- a/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/DefenseService.kt +++ b/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/DefenseService.kt @@ -102,7 +102,8 @@ class DefenseService( sessions.save(s) } - @Transactional(readOnly = true) + /** Не readOnly: внутри переносим истёкшие защиты в ready — изменения должны сохраниться. */ + @Transactional fun listForLab(labId: UUID): List { syncTimeouts() return sessions.findByLabIdOrderByStudentIdAsc(labId).map { toDto(it) } diff --git a/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/Dtos.kt b/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/Dtos.kt index f62e3c7..42ec706 100644 --- a/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/Dtos.kt +++ b/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/Dtos.kt @@ -180,3 +180,18 @@ data class PublicationSetRequest( val labId: String, val studentIds: List = emptyList(), ) + +/** Агрегированные данные одной лабораторной по всем студентам группы (для страницы группы одним запросом). */ +data class LabDataResponse( + val labId: String, + val assignments: List = emptyList(), + val levels: List = emptyList(), + val submissions: List = emptyList(), + val defenses: List = emptyList(), + val publication: PublicationState = PublicationState(), +) + +data class GroupLabDataResponse( + val groupId: String, + val labs: List = emptyList(), +) diff --git a/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/GroupLabDataController.kt b/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/GroupLabDataController.kt new file mode 100644 index 0000000..5d50eba --- /dev/null +++ b/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/GroupLabDataController.kt @@ -0,0 +1,24 @@ +package ru.ysu.lms.progress + +import org.springframework.security.access.prepost.PreAuthorize +import org.springframework.security.core.annotation.AuthenticationPrincipal +import org.springframework.security.oauth2.jwt.Jwt +import org.springframework.web.bind.annotation.GetMapping +import org.springframework.web.bind.annotation.PathVariable +import org.springframework.web.bind.annotation.RequestParam +import org.springframework.web.bind.annotation.RestController +import java.util.UUID + +@RestController +class GroupLabDataController( + private val service: GroupLabDataService, +) { + /** Данные всех студентов группы по выбранным лабораторным — только для админа. */ + @GetMapping("/api/v1/assignments/group/{groupId}") + @PreAuthorize("hasAuthority('ROLE_ADMIN')") + fun groupLabData( + @PathVariable groupId: UUID, + @RequestParam("labId") labIds: List, + @AuthenticationPrincipal jwt: Jwt, + ): GroupLabDataResponse = service.labData(groupId, labIds, jwt.tokenValue) +} diff --git a/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/GroupLabDataService.kt b/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/GroupLabDataService.kt new file mode 100644 index 0000000..b83a8f2 --- /dev/null +++ b/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/GroupLabDataService.kt @@ -0,0 +1,46 @@ +package ru.ysu.lms.progress + +import org.springframework.stereotype.Service +import java.util.UUID + +/** + * Агрегатор данных страницы группы: назначения, уровни, последние сдачи, + * защиты и состояние публикации по каждой лабораторной. Заменяет ~4N запросов + * BFF к сервисам одним вызовом. Без своей транзакции: внутренние сервисы + * управляют своими (внутри пишущий syncTimeouts — readOnly снаружи нельзя). + */ +@Service +class GroupLabDataService( + private val progress: ProgressService, + private val submissions: SubmissionService, + private val defenses: DefenseService, + private val publications: GradePublicationService, +) { + fun labData( + groupId: UUID, + labIds: List, + token: String, + ): GroupLabDataResponse = + GroupLabDataResponse( + groupId = groupId.toString(), + labs = + labIds.map { labId -> + LabDataResponse( + labId = labId.toString(), + assignments = progress.assignmentsForLabEnriched(labId, token), + levels = + progress.getLevelsForLab(labId).map { sll -> + LevelAssignmentResponse( + studentId = sll.studentId.toString(), + labId = sll.labId.toString(), + labLevelId = sll.labLevelId.toString(), + manual = sll.manual, + ) + }, + submissions = submissions.latestPerStudent(labId), + defenses = defenses.listForLab(labId), + publication = publications.state(groupId, labId), + ) + }, + ) +} diff --git a/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/SubmissionService.kt b/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/SubmissionService.kt index 4bff03e..90d750e 100644 --- a/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/SubmissionService.kt +++ b/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/SubmissionService.kt @@ -1,5 +1,6 @@ package ru.ysu.lms.progress +import org.springframework.dao.DataIntegrityViolationException import org.springframework.stereotype.Service import org.springframework.transaction.annotation.Transactional import org.springframework.web.multipart.MultipartFile @@ -8,6 +9,7 @@ import ru.ysu.lms.common.ForbiddenException import ru.ysu.lms.common.NotFoundException import ru.ysu.lms.progress.domain.Submission import ru.ysu.lms.progress.repo.SubmissionRepository +import java.io.IOException import java.nio.file.Files import java.time.Instant import java.util.UUID @@ -28,8 +30,22 @@ class SubmissionService( val payload = validatedPayload(files) ensureDeadline(requireLab(labId, token)) val seq = nextSeq(studentId, labId) - val stored = storage.store(studentId, labId, seq, payload) - val entity = submissions.save(Submission(studentId = studentId, labId = labId, seq = seq)) + // Сначала запись в БД: unique-констрейнт (student, lab, seq) гасит двойную отправку, + // и проигравшая попытка не трогает файлы на диске (иначе перезаписала бы предыдущую сдачу). + val entity = + try { + submissions.saveAndFlush(Submission(studentId = studentId, labId = labId, seq = seq)) + } catch (ignore: DataIntegrityViolationException) { + throw BadRequestException("Эта попытка уже отправлена — обновите страницу и попробуйте ещё раз") + } + val stored = + try { + storage.store(studentId, labId, seq, payload) + } catch (ignore: IOException) { + // строка откатится вместе с транзакцией; убираем недописанные файлы + runCatching { storage.deleteAttemptDir(studentId, labId, seq) } + throw BadRequestException("Не удалось сохранить файлы решения, попробуйте ещё раз") + } return toDto(entity, stored.files) } diff --git a/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/SubmissionStorage.kt b/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/SubmissionStorage.kt index 3a58bc4..b50e8d0 100644 --- a/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/SubmissionStorage.kt +++ b/services/progress-service/src/main/kotlin/ru/ysu/lms/progress/SubmissionStorage.kt @@ -37,12 +37,32 @@ class SubmissionStorage( val names = mutableListOf() for ((rawName, bytes) in files) { val safe = sanitize(rawName) - Files.write(dir.resolve(safe), bytes) + val target = dir.resolve(safe).normalize() + require(target.startsWith(dir)) { "path escape blocked" } + Files.write(target, bytes) names.add(safe) } return StoredSubmission(dir, names) } + /** Страховка после сбоя записи: недописанная попытка не должна оставлять мусор на диске. */ + fun deleteAttemptDir( + studentId: UUID, + labId: UUID, + seq: Int, + ) { + val dir = dirFor(studentId, labId, seq).normalize() + require(dir.startsWith(rootDir) && dir != rootDir) { "path escape blocked" } + if (!Files.exists(dir)) return + Files + .walk(dir) + .use { walk -> + walk + .sorted(Comparator.reverseOrder()) + .forEach { p -> runCatching { Files.delete(p) } } + } + } + /** Собирает zip из сохранённых файлов сдачи, сохраняя реальное время изменения файлов. */ fun zipOf(storedDir: Path): ByteArray { val out = ByteArrayOutputStream() diff --git a/services/progress-service/src/test/kotlin/ru/ysu/lms/progress/ProgressDistributionIntegrationTest.kt b/services/progress-service/src/test/kotlin/ru/ysu/lms/progress/ProgressDistributionIntegrationTest.kt index bb44e9a..003ab74 100644 --- a/services/progress-service/src/test/kotlin/ru/ysu/lms/progress/ProgressDistributionIntegrationTest.kt +++ b/services/progress-service/src/test/kotlin/ru/ysu/lms/progress/ProgressDistributionIntegrationTest.kt @@ -79,4 +79,46 @@ class ProgressDistributionIntegrationTest { assertThat(saved.map { it.studentId.toString() }).containsExactlyInAnyOrder(student1, student2) assertThat(saved.map { it.variantId.toString() }).containsAnyOf(variantA, variantB) } + + @Autowired + private lateinit var groupLabData: GroupLabDataService + + @Test + fun `group lab data aggregates per-lab data in one call`() { + val labId = UUID.randomUUID() + val student1 = UUID.randomUUID().toString() + val student2 = UUID.randomUUID().toString() + val variantA = UUID.randomUUID().toString() + + `when`(identity.listGroups("t")).thenReturn(listOf(mapOf("id" to "g1"))) + `when`(identity.getGroup("g1", "t")).thenReturn( + GroupInfo(id = "g1", name = "Группа", students = listOf(StudentInfo(student1), StudentInfo(student2))), + ) + `when`(academic.getLab(labId.toString(), "t")).thenReturn( + LabInfo( + id = labId.toString(), + levels = + listOf( + LevelInfo( + id = "lvl", + code = "basic", + title = "Основная", + variants = listOf(VariantInfo(id = variantA, number = "1")), + ), + ), + ), + ) + service.distribute(DistributeRequest(labIds = listOf(labId.toString()), groupIds = listOf("g1")), "t") + + val data = groupLabData.labData(UUID.randomUUID(), listOf(labId), "t") + + assertThat(data.labs).hasSize(1) + val lab = data.labs.first() + assertThat(lab.labId).isEqualTo(labId.toString()) + assertThat(lab.assignments).hasSize(2) + assertThat(lab.assignments.all { it.variantNumber == "1" }).isTrue() + assertThat(lab.defenses).isEmpty() + assertThat(lab.submissions).isEmpty() + assertThat(lab.publication.all).isFalse() + } } diff --git a/services/progress-service/src/test/kotlin/ru/ysu/lms/progress/SubmissionServiceTest.kt b/services/progress-service/src/test/kotlin/ru/ysu/lms/progress/SubmissionServiceTest.kt index 1feadb5..73ac55f 100644 --- a/services/progress-service/src/test/kotlin/ru/ysu/lms/progress/SubmissionServiceTest.kt +++ b/services/progress-service/src/test/kotlin/ru/ysu/lms/progress/SubmissionServiceTest.kt @@ -6,6 +6,7 @@ import org.assertj.core.api.Assertions.assertThat import org.assertj.core.api.Assertions.assertThatThrownBy import org.junit.jupiter.api.Test import org.junit.jupiter.api.io.TempDir +import org.springframework.dao.DataIntegrityViolationException import org.springframework.mock.web.MockMultipartFile import ru.ysu.lms.common.BadRequestException import ru.ysu.lms.progress.domain.Submission @@ -27,7 +28,7 @@ class SubmissionServiceTest { var saved: Submission? = null every { academic.getLabSummary(any(), any()) } returns LabInfo(id = labId.toString(), deadlineAt = null) every { repo.findFirstByStudentIdAndLabIdOrderBySeqDesc(any(), any()) } returns null - every { repo.save(any()) } answers { + every { repo.saveAndFlush(any()) } answers { firstArg().also { saved = it } } every { repo.findById(any()) } answers { java.util.Optional.ofNullable(saved) } @@ -73,6 +74,17 @@ class SubmissionServiceTest { assertThat(dto.files).containsExactly("project.zip") } + @Test + fun `duplicate concurrent attempt is rejected as bad request`() { + val (service, repo) = service() + val file = MockMultipartFile("files", "main.c", "text/plain", "#include \n".toByteArray()) + every { repo.saveAndFlush(any()) } throws DataIntegrityViolationException("duplicate key") + + assertThatThrownBy { service.submit(studentId, labId, listOf(file), "token") } + .isInstanceOf(BadRequestException::class.java) + .hasMessageContaining("уже отправлена") + } + private fun zipEntries(bytes: ByteArray): Map { val result = linkedMapOf() ZipInputStream(bytes.inputStream()).use { zin -> diff --git a/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminGroupController.kt b/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminGroupController.kt index 6d5241a..5702de1 100644 --- a/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminGroupController.kt +++ b/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminGroupController.kt @@ -31,10 +31,11 @@ import ru.ysu.lms.web.api.createGroup import ru.ysu.lms.web.api.deleteCommentTemplate import ru.ysu.lms.web.api.downloadSubmissionAsAdmin import ru.ysu.lms.web.api.getLabAssignments +import ru.ysu.lms.web.api.getLevels import ru.ysu.lms.web.api.groupCandidates import ru.ysu.lms.web.api.groupDetail +import ru.ysu.lms.web.api.groupLabData import ru.ysu.lms.web.api.importGroup -import ru.ysu.lms.web.api.labDefenses import ru.ysu.lms.web.api.labDetail import ru.ysu.lms.web.api.labSubmissions import ru.ysu.lms.web.api.listGrades @@ -66,7 +67,6 @@ class AdminGroupController( val grades = api.listGrades(t) model.addAttribute("groupCount", groups.size) model.addAttribute("studentCount", groups.sumOf { it.studentCount }) - model.addAttribute("groupCount", groups.size) model.addAttribute("labCount", labs.size) val gradedByStudent = grades.groupBy { it.studentId } val averages = @@ -147,9 +147,11 @@ class AdminGroupController( val commentsBy = grades.associate { "${it.studentId}|${it.labId}" to it.comments.map { ct -> ct.text to ct.tone } } val memberIds = group.students.map { it.id } val memberIdSet = memberIds.toSet() + // Данные всех лаб одним запросом вместо 4 вызовов на каждую лабу + val dataByLab = api.groupLabData(t, id, labs.map { it.id }).labs.associateBy { it.labId } val publishedByLab = labs.associate { lab -> - val state = api.publicationState(t, id, lab.id) + val state = dataByLab.getValue(lab.id).publication lab.id to if (state.all) memberIdSet else state.studentIds.filter { it in memberIdSet }.toSet() } val labPublication = @@ -162,9 +164,9 @@ class AdminGroupController( total = memberIds.size, ) } - val assignmentsBy = labs.associateWith { lab -> api.getLabAssignments(t, lab.id).associateBy { it.studentId } } - val submissionsBy = labs.associateWith { lab -> api.labSubmissions(t, lab.id).associateBy { it.studentId } } - val defensesBy = labs.associateWith { lab -> api.labDefenses(t, lab.id).associateBy { it.studentId } } + val assignmentsBy = labs.associateWith { lab -> dataByLab.getValue(lab.id).assignments.associateBy { it.studentId } } + val submissionsBy = labs.associateWith { lab -> dataByLab.getValue(lab.id).submissions.associateBy { it.studentId } } + val defensesBy = labs.associateWith { lab -> dataByLab.getValue(lab.id).defenses.associateBy { it.studentId } } val rows = buildRows(group.students, labs, gradeBy, commentsBy, publishedByLab, assignmentsBy, submissionsBy, defensesBy) @@ -281,6 +283,57 @@ class AdminGroupController( model.addAttribute("labSubmissionCounts", view.submissionCounts) } + /** Фрагмент большой модалки «Выдача вариантов»: вся группа одной таблицей по выбранной лабе. */ + @GetMapping("/groups/{id}/assign-modal") + fun assignModal( + @PathVariable id: String, + @RequestParam labId: String, + session: HttpSession, + model: Model, + ): String { + val t = token(session) + val group = api.groupDetail(t, id) + val lab = api.labDetail(t, labId) + val levels = api.getLevels(t, labId).associateBy { it.studentId } + val assignments = api.getLabAssignments(t, labId).associateBy { it.studentId } + val submissions = api.labSubmissions(t, labId).associateBy { it.studentId } + val variantUsage = java.util.HashMap() + assignments.values.forEach { variantUsage[it.variantId] = (variantUsage[it.variantId] ?: 0) + 1 } + val rows = + group.students.map { s -> + val assignment = assignments[s.id] + AssignModalRow( + studentId = s.id, + fullName = s.fullName, + studentNo = s.studentNo, + levelId = levels[s.id]?.labLevelId ?: "", + variantId = assignment?.variantId, + variantNumber = assignment?.variantNumber, + manual = assignment?.manual == true, + submissionId = submissions[s.id]?.id, + submissionSeq = submissions[s.id]?.seq, + ) + } + model.addAttribute("lab", lab) + model.addAttribute("groupId", id) + model.addAttribute("rows", rows) + model.addAttribute("variantUsage", variantUsage) + model.addAttribute("refreshUrl", "/admin/groups/$id/assign-modal?labId=$labId") + return "admin/assign-fragment" + } + + private data class AssignModalRow( + val studentId: String, + val fullName: String, + val studentNo: String, + val levelId: String, + val variantId: String?, + val variantNumber: String?, + val manual: Boolean, + val submissionId: String?, + val submissionSeq: Int?, + ) + @PostMapping("/groups/{id}/grades") fun setGrade( @PathVariable id: String, diff --git a/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminLabController.kt b/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminLabController.kt index cf08cb1..249f549 100644 --- a/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminLabController.kt +++ b/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminLabController.kt @@ -16,15 +16,10 @@ import ru.ysu.lms.web.api.ApiClient import ru.ysu.lms.web.api.ApiException import ru.ysu.lms.web.api.AssignVariantPayload import ru.ysu.lms.web.api.LabCreatePayload -import ru.ysu.lms.web.api.LabDetailDto -import ru.ysu.lms.web.api.LabLevelDetailDto import ru.ysu.lms.web.api.LabUpdatePayload import ru.ysu.lms.web.api.LevelAssignPayload -import ru.ysu.lms.web.api.LevelAssignmentResultDto import ru.ysu.lms.web.api.LevelPayload import ru.ysu.lms.web.api.LevelSavePayload -import ru.ysu.lms.web.api.SubmissionDto -import ru.ysu.lms.web.api.VariantAssignmentResultDto import ru.ysu.lms.web.api.VariantSavePayload import ru.ysu.lms.web.api.addLevel import ru.ysu.lms.web.api.assignVariant @@ -33,12 +28,9 @@ import ru.ysu.lms.web.api.deleteLab import ru.ysu.lms.web.api.deleteLevel import ru.ysu.lms.web.api.deleteVariant import ru.ysu.lms.web.api.getLabAssignments -import ru.ysu.lms.web.api.getLevels -import ru.ysu.lms.web.api.groupDetail import ru.ysu.lms.web.api.importLab import ru.ysu.lms.web.api.labDetail import ru.ysu.lms.web.api.labSubmissions -import ru.ysu.lms.web.api.listGroups import ru.ysu.lms.web.api.listLabs import ru.ysu.lms.web.api.renameLevel import ru.ysu.lms.web.api.saveVariant @@ -94,18 +86,25 @@ class AdminLabController( @PostMapping("/labs/{id}/toggle-status") fun toggleStatus( @PathVariable id: String, + @RequestParam(required = false) back: String?, session: HttpSession, redirect: RedirectAttributes, ): String = try { api.toggleLabStatus(token(session), id) redirect.addFlashAttribute("notice", "Статус лабораторной изменён") - "redirect:/admin/labs/$id" + "redirect:" + safeBack(back, "/admin/labs/$id") } catch (ex: ApiException) { redirect.addFlashAttribute("error", ex.message) "redirect:/admin/labs/$id" } + /** Возврат на страницу, откуда пришёл запрос; принимаем только внутренние пути. */ + private fun safeBack( + back: String?, + fallback: String, + ): String = if (!back.isNullOrBlank() && back.startsWith("/") && !back.startsWith("//")) back else fallback + @PostMapping("/labs/{id}/delete") fun deleteLab( @PathVariable id: String, @@ -129,23 +128,13 @@ class AdminLabController( ): String { val t = token(session) val lab = api.labDetail(t, id) - val submissionsByStudent = api.labSubmissions(t, id).associateBy { it.studentId } + val submissionCount = api.labSubmissions(t, id).size val assignments = api.getLabAssignments(t, id) - val assignedCount = assignments.size - val levelByStudent = api.getLevels(t, id).associateBy { it.studentId } - val assignmentByStudent = assignments.associateBy { it.studentId } val variantUsage = java.util.HashMap() assignments.forEach { variantUsage[it.variantId] = (variantUsage[it.variantId] ?: 0) + 1 } val allVariants = lab.levels.flatMap { it.variants } val totalVariants = allVariants.size val occupiedVariants = allVariants.count { (variantUsage[it.id] ?: 0) > 0 } - val defaultLevel = lab.levels.firstOrNull() - - val assignmentGroups = - buildAssignmentGroups( - t, - LabAssignmentData(lab, levelByStudent, assignmentByStudent, submissionsByStudent, defaultLevel), - ) model.addAttribute("lab", lab) model.addAttribute("descriptionHtml", markdown.render(lab.descriptionMd)) @@ -159,67 +148,12 @@ class AdminLabController( ru.ysu.lms.web.DateUtils .formatMsk(lab.defenseAt), ) - model.addAttribute( - "deadlineLocal", - ru.ysu.lms.web.DateUtils - .localInput(lab.deadlineAt), - ) - model.addAttribute( - "defenseLocal", - ru.ysu.lms.web.DateUtils - .localInput(lab.defenseAt), - ) - model.addAttribute("assignedCount", assignedCount) - model.addAttribute("submissionCount", submissionsByStudent.size) - model.addAttribute("assignmentGroups", assignmentGroups) - model.addAttribute("groups", api.listGroups(t)) - model.addAttribute("variantUsage", variantUsage) + model.addAttribute("submissionCount", submissionCount) model.addAttribute("totalVariants", totalVariants) model.addAttribute("occupiedVariants", occupiedVariants) - model.addAttribute("freeVariants", totalVariants - occupiedVariants) return "admin/lab" } - private data class LabAssignmentData( - val lab: LabDetailDto, - val levelByStudent: Map, - val assignmentByStudent: Map, - val submissionsByStudent: Map, - val defaultLevel: LabLevelDetailDto?, - ) - - /** Строки «группа → студенты» со их уровнем, вариантом и последней сдачей. */ - private fun buildAssignmentGroups( - token: String, - data: LabAssignmentData, - ): List = - api.listGroups(token).map { g -> - val detail = api.groupDetail(token, g.id) - AdminLabGroup( - groupName = g.name, - students = - detail.students.map { s -> - val levelState = data.levelByStudent[s.id] - val level = data.lab.levels.firstOrNull { it.id == levelState?.labLevelId } ?: data.defaultLevel - val va = data.assignmentByStudent[s.id] - AdminLabStudent( - studentId = s.id, - fullName = s.fullName, - studentNo = s.studentNo, - levelId = level?.id ?: "", - levelTitle = level?.title ?: "—", - levelChanged = levelState?.manual == true, - variantId = va?.variantId, - variantNumber = va?.variantNumber, - source = if (va?.manual == true) "вручную" else "авто", - submissionId = data.submissionsByStudent[s.id]?.id, - submissionSeq = data.submissionsByStudent[s.id]?.seq, - submissionFiles = data.submissionsByStudent[s.id]?.files ?: emptyList(), - ) - }, - ) - } - @PostMapping("/labs/{labId}/assign") fun assignVariant( @PathVariable labId: String, @@ -320,22 +254,21 @@ class AdminLabController( "redirect:/admin/labs/$labId" } - @GetMapping("/labs/{id}/edit") /** Быстрые действия «удалить уровень/вариант» на форме лабы; null — обычное сохранение. */ private fun handleEditDeleteAction( token: String, id: String, - action: String, + action: String?, redirect: RedirectAttributes, ): String? = when { - action.startsWith("delete-level-") -> { - api.deleteLevel(token, id, action.removePrefix("delete-level-")) + action.orEmpty().startsWith("delete-level-") -> { + api.deleteLevel(token, id, action.orEmpty().removePrefix("delete-level-")) redirect.addFlashAttribute("notice", "Уровень удалён") "redirect:/admin/labs/$id/edit" } - action.startsWith("delete-variant-") -> { - api.deleteVariant(token, id, action.removePrefix("delete-variant-")) + action.orEmpty().startsWith("delete-variant-") -> { + api.deleteVariant(token, id, action.orEmpty().removePrefix("delete-variant-")) redirect.addFlashAttribute("notice", "Вариант удалён") "redirect:/admin/labs/$id/edit" } @@ -392,6 +325,7 @@ class AdminLabController( } } + @GetMapping("/labs/{id}/edit") fun editLabForm( @PathVariable id: String, session: HttpSession, diff --git a/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminLabView.kt b/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminLabView.kt index 916a43a..0c60c54 100644 --- a/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminLabView.kt +++ b/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminLabView.kt @@ -1,31 +1,5 @@ package ru.ysu.lms.web.admin -data class AdminLabGroup( - val groupName: String, - val students: List, -) - -data class AdminLabStudent( - val studentId: String, - val fullName: String, - val studentNo: String, - val levelId: String, - val levelTitle: String, - val levelChanged: Boolean, - val variantId: String?, - val variantNumber: String?, - val source: String, - val submissionId: String? = null, - val submissionSeq: Int? = null, - val submissionFiles: List = emptyList(), - val defenseStatus: String? = null, - val defenseCode: String? = null, - val defenseSessionId: String? = null, - val defenseSecondsLeft: Long? = null, - val defenseAssigned: Boolean = false, - val adminAssignmentMd: String = "", -) - data class AdminStudentGrades( val studentId: String, val fullName: String, diff --git a/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminStudentController.kt b/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminStudentController.kt index f906c40..07b8358 100644 --- a/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminStudentController.kt +++ b/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminStudentController.kt @@ -19,6 +19,7 @@ import ru.ysu.lms.web.api.CreateStudentRequestDto import ru.ysu.lms.web.api.createStudent import ru.ysu.lms.web.api.credentials import ru.ysu.lms.web.api.enrollStudent +import ru.ysu.lms.web.api.groupDetail import ru.ysu.lms.web.api.resetPassword import ru.ysu.lms.web.api.unenrollStudent import tools.jackson.databind.ObjectMapper @@ -42,13 +43,25 @@ class AdminStudentController( try { val result = api.resetPassword(token(session), studentId) redirect.addFlashAttribute("resetPassword", result.password) - redirect.addFlashAttribute("resetFor", studentId) + // В тосте должно быть видно, ЧЬЙ пароль: за сеанс можно сбросить несколько подряд. + redirect.addFlashAttribute("resetFor", studentName(token(session), id, studentId)) "redirect:/admin/groups/$id" } catch (ex: ApiException) { redirect.addFlashAttribute("error", ex.message) "redirect:/admin/groups/$id" } + private fun studentName( + token: String, + groupId: String, + studentId: String, + ): String = + runCatching { api.groupDetail(token, groupId).students } + .getOrNull() + ?.firstOrNull { it.id == studentId } + ?.fullName + ?: studentId + @GetMapping("/groups/{id}/credentials") fun credentials( @PathVariable id: String, diff --git a/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminUsersController.kt b/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminUsersController.kt index 83150e2..8178f48 100644 --- a/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminUsersController.kt +++ b/web/src/main/kotlin/ru/ysu/lms/web/admin/AdminUsersController.kt @@ -54,9 +54,27 @@ class AdminUsersController( return try { val t = token(session) val student = api.searchStudents(t, null).firstOrNull { it.id == id } - api.purgeStudent(t, id) - val deleted = api.deleteStudent(t, id) - val name = deleted.fullName.ifBlank { student?.fullName ?: deleted.login } + // Сначала аккаунт, потом очистка данных: сбой очистки оставит безвредные + // данные-сироты (дочищаются повторным запуском), а не «живой» аккаунт без данных. + val deleted = + try { + api.deleteStudent(t, id) + } catch (alreadyGone: ApiException) { + if (alreadyGone.status != 404) throw alreadyGone + null // аккаунта уже нет — дочищаем данные после прошлого сбоя + } + val name = + (deleted?.fullName?.ifBlank { student?.fullName ?: deleted.login } ?: student?.fullName ?: id) + try { + api.purgeStudent(t, id) + } catch (purgeError: ApiException) { + redirect.addFlashAttribute( + "error", + "Аккаунт $name удалён, но данные прогресса не очищены (${purgeError.message}). " + + "Нажмите «Удалить» ещё раз, чтобы дочистить.", + ) + return redirectTo(query) + } redirect.addFlashAttribute("notice", "Пользователь $name удалён вместе со всеми данными") redirectTo(query) } catch (ex: ApiException) { diff --git a/web/src/main/kotlin/ru/ysu/lms/web/api/ApiClient.kt b/web/src/main/kotlin/ru/ysu/lms/web/api/ApiClient.kt index 9f162fa..385bb8a 100644 --- a/web/src/main/kotlin/ru/ysu/lms/web/api/ApiClient.kt +++ b/web/src/main/kotlin/ru/ysu/lms/web/api/ApiClient.kt @@ -2,18 +2,33 @@ package ru.ysu.lms.web.api import org.springframework.beans.factory.annotation.Value import org.springframework.http.HttpHeaders +import org.springframework.http.client.JdkClientHttpRequestFactory import org.springframework.stereotype.Component import org.springframework.web.client.RestClient import org.springframework.web.client.RestClientException import org.springframework.web.client.RestClientResponseException import java.net.URLDecoder +import java.net.http.HttpClient import java.nio.charset.StandardCharsets +import java.time.Duration @Component class ApiClient( @Value("\${lms.gateway.base-url}") baseUrl: String, ) { - internal val client: RestClient = RestClient.builder().baseUrl(baseUrl).build() + // readTimeout большой: через BFF ходят загрузки/выгрузки до 200 МБ (импорт ZIP, архивы сдач). + internal val client: RestClient = + RestClient + .builder() + .baseUrl(baseUrl) + .requestFactory( + JdkClientHttpRequestFactory( + HttpClient + .newBuilder() + .connectTimeout(Duration.ofSeconds(5)) + .build(), + ).apply { setReadTimeout(Duration.ofSeconds(300)) }, + ).build() internal fun bearer(token: String): String = "Bearer $token" diff --git a/web/src/main/kotlin/ru/ysu/lms/web/api/ApiClientGroup.kt b/web/src/main/kotlin/ru/ysu/lms/web/api/ApiClientGroup.kt new file mode 100644 index 0000000..1117e8c --- /dev/null +++ b/web/src/main/kotlin/ru/ysu/lms/web/api/ApiClientGroup.kt @@ -0,0 +1,25 @@ +package ru.ysu.lms.web.api + +import org.springframework.http.HttpHeaders + +/** Данные группы по всем лабораторным одним запросом (назначения, уровни, сдачи, защиты, публикация). */ +fun ApiClient.groupLabData( + token: String, + groupId: String, + labIds: List, +): GroupLabDataDto = + call { + val spec = + client + .get() + .uri { builder -> + builder + .path("/api/v1/assignments/group/{groupId}") + .queryParam("labId", *labIds.toTypedArray()) + .build(groupId) + } + spec + .header(HttpHeaders.AUTHORIZATION, bearer(token)) + .retrieve() + .body(GroupLabDataDto::class.java) + } diff --git a/web/src/main/kotlin/ru/ysu/lms/web/api/ApiDtos.kt b/web/src/main/kotlin/ru/ysu/lms/web/api/ApiDtos.kt index 8040728..a420d00 100644 --- a/web/src/main/kotlin/ru/ysu/lms/web/api/ApiDtos.kt +++ b/web/src/main/kotlin/ru/ysu/lms/web/api/ApiDtos.kt @@ -477,3 +477,19 @@ data class DefenseDto( ) { var assignmentHtml: String = "" } + +@JsonIgnoreProperties(ignoreUnknown = true) +data class LabDataDto( + val labId: String = "", + val assignments: List = emptyList(), + val levels: List = emptyList(), + val submissions: List = emptyList(), + val defenses: List = emptyList(), + val publication: PublicationStateDto = PublicationStateDto(), +) + +@JsonIgnoreProperties(ignoreUnknown = true) +data class GroupLabDataDto( + val groupId: String = "", + val labs: List = emptyList(), +) diff --git a/web/src/main/kotlin/ru/ysu/lms/web/config/WebConfig.kt b/web/src/main/kotlin/ru/ysu/lms/web/config/WebConfig.kt index 7a83107..bbc8865 100644 --- a/web/src/main/kotlin/ru/ysu/lms/web/config/WebConfig.kt +++ b/web/src/main/kotlin/ru/ysu/lms/web/config/WebConfig.kt @@ -1,9 +1,11 @@ package ru.ysu.lms.web.config import jakarta.servlet.http.HttpServletRequest +import jakarta.servlet.http.HttpServletResponse import org.springframework.context.annotation.Configuration import org.springframework.web.bind.annotation.ControllerAdvice import org.springframework.web.bind.annotation.ModelAttribute +import org.springframework.web.servlet.HandlerInterceptor import org.springframework.web.servlet.config.annotation.InterceptorRegistry import org.springframework.web.servlet.config.annotation.WebMvcConfigurer import ru.ysu.lms.web.session.AdminInterceptor @@ -12,6 +14,18 @@ import ru.ysu.lms.web.session.MustChangePasswordInterceptor import ru.ysu.lms.web.session.SessionKeys import ru.ysu.lms.web.session.SessionUser +/** Динамические страницы не кэшируются: браузер всегда берёт свежую разметку со свежими ссылками на css/js. */ +class NoCachePageInterceptor : HandlerInterceptor { + override fun preHandle( + request: HttpServletRequest, + response: HttpServletResponse, + handler: Any, + ): Boolean { + response.setHeader("Cache-Control", "no-store") + return true + } +} + @Configuration class WebConfig( private val authInterceptor: AuthInterceptor, @@ -21,6 +35,10 @@ class WebConfig( ) : WebMvcConfigurer { override fun addInterceptors(registry: InterceptorRegistry) { registry.addInterceptor(csrfInterceptor).addPathPatterns("/**") + registry + .addInterceptor(NoCachePageInterceptor()) + .addPathPatterns("/**") + .excludePathPatterns("/css/**", "/js/**", "/favicon.svg", "/error") registry .addInterceptor(mustChangePasswordInterceptor) .addPathPatterns("/**") diff --git a/web/src/main/kotlin/ru/ysu/lms/web/learn/ContentAssetProxy.kt b/web/src/main/kotlin/ru/ysu/lms/web/learn/ContentAssetProxy.kt index de40f1a..fefd3b3 100644 --- a/web/src/main/kotlin/ru/ysu/lms/web/learn/ContentAssetProxy.kt +++ b/web/src/main/kotlin/ru/ysu/lms/web/learn/ContentAssetProxy.kt @@ -24,9 +24,15 @@ class ContentAssetProxy( val media = runCatching { MediaType.parseMediaType(asset.contentType) } .getOrElse { MediaType.APPLICATION_OCTET_STREAM } - return ResponseEntity - .ok() - .contentType(media) - .body(asset.bytes) + val builder = + ResponseEntity + .ok() + .contentType(media) + // Проксируем аккуратно: SVG при прямом открытии не должен исполнять скрипты. + if (media.isCompatibleWith(MediaType.parseMediaType("image/svg+xml"))) { + builder.header("Content-Security-Policy", "script-src 'none'; object-src 'none'") + } + builder.header("X-Content-Type-Options", "nosniff") + return builder.body(asset.bytes) } } diff --git a/web/src/main/resources/application.yml b/web/src/main/resources/application.yml index 2fc5076..18e4885 100644 --- a/web/src/main/resources/application.yml +++ b/web/src/main/resources/application.yml @@ -8,6 +8,9 @@ server: cookie: http-only: true same-site: lax + # Прод работает по HTTPS (nginx + TLS); локально на http://localhost браузеры + # тоже принимают Secure-cookie. Отключается переменной LMS_SESSION_COOKIE_SECURE=false. + secure: ${LMS_SESSION_COOKIE_SECURE:true} max-age: 30d tracking-modes: cookie diff --git a/web/src/main/resources/static/css/app.css b/web/src/main/resources/static/css/app.css index 0f0ebf9..f8a4ee5 100644 --- a/web/src/main/resources/static/css/app.css +++ b/web/src/main/resources/static/css/app.css @@ -7,44 +7,66 @@ body { flex-direction: column; background: var(--c-bg); color: var(--c-text); - font-family: system-ui, -apple-system, "Segoe UI", Roboto, sans-serif; - line-height: 1.5; + font-family: "Inter", system-ui, -apple-system, "Segoe UI", Roboto, sans-serif; + line-height: 1.55; + -webkit-font-smoothing: antialiased; } +/* Палитра Б3: нейтральный каркас + бордовый акцент. + Цвет несёт смысл: акцент — действия и фокус, зелёный/янтарный — статусы защиты, + винный — деструктивные действия и негативная тональность. */ :root { - --c-bg: #f8fafc; + --c-bg: #fafafa; --c-surface: #ffffff; - --c-text: #0f172a; - --c-text-muted: #64748b; - --c-text-link: #6366f1; - --c-text-link-hover: #4f46e5; - --c-border: #e2e8f0; - --c-accent: #6366f1; - --c-badge-ok-bg: #dcfce7; - --c-badge-ok-text: #166534; - --c-badge-warn-bg: #fef3c7; - --c-badge-warn-text: #92400e; - --c-tone-pos: #16803c; - --c-tone-neg: #b3283b; - --c-tone-neu: #64748b; + --c-text: #131316; + --c-text-muted: #74747c; + --c-text-link: #131316; + --c-text-link-hover: #7a2333; + --c-border: #e6e6e9; + --c-border-strong: #d4d4da; + --c-accent: #7a2333; + --c-accent-hover: #5f1b28; + --c-accent-soft: #f7edee; + --c-badge-ok-bg: #f2f7f3; + --c-badge-ok-text: #2f6b3a; + --c-badge-ok-border: #b7cdbd; + --c-badge-warn-bg: #faf5e9; + --c-badge-warn-text: #8a5a12; + --c-badge-warn-border: #d3bd8f; + --c-tone-pos: #2f6b3a; + --c-tone-neg: #9d2b3c; + --c-tone-neu: #74747c; + --c-btn-secondary-bg: #f1f1f3; + --c-btn-secondary-bg-hover: #e6e6ea; + --radius-xs: 0.125rem; + --radius-sm: 0.1875rem; + --radius: 0.25rem; + --radius-lg: 0.375rem; } [data-theme="dark"] { - --c-bg: #0f1115; - --c-surface: #171a21; - --c-text: #e7e9ee; - --c-text-muted: #9aa3b2; - --c-text-link: #6ea8fe; - --c-text-link-hover: #8bb9fe; - --c-border: #262b36; - --c-accent: #6ea8fe; - --c-badge-ok-bg: rgba(16, 185, 129, 0.15); - --c-badge-ok-text: #6ee7b7; - --c-badge-warn-bg: rgba(251, 191, 36, 0.15); - --c-badge-warn-text: #fcd34d; - --c-tone-pos: #4ade80; - --c-tone-neg: #f87171; - --c-tone-neu: #9aa3b2; + --c-bg: #101014; + --c-surface: #17181d; + --c-text: #e8e9ec; + --c-text-muted: #9a9ba4; + --c-text-link: #e8e9ec; + --c-text-link-hover: #d98b98; + --c-border: #26272e; + --c-border-strong: #34353d; + --c-accent: #a64d5f; + --c-accent-hover: #b86070; + --c-accent-soft: rgba(166, 77, 95, 0.16); + --c-badge-ok-bg: rgba(46, 107, 58, 0.25); + --c-badge-ok-text: #8fd4a1; + --c-badge-ok-border: rgba(143, 212, 161, 0.4); + --c-badge-warn-bg: rgba(138, 90, 18, 0.25); + --c-badge-warn-text: #e3b766; + --c-badge-warn-border: rgba(227, 183, 102, 0.4); + --c-tone-pos: #6fcf8f; + --c-tone-neg: #d98b98; + --c-tone-neu: #9a9ba4; + --c-btn-secondary-bg: #26272e; + --c-btn-secondary-bg-hover: #2f3037; } a { color: var(--c-text-link); text-decoration: none; } @@ -55,7 +77,7 @@ a:not(.btn):hover { color: var(--c-text-link-hover); } /* Layout */ .page { display: flex; flex-direction: column; min-height: 100vh; } -.page-header { position: sticky; top: 0; z-index: 10; border-bottom: 1px solid var(--c-border); background: var(--c-surface); } +.page-header { position: sticky; top: 0; z-index: 10; border-bottom: 2px solid var(--c-accent); background: var(--c-surface); } .page-header-inner { max-width: 1152px; margin: 0 auto; display: flex; align-items: center; justify-content: space-between; gap: 0.5rem 1rem; flex-wrap: wrap; padding: 0.75rem 1.5rem; } @media (max-width: 640px) { .page-header-inner { padding: 0.5rem 0.75rem; } @@ -68,10 +90,11 @@ a:not(.btn):hover { color: var(--c-text-link-hover); } /* Nav */ .nav { display: flex; align-items: center; gap: 1rem; font-size: 0.875rem; } -.nav-link { color: var(--c-text-link); text-decoration: none; } -.nav-link:hover { color: var(--c-text-link-hover); } +.nav-link { color: var(--c-text-muted); text-decoration: none; } +.nav-link:hover { color: var(--c-accent); } .nav-sep { color: var(--c-text-muted); } -.logo { font-size: 1.125rem; font-weight: 600; color: var(--c-text); text-decoration: none; } +.logo { font-size: 1.25rem; font-weight: 700; color: var(--c-accent); text-decoration: none; letter-spacing: -0.01em; } +.logo:hover { color: var(--c-accent-hover); } .nav-user { font-weight: 500; } .theme-btn { display: inline-flex; align-items: center; justify-content: center; padding: 0.3rem 0.5rem; } .theme-icon { width: 1.05rem; height: 1.05rem; display: block; } @@ -79,36 +102,39 @@ a:not(.btn):hover { color: var(--c-text-link-hover); } .breadcrumb .nav-sep { user-select: none; } .check-list { display: flex; flex-direction: column; gap: 0.25rem; margin-top: 0.25rem; max-height: 12rem; overflow-y: auto; } -/* Buttons */ +/* Buttons — по спецификации Б3: настоящая рамка вместо box-shadow, + компактный кегль 13px, бордо только на основном действии */ .btn { display: inline-flex; align-items: center; justify-content: center; gap: 0.5rem; - border-radius: 0.5rem; padding: 0.5rem 1rem; font-size: 0.875rem; font-weight: 500; - border: none; cursor: pointer; transition: opacity 0.15s; + border: 1px solid transparent; border-radius: var(--radius); + padding: 0.5rem 1rem; font-size: 0.8125rem; font-weight: 500; + cursor: pointer; text-decoration: none; + transition: background 0.15s, border-color 0.15s, color 0.15s; } -.btn:active { opacity: 0.8; } -.btn-sm { padding: 0.25rem 0.5rem; font-size: 0.75rem; } -.btn-primary { background: var(--c-accent); color: white; text-decoration: none; } -.btn-primary:hover { opacity: 0.9; } -.btn-secondary { - background: var(--c-surface); color: var(--c-text); text-decoration: none; - box-shadow: inset 0 0 0 1px var(--c-border); -} -.btn-secondary:hover { background: var(--c-border); } +.btn:active { opacity: 0.85; } +.btn:focus-visible { outline: 2px solid var(--c-accent); outline-offset: 2px; } +.btn-primary { background: var(--c-accent); border-color: var(--c-accent); color: #ffffff; } +.btn-primary:hover { background: var(--c-accent-hover); border-color: var(--c-accent-hover); } +.btn-secondary { background: var(--c-btn-secondary-bg); border-color: var(--c-btn-secondary-bg); color: var(--c-text); } +.btn-secondary:hover { background: var(--c-btn-secondary-bg-hover); border-color: var(--c-btn-secondary-bg-hover); } +/* Деструктивные действия — залиты мягким бордо, текст и рамка акцентные */ +.btn-danger { background: var(--c-accent-soft); border-color: #dcb9be; color: var(--c-accent); } +.btn-danger:hover { background: var(--c-accent); border-color: var(--c-accent); color: #ffffff; } /* Forms */ .input { - width: 100%; border-radius: 0.5rem; background: var(--c-bg); color: var(--c-text); - padding: 0.5rem 0.75rem; font-size: 0.875rem; border: 1px solid var(--c-border); + width: 100%; border-radius: var(--radius); background: var(--c-surface); color: var(--c-text); + padding: 0.5rem 0.75rem; font-size: 0.875rem; border: 1px solid var(--c-border-strong); outline: none; } -.input:focus { border-color: var(--c-accent); box-shadow: 0 0 0 2px rgba(99, 102, 241, 0.2); } +.input:focus { border-color: var(--c-accent); box-shadow: 0 0 0 3px var(--c-accent-soft); } .input::placeholder { color: var(--c-text-muted); } .label { display: block; margin-bottom: 0.25rem; font-size: 0.875rem; font-weight: 500; } .hint { margin-top: 0.25rem; font-size: 0.75rem; color: var(--c-text-muted); } /* Cards */ -.card { border-radius: 0.75rem; background: var(--c-surface); border: 1px solid var(--c-border); } -.card-flush { border-radius: 0.75rem; background: var(--c-surface); border: 1px solid var(--c-border); padding: 0; overflow: hidden; } +.card { border-radius: var(--radius-lg); background: var(--c-surface); border: 1px solid var(--c-border); } +.card-flush { border-radius: var(--radius-lg); background: var(--c-surface); border: 1px solid var(--c-border); padding: 0; overflow: hidden; } .card-body { padding: 1.5rem; } /* Grid */ @@ -153,11 +179,15 @@ a:not(.btn):hover { color: var(--c-text-link-hover); } .w-full { width: 100%; } .overflow-x-auto { overflow-x: auto; } -/* Badges */ -.badge { display: inline-flex; align-items: center; border-radius: 9999px; padding: 0.125rem 0.625rem; font-size: 0.75rem; font-weight: 500; } -.badge-ok { background: var(--c-badge-ok-bg); color: var(--c-badge-ok-text); } -.badge-warn { background: var(--c-badge-warn-bg); color: var(--c-badge-warn-text); } -.badge-btn { border: none; cursor: pointer; padding: 0.125rem 0.625rem; } +/* Badges — «этикетки»: строгая рамка, разрядка капсом; только индикаторы статуса */ +.badge { + display: inline-flex; align-items: center; + border-radius: var(--radius-sm); padding: 0.1rem 0.5rem; + font-size: 0.6875rem; font-weight: 600; letter-spacing: 0.06em; text-transform: uppercase; + border: 1px solid transparent; +} +.badge-ok { background: var(--c-badge-ok-bg); color: var(--c-badge-ok-text); border-color: var(--c-badge-ok-border); } +.badge-warn { background: var(--c-badge-warn-bg); color: var(--c-badge-warn-text); border-color: var(--c-badge-warn-border); } /* Tables */ table { width: 100%; font-size: 0.875rem; border-collapse: collapse; } @@ -168,7 +198,7 @@ td { padding: 0.75rem 1.5rem; border-bottom: 1px solid var(--c-border); } tr:last-child td { border-bottom: none; } /* Alerts */ -.alert { padding: 0.75rem 1rem; border-radius: 0.5rem; font-size: 0.875rem; margin-bottom: 1rem; } +.alert { padding: 0.75rem 1rem; border-radius: var(--radius); font-size: 0.875rem; margin-bottom: 1rem; } .alert-error { background: #fef2f2; color: #991b1b; } .alert-success { background: #dcfce7; color: #166534; } .alert-warning { background: #fef3c7; color: #92400e; } @@ -176,7 +206,7 @@ tr:last-child td { border-bottom: none; } .ml-2 { margin-left: 0.5rem; } .list-pad { padding-left: 1.25rem; } .import-card { border-color: rgba(16,185,129,0.4); } -.mini-alert { padding: 0.5rem 0.75rem; border-radius: 0.25rem; margin-bottom: 0.5rem; } +.mini-alert { padding: 0.5rem 0.75rem; border-radius: var(--radius-xs); margin-bottom: 0.5rem; } .font-semibold { font-weight: 600; } /* Theme toggle */ @@ -185,10 +215,10 @@ tr:last-child td { border-bottom: none; } /* Group assignment page */ .group-title { margin: 0 0 0.5rem; font-size: 1rem; font-weight: 600; } .inline-form { display: inline-flex; align-items: center; gap: 0.375rem; margin: 0; } -.inline-form .input-sm { width: auto; min-width: 10rem; } -.input-sm { padding: 0.25rem 0.5rem; font-size: 0.8125rem; } +/* Селект фокуса на лабу в тулбаре ведомости — по содержимому, не на всю ширину */ +#lab-focus { width: auto; min-width: 11rem; } .input-score { width: 4.5rem; } -.inline-form .input-sm { width: auto; min-width: 0; } + .inline-form .input-score, input[type="number"].input-score { width: 4.5rem; min-width: 4.5rem; flex: 0 0 auto; } .filter-select { width: auto; } .hidden { display: none !important; } @@ -214,7 +244,7 @@ tr:last-child td { border-bottom: none; } } .toast-item { pointer-events: auto; display: flex; align-items: flex-start; gap: 0.5rem; - padding: 0.7rem 0.85rem; border-radius: 0.5rem; border: 1px solid var(--c-border); + padding: 0.7rem 0.85rem; border-radius: var(--radius); border: 1px solid var(--c-border); border-left-width: 4px; background: var(--c-surface); color: var(--c-text); box-shadow: 0 10px 25px rgba(0, 0, 0, 0.15); animation: toast-in 0.18s ease-out; @@ -235,12 +265,12 @@ tr:last-child td { border-bottom: none; } /* Modals */ dialog.modal { - border: none; border-radius: 0.75rem; padding: 0; + border: 1px solid var(--c-border); border-radius: var(--radius-lg); padding: 0; background: var(--c-surface); color: var(--c-text); width: min(34rem, calc(100vw - 2rem)); - box-shadow: 0 20px 60px rgba(0,0,0,0.35); + box-shadow: 0 16px 48px rgba(19, 19, 22, 0.28); } -dialog.modal::backdrop { background: rgba(0,0,0,0.55); } +dialog.modal::backdrop { background: rgba(19, 19, 22, 0.5); } /* Пока открыта модалка — страница не прокручивается, позиция скролла сохраняется; прокрутка остаётся внутри .modal-body */ html.modal-open body { @@ -277,10 +307,13 @@ html.modal-open body { .md-body h2 { font-size: 1.125rem; } .md-body p, .md-body ul, .md-body ol { margin: 0.5rem 0; } .md-body ul, .md-body ol { padding-left: 1.5rem; } -.md-body pre { background: var(--c-bg); border: 1px solid var(--c-border); border-radius: 0.5rem; padding: 0.75rem 1rem; overflow-x: auto; font-size: 0.8125rem; } +.md-body pre { background: var(--c-bg); border: 1px solid var(--c-border); border-radius: var(--radius); padding: 0.75rem 1rem; overflow-x: auto; font-size: 0.8125rem; } .md-body code { font-family: monospace; } -.md-body details.spoiler { border: 1px solid var(--c-border); border-radius: 0.5rem; padding: 0.5rem 0.75rem; margin: 0.5rem 0; } -.md-body details.spoiler summary { cursor: pointer; font-weight: 500; color: var(--c-text-link); } +/* Ссылки в учебных материалах — акцентные, с подчёркиванием: их видно в тексте */ +.md-body a { color: var(--c-accent); text-decoration: underline; text-underline-offset: 2px; } +.md-body a:hover { color: var(--c-accent-hover); } +.md-body details.spoiler { border: 1px solid var(--c-border); border-radius: var(--radius); padding: 0.5rem 0.75rem; margin: 0.5rem 0; } +.md-body details.spoiler summary { cursor: pointer; font-weight: 500; color: var(--c-accent); } .md-body img { max-width: 100%; } .md-body table { margin: 0; } /* Таблицы в статьях прокручиваются по горизонтали, как блоки кода */ @@ -293,7 +326,7 @@ html.modal-open body { font: inherit; font-size: 0.72rem; line-height: 1; padding: 0.25rem 0.5rem; cursor: pointer; color: var(--c-text-muted); background: var(--c-surface); - border: 1px solid var(--c-border); border-radius: 0.35rem; opacity: 0.85; + border: 1px solid var(--c-border); border-radius: var(--radius-sm); opacity: 0.85; } .md-copy:hover { color: var(--c-text); opacity: 1; } .md-copy.copied { color: var(--c-text-link); } @@ -302,7 +335,7 @@ html.modal-open body { .empty { text-align: center; color: var(--c-text-muted); padding: 1.5rem; } /* Mono inline code */ -code.code { padding: 0.125rem 0.5rem; border-radius: 0.25rem; font-family: monospace; background: var(--c-bg); } +code.code { padding: 0.125rem 0.5rem; border-radius: var(--radius-xs); font-family: monospace; background: var(--c-bg); } /* Containers */ .container-sm { max-width: 28rem; margin: 0 auto; } @@ -311,10 +344,10 @@ code.code { padding: 0.125rem 0.5rem; border-radius: 0.25rem; font-family: monos /* Header logo */ .link-inherit { font-size: 0.875rem; }/* Chips (оценочные комментарии) */ -.chip { display: inline-block; margin: 2px 4px 2px 0; padding: 2px 8px; border-radius: 9999px; font-size: 11.5px; line-height: 1.5; } -.chip-pos { background: #16803c; color: #fff; } +.chip { display: inline-block; margin: 2px 4px 2px 0; padding: 2px 8px; border-radius: var(--radius-sm); font-size: 11.5px; line-height: 1.5; } +.chip-pos { background: #2e6b3a; color: #fff; } .chip-neu { background: #5d6570; color: #fff; } -.chip-neg { background: #b3283b; color: #fff; } +.chip-neg { background: #8f2d3c; color: #fff; } /* Слоты комментариев в диалоге */ .cs-row { display: flex; gap: 6px; align-items: center; } .cs-text-wrap { position: relative; flex: 1 1 auto; min-width: 0; } @@ -325,7 +358,7 @@ code.code { padding: 0.125rem 0.5rem; border-radius: 0.25rem; font-family: monos position: absolute; left: 0; right: 0; top: 100%; margin-top: 2px; z-index: 20; max-height: 14rem; overflow-y: auto; background: var(--c-surface); border: 1px solid var(--c-border); - border-radius: 0.4rem; box-shadow: 0 8px 20px rgba(0, 0, 0, 0.15); + border-radius: var(--radius-sm); box-shadow: 0 8px 20px rgba(0, 0, 0, 0.15); } .ac-list.drop-up { top: auto; bottom: 100%; margin-top: 0; margin-bottom: 2px; } .ac-item { display: flex; align-items: center; gap: 0.5rem; padding: 0.4rem 0.6rem; padding-left: 0.45rem; border-left: 3px solid transparent; cursor: pointer; font-size: 0.875rem; } @@ -339,55 +372,86 @@ code.code { padding: 0.125rem 0.5rem; border-radius: 0.25rem; font-family: monos .ac-tone-neu { color: var(--c-tone-neu); } .ac-text { flex: 1 1 auto; min-width: 0; overflow: hidden; text-overflow: ellipsis; white-space: nowrap; } .ac-count { flex: 0 0 auto; background: var(--c-border); color: var(--c-text-muted); } -.ac-del { flex: 0 0 auto; border: none; background: transparent; color: var(--c-text-muted); cursor: pointer; padding: 0 0.25rem; border-radius: 0.25rem; font-size: 0.75rem; line-height: 1.4; } +.ac-del { flex: 0 0 auto; border: none; background: transparent; color: var(--c-text-muted); cursor: pointer; padding: 0 0.25rem; border-radius: var(--radius-xs); font-size: 0.75rem; line-height: 1.4; } .ac-del:not(:disabled):hover { color: #dc2626; background: var(--c-bg); } .ac-del:disabled { opacity: 0.3; cursor: not-allowed; } .ac-empty { padding: 0.4rem 0.6rem; font-size: 0.875rem; color: var(--c-text-muted); } -/* Ячейка матрицы: вариант и кнопки действий под ним; кнопки в шапке ЛР */ +/* Ячейка матрицы в стиле Б3: вариант и оценка — текстом, подстрока — сдача и защита с точкой */ .cell-stack { display: flex; flex-direction: column; align-items: center; gap: 0.3rem; } +.cell-top { display: flex; align-items: center; justify-content: center; gap: 0.4rem; } +/* Публикация в ячейке — «глаз»-точка: ○ скрыто, ◉ опубликовано (нативный checkbox под капотом) */ +.pub-toggle { + appearance: none; -webkit-appearance: none; + width: 1rem; height: 1rem; margin: 0; padding: 0; cursor: pointer; + border-radius: 50%; border: 1.5px solid var(--c-border-strong); + background: transparent; vertical-align: middle; + transition: border-color 0.15s, box-shadow 0.15s; +} +.pub-toggle:checked { + border-color: var(--c-accent); + background: radial-gradient(circle, var(--c-accent) 0 42%, transparent 48%); +} +.pub-toggle:focus-visible { outline: 2px solid var(--c-accent); outline-offset: 2px; } .variant-btn { - display: inline-block; padding: 0.18rem 0.5rem; border-radius: 0.4rem; - border: 1px solid var(--c-border); background: var(--c-bg); - font-size: 0.8rem; white-space: nowrap; cursor: pointer; color: var(--c-text); + display: inline-block; padding: 0; border: none; background: none; + font-size: 0.8125rem; font-weight: 600; white-space: nowrap; cursor: pointer; + color: var(--c-accent); text-decoration: none; } -.variant-btn:hover { border-color: var(--c-accent); color: var(--c-accent); } -.pub-toggle { width: 1.05rem; height: 1.05rem; margin: 0; cursor: pointer; accent-color: #16a34a; } -.variant-btn.manual { border-color: var(--c-border-strong, var(--c-border)); font-weight: 600; } -.cell-btns { display: flex; gap: 0.2rem; } -.cell-btn { - display: inline-flex; align-items: center; justify-content: center; gap: 0.15rem; - min-width: 1.6rem; height: 1.6rem; padding: 0 0.35rem; cursor: pointer; - border: 1px solid var(--c-border); border-radius: 0.35rem; - background: var(--c-surface); color: var(--c-text); text-decoration: none; font-size: 0.85rem; +.variant-btn:hover { text-decoration: underline; } +.variant-btn.manual { font-weight: 700; } +.variant-btn .cell-none { color: var(--c-text-muted); font-weight: 400; } +.variant-btn.manual .cell-none { font-weight: 600; } +.score-link { + display: inline-block; padding: 0; border: none; background: none; + font-size: 0.8125rem; font-weight: 600; font-variant-numeric: tabular-nums; + color: var(--c-text); cursor: pointer; } -.cell-btn:hover { background: var(--c-bg); } -.cell-btn.dim { color: var(--c-text-muted); } -.cell-btn.warn { color: #d97706; } -.cell-btn.ok { color: #16a34a; } -.cell-num { font-size: 0.72rem; opacity: 0.75; } +.score-link:hover { color: var(--c-accent); text-decoration: underline; } +.score-link.none { color: var(--c-text-muted); font-weight: 400; } +.cell-sub { display: flex; align-items: center; justify-content: center; gap: 0.75rem; font-size: 0.75rem; } +.sub-link { color: var(--c-text-muted); text-decoration: none; font-variant-numeric: tabular-nums; } +.sub-link:hover { color: var(--c-text); text-decoration: underline; } +.sub-dim { color: var(--c-text-muted); } +.cell-dim { color: var(--c-text-muted); } +/* Статус защиты: текст с цветной точкой вместо кнопки-плашки. + Класс cell-btn сохранён — по нему JS находит ячейку (defenseBtn). */ +.cell-btn.st { + display: inline-flex; align-items: center; gap: 0.35rem; + min-width: 0; height: auto; padding: 0; + border: none; border-radius: 0; background: none; + color: var(--c-text-muted); font-size: 0.75rem; text-decoration: none; cursor: pointer; +} +.cell-btn.st:hover { background: none; color: var(--c-text); } +.cell-btn.st::before { content: ""; width: 6px; height: 6px; border-radius: 50%; background: var(--c-border-strong); } +.cell-btn.st.ok { color: var(--c-tone-pos); } +.cell-btn.st.ok::before { background: var(--c-tone-pos); } +.cell-btn.st.warn { color: #b45309; } +.cell-btn.st.warn::before { background: #b45309; } +.cell-btn.st.dim { color: var(--c-text-muted); } +.cell-btn.st.dim::before { background: var(--c-border-strong); } .mini-count { font-size: 0.72rem; margin-left: 0.15rem; color: var(--c-text-muted); } .defense-cell-timer { font-variant-numeric: tabular-nums; } .lab-head-actions { display: flex; gap: 0.2rem; } .mini-btn { display: inline-flex; align-items: center; justify-content: center; min-width: 1.5rem; height: 1.5rem; padding: 0 0.3rem; cursor: pointer; - border: 1px solid var(--c-border); border-radius: 0.35rem; + border: 1px solid var(--c-border); border-radius: var(--radius-sm); background: var(--c-surface); color: var(--c-text); text-decoration: none; font-size: 0.85rem; } .mini-btn:hover { background: var(--c-bg); } -.mini-btn.ok { color: #16a34a; } -.mini-btn.warn { color: #d97706; } -.mini-btn.partial { color: #2563eb; } +.mini-btn.ok { color: var(--c-tone-pos); } +.mini-btn.warn { color: #b45309; } +.mini-btn.partial { color: var(--c-accent); } /* Увеличенный диалог (оценка + комментарии) */ dialog.modal.modal-lg { width: min(72rem, calc(100vw - 2rem)); } dialog.modal.modal-lg .modal-body { max-height: 88vh; min-height: 62vh; } .mini-btn svg, .cell-btn svg { width: 1rem; height: 1rem; } -.chip { display: inline-block; margin: 2px 4px 2px 0; padding: 2px 8px; border-radius: 9999px; font-size: 11.5px; line-height: 1.5; } -.chip-pos { background: #16803c; color: #fff; } +.chip { display: inline-block; margin: 2px 4px 2px 0; padding: 2px 8px; border-radius: var(--radius-sm); font-size: 11.5px; line-height: 1.5; } +.chip-pos { background: #2e6b3a; color: #fff; } .chip-neu { background: #5d6570; color: #fff; } -.chip-neg { background: #b3283b; color: #fff; } +.chip-neg { background: #8f2d3c; color: #fff; } /* Defense timer */ /* Материалы: ряды и ручки перетаскивания слева, одним столбцом */ @@ -430,6 +494,10 @@ dialog.modal.modal-lg .modal-body { max-height: 88vh; min-height: 62vh; } .matrix td.cell-lab { min-width: 9.5rem; border-left: 1px solid var(--c-border); } .matrix td.student-cell { min-width: 12rem; } .lab-col-head { display: flex; flex-direction: column; align-items: center; gap: 0.375rem; } +.lab-col-title { + max-width: 9rem; overflow: hidden; text-overflow: ellipsis; white-space: nowrap; + font-size: 0.7rem; font-weight: 400; color: var(--c-text-muted); letter-spacing: 0; +} .cell-line { display: flex; align-items: center; gap: 0.375rem; font-size: 0.8125rem; text-align: left; } .cell-label { color: var(--c-text-muted); min-width: 3.5rem; } .cell-link { font-size: 0.8125rem; } @@ -455,7 +523,7 @@ dialog.modal.modal-lg .modal-body { max-height: 88vh; min-height: 62vh; } .pre-wrap { white-space: pre-wrap; word-break: break-word; } /* Редактируемая матрица: закреплённый столбец и компактные селекты */ -.matrix-sticky { position: sticky; left: 0; z-index: 2; background: var(--c-surface); } +.matrix-sticky { position: sticky; left: 0; z-index: 2; background: var(--c-surface); box-shadow: 8px 0 8px -8px rgba(19, 19, 22, 0.1); } .matrix thead .matrix-sticky { z-index: 3; } .level-form { margin: 0; } .level-select { @@ -474,7 +542,7 @@ dialog.modal.modal-lg .modal-body { max-height: 88vh; min-height: 62vh; } .cell-chip { display: inline-flex; align-items: center; gap: 0.15rem; font-size: 0.72rem; line-height: 1.7; padding: 0.05rem 0.45rem; - border: 1px solid var(--c-border); border-radius: 99px; + border: 1px solid var(--c-border); border-radius: var(--radius-sm); background: var(--c-bg); color: var(--c-text); white-space: nowrap; cursor: pointer; text-decoration: none; font-family: inherit; } @@ -482,12 +550,21 @@ dialog.modal.modal-lg .modal-body { max-height: 88vh; min-height: 62vh; } .chip-dim { color: var(--c-text-muted); } .chip-warn { color: #92400e; background: #fef3c7; border-color: #fde68a; } .chip-ok { color: #166534; background: #dcfce7; border-color: #bbf7d0; } -.chip-link:hover { color: var(--c-text-link); } +.chip-link:hover { color: var(--c-accent); } [data-theme="dark"] .chip-warn { color: #fcd34d; background: rgba(146, 64, 14, 0.25); border-color: rgba(146, 64, 14, 0.5); } [data-theme="dark"] .chip-ok { color: #86efac; background: rgba(22, 101, 52, 0.25); border-color: rgba(22, 101, 52, 0.5); } .lab-head-row { display: flex; align-items: center; justify-content: center; gap: 0.3rem; } .actions-cell { white-space: nowrap; } .actions-cell .inline-form { display: inline-block; } +/* Действия столбиком: ровные одинаковые кнопки друг под другом */ +.actions-col { display: inline-flex; flex-direction: column; gap: 0.35rem; align-items: stretch; min-width: 6.5rem; } +.actions-col .btn { text-align: center; } +/* Монохромные SVG-иконки в ячейках матрицы (защита, оценка) */ +.cell-ico { + width: 0.95rem; height: 0.95rem; display: block; + fill: none; stroke: currentColor; stroke-width: 2; + stroke-linecap: round; stroke-linejoin: round; +} /* tablekit: сортируемые таблицы и поиск */ th.th-sortable { cursor: pointer; user-select: none; } @@ -500,7 +577,7 @@ th.th-sortable.is-desc::after { content: " ▼"; color: var(--c-text-muted); fon .filter-row th { padding: 0.25rem 0.35rem; background: var(--c-bg); } .filter-control { width: 100%; min-width: 3.5rem; font-size: 0.72rem; line-height: 1.4; - padding: 0.1rem 0.25rem; border: 1px solid var(--c-border); border-radius: 0.35rem; + padding: 0.1rem 0.25rem; border: 1px solid var(--c-border); border-radius: var(--radius-sm); background: var(--c-surface); color: var(--c-text); font-family: inherit; } th.th-filterable { padding-bottom: 0.35rem; } @@ -509,14 +586,14 @@ th.th-filterable { padding-bottom: 0.35rem; } .drag-handle { display: inline-block; flex: 0 0 auto; width: 1.5rem; text-align: center; cursor: grab; user-select: none; color: var(--c-text-muted); - font-size: 1rem; line-height: 1; padding: 0.15rem 0; border-radius: 0.35rem; + font-size: 1rem; line-height: 1; padding: 0.15rem 0; border-radius: var(--radius-sm); } .drag-handle:hover { background: var(--c-bg); color: var(--c-text); } .drag-handle:active { cursor: grabbing; } .dragging { opacity: 0.45; } .drop-before { box-shadow: 0 -3px 0 0 var(--c-text); } .drop-after { box-shadow: 0 3px 0 0 var(--c-text); } -.drop-into { outline: 2px dashed var(--c-border); outline-offset: 3px; border-radius: 0.35rem; } +.drop-into { outline: 2px dashed var(--c-border); outline-offset: 3px; border-radius: var(--radius-sm); } /* индикатор сохранения порядка материалов */ .sort-status { @@ -524,7 +601,7 @@ th.th-filterable { padding-bottom: 0.35rem; } display: flex; align-items: center; gap: 0.6rem; max-width: min(24rem, calc(100vw - 2rem)); background: var(--c-surface); border: 1px solid var(--c-border); - border-radius: 0.5rem; padding: 0.5rem 0.75rem; + border-radius: var(--radius); padding: 0.5rem 0.75rem; box-shadow: 0 6px 24px rgba(0, 0, 0, 0.15); font-size: 0.85rem; } @@ -555,13 +632,13 @@ th.th-filterable { padding-bottom: 0.35rem; } .md-toolbar { display: flex; flex-wrap: wrap; align-items: center; gap: 0.1rem; padding: 0.3rem; margin-bottom: 0.5rem; - border: 1px solid var(--c-border); border-radius: 0.5rem; background: var(--c-bg); + border: 1px solid var(--c-border); border-radius: var(--radius); background: var(--c-bg); } .md-sprite { display: none; } .md-btn { display: inline-flex; align-items: center; justify-content: center; width: 2rem; height: 2rem; padding: 0; cursor: pointer; - border: 1px solid transparent; border-radius: 0.375rem; + border: 1px solid transparent; border-radius: var(--radius-sm); background: transparent; color: var(--c-text); } .md-btn:hover { background: var(--c-surface); border-color: var(--c-border); } @@ -587,4 +664,4 @@ th.th-filterable { padding-bottom: 0.35rem; } margin-top: 0.4rem; font-size: 0.875rem; color: var(--c-text-muted); } .md-video-figure { margin: 1rem 0; text-align: center; } -.md-video { width: 100%; max-width: 100%; border-radius: 0.5rem; background: #000; } +.md-video { width: 100%; max-width: 100%; border-radius: var(--radius); background: #000; } diff --git a/web/src/main/resources/static/favicon.svg b/web/src/main/resources/static/favicon.svg index 9515f38..828ab7f 100644 --- a/web/src/main/resources/static/favicon.svg +++ b/web/src/main/resources/static/favicon.svg @@ -1,7 +1,4 @@ - - - - - + + diff --git a/web/src/main/resources/static/js/admin-group.js b/web/src/main/resources/static/js/admin-group.js index 9cab501..6ff6fd3 100644 --- a/web/src/main/resources/static/js/admin-group.js +++ b/web/src/main/resources/static/js/admin-group.js @@ -57,7 +57,7 @@ function csRow(text, tone) { var rm = document.createElement('button'); rm.type = 'button'; - rm.className = 'btn btn-secondary btn-sm'; + rm.className = 'btn btn-secondary'; rm.textContent = 'удалить'; rm.onclick = function () { row.remove(); }; @@ -361,13 +361,13 @@ function applyDefenseToCell(s) { } if (label) { label.textContent = started ? '' - : (s.status == null ? 'нет' - : (s.status === 'ready' ? 'готов' - : (s.code ? 'код' : 'ожид.'))); + : (s.status == null ? '—' + : (s.status === 'ready' ? 'готова' + : (s.code ? 'код' : 'ожидает'))); } if (s.status === 'ready') { btn.title = 'Защита завершена: ' + (s.readyReason === 'timeout' ? 'время вышло' : 'студент нажал «Готов»'); - if (label) label.textContent = 'готов'; + if (label) label.textContent = 'готова'; } } @@ -521,7 +521,11 @@ function openVariant(btn) { levelSel.innerHTML = ''; VARIANT_CACHE = {}; fetch('/admin/labs/' + encodeURIComponent(lab) + '/schema') - .then(function (r) { return r.ok ? r.json() : { levels: [] }; }) + .then(function (r) { return r.ok && !r.redirected ? r.json() : { levels: [] }; }) + .catch(function () { + alert('Не удалось загрузить уровни и варианты (возможно, сессия истекла). Обновите страницу и откройте модалку заново.'); + return { levels: [] }; + }) .then(function (data) { (data.levels || []).forEach(function (lvl, i) { var opt = document.createElement('option'); @@ -607,3 +611,55 @@ window.addEventListener('DOMContentLoaded', function () { startDefenseWatch(); attachPublishToggles(); }); + +/* ---------- Большая модалка «Выдача вариантов» ---------- */ + +function refreshAssignModal(url) { + var body = document.getElementById('assign-modal-body'); + if (!body) return Promise.resolve(); + return fetch(url, { headers: { 'X-Requested-With': 'fetch' } }) + .then(function (r) { + if (r.redirected) throw new Error('сессия истекла — обновите страницу'); + return r.text(); + }) + .then(function (html) { body.innerHTML = html; }); +} + +window.openLabAssign = function (btn) { + var body = document.getElementById('assign-modal-body'); + if (!body) return; + var title = document.getElementById('assign-modal-title'); + if (title) title.textContent = 'Выдача вариантов — ' + (btn.getAttribute('data-lab-name') || ''); + body.innerHTML = '

Загрузка…

'; + openModal('modal-lab-assign'); + refreshAssignModal(btn.getAttribute('data-url')).catch(function (e) { + body.innerHTML = '

Не удалось загрузить: ' + e.message + '

'; + }); +}; + +window.confirmAssignVariant = function (sel) { + var opt = sel.selectedOptions[0]; + var taken = opt ? parseInt(opt.getAttribute('data-taken') || '0', 10) : 0; + if (opt && opt.value && taken > 0 && sel.value !== sel.dataset.prev) { + if (!confirm('Вариант уже занят (' + taken + '). Назначить всё равно?')) { + sel.value = sel.dataset.prev; + return; + } + } + sel.dataset.prev = sel.value; + sel.closest('form').requestSubmit(); +}; + +/* Формы внутри модалки сохраняются через fetch — модалка не закрывается и обновляется */ +document.addEventListener('submit', function (ev) { + var form = ev.target; + if (!form.classList || !form.classList.contains('js-assign-form')) return; + ev.preventDefault(); + var btn = form.querySelector('button[type="submit"]'); + if (btn) btn.disabled = true; + fetch(form.action, { method: 'POST', body: new FormData(form), headers: { 'X-Requested-With': 'fetch' } }) + .then(function (r) { if (r.redirected) throw new Error('сессия истекла — обновите страницу'); return r.text(); }) + .then(function () { return refreshAssignModal(form.getAttribute('data-refresh')); }) + .catch(function (e) { showToast('Не удалось сохранить: ' + e.message, 'error'); }) + .finally(function () { if (btn) btn.disabled = false; }); +}); diff --git a/web/src/main/resources/static/js/admin-lab.js b/web/src/main/resources/static/js/admin-lab.js deleted file mode 100644 index 9194263..0000000 --- a/web/src/main/resources/static/js/admin-lab.js +++ /dev/null @@ -1,36 +0,0 @@ -function toggleAllStudents(cb) { - document.querySelectorAll('.pick-student').forEach(function (c) { c.checked = cb.checked; }); -} - -function collectPicked(form) { - var picked = Array.prototype.slice.call(document.querySelectorAll('.pick-student:checked')); - if (!picked.length) { - alert('Отметьте хотя бы одного студента.'); - return false; - } - picked.forEach(function (c) { - var input = document.createElement('input'); - input.type = 'hidden'; - input.name = 'studentIds'; - input.value = c.value; - form.appendChild(input); - }); - return true; -} - -function confirmOccupied(sel) { - var opt = sel.options[sel.selectedIndex]; - var taken = opt ? parseInt(opt.getAttribute('data-taken') || '0', 10) : 0; - if (opt && opt.value && taken > 0 && sel.value !== sel.dataset.prev) { - if (!confirm('Вариант уже занят (' + taken + '). Назначить всё равно?')) { - if (sel.dataset.prev !== undefined) sel.value = sel.dataset.prev; - return; - } - } - sel.dataset.prev = sel.value; - if (sel.form) sel.form.submit(); -} - -window.addEventListener('DOMContentLoaded', function () { - document.querySelectorAll('.pick-variant').forEach(function (s) { s.dataset.prev = s.value; }); -}); diff --git a/web/src/main/resources/static/js/content-sort.js b/web/src/main/resources/static/js/content-sort.js index 24f6a3c..8803a51 100644 --- a/web/src/main/resources/static/js/content-sort.js +++ b/web/src/main/resources/static/js/content-sort.js @@ -239,6 +239,9 @@ headers: { 'Content-Type': 'application/json' }, body: JSON.stringify(collectPayload()) }).then(function (response) { + // после истечения сессии сервер отвечает редиректом на /login: fetch вернёт 200 с HTML — + // считать такое успехом нельзя, иначе порядок «сохранится» только на экране + if (response.redirected) throw new Error('сессия истекла — обновите страницу'); if (!response.ok) throw new Error('HTTP ' + response.status); pending = false; saving = false; diff --git a/web/src/main/resources/static/js/htmx.min.js b/web/src/main/resources/static/js/htmx.min.js deleted file mode 100644 index 3b7ac1a..0000000 --- a/web/src/main/resources/static/js/htmx.min.js +++ /dev/null @@ -1 +0,0 @@ -var htmx=function(){"use strict";const Q={onLoad:null,process:null,on:null,off:null,trigger:null,ajax:null,find:null,findAll:null,closest:null,values:function(e,t){const n=dn(e,t||"post");return n.values},remove:null,addClass:null,removeClass:null,toggleClass:null,takeClass:null,swap:null,defineExtension:null,removeExtension:null,logAll:null,logNone:null,logger:null,config:{historyEnabled:true,historyCacheSize:10,refreshOnHistoryMiss:false,defaultSwapStyle:"innerHTML",defaultSwapDelay:0,defaultSettleDelay:20,includeIndicatorStyles:true,indicatorClass:"htmx-indicator",requestClass:"htmx-request",addedClass:"htmx-added",settlingClass:"htmx-settling",swappingClass:"htmx-swapping",allowEval:true,allowScriptTags:true,inlineScriptNonce:"",inlineStyleNonce:"",attributesToSettle:["class","style","width","height"],withCredentials:false,timeout:0,wsReconnectDelay:"full-jitter",wsBinaryType:"blob",disableSelector:"[hx-disable], [data-hx-disable]",scrollBehavior:"instant",defaultFocusScroll:false,getCacheBusterParam:false,globalViewTransitions:false,methodsThatUseUrlParams:["get","delete"],selfRequestsOnly:true,ignoreTitle:false,scrollIntoViewOnBoost:true,triggerSpecsCache:null,disableInheritance:false,responseHandling:[{code:"204",swap:false},{code:"[23]..",swap:true},{code:"[45]..",swap:false,error:true}],allowNestedOobSwaps:true,historyRestoreAsHxRequest:true,reportValidityOfForms:false},parseInterval:null,location:location,_:null,version:"2.0.10"};Q.onLoad=j;Q.process=Ft;Q.on=ye;Q.off=xe;Q.trigger=ae;Q.ajax=Nn;Q.find=f;Q.findAll=y;Q.closest=g;Q.remove=z;Q.addClass=w;Q.removeClass=b;Q.toggleClass=G;Q.takeClass=W;Q.swap=_e;Q.defineExtension=_n;Q.removeExtension=zn;Q.logAll=$;Q.logNone=_;Q.parseInterval=d;Q._=e;const n={addTriggerHandler:St,bodyContains:se,canAccessLocalStorage:U,findThisElement:we,filterValues:yn,swap:_e,hasAttribute:s,getAttributeValue:a,getClosestAttributeValue:ne,getClosestMatch:A,getExpressionVars:Rn,getHeaders:mn,getInputValues:dn,getInternalData:oe,getSwapSpecification:bn,getTriggerSpecs:st,getTarget:Se,makeFragment:P,mergeObjects:le,makeSettleInfo:Sn,oobSwap:He,querySelectorExt:ce,settleImmediately:Yt,shouldCancel:ht,triggerEvent:ae,triggerErrorEvent:fe,withExtensions:Vt};const de=["get","post","put","delete","patch"];const R=de.map(function(e){return"[hx-"+e+"], [data-hx-"+e+"]"}).join(", ");function d(e){if(e==undefined){return undefined}let t=NaN;if(e.slice(-2)=="ms"){t=parseFloat(e.slice(0,-2))}else if(e.slice(-1)=="s"){t=parseFloat(e.slice(0,-1))*1e3}else if(e.slice(-1)=="m"){t=parseFloat(e.slice(0,-1))*1e3*60}else{t=parseFloat(e)}return isNaN(t)?undefined:t}function ee(e,t){return e instanceof Element&&e.getAttribute(t)}function s(e,t){return!!e.hasAttribute&&(e.hasAttribute(t)||e.hasAttribute("data-"+t))}function a(e,t){return ee(e,t)||ee(e,"data-"+t)}function c(e){const t=e.parentElement;if(!t&&e.parentNode instanceof ShadowRoot)return e.parentNode;return t}function te(){return document}function q(e,t){return e.getRootNode?e.getRootNode({composed:t}):te()}function A(e,t){while(e&&!t(e)){e=c(e)}return e||null}function o(e,t,n){const r=a(t,n);const o=a(t,"hx-disinherit");var i=a(t,"hx-inherit");if(e!==t){if(Q.config.disableInheritance){if(i&&(i==="*"||i.split(" ").indexOf(n)>=0)){return r}else{return null}}if(o&&(o==="*"||o.split(" ").indexOf(n)>=0)){return"unset"}}return r}function ne(t,n){let r=null;A(t,function(e){return!!(r=o(t,ue(e),n))});if(r!=="unset"){return r}}function h(e,t){return e instanceof Element&&e.matches(t)}function N(e){const t=/<([a-z][^\/\0>\x20\t\r\n\f]*)/i;const n=t.exec(e);if(n){return n[1].toLowerCase()}else{return""}}function I(e){if("parseHTMLUnsafe"in Document){return Document.parseHTMLUnsafe(e)}const t=new DOMParser;return t.parseFromString(e,"text/html")}function L(e,t){while(t.childNodes.length>0){e.append(t.childNodes[0])}}function r(e){const t=te().createElement("script");ie(e.attributes,function(e){t.setAttribute(e.name,e.value)});t.textContent=e.textContent;t.async=false;if(Q.config.inlineScriptNonce){t.nonce=Q.config.inlineScriptNonce}return t}function i(e){return e.matches("script")&&(e.type==="text/javascript"||e.type==="module"||e.type==="")}function D(e){Array.from(e.querySelectorAll("script")).forEach(e=>{if(i(e)){const t=r(e);const n=e.parentNode;try{n.insertBefore(t,e)}catch(e){H(e)}finally{e.remove()}}})}function P(e){const t=e.replace(/]*)?>[\s\S]*?<\/head>/i,"");const n=N(t);let r;if(n==="html"){r=new DocumentFragment;const i=I(e);L(r,i.body);r.title=i.title}else if(n==="body"){r=new DocumentFragment;const i=I(t);L(r,i.body);r.title=i.title}else{const i=I('");r=i.querySelector("template").content;r.title=i.title;var o=r.querySelector("title");if(o&&o.parentNode===r){o.remove();r.title=o.innerText}}if(r){if(Q.config.allowScriptTags){D(r)}else{r.querySelectorAll("script").forEach(e=>e.remove())}}return r}function re(e){if(e){e()}}function t(e,t){return Object.prototype.toString.call(e)==="[object "+t+"]"}function k(e){return typeof e==="function"}function M(e){return t(e,"Object")}function oe(e){const t="htmx-internal-data";let n=e[t];if(!n){n=e[t]={}}return n}function F(t){const n=[];if(t){for(let e=0;e=0}function se(e){return e.getRootNode({composed:true})===document}function X(e){return e.trim().split(/\s+/)}function le(e,t){for(const n in t){if(t.hasOwnProperty(n)){e[n]=t[n]}}return e}function v(e){try{return JSON.parse(e)}catch(e){H(e);return null}}function U(){const e="htmx:sessionStorageTest";try{sessionStorage.setItem(e,e);sessionStorage.removeItem(e);return true}catch(e){return false}}function V(e){try{const t=new URL(e,window.location.href);e=t.pathname+t.search}catch(e){}if(e!="/"){e=e.replace(/\/+$/,"")}return e}function e(e){return On(te().body,function(){return eval(e)})}function j(t){const e=Q.on("htmx:load",function(e){t(e.detail.elt)});return e}function $(){Q.logger=function(e,t,n){if(console){console.log(t,e,n)}}}function _(){Q.logger=null}function f(e,t){if(typeof e!=="string"){return e.querySelector(t)}else{return f(te(),e)}}function y(e,t){if(typeof e!=="string"){return e.querySelectorAll(t)}else{return y(te(),e)}}function x(){return window}function z(e,t){e=S(e);if(t){x().setTimeout(function(){z(e);e=null},t)}else{c(e).removeChild(e)}}function ue(e){return e instanceof Element?e:null}function J(e){return e instanceof HTMLElement?e:null}function K(e){return typeof e==="string"?e:null}function p(e){return e instanceof Element||e instanceof Document||e instanceof DocumentFragment?e:null}function w(e,t,n){e=ue(S(e));if(!e){return}if(n){x().setTimeout(function(){w(e,t);e=null},n)}else{e.classList&&e.classList.add(t)}}function b(e,t,n){let r=ue(S(e));if(!r){return}if(n){x().setTimeout(function(){b(r,t);r=null},n)}else{if(r.classList){r.classList.remove(t);if(r.classList.length===0){r.removeAttribute("class")}}}}function G(e,t){e=S(e);e.classList.toggle(t)}function W(e,t){e=S(e);ie(e.parentElement.children,function(e){b(e,t)});w(ue(e),t)}function g(e,t){e=ue(S(e));if(e){return e.closest(t)}return null}function l(e,t){return e.substring(0,t.length)===t}function Z(e,t){return e.substring(e.length-t.length)===t}function Y(e){const t=e.trim();if(l(t,"<")&&Z(t,"/>")){return t.substring(1,t.length-2)}else{return t}}function m(t,r,n){if(r.indexOf("global ")===0){return m(t,r.slice(7),true)}t=S(t);const o=[];{let t=0;let n=0;for(let e=0;e"){t--}}if(n0){const r=Y(o.shift());let e;if(r.indexOf("closest ")===0){e=g(ue(t),Y(r.slice(8)))}else if(r.indexOf("find ")===0){e=f(p(t),Y(r.slice(5)))}else if(r==="next"||r==="nextElementSibling"){e=ue(t).nextElementSibling}else if(r.indexOf("next ")===0){e=pe(t,Y(r.slice(5)),!!n)}else if(r==="previous"||r==="previousElementSibling"){e=ue(t).previousElementSibling}else if(r.indexOf("previous ")===0){e=ge(t,Y(r.slice(9)),!!n)}else if(r==="document"){e=document}else if(r==="window"){e=window}else if(r==="body"){e=document.body}else if(r==="root"){e=q(t,!!n)}else if(r==="host"){e=t.getRootNode().host}else{s.push(r)}if(e){i.push(e)}}if(s.length>0){const e=s.join(",");const u=p(q(t,!!n));i.push(...F(u.querySelectorAll(e)))}return i}var pe=function(t,e,n){const r=p(q(t,n)).querySelectorAll(e);for(let e=0;e=0;e--){const o=r[e];if(o.compareDocumentPosition(t)===Node.DOCUMENT_POSITION_FOLLOWING){return o}}};function ce(e,t){if(typeof e!=="string"){return m(e,t)[0]}else{return m(te().body,e)[0]}}function S(e,t){if(typeof e==="string"){return f(p(t)||document,e)}else{return e}}function me(e,t,n,r){if(k(t)){return{target:te().body,event:K(e),listener:t,options:n}}else{return{target:S(e),event:K(t),listener:n,options:r}}}function ye(t,n,r,o){Gn(function(){const e=me(t,n,r,o);e.target.addEventListener(e.event,e.listener,e.options)});const e=k(n);return e?n:r}function xe(t,n,r){Gn(function(){const e=me(t,n,r);e.target.removeEventListener(e.event,e.listener)});return k(n)?n:r}const be=te().createElement("output");function ve(t,n){const e=ne(t,n);if(e){if(e==="this"){return[we(t,n)]}else{const r=m(t,e);const o=/(^|,)(\s*)inherit(\s*)($|,)/.test(e);if(o){const i=ue(A(t,function(e){return e!==t&&s(ue(e),n)}));if(i){r.push(...ve(i,n))}}if(r.length===0){H('The selector "'+e+'" on '+n+" returned no matches!");return[be]}else{return r}}}}function we(e,t){return ue(A(e,function(e){return a(ue(e),t)!=null}))}function Se(e){const t=ne(e,"hx-target");if(t){if(t==="this"){return we(e,"hx-target")}else{return ce(e,t)}}else{const n=oe(e);if(n.boosted){return te().body}else{return e}}}function Ee(e){return Q.config.attributesToSettle.includes(e)}function Ce(t,n){ie(Array.from(t.attributes),function(e){if(!n.hasAttribute(e.name)&&Ee(e.name)){t.removeAttribute(e.name)}});ie(n.attributes,function(e){if(Ee(e.name)){t.setAttribute(e.name,e.value)}})}function Oe(t,e){const n=Jn(e);for(let e=0;e0){s=e.substring(0,e.indexOf(":"));n=e.substring(e.indexOf(":")+1)}else{s=e}o.removeAttribute("hx-swap-oob");o.removeAttribute("data-hx-swap-oob");const r=m(t,n,false);if(r.length){ie(r,function(e){let t;const n=o.cloneNode(true);t=te().createDocumentFragment();t.appendChild(n);if(!Oe(s,e)){t=p(n)}const r={shouldSwap:true,target:e,fragment:t};if(!ae(e,"htmx:oobBeforeSwap",r))return;e=r.target;if(r.shouldSwap){Re(t);je(s,e,e,t,i);Te()}ie(i.elts,function(e){ae(e,"htmx:oobAfterSwap",r)})});o.parentNode.removeChild(o)}else{o.parentNode.removeChild(o);fe(te().body,"htmx:oobErrorNoTarget",{content:o,target:n})}return e}function Te(){const e=f("#--htmx-preserve-pantry--");if(e){for(const t of[...e.children]){const n=f("#"+t.id);n.parentNode.moveBefore(t,n);n.remove()}e.remove()}}function Re(e){ie(y(e,"[hx-preserve], [data-hx-preserve]"),function(e){const t=a(e,"id");const n=te().getElementById(t);if(n!=null){if(e.moveBefore){let e=f("#--htmx-preserve-pantry--");if(e==null){te().body.insertAdjacentHTML("afterend","
");e=f("#--htmx-preserve-pantry--")}e.moveBefore(n,null)}else{e.parentNode.replaceChild(n,e)}}})}function qe(i,e,s){ie(e.querySelectorAll("[id]"),function(t){const n=ee(t,"id");if(n&&n.length>0){const e=p(i);const r=e&&e.querySelector(CSS.escape(t.tagName)+"#"+CSS.escape(n));if(r&&r!==e){const o=t.cloneNode();Ce(t,r);s.tasks.push(function(){Ce(t,o)})}}})}function Ae(e){return function(){b(e,Q.config.addedClass);Ft(ue(e));Ne(p(e));ae(e,"htmx:load")}}function Ne(e){const t="[autofocus]";const n=J(h(e,t)?e:e.querySelector(t));if(n!=null){n.focus()}}function u(e,t,n,r){qe(e,n,r);while(n.childNodes.length>0){const o=n.firstChild;w(ue(o),Q.config.addedClass);e.insertBefore(o,t);if(o.nodeType!==Node.TEXT_NODE&&o.nodeType!==Node.COMMENT_NODE){r.tasks.push(Ae(o))}}}function Ie(e,t){let n=0;while(n0}function _e(h,d,p,g){if(!g){g={}}let m=null;let n=null;let e=function(){re(g.beforeSwapCallback);h=S(h);const r=g.contextElement?q(g.contextElement,false):te();const e=document.activeElement;let t={};t={elt:e,start:e?e.selectionStart:null,end:e?e.selectionEnd:null};const o=Sn(h);if(p.swapStyle==="textContent"){h.textContent=d}else{let n=P(d);o.title=g.title||n.title;if(g.historyRequest){n=n.querySelector("[hx-history-elt],[data-hx-history-elt]")||n}if(g.selectOOB){const i=g.selectOOB.split(",");for(let t=0;t0){x().setTimeout(n,p.settleDelay)}else{n()}};let t=Q.config.globalViewTransitions;if(p.hasOwnProperty("transition")){t=p.transition}const r=g.contextElement||te();if(t&&ae(r,"htmx:beforeTransition",g.eventInfo)&&typeof Promise!=="undefined"&&document.startViewTransition){const o=new Promise(function(e,t){m=e;n=t});const i=e;e=function(){document.startViewTransition(function(){i();return o})}}try{if(p?.swapDelay&&p.swapDelay>0){x().setTimeout(e,p.swapDelay)}else{e()}}catch(e){fe(r,"htmx:swapError",g.eventInfo);re(n);throw e}}function ze(e,t,n){const r=e.getResponseHeader(t);if(r.indexOf("{")===0){const o=v(r);for(const i in o){if(o.hasOwnProperty(i)){let e=o[i];if(M(e)){n=e.target!==undefined?e.target:n}else{e={value:e}}ae(n,i,e)}}}else{const s=r.split(",");for(let e=0;e0){const s=o[0];if(s==="]"){e--;if(e===0){if(n===null){t=t+"true"}o.shift();t+=")})";try{const l=On(r,function(){return Function(t)()},function(){return true});l.source=t;return l}catch(e){fe(te().body,"htmx:syntax:error",{error:e,source:t});return null}}}else if(s==="["){e++}if(tt(s,n,i)){t+="(("+i+"."+s+") ? ("+i+"."+s+") : (window."+s+"))"}else{t=t+s}n=o.shift()}}}function O(e,t){let n="";while(e.length>0&&!t.test(e[0])){n+=e.shift()}return n}function rt(e){let t;if(e.length>0&&Ye.test(e[0])){e.shift();t=O(e,Qe).trim();e.shift()}else{t=O(e,C)}return t}const ot="input, textarea, select";function it(e,t,n){const r=[];const o=et(t);do{O(o,Ze);const l=o.length;const u=O(o,/[,\[\s]/);if(u!==""){if(u==="every"){const c={trigger:"every"};O(o,Ze);c.pollInterval=d(O(o,/[,\[\s]/));O(o,Ze);var i=nt(e,o,"event");if(i){c.eventFilter=i}r.push(c)}else{const f={trigger:u};var i=nt(e,o,"event");if(i){f.eventFilter=i}O(o,Ze);while(o.length>0&&o[0]!==","){const a=o.shift();if(a==="changed"){f.changed=true}else if(a==="once"){f.once=true}else if(a==="consume"){f.consume=true}else if(a==="delay"&&o[0]===":"){o.shift();f.delay=d(O(o,C))}else if(a==="from"&&o[0]===":"){o.shift();if(Ye.test(o[0])){var s=rt(o)}else{var s=O(o,C);if(s==="closest"||s==="find"||s==="next"||s==="previous"){o.shift();const h=rt(o);if(h.length>0){s+=" "+h}}}f.from=s}else if(a==="target"&&o[0]===":"){o.shift();f.target=rt(o)}else if(a==="throttle"&&o[0]===":"){o.shift();f.throttle=d(O(o,C))}else if(a==="queue"&&o[0]===":"){o.shift();f.queue=O(o,C)}else if(a==="root"&&o[0]===":"){o.shift();f[a]=rt(o)}else if(a==="threshold"&&o[0]===":"){o.shift();f[a]=O(o,C)}else{fe(e,"htmx:syntax:error",{token:o.shift()})}O(o,Ze)}r.push(f)}}if(o.length===l){fe(e,"htmx:syntax:error",{token:o.shift()})}O(o,Ze)}while(o[0]===","&&o.shift());if(n){n[t]=r}return r}function st(e){const t=a(e,"hx-trigger");let n=[];if(t){const r=Q.config.triggerSpecsCache;n=r&&r[t]||it(e,t,r)}if(n.length>0){return n}else if(h(e,"form")){return[{trigger:"submit"}]}else if(h(e,'input[type="button"], input[type="submit"]')){return[{trigger:"click"}]}else if(h(e,ot)){return[{trigger:"change"}]}else{return[{trigger:"click"}]}}function lt(e){oe(e).cancelled=true}function ut(e,t,n){const r=oe(e);r.timeout=x().setTimeout(function(){if(se(e)&&r.cancelled!==true){if(!pt(n,e,Xt("hx:poll:trigger",{triggerSpec:n,target:e}))){t(e)}ut(e,t,n)}},n.pollInterval)}function ct(e){return location.hostname===e.hostname&&ee(e,"href")&&ee(e,"href").indexOf("#")!==0}function ft(e){return g(e,Q.config.disableSelector)}function at(t,n,e){if(t instanceof HTMLAnchorElement&&ct(t)&&(t.target===""||t.target==="_self")||t.tagName==="FORM"&&String(ee(t,"method")).toLowerCase()!=="dialog"){n.boosted=true;let r,o;if(t.tagName==="A"){r="get";o=ee(t,"href")}else{const i=ee(t,"method");r=i?i.toLowerCase():"get";o=ee(t,"action");if(o==null||o===""){o=location.href}if(r==="get"&&o.includes("?")){o=o.replace(/\?[^#]+/,"")}}e.forEach(function(e){gt(t,function(e,t){const n=ue(e);if(ft(n)){E(n);return}he(r,o,n,t)},n,e,true)})}}function ht(e,t){if(e.type==="submit"&&t.tagName==="FORM"){return true}else if(e.type==="click"){const n=t.closest('input[type="submit"], button');if(n&&n.form&&n.type==="submit"){return true}const r=t.closest("a");const o=/^#.+/;if(r&&r.href&&!o.test(r.getAttribute("href"))){return true}}return false}function dt(e,t){return oe(e).boosted&&e instanceof HTMLAnchorElement&&t.type==="click"&&(t.ctrlKey||t.metaKey)}function pt(e,t,n){const r=e.eventFilter;if(r){try{return r.call(t,n)!==true}catch(e){const o=r.source;fe(te().body,"htmx:eventFilter:error",{error:e,source:o});return true}}return false}function gt(l,u,e,c,f){const a=oe(l);let t;if(c.from){t=m(l,c.from)}else{t=[l]}if(c.changed){if(!("lastValue"in a)){a.lastValue=new WeakMap}t.forEach(function(e){if(!a.lastValue.has(c)){a.lastValue.set(c,new WeakMap)}a.lastValue.get(c).set(e,e.value)})}ie(t,function(i){const s=function(e){if(!se(l)){i.removeEventListener(c.trigger,s);return}if(dt(l,e)){return}if(f||ht(e,i)){e.preventDefault()}if(pt(c,l,e)){return}const t=oe(e);t.triggerSpec=c;if(t.handledFor==null){t.handledFor=[]}if(t.handledFor.indexOf(l)<0){t.handledFor.push(l);if(c.consume){e.stopPropagation()}if(c.target&&e.target){if(!h(ue(e.target),c.target)){return}}if(c.once){if(a.triggeredOnce){return}else{a.triggeredOnce=true}}if(c.changed){const n=e.target;const r=n.value;const o=a.lastValue.get(c);if(o.has(n)&&o.get(n)===r){return}o.set(n,r)}if(a.delayed){clearTimeout(a.delayed)}if(a.throttle){return}if(c.throttle>0){if(!a.throttle){ae(l,"htmx:trigger");u(l,e);a.throttle=x().setTimeout(function(){a.throttle=null},c.throttle)}}else if(c.delay>0){a.delayed=x().setTimeout(function(){ae(l,"htmx:trigger");u(l,e)},c.delay)}else{ae(l,"htmx:trigger");u(l,e)}}};if(e.listenerInfos==null){e.listenerInfos=[]}e.listenerInfos.push({trigger:c.trigger,listener:s,on:i});i.addEventListener(c.trigger,s)})}let mt=false;let yt=null;function xt(){if(!yt){yt=function(){mt=true};window.addEventListener("scroll",yt);window.addEventListener("resize",yt);setInterval(function(){if(mt){mt=false;ie(te().querySelectorAll("[hx-trigger*='revealed'],[data-hx-trigger*='revealed']"),function(e){bt(e)})}},200)}}function bt(e){if(!s(e,"data-hx-revealed")&&B(e)){e.setAttribute("data-hx-revealed","true");const t=oe(e);if(t.initHash){ae(e,"revealed")}else{e.addEventListener("htmx:afterProcessNode",function(){ae(e,"revealed")},{once:true})}}}function vt(e,t,n,r){const o=function(){if(!n.loaded){n.loaded=true;ae(e,"htmx:trigger");t(e)}};if(r>0){x().setTimeout(o,r)}else{o()}}function wt(t,n,e){let i=false;ie(de,function(r){if(s(t,"hx-"+r)){const o=a(t,"hx-"+r);i=true;n.path=o;n.verb=r;e.forEach(function(e){St(t,e,n,function(e,t){const n=ue(e);if(ft(n)){E(n);return}he(r,o,n,t)})})}});return i}function St(r,e,t,n){if(e.trigger==="revealed"){xt();gt(r,n,t,e);bt(ue(r))}else if(e.trigger==="intersect"){const o={};if(e.root){o.root=ce(r,e.root)}if(e.threshold){o.threshold=parseFloat(e.threshold)}const i=new IntersectionObserver(function(t){for(let e=0;e0){t.polling=true;ut(ue(r),n,e)}else{gt(r,n,t,e)}}function Et(e){const t=ue(e);if(!t){return false}const n=t.attributes;for(let e=0;e", "+e).join(""));return o}else{return[]}}function Rt(e){const t=At(e.target);const n=It(e);if(n){n.lastButtonClicked=t}}function qt(e){const t=It(e);if(t){t.lastButtonClicked=null}}function At(e){return g(ue(e),"button, input[type='submit']")}function Nt(e){return e.form||g(e,"form")}function It(e){const t=At(e.target);if(!t){return}const n=Nt(t);if(!n){return}return oe(n)}function Lt(e){e.addEventListener("click",Rt);e.addEventListener("focusin",Rt);e.addEventListener("focusout",qt)}function Dt(t,e,n){const r=oe(t);if(!Array.isArray(r.onHandlers)){r.onHandlers=[]}let o;const i=function(e){On(t,function(){if(ft(t)){return}if(!o){o=new Function("event",n)}o.call(t,e)})};t.addEventListener(e,i);r.onHandlers.push({event:e,listener:i})}function Pt(t){De(t);for(let e=0;eQ.config.historyCacheSize){i.shift()}while(i.length>0){try{sessionStorage.setItem("htmx-history-cache",JSON.stringify(i));break}catch(e){fe(te().body,"htmx:historyCacheError",{cause:e,cache:i});i.shift()}}}function Jt(t){if(!U()){return null}t=V(t);const n=v(sessionStorage.getItem("htmx-history-cache"))||[];for(let e=0;e=200&&this.status<400){r.response=this.response;ae(te().body,"htmx:historyCacheMissLoad",r);_e(r.historyElt,r.response,n,{contextElement:r.historyElt,historyRequest:true});$t(r.path);ae(te().body,"htmx:historyRestore",{path:e,cacheMiss:true,serverResponse:r.response})}else{fe(te().body,"htmx:historyCacheMissLoadError",r)}};if(ae(te().body,"htmx:historyCacheMiss",r)){t.send()}}function en(e){Gt();e=e||location.pathname+location.search;const t=Jt(e);if(t){const n={swapStyle:"innerHTML",swapDelay:0,settleDelay:0,scroll:t.scroll};const r={path:e,item:t,historyElt:_t(),swapSpec:n};if(ae(te().body,"htmx:historyCacheHit",r)){_e(r.historyElt,t.content,n,{contextElement:r.historyElt,title:t.title});$t(r.path);ae(te().body,"htmx:historyRestore",r)}}else{if(Q.config.refreshOnHistoryMiss){Q.location.reload(true)}else{Qt(e)}}}function tn(e){let t=ve(e,"hx-indicator");if(t==null){t=[e]}ie(t,function(e){const t=oe(e);t.requestCount=(t.requestCount||0)+1;w(e,Q.config.requestClass)});return t}function nn(e){let t=ve(e,"hx-disabled-elt");if(t==null){t=[]}ie(t,function(e){const t=oe(e);t.requestCount=(t.requestCount||0)+1;if(!e.hasAttribute("disabled")){e.setAttribute("disabled","");e.setAttribute("data-disabled-by-htmx","")}});return t}function rn(e,t){ie(e.concat(t),function(e){const t=oe(e);t.requestCount=(t.requestCount||1)-1});ie(e,function(e){const t=oe(e);if(t.requestCount===0){b(e,Q.config.requestClass)}});ie(t,function(e){const t=oe(e);if(t.requestCount===0&&e.hasAttribute("data-disabled-by-htmx")){e.removeAttribute("disabled");e.removeAttribute("data-disabled-by-htmx")}})}function on(t,n){for(let e=0;en.indexOf(e)<0)}else{e=e.filter(e=>e!==n)}r.delete(t);ie(e,e=>r.append(t,e))}}function cn(e){if(e instanceof HTMLSelectElement&&e.multiple){return F(e.querySelectorAll("option:checked")).map(function(e){return e.value})}if(e instanceof HTMLInputElement&&e.files){return F(e.files)}return e.value}function fn(t,n,r,e,o){if(e==null||on(t,e)){return}else{t.push(e)}if(sn(e)){const i=ee(e,"name");ln(i,cn(e),n);if(o){an(e,r)}}if(e instanceof HTMLFormElement){ie(e.elements,function(e){if(t.indexOf(e)>=0){un(e.name,cn(e),n)}else{t.push(e)}if(o){an(e,r)}});new FormData(e).forEach(function(e,t){if(e instanceof File&&e.name===""){return}ln(t,e,n)})}}function an(e,t){const n=e;if(n.willValidate){ae(n,"htmx:validation:validate");if(!n.checkValidity()){if(ae(n,"htmx:validation:failed",{message:n.validationMessage,validity:n.validity})&&!t.length&&Q.config.reportValidityOfForms){n.reportValidity()}t.push({elt:n,message:n.validationMessage,validity:n.validity})}}}function hn(n,e){for(const t of e.keys()){n.delete(t)}e.forEach(function(e,t){n.append(t,e)});return n}function dn(e,t){const n=[];const r=new FormData;const o=new FormData;const i=[];const s=oe(e);if(s.lastButtonClicked&&!se(s.lastButtonClicked)){s.lastButtonClicked=null}let l=e instanceof HTMLFormElement&&e.noValidate!==true||a(e,"hx-validate")==="true";if(s.lastButtonClicked){l=l&&s.lastButtonClicked.formNoValidate!==true}if(t!=="get"){fn(n,o,i,Nt(e),l)}fn(n,r,i,e,l);if(s.lastButtonClicked||e.tagName==="BUTTON"||e.tagName==="INPUT"&&ee(e,"type")==="submit"){const c=s.lastButtonClicked||e;const f=ee(c,"name");ln(f,c.value,o)}const u=ve(e,"hx-include");ie(u,function(e){fn(n,r,i,ue(e),l);if(!h(e,"form")){ie(p(e).querySelectorAll(ot),function(e){fn(n,r,i,e,l)})}});hn(r,o);return{errors:i,formData:r,values:kn(r)}}function pn(e,t,n){if(e!==""){e+="&"}if(String(n)==="[object Object]"){n=JSON.stringify(n)}const r=encodeURIComponent(n);e+=encodeURIComponent(t)+"="+r;return e}function gn(e){e=Dn(e);let n="";e.forEach(function(e,t){n=pn(n,t,e)});return n}function mn(e,t,n){const r={"HX-Request":"true","HX-Trigger":ee(e,"id"),"HX-Trigger-Name":ee(e,"name"),"HX-Target":a(t,"id"),"HX-Current-URL":location.href};Cn(e,"hx-headers",false,r);if(n!==undefined){r["HX-Prompt"]=n}if(oe(e).boosted){r["HX-Boosted"]="true"}return r}function yn(n,e){const t=ne(e,"hx-params");if(t){if(t==="none"){return new FormData}else if(t==="*"){return n}else if(t.indexOf("not ")===0){ie(t.slice(4).split(","),function(e){e=e.trim();n.delete(e)});return n}else{const r=new FormData;ie(t.split(","),function(t){t=t.trim();if(n.has(t)){n.getAll(t).forEach(function(e){r.append(t,e)})}});return r}}else{return n}}function xn(e){return!!ee(e,"href")&&ee(e,"href").indexOf("#")>=0}function bn(e,t){const n=t||ne(e,"hx-swap");const r={swapStyle:oe(e).boosted?"innerHTML":Q.config.defaultSwapStyle,swapDelay:Q.config.defaultSwapDelay,settleDelay:Q.config.defaultSettleDelay};if(Q.config.scrollIntoViewOnBoost&&oe(e).boosted&&!xn(e)){r.show="top"}if(n){const s=X(n);if(s.length>0){for(let e=0;e0?o.join(":"):null;r.scroll=c;r.scrollTarget=i}else if(l.indexOf("show:")===0){const f=l.slice(5);var o=f.split(":");const a=o.pop();var i=o.length>0?o.join(":"):null;r.show=a;r.showTarget=i}else if(l.indexOf("focus-scroll:")===0){const h=l.slice("focus-scroll:".length);r.focusScroll=h=="true"}else if(e==0){r.swapStyle=l}else{H("Unknown modifier in hx-swap: "+l)}}}}return r}function vn(e){return ne(e,"hx-encoding")==="multipart/form-data"||h(e,"form")&&ee(e,"enctype")==="multipart/form-data"}function wn(t,n,r){let o=null;Vt(n,function(e){if(o==null){o=e.encodeParameters(t,r,n)}});if(o!=null){return o}else{if(vn(n)){return hn(new FormData,Dn(r))}else{return gn(r)}}}function Sn(e){return{tasks:[],elts:[e]}}function En(e,t){const n=e[0];const r=e[e.length-1];if(t.scroll){var o=null;if(t.scrollTarget){o=ue(ce(n,t.scrollTarget))}if(t.scroll==="top"&&(n||o)){o=o||n;o.scrollTop=0}if(t.scroll==="bottom"&&(r||o)){o=o||r;o.scrollTop=o.scrollHeight}if(typeof t.scroll==="number"){x().setTimeout(function(){window.scrollTo(0,t.scroll)},0)}}if(t.show){var o=null;if(t.showTarget){let e=t.showTarget;if(t.showTarget==="window"){e="body"}o=ue(ce(n,e))}if(t.show==="top"&&(n||o)){o=o||n;o.scrollIntoView({block:"start",behavior:Q.config.scrollBehavior})}if(t.show==="bottom"&&(r||o)){o=o||r;o.scrollIntoView({block:"end",behavior:Q.config.scrollBehavior})}}}function Cn(r,e,o,i,s){if(i==null){i={}}if(r==null){return i}const l=a(r,e);if(l){let e=l.trim();let t=o;if(e==="unset"){return null}if(e.indexOf("javascript:")===0){e=e.slice(11);t=true}else if(e.indexOf("js:")===0){e=e.slice(3);t=true}if(e.indexOf("{")!==0){e="{"+e+"}"}let n;if(t){n=On(r,function(){if(s){return Function("event","return ("+e+")").call(r,s)}else{return Function("return ("+e+")").call(r)}},{})}else{n=v(e)}for(const u in n){if(n.hasOwnProperty(u)){if(i[u]==null){i[u]=n[u]}}}}return Cn(ue(c(r)),e,o,i,s)}function On(e,t,n){if(Q.config.allowEval){return t()}else{fe(e,"htmx:evalDisallowedError");return n}}function Hn(e,t,n){return Cn(e,"hx-vars",true,n,t)}function Tn(e,t,n){return Cn(e,"hx-vals",false,n,t)}function Rn(e,t){return le(Hn(e,t),Tn(e,t))}function qn(t,n,r){if(r!==null){try{t.setRequestHeader(n,r)}catch(e){t.setRequestHeader(n,encodeURIComponent(r));t.setRequestHeader(n+"-URI-AutoEncoded","true")}}}function An(t){if(t.responseURL){try{const e=new URL(t.responseURL);return e.pathname+e.search}catch(e){fe(te().body,"htmx:badResponseUrl",{url:t.responseURL})}}}function T(e,t){return t.test(e.getAllResponseHeaders())}function Nn(t,n,r){t=t.toLowerCase();if(r){if(r instanceof Element||typeof r==="string"){return he(t,n,null,null,{targetOverride:S(r)||be,returnPromise:true})}else{let e=S(r.target);if(r.target&&!e||r.source&&!e&&!S(r.source)){e=be}return he(t,n,S(r.source),r.event,{handler:r.handler,headers:r.headers,values:r.values,targetOverride:e,swapOverride:r.swap,select:r.select,returnPromise:true,push:r.push,replace:r.replace,selectOOB:r.selectOOB})}}else{return he(t,n,null,null,{returnPromise:true})}}function In(e){const t=[];while(e){t.push(e);e=e.parentElement}return t}function Ln(e,t,n){const r=new URL(t,location.protocol!=="about:"?location.href:window.origin);const o=location.protocol!=="about:"?location.origin:window.origin;const i=o===r.origin;if(Q.config.selfRequestsOnly){if(!i){return false}}return ae(e,"htmx:validateUrl",le({url:r,sameHost:i},n))}function Dn(e){if(e instanceof FormData)return e;const t=new FormData;for(const n in e){if(e.hasOwnProperty(n)){if(e[n]&&typeof e[n].forEach==="function"){e[n].forEach(function(e){t.append(n,e)})}else if(typeof e[n]==="object"&&!(e[n]instanceof Blob)){t.append(n,JSON.stringify(e[n]))}else{t.append(n,e[n])}}}return t}function Pn(r,o,e){return new Proxy(e,{get:function(t,e){if(typeof e==="number")return t[e];if(e==="length")return t.length;if(e==="push"){return function(e){t.push(e);r.append(o,e)}}if(typeof t[e]==="function"){return function(){t[e].apply(t,arguments);r.delete(o);t.forEach(function(e){r.append(o,e)})}}if(t[e]&&t[e].length===1){return t[e][0]}else{return t[e]}},set:function(e,t,n){e[t]=n;r.delete(o);e.forEach(function(e){r.append(o,e)});return true}})}function kn(o){return new Proxy(o,{get:function(e,t){if(typeof t==="symbol"){const r=Reflect.get(e,t);if(typeof r==="function"){return function(){return r.apply(o,arguments)}}else{return r}}if(t==="toJSON"){return()=>Object.fromEntries(o)}if(t in e){if(typeof e[t]==="function"){return function(){return o[t].apply(o,arguments)}}}const n=o.getAll(t);if(n.length===0){return undefined}else if(n.length===1){return n[0]}else{return Pn(e,t,n)}},set:function(t,n,e){if(typeof n!=="string"){return false}t.delete(n);if(e&&typeof e.forEach==="function"){e.forEach(function(e){t.append(n,e)})}else if(typeof e==="object"&&!(e instanceof Blob)){t.append(n,JSON.stringify(e))}else{t.append(n,e)}return true},deleteProperty:function(e,t){if(typeof t==="string"){e.delete(t)}return true},ownKeys:function(e){return Reflect.ownKeys(Object.fromEntries(e))},getOwnPropertyDescriptor:function(e,t){return Reflect.getOwnPropertyDescriptor(Object.fromEntries(e),t)}})}function he(t,n,r,o,i,k){let s=null;let l=null;i=i!=null?i:{};if(i.returnPromise&&typeof Promise!=="undefined"){var e=new Promise(function(e,t){s=e;l=t})}if(r==null){r=te().body}const M=i.handler||Vn;const F=i.select||null;if(!se(r)){re(s);return e}const u=i.targetOverride||ue(Se(r));if(u==null||u==be){fe(r,"htmx:targetError",{target:ne(r,"hx-target")});re(l);return e}let c=oe(r);const f=c.lastButtonClicked;if(f){const A=ee(f,"formaction");if(A!=null){n=A}const N=ee(f,"formmethod");if(N!=null){if(de.includes(N.toLowerCase())){t=N}else{re(s);return e}}}const a=ne(r,"hx-confirm");if(k===undefined){const K=function(e){return he(t,n,r,o,i,!!e)};const G={target:u,elt:r,path:n,verb:t,triggeringEvent:o,etc:i,issueRequest:K,question:a};if(ae(r,"htmx:confirm",G)===false){re(s);return e}}let h=r;let d=ne(r,"hx-sync");let p=null;let B=false;if(d){const I=d.split(":");const L=I[0].trim();if(L==="this"){h=we(r,"hx-sync")}else{h=ue(ce(r,L))}d=(I[1]||"drop").trim();c=oe(h);if(d==="drop"&&c.xhr&&c.abortable!==true){re(s);return e}else if(d==="abort"){if(c.xhr){re(s);return e}else{B=true}}else if(d==="replace"){ae(h,"htmx:abort")}else if(d.indexOf("queue")===0){const W=d.split(" ");p=(W[1]||"last").trim()}}if(c.xhr){if(c.abortable){ae(h,"htmx:abort")}else{if(p==null){if(o){const D=oe(o);if(D&&D.triggerSpec&&D.triggerSpec.queue){p=D.triggerSpec.queue}}if(p==null){p="last"}}if(c.queuedRequests==null){c.queuedRequests=[]}if(p==="first"&&c.queuedRequests.length===0){c.queuedRequests.push(function(){he(t,n,r,o,i)})}else if(p==="all"){c.queuedRequests.push(function(){he(t,n,r,o,i)})}else if(p==="last"){c.queuedRequests=[];c.queuedRequests.push(function(){he(t,n,r,o,i)})}re(s);return e}}const g=new XMLHttpRequest;c.xhr=g;c.abortable=B;const m=function(){c.xhr=null;c.abortable=false;if(c.queuedRequests!=null&&c.queuedRequests.length>0){const e=c.queuedRequests.shift();e()}};const X=ne(r,"hx-prompt");if(X){var y=prompt(X);if(y===null||!ae(r,"htmx:prompt",{prompt:y,target:u})){re(s);m();return e}}if(a&&!k){if(!confirm(a)){re(s);m();return e}}let x=mn(r,u,y);if(t!=="get"&&!vn(r)){x["Content-Type"]="application/x-www-form-urlencoded"}if(i.headers){x=le(x,i.headers)}const U=dn(r,t);let b=U.errors;const V=U.formData;if(i.values){hn(V,Dn(i.values))}const j=Dn(Rn(r,o));const v=hn(V,j);let w=yn(v,r);if(Q.config.getCacheBusterParam&&t==="get"){w.set("org.htmx.cache-buster",ee(u,"id")||"true")}if(n==null||n===""){n=location.href}const S=Cn(r,"hx-request");const $=oe(r).boosted;let E=Q.config.methodsThatUseUrlParams.indexOf(t)>=0;const C={boosted:$,useUrlParams:E,formData:w,parameters:kn(w),unfilteredFormData:v,unfilteredParameters:kn(v),headers:x,elt:r,target:u,verb:t,errors:b,withCredentials:i.credentials||S.credentials||Q.config.withCredentials,timeout:i.timeout||S.timeout||Q.config.timeout,path:n,triggeringEvent:o};if(!ae(r,"htmx:configRequest",C)){re(s);m();return e}n=C.path;t=C.verb;x=C.headers;w=Dn(C.parameters);b=C.errors;E=C.useUrlParams;if(b&&b.length>0){ae(r,"htmx:validation:halted",C);re(s);m();return e}const _=n.split("#");const z=_[0];const O=_[1];let H=n;if(E){H=z;const Z=!w.keys().next().done;if(Z){if(H.indexOf("?")<0){H+="?"}else{H+="&"}H+=gn(w);if(O){H+="#"+O}}}if(!Ln(r,H,C)){fe(r,"htmx:invalidPath",C);re(l);m();return e}g.open(t.toUpperCase(),H,true);g.overrideMimeType("text/html");g.withCredentials=C.withCredentials;g.timeout=C.timeout;if(S.noHeaders){}else{for(const P in x){if(x.hasOwnProperty(P)){const Y=x[P];qn(g,P,Y)}}}const T={xhr:g,target:u,requestConfig:C,etc:i,boosted:$,select:F,pathInfo:{requestPath:n,finalRequestPath:H,responsePath:null,anchor:O}};g.onload=function(){try{const t=In(r);T.pathInfo.responsePath=An(g);M(r,T);if(T.keepIndicators!==true){rn(R,q)}ae(r,"htmx:afterRequest",T);ae(r,"htmx:afterOnLoad",T);if(!se(r)){let e=null;while(t.length>0&&e==null){const n=t.shift();if(se(n)){e=n}}if(e){ae(e,"htmx:afterRequest",T);ae(e,"htmx:afterOnLoad",T)}}re(s)}catch(e){fe(r,"htmx:onLoadError",le({error:e},T));throw e}finally{m()}};g.onerror=function(){rn(R,q);fe(r,"htmx:afterRequest",T);fe(r,"htmx:sendError",T);re(l);m()};g.onabort=function(){rn(R,q);fe(r,"htmx:afterRequest",T);fe(r,"htmx:sendAbort",T);re(l);m()};g.ontimeout=function(){rn(R,q);fe(r,"htmx:afterRequest",T);fe(r,"htmx:timeout",T);re(l);m()};if(!ae(r,"htmx:beforeRequest",T)){re(s);m();return e}var R=tn(r);var q=nn(r);ie(["loadstart","loadend","progress","abort"],function(t){ie([g,g.upload],function(e){e.addEventListener(t,function(e){ae(r,"htmx:xhr:"+t,{lengthComputable:e.lengthComputable,loaded:e.loaded,total:e.total})})})});ae(r,"htmx:beforeSend",T);const J=E?null:wn(g,r,w);g.send(J);return e}function Mn(e,t){const n=t.xhr;let r=null;let o=null;if(T(n,/HX-Push:/i)){r=n.getResponseHeader("HX-Push");o="push"}else if(T(n,/HX-Push-Url:/i)){r=n.getResponseHeader("HX-Push-Url");o="push"}else if(T(n,/HX-Replace-Url:/i)){r=n.getResponseHeader("HX-Replace-Url");o="replace"}if(r){if(r==="false"){return{}}else{return{type:o,path:r}}}const i=t.pathInfo.finalRequestPath;const s=t.pathInfo.responsePath;let l=t.etc.push||ne(e,"hx-push-url");let u=t.etc.replace||ne(e,"hx-replace-url");if(l==="false")l=null;if(u==="false")u=null;const c=oe(e).boosted;let f=null;let a=null;if(l){f="push";a=l}else if(u){f="replace";a=u}else if(c){f="push";a=s||i}if(a){if(a==="true"){a=s||i}if(t.pathInfo.anchor&&a.indexOf("#")===-1){a=a+"#"+t.pathInfo.anchor}return{type:f,path:a}}else{return{}}}function Fn(e,t){var n=new RegExp(e.code);return n.test(t.toString(10))}function Bn(e){for(var t=0;t`+`.${t}{opacity:0;visibility: hidden} `+`.${n} .${t}, .${n}.${t}{opacity:1;visibility: visible;transition: opacity 200ms ease-in}`+"")}}function Zn(){const e=te().querySelector('meta[name="htmx-config"]');if(e){return v(e.content)}else{return null}}function Yn(){const e=Zn();if(e){Q.config=le(Q.config,e)}}Gn(function(){Yn();Wn();let e=te().body;Ft(e);const t=te().querySelectorAll("[hx-trigger='restored'],[data-hx-trigger='restored']");e.addEventListener("htmx:abort",function(e){const t=e.detail.elt||e.target;const n=oe(t);if(n&&n.xhr){n.xhr.abort()}});const n=window.onpopstate?window.onpopstate.bind(window):null;window.onpopstate=function(e){if(e.state&&e.state.htmx){en();ie(t,function(e){ae(e,"htmx:restored",{document:te(),triggerEvent:ae})})}else{if(n){n(e)}}};x().setTimeout(function(){ae(e,"htmx:load",{});e=null},0)});return Q}(); \ No newline at end of file diff --git a/web/src/main/resources/static/js/mdeditor.js b/web/src/main/resources/static/js/mdeditor.js index 9d08eb9..31065f1 100644 --- a/web/src/main/resources/static/js/mdeditor.js +++ b/web/src/main/resources/static/js/mdeditor.js @@ -113,7 +113,8 @@ }; document.addEventListener('input', function (event) { - if (event.target && event.target.id === 'article-body') dirty = true; + var id = event.target && event.target.id; + if (id === 'article-body' || id === 'article-title') dirty = true; }); /* Сохранение — это осознанный уход со страницы, подтверждение не нужно */ diff --git a/web/src/main/resources/static/js/submit-guard.js b/web/src/main/resources/static/js/submit-guard.js new file mode 100644 index 0000000..f4435cb --- /dev/null +++ b/web/src/main/resources/static/js/submit-guard.js @@ -0,0 +1,32 @@ +/* Защита от двойной отправки: после сабмита формы кнопки блокируются до перехода. + Инлайн-onsubmit с confirm() успевает отработать раньше (обработчик из атрибута + регистрируется первым) — если сабмит отменён, кнопки не блокируются. */ +(function () { + function buttonsOf(form) { + return form.querySelectorAll('button[type="submit"]'); + } + + function setDisabled(form, disabled) { + buttonsOf(form).forEach(function (btn) { btn.disabled = disabled; }); + } + + function arm(form) { + if (form.dataset.submitGuard) return; + form.dataset.submitGuard = '1'; + form.addEventListener('submit', function (event) { + if (event.defaultPrevented) return; + window.setTimeout(function () { + // кто-то из более поздних обработчиков мог отменить сабмит + if (!event.defaultPrevented) setDisabled(form, true); + }, 0); + }); + // возврат на страницу по Back/Forward из кэша — разблокируем + window.addEventListener('pageshow', function (event) { + if (event.persisted) setDisabled(form, false); + }); + } + + window.addEventListener('DOMContentLoaded', function () { + document.querySelectorAll('form').forEach(arm); + }); +})(); diff --git a/web/src/main/resources/static/js/tablekit.js b/web/src/main/resources/static/js/tablekit.js index efbf031..fbd5717 100644 --- a/web/src/main/resources/static/js/tablekit.js +++ b/web/src/main/resources/static/js/tablekit.js @@ -158,13 +158,26 @@ if (th.hasAttribute('data-nosort')) return; th.classList.add('th-sortable'); if (!th.title) th.title = 'Сортировать по этому столбцу'; - th.addEventListener('click', function (event) { - if (event.target.closest('form, button, a, select, input')) return; + // сортировка с клавиатуры: фокус + Enter/пробел, состояние доступно скринридеру + th.tabIndex = 0; + if (th.classList.contains('is-asc')) th.setAttribute('aria-sort', 'ascending'); + function doSort() { var asc = !th.classList.contains('is-asc'); clearAllMarkers(table); th.classList.add(asc ? 'is-asc' : 'is-desc'); + th.setAttribute('aria-sort', asc ? 'ascending' : 'descending'); sortColumn(table, index, asc ? 1 : -1); th.title = (th.textContent.trim() || 'Столбец') + ' — ' + (asc ? 'по возрастанию' : 'по убыванию'); + } + th.addEventListener('click', function (event) { + if (event.target.closest('form, button, a, select, input')) return; + doSort(); + }); + th.addEventListener('keydown', function (event) { + if (event.target !== th) return; + if (event.key !== 'Enter' && event.key !== ' ') return; + event.preventDefault(); + doSort(); }); }); buildFilters(table); diff --git a/web/src/main/resources/templates/admin/assign-fragment.html b/web/src/main/resources/templates/admin/assign-fragment.html new file mode 100644 index 0000000..1c693fc --- /dev/null +++ b/web/src/main/resources/templates/admin/assign-fragment.html @@ -0,0 +1,76 @@ + + + + +
+
+
+ + + + +
+ — +
+ +
+ + + + + + + + + + + + + + + +
СтудентУровеньВариантИсточникСдача
+
—
+
—
+ 		+
+ + + +
+ 		+
+ + + +
+ 		+ авто + + сдача ↓ 1 + не сдано +
В группе нет студентов.
+
+
+ + diff --git a/web/src/main/resources/templates/admin/dashboard.html b/web/src/main/resources/templates/admin/dashboard.html index 8d77b9f..5f7b410 100644 --- a/web/src/main/resources/templates/admin/dashboard.html +++ b/web/src/main/resources/templates/admin/dashboard.html @@ -19,7 +19,7 @@

Лабораторные

- Все → + Все лабораторные →
@@ -36,7 +36,7 @@
diff --git a/web/src/main/resources/templates/admin/group.html b/web/src/main/resources/templates/admin/group.html index 1c81406..9014096 100644 --- a/web/src/main/resources/templates/admin/group.html +++ b/web/src/main/resources/templates/admin/group.html @@ -9,8 +9,10 @@

Группа

Студентов: 0

-
- +
+ Скачать работы + Скачать пароли
@@ -18,14 +20,14 @@

Студенты и лабораторные

-
- - -
+
+
+ +
Опубликованных лабораторных пока нет.
@@ -37,10 +39,17 @@
- @@ -140,7 +155,7 @@
- +
@@ -153,7 +168,7 @@
- +
@@ -165,7 +180,7 @@

Назначить вариант

- +

—

@@ -201,7 +216,7 @@

Индивидуальное задание

- +
@@ -249,7 +264,7 @@

Импорт заданий

- +
@@ -277,7 +292,7 @@

Добавить студента

- +
- + +
+
+

+ Группа — импортирована +

+

+ Новых студентов: 0 — логины и одноразовые пароли: +

+
Студент +
ЛР + —
+
@@ -80,54 +89,60 @@ th:classappend="${cell.variantManual} ? 'manual' : ''" th:attr="data-student=${row.studentId},data-student-name=${row.fullName},data-lab=${cell.labId},data-lab-name='ЛР ' + ${cell.labNumber},data-variant=${cell.variantId}" onclick="openVariant(this)" - title="Назначить вариант вручную или выдать случайный">Вариант — + title="Назначить вариант вручную или выдать случайный"> + В-1 + выдать + +
-
- ↓ - ↓ - -
-
- временный - изменён - - - - -
+ временный + изменён 		-
- - -
+
+
+ + +
+
+ + +
+
В группе нет студентов.
+ + + + + + + + +
СтудентЛогинПароль
———
+

+ Обновлено существующих студентов: 0 +

+

Пароли показываются один раз — сохраните их или воспользуйтесь «Скачать пароли».

+
+
+ + + +
+ + +
+
+ + + diff --git a/web/src/main/resources/templates/admin/groups.html b/web/src/main/resources/templates/admin/groups.html index 35dc7ba..2963649 100644 --- a/web/src/main/resources/templates/admin/groups.html +++ b/web/src/main/resources/templates/admin/groups.html @@ -3,31 +3,19 @@ Группы — LMS
-

Группы

-
- -

Создать группу

-
- - - -
-

Импорт студентов

-

Файл .xls, .xlsx или .csv со столбцами: студенческий номер, ФИО, основа обучения.

-
-
- - -
-
-
-

Список групп

- +
+

Группы

+
+ + + +
+
- + @@ -38,6 +26,39 @@
НазваниеСтудентовВыгрузка
НазваниеСтудентовПароли
—
+ + +
+

Новая группа

+ +
+
+
+
+
+ + +
+
+ + + +
+

Импорт студентов

+ +
+
+
+

Файл .xls, .xlsx или .csv со столбцами: студенческий номер, ФИО, основа обучения. Логин — студенческий номер, пароль сгенерируется и покажется после импорта.

+
+
+
+
+ + +
+
+
- \ No newline at end of file + diff --git a/web/src/main/resources/templates/admin/lab-edit.html b/web/src/main/resources/templates/admin/lab-edit.html index 07ffdaa..fba3f98 100644 --- a/web/src/main/resources/templates/admin/lab-edit.html +++ b/web/src/main/resources/templates/admin/lab-edit.html @@ -24,9 +24,9 @@
@@ -45,9 +45,9 @@ - @@ -81,9 +81,9 @@
- + onclick="return confirm('Удалить вариант? Несохранённые правки остальных полей формы будут потеряны.')">Удалить вариант diff --git a/web/src/main/resources/templates/admin/lab.html b/web/src/main/resources/templates/admin/lab.html index 1af052f..059ea09 100644 --- a/web/src/main/resources/templates/admin/lab.html +++ b/web/src/main/resources/templates/admin/lab.html @@ -7,138 +7,31 @@

ЛР 1

-

- Статус: -

+
+ Статус: draft + - + · Срок: · Защита: · Максимум: б. - Вариантов назначено: - Занято вариантов: из - Свободно: - Сдано: -

+ · Вариантов занято: из + · Сдач: +
Редактировать -

Условие лабораторной

-
-

Назначения студентов

-

- Смена уровня сразу подбирает вариант; конкретный вариант можно закрепить вручную. -

-
-
Нет студентов для назначения.
- -
-
- -
- - - -
-
- - - -
-
-
- -
-
-
-

Группа

- -
-
- - - - - - - - - - - - - - - -
ВыборСтудентУровеньВариантИсточникСдача
-
—
-
—
- 		-
- - - - -
- 		-
- - - - -
- 		- авто - - - попытка 1 ↓ -
- main.c -
- — -
-
-
-
-

Уровни и варианты

Уровней пока нет — добавьте на экране настройки.
@@ -162,40 +55,6 @@
- - -
-

Распределение вариантов

- -
-
-
- -

- Студенты выбранных групп раздаются одной общей очерёдностью. Каждый получает - вариант со своего уровня (без уровня — с первого уровня лабораторной): - пока внутри уровня есть не выданные никому варианты — они выдаются без повторов, - затем вариант берётся из наименее занятых. Ручные закрепления сохраняются всегда. -

-
- -
- -
-
- - -
-
- - - diff --git a/web/src/main/resources/templates/admin/labs.html b/web/src/main/resources/templates/admin/labs.html index 0027ff4..9d76d04 100644 --- a/web/src/main/resources/templates/admin/labs.html +++ b/web/src/main/resources/templates/admin/labs.html @@ -6,34 +6,41 @@

Лабораторные

-
-

Статус draft — черновик (редактирование, студентам не видна). - published — опубликована (доступна студентам, видны распределённые варианты). Нажмите на бейдж статуса, чтобы переключить.

- + - + - + - +
№НазваниеВариантовСтатус
№НазваниеВариантовСтатусДействие
1 — 0 -
+
+ — + + - + +
Лабораторных пока нет.
Лабораторных пока нет.
@@ -42,7 +49,7 @@

Импорт лабораторной

- +
@@ -65,7 +72,7 @@

Новая лабораторная

- +
diff --git a/web/src/main/resources/templates/admin/users.html b/web/src/main/resources/templates/admin/users.html index a7bf006..e8ce7f3 100644 --- a/web/src/main/resources/templates/admin/users.html +++ b/web/src/main/resources/templates/admin/users.html @@ -6,9 +6,9 @@

Пользователи

- - - Сбросить + + + Сбросить Найдено: 0 @@ -34,7 +34,7 @@ onsubmit="return confirm('Удалить пользователя безвозвратно? Будут удалены аккаунт, группа, оценки с комментариями, сдачи с файлами и задания защит.');"> - + diff --git a/web/src/main/resources/templates/change_password.html b/web/src/main/resources/templates/change_password.html index 21ae338..3a096da 100644 --- a/web/src/main/resources/templates/change_password.html +++ b/web/src/main/resources/templates/change_password.html @@ -13,9 +13,9 @@ Смените пароль, если хотите. После сохранения вы останетесь в системе.

-
-

Минимум 8 символов.

-
+
+

Минимум 8 символов.

+
Отмена diff --git a/web/src/main/resources/templates/fragments/base.html b/web/src/main/resources/templates/fragments/base.html index 3188d61..4b40e29 100644 --- a/web/src/main/resources/templates/fragments/base.html +++ b/web/src/main/resources/templates/fragments/base.html @@ -4,16 +4,26 @@ LMS - - - - - - - - - - + + + + + + + + + + + + + + + + +
diff --git a/web/src/main/resources/templates/learn/chapter.html b/web/src/main/resources/templates/learn/chapter.html index 7a6f3fd..aecf4b3 100644 --- a/web/src/main/resources/templates/learn/chapter.html +++ b/web/src/main/resources/templates/learn/chapter.html @@ -15,14 +15,14 @@
- - + + - @@ -30,7 +30,7 @@ onsubmit="return confirm('Удалить главу со всеми разделами и статьями?')"> - +
@@ -62,24 +62,24 @@ th:text="${section.title}">Раздел скрыто
-
-
-
- +
@@ -95,14 +95,14 @@ th:text="${article.title}">Статья скрыто
- Править
-
@@ -116,7 +116,7 @@

Переименовать раздел

-
@@ -132,7 +132,7 @@

Перенести статьи из раздела

-
@@ -140,6 +140,9 @@

Выберите раздел-приёмник: статьи перенесутся туда, а текущий раздел удалится.

+

+ Других разделов нет — при подтверждении статьи будут удалены вместе с разделом. +

- -
- +
@@ -73,18 +68,18 @@ - - -
- +
@@ -104,16 +99,16 @@ - - Править
- +
@@ -125,7 +120,7 @@

Переименовать раздел

-
@@ -140,7 +135,7 @@

Перенести статьи из раздела

-
@@ -150,6 +145,7 @@
@@ -164,7 +160,7 @@

Переименовать главу

-
@@ -182,7 +178,7 @@

Импорт материалов (zip)

- +
@@ -208,7 +204,7 @@

Новая глава

- +
@@ -223,7 +219,7 @@

Новая статья

- +
@@ -245,8 +241,8 @@
- - + +
diff --git a/web/src/main/resources/templates/learn/section.html b/web/src/main/resources/templates/learn/section.html index 2efdeee..feb2c34 100644 --- a/web/src/main/resources/templates/learn/section.html +++ b/web/src/main/resources/templates/learn/section.html @@ -17,23 +17,23 @@
- - + + - - +
- +
@@ -58,14 +58,14 @@ скрыто - Править
-
@@ -73,7 +73,7 @@ onsubmit="return confirm('Удалить статью?')"> - + @@ -85,7 +85,7 @@

Переименовать раздел

- +
@@ -101,13 +101,16 @@

Перенести статьи и удалить раздел

- +

Выберите раздел-приёмник: его статьи перенесутся туда, а текущий раздел удалится.

+

+ Других разделов нет — при подтверждении статьи будут удалены вместе с разделом. +

-
+
+
diff --git a/web/src/main/resources/templates/setup.html b/web/src/main/resources/templates/setup.html index b9b0df5..c4b130c 100644 --- a/web/src/main/resources/templates/setup.html +++ b/web/src/main/resources/templates/setup.html @@ -7,9 +7,9 @@

Создание администратора

Это разовая настройка. Аккаунт преподавателя создаётся один раз.

-
-

Минимум 8 символов.

-
+
+

Минимум 8 символов.

+
diff --git a/web/src/main/resources/templates/student/dashboard.html b/web/src/main/resources/templates/student/dashboard.html index c3b1993..18a1fca 100644 --- a/web/src/main/resources/templates/student/dashboard.html +++ b/web/src/main/resources/templates/student/dashboard.html @@ -33,7 +33,7 @@

—

- вариант + вариант сдано (попытка 1) не сдано оценка: diff --git a/web/src/main/resources/templates/student/lab.html b/web/src/main/resources/templates/student/lab.html index 151da9b..be7df8e 100644 --- a/web/src/main/resources/templates/student/lab.html +++ b/web/src/main/resources/templates/student/lab.html @@ -46,8 +46,8 @@ 1 - - Скачать + main.c + Скачать Нет сдач. @@ -89,7 +89,8 @@
- +
@@ -97,7 +98,8 @@

Ваше индивидуальное задание

-
+
@@ -107,7 +109,7 @@

Ожидайте преподавателя с проверкой.

- +
diff --git a/web/src/main/resources/templates/student/labs.html b/web/src/main/resources/templates/student/labs.html index fc4822d..efea1bb 100644 --- a/web/src/main/resources/templates/student/labs.html +++ b/web/src/main/resources/templates/student/labs.html @@ -11,7 +11,7 @@ ЛР
- Вариант + Вариант сдано (попытка ) не сдано оценка: