Add delete email support

Signed-off-by: Andrew Thornton <[email protected]>

Author: zeripath

cmd/admin.go

@@ -10,6 +10,7 @@ import (
 	"errors"
 	"fmt"
 	"os"
+	"strings"
 	"text/tabwriter"
 
 	"code.gitea.io/gitea/models"
@@ -126,7 +127,7 @@ var (
 
 	microcmdUserDelete = cli.Command{
 		Name:  "delete",
-		Usage: "Delete specific user",
+		Usage: "Delete specific user by id, name or email",
 		Flags: []cli.Flag{
 			cli.Int64Flag{
 				Name:  "id",
@@ -136,6 +137,10 @@ var (
 				Name:  "username,u",
 				Usage: "Username of the user to delete",
 			},
+			cli.StringFlag{
+				Name:  "email,e",
+				Usage: "Email of the user to delete",
+			},
 		},
 		Action: runDeleteUser,
 	}
@@ -472,8 +477,8 @@ func runListUsers(c *cli.Context) error {
 }
 
 func runDeleteUser(c *cli.Context) error {
-	if !c.IsSet("id") && !c.IsSet("username") {
-		return fmt.Errorf("--id or --username missing")
+	if !c.IsSet("id") && !c.IsSet("username") && !c.IsSet("email") {
+		return fmt.Errorf("You must provide the id, username or email of a user to delete")
 	}
 
 	if err := initDB(); err != nil {
@@ -482,14 +487,23 @@ func runDeleteUser(c *cli.Context) error {
 
 	var err error
 	var user *models.User
-	if c.IsSet("id") {
-		user, err = models.GetUserByID(c.Int64("id"))
-	} else {
+	if c.IsSet("email") {
+		user, err = models.GetUserByEmail(c.String("email"))
+	} else if c.IsSet("username") {
 		user, err = models.GetUserByName(c.String("username"))
+	} else {
+		user, err = models.GetUserByID(c.Int64("id"))
 	}
 	if err != nil {
 		return err
 	}
+	if c.IsSet("username") && user.LowerName != strings.ToLower(strings.TrimSpace(c.String("username"))) {
+		return fmt.Errorf("The user %s who has email %s does not match the provided username %s", user.Name, c.String("email"), c.String("username"))
+	}
+
+	if c.IsSet("id") && user.ID != c.Int64("id") {
+		return fmt.Errorf("The user %s does not match the provided id %d", user.Name, c.Int64("id"))
+	}
 
 	return models.DeleteUser(user)
 }

docs/content/doc/usage/command-line.en-us.md

@@ -69,9 +69,10 @@ Admin operations:
         - `gitea admin user list`
     - `delete`:
       - Options:
+        - `--email`: Email of the user to be deleted.
         - `--username`: Username of user to be deleted.
         - `--id`: ID of user to be deleted.
-        - One of `--id` or `--username` is required.
+        - One of `--id`, `--username` or `--email` is required. If more than one is provided then all have to match.
       - Examples:
         - `gitea admin user delete --id 1`
     - `create`: - Options: - `--name value`: Username. Required. As of gitea 1.9.0, use the `--username` flag instead. - `--username value`: Username. Required. New in gitea 1.9.0. - `--password value`: Password. Required. - `--email value`: Email. Required. - `--admin`: If provided, this makes the user an admin. Optional. - `--access-token`: If provided, an access token will be created for the user. Optional. (default: false). - `--must-change-password`: If provided, the created user will be required to choose a newer password after