From 9432be14aa95abb0c095f8207f55906f6448b377 Mon Sep 17 00:00:00 2001 From: Chris Lemaire Date: Wed, 13 Mar 2024 11:03:14 +0100 Subject: [PATCH] [gitlab] Cleanup unnecessary comparison GitOrigin-RevId: daff3adb49fba93a167cb4d1a96dba79139b1807 --- .../ui/create/model/GitLabMergeRequestCreateViewModel.kt | 2 +- .../ui/details/model/GitLabMergeRequestReviewFlowViewModel.kt | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/plugins/gitlab/src/org/jetbrains/plugins/gitlab/mergerequest/ui/create/model/GitLabMergeRequestCreateViewModel.kt b/plugins/gitlab/src/org/jetbrains/plugins/gitlab/mergerequest/ui/create/model/GitLabMergeRequestCreateViewModel.kt index 4f0e4372fbb3..9c48b4dd979a 100644 --- a/plugins/gitlab/src/org/jetbrains/plugins/gitlab/mergerequest/ui/create/model/GitLabMergeRequestCreateViewModel.kt +++ b/plugins/gitlab/src/org/jetbrains/plugins/gitlab/mergerequest/ui/create/model/GitLabMergeRequestCreateViewModel.kt @@ -181,7 +181,7 @@ internal class GitLabMergeRequestCreateViewModelImpl( cs.launchNow(Dispatchers.Main) { val allowsMultipleReviewers = allowsMultipleReviewers.first() val originalReviewersIds = adjustedReviewers.value.mapTo(mutableSetOf(), GitLabUserDTO::id) - val updatedReviewers = if (allowsMultipleReviewers == true) + val updatedReviewers = if (allowsMultipleReviewers) GitLabMergeRequestReviewersUtil.selectReviewers(point, originalReviewersIds, potentialReviewers, avatarIconProvider) else GitLabMergeRequestReviewersUtil.selectReviewer(point, originalReviewersIds, potentialReviewers, avatarIconProvider) diff --git a/plugins/gitlab/src/org/jetbrains/plugins/gitlab/mergerequest/ui/details/model/GitLabMergeRequestReviewFlowViewModel.kt b/plugins/gitlab/src/org/jetbrains/plugins/gitlab/mergerequest/ui/details/model/GitLabMergeRequestReviewFlowViewModel.kt index 6a37302c3635..de34cf3530b5 100644 --- a/plugins/gitlab/src/org/jetbrains/plugins/gitlab/mergerequest/ui/details/model/GitLabMergeRequestReviewFlowViewModel.kt +++ b/plugins/gitlab/src/org/jetbrains/plugins/gitlab/mergerequest/ui/details/model/GitLabMergeRequestReviewFlowViewModel.kt @@ -266,7 +266,7 @@ internal class GitLabMergeRequestReviewFlowViewModelImpl( @SinceGitLab("13.8") override fun setMyselfAsReviewer() = runAction { val allowsMultipleReviewers = allowsMultipleReviewers.first() - if (allowsMultipleReviewers == true) { + if (allowsMultipleReviewers) { mergeRequest.setReviewers(listOf(currentUser) + reviewers.value) } else {