Merge branch 'main' into refactor-webhook

Author: techknowlogick

integrations/api_pull_review_test.go

@@ -11,6 +11,7 @@ import (
 
 	"code.gitea.io/gitea/models"
 	api "code.gitea.io/gitea/modules/structs"
+	jsoniter "github.com/json-iterator/go"
 	"github.com/stretchr/testify/assert"
 )
 
@@ -139,6 +140,59 @@ func TestAPIPullReview(t *testing.T) {
 	req = NewRequestf(t, http.MethodDelete, "/api/v1/repos/%s/%s/pulls/%d/reviews/%d?token=%s", repo.OwnerName, repo.Name, pullIssue.Index, review.ID, token)
 	resp = session.MakeRequest(t, req, http.StatusNoContent)
 
+	// test CreatePullReview Comment without body but with comments
+	req = NewRequestWithJSON(t, http.MethodPost, fmt.Sprintf("/api/v1/repos/%s/%s/pulls/%d/reviews?token=%s", repo.OwnerName, repo.Name, pullIssue.Index, token), &api.CreatePullReviewOptions{
+		// Body:  "",
+		Event: "COMMENT",
+		Comments: []api.CreatePullReviewComment{{
+			Path:       "README.md",
+			Body:       "first new line",
+			OldLineNum: 0,
+			NewLineNum: 1,
+		}, {
+			Path:       "README.md",
+			Body:       "first old line",
+			OldLineNum: 1,
+			NewLineNum: 0,
+		},
+		},
+	})
+	var commentReview api.PullReview
+
+	resp = session.MakeRequest(t, req, http.StatusOK)
+	DecodeJSON(t, resp, &commentReview)
+	assert.EqualValues(t, "COMMENT", commentReview.State)
+	assert.EqualValues(t, 2, commentReview.CodeCommentsCount)
+	assert.EqualValues(t, "", commentReview.Body)
+	assert.EqualValues(t, false, commentReview.Dismissed)
+
+	// test CreatePullReview Comment with body but without comments
+	commentBody := "This is a body of the comment."
+	req = NewRequestWithJSON(t, http.MethodPost, fmt.Sprintf("/api/v1/repos/%s/%s/pulls/%d/reviews?token=%s", repo.OwnerName, repo.Name, pullIssue.Index, token), &api.CreatePullReviewOptions{
+		Body:     commentBody,
+		Event:    "COMMENT",
+		Comments: []api.CreatePullReviewComment{},
+	})
+
+	resp = session.MakeRequest(t, req, http.StatusOK)
+	DecodeJSON(t, resp, &commentReview)
+	assert.EqualValues(t, "COMMENT", commentReview.State)
+	assert.EqualValues(t, 0, commentReview.CodeCommentsCount)
+	assert.EqualValues(t, commentBody, commentReview.Body)
+	assert.EqualValues(t, false, commentReview.Dismissed)
+
+	// test CreatePullReview Comment without body and no comments
+	req = NewRequestWithJSON(t, http.MethodPost, fmt.Sprintf("/api/v1/repos/%s/%s/pulls/%d/reviews?token=%s", repo.OwnerName, repo.Name, pullIssue.Index, token), &api.CreatePullReviewOptions{
+		Body:     "",
+		Event:    "COMMENT",
+		Comments: []api.CreatePullReviewComment{},
+	})
+	resp = session.MakeRequest(t, req, http.StatusUnprocessableEntity)
+	errMap := make(map[string]interface{})
+	json := jsoniter.ConfigCompatibleWithStandardLibrary
+	json.Unmarshal(resp.Body.Bytes(), &errMap)
+	assert.EqualValues(t, "review event COMMENT requires a body or a comment", errMap["message"].(string))
+
 	// test get review requests
 	// to make it simple, use same api with get review
 	pullIssue12 := models.AssertExistsAndLoadBean(t, &models.Issue{ID: 12}).(*models.Issue)

modules/git/blob.go

@@ -34,25 +34,30 @@ func (b *Blob) GetBlobContent() (string, error) {
 	return string(buf), nil
 }
 
-// GetBlobLineCount gets line count of lob as raw text
+// GetBlobLineCount gets line count of the blob
 func (b *Blob) GetBlobLineCount() (int, error) {
 	reader, err := b.DataAsync()
 	if err != nil {
 		return 0, err
 	}
 	defer reader.Close()
 	buf := make([]byte, 32*1024)
-	count := 0
+	count := 1
 	lineSep := []byte{'\n'}
+
+	c, err := reader.Read(buf)
+	if c == 0 && err == io.EOF {
+		return 0, nil
+	}
 	for {
-		c, err := reader.Read(buf)
 		count += bytes.Count(buf[:c], lineSep)
 		switch {
 		case err == io.EOF:
 			return count, nil
 		case err != nil:
 			return count, err
 		}
+		c, err = reader.Read(buf)
 	}
 }
 

modules/highlight/highlight.go

@@ -33,7 +33,7 @@ var (
 
 	once sync.Once
 
-	cache *lru.ARCCache
+	cache *lru.TwoQueueCache
 )
 
 // NewContext loads custom highlight map from local config
@@ -45,7 +45,7 @@ func NewContext() {
 		}
 
 		// The size 512 is simply a conservative rule of thumb
-		c, err := lru.NewARC(512)
+		c, err := lru.New2Q(512)
 		if err != nil {
 			panic(fmt.Sprintf("failed to initialize LRU cache for highlighter: %s", err))
 		}

options/locale/locale_bg-BG.ini

@@ -259,11 +259,18 @@ authorization_failed=Оторизацията беше неуспешна
 sspi_auth_failed=SSPI удостоверяването беше неуспешно
 
 [mail]
+
 activate_account=Моля активирайте Вашия профил
+
 activate_email=Провери адрес на ел. поща
+
+register_notify=Добре дошли в Gitea
+
 reset_password=Възстановете акаунта си
+
 register_success=Успешна регистрация
-register_notify=Добре дошли в Gitea
+
+
 
 
 

options/locale/locale_cs-CZ.ini

@@ -314,11 +314,18 @@ password_pwned=Heslo, které jste zvolili, je na <a target="_blank" rel="noopene
 password_pwned_err=Nelze dokončit požadavek na HaveIBeenPwned
 
 [mail]
+
 activate_account=Prosíme, aktivujte si váš účet
+
 activate_email=Ověřte vaši e-mailovou adresu
+
+register_notify=Vítejte v Gitea
+
 reset_password=Obnovit váš účet
+
 register_success=Registrace byla úspěšná
-register_notify=Vítejte v Gitea
+
+
 
 release.new.subject=%s v %s vydáno
 

options/locale/locale_de-DE.ini

@@ -316,19 +316,64 @@ password_pwned=Das von dir gewählte Passwort ist auf einer <a target="_blank" r
 password_pwned_err=Anfrage an HaveIBeenPwned konnte nicht abgeschlossen werden
 
 [mail]
+view_it_on=Auf %s ansehen
+link_not_working_do_paste=Link funktioniert nicht? Versuche ihn zu kopieren und im Browser einzufügen.
+hi_user_x=Hallo <b>%s</b>,
+
 activate_account=Bitte aktiviere dein Konto
+activate_account.title=%s, bitte aktiviere dein Konto
+activate_account.test_1=Hallo <b>%[1]s</b>, danke für deine Registrierung bei %[2]!
+activate_account.test_2=Bitte klicke innerhalb von <b>%s</b> auf folgenden Link, um dein Konto zu aktivieren:
+
 activate_email=Bestätige deine E-Mail-Adresse
+activate_email.title=%s, bitte verifiziere deine E-Mail-Adresse
+activate_email.text=Bitte klicke innerhalb von <b>%s</b> auf folgenden Link, um dein Konto zu aktivieren:
+
+register_notify=Willkommen bei Gitea
+register_notify.title=%[1]s, willkommen bei %[2]s
+register_notify.text_1=dies ist deine Bestätigungs-E-Mail für %s!
+register_notify.text_2=Du kannst dich jetzt mit dem Benutzernamen "%s" anmelden.
+register_notify.text_3=Wenn dieser Account von dir erstellt wurde, musst du zuerst <a href="%s">dein Passwort setzen</a>.
+
 reset_password=Stelle dein Konto wieder her
+reset_password.title=%s, du hast um Wiederherstellung deines Kontos gebeten
+reset_password.text=Bitte klicke innerhalb von <b>%s</b> auf folgenden Link, um dein Konto wiederherzustellen:
+
 register_success=Registrierung erfolgreich
-register_notify=Willkommen bei Gitea
+
+issue_assigned.pull=@%[1]s hat dich im Repository %[3]s dem Pull Request %[2]s zugewiesen.
+issue_assigned.issue=@%[1]s hat dich im Repository %[3]s dem Issue %[2]s zugewiesen.
+
+issue.x_mentioned_you=<b>@%s</b> hat dich erwähnt:
+issue.action.force_push=<b>%[1]s</b> hat %[3]s mit %[4]s auf <b>%[2]s</b> überschrieben.
+issue.action.push_1=<b>@%[1]s</b> hat einen Commit auf %[2]s gepusht
+issue.action.push_n=<b>@%[1]s</b> hat %[3]d Commits auf %[2]s gepusht
+issue.action.close=<b>@%[1]s</b> hat #%[2]d geschlossen.
+issue.action.reopen=<b>@%[1]s</b> hat #%[2]d wieder geöffnet.
+issue.action.merge=<b>@%[1]s</b> hat #%[2]d in %[3]s gemergt.
+issue.action.approve=<b>@%[1]s</b> hat diesen Pull-Request approved.
+issue.action.reject=<b>@%[1]s</b> hat Änderungen auf diesem Pull-Request angefordert.
+issue.action.review=<b>@%[1]s</b> hat diesen Pull-Request kommentiert.
+issue.action.review_dismissed=<b>@%[1]s</b> hat das letzte Review von %[2]s für diesen Pull Request verworfen.
+issue.action.ready_for_review=<b>@%[1]s</b> hat diesen Pull Request zum Review freigegeben.
+issue.action.new=<b>@%[1]s</b> hat #%[2]d geöffnet.
+issue.in_tree_path=In %s:
 
 release.new.subject=Release %s in %s erschienen
+release.new.text=<b>@%[1]s</b> hat %[2]s in %[3]s released
+release.title=Titel: %s
+release.note=Anmerkung:
+release.downloads=Downloads:
+release.download.zip=Quellcode (ZIP Datei)
+release.download.targz=Quellcode (TAR.GZ Datei)
 
 repo.transfer.subject_to=%s möchte "%s" an %s übertragen
 repo.transfer.subject_to_you=%s möchte dir "%s" übertragen
 repo.transfer.to_you=dir
+repo.transfer.body=Um es anzunehmen oder abzulehnen, öffne %s, oder ignoriere es einfach.
 
 repo.collaborator.added.subject=%s hat dich zu %s hinzugefügt
+repo.collaborator.added.text=Du wurdest als Mitarbeiter für folgendes Repository hinzugefügt:
 
 [modal]
 yes=Ja
@@ -830,6 +875,7 @@ migrate.gitlab.description=Migriere Daten von GitLab.com oder einem selbst gehos
 migrate.gitea.description=Migriere Daten von Gitea.com oder einem selbst gehostetem Gitea Server.
 migrate.gogs.description=Migriere Daten von notabug.org oder einem anderen, selbst gehosteten Gogs Server.
 migrate.migrating_git=Git Daten werden migriert
+migrate.migrating_topics=Themen werden migriert
 migrate.migrating_milestones=Meilensteine werden migriert
 migrate.migrating_labels=Labels werden migriert
 migrate.migrating_releases=Releases werden migriert
@@ -1557,6 +1603,8 @@ settings.hooks=Webhooks
 settings.githooks=Git-Hooks
 settings.basic_settings=Grundeinstellungen
 settings.mirror_settings=Mirror-Einstellungen
+settings.mirror_settings.docs=Richte dein Projekt so ein, dass Änderungen automatisch in ein anderes Repository gepusht, oder aus einem anderen Repository gepullt werden. Branches, tags und commits werden dann automatisch synchronisiert. <a target="_blank" rel="noopener noreferrer" href="https://docs.gitea.io/en-us/repo-mirror/">Wie kann ich ein Repository spiegeln? (Englisch)</a>
+settings.mirror_settings.mirrored_repository=Gespiegeltes Repository
 settings.mirror_settings.direction=Richtung
 settings.mirror_settings.direction.pull=Pull
 settings.mirror_settings.direction.push=Push

options/locale/locale_es-ES.ini

@@ -316,19 +316,64 @@ password_pwned=La contraseña que eligió está en una <a target="_blank" rel="n
 password_pwned_err=No se pudo completar la solicitud a HaveIBeenPwned
 
 [mail]
+view_it_on=Ver en %s
+link_not_working_do_paste=¿No funciona? Intenta copiarlo y pegarlo en tu navegador.
+hi_user_x=Hola <b>%s</b>,
+
 activate_account=Por favor, active su cuenta
+activate_account.title=%s, por favor activa tu cuenta
+activate_account.test_1=¡Hola <b>%[1]s</b>, gracias por registrarte en %[2]!
+activate_account.test_2=Haga clic en el siguiente enlace para activar su cuenta dentro de <b>%s</b>:
+
 activate_email=Verifique su correo electrónico
+activate_email.title=%s, por favor verifica tu dirección de correo electrónico
+activate_email.text=Por favor, haga clic en el siguiente enlace para verificar su dirección de correo electrónico dentro de <b>%s</b>:
+
+register_notify=¡Bienvenido a Gitea
+register_notify.title=%[1]s, bienvenido a %[2]s
+register_notify.text_1=este es tu correo de confirmación de registro para %s!
+register_notify.text_2=Ahora puede iniciar sesión vía nombre de usuario: %s.
+register_notify.text_3=Si esta cuenta ha sido creada para usted, por favor <a href="%s">establezca su contraseña</a> primero.
+
 reset_password=Recupere su cuenta
+reset_password.title=%s, has solicitado recuperar tu cuenta
+reset_password.text=Haga clic en el siguiente enlace para recuperar su cuenta dentro de <b>%s</b>:
+
 register_success=Registro completado
-register_notify=¡Bienvenido a Gitea
+
+issue_assigned.pull=@%[1]s le asignó al pull request %[2]s en el repositorio %[3]s.
+issue_assigned.issue=@%[1]s le asignó a la incidencia %[2]s en el repositorio %[3]s.
+
+issue.x_mentioned_you=<b>@%s</b> te mencionó:
+issue.action.force_push=<b>%[1]s</b> empujó a la fuerza el <b>%[2]s</b> de %[3]s a %[4]s.
+issue.action.push_1=<b>@%[1]s</b> hizo 1 commit al %[2]s
+issue.action.push_n=<b>@%[1]s</b> push %[3]d commits a %[2]s
+issue.action.close=<b>@%[1]s</b> cerró #%[2]d.
+issue.action.reopen=<b>@%[1]s</b> reabrió #%[2]d.
+issue.action.merge=<b>@%[1]s</b> fusionó #%[2]d en %[3]s.
+issue.action.approve=<b>@%[1]s</b> aprobó este pull request.
+issue.action.reject=<b>@%[1]s</b> solicitó cambios en este pull request.
+issue.action.review=<b>@%[1]s</b> comentó en este pull request.
+issue.action.review_dismissed=<b>@%[1]s</b> descartó la última revisión de %[2]s para este pull request.
+issue.action.ready_for_review=<b>@%[1]s</b> marcó este pull request listo para ser revisado.
+issue.action.new=<b>@%[1]s</b> creó #%[2]d.
+issue.in_tree_path=En %s:
 
 release.new.subject=%s en %s publicado
+release.new.text=<b>@%[1]s</b> lanzó %[2]s en %[3]s
+release.title=Título: %s
+release.note=Nota:
+release.downloads=Descargas:
+release.download.zip=Código fuente (ZIP)
+release.download.targz=Código fuente (TAR.GZ)
 
 repo.transfer.subject_to=%s desea transferir "%s" a %s
 repo.transfer.subject_to_you=%s desea transferir "%s" a usted
 repo.transfer.to_you=usted
+repo.transfer.body=Para aceptarlo o rechazarlo, visita %s o simplemente ignórelo.
 
 repo.collaborator.added.subject=%s le añadió en %s
+repo.collaborator.added.text=Has sido añadido como colaborador del repositorio:
 
 [modal]
 yes=Sí
@@ -726,6 +771,7 @@ mirror_prune_desc=Eliminar referencias de seguimiento de remotes obsoletas
 mirror_interval=Intervalo de réplica (Las unidades de tiempo válidas son 'h', 'm', 's'). Pone 0 para deshabilitar la sincronización automática.
 mirror_interval_invalid=El intervalo de réplica no es válido.
 mirror_address=Clonar desde URL
+mirror_address_desc=Ponga cualquier credencial requerida en la sección de Autorización.
 mirror_address_url_invalid=La url proporcionada no es válida. Debe escapar correctamente de todos los componentes de la url.
 mirror_address_protocol_invalid=La url proporcionada no es válida. Sólo las ubicaciones http(s):// o git:// pueden ser replicadas desde.
 mirror_lfs=Almacenamiento de archivos grande (LFS)
@@ -734,6 +780,8 @@ mirror_lfs_endpoint=Punto final de LFS
 mirror_lfs_endpoint_desc=Sync intentará usar la url del clon para <a target="_blank" rel="noopener noreferrer" href="%s">determinar el servidor LFS</a>. También puede especificar un punto final personalizado si los datos LFS del repositorio se almacenan en otro lugar.
 mirror_last_synced=Sincronizado por última vez
 mirror_password_placeholder=(Sin cambios)
+mirror_password_blank_placeholder=(Indefinido)
+mirror_password_help=Cambie el nombre de usario para eliminar una contraseña almacenada.
 watchers=Seguidores
 stargazers=Fans
 forks=Forks
@@ -820,11 +868,19 @@ migrated_from_fake=Migrado desde %[1]s
 migrate.migrate=Migrar desde %s
 migrate.migrating=Migrando desde <b>%s</b>...
 migrate.migrating_failed=La migración desde <b>%s</b> ha fallado.
+migrate.migrating_failed.error=Error: %s
 migrate.github.description=Migrar datos de Github.com o Github Enterprise.
 migrate.git.description=Migrar o replicar de datos de git desde los servicios de Git
 migrate.gitlab.description=Migrar datos de GitLab.com o servidor gitlab autoalojado.
 migrate.gitea.description=Migrando datos de Gitea.com o servidor Gitea autoalojado.
 migrate.gogs.description=Migrando datos de notabug.org u otro servidor de Gogs autoalojado.
+migrate.migrating_git=Migrando datos de Git
+migrate.migrating_topics=Migrando Temas
+migrate.migrating_milestones=Migrando Hitos
+migrate.migrating_labels=Migrando etiquetas
+migrate.migrating_releases=Migrando Lanzamientos
+migrate.migrating_issues=Migrando Incidencías
+migrate.migrating_pulls=Migrando Pull Requests
 
 mirror_from=réplica de
 forked_from=forkeado de
@@ -1318,7 +1374,10 @@ pulls.manually_merged_as=El Pull Request se ha fusionado manualmente como <a rel
 pulls.is_closed=El pull request ha sido cerrado.
 pulls.has_merged=El pull request ha sido fusionado.
 pulls.title_wip_desc=`<a href="#">Comience el título con <strong>%s</strong></a> para prevenir que el pull request se fusione accidentalmente.`
+pulls.cannot_merge_work_in_progress=Este pull request está marcado como un trabajo en curso.
 pulls.still_in_progress=¿Aún en curso?
+pulls.add_prefix=Añadir prefijo <strong>%s</strong>
+pulls.remove_prefix=Eliminar prefijo <strong>%s</strong>
 pulls.data_broken=Este pull request está rota debido a que falta información del fork.
 pulls.files_conflicted=Este pull request tiene cambios en conflicto con la rama de destino.
 pulls.is_checking=La comprobación de conflicto de fusión está en progreso. Inténtalo de nuevo en unos momentos.
@@ -1545,10 +1604,14 @@ settings.githooks=Git Hooks
 settings.basic_settings=Configuración Básica
 settings.mirror_settings=Configuración de réplica
 settings.mirror_settings.docs=Configure su proyecto para insertar y/o extraer automáticamente los cambios hacia/desde otro repositorio. Las ramas, etiquetas y commits se sincronizarán automáticamente. <a target="_blank" rel="noopener noreferrer" href="https://docs.gitea.io/en-us/repo-mirror/">¿Cómo replico los repositorios?</a>
+settings.mirror_settings.mirrored_repository=Repositorio Replicado
 settings.mirror_settings.direction=Dirección
 settings.mirror_settings.direction.pull=Pull
+settings.mirror_settings.direction.push=Push
 settings.mirror_settings.last_update=Última actualización
+settings.mirror_settings.push_mirror.none=No hay Réplicas de Push configurados
 settings.mirror_settings.push_mirror.remote_url=URL del repositorio remoto de Git
+settings.mirror_settings.push_mirror.add=Añadir Réplica de Push
 settings.sync_mirror=Sincronizar ahora
 settings.mirror_sync_in_progress=La sincronización del repositorio replicado está en curso. Vuelva a intentarlo más tarde.
 settings.email_notifications.enable=Habilitar las notificaciones por correo electrónico

options/locale/locale_fa-IR.ini

@@ -299,11 +299,18 @@ authorization_failed_desc=تاییدیه ناموفق بود. لذا ما درخ
 sspi_auth_failed=SSPI عدم احراز هویت
 
 [mail]
+
 activate_account=لطفا حساب خود را فعال کنید
+
 activate_email=نشانی ایمیل خود را تایید کنید
+
+register_notify=به گیتی یا گیت‌گو خوش آمدید
+
 reset_password=حساب خود را دوباره فعال کنید
+
 register_success=ثبت‌نام با موفقیت انجام شد
-register_notify=به گیتی یا گیت‌گو خوش آمدید
+
+