From 4b2010bf38e98d84fc8d22720a412c728792fd53 Mon Sep 17 00:00:00 2001
From: Kirill Likhodedov <Kirill.Likhodedov@jetbrains.com>
Date: Tue, 11 Dec 2012 16:17:41 +0400
Subject: [PATCH] [git] IDEA-83559 Warn if user is about to commit unresolved
 conflicts

---
 plugins/git4idea/src/META-INF/plugin.xml      |  2 +
 .../checkin/UnresolvedMergeCheckFactory.java  | 59 +++++++++++++++++++
 2 files changed, 61 insertions(+)
 create mode 100644 plugins/git4idea/src/git4idea/checkin/UnresolvedMergeCheckFactory.java

diff --git a/plugins/git4idea/src/META-INF/plugin.xml b/plugins/git4idea/src/META-INF/plugin.xml
index ead2ca5e4a5c..2d3afc086997 100644
--- a/plugins/git4idea/src/META-INF/plugin.xml
+++ b/plugins/git4idea/src/META-INF/plugin.xml
@@ -93,6 +93,8 @@
     <checkoutProvider implementation="git4idea.checkout.GitCheckoutProvider"/>
     <vcsPopupProvider implementation="git4idea.actions.GitQuickListContentProvider"/>
     <vcsCheckinHandlerFactory implementation="git4idea.checkin.GitCheckinHandlerFactory"/>
+    <checkinHandlerFactory implementation="git4idea.checkin.UnresolvedMergeCheckFactory"/>
+
     <vcsChangesViewRefresher implementation="git4idea.changes.GitChangesViewRefresher" />
 
     <projectService
diff --git a/plugins/git4idea/src/git4idea/checkin/UnresolvedMergeCheckFactory.java b/plugins/git4idea/src/git4idea/checkin/UnresolvedMergeCheckFactory.java
new file mode 100644
index 000000000000..1388ee09a333
--- /dev/null
+++ b/plugins/git4idea/src/git4idea/checkin/UnresolvedMergeCheckFactory.java
@@ -0,0 +1,59 @@
+/*
+ * Copyright 2000-2012 JetBrains s.r.o.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package git4idea.checkin;
+
+import com.intellij.openapi.ui.Messages;
+import com.intellij.openapi.vcs.CheckinProjectPanel;
+import com.intellij.openapi.vcs.FileStatus;
+import com.intellij.openapi.vcs.changes.Change;
+import com.intellij.openapi.vcs.changes.CommitContext;
+import com.intellij.openapi.vcs.checkin.CheckinHandler;
+import com.intellij.openapi.vcs.checkin.CheckinHandlerFactory;
+import org.jetbrains.annotations.NotNull;
+
+/**
+ * Checks if there are unresolved conflicts selected to commit.
+ *
+ * @author Kirill Likhodedov
+ */
+public class UnresolvedMergeCheckFactory extends CheckinHandlerFactory {
+  @NotNull
+  @Override
+  public CheckinHandler createHandler(final CheckinProjectPanel panel, CommitContext commitContext) {
+    return new CheckinHandler() {
+      @Override
+      public ReturnResult beforeCheckin() {
+        if (containsUnresolvedConflicts(panel)) {
+          int answer = Messages.showYesNoDialog(panel.getComponent(), "Are you sure you want to commit changes with unresolved conflicts?",
+                                                "Unresolved Conflicts", Messages.getWarningIcon());
+          return answer == Messages.OK ? ReturnResult.COMMIT : ReturnResult.CANCEL;
+        }
+        return ReturnResult.COMMIT;
+      }
+    };
+  }
+
+  private static boolean containsUnresolvedConflicts(@NotNull CheckinProjectPanel panel) {
+    for (Change change : panel.getSelectedChanges()) {
+      FileStatus status = change.getFileStatus();
+      if (status.equals(FileStatus.MERGE) || status.equals(FileStatus.MERGED_WITH_BOTH_CONFLICTS) ||
+          status.equals(FileStatus.MERGED_WITH_CONFLICTS) || status.equals(FileStatus.MERGED_WITH_PROPERTY_CONFLICTS)) {
+        return true;
+      }
+    }
+    return false;
+  }
+}