[DURACOM-367] fix matomo SSR error on bitstream download page

(cherry picked from commit 37cef0f)

# Conflicts:
#	src/app/shared/cookies/klaro.service.ts
#	src/app/statistics/matomo.service.spec.ts
#	src/app/statistics/matomo.service.ts

Author: Andrea Barbasso

src/app/footer/footer.component.ts

@@ -61,7 +61,7 @@ export class FooterComponent implements OnInit {
   }
 
   showCookieSettings() {
-    if (hasValue(this.cookies)) {
+    if (hasValue(this.cookies) && this.cookies.showSettings instanceof Function) {
       this.cookies.showSettings();
     }
     return false;

src/app/shared/cookies/klaro.service.ts

@@ -4,7 +4,7 @@ import { Observable } from 'rxjs';
 /**
  * Abstract class representing a service for handling Klaro consent preferences and UI
  */
-@Injectable()
+@Injectable({ providedIn: 'root' })
 export abstract class KlaroService {
   /**
    * Initializes the service

src/app/statistics/matomo.service.spec.ts

@@ -1,3 +1,7 @@
+import {
+  Injector,
+  runInInjectionContext,
+} from '@angular/core';
 import {
   fakeAsync,
   TestBed,
@@ -53,6 +57,7 @@ describe('MatomoService', () => {
         { provide: KlaroService, useValue: klaroService },
         { provide: NativeWindowService, useValue: nativeWindowService },
         { provide: ConfigurationDataService, useValue: configService },
+        { provide: Injector, useValue: TestBed },
       ],
     });
 
@@ -70,11 +75,13 @@ describe('MatomoService', () => {
   });
 
   it('should call setConsentGiven when consent is true', () => {
+    service.matomoTracker = matomoTracker;
     service.changeMatomoConsent(true);
     expect(matomoTracker.setConsentGiven).toHaveBeenCalled();
   });
 
   it('should call forgetConsentGiven when consent is false', () => {
+    service.matomoTracker = matomoTracker;
     service.changeMatomoConsent(false);
     expect(matomoTracker.forgetConsentGiven).toHaveBeenCalled();
   });
@@ -91,7 +98,10 @@ describe('MatomoService', () => {
     configService.findByPropertyName.withArgs(MATOMO_SITE_ID).and.returnValue(
       createSuccessfulRemoteDataObject$(Object.assign(new ConfigurationProperty(), { values: ['1'] })));
     klaroService.getSavedPreferences.and.returnValue(of({ matomo: true }));
-    service.init();
+
+    runInInjectionContext(TestBed, () => {
+      service.init();
+    });
 
     expect(matomoTracker.setConsentGiven).toHaveBeenCalled();
     expect(matomoInitializer.initializeTracker).toHaveBeenCalledWith({
@@ -100,7 +110,7 @@ describe('MatomoService', () => {
     });
   });
 
-  it('should initialize tracker with REST configuration correct parameters in production', () => {
+  it('should initialize tracker with REST configuration correct parameters in production', fakeAsync(() => {
     environment.production = true;
     environment.matomo = { trackerUrl: '' };
     configService.findByPropertyName.withArgs(MATOMO_TRACKER_URL).and.returnValue(
@@ -113,19 +123,25 @@ describe('MatomoService', () => {
       createSuccessfulRemoteDataObject$(Object.assign(new ConfigurationProperty(), { values: ['1'] })));
     klaroService.getSavedPreferences.and.returnValue(of({ matomo: true }));
 
-    service.init();
+    runInInjectionContext(TestBed, () => {
+      service.init();
+    });
+
+    tick();
 
     expect(matomoTracker.setConsentGiven).toHaveBeenCalled();
     expect(matomoInitializer.initializeTracker).toHaveBeenCalledWith({
       siteId: '1',
       trackerUrl: 'http://matomo',
     });
-  });
+  }));
 
   it('should not initialize tracker if not in production', () => {
     environment.production = false;
 
-    service.init();
+    runInInjectionContext(TestBed, () => {
+      service.init();
+    });
 
     expect(matomoInitializer.initializeTracker).not.toHaveBeenCalled();
   });
@@ -141,16 +157,19 @@ describe('MatomoService', () => {
     );
     configService.findByPropertyName.withArgs(MATOMO_SITE_ID).and.returnValue(
       createSuccessfulRemoteDataObject$(Object.assign(new ConfigurationProperty(), { values: ['1'] })));
-    orejimeService.getSavedPreferences.and.returnValue(of({ matomo: true }));
+    klaroService.getSavedPreferences.and.returnValue(of({ matomo: true }));
 
-    service.init();
+    runInInjectionContext(TestBed, () => {
+      service.init();
+    });
 
     expect(matomoInitializer.initializeTracker).not.toHaveBeenCalled();
   });
 
   describe('with visitorId set', () => {
     beforeEach(() => {
       matomoTracker.getVisitorId.and.returnValue(Promise.resolve('12345'));
+      service.matomoTracker = matomoTracker;
     });
 
     it('should add trackerId parameter', fakeAsync(() => {

src/app/statistics/matomo.service.ts

@@ -1,6 +1,8 @@
 import {
+  EnvironmentInjector,
   inject,
   Injectable,
+  runInInjectionContext,
 } from '@angular/core';
 import {
   MatomoInitializerService,
@@ -11,12 +13,10 @@ import {
   from as fromPromise,
   Observable,
   of,
-  switchMap,
 } from 'rxjs';
 import {
   map,
   take,
-  tap,
 } from 'rxjs/operators';
 
 import { environment } from '../../environments/environment';
@@ -31,6 +31,12 @@ import { isNotEmpty } from '../shared/empty.util';
 export const MATOMO_TRACKER_URL = 'matomo.tracker.url';
 export const MATOMO_SITE_ID = 'matomo.request.siteid';
 
+export const MATOMO_ENABLED = 'matomo.enabled';
+
+/**
+ * Service to manage Matomo analytics integration.
+ * Handles initialization and consent management for Matomo tracking.
+ */
 @Injectable({
   providedIn: 'root',
 })
@@ -40,14 +46,21 @@ export const MATOMO_SITE_ID = 'matomo.request.siteid';
  */
 export class MatomoService {
 
-  matomoInitializer = inject(MatomoInitializerService);
-  matomoTracker = inject(MatomoTracker);
+  /** Injects the MatomoInitializerService to initialize the Matomo tracker. */
+  matomoInitializer: MatomoInitializerService;
+
+  /** Injects the MatomoTracker to manage Matomo tracking operations. */
+  matomoTracker: MatomoTracker;
   klaroService = inject(KlaroService);
   _window = inject(NativeWindowService);
 
   /** Injects the ConfigurationService. */
   configService = inject(ConfigurationDataService);
 
+  constructor(private injector: EnvironmentInjector) {
+
+  }
+
   /**
    * Initializes the Matomo tracker if in production environment.
    * Sets up the changeMatomoConsent function on the native window object.
@@ -61,24 +74,29 @@ export class MatomoService {
     if (environment.production) {
       const preferences$ = this.klaroService.getSavedPreferences();
 
-      preferences$
-        .pipe(
-          tap(preferences => this.changeMatomoConsent(preferences?.matomo)),
-          switchMap(_ => combineLatest([this.isMatomoEnabled$(), this.getSiteId$(), this.getTrackerUrl$()])),
-        )
-        .subscribe(([isMatomoEnabled, siteId, trackerUrl]) => {
+      combineLatest([preferences$, this.isMatomoEnabled$(), this.getSiteId$(), this.getTrackerUrl$()])
+        .subscribe(([preferences, isMatomoEnabled, siteId, trackerUrl]) => {
           if (isMatomoEnabled && siteId && trackerUrl) {
+            runInInjectionContext(this.injector, () => {
+              this.matomoTracker = inject(MatomoTracker);
+              this.matomoInitializer = inject(MatomoInitializerService);
+            });
             this.matomoInitializer.initializeTracker({ siteId, trackerUrl });
+            this.changeMatomoConsent(preferences?.matomo);
           }
         });
     }
   }
 
+  /**
+   * Changes the Matomo consent status based on the given consent value.
+   * @param consent - A boolean indicating whether consent is given for Matomo tracking.
+   */
   changeMatomoConsent = (consent: boolean) => {
     if (consent) {
-      this.matomoTracker.setConsentGiven();
+      this.matomoTracker?.setConsentGiven();
     } else {
-      this.matomoTracker.forgetConsentGiven();
+      this.matomoTracker?.forgetConsentGiven();
     }
   };
 
@@ -88,7 +106,7 @@ export class MatomoService {
    * @returns An Observable that emits the URL with the visitor ID appended.
    */
   appendVisitorId(url: string): Observable<string> {
-    return fromPromise(this.matomoTracker.getVisitorId())
+    return fromPromise(this.matomoTracker?.getVisitorId())
       .pipe(
         map(visitorId => this.appendTrackerId(url, visitorId)),
         take(1),
@@ -128,6 +146,21 @@ export class MatomoService {
       );
   }
 
+  /**
+   * Checks if Matomo tracking is enabled by retrieving the configuration property.
+   * @returns An Observable that emits a boolean indicating whether Matomo tracking is enabled.
+   */
+  isMatomoEnabled$(): Observable<boolean> {
+    return this.configService.findByPropertyName(MATOMO_ENABLED)
+      .pipe(
+        getFirstCompletedRemoteData(),
+        map((res: RemoteData<ConfigurationProperty>) => {
+          return res.hasSucceeded && res.payload && isNotEmpty(res.payload.values) &&
+            res.payload.values[0]?.toLowerCase() === 'true';
+        }),
+      );
+  }
+
   /**
    * Appends the visitor ID as a query parameter to the given URL.
    * @param url - The original URL to modify