Merge pull request #419 from aziiee/export_limit

Fix(analysis): Set limit for reports export

Author: Akanksh Saxena

app/analysis/index/controller.js

@@ -184,8 +184,18 @@ const AnalysisController = Controller.extend(AnalysisQueryParams.Mixin, {
     }
   ),
 
+  exportLimitExceeded: computed("totalItems", function() {
+    return this.totalItems > this.exportLimit;
+  }),
+
+  exportLimitMessage: computed("exportLimit", function() {
+    return `The export limit is ${this.exportLimit}. Please use filters to reduce the amount of reports.`;
+  }),
+
   exportLinks: config.APP.REPORTEXPORTS,
 
+  exportLimit: config.APP.EXPORT_LIMIT,
+
   session: service("session"),
 
   notify: service("notify"),

app/analysis/index/template.hbs

@@ -305,6 +305,8 @@
             type="button"
             data-download-count="{{download.performCount}}"
             class="btn btn-primary {{if (and (eq download.last.url link.url) (eq download.last.params link.params) download.isRunning) 'loading'}}"
+            disabled={{exportLimitExceeded}}
+            title={{if exportLimitExceeded exportLimitMessage}}
           >
             {{fa-icon 'download'}} {{link.label}}
           </button>

config/environment.js

@@ -44,7 +44,8 @@ module.exports = function(environment) {
           url: "/api/v1/work-reports",
           params: {}
         }
-      ]
+      ],
+      EXPORT_LIMIT: 100000
     },
 
     "ember-simple-auth": {

tests/acceptance/analysis-test.js

@@ -11,6 +11,7 @@ import { setupApplicationTest } from "ember-qunit";
 import { authenticateSession } from "ember-simple-auth/test-support";
 import { module, test } from "qunit";
 
+import config from "../../config/environment";
 import userSelect from "../helpers/user-select";
 
 module("Acceptance | analysis", function(hooks) {
@@ -42,6 +43,19 @@ module("Acceptance | analysis", function(hooks) {
     assert.dom('[data-download-count="1"]').exists();
   });
 
+  test("disables buttons if export limit exceeded", async function(assert) {
+    this.server.get("/reports", function({ reports }) {
+      return {
+        ...this.serialize(reports.all()),
+        meta: { pagination: { count: config.APP.EXPORT_LIMIT + 1 } }
+      };
+    });
+
+    await visit("/analysis");
+
+    assert.dom(".export-buttons .btn:first-child").isDisabled();
+  });
+
   test("can filter and reset filter", async function(assert) {
     await visit("/analysis");