aboutsummaryrefslogtreecommitdiff
path: root/internal/route/api/v1/user/email.go
diff options
context:
space:
mode:
Diffstat (limited to 'internal/route/api/v1/user/email.go')
-rw-r--r--internal/route/api/v1/user/email.go15
1 files changed, 8 insertions, 7 deletions
diff --git a/internal/route/api/v1/user/email.go b/internal/route/api/v1/user/email.go
index e211221c..07fd4f8a 100644
--- a/internal/route/api/v1/user/email.go
+++ b/internal/route/api/v1/user/email.go
@@ -5,25 +5,26 @@
package user
import (
- convert2 "gogs.io/gogs/internal/route/api/v1/convert"
"net/http"
api "github.com/gogs/go-gogs-client"
+ "github.com/pkg/errors"
"gogs.io/gogs/internal/conf"
"gogs.io/gogs/internal/context"
"gogs.io/gogs/internal/db"
+ "gogs.io/gogs/internal/route/api/v1/convert"
)
func ListEmails(c *context.APIContext) {
emails, err := db.GetEmailAddresses(c.User.ID)
if err != nil {
- c.ServerError("GetEmailAddresses", err)
+ c.Error(err, "get email addresses")
return
}
apiEmails := make([]*api.Email, len(emails))
for i := range emails {
- apiEmails[i] = convert2.ToEmail(emails[i])
+ apiEmails[i] = convert.ToEmail(emails[i])
}
c.JSONSuccess(&apiEmails)
}
@@ -45,16 +46,16 @@ func AddEmail(c *context.APIContext, form api.CreateEmailOption) {
if err := db.AddEmailAddresses(emails); err != nil {
if db.IsErrEmailAlreadyUsed(err) {
- c.Error(http.StatusUnprocessableEntity, "", "email address has been used: "+err.(db.ErrEmailAlreadyUsed).Email)
+ c.ErrorStatus(http.StatusUnprocessableEntity, errors.New("email address has been used: "+err.(db.ErrEmailAlreadyUsed).Email))
} else {
- c.Error(http.StatusInternalServerError, "AddEmailAddresses", err)
+ c.Error(err, "add email addresses")
}
return
}
apiEmails := make([]*api.Email, len(emails))
for i := range emails {
- apiEmails[i] = convert2.ToEmail(emails[i])
+ apiEmails[i] = convert.ToEmail(emails[i])
}
c.JSON(http.StatusCreated, &apiEmails)
}
@@ -74,7 +75,7 @@ func DeleteEmail(c *context.APIContext, form api.CreateEmailOption) {
}
if err := db.DeleteEmailAddresses(emails); err != nil {
- c.Error(http.StatusInternalServerError, "DeleteEmailAddresses", err)
+ c.Error(err, "delete email addresses")
return
}
c.NoContent()