Fix for #1529: Pagination does not work when the aggregation queries have dotted fields

Author: simsekgokhan

src/robomongo/core/domain/MongoShell.h

@@ -27,6 +27,7 @@ namespace Robomongo
         std::string dbname() const { return _scriptInfo.dbname(); }
         const CursorPosition &cursor() const { return _scriptInfo.cursor(); }
         void setScript(const QString &script) { return _scriptInfo.setScript(script); }
+        void setScriptExecutable(bool execute) { return _scriptInfo.setExecutable(execute); }
         void setAggrInfo(AggrInfo const& aggrInfo) { _aggrInfo = aggrInfo; }
         QString filePath() const { return _scriptInfo.filePath(); }
 

src/robomongo/core/domain/ScriptInfo.h

@@ -14,6 +14,7 @@ namespace Robomongo
                    const QString &title = QString(), const QString &filePath = QString());
 
         bool execute() const { return _execute; }
+        void setExecutable(bool execute) { _execute = execute; }
         QString script() const { return _script; }
         std::string dbname() const { return _dbname; }
         const QString &title() const { return _title; }
@@ -28,7 +29,7 @@ namespace Robomongo
     private:
         QString _script;
         std::string _dbname;
-        const bool _execute;
+        bool _execute;
         const QString _title;
         const CursorPosition _cursor;
         QString _filePath;

src/robomongo/gui/widgets/workarea/OutputItemContentWidget.cpp

@@ -183,14 +183,17 @@ namespace Robomongo
         info._skip = skip;
         info._batchSize = batchSize;
         _outputWidget->showProgress();
-        
+                
+        _shell->setScriptExecutable(true);
         if (_aggrInfo.isValid) {
             // Build original pipeline object, and append extra skip and limit for paging
             std::string pipelineModified = "[";
-            int i = 0;
-            while (!_aggrInfo.pipeline.getObjectField(std::to_string(i)).isEmpty()) {
-                pipelineModified.append(_aggrInfo.pipeline.getObjectField(std::to_string(i)).toString() + ",");
-                ++i;
+            for (int i = 0; ; i++) {
+                auto const obj = mongo::tojson(_aggrInfo.pipeline.getObjectField(std::to_string(i)));
+                if (obj.empty() || "{}" == obj)
+                    break;
+
+                pipelineModified.append(obj + ",");
             }
             pipelineModified.append("{$skip:" + std::to_string(skip) + "}, " +
                                     "{$limit:" + std::to_string(batchSize) + "}" +