creating rows changed into promises

frontend-js/src/main/js/gui/export/AbstractExportPanel.js

@@ -361,7 +361,7 @@ AbstractExportPanel.prototype._getAllCompartments = function () {
 /**
  *
  * @returns {HTMLElement}
- * @private
+ * @protected
  */
 AbstractExportPanel.prototype._createDownloadButton = function () {
   var self = this;
@@ -445,7 +445,7 @@ AbstractExportPanel.prototype._createSelectIncludedCompartmentDiv = function ()
 /**
  *
  * @returns {Promise<HTMLElement>}
- * @private
+ * @protected
  */
 AbstractExportPanel.prototype._createSelectExcludedCompartmentDiv = function () {
   var self = this;
@@ -563,8 +563,8 @@ AbstractExportPanel.prototype.getSelectedExcludedCompartments = function () {
 /**
  *
  * @param {{name:string}[]} columns
- * @param {MiriamType[]> miriamTypes
- * @returns {string}
+ * @param {MiriamType[]} miriamTypes
+ * @returns {Promise<string>}
  */
 AbstractExportPanel.prototype.createResponseHeader = function (columns, miriamTypes) {
   var stringBuilder = [];

frontend-js/src/main/js/gui/export/ElementExportPanel.js

@@ -112,12 +112,14 @@ ElementExportPanel.prototype.createResponseString = function () {
       return Promise.reject(new GuiMessageError("You must select at least one column"));
     }
 
-    var rows = [];
-    rows.push(self.createResponseHeader(selectedColumns, miriamTypes));
+    var rowPromises = [];
+    rowPromises.push(self.createResponseHeader(selectedColumns, miriamTypes));
     for (var i = 0; i < elements.length; i++) {
-      rows.push(self.createResponseRow(elements[i], selectedColumns, miriamTypes));
+      rowPromises.push(self.createResponseRow(elements[i], selectedColumns, miriamTypes));
     }
-    return rows.join("\n");
+    return Promise.all(rowPromises).then(function(rows){
+      return rows.join("\n");
+    });
   });
 };
 
@@ -126,7 +128,7 @@ ElementExportPanel.prototype.createResponseString = function () {
  * @param {Alias} alias
  * @param {Array} columns
  * @param {MiriamType[]} miriamTypes
- * @returns {string}
+ * @returns {Promise<string>}
  */
 ElementExportPanel.prototype.createResponseRow = function (alias, columns, miriamTypes) {
   var stringBuilder = [];
@@ -154,7 +156,7 @@ ElementExportPanel.prototype.createResponseRow = function (alias, columns, miria
     }
     stringBuilder.push(value);
   }
-  return stringBuilder.join("\t");
+  return Promise.resolve(stringBuilder.join("\t"));
 };
 
 /**

frontend-js/src/test/js/gui/export/ElementExportPanel-test.js

@@ -283,8 +283,9 @@ describe('ElementExportPanel', function () {
       var alias = helper.createAlias();
       var desc = "test\ntest2\n";
       alias.setDescription(desc);
-      var rowString = exportObject.createResponseRow(alias, exportObject.getAllColumns(), []);
-      assert.ok(rowString.indexOf("test\ntest2\n") === -1);
+      return exportObject.createResponseRow(alias, exportObject.getAllColumns(), []).then (function(rowString){
+        assert.ok(rowString.indexOf("test\ntest2\n") === -1);
+      });
     });
   });