87968: Update ESLint disable/enable comments

eslint-disable max-classes-per-file only works at the top of the file now

src/app/core/data/base-response-parsing.service.spec.ts

@@ -1,10 +1,10 @@
+/* eslint-disable max-classes-per-file */
 import { BaseResponseParsingService } from './base-response-parsing.service';
 import { ObjectCacheService } from '../cache/object-cache.service';
 import { CacheableObject } from '../cache/object-cache.reducer';
 import { GetRequest, RestRequest } from './request.models';
 import { DSpaceObject } from '../shared/dspace-object.model';
 
-/* eslint-disable max-classes-per-file */
 class TestService extends BaseResponseParsingService {
   toCache = true;
 
@@ -101,4 +101,3 @@ describe('BaseResponseParsingService', () => {
     });
   });
 });
-/* eslint-enable max-classes-per-file */

src/app/core/data/base-response-parsing.service.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-classes-per-file */
 import { hasNoValue, hasValue, isNotEmpty } from '../../shared/empty.util';
 import { DSpaceSerializer } from '../dspace-rest/dspace.serializer';
 import { CacheableObject } from '../cache/object-cache.reducer';
@@ -10,7 +11,6 @@ import { getClassForType } from '../cache/builders/build-decorators';
 import { RestRequest } from './request.models';
 import { environment } from '../../../environments/environment';
 
-/* eslint-disable max-classes-per-file */
 
 /**
  * Return true if halObj has a value for `_links.self`
@@ -180,4 +180,3 @@ export abstract class BaseResponseParsingService {
     return statusCode >= 200 && statusCode < 300;
   }
 }
-/* eslint-enable max-classes-per-file */

src/app/core/data/configuration-data.service.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-classes-per-file */
 import { HttpClient } from '@angular/common/http';
 import { Injectable } from '@angular/core';
 import { Store } from '@ngrx/store';
@@ -15,7 +16,6 @@ import { ConfigurationProperty } from '../shared/configuration-property.model';
 import { DefaultChangeAnalyzer } from './default-change-analyzer.service';
 import { CONFIG_PROPERTY } from '../shared/config-property.resource-type';
 
-/* eslint-disable max-classes-per-file */
 class DataServiceImpl extends DataService<ConfigurationProperty> {
   protected linkPath = 'properties';
 
@@ -60,4 +60,3 @@ export class ConfigurationDataService {
     return this.dataService.findById(name);
   }
 }
-/* eslint-enable max-classes-per-file */

src/app/core/data/data.service.spec.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-classes-per-file */
 import { HttpClient } from '@angular/common/http';
 import { Store } from '@ngrx/store';
 import { compare, Operation } from 'fast-json-patch';
@@ -26,7 +27,6 @@ import { RequestEntryState } from './request.reducer';
 
 const endpoint = 'https://rest.api/core';
 
-/* eslint-disable max-classes-per-file */
 class TestService extends DataService<any> {
 
   constructor(
@@ -833,4 +833,3 @@ describe('DataService', () => {
     });
   });
 });
-/* eslint-enable max-classes-per-file */

src/app/core/data/dspace-object-data.service.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-classes-per-file */
 import { HttpClient } from '@angular/common/http';
 import { Injectable } from '@angular/core';
 import { Store } from '@ngrx/store';
@@ -18,7 +19,6 @@ import { RequestService } from './request.service';
 import { FindListOptions } from './request.models';
 import { PaginatedList } from './paginated-list.model';
 
-/* eslint-disable max-classes-per-file */
 class DataServiceImpl extends DataService<DSpaceObject> {
   protected linkPath = 'dso';
 
@@ -104,4 +104,3 @@ export class DSpaceObjectDataService {
   }
 
 }
-/* eslint-enable max-classes-per-file */

src/app/core/data/dspace-rest-response-parsing.service.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-classes-per-file */
 import { hasNoValue, hasValue, isNotEmpty } from '../../shared/empty.util';
 import { DSpaceSerializer } from '../dspace-rest/dspace.serializer';
 import { CacheableObject } from '../cache/object-cache.reducer';
@@ -18,7 +19,6 @@ import { RestRequestMethod } from './rest-request-method';
 import { getUrlWithoutEmbedParams, getEmbedSizeParams } from '../index/index.selectors';
 import { URLCombiner } from '../url-combiner/url-combiner';
 
-/* eslint-disable max-classes-per-file */
 
 /**
  * Return true if obj has a value for `_links.self`
@@ -271,4 +271,3 @@ export class DspaceRestResponseParsingService implements ResponseParsingService
     return statusCode >= 200 && statusCode < 300;
   }
 }
-/* eslint-enable max-classes-per-file */

src/app/core/data/href-only-data.service.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-classes-per-file */
 import { DataService } from './data.service';
 import { RequestService } from './request.service';
 import { RemoteDataBuildService } from '../cache/builders/remote-data-build.service';
@@ -20,7 +21,6 @@ import { ITEM_TYPE } from '../shared/item-relationships/item-type.resource-type'
 import { LICENSE } from '../shared/license.resource-type';
 import { CacheableObject } from '../cache/object-cache.reducer';
 
-/* eslint-disable max-classes-per-file */
 class DataServiceImpl extends DataService<any> {
   // linkPath isn't used if we're only searching by href.
   protected linkPath = undefined;

src/app/core/data/item-template-data.service.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-classes-per-file */
 import { Injectable } from '@angular/core';
 import { ItemDataService } from './item-data.service';
 import { UpdateDataService } from './update-data.service';
@@ -24,7 +25,6 @@ import { hasValue } from '../../shared/empty.util';
 import { Operation } from 'fast-json-patch';
 import { getFirstCompletedRemoteData } from '../shared/operators';
 
-/* eslint-disable max-classes-per-file */
 /**
  * A custom implementation of the ItemDataService, but for collection item templates
  * Makes sure to change the endpoint before sending out CRUD requests for the item template
@@ -228,4 +228,3 @@ export class ItemTemplateDataService implements UpdateDataService<Item> {
     return this.dataService.getCollectionEndpoint(collectionID);
   }
 }
-/* eslint-enable max-classes-per-file */

src/app/core/data/object-updates/object-updates.actions.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-classes-per-file */
 import {type} from '../../../shared/ngrx/type';
 import {Action} from '@ngrx/store';
 import {Identifiable} from './object-updates.reducer';
@@ -21,7 +22,6 @@ export const ObjectUpdatesActionTypes = {
   REMOVE_FIELD: type('dspace/core/cache/object-updates/REMOVE_FIELD')
 };
 
-/* eslint-disable max-classes-per-file */
 
 /**
  * Enum that represents the different types of updates that can be performed on a field in the ObjectUpdates store
@@ -283,7 +283,6 @@ export class RemoveFieldUpdateAction implements Action {
   }
 }
 
-/* eslint-enable max-classes-per-file */
 
 /**
  * A type to encompass all ObjectUpdatesActions

src/app/core/data/request.actions.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-classes-per-file */
 import { Action } from '@ngrx/store';
 import { type } from '../../shared/ngrx/type';
 import { RestRequest } from './request.models';
@@ -17,7 +18,6 @@ export const RequestActionTypes = {
   REMOVE: type('dspace/core/data/request/REMOVE')
 };
 
-/* eslint-disable max-classes-per-file */
 export abstract class RequestUpdateAction implements Action {
   abstract type: string;
   lastUpdated: number;
@@ -185,7 +185,6 @@ export class RequestRemoveAction implements Action {
   }
 }
 
-/* eslint-enable max-classes-per-file */
 
 /**
  * A type to encompass all RequestActions

src/app/core/data/request.models.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-classes-per-file */
 import { SortOptions } from '../cache/models/sort-options.model';
 import { GenericConstructor } from '../shared/generic-constructor';
 import { ResponseParsingService } from './parsing.service';
@@ -11,7 +12,6 @@ import { ContentSourceResponseParsingService } from './content-source-response-p
 import { DspaceRestResponseParsingService } from './dspace-rest-response-parsing.service';
 import { environment } from '../../../environments/environment';
 
-/* eslint-disable max-classes-per-file */
 
 // uuid and handle requests have separate endpoints
 export enum IdentifierType {
@@ -280,4 +280,3 @@ export class RequestError extends Error {
   statusCode: number;
   statusText: string;
 }
-/* eslint-enable max-classes-per-file */

src/app/core/data/request.reducer.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-classes-per-file */
 import {
   RequestAction,
   RequestActionTypes,
@@ -112,7 +113,6 @@ export class ResponseState {
   unCacheableObject?: UnCacheableObject;
 }
 
-// eslint-disable-next-line max-classes-per-file
 export class RequestEntry {
   request: RestRequest;
   state: RequestEntryState;

src/app/core/data/root-data.service.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-classes-per-file */
 import { DataService } from './data.service';
 import { Root } from './root.model';
 import { Injectable } from '@angular/core';
@@ -21,7 +22,6 @@ import { DspaceRestService } from '../dspace-rest/dspace-rest.service';
 import { RawRestResponse } from '../dspace-rest/raw-rest-response.model';
 import { catchError, map } from 'rxjs/operators';
 
-/* eslint-disable max-classes-per-file */
 
 /**
  * A private DataService implementation to delegate specific methods to.
@@ -131,4 +131,3 @@ export class RootDataService {
     this.requestService.setStaleByHrefSubstring(this.halService.getRootHref());
   }
 }
-/* eslint-enable max-classes-per-file */