fix: messed up on merge-conflicts (#7873)

When fixing conflicts accidentally I've undone my changes

src/lib/routes/admin-api/user-admin.ts

@@ -520,22 +520,30 @@ export default class UserAdminController extends Controller {
             ? Number(rootRole)
             : (rootRole as RoleName);
 
-        const { createdUser, inviteLink, emailSent } =
+        const createdUser = await this.userService.createUser(
-            await this.userService.createUserWithEmail(
+            {
-                {
+                username,
-                    username,
+                email,
-                    email,
+                name,
-                    name,
+                password,
-                    password,
+                rootRole: normalizedRootRole,
-                    rootRole: normalizedRootRole,
+            },
-                },
+            req.audit,
-                sendEmail,
+        );
-                req.audit,
+
-            );
+        const inviteLink = await this.userService.newUserInviteLink(
+            createdUser,
+            req.audit,
+        );
+
+        // send email defaults to true
+        const emailSent = (sendEmail !== undefined ? sendEmail : true)
+            ? await this.userService.sendWelcomeEmail(createdUser, inviteLink)
+            : false;
 
         const responseData: CreateUserResponseSchema = {
             ...serializeDates(createdUser),
-            inviteLink: inviteLink,
+            inviteLink,
             emailSent,
             rootRole: normalizedRootRole,
         };

src/lib/services/user-service.ts

@@ -259,20 +259,10 @@ class UserService {
         return userCreated;
     }
 
-    async createUserWithEmail(
+    async newUserInviteLink(
-        { username, email, name, password, rootRole }: ICreateUser,
+        user: IUserWithRootRole,
-        sendEmail = true,
         auditUser: IAuditUser = SYSTEM_USER_AUDIT,
-    ): Promise<{
+    ): Promise<string> {
-        createdUser: IUserWithRootRole;
-        inviteLink: string;
-        emailSent: boolean;
-    }> {
-        const createdUser = await this.createUser(
-            { username, email, name, password, rootRole },
-            auditUser,
-        );
-
         const passwordAuthSettings =
             await this.settingService.getWithDefault<SimpleAuthSettings>(
                 simpleAuthSettingsKey,
@@ -282,20 +272,26 @@ class UserService {
         let inviteLink = this.unleashUrl;
         if (!passwordAuthSettings.disabled) {
             const inviteUrl = await this.resetTokenService.createNewUserUrl(
-                createdUser.id,
+                user.id,
                 auditUser.username,
             );
             inviteLink = inviteUrl.toString();
         }
+        return inviteLink;
+    }
 
+    async sendWelcomeEmail(
+        user: IUserWithRootRole,
+        inviteLink: string,
+    ): Promise<boolean> {
         let emailSent = false;
         const emailConfigured = this.emailService.configured();
 
-        if (emailConfigured && sendEmail && createdUser.email) {
+        if (emailConfigured && user.email) {
             try {
                 await this.emailService.sendGettingStartedMail(
-                    createdUser.name || '',
+                    user.name || '',
-                    createdUser.email,
+                    user.email,
                     this.unleashUrl,
                     inviteLink,
                 );
@@ -312,7 +308,7 @@ class UserService {
             );
         }
 
-        return { createdUser, inviteLink, emailSent };
+        return emailSent;
     }
 
     async updateUser(