[#141] if/else instead of ternary op

dedica-team · Feb 2, 2022 · 4c6050c · 4c6050c
1 parent a018822
commit 4c6050c
Showing 1 changed file with 14 additions and 2 deletions.
diff --git a/src/main/java/de/bonndan/nivio/security/CustomOAuth2UserService.java b/src/main/java/de/bonndan/nivio/security/CustomOAuth2UserService.java
@@ -50,7 +50,12 @@ public static CustomOAuth2User fromGitHubUser(@NonNull final OAuth2User user,
     ) {
         var externalId = "";
         if (StringUtils.hasLength(nameAttribute)) {
-            externalId = String.valueOf(user.getAttribute("id") == null ? "" : user.getAttribute("id"));
+            final var id = user.getAttribute("id");
+            if (id == null) {
+                externalId = "";
+            } else {
+                externalId = String.valueOf(id);
+            }
         }
 
         var name = "";
@@ -64,9 +69,16 @@ public static CustomOAuth2User fromGitHubUser(@NonNull final OAuth2User user,
             }
         }
 
+        var alias = "";
+        if (StringUtils.hasLength(aliasAttribute)) {
+            alias = Optional.ofNullable((String) user.getAttribute(aliasAttribute)).orElse("");
+        } else {
+            alias = "";
+        }
+
         return new CustomOAuth2User(
                 externalId,
-                StringUtils.hasLength(aliasAttribute) ? Optional.ofNullable((String) user.getAttribute(aliasAttribute)).orElse("") : "",
+                alias,
                 name,
                 user.getAttributes(),
                 user.getAuthorities(),