Manually fix lint issues

ybnd · ybnd · commit 3b48d9a26d46 · 2024-03-06T11:56:17.000+01:00
diff --git 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'),
diff --git 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++) {
diff --git 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) {
diff --git 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;' }) : '',
diff --git 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">
diff --git 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;
   };
diff --git 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';
diff --git 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';
diff --git 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)"
diff --git 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>
diff --git 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
     }}"
diff --git 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 @@ <h4 class="border-bottom pb-2">
                         <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"
diff --git 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,19 +4,19 @@
       <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"
                      [retainScrollPosition]="false"
                      (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">
diff --git 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.
diff --git 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());
+          }),
         );
     })),
   );
diff --git 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 @@ <h2 class="flex-grow-1">
 
       </ds-pagination>
 
-      <div *ngIf="processesRD?.payload?.totalElements == 0">
+      <div *ngIf="processesRD?.payload?.totalElements === 0">
         <p>{{'process.overview.table.empty' | translate}}</p>
       </div>
     </ng-container>
diff --git 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">
diff --git 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">
diff --git 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(
diff --git a/src/app/submission/sections/duplicates/section-duplicates.component.html b/src/app/submission/sections/duplicates/section-duplicates.component.html
@@ -3,7 +3,7 @@
 @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">
diff --git 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)"
                     >
diff --git a/src/app/suggestions-page/suggestions-page.component.html b/src/app/suggestions-page/suggestions-page.component.html
@@ -22,7 +22,7 @@ <h1>
               <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"
diff --git a/src/environments/environment.test.ts b/src/environments/environment.test.ts
diff --git a/src/themes/dspace/app/header/header.component.html b/src/themes/dspace/app/header/header.component.html

Original file line number	Diff line number	Diff line change
`@@ -111,7 +111,7 @@ export class FiltersComponent {`
`111`	`111`	`return undefined;`
`112`	`112`	`}`
`113`	`113`
`114`		`- static toQueryString(filters: Object): string {`
	`114`	`+ static toQueryString(filters: { [key: string]: any }): string {`
`115`	`115`	`let params = '';`
`116`	`116`	`let first = true;`
`117`	`117`	`for (const key in filters) {`