From e18ee1f0fe5afe95800f10ad7c41ad71b2e5e5be Mon Sep 17 00:00:00 2001
From: David Mehren <git@herrmehren.de>
Date: Thu, 29 Apr 2021 16:40:13 +0200
Subject: [PATCH] UsersService: Remove null from toUserDto return type

toUserDto won't return null, as TS's strict mode
prevents user from being nullish

Signed-off-by: David Mehren <git@herrmehren.de>
---
 src/users/users.service.ts | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/src/users/users.service.ts b/src/users/users.service.ts
index 7a10b8fac..f7ce84217 100644
--- a/src/users/users.service.ts
+++ b/src/users/users.service.ts
@@ -108,13 +108,9 @@ export class UsersService {
   /**
    * Build UserInfoDto from a user.
    * @param {User=} user - the user to use
-   * @return {(UserInfoDto|null)} the built UserInfoDto
+   * @return {(UserInfoDto)} the built UserInfoDto
    */
-  toUserDto(user: User | null | undefined): UserInfoDto | null {
-    if (!user) {
-      this.logger.warn(`Recieved ${String(user)} argument!`, 'toUserDto');
-      return null;
-    }
+  toUserDto(user: User): UserInfoDto {
     return {
       userName: user.userName,
       displayName: user.displayName,