From efc22b39075d0b777d0953b77d8f073747d82360 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Pau=20Ferrer=20Oca=C3=B1a?= <crazyserver@gmail.com>
Date: Mon, 11 Mar 2024 12:40:50 +0100
Subject: [PATCH] MOBILE-4329 dataprivacy: Add download button when download is
 ready

---
 scripts/langindex.json                        |  3 +-
 src/core/features/dataprivacy/lang.json       |  3 +-
 .../features/dataprivacy/pages/main/main.html | 88 ++++++++++---------
 .../dataprivacy/services/dataprivacy.ts       |  1 +
 .../tests/behat/create_data_request.feature   |  2 +-
 5 files changed, 54 insertions(+), 43 deletions(-)

diff --git a/scripts/langindex.json b/scripts/langindex.json
index 3140fadfa2c..009ac93f884 100644
--- a/scripts/langindex.json
+++ b/scripts/langindex.json
@@ -1684,9 +1684,10 @@
   "core.dataprivacy.datarequests": "tool_dataprivacy",
   "core.dataprivacy.daterequested": "tool_dataprivacy",
   "core.dataprivacy.deletemyaccount": "tool_dataprivacy",
+  "core.dataprivacy.download": "tool_dataprivacy",
   "core.dataprivacy.message": "tool_dataprivacy",
   "core.dataprivacy.newrequest": "tool_dataprivacy",
-  "core.dataprivacy.nodatarequests": "tool_dataprivacy",
+  "core.dataprivacy.nopersonaldatarequests": "tool_dataprivacy",
   "core.dataprivacy.pluginname": "tool_dataprivacy",
   "core.dataprivacy.replyto": "tool_dataprivacy",
   "core.dataprivacy.requestactions": "tool_dataprivacy",
diff --git a/src/core/features/dataprivacy/lang.json b/src/core/features/dataprivacy/lang.json
index 13f0290bd50..3304876cd52 100644
--- a/src/core/features/dataprivacy/lang.json
+++ b/src/core/features/dataprivacy/lang.json
@@ -6,9 +6,10 @@
   "datarequests": "Data requests",
   "daterequested": "Date requested",
   "deletemyaccount": "Delete my account",
+  "download": "Download",
   "message": "Message",
   "newrequest": "New request",
-  "nodatarequests": "There are no data requests",
+  "nopersonaldatarequests": "You don't have any personal data requests",
   "pluginname": "Data privacy",
   "replyto": "Reply to",
   "requestactions": "Actions",
diff --git a/src/core/features/dataprivacy/pages/main/main.html b/src/core/features/dataprivacy/pages/main/main.html
index 5e42746b8e7..38ffc3aa15b 100644
--- a/src/core/features/dataprivacy/pages/main/main.html
+++ b/src/core/features/dataprivacy/pages/main/main.html
@@ -35,24 +35,27 @@ <h2 class="big">{{ 'core.dataprivacy.datarequests' | translate }}</h2>
                         </ion-row>
                     </ion-label>
                 </ion-item>
-                <ion-item button [detail]="false" *ngIf="request.requestedbyuser" core-user-link [userId]="request.requestedbyuser.id"
-                    [attr.aria-label]="request.requestedbyuser.fullname" class="ion-text-wrap">
+                <ion-item *ngIf="request.requestedbyuser" class="ion-text-wrap">
                     <ion-label>
                         <p class="item-heading">{{ 'core.dataprivacy.requestby' | translate }}</p>
                         <p>{{ request.requestedbyuser.fullname }}</p>
                     </ion-label>
                 </ion-item>
-                <ion-item class="ion-text-wrap" [lines]="request.canCancel ? 'full' : 'none'">
+                <ion-item class="ion-text-wrap" [lines]="request.canCancel || request.downloadlink ? 'full' : 'none'">
                     <ion-label>
                         <p class="item-heading">{{ 'core.dataprivacy.message' | translate }}</p>
                         <p><core-format-text [text]="request.messagehtml" /></p>
                     </ion-label>
                 </ion-item>
-                <ion-item class="ion-text-wrap ion-text-end" *ngIf="request.canCancel">
+                <ion-item class="ion-text-wrap ion-text-end" *ngIf="request.canCancel || request.downloadlink">
                     <ion-label>
-                        <ion-button fill="outline" expand="block" (click)="cancelRequest(request.id)">
+                        <ion-button fill="outline" expand="block" (click)="cancelRequest(request.id)" *ngIf="request.canCancel">
                             {{ 'core.dataprivacy.cancelrequest' | translate }}
                         </ion-button>
+                        <ion-button fill="outline" expand="block" *ngIf="request.downloadlink" [href]="request.downloadlink" core-link
+                            [showBrowserWarning]="false">
+                            {{ 'core.dataprivacy.download' | translate }}
+                        </ion-button>
                     </ion-label>
                 </ion-item>
             </ion-card>
@@ -90,12 +93,17 @@ <h2 class="big">{{ 'core.dataprivacy.datarequests' | translate }}</h2>
                         <ion-button fill="outline" size="small" (click)="cancelRequest(request.id)" *ngIf="request.canCancel">
                             {{ 'core.dataprivacy.cancelrequest' | translate }}
                         </ion-button>
+                        <ion-button fill="outline" size="small" *ngIf="request.downloadlink" [href]="request.downloadlink" core-link
+                            [showBrowserWarning]="false">
+                            {{ 'core.dataprivacy.download' | translate }}
+                        </ion-button>
                     </td>
                 </tr>
             </tbody>
         </table>
 
-        <core-empty-box *ngIf="!requests.length" icon="fas-bell-concierge" [message]="'core.dataprivacy.nodatarequests' | translate" />
+        <core-empty-box *ngIf="!requests.length" icon="fas-bell-concierge"
+            [message]="'core.dataprivacy.nopersonaldatarequests' | translate" />
 
         <div collapsible-footer *ngIf="loaded" slot="fixed">
             <div class="list-item-limited-width adaptable-buttons-row">
@@ -117,44 +125,44 @@ <h2 class="big">{{ 'core.dataprivacy.datarequests' | translate }}</h2>
 
 <ng-template #statusBadge let-request="request">
     @switch (request.status) {
-    @case (0) {
-    <ion-badge color="info">{{'core.dataprivacy.statuspending' | translate }}</ion-badge>
-    } @case (1) {
-    <ion-badge color="info">{{'core.dataprivacy.statuspreprocessing' | translate }}</ion-badge>
-    } @case (2) {
-    <ion-badge color="info">{{'core.dataprivacy.statusawaitingapproval' | translate }}</ion-badge>
-    } @case (3) {
-    <ion-badge color="info">{{'core.dataprivacy.statusapproved' | translate }}</ion-badge>
-    } @case (4) {
-    <ion-badge color="info">{{'core.dataprivacy.statusprocessing' | translate }}</ion-badge>
-    } @case (5) {
-    <ion-badge color="success">{{'core.dataprivacy.statuscomplete' | translate }}</ion-badge>
-    } @case (6) {
-    <ion-badge color="warning">{{'core.dataprivacy.statuscancelled' | translate }}</ion-badge>
-    } @case (7) {
-    <ion-badge color="danger">{{'core.dataprivacy.statusrejected' | translate }}</ion-badge>
-    } @case (8) {
-    <ion-badge color="success">{{'core.dataprivacy.statusready' | translate }}</ion-badge>
-    } @case (9) {
-    <ion-badge color="secondary">{{'core.dataprivacy.statusexpired' | translate }}</ion-badge>
-    } @case (10) {
-    <ion-badge color="success">{{'core.dataprivacy.statusdeleted' | translate }}</ion-badge>
-    } @default {
-    <ion-badge class="ion-text-wrap">{{request.statuslabel}}</ion-badge>
-    }
+        @case (0) {
+            <ion-badge color="info">{{'core.dataprivacy.statuspending' | translate }}</ion-badge>
+        } @case (1) {
+            <ion-badge color="info">{{'core.dataprivacy.statuspreprocessing' | translate }}</ion-badge>
+        } @case (2) {
+            <ion-badge color="info">{{'core.dataprivacy.statusawaitingapproval' | translate }}</ion-badge>
+        } @case (3) {
+            <ion-badge color="info">{{'core.dataprivacy.statusapproved' | translate }}</ion-badge>
+        } @case (4) {
+            <ion-badge color="info">{{'core.dataprivacy.statusprocessing' | translate }}</ion-badge>
+        } @case (5) {
+            <ion-badge color="success">{{'core.dataprivacy.statuscomplete' | translate }}</ion-badge>
+        } @case (6) {
+            <ion-badge color="warning">{{'core.dataprivacy.statuscancelled' | translate }}</ion-badge>
+        } @case (7) {
+            <ion-badge color="danger">{{'core.dataprivacy.statusrejected' | translate }}</ion-badge>
+        } @case (8) {
+            <ion-badge color="success">{{'core.dataprivacy.statusready' | translate }}</ion-badge>
+        } @case (9) {
+            <ion-badge color="secondary">{{'core.dataprivacy.statusexpired' | translate }}</ion-badge>
+        } @case (10) {
+            <ion-badge color="success">{{'core.dataprivacy.statusdeleted' | translate }}</ion-badge>
+        } @default {
+            <ion-badge class="ion-text-wrap">{{request.statuslabel}}</ion-badge>
+        }
     }
 </ng-template>
 
 <ng-template #type let-request="request">
     @switch (request.type) {
-    @case (1) {
-    {{ 'core.dataprivacy.requesttypeexport' | translate }}
-    } @case (2) {
-    {{ 'core.dataprivacy.requesttypedelete' | translate }}
-    } @case (3) {
-    {{ 'core.dataprivacy.requesttypeothers' | translate }}
-    } @default {
-    {{request.typename}}
-    }
+        @case (1) {
+            {{ 'core.dataprivacy.requesttypeexport' | translate }}
+        } @case (2) {
+            {{ 'core.dataprivacy.requesttypedelete' | translate }}
+        } @case (3) {
+            {{ 'core.dataprivacy.requesttypeothers' | translate }}
+        } @default {
+            {{request.typename}}
+        }
     }
 </ng-template>
diff --git a/src/core/features/dataprivacy/services/dataprivacy.ts b/src/core/features/dataprivacy/services/dataprivacy.ts
index 9305503cbf1..516de5bb49a 100644
--- a/src/core/features/dataprivacy/services/dataprivacy.ts
+++ b/src/core/features/dataprivacy/services/dataprivacy.ts
@@ -380,4 +380,5 @@ export type CoreDataPrivacyRequest = {
     approvedeny?: boolean; // Approvedeny.
     allowfiltering?: boolean; // Allowfiltering.
     canmarkcomplete?: boolean; // Canmarkcomplete.
+    downloadlink?: string; // Downloadlink.
 };
diff --git a/src/core/features/dataprivacy/tests/behat/create_data_request.feature b/src/core/features/dataprivacy/tests/behat/create_data_request.feature
index 4be00278772..bd848bcd854 100644
--- a/src/core/features/dataprivacy/tests/behat/create_data_request.feature
+++ b/src/core/features/dataprivacy/tests/behat/create_data_request.feature
@@ -59,4 +59,4 @@ Feature: Data export and delete from the privacy API
     When I run all adhoc tasks
     And I pull to refresh in the app
     And I should find "Download ready" near "Export all of my personal data" in the app
-    # TODO: Add download link and test it.
+    And I press "Download" in the app