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 /modules/repository | |
| 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.
Diffstat (limited to 'modules/repository')
| -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 { |