Migrate to new swagger docs generation (#18)

pkg/routes/api/v1/user_add_update.go

@@ -5,78 +5,28 @@ import (
 	"code.vikunja.io/api/pkg/routes/crud"
 	"github.com/labstack/echo"
 	"net/http"
-	"strconv"
 )
 
-// RegisterUser ...
+// RegisterUser is the register handler
+// @Summary Register
+// @Description Creates a new user account.
+// @tags user
+// @Accept json
+// @Produce json
+// @Param credentials body models.APIUserPassword true "The user credentials"
+// @Success 200 {object} models.User
+// @Failure 400 {object} models.HTTPError "No or invalid user register object provided / User already exists."
+// @Failure 500 {object} models.Message "Internal error"
+// @Router /register [post]
 func RegisterUser(c echo.Context) error {
-
-	// swagger:operation POST /register user register
-	// ---
-	// summary: Creates a new user account
-	// consumes:
-	// - application/json
-	// produces:
-	// - application/json
-	// parameters:
-	// - name: body
-	//   in: body
-	//   schema:
-	//     "$ref": "#/definitions/APIUserPassword"
-	// responses:
-	//   "200":
-	//     "$ref": "#/responses/User"
-	//   "400":
-	//     "$ref": "#/responses/Message"
-	//   "500":
-	//     "$ref": "#/responses/Message"
-
-	return userAddOrUpdate(c)
-}
-
-// userAddOrUpdate is the handler to add a user
-func userAddOrUpdate(c echo.Context) error {
-
-	// TODO: prevent everyone from updating users
-
 	// Check for Request Content
 	var datUser *models.APIUserPassword
-
 	if err := c.Bind(&datUser); err != nil {
-		return c.JSON(http.StatusBadRequest, models.Message{"No user model provided."})
-	}
-
-	// Check if we have an ID other than the one in the struct
-	id := c.Param("id")
-	if id != "" {
-		// Make int
-		userID, err := strconv.ParseInt(id, 10, 64)
-
-		if err != nil {
-			return c.JSON(http.StatusBadRequest, models.Message{"Invalid ID."})
-		}
-		datUser.ID = userID
-	}
-
-	// Check if the user exists
-	var exists = true
-	_, err := models.GetUserByID(datUser.ID)
-	if err != nil {
-		if models.IsErrUserDoesNotExist(err) {
-			exists = false
-		} else {
-			return c.JSON(http.StatusInternalServerError, models.Message{"Could not check if the user exists."})
-		}
-	}
-
-	// Insert or update the user
-	var newUser models.User
-	if exists {
-		newUser, err = models.UpdateUser(datUser.APIFormat())
-	} else {
-		newUser, err = models.CreateUser(datUser.APIFormat())
+		return c.JSON(http.StatusBadRequest, models.Message{"No or invalid user model provided."})
 	}
 
+	// Insert the user
+	newUser, err := models.CreateUser(datUser.APIFormat())
 	if err != nil {
 		return crud.HandleHTTPError(err)
 	}