Added initial wizard

Small refactoring

Keep going!

commitmodel.cpp

@@ -26,7 +26,7 @@ CommitModel* CommitModel::fromBranch(GitBranch* branch)
         GitOid commitOid(&newOid, branch->repository());
         GitCommit *commit = GitCommit::fromOid(commitOid);
         if(commit != nullptr) {
-            tmpModel->add(commit);
+            tmpModel->append(commit);
         } else {
             qDebug() << "Commit is null";
         }

githandler.cpp

@@ -6,6 +6,7 @@
 #include <QClipboard>
 #include <QtConcurrentRun>
 #include <QDebug>
+#include <QDir>
 #include <qqml.h>
 
 #include <gitrepository.h>
@@ -34,6 +35,7 @@ GitHandler::GitHandler() : QObject()
   ,m_tagList(new TagListModel(this))
   ,m_activeRepoWatcher(new QFileSystemWatcher(this))
   ,m_console(new GitConsole(this))
+  ,m_homePath(QUrl::fromLocalFile(QDir::homePath()))
 {
     git_libgit2_init();
     connect(&m_diffTask, &QFutureWatcher<GitDiff*>::finished, this, &GitHandler::onDiffReady);

githandler.h

@@ -27,6 +27,7 @@ class GitHandler : public QObject
     Q_PROPERTY(TagListModel* tagList READ tagList CONSTANT)
     Q_PROPERTY(GitConsole* console READ console CONSTANT)
     Q_PROPERTY(bool isBusy READ isBusy NOTIFY isBusyChanged)
+    Q_PROPERTY(QUrl homePath READ homePath CONSTANT)
 
 public:
     GitHandler();
@@ -87,6 +88,11 @@ public:
         return !m_graphTask.isCanceled() && !m_graphTask.isFinished();
     }
 
+    QUrl homePath() const
+    {
+        return m_homePath;
+    }
+
 public slots:
     void setActiveDiff(GitDiff* activeDiff);
 
@@ -131,6 +137,7 @@ private:
     QFutureWatcher<GitDiff*> m_diffTask;
     QFutureWatcher<CommitGraph*> m_graphTask;
     bool m_isBusy;
+    QUrl m_homePath;
 
     //    GitOid m_constantHead; //TODO:
 };

qml/InitialWizard.qml

@@ -0,0 +1,31 @@
+import QtQuick 2.0
+import QtQuick.Dialogs 1.2
+import QtQuick.Controls 1.4
+import QtQuick.Controls.Styles 1.4
+
+FocusScope {
+    Rectangle {
+        anchors.fill: parent
+    }
+
+    Text {
+        textFormat: Text.RichText
+        font.pointSize: 10
+        text: qsTr("Hi!<\br> Seems you started CuteGit for the first time. So lets <a href=\"#open\">open</a> one of your repository.");
+        onLinkActivated: {
+            if(link == "#open") {
+                repoOpenDialog.open()
+            }
+        }
+    }
+
+    FileDialog {
+        id: repoOpenDialog
+        folder: _handler.homePath
+        selectFolder: true
+        selectMultiple: false
+        onAccepted: {
+            _handler.open(repoOpenDialog.fileUrl, true)
+        }
+    }
+}

qml/MainView.qml

@@ -146,4 +146,9 @@ FocusScope {
 
     Tooltip {
     }
+
+    InitialWizard {
+        visible: _handler.repositories.count <= 0
+        anchors.fill: parent
+    }
 }

qml/TopBar.qml

@@ -42,7 +42,10 @@ Item {
         Button {
             anchors.verticalCenter: parent.verticalCenter
             text: qsTr("Add...")
-            onClicked: repoOpenDialog.open()
+            onClicked: {
+                console.log("repoOpenDialog.folder: " + repoOpenDialog.folder)
+                repoOpenDialog.open()
+            }
         }
     }
 
@@ -78,7 +81,7 @@ Item {
 
     FileDialog {
         id: repoOpenDialog
-        folder: "."
+        folder: _handler.homePath
         selectFolder: true
         selectMultiple: false
         onAccepted: {

repositorymodel.cpp

@@ -19,7 +19,7 @@ void RepositoryModel::addRepository(GitRepository *repository)
     }
 
     m_repolist.append(repository->name());
-    add(repository);
+    append(repository);
     setActiveRepositoryIndex(m_repolist.count() - 1);
 }
 

resources.qrc

@@ -53,5 +53,6 @@
         <file>qml/Separator.qml</file>
         <file>images/question-mark-4-24.png</file>
         <file>images/question-mark-4-24_active.png</file>
+        <file>qml/InitialWizard.qml</file>
     </qresource>
 </RCC>

settings.cpp

@@ -17,7 +17,7 @@ Settings::Settings() : QObject()
 {
 }
 
-void Settings::load(QStringList& repos)
+void Settings::load(QStringList &repos)
 {
     QStringList groups = m_settings.childGroups();
 
@@ -34,7 +34,7 @@ void Settings::load(QStringList& repos)
     }
 }
 
-void Settings::add(GitRepository* repo)
+void Settings::add(GitRepository *repo)
 {
     if(repo == nullptr) {
         return;
@@ -46,7 +46,7 @@ void Settings::add(GitRepository* repo)
     m_settings.endGroup();
 }
 
-void Settings::saveLastRepo(GitRepository* repo)
+void Settings::saveLastRepo(GitRepository *repo)
 {
     m_settings.beginGroup(GeneralGroupKey);
     m_settings.setValue(LastRepoKey, repo->id());

settings.h

@@ -15,10 +15,10 @@ public:
         return &settings;
     }
 
-    void load(QStringList& repos);
-    void add(GitRepository* repo);
+    void load(QStringList &repos);
+    void add(GitRepository *repo);
 
-    void saveLastRepo(GitRepository* activeRepoId);
+    void saveLastRepo(GitRepository *activeRepoId);
     QString loadLastRepo();
 
 private:

universallistmodel.h

@@ -1,5 +1,4 @@
-#ifndef UNIVERSALLISTMODEL_H
-#define UNIVERSALLISTMODEL_H
+#pragma once
 
 #include <QAbstractListModel>
 
@@ -12,6 +11,11 @@
 
 #include <QDebug>
 
+
+/*! Universal list model is QObject-base list model abstraction.
+ * It exposes all objects properties as data-roles.
+ *
+ */
 template <typename T>
 class UniversalListModel : public QAbstractListModel
 {
@@ -53,21 +57,50 @@ public:
         return dataPtr->property(s_roleNames.value(role));
     }
 
-    bool add(T* value) {
+    /*!
+     * \brief append
+     * \param value
+     * \return
+     */
+    int append(T* value) {
         Q_ASSERT_X(value != nullptr, fullTemplateName(), "Trying to add member of NULL");
 
         if(m_container.indexOf(value) >= 0) {
 #ifdef DEBUG
             qDebug() << fullTemplateName() << "Member already exists";
 #endif
-            return false;
+            return -1;
         }
         beginInsertRows(QModelIndex(), m_container.count(), m_container.count());
         m_container.append(QPointer<T>(value));
         endInsertRows();
-        return true;
+        return m_container.count() - 1;
+    }
+
+    /*!
+     * \brief prepend
+     * \param value
+     * \return
+     */
+    int prepend(T* value) {
+        Q_ASSERT_X(value != nullptr, fullTemplateName(), "Trying to add member of NULL");
+
+        if(m_container.indexOf(value) >= 0) {
+#ifdef DEBUG
+            qDebug() << fullTemplateName() << "Member already exists";
+#endif
+            return -1;
+        }
+        beginInsertRows(QModelIndex(), 0, 0);
+        m_container.prepend(QPointer<T>(value));
+        endInsertRows();
+        return 0;
     }
 
+    /*!
+     * \brief remove
+     * \param value
+     */
     void remove(T* value) {
         Q_ASSERT_X(value != nullptr, fullTemplateName(), ": Trying to remove member of NULL");
 
@@ -80,6 +113,11 @@ public:
         }
     }
 
+    /*!
+     * \brief Resets container with new container passed as parameter
+     * \param container a data for model. Should contain QPointer's to objects.
+     * Passing empty container makes model empty. This method should be used to cleanup model.
+     */
     void reset(const QList<QPointer<T> >& container) {
         beginResetModel();
         clear();
@@ -87,10 +125,21 @@ public:
         endResetModel();
     }
 
+    /*!
+     * \brief Returns the item at index position i in the list. i must be a valid index position in the list (i.e., 0 <= i < rowCount()).
+     * This function is very fast (constant time).
+     * \param i index of looking object
+     * \return Object at provided index
+     */
     T* at(int i) const {
         return m_container.at(i);
     }
 
+    /*!
+     * \brief Looking for index of objec
+     * \param value
+     * \return
+     */
     int indexOf(T* value) const {
         return m_container.indexOf(value);
     }
@@ -98,12 +147,6 @@ public:
 
 protected:
     void clear() {
-//TODO: need to verify if wee really should cleanup these commits.
-//        foreach (const QPointer<T>& value, m_container) {
-//            if(!value.isNull()) {
-//                delete value.data();
-//            }
-//        }
         m_container.clear();
     }
 
@@ -119,5 +162,3 @@ private:
 
 template<typename T>
 QHash<int, QByteArray> UniversalListModel<T>::s_roleNames;
-
-#endif // UNIVERSALLISTMODEL_H