Merge branch 'main' into mouse-wheel-click-file-tree

Author: wxiaoguang

Makefile

@@ -36,7 +36,8 @@ XGO_PACKAGE ?= src.techknowlogick.com/xgo@latest
 GO_LICENSES_PACKAGE ?= github.com/google/go-licenses@v1
 GOVULNCHECK_PACKAGE ?= golang.org/x/vuln/cmd/govulncheck@v1
 ACTIONLINT_PACKAGE ?= github.com/rhysd/actionlint/cmd/actionlint@v1
-GOPLS_PACKAGE ?= golang.org/x/tools/[email protected]
+GOPLS_PACKAGE ?= golang.org/x/tools/[email protected]
+GOPLS_MODERNIZE_PACKAGE ?= golang.org/x/tools/gopls/internal/analysis/modernize/cmd/[email protected]
 
 DOCKER_IMAGE ?= gitea/gitea
 DOCKER_TAG ?= latest
@@ -230,7 +231,7 @@ clean: ## delete backend and integration files
 		tests/e2e/reports/ tests/e2e/test-artifacts/ tests/e2e/test-snapshots/
 
 .PHONY: fmt
-fmt: ## format the Go code
+fmt: ## format the Go and template code
 	@GOFUMPT_PACKAGE=$(GOFUMPT_PACKAGE) $(GO) run build/code-batch-process.go gitea-fmt -w '{file-list}'
 	$(eval TEMPLATES := $(shell find templates -type f -name '*.tmpl'))
 	@# strip whitespace after '{{' or '(' and before '}}' or ')' unless there is only
@@ -249,6 +250,19 @@ fmt-check: fmt
 	  exit 1; \
 	fi
 
+.PHONY: fix
+fix: ## apply automated fixes to Go code
+	$(GO) run $(GOPLS_MODERNIZE_PACKAGE) -fix ./...
+
+.PHONY: fix-check
+fix-check: fix
+	@diff=$$(git diff --color=always $(GO_SOURCES)); \
+	if [ -n "$$diff" ]; then \
+	  echo "Please run 'make fix' and commit the result:"; \
+	  printf "%s" "$${diff}"; \
+	  exit 1; \
+	fi
+
 .PHONY: $(TAGS_EVIDENCE)
 $(TAGS_EVIDENCE):
 	@mkdir -p $(MAKE_EVIDENCE_DIR)
@@ -288,7 +302,7 @@ checks: checks-frontend checks-backend ## run various consistency checks
 checks-frontend: lockfile-check svg-check ## check frontend files
 
 .PHONY: checks-backend
-checks-backend: tidy-check swagger-check fmt-check swagger-validate security-check ## check backend files
+checks-backend: tidy-check swagger-check fmt-check fix-check swagger-validate security-check ## check backend files
 
 .PHONY: lint
 lint: lint-frontend lint-backend lint-spell ## lint everything
@@ -809,6 +823,7 @@ deps-tools: ## install tool dependencies
 	$(GO) install $(GOVULNCHECK_PACKAGE) & \
 	$(GO) install $(ACTIONLINT_PACKAGE) & \
 	$(GO) install $(GOPLS_PACKAGE) & \
+	$(GO) install $(GOPLS_MODERNIZE_PACKAGE) & \
 	wait
 
 node_modules: package-lock.json

models/migrations/v1_22/v294_test.go

@@ -4,7 +4,6 @@
 package v1_22 //nolint
 
 import (
-	"slices"
 	"testing"
 
 	"code.gitea.io/gitea/models/migrations/base"
@@ -44,7 +43,7 @@ func Test_AddUniqueIndexForProjectIssue(t *testing.T) {
 	for _, index := range tables[0].Indexes {
 		if index.Type == schemas.UniqueType {
 			found = true
-			slices.Equal(index.Cols, []string{"project_id", "issue_id"})
+			assert.ElementsMatch(t, index.Cols, []string{"project_id", "issue_id"})
 			break
 		}
 	}

models/packages/package_property.go

@@ -92,8 +92,8 @@ func DeletePropertyByID(ctx context.Context, propertyID int64) error {
 	return err
 }
 
-// DeletePropertyByName deletes properties by name
-func DeletePropertyByName(ctx context.Context, refType PropertyType, refID int64, name string) error {
+// DeletePropertiesByName deletes properties by name
+func DeletePropertiesByName(ctx context.Context, refType PropertyType, refID int64, name string) error {
 	_, err := db.GetEngine(ctx).Where("ref_type = ? AND ref_id = ? AND name = ?", refType, refID, name).Delete(&PackageProperty{})
 	return err
 }

modules/actions/workflows.go

@@ -565,18 +565,12 @@ func matchPullRequestReviewEvent(prPayload *api.PullRequestPayload, evt *jobpars
 				actions = append(actions, "submitted", "edited")
 			}
 
-			matched := false
 			for _, val := range vals {
 				if slices.ContainsFunc(actions, glob.MustCompile(val, '/').Match) {
-					matched = true
-				}
-				if matched {
+					matchTimes++
 					break
 				}
 			}
-			if matched {
-				matchTimes++
-			}
 		default:
 			log.Warn("pull request review event unsupported condition %q", cond)
 		}
@@ -611,18 +605,12 @@ func matchPullRequestReviewCommentEvent(prPayload *api.PullRequestPayload, evt *
 				actions = append(actions, "created", "edited")
 			}
 
-			matched := false
 			for _, val := range vals {
 				if slices.ContainsFunc(actions, glob.MustCompile(val, '/').Match) {
-					matched = true
-				}
-				if matched {
+					matchTimes++
 					break
 				}
 			}
-			if matched {
-				matchTimes++
-			}
 		default:
 			log.Warn("pull request review comment event unsupported condition %q", cond)
 		}

modules/packages/content_store.go

@@ -28,8 +28,7 @@ func NewContentStore() *ContentStore {
 	return contentStore
 }
 
-// Get gets a package blob
-func (s *ContentStore) Get(key BlobHash256Key) (storage.Object, error) {
+func (s *ContentStore) OpenBlob(key BlobHash256Key) (storage.Object, error) {
 	return s.store.Open(KeyToRelativePath(key))
 }
 

modules/public/public.go

@@ -110,5 +110,4 @@ func servePublicAsset(w http.ResponseWriter, req *http.Request, fi os.FileInfo,
 		}
 	}
 	http.ServeContent(w, req, fi.Name(), modtime, content)
-	return
 }

modules/setting/markup.go

@@ -149,7 +149,7 @@ func loadMarkupFrom(rootCfg ConfigProvider) {
 func newMarkupSanitizer(name string, sec ConfigSection) {
 	rule, ok := createMarkupSanitizerRule(name, sec)
 	if ok {
-		if after, ok0 := strings.CutPrefix(name, "sanitizer."); ok0 {
+		if after, found := strings.CutPrefix(name, "sanitizer."); found {
 			names := strings.SplitN(after, ".", 2)
 			name = names[0]
 		}

options/locale/locale_uk-UA.ini

@@ -1334,6 +1334,7 @@ editor.new_branch_name_desc=Назва нової гілки…
 editor.cancel=Відмінити
 editor.filename_cannot_be_empty=Назва файлу не може бути порожньою.
 editor.filename_is_invalid=Назва файлу недійсна: "%s".
+editor.commit_email=Електронна пошта коміту
 editor.invalid_commit_email=Адреса електронної пошти для коміту недійсна.
 editor.file_is_a_symlink=`"%s" - це символічне посилання. Символічні посилання не можна редагувати у веб-редакторі`
 editor.filename_is_a_directory=Назва файлу '%s' вже використовується як назва каталогу у цьому сховищі.
@@ -1369,6 +1370,7 @@ commits.signed_by_untrusted_user=Підписаний недовіреним к
 commits.signed_by_untrusted_user_unmatched=Підписано недовіреним користувачем, який не відповідає комітеру
 commits.gpg_key_id=Ідентифікатор GPG ключа
 commits.ssh_key_fingerprint=Відбиток ключа SSH
+commits.view_path=Переглянути в історії
 commits.view_file_diff=Переглянути зміни до цього файлу в цьому коміті
 
 commit.revert=Повернути до попереднього стану
@@ -1497,6 +1499,7 @@ issues.filter_project=Проєкт
 issues.filter_project_all=Всі проєкти
 issues.filter_project_none=Проєкт відсутній
 issues.filter_assignee=Виконавець
+issues.filter_assignee_any_assignee=Призначено будь-кому
 issues.filter_poster=Автор
 issues.filter_user_placeholder=Пошук користувачів
 issues.filter_user_no_select=Усі користувачі
@@ -1527,13 +1530,16 @@ issues.action_milestone=Етап
 issues.action_milestone_no_select=Етап відсутній
 issues.action_assignee=Виконавець
 issues.action_assignee_no_select=Немає виконавця
+issues.action_check=Встановити/зняти позначку
+issues.action_check_all=Встановити/зняти позначку з усіх елементів
 issues.opened_by=%[1]s відкрито <a href="%[2]s">%[3]s</a>
 issues.opened_by_fake=%[1]s відкрито користувачем %[2]s
 issues.previous=Попередній
 issues.next=Далі
 issues.open_title=Відкрито
 issues.closed_title=Закрито
 issues.draft_title=Чернетка
+issues.num_comments_1=%d коментар
 issues.num_comments=%d коментарів
 issues.commented_at=`прокоментував(ла) <a href="#%s">%s</a>`
 issues.delete_comment_confirm=Ви впевнені, що хочете видалити цей коментар?
@@ -1542,6 +1548,7 @@ issues.context.quote_reply=Цитувати відповідь
 issues.context.reference_issue=Посилання в новій задачі
 issues.context.edit=Редагувати
 issues.context.delete=Видалити
+issues.close=Закрити задачу
 issues.comment_manually_pull_merged_at=вручну об'єднав(-ла) коміти %[1]s в %[2]s %[3]s
 issues.close_comment_issue=Закрити з коментарем
 issues.reopen_issue=Відкрити знову
@@ -1568,6 +1575,7 @@ issues.role.collaborator=Співавтор
 issues.role.collaborator_helper=Цей користувач був запрошений до співпраці у сховищі.
 issues.role.first_time_contributor=Учасник, який вперше долучився
 issues.role.first_time_contributor_helper=Це перший внесок цього користувача в сховищі.
+issues.role.contributor=Співавтор
 issues.role.contributor_helper=Цей користувач раніше вже вносив зміни до сховища.
 issues.re_request_review=Повторно попросити рецензію
 issues.is_stale=З часу останньої перевірки в цей PR було внесено деякі зміни
@@ -2190,6 +2198,7 @@ settings.webhook.response=Відповідь
 settings.webhook.headers=Заголовки
 settings.webhook.payload=Зміст
 settings.webhook.body=Тіло
+settings.webhook.replay.description=Повторити цей веб-хук.
 settings.githook_edit_desc=Якщо хук неактивний, буде показано зразок вмісту. Якщо залишити вміст порожнім, хук буде вимкнено.
 settings.githook_name=Назва хуку
 settings.githook_content=Зміст хука
@@ -2246,6 +2255,7 @@ settings.event_pull_request_sync=Запит на злиття синхроніз
 settings.event_pull_request_sync_desc=Запит до злиття синхронізовано.
 settings.event_package=Пакет
 settings.branch_filter=Фільтр гілок
+settings.authorization_header=Заголовок авторизації
 settings.active=Активний
 settings.active_helper=Інформацію про викликані події буде надіслано за цією веб-хук URL-адресою.
 settings.add_hook_success=Веб-хук було додано.
@@ -2294,6 +2304,7 @@ settings.protect_disable_push=Заборонити Push
 settings.protect_disable_push_desc=Для цієї гілки буде заборонено виконання push.
 settings.protect_enable_push=Дозволити Push
 settings.protect_enable_push_desc=Будь-хто із правом запису зможе виконувати push для цієї гілки (за виключенням force push).
+settings.protect_enable_merge=Увімкнути об’єднання
 settings.protect_check_status_contexts=Увімкнути перевірку стану
 settings.protect_status_check_patterns=Шаблони перевірки стану:
 settings.protect_status_check_patterns_desc=Введіть шаблони, щоб вказати, які перевірки стану повинні пройти гілки, перш ніж їх буде об'єднано у гілку, що відповідає цьому правилу. Кожен рядок визначає шаблон. Шаблони не можуть бути порожніми.
@@ -2327,6 +2338,8 @@ settings.block_outdated_branch_desc=Об'єднання буде неможли
 settings.block_admin_merge_override=Адміністратори повинні дотримуватися правил захисту гілки
 settings.block_admin_merge_override_desc=Адміністратори повинні дотримуватися правил захисту гілки і не можуть їх обійти.
 settings.default_branch_desc=Обрати типову гілку сховища для запитів на злиття і комітів:
+settings.merge_style_desc=Стилі об'єднання
+settings.default_merge_style_desc=Типовий стиль об'єднання
 settings.choose_branch=Оберіть гілку…
 settings.no_protected_branch=Немає захищених гілок.
 settings.edit_protected_branch=Редагувати
@@ -2348,12 +2361,14 @@ settings.chat_id=ID чату
 settings.matrix.homeserver_url=URL домашньої сторінки
 settings.matrix.room_id=ID кімнати
 settings.matrix.message_type=Тип повідомлення
+settings.visibility.public.button=Зробити публічним
 settings.archive.header=Відправити репозиторій в архів
 settings.archive.success=Сховище успішно заархівовано.
 settings.archive.error=Сталася помилка при спробі архівувати репозиторій. Докладнішу інформацію дивіться у журналі.
 settings.archive.error_ismirror=Неможливо архівувати дзеркальне сховище.
 settings.archive.branchsettings_unavailable=Параметри гілки не доступні, якщо репозиторій архівний.
 settings.archive.tagsettings_unavailable=Параметри міток недоступні, якщо репозиторій архівний.
+settings.unarchive.header=Розархівувати це сховище
 settings.unarchive.success=Сховище успішно розархівовано.
 settings.update_avatar_success=Аватар репозиторію оновлений.
 settings.lfs=LFS
@@ -2437,6 +2452,7 @@ diff.protected=Захищений
 diff.image.side_by_side=Поруч
 diff.image.swipe=Провести пальцем
 diff.image.overlay=Накласти
+diff.has_escaped=Цей рядок містить приховані символи Юнікоду
 diff.show_file_tree=Показати дерево файлів
 diff.hide_file_tree=Сховати дерево файлів
 diff.submodule_added=Підмодуль %[1]s додано в %[2]s
@@ -2749,6 +2765,7 @@ dashboard.resync_all_hooks=Заново синхронізувати хуки п
 dashboard.reinit_missing_repos=Заново ініціалізувати всі відсутні сховища Git'а, для яких існують записи
 dashboard.sync_external_users=Синхронізувати дані зовнішніх користувачів
 dashboard.cleanup_hook_task_table=Очистити таблицю hook_task
+dashboard.cleanup_actions=Очищення ресурсів прострочених дій
 dashboard.server_uptime=Час роботи сервера
 dashboard.current_goroutine=Поточна кількість Goroutines
 dashboard.current_memory_usage=Поточне використання пам'яті
@@ -2778,7 +2795,11 @@ dashboard.total_gc_time=Загальна пауза збирача сміття
 dashboard.total_gc_pause=Загальна пауза збирача сміття (GC)
 dashboard.last_gc_pause=Остання пауза збирача сміття (GC)
 dashboard.gc_times=Кількість запусків збирача сміття (GC)
+dashboard.delete_old_actions=Видалити всі старі дії з бази даних
 dashboard.update_checker=Перевірка оновлень
+dashboard.gc_lfs=Збір сміття мета-об'єктів LFS
+dashboard.cancel_abandoned_jobs=Скасувати покинуті завдання
+dashboard.start_schedule_tasks=Запуск запланованих завдань
 dashboard.sync_branch.started=Розпочато синхронізацію гілок
 dashboard.rebuild_issue_indexer=Перебудувати індексатор задач
 dashboard.sync_repo_licenses=Синхронізувати ліцензії сховища
@@ -2851,6 +2872,7 @@ emails.not_updated=Не вдалось оновити адресу електр
 emails.duplicate_active=Ця адреса електронної пошти вже активна для іншого користувача.
 emails.change_email_header=Редагувати властивості електронної пошти
 emails.change_email_text=Ви впевнені, що хочете оновити адресу електронної пошти?
+emails.delete=Видалити адресу електронної пошти
 emails.delete_desc=Ви впевнені, що хочете видалити адресу електронної пошти?
 emails.deletion_success=Адресу електронної пошти видалено.
 emails.delete_primary_email_error=Ви не можете видалити основну адресу електронної пошти.
@@ -2871,6 +2893,7 @@ repos.issues=Задачі
 repos.size=Розмір
 repos.lfs_size=Розмір LFS
 
+packages.package_manage_panel=Керування пакетами
 packages.total_size=Загальний розмір: %s
 packages.unreferenced_size=Розмір без посилань: %s
 packages.cleanup=Очистити прострочені дані
@@ -3393,6 +3416,7 @@ settings.link=Прив'язати пакет до сховища
 settings.link.description=Якщо ви зв'яжете пакет зі сховищем, його буде вказано у списку пакетів сховища.
 settings.link.select=Обрати сховище
 settings.link.button=Оновити посилання на сховище
+settings.link.error=Не вдалося оновити посилання на сховище.
 settings.delete=Видалити пакет
 settings.delete.description=Видалення пакета є остаточним і не може бути скасоване.
 settings.delete.notice=Ви збираєтесь видалити %s (%s). Цю операцію неможливо скасувати, ви впевнені?
@@ -3516,6 +3540,7 @@ logs.always_expand_running=Завжди розгортати поточні жу
 [projects]
 deleted.display_name=Видалений проєкт
 type-1.display_name=Індивідуальний проєкт
+type-2.display_name=Проєкт сховища
 type-3.display_name=Проєкт організації
 enter_fullscreen=Повноекранний режим
 exit_fullscreen=Вийти з повноекранного режиму

routers/api/packages/alpine/alpine.go

@@ -68,7 +68,7 @@ func GetRepositoryFile(ctx *context.Context) {
 		return
 	}
 
-	s, u, pf, err := packages_service.GetFileStreamByPackageVersion(
+	s, u, pf, err := packages_service.OpenFileForDownloadByPackageVersion(
 		ctx,
 		pv,
 		&packages_service.PackageFileInfo{
@@ -216,7 +216,7 @@ func DownloadPackageFile(ctx *context.Context) {
 		}
 	}
 
-	s, u, pf, err := packages_service.GetPackageFileStream(ctx, pfs[0])
+	s, u, pf, err := packages_service.OpenFileForDownload(ctx, pfs[0])
 	if err != nil {
 		if errors.Is(err, util.ErrNotExist) {
 			apiError(ctx, http.StatusNotFound, err)

routers/api/packages/arch/arch.go

@@ -239,7 +239,7 @@ func GetPackageOrRepositoryFile(ctx *context.Context) {
 		return
 	}
 
-	s, u, pf, err := packages_service.GetPackageFileStream(ctx, pfs[0])
+	s, u, pf, err := packages_service.OpenFileForDownload(ctx, pfs[0])
 	if err != nil {
 		if errors.Is(err, util.ErrNotExist) {
 			apiError(ctx, http.StatusNotFound, err)