comparison mainwindow.cpp @ 157:e411bb42d934

* Default merge comments

 void MainWindow::hgCommit()
 {
     QStringList params;
     QString comment;
 
+    if (justMerged) {
+        comment = mergeCommitComment;
+    }
+
     QStringList files = hgTabs->getSelectedCommittableFiles();
     QStringList reportFiles = files;
     if (reportFiles.empty()) reportFiles = hgTabs->getAllCommittableFiles();
 
     QString cf(tr("Commit files"));

          tr("<h3>%1</h3><p>%2").arg(cf)
          .arg(tr("You are about to commit %n file(s):", "", reportFiles.size())),
          reportFiles,
          comment)) {
 
-        if ((justMerged == false) && //!!! review usage of justMerged, and how it interacts with asynchronous request queue
-            !files.empty()) {
-            // User wants to commit selected file(s) (and this is not merge commit, which would fail if we selected files)
-            params << "commit" << "--message" << comment << "--user" << getUserInfo() << "--" << files;
+        if (!justMerged && !files.empty()) {
+            // User wants to commit selected file(s) (and this is not
+            // merge commit, which would fail if we selected files)
+            params << "commit" << "--message" << comment
+                   << "--user" << getUserInfo() << "--" << files;
         } else {
             // Commit all changes
-            params << "commit" << "--message" << comment << "--user" << getUserInfo();
+            params << "commit" << "--message" << comment
+                   << "--user" << getUserInfo();
         }
         
         runner->requestAction(HgAction(ACT_COMMIT, workFolderPath, params));
     }
+    
+    justMerged = false;
 }
 
 QString MainWindow::filterTag(QString tag)
 {
     for(int i = 0; i < tag.size(); i++)

     QStringList params;
 
     params << "merge";
     
     runner->requestAction(HgAction(ACT_MERGE, workFolderPath, params));
+
+    mergeCommitComment = tr("Merge");
 }
 
 
 void MainWindow::hgMergeFrom(QString id)
 {

 
     params << "merge";
     params << "--rev" << Changeset::hashOf(id);
     
     runner->requestAction(HgAction(ACT_MERGE, workFolderPath, params));
+
+    mergeCommitComment = "";
+
+    foreach (Changeset *cs, currentHeads) {
+        if (cs->id() == id && !cs->isOnBranch(currentBranch)) {
+            if (cs->branch() == "" || cs->branch() == "default") {
+                mergeCommitComment = tr("Merge from the default branch");
+            } else {
+                mergeCommitComment = tr("Merge from branch \"%1\"").arg(cs->branch());
+            }
+        }
+    }
+
+    if (mergeCommitComment == "") {
+        mergeCommitComment = tr("Merge from %1").arg(id);
+    }
 }
 
 
 void MainWindow::hgCloneFromRemote()
 {

     //!!! Want "merge failed" report
 
     if (canMerge) {
         hgTabs->setState(tr("<b>Awaiting merge</b> on %1").arg(branchText));
     } else if (haveMerge) {
-        hgTabs->setState(tr("Have merged but not committed on %1").arg(branchText));
+        hgTabs->setState(tr("Have merged but not yet committed on %1").arg(branchText));
     } else if (canUpdate) {
         hgTabs->setState(tr("On %1. Not at the head of the branch: consider updating").arg(branchText));
     } else if (currentBranchHeads > 1) {
         hgTabs->setState(tr("At one of %n heads of %1", "", currentBranchHeads).arg(branchText));
     } else {