Make sure to create a new namespace for new users

pkg/cmd/user.go

@@ -18,6 +18,7 @@
 package cmd
 
 import (
+	"code.vikunja.io/api/pkg/models"
 	"fmt"
 	"os"
 	"strconv"
@@ -157,11 +158,16 @@ var userCreateCmd = &cobra.Command{
 			Email:    userFlagEmail,
 			Password: getPasswordFromFlagOrInput(),
 		}
-		_, err := user.CreateUser(u)
+		newUser, err := user.CreateUser(u)
 		if err != nil {
 			log.Fatalf("Error creating new user: %s", err)
 		}
 
+		err = models.CreateNewNamespaceForUser(newUser)
+		if err != nil {
+			log.Fatalf("Error creating new namespace for user: %s", err)
+		}
+
 		fmt.Printf("\nUser was created successfully.\n")
 	},
 }

pkg/models/namespace.go

@@ -445,6 +445,16 @@ func (n *Namespace) Create(a web.Auth) (err error) {
 	return
 }
 
+// CreateNewNamespaceForUser creates a new namespace for a user. To prevent import cycles, we can't do that
+// directly in the user.Create function.
+func CreateNewNamespaceForUser(user *user.User) (err error) {
+	newN := &Namespace{
+		Title:       user.Username,
+		Description: user.Username + "'s namespace.",
+	}
+	return newN.Create(user)
+}
+
 // Delete deletes a namespace
 // @Summary Deletes a namespace
 // @Description Delets a namespace

pkg/modules/auth/openid/openid.go

@@ -243,6 +243,12 @@ func getOrCreateUser(cl *claims, issuer, subject string) (u *user.User, err erro
 			uu.Username = petname.Generate(3, "-")
 			u, err = user.CreateUser(uu)
 		}
+
+		// And create its namespace
+		err = models.CreateNewNamespaceForUser(u)
+		if err != nil {
+			return nil, err
+		}
 	} else {
 		// If it exists, check if the email address changed and change it if not
 		if cl.Email != u.Email {

pkg/routes/api/v1/user_register.go

@@ -57,8 +57,7 @@ func RegisterUser(c echo.Context) error {
 	}
 
 	// Add its namespace
-	newN := &models.Namespace{Title: newUser.Username, Description: newUser.Username + "'s namespace.", Owner: newUser}
-	err = newN.Create(newUser)
+	err = models.CreateNewNamespaceForUser(newUser)
 	if err != nil {
 		return handler.HandleHTTPError(err, c)
 	}