diff options
| author | wxiaoguang <wxiaoguang@gmail.com> | 2022-09-18 10:35:24 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-09-18 10:35:24 +0800 |
| commit | 395f65c65a4b3d2bb06056b9bba3f4c016ab03e9 (patch) | |
| tree | 380f6445cbcbd5849541fa313eb0949ccc9296d9 | |
| parent | 321964155a605900bad1c3c97163bf33b4ae5d65 (diff) | |
| download | gitea-395f65c65a4b3d2bb06056b9bba3f4c016ab03e9.tar.gz gitea-395f65c65a4b3d2bb06056b9bba3f4c016ab03e9.zip | |
Remove unnecessary length check for repo's Description & Website (#21194)
Follows #21119
The manual length check doesn't make sense nowadays:
1. The length check is already done by form's `binding:MaxSize` (then the manual check is unnecessary)
2. The CreateRepository doesn't have such check (then the manual check is inconsistent)
So this PR removes these manual length checks.
| -rw-r--r-- | modules/repository/create.go | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/modules/repository/create.go b/modules/repository/create.go index 7a25323def..966a6a2f21 100644 --- a/modules/repository/create.go +++ b/modules/repository/create.go @@ -10,7 +10,6 @@ import ( "os" "path" "strings" - "unicode/utf8" "code.gitea.io/gitea/models" activities_model "code.gitea.io/gitea/models/activities" @@ -337,13 +336,6 @@ func CheckDaemonExportOK(ctx context.Context, repo *repo_model.Repository) error func UpdateRepository(ctx context.Context, repo *repo_model.Repository, visibilityChanged bool) (err error) { repo.LowerName = strings.ToLower(repo.Name) - if utf8.RuneCountInString(repo.Description) > 255 { - repo.Description = string([]rune(repo.Description)[:255]) - } - if utf8.RuneCountInString(repo.Website) > 255 { - repo.Website = string([]rune(repo.Website)[:255]) - } - e := db.GetEngine(ctx) if _, err = e.ID(repo.ID).AllCols().Update(repo); err != nil { |