MOBILE-4406 course-storage: Hide downloaded button from resources typ…

…e label
moodlehq · Sep 4, 2023 · c66b33b · c66b33b
1 parent b5a2ddd
commit c66b33b
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 6 deletions.
diff --git a/src/addons/storagemanager/pages/course-storage/course-storage.html b/src/addons/storagemanager/pages/course-storage/course-storage.html
@@ -19,7 +19,10 @@ <h1>{{ 'addon.storagemanager.coursedownloads' | translate }}</h1>
 
         <ion-card class="wholecourse">
             <ion-card-header>
-                <ion-card-title>{{ title }}</ion-card-title>
+                <ion-card-title>
+                    <core-format-text [text]="title" contextLevel="course" [contextInstanceId]="courseId">
+                    </core-format-text>
+                </ion-card-title>
                 <ion-item class="size ion-text-wrap ion-no-padding">
                     <ion-label>
                         <p class="item-heading ion-text-wrap">{{ 'addon.storagemanager.totaldownloads' | translate }}</p>
@@ -77,7 +80,7 @@ <h1>{{ 'addon.storagemanager.coursedownloads' | translate }}</h1>
                         </ion-label>
                         <div class="storage-buttons" slot="end"
                             *ngIf="(!section.calculatingSize && section.totalSize > 0) || downloadEnabled">
-                            <div *ngIf="downloadEnabled" slot="end" class="core-button-spinner">
+                            <div *ngIf="downloadEnabled && canDownloadSection(section)" slot="end" class="core-button-spinner">
                                 <core-download-refresh *ngIf="!section.isDownloading && section.downloadStatus != statusDownloaded"
                                     [status]="section.downloadStatus" [enabled]="true" (action)="prefecthSection(section)"
                                     [loading]="section.isDownloading || section.isCalculating" [canTrustDownload]="true">
@@ -126,17 +129,18 @@ <h1>{{ 'addon.storagemanager.coursedownloads' | translate }}</h1>
 
                                 <div class="storage-buttons" slot="end">
                                     <core-download-refresh *ngIf="downloadEnabled && module.handlerData?.showDownloadButton &&
-                                        module.downloadStatus != statusDownloaded" [status]="module.downloadStatus" [enabled]="true"
-                                        [canTrustDownload]="true" [loading]="module.spinner || module.handlerData.spinner"
-                                        (action)="prefetchModule(module)">
+                                        module.downloadStatus != statusDownloaded && module.modname !== 'label'"
+                                        [status]="module.downloadStatus" [enabled]="true" [canTrustDownload]="true"
+                                        [loading]="module.spinner || module.handlerData.spinner" (action)="prefetchModule(module)">
                                     </core-download-refresh>
                                     <ion-button fill="clear" (click)="deleteForModule($event, module, section)"
                                         *ngIf="!module.calculatingSize && module.totalSize > 0" color="danger">
                                         <ion-icon name="fas-trash" slot="icon-only"
                                             [attr.aria-label]="'addon.storagemanager.deletedatafrom' | translate: { name: module.name }">
                                         </ion-icon>
                                     </ion-button>
-                                    <p *ngIf="!downloadEnabled || !module.handlerData?.showDownloadButton" class="sr-only">
+                                    <p *ngIf="(!downloadEnabled || !module.handlerData?.showDownloadButton) && module.modname === 'label'"
+                                        class="sr-only">
                                         {{ 'core.notdownloadable' | translate }}
                                     </p>
                                 </div>

diff --git a/src/addons/storagemanager/pages/course-storage/course-storage.ts b/src/addons/storagemanager/pages/course-storage/course-storage.ts
@@ -715,6 +715,16 @@ export class AddonStorageManagerCourseStoragePage implements OnInit, OnDestroy {
         }
     }
 
+    /**
+     * Check if the provided section can be downloaded.
+     *
+     * @param section Section to check.
+     * @returns Can be downloaded or not.
+     */
+    canDownloadSection(section: AddonStorageManagerCourseSection): boolean {
+        return section.modules.some(module => module.modname !== 'label');
+    }
+
     /**
      * Toggle expand status.
      *