Limit maximum amount of child items in treeview

Fix #3994

Author: uglide

.gitignore

@@ -80,4 +80,5 @@ parts
 prime
 stage
 3rdparty/python*
-__pycache__/
+__pycache__/
+qml_*.cpp

src/app/models/connectionsmanager.cpp

@@ -60,7 +60,7 @@ void ConnectionsManager::updateGroup(const ConnectionGroup &group)
         return;
     }
 
-    onItemChanged(serverGroup);
+    itemChanged(serverGroup);
 
     saveConfig();
 
@@ -302,7 +302,7 @@ void ConnectionsManager::createServerItemForConnection(
       [this, serverItem]() {
         if (!serverItem) return;
 
-        onItemChanged(
+        itemChanged(
             serverItem.dynamicCast<ConnectionsTree::TreeItem>().toWeakRef());
       });
 

src/app/models/treeoperations.cpp

@@ -183,57 +183,41 @@ QFuture<void> TreeOperations::getDatabases(
 }
 
 void TreeOperations::loadNamespaceItems(
-    QSharedPointer<ConnectionsTree::AbstractNamespaceItem> parent,
-    const QString& filter, std::function<void(const QString& err)> callback,
-    QSet<QByteArray> expandedNs) {
+    uint dbIndex,
+    const QString& filter,
+    std::function<void(const RedisClient::Connection::RawKeysList& keylist,
+                       const QString& err)>
+        callback) {
   QString keyPattern = filter.isEmpty() ? m_config.keysPattern() : filter;
 
   if (m_filterHistory.contains(keyPattern)) {
-      m_filterHistory[keyPattern] = m_filterHistory[keyPattern].toInt() + 1;
+    m_filterHistory[keyPattern] = m_filterHistory[keyPattern].toInt() + 1;
   } else {
-      m_filterHistory[keyPattern] = 1;
+    m_filterHistory[keyPattern] = 1;
   }
   m_config.setFilterHistory(m_filterHistory);
   emit filterHistoryUpdated();
 
-  auto renderingCallback =
-      [this, callback, filter, parent, expandedNs](
-          const RedisClient::Connection::RawKeysList& keylist,
-          const QString& err) {
-        if (!err.isEmpty()) {
-          return callback(err);
-        }
-
-        auto settings = ConnectionsTree::KeysTreeRenderer::RenderingSettigns{
-            QRegExp(filter), getNamespaceSeparator(), parent->getDbIndex(),
-            true};
-
-        AsyncFuture::observe(
-            QtConcurrent::run(&ConnectionsTree::KeysTreeRenderer::renderKeys,
-                              sharedFromThis(), keylist, parent, settings,
-                              expandedNs))
-            .subscribe([callback]() { callback(QString()); });
-      };
-
   if (!connect(m_connection)) return;
 
   auto processErr = [callback](const QString& err) {
     return callback(
+        RedisClient::Connection::RawKeysList(),
         QCoreApplication::translate("RDM", "Cannot load keys: %1").arg(err));
-  };  
+  };
 
   try {
     if (m_connection->mode() == RedisClient::Connection::Mode::Cluster) {
-      m_connection->getClusterKeys(renderingCallback, keyPattern);
+      m_connection->getClusterKeys(callback, keyPattern);
     } else {
       m_connection->cmd(
-          {"ping"}, this, parent->getDbIndex(),
-          [this, callback, renderingCallback, keyPattern,
+          {"ping"}, this, dbIndex,
+          [this, callback, keyPattern,
            processErr](const RedisClient::Response& r) {
             if (r.isErrorMessage()) {
               return processErr(r.value().toString());
             }
-            m_connection->getDatabaseKeys(renderingCallback, keyPattern, -1);
+            m_connection->getDatabaseKeys(callback, keyPattern, -1);
           },
           [processErr](const QString& err) { return processErr(err); });
     }
@@ -394,7 +378,8 @@ void TreeOperations::openKeyIfExists(
         if (result.toByteArray() == "1") {
           auto key = QSharedPointer<ConnectionsTree::KeyItem>(
               new ConnectionsTree::KeyItem(fullPath, parent.toWeakRef(),
-                                           parent->model()));
+                                           parent->model(),
+                                           parent->keysShortNameRendering()));
 
           emit m_events->openValueTab(m_connection, key, true);
 

src/app/models/treeoperations.h

@@ -27,9 +27,10 @@ class TreeOperations : public QObject,
   QFuture<void> getDatabases(std::function<void (RedisClient::DatabaseList, const QString&)>) override;
 
   void loadNamespaceItems(
-      QSharedPointer<ConnectionsTree::AbstractNamespaceItem> parent,
-      const QString& filter, std::function<void(const QString& err)> callback,
-      QSet<QByteArray> expandedNs) override;
+      uint dbIndex, const QString& filter,
+      std::function<void(const RedisClient::Connection::RawKeysList& keylist,
+                         const QString& err)>
+          callback) override;
 
   void disconnect() override;