comparison mainwindow.cpp @ 98:3e6c3281fb97

* Wire up the basic Hg actions again

 /* -*- c-basic-offset: 4 indent-tabs-mode: nil -*-  vi:set ts=8 sts=4 sw=4: */
 
 /*
     EasyMercurial
 
-    Based on HgExplorer by Jari Korhonen
+    Based on hgExplorer by Jari Korhonen
     Copyright (c) 2010 Jari Korhonen
     Copyright (c) 2010 Chris Cannam
     Copyright (c) 2010 Queen Mary, University of London
     
     This program is free software; you can redistribute it and/or

 
     readSettings();
 
     tabPage = 0;
     justMerged = false;
-    hgExp = new HgTabWidget((QWidget *) this, remoteRepoPath, workFolderPath);
-    setCentralWidget(hgExp);
-
-    connect(hgExp, SIGNAL(selectionChanged()),
+    hgTabs = new HgTabWidget((QWidget *) this, remoteRepoPath, workFolderPath);
+    setCentralWidget(hgTabs);
+
+    connect(hgTabs, SIGNAL(selectionChanged()),
             this, SLOT(enableDisableActions()));
 
     setUnifiedTitleAndToolBarOnMac(true);
     connectActions();
     enableDisableActions();

 {
    QMessageBox::about(this, tr("About EasyMercurial"),
                       tr("<qt><h2>About EasyMercurial</h2>"
                          "<p>EasyMercurial is a simple user interface for the "
                          "Mercurial version control system.</p>"
-                         "<p>EasyMercurial is based on HgExplorer by "
+                         "<p>EasyMercurial is based on hgExplorer by "
                          "Jari Korhonen, with thanks.<br>EasyMercurial development carried out by "
                          "Chris Cannam for soundsoftware.ac.uk at the Centre for Digital Music, Queen Mary, University of London."
                          "<ul><li>Copyright &copy; 2010 Jari Korhonen</li>"
                          "<li>Copyright &copy; 2010 Chris Cannam</li>"
                          "<li>Copyright &copy; 2010 Queen Mary, University of London</li>"

                          "COPYING included with this distribution for more information."));
 }
 
 void MainWindow::clearSelections()
 {
-    hgExp->clearSelections();
+    hgTabs->clearSelections();
 }
 
 void MainWindow::hgStat()
 {
     if (runningAction == ACT_NONE)

         runningAction = ACT_PARENTS;
     }
 }
 
 
-
-void MainWindow::hgRemove()
-{
-    if (runningAction == ACT_NONE)
-    {
-        QStringList params;
-        QString currentFile;//!!! = hgExp -> getCurrentFileListLine();
-
-        if (!currentFile.isEmpty())
-        {
-            if (QMessageBox::Ok == QMessageBox::warning(this, "Remove file", "Really remove file " + currentFile.mid(2) + "?",
-                QMessageBox::Ok | QMessageBox::Cancel, QMessageBox::Cancel))
-            {
-                params << "remove" << "--after" << "--force" << "--" << currentFile.mid(2);   //Jump over status marker characters (e.g "M ")
-
-                runner -> startHgCommand(workFolderPath, params);
-                runningAction = ACT_REMOVE;
-            }
-        }
-    }
-}
-
 void MainWindow::hgAnnotate()
 {
     if (runningAction == ACT_NONE)
     {
         QStringList params;
-        QString currentFile;//!!! = hgExp -> getCurrentFileListLine();
+        QString currentFile;//!!! = hgTabs -> getCurrentFileListLine();
 
         if (!currentFile.isEmpty())
         {
             params << "annotate" << "--" << currentFile.mid(2);   //Jump over status marker characters (e.g "M ")
 

 void MainWindow::hgResolveMark()
 {
     if (runningAction == ACT_NONE)
     {
         QStringList params;
-        QString currentFile;//!!! = hgExp -> getCurrentFileListLine();
+        QString currentFile;//!!! = hgTabs -> getCurrentFileListLine();
 
         if (!currentFile.isEmpty())
         {
             params << "resolve" << "--mark" << "--" << currentFile.mid(2);   //Jump over status marker characters (e.g "M ")
 

 {
     if (runningAction == ACT_NONE)
     {
         QStringList params;
 
-        QString currentFile;//!!! = hgExp -> getCurrentFileListLine();
+        // hgExplorer permitted adding "all" files -- I'm not sure
+        // that one is a good idea, let's require the user to select
+
+        QStringList files = hgTabs->getSelectedAddableFiles();
+
+        if (!files.empty()) {
+            params << "add" << "--" << files;
+            runner -> startHgCommand(workFolderPath, params);
+            runningAction = ACT_ADD;
+        }
+    }
+}
+
+
+void MainWindow::hgRemove()
+{
+    if (runningAction == ACT_NONE)
+    {
+        QStringList params;
+
+        QStringList files = hgTabs->getSelectedRemovableFiles();
+
+        //!!! todo: confirmation dialog (with file list in it)
+
+        if (!files.empty()) {
+
+            params << "remove" << "--after" << "--force" << "--" << files;
+            runner -> startHgCommand(workFolderPath, params);
+            runningAction = ACT_REMOVE;
+        }
+
 /*!!!
-        if (areAllSelectedUntracked(hgExp -> workFolderFileList))
-        {
-            //User wants to add selected file(s)
-            params << "add" << "--";
-
-            QList <QListWidgetItem *> selList = hgExp -> workFolderFileList -> selectedItems();
-
-            for (int i = 0; i < selList.size(); ++i)
-            {
-                QString tmp = selList.at(i)->text();
-                params.append(tmp.mid(2));
-            }
-        }
-        else
-        {
-            //Add all untracked files
-            params << "add";
-        }
-
-        runner -> startHgCommand(workFolderPath, params);
-        runningAction = ACT_ADD;
-    */
-    }
-}
+        QString currentFile;//!!! = hgTabs -> getCurrentFileListLine();
+
+        if (!currentFile.isEmpty())
+        {
+            if (QMessageBox::Ok == QMessageBox::warning(this, "Remove file", "Really remove file " + currentFile.mid(2) + "?",
+                QMessageBox::Ok | QMessageBox::Cancel, QMessageBox::Cancel))
+            {
+                params << "remove" << "--after" << "--force" << "--" << currentFile.mid(2);   //Jump over status marker characters (e.g "M ")
+
+                runner -> startHgCommand(workFolderPath, params);
+                runningAction = ACT_REMOVE;
+            }
+        }
+        */
+    }
+}
+
 
 bool MainWindow::getCommentOrTag(QString& commentOrTag,
                                  QString question,
                                  QString dlgTitle)
 {
     bool ok = false;
     QString text = QInputDialog::getText(this, dlgTitle, question, QLineEdit::Normal, commentOrTag, &ok);
     commentOrTag = text;
     return ok;
-
-            /*!!!
-    int ret;
-
-    QDialog dlg(this);
-
-    QLabel *commentLabel = new QLabel(question);
-    QLineEdit *commentOrTagEdit = new QLineEdit;
-    commentOrTagEdit -> setFixedWidth(400);
-    QHBoxLayout *commentLayout = new QHBoxLayout;
-    commentLayout -> addWidget(commentLabel);
-    commentLayout -> addWidget(commentOrTagEdit);
-
-    QPushButton *btnOk = new QPushButton(tr("Ok"));
-    QPushButton *btnCancel = new QPushButton(tr("Cancel"));
-    QHBoxLayout *btnLayout = new QHBoxLayout;
-    btnLayout -> addWidget(btnOk);
-    btnLayout -> addWidget(btnCancel);
-
-    QVBoxLayout *mainLayout = new QVBoxLayout;
-    mainLayout -> addLayout(commentLayout);
-    mainLayout -> addLayout(btnLayout);
-
-    dlg.setLayout(mainLayout);
-
-    dlg.setWindowTitle(dlgTitle);
-
-    connect(btnOk, SIGNAL(clicked()), &dlg, SLOT(accept()));
-    connect(btnCancel, SIGNAL(clicked()), &dlg, SLOT(reject()));
-
-    ret = dlg.exec();
-    commentOrTag = commentOrTagEdit -> text();
-    return ret;
-    */
 }
 
 void MainWindow::hgCommit()
 {
     //!!! Now that hg actions can be fired asynchronously (e.g. from

         QStringList params;
         QString comment;
         
         if (getCommentOrTag(comment, tr("Comment:"), tr("Save (commit)")))
         {
-            if (!comment.isEmpty())
-            {
-                /*!!!
-                if ((justMerged == false) && (areAllSelectedCommitable(hgExp -> workFolderFileList)))
-                {
-                    //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() << "--";
-
-                    QList <QListWidgetItem *> selList = hgExp -> workFolderFileList -> selectedItems();
-                    for (int i = 0; i < selList.size(); ++i)
-                    {
-                        QString tmp = selList.at(i)->text();
-                        params.append(tmp.mid(2));
-                    }
-                }
-                else
-                {
-                    //Commit all changes
-                    params << "commit" << "--message" << comment << "--user" << getUserInfo();
-                }
-
-                runner -> startHgCommand(workFolderPath, params);
-                runningAction = ACT_COMMIT;
-                */
-            }
+            //!!! do something more sensible when the comment is empty
+            // (i.e. tell the user about it?)
+
+            QStringList files = hgTabs->getSelectedCommittableFiles();
+
+            if ((justMerged == false) && //!!! review usage of 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();
+            }
+
+            runner -> startHgCommand(workFolderPath, params);
+            runningAction = ACT_COMMIT;
         }
     }
 }
 
 QString MainWindow::filterTag(QString tag)

 {
     if (runningAction == ACT_NONE)
     {
         QStringList params;
 /*!!!
-        QString currentFile = hgExp -> getCurrentFileListLine();
+        QString currentFile = hgTabs -> getCurrentFileListLine();
 
         if (!currentFile.isEmpty())
         {
             //Diff parent file against working folder file
             params << "kdiff3" << "--" << currentFile.mid(2);

 
         //Diff 2 history log versions against each other
         QString revA;
         QString revB;
 /*!!!
-        hgExp -> getHistoryDiffRevisions(revA, revB);
+        hgTabs -> getHistoryDiffRevisions(revA, revB);
 
         if ((!revA.isEmpty()) && (!revB.isEmpty()))
         {
             params << "kdiff3" << "--rev" << revA << "--rev" << revB;
             runner -> startHgCommand(workFolderPath, params);

 {
     if (runningAction == ACT_NONE)
     {
         QStringList params;
 
-
         params << "update";
-
 
         runner -> startHgCommand(workFolderPath, params);
         runningAction = ACT_UPDATE;
     }
 }

     if (runningAction == ACT_NONE)
     {
         QStringList params;
         QString rev;
 /*!!!
-        hgExp -> getUpdateToRevRevision(rev);
-
-        hgExp -> setCurrentIndex(WORKTAB);
+        hgTabs -> getUpdateToRevRevision(rev);
+
+        hgTabs -> setCurrentIndex(WORKTAB);
         enableDisableActions();
 
         params << "update" << "--rev" << rev << "--clean";
 
         runner -> startHgCommand(workFolderPath, params);

 
 void MainWindow::hgRevert()
 {
     if (runningAction == ACT_NONE)
     {
-        /*!!!
-        QStringList params;
-        QString currentFile = hgExp -> getCurrentFileListLine();
-
-        params << "revert" << "--no-backup" << "--" << currentFile.mid(2);
+        //!!! todo: ask user!
+
+        QStringList params;
+
+        QStringList files = hgTabs->getSelectedCommittableFiles();
+
+        if (files.empty()) {
+            params << "revert" << "--no-backup";
+        } else {
+            params << "revert" << "--no-backup" << "--" << files;
+        }
 
         runner -> startHgCommand(workFolderPath, params);
         runningAction = ACT_REVERT;
-        */
     }
 }
 
 void MainWindow::hgRetryMerge()
 {

 {
 /*!!!
     SettingsDialog *settingsDlg = new SettingsDialog(this);
     settingsDlg->setModal(true);
     settingsDlg->exec();
-    hgExp -> clearLists();
+    hgTabs -> clearLists();
     enableDisableActions();
     hgStat();
 */
 }
 

 
         dlg.exec();
     }
     else
     {
-        QMessageBox::information(this, tr("HgExplorer"), tr("Mercurial command did not return any output."));
+        QMessageBox::information(this, tr("EasyMercurial"), tr("Mercurial command did not return any output."));
     }
 }
 
 void MainWindow::updateFileSystemWatcher()
 {

                         enableDisableActions();
                         break;
                     }
 
                     case ACT_STAT:
-                        hgExp -> updateWorkFolderFileList(runner -> getOutput());
+                        hgTabs -> updateWorkFolderFileList(runner -> getOutput());
                         updateFileSystemWatcher();
                         break;
 
                     case ACT_INCOMING:
                     case ACT_ANNOTATE:

                         enableDisableActions();
                         shouldHgStat = true;
                         break;
 
                     case ACT_LOG:
-                        hgExp -> updateLocalRepoHgLogList(runner -> getOutput());
+                        hgTabs -> updateLocalRepoHgLogList(runner -> getOutput());
                         break;
 
                     case ACT_PARENTS:
                         {
-                        //!!!    hgExp -> updateLocalRepoParentsList(runner -> getOutput());
+                        //!!!    hgTabs -> updateLocalRepoParentsList(runner -> getOutput());
                         }
                         break;
 
                     case ACT_HEADS:
                         {
                             QString stdOut = runner -> getOutput();
-                        //!!!    hgExp -> updateLocalRepoHeadsList(stdOut);
+                        //!!!    hgTabs -> updateLocalRepoHeadsList(stdOut);
                         }
                         break;
 
                     case ACT_REMOVE:
                     case ACT_ADD:

     connect(exitAct, SIGNAL(triggered()), this, SLOT(close()));
     connect(aboutAct, SIGNAL(triggered()), this, SLOT(about()));
     connect(aboutQtAct, SIGNAL(triggered()), qApp, SLOT(aboutQt()));
 
     connect(hgStatAct, SIGNAL(triggered()), this, SLOT(hgPaths()));
-    connect(hgExp, SIGNAL(workFolderViewTypesChanged()), this, SLOT(hgPaths()));
+    connect(hgTabs, SIGNAL(workFolderViewTypesChanged()), this, SLOT(hgPaths()));
     connect(hgRemoveAct, SIGNAL(triggered()), this, SLOT(hgRemove()));
     connect(hgAddAct, SIGNAL(triggered()), this, SLOT(hgAdd()));
     connect(hgCommitAct, SIGNAL(triggered()), this, SLOT(hgCommit()));
     connect(hgFileDiffAct, SIGNAL(triggered()), this, SLOT(hgFileDiff()));
     connect(hgFolderDiffAct, SIGNAL(triggered()), this, SLOT(hgFolderDiff()));

     connect(hgCloneFromRemoteAct, SIGNAL(triggered()), this, SLOT(hgCloneFromRemote()));
     connect(hgIncomingAct, SIGNAL(triggered()), this, SLOT(hgIncoming()));
     connect(hgPullAct, SIGNAL(triggered()), this, SLOT(hgPull()));
     connect(hgPushAct, SIGNAL(triggered()), this, SLOT(hgPush()));
 
-    connect(hgExp, SIGNAL(currentChanged(int)), this, SLOT(tabChanged(int)));
+    connect(hgTabs, SIGNAL(currentChanged(int)), this, SLOT(tabChanged(int)));
 
     connect(hgUpdateToRevAct, SIGNAL(triggered()), this, SLOT(hgUpdateToRev()));
     connect(hgAnnotateAct, SIGNAL(triggered()), this, SLOT(hgAnnotate()));
     connect(hgResolveListAct, SIGNAL(triggered()), this, SLOT(hgResolveList()));
     connect(hgResolveMarkAct, SIGNAL(triggered()), this, SLOT(hgResolveMark()));

     hgAnnotateAct -> setEnabled(localRepoActionsEnabled);
     hgServeAct -> setEnabled(localRepoActionsEnabled);
     hgTagAct -> setEnabled(localRepoActionsEnabled);
     hgIgnoreAct -> setEnabled(localRepoActionsEnabled);
 
-    //!!!hgExp -> enableDisableOtherTabs(tabPage);
+    //!!!hgTabs -> enableDisableOtherTabs(tabPage);
 
     DEBUG << "localRepoActionsEnabled = " << localRepoActionsEnabled << endl;
-    DEBUG << "canCommit = " << hgExp->canCommit() << endl;
+    DEBUG << "canCommit = " << hgTabs->canCommit() << endl;
 
     //!!! new stuff:
-    hgAddAct->setEnabled(localRepoActionsEnabled && hgExp->canAdd());
-    hgRemoveAct->setEnabled(localRepoActionsEnabled && hgExp->canRemove());
-    hgCommitAct->setEnabled(localRepoActionsEnabled && hgExp->canCommit());
-    hgRevertAct->setEnabled(localRepoActionsEnabled && hgExp->canCommit());
-    hgFolderDiffAct->setEnabled(localRepoActionsEnabled && hgExp->canDoDiff());
+    hgAddAct->setEnabled(localRepoActionsEnabled && hgTabs->canAdd());
+    hgRemoveAct->setEnabled(localRepoActionsEnabled && hgTabs->canRemove());
+    hgCommitAct->setEnabled(localRepoActionsEnabled && hgTabs->canCommit());
+    hgRevertAct->setEnabled(localRepoActionsEnabled && hgTabs->canCommit());
+    hgFolderDiffAct->setEnabled(localRepoActionsEnabled && hgTabs->canDoDiff());
 
 /*!!!
     int added, modified, removed, notTracked, selected, selectedAdded, selectedModified, selectedRemoved, selectedNotTracked;
 
-    countModifications(hgExp -> workFolderFileList,
+    countModifications(hgTabs -> workFolderFileList,
         added, modified, removed, notTracked,
         selected,
         selectedAdded, selectedModified, selectedRemoved, selectedNotTracked);
 
     if (tabPage == WORKTAB)

             if (modified == 0)
             {
                 hgFolderDiffAct -> setEnabled(false);
             }
 
-            if (!isSelectedModified(hgExp -> workFolderFileList))
+            if (!isSelectedModified(hgTabs -> workFolderFileList))
             {
                 hgFileDiffAct -> setEnabled(false);
                 hgRevertAct -> setEnabled(false);
             }
 

             if (notTracked == 0)
             {
                 hgAddAct -> setEnabled(false);
             }
 
-            if (!isSelectedDeletable(hgExp -> workFolderFileList))
+            if (!isSelectedDeletable(hgTabs -> workFolderFileList))
             {
                 hgRemoveAct -> setEnabled(false);
             }
 
             hgResolveListAct -> setEnabled(true);
 
-            if (hgExp -> localRepoHeadsList->count() < 2)
+            if (hgTabs -> localRepoHeadsList->count() < 2)
             {
                 hgMergeAct -> setEnabled(false);
                 hgRetryMergeAct -> setEnabled(false);
             }
 
-            if (hgExp -> localRepoHeadsList->count() < 1)
+            if (hgTabs -> localRepoHeadsList->count() < 1)
             {
                 hgTagAct -> setEnabled(false);
             }
 
-            QString currentFile = hgExp -> getCurrentFileListLine();
+            QString currentFile = hgTabs -> getCurrentFileListLine();
             if (!currentFile.isEmpty())
             {
                 hgAnnotateAct -> setEnabled(true);
                 hgResolveMarkAct -> setEnabled(true);
             }

             }
         }
     }
     else
     {
-        QList <QListWidgetItem *> headSelList = hgExp -> localRepoHeadsList->selectedItems();
-        QList <QListWidgetItem *> historySelList = hgExp -> localRepoHgLogList->selectedItems();
+        QList <QListWidgetItem *> headSelList = hgTabs -> localRepoHeadsList->selectedItems();
+        QList <QListWidgetItem *> historySelList = hgTabs -> localRepoHgLogList->selectedItems();
 
         if ((historySelList.count() == 2) || (headSelList.count() == 2))
         {
             hgChgSetDiffAct -> setEnabled(true);
         }

     settings.setValue("pos", pos());
     settings.setValue("size", size());
     settings.setValue("remoterepopath", remoteRepoPath);
     settings.setValue("workfolderpath", workFolderPath);
     settings.setValue("firststart", firstStart);
-    //!!!settings.setValue("viewFileTypes", hgExp -> getFileTypesBits());
-}
-
-
-
-
+    //!!!settings.setValue("viewFileTypes", hgTabs -> getFileTypesBits());
+}
+
+
+
+