Remove signals from RM::Repo

Author: scunz

Libs/libMacGitverCore/RepoMan/Frontend/Repo.cpp

@@ -199,7 +199,8 @@ namespace RM
 
         if (d->mDisplayAlias != alias) {
             d->mDisplayAlias = alias;
-            emit aliasChanged(alias);
+            // ###REPOMAN Create new Event "RepoAliasChanged"
+            // emit aliasChanged(alias);
         }
     }
 
@@ -218,7 +219,6 @@ namespace RM
         }
 
         Events::self()->repositoryAboutToClose(this);
-        emit aboutToClose(this);
 
         foreach (Repo* child, submodules()) {
             child->close();
@@ -421,13 +421,10 @@ namespace RM
             mUnloadTimer = NULL;
         }
 
-        Repo* r = pub<Repo>();
-        emit r->aboutToUnload(r);
-
+        // ####REPOMAN Do we really need to send out events for Unload/Load? If yes, create
+        //             real events for that!
         mIsLoaded = false;
         mRepo = Git::Repository();
-
-        emit r->unloaded(r);
     }
 
 
@@ -474,10 +471,7 @@ namespace RM
 
             subInfo = repoByPath(path, true);
             if (!subInfo) {
-
                 subInfo = new Submodule(subRepo, p);
-                emit p->childAdded(p, subInfo);
-
             }
             else {
                 oldSubmodules.remove(subInfo);

Libs/libMacGitverCore/RepoMan/Repo.hpp

@@ -98,14 +98,6 @@ namespace RM
 
     private slots:
         void unloadTimer();
-
-    signals:
-        void aboutToClose   (RM::Repo* repo);
-        void aboutToUnload  (RM::Repo* repo);
-        void unloaded       (RM::Repo* repo);
-        void loaded         (RM::Repo* repo);
-        void childAdded     (RM::Repo* parent, RM::Repo* child);
-        void aliasChanged   (const QString& newAlias);
     };
 
 }

Modules/Repository/RepoInfoModel.cpp

@@ -197,11 +197,13 @@ void RepoInfoModel::repositoryOpened(RM::Repo *info)
         return;
     }
 
+    // ###REPOMAN Instead of this, listen to SubmoduleCreated events
+
     // This connection is wrong. There could be any children added below the repository (i.e.
     // HEAD, SubModule, CollectionNode). But the slot will unconditionally reserve a new space
     // for a submodule...
-    connect(info, &RM::Repo::childAdded,
-            this, &RepoInfoModel::repositoryChildAdded);
+    //connect(info, &RM::Repo::childAdded,
+    //        this, &RepoInfoModel::repositoryChildAdded);
 
     // we add a row just at the end of the root. This is stupid. But that's the way it works when
     // a model actually isn't a model...