usememos/memos
1
1
Fork 0
mirror of https://github.com/usememos/memos.git synced 2025-12-20 07:38:43 +08:00
Code Issues Projects Releases Packages Wiki Activity

style: fix golangci-lint issues in memo_service.go

Browse source 
- Replace 'else {if}' with 'else if' (gocritic)
- Fix multi-line condition indentation (goimports)
This commit is contained in:
Claude 2025-11-08 01:36:32 +00:00
parent 652c855145
commit 9fcf654b40
No known key found for this signature in database
1 changed files with 4 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
server/router/api/v1/memo_service.go
View file

@ -34,11 +34,9 @@ func (s *APIV1Service) CreateMemo(ctx context.Context, request *v1pb.CreateMemoR
memoUID := strings.TrimSpace(request.MemoId) memoUID := strings.TrimSpace(request.MemoId)
if memoUID == "" { if memoUID == "" {
memoUID = shortuuid.New() memoUID = shortuuid.New()
} else { } else if !base.UIDMatcher.MatchString(memoUID) {
// Validate custom memo ID format // Validate custom memo ID format
if !base.UIDMatcher.MatchString(memoUID) { return nil, status.Errorf(codes.InvalidArgument, "invalid memo_id format: must be 1-32 characters, alphanumeric and hyphens only, cannot start or end with hyphen")
return nil, status.Errorf(codes.InvalidArgument, "invalid memo_id format: must be 1-32 characters, alphanumeric and hyphens only, cannot start or end with hyphen")
}
} }
create := &store.Memo{ create := &store.Memo{
@ -73,8 +71,8 @@ func (s *APIV1Service) CreateMemo(ctx context.Context, request *v1pb.CreateMemoR
// Check for unique constraint violation (AIP-133 compliance) // Check for unique constraint violation (AIP-133 compliance)
errMsg := err.Error() errMsg := err.Error()
if strings.Contains(errMsg, "UNIQUE constraint failed") || if strings.Contains(errMsg, "UNIQUE constraint failed") ||
strings.Contains(errMsg, "duplicate key") || strings.Contains(errMsg, "duplicate key") ||
strings.Contains(errMsg, "Duplicate entry") { strings.Contains(errMsg, "Duplicate entry") {
return nil, status.Errorf(codes.AlreadyExists, "memo with ID %q already exists", memoUID) return nil, status.Errorf(codes.AlreadyExists, "memo with ID %q already exists", memoUID)
} }
return nil, err return nil, err