feat(#8462): support exporting users devices details

admin/src/js/controllers/export-contacts.js

@@ -17,7 +17,16 @@ angular.module('controllers').controller('ExportContactsCtrl',
     $scope.overwrite = false;
 
     $scope.export = function() {
-      Export('contacts', {}, { humanReadable: true });
+      if ($scope.exporting) {
+        return;
+      }
+
+      $scope.exporting = true;
+      Export('contacts', {}, { humanReadable: true })
+        .finally(() => {
+          $scope.exporting = false;
+          $scope.$apply();
+        });
     };
 
     $scope.import = function() {

admin/src/js/controllers/export-dhis.js

@@ -63,6 +63,10 @@ angular.module('controllers').controller('ExportDhisCtrl',
     });
 
     $scope.export = () => {
+      if ($scope.exporting) {
+        return;
+      }
+
       const { dataSet, period, place } = $scope.selected;
       const filters = {
         dataSet,
@@ -75,6 +79,11 @@ angular.module('controllers').controller('ExportDhisCtrl',
         filters.orgUnit = place;
       }
 
-      Export('dhis', filters, {});
+      $scope.exporting = true;
+      Export('dhis', filters, {})
+        .finally(() => {
+          $scope.exporting = false;
+          $scope.$apply();
+        });
     };
   });

admin/src/js/controllers/export-feedback.js

@@ -62,6 +62,15 @@ angular
       });
 
     $scope.export = function() {
-      Export('feedback');
+      if ($scope.exporting) {
+        return;
+      }
+
+      $scope.exporting = true;
+      Export('feedback')
+        .finally(() => {
+          $scope.exporting = false;
+          $scope.$apply();
+        });
     };
   });

admin/src/js/controllers/export-messages.js

@@ -8,7 +8,16 @@ angular.module('controllers').controller('ExportMessagesCtrl',
     'ngInject';
 
     $scope.export = function() {
-      Export('messages', {}, { humanReadable: true });
+      if ($scope.exporting) {
+        return;
+      }
+
+      $scope.exporting = true;
+      Export('messages', {}, { humanReadable: true })
+        .finally(() => {
+          $scope.exporting = false;
+          $scope.$apply();
+        });
     };
 
   });

admin/src/js/controllers/export-reports.js

@@ -8,7 +8,16 @@ angular.module('controllers').controller('ExportReportsCtrl',
     'ngInject';
 
     $scope.export = function() {
-      Export('reports', {}, { humanReadable: true });
+      if ($scope.exporting) {
+        return;
+      }
+
+      $scope.exporting = true;
+      Export('reports', {}, { humanReadable: true })
+        .finally(() => {
+          $scope.exporting = false;
+          $scope.$apply();
+        });
     };
 
   });

admin/src/js/controllers/export-user-devices.js

@@ -0,0 +1,23 @@
+angular.module('controllers').controller('ExportUserDevicesCtrl',
+  function (
+    $scope,
+    Export
+  ) {
+
+    'use strict';
+    'ngInject';
+
+    $scope.export = function() {
+      if ($scope.exporting) {
+        return;
+      }
+
+      $scope.exporting = true;
+      Export('user-devices', {}, {})
+        .finally(() => {
+          $scope.exporting = false;
+          $scope.$apply();
+        });
+    };
+
+  });

admin/src/js/main.js

@@ -46,6 +46,7 @@ require('./controllers/edit-translation');
 require('./controllers/edit-user');
 require('./controllers/export-contacts');
 require('./controllers/export-dhis');
+require('./controllers/export-user-devices');
 require('./controllers/export-feedback');
 require('./controllers/export-messages');
 require('./controllers/export-reports');
@@ -277,6 +278,15 @@ angular.module('adminApp').config(function(
         }
       }
     })
+    .state('export.user-devices', {
+      url: '/user-devices',
+      views: {
+        tab: {
+          controller: 'ExportUserDevicesCtrl',
+          templateUrl: 'templates/export_user-devices.html'
+        }
+      }
+    })
     .state('export.feedback', {
       url: '/feedback',
       views: {

admin/src/js/modules/ajax-download.js

@@ -4,16 +4,27 @@
 
   const DEFAULT_FILE_NAME = 'download';
 
+  const getFileName = (headers) => {
+    const contentDisposition = headers.get('content-disposition');
+    const match = /filename=(?:"|)(.*?)(?:"|;|$)/.exec(contentDisposition);
+    return match ? match[1].trim() : DEFAULT_FILE_NAME;
+  };
+
   /**
    * Prompts the user to download a file given a url.
    */
   exports.download = function(url) {
-    const element = document.createElement('a');
-    element.setAttribute('href', url);
-    element.setAttribute('download', DEFAULT_FILE_NAME);
-    element.style.display = 'none';
-    document.body.appendChild(element);
-    element.click();
-    document.body.removeChild(element);
+    return fetch(url)
+      .then(response => response.blob()
+        .then(blob => {
+          const blobUrl = URL.createObjectURL(blob);
+          const link = document.createElement('a');
+          link.style.display = 'none';
+          link.href = blobUrl;
+          link.setAttribute('download', getFileName(response.headers));
+          document.body.appendChild(link);
+          link.click();
+          URL.revokeObjectURL(blobUrl);
+        }));
   };
 }());

admin/src/js/services/export.js

@@ -6,6 +6,7 @@ const KNOWN_EXPORTS = [
   'feedback',
   'messages',
   'reports',
+  'user-devices',
 ];
 
 angular.module('inboxServices').factory('Export',
@@ -20,6 +21,6 @@ angular.module('inboxServices').factory('Export',
       }
       const params = '?' + $.param({ filters, options });
       const url = '/api/v2/export/' + type + params;
-      ajaxDownload.download(url);
+      return ajaxDownload.download(url);
     };
   });

admin/src/templates/export.html

@@ -14,5 +14,8 @@
   <li role="presentation" ui-sref-active="active" mm-auth="can_export_dhis">
     <a ui-sref="export.dhis" translate>DHIS2</a>
   </li>
+  <li role="presentation" ui-sref-active="active">
+    <a ui-sref="export.user-devices" translate>export.tabs.user_devices</a>
+  </li>
 </ul>
 <div class="tab-content" ui-view="tab" mm-auth="can_export_all"></div>

admin/src/templates/export_contacts.html

@@ -1,11 +1,11 @@
 <div class="col-sm-12">
-  
+
   <div class="people-section">
     <legend translate>Export</legend>
     <p translate>export.people.description</p>
     <a class="btn btn-default" ng-click="export()" ng-disabled="exporting">
       <i class="fa fa-arrow-down"></i>
-      <span translate>Download</span>
+      <span translate>export.button.download</span>
     </a>
   </div>
 

admin/src/templates/export_dhis.html

@@ -31,7 +31,7 @@
 
     <a class="btn btn-default" ng-click="export()" ng-disabled="exporting || !selected.dataSet">
       <i class="fa fa-arrow-down"></i>
-      <span translate>Download</span>
+      <span translate>export.button.download</span>
     </a>
   </div>
 </div>

admin/src/templates/export_feedback.html

@@ -3,7 +3,7 @@
     <p translate>export.feedback.description</p>
     <a class="btn btn-default" ng-click="export()" ng-disabled="exporting">
       <i class="fa fa-arrow-down"></i>
-      <span translate>Download</span>
+      <span translate>export.button.download</span>
     </a>
   </div>
   <div class="section">

admin/src/templates/export_messages.html

@@ -3,7 +3,7 @@
     <p translate>export.messages.description</p>
     <a class="btn btn-default" ng-click="export()" ng-disabled="exporting">
       <i class="fa fa-arrow-down"></i>
-      <span translate>Download</span>
+      <span translate>export.button.download</span>
     </a>
   </div>
 </div>

admin/src/templates/export_reports.html

@@ -3,7 +3,7 @@
     <p translate>export.reports.description</p>
     <a class="btn btn-default" ng-click="export()" ng-disabled="exporting">
       <i class="fa fa-arrow-down"></i>
-      <span translate>Download</span>
+      <span translate>export.button.download</span>
     </a>
   </div>
 </div>

admin/src/templates/export_user-devices.html

@@ -0,0 +1,9 @@
+<div class="col-sm-12">
+  <div>
+    <p translate>export.user-devices.description</p>
+    <a class="btn btn-default" ng-click="export()" ng-disabled="exporting">
+      <i class="fa fa-arrow-down"></i>
+      <span translate>export.button.download</span>
+    </a>
+  </div>
+</div>

admin/tests/unit/controllers/export-dhis.spec.js

@@ -12,11 +12,13 @@ describe('dhis2 export controller', () => {
   let getService;
   let query;
   let Settings;
+  let Export;
 
   beforeEach(() => {
     module('adminApp');
     sinon.useFakeTimers(NOW);
     Settings = sinon.stub().resolves({ dhis_data_sets: dhisDataSets });
+    Export = sinon.stub().resolves();
     query = sinon.stub().resolves({ rows: [
       mockContact('p1'),
       mockContact('p2', { dhis: { orgUnit: 'ou-p2'}}),
@@ -28,6 +30,7 @@ describe('dhis2 export controller', () => {
 
     module($provide => {
       $provide.value('Settings', Settings);
+      $provide.value('Export', Export);
       $provide.value('$scope', scope);
     });
 
@@ -79,6 +82,19 @@ describe('dhis2 export controller', () => {
     expect(!!scope.dataSets).to.be.false;
   });
 
+  it('doesn\'t trigger additional exports when an export is ongoing', async () => {
+    expect(scope.exporting).to.be.undefined;
+    await getService();
+    scope.export();
+    expect(scope.exporting).to.be.true;
+    scope.export();
+    scope.export();
+    scope.export();
+    expect(Export.returnValues.length).to.equal(1);
+    await Export.returnValues[0];
+    expect(scope.exporting).to.be.false;
+  });
+
   const mockContact = (name, assign) => ({
     id: `contact-${name}`,
     doc: Object.assign({

admin/tests/unit/controllers/export-user-devices.spec.js

@@ -0,0 +1,55 @@
+describe('User devices export controller', () => {
+  'use strict';
+
+  const { expect } = chai;
+
+  let scope;
+  let getService;
+  let Export;
+
+  beforeEach(() => {
+    module('adminApp');
+    Export = sinon.stub().resolves();
+    module($provide => {
+      $provide.value('Export', Export);
+      $provide.value('$scope', scope);
+    });
+    inject(($controller, _$rootScope_) => {
+      scope = _$rootScope_.$new();
+      getService = async () => {
+        const result = $controller('ExportUserDevicesCtrl', {
+          $scope: scope,
+        });
+        return result;
+      };
+    });
+  });
+
+  afterEach(() => sinon.restore());
+
+  it('exports user-devices', async () => {
+    sinon.spy(scope, '$apply');
+    await getService();
+    scope.export();
+    expect(scope.exporting).to.be.true;
+    expect(Export.returnValues.length).to.equal(1);
+    await Export.returnValues[0];
+    expect(scope.exporting).to.be.false;
+    expect(Export.callCount).to.equal(1);
+    expect(Export.args[0]).to.deep.equal(['user-devices', {}, {}]);
+    expect(scope.$apply.callCount).to.equal(1);
+  });
+
+  it('doesn\'t trigger additional exports when an export is ongoing', async () => {
+    expect(scope.exporting).to.be.undefined;
+    await getService();
+    scope.export();
+    expect(scope.exporting).to.be.true;
+    scope.export();
+    scope.export();
+    scope.export();
+    expect(Export.returnValues.length).to.equal(1);
+    await Export.returnValues[0];
+    expect(scope.exporting).to.be.false;
+  });
+});