diff --git a/lib/content-services/src/lib/content-metadata/services/property-groups-translator.service.spec.ts b/lib/content-services/src/lib/content-metadata/services/property-groups-translator.service.spec.ts index c0f05551558..81a732406f3 100644 --- a/lib/content-services/src/lib/content-metadata/services/property-groups-translator.service.spec.ts +++ b/lib/content-services/src/lib/content-metadata/services/property-groups-translator.service.spec.ts @@ -25,11 +25,11 @@ import { CardViewDateItemModel, CardViewIntItemModel, CardViewFloatItemModel, - NotificationService, CardViewBoolItemModel, CardViewDatetimeItemModel, CardViewSelectItemModel, - CardViewSelectItemProperties + CardViewSelectItemProperties, + LogService } from '@alfresco/adf-core'; import { ContentTestingModule } from '../../testing/content.testing.module'; import { TranslateModule } from '@ngx-translate/core'; @@ -42,7 +42,7 @@ describe('PropertyGroupTranslatorService', () => { let propertyGroup: OrganisedPropertyGroup; let property: Property; let propertyValues: { [key: string]: any }; - let notificationService: NotificationService; + let logService: LogService; beforeEach(() => { TestBed.configureTestingModule({ @@ -51,7 +51,7 @@ describe('PropertyGroupTranslatorService', () => { ContentTestingModule ] }); - notificationService = TestBed.inject(NotificationService); + logService = TestBed.inject(LogService); service = TestBed.inject(PropertyGroupTranslatorService); property = { @@ -137,7 +137,7 @@ describe('PropertyGroupTranslatorService', () => { }); it('should log an error if unrecognised type is found', () => { - const showError = spyOn(notificationService, 'showError').and.stub(); + const logServiceError = spyOn(logService, 'error').and.stub(); property.name = 'FAS:PLAGUE'; property.title = 'The Faro Plague'; @@ -149,7 +149,7 @@ describe('PropertyGroupTranslatorService', () => { propertyGroups.push(Object.assign({}, propertyGroup)); service.translateToCardViewGroups(propertyGroups, propertyValues, null); - expect(showError).toHaveBeenCalledWith('Unknown type for mapping: daemonic:scorcher'); + expect(logServiceError).toHaveBeenCalledWith('Unknown type for mapping: daemonic:scorcher'); }); it('should fall back to single-line property type if unrecognised type is found', () => { diff --git a/lib/content-services/src/lib/content-metadata/services/property-groups-translator.service.ts b/lib/content-services/src/lib/content-metadata/services/property-groups-translator.service.ts index 2126dc7c74f..e2696fb7870 100644 --- a/lib/content-services/src/lib/content-metadata/services/property-groups-translator.service.ts +++ b/lib/content-services/src/lib/content-metadata/services/property-groups-translator.service.ts @@ -15,7 +15,7 @@ * limitations under the License. */ -import { Injectable, inject } from '@angular/core'; +import { Injectable } from '@angular/core'; import { CardViewItemProperties, CardViewItem, @@ -29,7 +29,7 @@ import { MultiValuePipe, AppConfigService, DecimalNumberPipe, - NotificationService + LogService } from '@alfresco/adf-core'; import { Property, CardViewGroup, OrganisedPropertyGroup } from '../interfaces/content-metadata.interfaces'; import { of } from 'rxjs'; @@ -51,12 +51,12 @@ export const RECOGNISED_ECM_TYPES = [D_TEXT, D_MLTEXT, D_DATE, D_DATETIME, D_INT providedIn: 'root' }) export class PropertyGroupTranslatorService { - private notificationService = inject(NotificationService); valueSeparator: string; constructor(private multiValuePipe: MultiValuePipe, private decimalNumberPipe: DecimalNumberPipe, - private appConfig: AppConfigService) { + private appConfig: AppConfigService, + private logService: LogService) { this.valueSeparator = this.appConfig.get('content-metadata.multi-value-pipe-separator'); }