internal/short: avoid uncessary assignment

changkun · Dec 20, 2023 · 29ed6e5 · 29ed6e5
1 parent 04d5153
commit 29ed6e5
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/internal/short/cmd.go b/internal/short/cmd.go
@@ -72,9 +72,7 @@ func Edit(ctx context.Context, s *db.Store, operate Op, a string, r *models.Redi
 		// into concurrent inconsistent issue. But for small scale
 		// use, it is fine for now.
 		rr, err = s.FetchAlias(ctx, a)
-		if err != nil {
-			err = nil
-		} else {
+		if err == nil {
 			// use old values if not presents
 			if r.URL == "" {
 				r.URL = rr.URL