fix: hide Download All button when embargoed, clean up badge styling and i18n

Author: milanmajchrak

src/app/shared/object-collection/shared/badges/access-status-badge/access-status-badge.component.html

@@ -1,6 +1,6 @@
 @if (showAccessStatus) {
   @if ((accessStatus$ | async); as status) {
-    <span [class]="'badge dont-break-out access-status-list-element-badge ' + accessStatusClass">
+    <span [class]="'badge badge-secondary dont-break-out access-status-list-element-badge ' + accessStatusClass">
       <span class="sr-only">{{ 'listelement.badge.access-status' | translate }}</span>
       {{ status | translate: { date: (embargoDate$ | async) } }}
       <span class="sr-only">, </span>

src/assets/i18n/en.json5

@@ -6972,8 +6972,6 @@
 
   "file-download-link.download": "Download ",
 
-  "file-download-link.embargo-until": "Embargo until {{ date }}",
-
   "register-page.registration.aria.label": "Enter your e-mail address",
 
   "forgot-email.form.aria.label": "Enter your e-mail address",

src/themes/datashare/app/item-page/simple/field-components/file-section/file-section.component.html

@@ -11,7 +11,7 @@
 <ng-container *ngVar="(bitstreams$ | async) as bitstreams">
   <br>
   <div class="download-all-container" *ngIf="(downloadLink$ | async) as downloadLink">
-    <a class="btn btn-primary" href="{{downloadLink}}" target="_blank">Download all files <i class="fa-solid fa-download"></i></a>
+    <a *ngIf="(hasEmbargo$ | async) !== true" class="btn btn-primary" href="{{downloadLink}}" target="_blank">Download all files <i class="fa-solid fa-download"></i></a>
   </div>
   <br/>
   <ds-metadata-field-wrapper *ngIf="bitstreams?.length > 0" [label]="label | translate">

src/themes/datashare/app/item-page/simple/field-components/file-section/file-section.component.ts

@@ -19,6 +19,7 @@ import { hasValue, isEmpty } from '../../../../../../../app/shared/empty.util';
 import { PaginationComponentOptions } from '../../../../../../../app/shared/pagination/pagination-component-options.model';
 import { followLink } from '../../../../../../../app/shared/utils/follow-link-config.model';
 import { BitstreamDataService } from '../../../../../../../app/core/data/bitstream-data.service';
+import { AccessStatusDataService } from '../../../../../../../app/core/data/access-status-data.service';
 import { NotificationsService } from '../../../../../../../app/shared/notifications/notifications.service';
 import { DSONameService } from '../../../../../../../app/core/breadcrumbs/dso-name.service';
 import { APP_CONFIG, AppConfig } from '../../../../../../../config/app-config.interface';
@@ -54,6 +55,7 @@ export class FileSectionComponent extends BaseComponent {
   licenses$: Observable<RemoteData<PaginatedList<Bitstream>>>;
   downloadLink$: Observable<string>;
   downloadLinkAvailable$: Observable<boolean>;
+  hasEmbargo$: Observable<boolean>;
 
   cclicenseOptions = Object.assign(new PaginationComponentOptions(), {
     id: 'cclbo',
@@ -76,6 +78,7 @@ export class FileSectionComponent extends BaseComponent {
     public dsoNameService: DSONameService,
     @Inject(APP_CONFIG) protected appConfig: AppConfig,
     protected downloadLinkService: DownloadLinkService,
+    protected accessStatusDataService: AccessStatusDataService,
   ) {
     super(bitstreamDataService, notificationsService, translateService, dsoNameService, appConfig);
 
@@ -124,6 +127,11 @@ export class FileSectionComponent extends BaseComponent {
       map(link => link)
     );
 
+    this.hasEmbargo$ = this.accessStatusDataService.findAccessStatusFor(this.item).pipe(
+      getFirstCompletedRemoteData(),
+      map(rd => rd?.hasSucceeded && rd.payload?.status === 'embargo'),
+    );
+
   }
 
   hasValuesInBundle(bundle: PaginatedList<Bitstream>) {