Manually fix lint issues

2025-10-11 20:13:07 +00:00 · 2024-03-06 10:45:59 +01:00
parent 2b540cd91c
commit 3b48d9a26d
24 changed files with 82 additions and 58 deletions
--- a/src/app/admin/admin-ldn-services/ldn-service-form/ldn-service-form.component.ts
+++ b/src/app/admin/admin-ldn-services/ldn-service-form/ldn-service-form.component.ts
@@ -261,8 +261,7 @@ export class LdnServiceFormComponent implements OnInit, OnDestroy {
    const servicesToUse = this.ldnService.notifyServiceInboundPatterns;

    servicesToUse.forEach((patternObj: NotifyServicePattern) => {
-      let patternFormGroup;
-      patternFormGroup = this.initializeInboundPatternFormGroup();
+      const patternFormGroup = this.initializeInboundPatternFormGroup();
      const newPatternObjWithLabel = Object.assign(new NotifyServicePattern(), {
        ...patternObj,
        patternLabel: this.translateService.instant('ldn-service.form.pattern.' + patternObj?.pattern + '.label'),
--- a/src/app/admin/admin-reports/filters-section/filters-section.component.spec.ts
+++ b/src/app/admin/admin-reports/filters-section/filters-section.component.spec.ts
@@ -45,7 +45,7 @@ describe('FiltersComponent', () => {
    fixture.detectChanges();
  }));

-  const isOneSelected = (values: {}): boolean => {
+  const isOneSelected = (values: any): boolean => {
    let oneSelected = false;
    let allFilters = FiltersComponent.FILTERS;
    for (let i = 0; !oneSelected && i < allFilters.length; i++) {
@@ -58,7 +58,7 @@ describe('FiltersComponent', () => {
    return oneSelected;
  };

-  const isAllSelected = (values: {}): boolean => {
+  const isAllSelected = (values: any): boolean => {
    let allSelected = true;
    let allFilters = FiltersComponent.FILTERS;
    for (let i = 0; allSelected && i < allFilters.length; i++) {
--- a/src/app/admin/admin-reports/filters-section/filters-section.component.ts
+++ b/src/app/admin/admin-reports/filters-section/filters-section.component.ts
@@ -111,7 +111,7 @@ export class FiltersComponent {
    return undefined;
  }

-  static toQueryString(filters: Object): string {
+  static toQueryString(filters: { [key: string]: any }): string {
    let params = '';
    let first = true;
    for (const key in filters) {
--- a/src/app/browse-by/browse-by-metadata/browse-by-metadata.component.html
+++ b/src/app/browse-by/browse-by-metadata/browse-by-metadata.component.html
@@ -1,6 +1,6 @@
 <section class="comcol-page-browse-section">
  <div class="browse-by-metadata w-100">
-    <ds-themed-browse-by *ngIf="!(loading$ | async)" class="col-xs-12 w-100"
+    <ds-themed-browse-by *ngIf="(loading$ | async) === false" class="col-xs-12 w-100"
                         title="{{'browse.title' | translate:{
                           field: 'browse.metadata.' + browseId | translate,
                           startsWith: (startsWith)? ('browse.startsWith' | translate: { startsWith: '&quot;' + startsWith + '&quot;' }) : '',
--- a/src/app/community-list-page/community-list/community-list.component.html
+++ b/src/app/community-list-page/community-list/community-list.component.html
@@ -33,7 +33,7 @@
        <span class="sr-only">{{ (node.isExpanded ? 'communityList.collapse' : 'communityList.expand') | translate:{ name: dsoNameService.getName(node.payload) } }}</span>
      </button>
      <!--Don't render the button when non-expandable otherwise it's still accessible, instead render this placeholder-->
-      <span *ngIf="!(hasChild(null, node) | async)" aria-hidden="true" class="btn btn-default invisible">
+      <span *ngIf="(hasChild(null, node) | async) === false" aria-hidden="true" class="btn btn-default invisible">
        <span class="fa fa-chevron-right"></span>
      </span>
      <div class="d-flex flex-row">
--- a/src/app/core/data/processes/process-data.service.spec.ts
+++ b/src/app/core/data/processes/process-data.service.spec.ts
@@ -41,7 +41,7 @@ import {
 describe('ProcessDataService', () => {
  let testScheduler;

-  const mockTimer = (fn: () => {}, interval: number) => {
+  const mockTimer = (fn: () => any, interval: number) => {
    fn();
    return 555;
  };
--- a/src/app/core/notifications/source/suggestion-source-data.service.ts
+++ b/src/app/core/notifications/source/suggestion-source-data.service.ts
@@ -1,7 +1,7 @@
 import { HttpClient } from '@angular/common/http';
 import { Injectable } from '@angular/core';
 import { Store } from '@ngrx/store';
-import { Observable } from 'rxjs/internal/Observable';
+import { Observable } from 'rxjs';

 import { NotificationsService } from '../../../shared/notifications/notifications.service';
 import { FollowLinkConfig } from '../../../shared/utils/follow-link-config.model';
--- a/src/app/core/notifications/target/suggestion-target-data.service.ts
+++ b/src/app/core/notifications/target/suggestion-target-data.service.ts
@@ -1,7 +1,7 @@
 import { HttpClient } from '@angular/common/http';
 import { Injectable } from '@angular/core';
 import { Store } from '@ngrx/store';
-import { Observable } from 'rxjs/internal/Observable';
+import { Observable } from 'rxjs';

 import { NotificationsService } from '../../../shared/notifications/notifications.service';
 import { FollowLinkConfig } from '../../../shared/utils/follow-link-config.model';
--- a/src/app/dso-shared/dso-edit-metadata/dso-edit-metadata-value/dso-edit-metadata-value.component.html
+++ b/src/app/dso-shared/dso-edit-metadata/dso-edit-metadata-value/dso-edit-metadata-value.component.html
@@ -3,7 +3,7 @@
     [ngClass]="{ 'ds-warning': mdValue.reordered || mdValue.change === DsoEditMetadataChangeTypeEnum.UPDATE, 'ds-danger': mdValue.change === DsoEditMetadataChangeTypeEnum.REMOVE, 'ds-success': mdValue.change === DsoEditMetadataChangeTypeEnum.ADD, 'h-100': isOnlyValue }">
  <div class="flex-grow-1 ds-flex-cell ds-value-cell d-flex flex-column" *ngVar="(mdRepresentation$ | async) as mdRepresentation" role="cell">
    <div class="dont-break-out preserve-line-breaks" *ngIf="!mdValue.editing && !mdRepresentation">{{ mdValue.newValue.value }}</div>
-    <textarea class="form-control" rows="5" *ngIf="mdValue.editing && !mdRepresentation && !(isAuthorityControlled() | async)" [(ngModel)]="mdValue.newValue.value"
+    <textarea class="form-control" rows="5" *ngIf="mdValue.editing && !mdRepresentation && (isAuthorityControlled() | async) === false" [(ngModel)]="mdValue.newValue.value"
              [attr.aria-label]="(dsoType + '.edit.metadata.edit.value') | translate"
              [dsDebounce]="300" (onDebounce)="confirm.emit(false)"></textarea>
    <ds-dynamic-scrollable-dropdown *ngIf="mdValue.editing && (isScrollableVocabulary() | async)"
--- a/src/app/item-page/field-components/metadata-values/metadata-values.component.html
+++ b/src/app/item-page/field-components/metadata-values/metadata-values.component.html
@@ -3,7 +3,7 @@
      <!--
      Choose a template. Priority: markdown, link, browse link.
      -->
-      <ng-container *ngTemplateOutlet="(renderMarkdown ? markdown : (hasLink(mdValue) ? (img != null ? linkImg : link) : (hasBrowseDefinition() ? browselink : simple)));
+      <ng-container *ngTemplateOutlet="(renderMarkdown ? markdown : (hasLink(mdValue) ? (img !== null ? linkImg : link) : (hasBrowseDefinition() ? browselink : simple)));
        context: {value: mdValue.value, img}">
      </ng-container>
      <span class="separator" *ngIf="!last" [innerHTML]="separator"></span>
--- a/src/app/item-page/simple/notify-requests-status/request-status-alert-box/request-status-alert-box.component.html
+++ b/src/app/item-page/simple/notify-requests-status/request-status-alert-box/request-status-alert-box.component.html
@@ -1,6 +1,6 @@
 <ng-container *ngIf="data?.length > 0 && displayOptions">
  <div
-    [ngClass]="{'align-items-center': data.length == 1}"
+    [ngClass]="{'align-items-center': data.length === 1}"
    class="alert d-flex flex-row sections-gap {{
      displayOptions.alertType
    }}"
--- a/src/app/notifications/qa/events/quality-assurance-events.component.html
+++ b/src/app/notifications/qa/events/quality-assurance-events.component.html
@@ -196,7 +196,7 @@
                        <i class="fas fa-trash-alt"></i>
                      </button>
                    </div>
-                    <div *ngIf="!(isAdmin$ | async)" class="btn-group button-width">
+                    <div *ngIf="(isAdmin$ | async) === false" class="btn-group button-width">
                      <button class="btn btn-outline-danger btn-sm button-width"
                              ngbTooltip="{{'quality-assurance.event.action.undo' | translate}}"
                              container="body"
--- a/src/app/notifications/suggestion-targets/publication-claim/publication-claim.component.html
+++ b/src/app/notifications/suggestion-targets/publication-claim/publication-claim.component.html
@@ -4,7 +4,7 @@
      <h1 id="header" class="border-bottom pb-2">{{'suggestion.title'| translate}}</h1>

      <ds-loading class="container" *ngIf="(isTargetsLoading() | async)" message="{{'suggestion.loading' | translate}}"></ds-loading>
-      <ds-pagination *ngIf="!(isTargetsLoading() | async)"
+      <ds-pagination *ngIf="(isTargetsLoading() | async) === false"
                     [paginationOptions]="paginationConfig"
                     [collectionSize]="(totalElements$ | async)"
                     [hideGear]="false"
@@ -12,11 +12,11 @@
                     (paginationChange)="getSuggestionTargets()">

        <ds-loading class="container" *ngIf="(isTargetsProcessing() | async)" message="'suggestion.loading' | translate"></ds-loading>
-        <ng-container *ngIf="!(isTargetsProcessing() | async)">
-          <div *ngIf="!(targets$|async) || (targets$|async)?.length == 0" class="alert alert-info w-100 mb-2 mt-2" role="alert">
+        <ng-container *ngIf="(isTargetsProcessing() | async) === false">
+          <div *ngIf="(targets$ | async)?.length === 0" class="alert alert-info w-100 mb-2 mt-2" role="alert">
            {{'suggestion.noTargets' | translate}}
          </div>
-          <div *ngIf="(targets$|async) && (targets$|async)?.length != 0" class="table-responsive mt-2">
+          <div *ngIf="(targets$|async)?.length !== 0" class="table-responsive mt-2">
            <table id="epeople" class="table table-striped table-hover table-bordered">
              <thead>
                <tr class="text-center">
--- a/src/app/notifications/suggestion-targets/suggestion-targets.actions.ts
+++ b/src/app/notifications/suggestion-targets/suggestion-targets.actions.ts
@@ -19,6 +19,7 @@ export const SuggestionTargetActionTypes = {
  RETRIEVE_TARGETS_BY_SOURCE_ERROR: type('dspace/integration/openaire/suggestions/target/RETRIEVE_TARGETS_BY_SOURCE_ERROR'),
  ADD_USER_SUGGESTIONS: type('dspace/integration/openaire/suggestions/target/ADD_USER_SUGGESTIONS'),
  REFRESH_USER_SUGGESTIONS: type('dspace/integration/openaire/suggestions/target/REFRESH_USER_SUGGESTIONS'),
+  REFRESH_USER_SUGGESTIONS_ERROR: type('dspace/integration/openaire/suggestions/target/REFRESH_USER_SUGGESTIONS_ERROR'),
  MARK_USER_SUGGESTIONS_AS_VISITED: type('dspace/integration/openaire/suggestions/target/MARK_USER_SUGGESTIONS_AS_VISITED'),
 };

@@ -126,6 +127,13 @@ export class RefreshUserSuggestionsAction implements Action {
  type = SuggestionTargetActionTypes.REFRESH_USER_SUGGESTIONS;
 }

+/**
+ * NgRx action to signify an error while handling {@link RefreshUserSuggestionsAction}
+ */
+export class RefreshUserSuggestionsErrorAction implements Action {
+  type = SuggestionTargetActionTypes.REFRESH_USER_SUGGESTIONS_ERROR;
+}
+
 /**
 * An ngrx action to Mark User Suggestions As Visited.
 * Called by the ??? effect.
--- a/src/app/notifications/suggestion-targets/suggestion-targets.effects.ts
+++ b/src/app/notifications/suggestion-targets/suggestion-targets.effects.ts
@@ -21,6 +21,7 @@ import { SuggestionsService } from '../suggestions.service';
 import {
  AddTargetAction,
  AddUserSuggestionsAction,
+  RefreshUserSuggestionsErrorAction,
  RetrieveAllTargetsErrorAction,
  RetrieveTargetsBySourceAction,
  SuggestionTargetActionTypes,
@@ -46,8 +47,8 @@ export class SuggestionTargetsEffects {
        map((targets: PaginatedList<SuggestionTarget>) =>
          new AddTargetAction(targets.page, targets.totalPages, targets.currentPage, targets.totalElements),
        ),
-        catchError((error: Error) => {
-          if (error) {
+        catchError((error: unknown) => {
+          if (error instanceof Error) {
            console.error(error.message);
          }
          return of(new RetrieveAllTargetsErrorAction());
@@ -78,10 +79,20 @@ export class SuggestionTargetsEffects {
            return this.suggestionsService.retrieveCurrentUserSuggestions(userId)
              .pipe(
                map((suggestionTargets: SuggestionTarget[]) => new AddUserSuggestionsAction(suggestionTargets)),
-                catchError((errors: unknown) => of(errors)),
+                catchError((error: unknown) => {
+                  if (error instanceof Error) {
+                    console.error(error.message);
+                  }
+                  return of(new RefreshUserSuggestionsErrorAction());
+                }),
              );
          }),
-          catchError((errors: unknown) => of(errors)),
+          catchError((error: unknown) => {
+            if (error instanceof Error) {
+              console.error(error.message);
+            }
+            return of(new RefreshUserSuggestionsErrorAction());
+          }),
        );
    })),
  );
--- a/src/app/process-page/overview/table/process-overview-table.component.html
+++ b/src/app/process-page/overview/table/process-overview-table.component.html
@@ -57,7 +57,7 @@

      </ds-pagination>

-      <div *ngIf="processesRD?.payload?.totalElements == 0">
+      <div *ngIf="processesRD?.payload?.totalElements === 0">
        <p>{{'process.overview.table.empty' | translate}}</p>
      </div>
    </ng-container>
--- a/src/app/shared/auth-nav-menu/auth-nav-menu.component.html
+++ b/src/app/shared/auth-nav-menu/auth-nav-menu.component.html
@@ -39,7 +39,7 @@

 <ng-template #mobileButtons>
  <div data-test="auth-nav">
-    <a *ngIf="!(isAuthenticated | async)" routerLink="/login" routerLinkActive="active" class="loginLink px-0.5" role="button">
+    <a *ngIf="(isAuthenticated | async) === false" routerLink="/login" routerLinkActive="active" class="loginLink px-0.5" role="button">
      {{ 'nav.login' | translate }}<span class="sr-only">(current)</span>
    </a>
    <a *ngIf="(isAuthenticated | async)" role="button" [attr.aria-label]="'nav.logout' |translate" [title]="'nav.logout' | translate" routerLink="/logout" routerLinkActive="active" class="logoutLink px-1">
--- a/src/app/shared/correction-suggestion/item-withdrawn-reinstate-modal.component.html
+++ b/src/app/shared/correction-suggestion/item-withdrawn-reinstate-modal.component.html
@@ -1,4 +1,4 @@
-<div *ngIf="!(this.submitted$ | async); else waiting">
+<div *ngIf="(this.submitted$ | async) === false; else waiting">
  <div *ngIf="this.canWithdraw; else reinstateHeader" class="modal-header">
    {{ 'item.qa.withdrawn.modal.header' | translate }}
    <button type="button" class="close" (click)="onModalClose()" aria-label="Close">
--- a/src/app/shared/log-in/methods/password/log-in-password.component.ts
+++ b/src/app/shared/log-in/methods/password/log-in-password.component.ts
@@ -152,8 +152,12 @@ export class LogInPasswordComponent implements OnInit {
      }),
    );

-    this.canRegister$ = this.authorizationService.isAuthorized(FeatureID.EPersonRegistration).pipe(shareReplay(1));
-    this.canForgot$ = this.authorizationService.isAuthorized(FeatureID.EPersonForgotPassword).pipe(shareReplay(1));
+    this.canRegister$ = this.authorizationService.isAuthorized(FeatureID.EPersonRegistration).pipe(
+      shareReplay({ refCount: false, bufferSize: 1 }),
+    );
+    this.canForgot$ = this.authorizationService.isAuthorized(FeatureID.EPersonForgotPassword).pipe(
+      shareReplay({ refCount: false, bufferSize: 1 }),
+    );
    this.canShowDivider$ =
        combineLatest([this.canRegister$, this.canForgot$])
          .pipe(
--- a/src/app/submission/sections/duplicates/section-duplicates.component.html
+++ b/src/app/submission/sections/duplicates/section-duplicates.component.html
@@ -3,7 +3,7 @@ Template for the detect duplicates submission section component
@author Kim Shepherd
 -->
 <div class="text-sm-left" *ngVar="(this.getDuplicateData() | async) as data">
-  <ng-container *ngIf="data?.potentialDuplicates.length == 0">
+  <ng-container *ngIf="data?.potentialDuplicates.length === 0">
    <div class="alert alert-success w-100">{{ 'submission.sections.duplicates.none' | translate }}</div>
  </ng-container>
    <ng-container *ngIf="data?.potentialDuplicates.length > 0">
--- a/src/app/submission/sections/section-coar-notify/section-coar-notify.component.html
+++ b/src/app/submission/sections/section-coar-notify/section-coar-notify.component.html
@@ -44,7 +44,7 @@
                    [scrollWindow]="false"
                  >
                    <button
-                      *ngIf="(filterServices(ldnPattern.pattern) | async)?.length == 0"
+                      *ngIf="(filterServices(ldnPattern.pattern) | async)?.length === 0"
                      class="dropdown-item collection-item text-truncate w-100"
                    >
                      {{'submission.section.section-coar-notify.dropdown.no-data' | translate}}
@@ -58,7 +58,7 @@
                    </button>
                    <button
                      *ngFor="let serviceOption of filterServices(ldnPattern.pattern ) | async"
-                      [ngClass]="{'bg-light': ldnServiceByPattern[ldnPattern.pattern ].services[serviceIndex]?.id == serviceOption.id}"
+                      [ngClass]="{'bg-light': ldnServiceByPattern[ldnPattern.pattern ].services[serviceIndex]?.id === serviceOption.id}"
                      class="dropdown-item collection-item text-truncate w-100"
                      (click)="onChange(ldnPattern.pattern, serviceIndex, serviceOption)"
                    >
--- a/src/app/suggestions-page/suggestions-page.component.html
+++ b/src/app/suggestions-page/suggestions-page.component.html
@@ -22,7 +22,7 @@
              <i class='fas fa-circle-notch fa-spin' *ngIf="isBulkOperationPending"></i>
            </div>
            <ds-loading *ngIf="(processing$ | async)"></ds-loading>
-            <ds-pagination *ngIf="!(processing$ | async)"
+            <ds-pagination *ngIf="(processing$ | async) === false"
                           [paginationOptions]="paginationOptions"
                           [sortOptions]="paginationSortConfig"
                           [collectionSize]="suggestionsRD?.pageInfo?.totalElements" [hideGear]="false"
--- a/src/environments/environment.test.ts
+++ b/src/environments/environment.test.ts
@@ -123,7 +123,7 @@ export const environment: BuildConfig = {
      timer: 5,
    },
    duplicateDetection: {
-      alwaysShowSection: false
+      alwaysShowSection: false,
    },
    typeBind: {
      field: 'dc.type',
@@ -152,22 +152,24 @@ export const environment: BuildConfig = {
          {
            value: 600,
            style: 'text-success',
-            icon: 'fa-circle-check'
+            icon: 'fa-circle-check',
          },
          {
            value: 500,
            style: 'text-info',
-            icon: 'fa-gear'
+            icon: 'fa-gear',
          },
          {
            value: 400,
            style: 'text-warning',
-            icon: 'fa-circle-question'
+            icon: 'fa-circle-question',
          },
          {
            value: 'default',
            style: 'text-muted',
-            icon: 'fa-circle-xmark'
+            icon: 'fa-circle-xmark',
+          },
+        ],
      },
    },
  },
@@ -247,9 +249,9 @@ export const environment: BuildConfig = {
      sortField: 'dc.date.accessioned',
    },
    topLevelCommunityList: {
-      pageSize: 5
+      pageSize: 5,
    },
-    showDiscoverFilters: false
+    showDiscoverFilters: false,
  },
  item: {
    edit: {
@@ -332,8 +334,8 @@ export const environment: BuildConfig = {
    {
      filter: 'subject',
      vocabulary: 'srsc',
-      enabled: true
-    }
+      enabled: true,
+    },
  ],

  suggestion: [],
@@ -341,8 +343,8 @@ export const environment: BuildConfig = {
  search: {
    advancedFilters: {
      enabled: false,
-      filter: ['title', 'author', 'subject', 'entityType']
-    }
+      filter: ['title', 'author', 'subject', 'entityType'],
+    },
  },

  notifyMetrics: [
@@ -353,34 +355,34 @@ export const environment: BuildConfig = {
          color: '#B8DAFF',
          title: 'admin-notify-dashboard.NOTIFY.incoming.accepted',
          config: 'NOTIFY.incoming.accepted',
-          description: 'admin-notify-dashboard.NOTIFY.incoming.accepted.description'
+          description: 'admin-notify-dashboard.NOTIFY.incoming.accepted.description',
        },
        {
          color: '#D4EDDA',
          title: 'admin-notify-dashboard.NOTIFY.incoming.processed',
          config: 'NOTIFY.incoming.processed',
-          description: 'admin-notify-dashboard.NOTIFY.incoming.processed.description'
+          description: 'admin-notify-dashboard.NOTIFY.incoming.processed.description',
        },
        {
          color: '#FDBBC7',
          title: 'admin-notify-dashboard.NOTIFY.incoming.failure',
          config: 'NOTIFY.incoming.failure',
-          description: 'admin-notify-dashboard.NOTIFY.incoming.failure.description'
+          description: 'admin-notify-dashboard.NOTIFY.incoming.failure.description',
        },
        {
          color: '#FDBBC7',
          title: 'admin-notify-dashboard.NOTIFY.incoming.untrusted',
          config: 'NOTIFY.incoming.untrusted',
-          description: 'admin-notify-dashboard.NOTIFY.incoming.untrusted.description'
+          description: 'admin-notify-dashboard.NOTIFY.incoming.untrusted.description',
        },
        {
          color: '#43515F',
          title: 'admin-notify-dashboard.NOTIFY.incoming.involvedItems',
          textColor: '#fff',
          config: 'NOTIFY.incoming.involvedItems',
-          description: 'admin-notify-dashboard.NOTIFY.incoming.involvedItems.description'
+          description: 'admin-notify-dashboard.NOTIFY.incoming.involvedItems.description',
        },
-      ]
+      ],
    },
    {
      title: 'admin-notify-dashboard.generated-ldn',
@@ -389,34 +391,34 @@ export const environment: BuildConfig = {
          color: '#D4EDDA',
          title: 'admin-notify-dashboard.NOTIFY.outgoing.delivered',
          config: 'NOTIFY.outgoing.delivered',
-          description: 'admin-notify-dashboard.NOTIFY.outgoing.delivered.description'
+          description: 'admin-notify-dashboard.NOTIFY.outgoing.delivered.description',
        },
        {
          color: '#B8DAFF',
          title: 'admin-notify-dashboard.NOTIFY.outgoing.queued',
          config: 'NOTIFY.outgoing.queued',
-          description: 'admin-notify-dashboard.NOTIFY.outgoing.queued.description'
+          description: 'admin-notify-dashboard.NOTIFY.outgoing.queued.description',
        },
        {
          color: '#FDEEBB',
          title: 'admin-notify-dashboard.NOTIFY.outgoing.queued_for_retry',
          config: 'NOTIFY.outgoing.queued_for_retry',
-          description: 'admin-notify-dashboard.NOTIFY.outgoing.queued_for_retry.description'
+          description: 'admin-notify-dashboard.NOTIFY.outgoing.queued_for_retry.description',
        },
        {
          color: '#FDBBC7',
          title: 'admin-notify-dashboard.NOTIFY.outgoing.failure',
          config: 'NOTIFY.outgoing.failure',
-          description: 'admin-notify-dashboard.NOTIFY.outgoing.failure.description'
+          description: 'admin-notify-dashboard.NOTIFY.outgoing.failure.description',
        },
        {
          color: '#43515F',
          title: 'admin-notify-dashboard.NOTIFY.outgoing.involvedItems',
          textColor: '#fff',
          config: 'NOTIFY.outgoing.involvedItems',
-          description: 'admin-notify-dashboard.NOTIFY.outgoing.involvedItems.description'
+          description: 'admin-notify-dashboard.NOTIFY.outgoing.involvedItems.description',
        },
-      ]
-    }
-  ]
+      ],
+    },
+  ],
 };
--- a/src/themes/dspace/app/header/header.component.html
+++ b/src/themes/dspace/app/header/header.component.html
@@ -8,7 +8,7 @@
      <a class="d-block my-2 my-md-0" routerLink="/home" [attr.aria-label]="'home.title' | translate">
        <img id="header-logo" src="assets/images/dspace-logo.svg" [attr.alt]="'menu.header.image.logo' | translate"/>
      </a>
-      <nav *ngIf="!(isMobile$ | async)" class="navbar navbar-expand p-0 align-items-stretch align-self-stretch" id="desktop-navbar" [attr.aria-label]="'nav.main.description' | translate">
+      <nav *ngIf="(isMobile$ | async) === false" class="navbar navbar-expand p-0 align-items-stretch align-self-stretch" id="desktop-navbar" [attr.aria-label]="'nav.main.description' | translate">
        <ds-themed-navbar></ds-themed-navbar>
      </nav>
    </div>
@@ -24,7 +24,7 @@

      <div id="mobile-navbar-toggler" class="d-block d-lg-none ml-3" *ngIf="(isMobile$ | async)">
        <button id="navbar-toggler" class="btn" type="button" (click)="toggleNavbar()"
-                [attr.aria-label]="'nav.toggle' | translate" aria-controls="collapsible-mobile-navbar" [attr.aria-expanded]="!(isNavBarCollapsed$ | async)">
+                [attr.aria-label]="'nav.toggle' | translate" aria-controls="collapsible-mobile-navbar" [attr.aria-expanded]="(isNavBarCollapsed$ | async) === false">
          <span class="fas fa-bars fa-fw fa-xl toggler-icon" aria-hidden="true"></span>
        </button>
      </div>