diff --git a/bot-core/src/main/java/dev/struchkov/bot/gitlab/core/handler/MergeRequestHandler.java b/bot-core/src/main/java/dev/struchkov/bot/gitlab/core/handler/MergeRequestHandler.java index 4b3ab16..c91a5ad 100644 --- a/bot-core/src/main/java/dev/struchkov/bot/gitlab/core/handler/MergeRequestHandler.java +++ b/bot-core/src/main/java/dev/struchkov/bot/gitlab/core/handler/MergeRequestHandler.java @@ -26,6 +26,7 @@ import org.springframework.context.event.EventListener; import org.springframework.stereotype.Component; import java.util.List; +import java.util.Objects; import java.util.Optional; import java.util.Set; import java.util.stream.Collectors; @@ -169,7 +170,7 @@ public class MergeRequestHandler { final Long botUserGitlabId = personInformation.getId(); if ( !botUserGitlabId.equals(mergeRequest.getAuthor().getId()) // Автор MR не пользователь приложения - && !oldMergeRequest.getDateLastCommit().equals(mergeRequest.getDateLastCommit()) // Изменилась дата последнего коммита + && !Objects.equals(oldMergeRequest.getDateLastCommit(), mergeRequest.getDateLastCommit()) && !mergeRequest.isConflict() // MR не находится в состоянии конфликта && !botUserGitlabId.equals(oldMergeRequest.getAuthor().getId()) // и MR создан НЕ пользователем бота ) { @@ -239,7 +240,7 @@ public class MergeRequestHandler { final Long gitlabUserId = personInformation.getId(); if (oldMergeRequest.isConflict() && !mergeRequest.isConflict()) { // проверяем даты коммитов, так как при пуше в target ветку MR у которого есть конфликт, конфликт на время пропадает. Судя по всему GitLab после пуша заново проверяет вероятность конфликта. Чаще всего конфликт никуда не девается. - if (oldMergeRequest.getDateLastCommit().equals(mergeRequest.getDateLastCommit())) { + if (Objects.equals(oldMergeRequest.getDateLastCommit(), mergeRequest.getDateLastCommit())) { mergeRequest.setConflict(true); } else { if (gitlabUserId.equals(oldMergeRequest.getAuthor().getId())) {