Skip to content

Commit 0ae6ceb

Browse files
fix: GitHub user sync to refresh profile fields (#505)
1 parent f4ad51b commit 0ae6ceb

1 file changed

Lines changed: 4 additions & 19 deletions

File tree

server/application-server/src/main/java/de/tum/in/www1/hephaestus/gitprovider/user/github/GitHubUserSyncService.java

Lines changed: 4 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -52,30 +52,15 @@ public void syncUser(String login) {
5252
* repository or creating a new one.
5353
*
5454
* @param ghUser The GitHub user data to process.
55-
* @return The updated or newly created User entity, or {@code null} if an error
56-
* occurred during update.
55+
* @return The updated or newly created User entity.
5756
*/
5857
@Transactional
5958
public User processUser(GHUser ghUser) {
60-
var result = userRepository
59+
var user = userRepository
6160
.findById(ghUser.getId())
62-
.map(user -> {
63-
try {
64-
if (user.getUpdatedAt() == null || user.getUpdatedAt().isBefore(ghUser.getUpdatedAt())) {
65-
return userConverter.update(ghUser, user);
66-
}
67-
return user;
68-
} catch (IOException e) {
69-
logger.error("Failed to update repository {}: {}", ghUser.getId(), e.getMessage());
70-
return null;
71-
}
72-
})
61+
.map(existingUser -> userConverter.update(ghUser, existingUser))
7362
.orElseGet(() -> userConverter.convert(ghUser));
7463

75-
if (result == null) {
76-
return null;
77-
}
78-
79-
return userRepository.save(result);
64+
return userRepository.save(user);
8065
}
8166
}

0 commit comments

Comments
 (0)