From abd540325f923117bbe3d3433e242d1db530c85c Mon Sep 17 00:00:00 2001
From: Eugenio Romano <eromano@users.noreply.github.com>
Date: Fri, 22 Apr 2022 16:26:30 +0100
Subject: [PATCH] [AAE-8565] multiple use viewer in form (#7594)

* multiple use viewer

multiple use in the form of viewer needs multiple instance of ViewUtilService

* Update viewer.component.spec.ts

* Update viewer.component.spec.ts

* Update viewer.component.spec.ts

* Update viewer.component.spec.ts
---
 .../components/viewer.component.spec.ts       | 58 ++++++++++++++++++-
 .../viewer/components/viewer.component.ts     |  3 +-
 2 files changed, 58 insertions(+), 3 deletions(-)

diff --git a/lib/core/viewer/components/viewer.component.spec.ts b/lib/core/viewer/components/viewer.component.spec.ts
index 375ce7b822b..9c84e42f60b 100644
--- a/lib/core/viewer/components/viewer.component.spec.ts
+++ b/lib/core/viewer/components/viewer.component.spec.ts
@@ -17,7 +17,7 @@
 
 import { Location } from '@angular/common';
 import { SpyLocation } from '@angular/common/testing';
-import { Component } from '@angular/core';
+import { Component, ViewChild } from '@angular/core';
 import { ComponentFixture, TestBed, fakeAsync, tick } from '@angular/core/testing';
 import { AlfrescoApiService, RenditionsService } from '../../services';
 
@@ -132,6 +132,23 @@ class ViewerWithCustomOpenWithComponent {
 })
 class ViewerWithCustomMoreActionsComponent {
 }
+
+@Component({
+    selector: 'adf-double-viewer',
+    template: `
+        <adf-viewer #viewer1></adf-viewer>
+        <adf-viewer #viewer2></adf-viewer>
+    `
+})
+class DoubleViewerComponent {
+    @ViewChild('viewer1')
+    viewer1: ViewerComponent;
+
+    @ViewChild('viewer2')
+    viewer2: ViewerComponent;
+
+}
+
 describe('ViewerComponent', () => {
 
     let component: ViewerComponent;
@@ -151,6 +168,7 @@ describe('ViewerComponent', () => {
             MatIconModule
         ],
         declarations: [
+            DoubleViewerComponent,
             ViewerWithCustomToolbarComponent,
             ViewerWithCustomSidebarComponent,
             ViewerWithCustomOpenWithComponent,
@@ -184,6 +202,36 @@ describe('ViewerComponent', () => {
         fixture.destroy();
     });
 
+    describe('Double viewer Test', () => {
+
+        it('should not reload the content of all the viewer after type change', async () => {
+            const fixtureDouble = TestBed.createComponent(DoubleViewerComponent);
+
+            await fixtureDouble.detectChanges();
+            await fixtureDouble.whenStable();
+
+            fixtureDouble.componentInstance.viewer1.urlFile = 'fake-test-file.pdf';
+            fixtureDouble.componentInstance.viewer2.urlFile = 'fake-test-file-two.xls';
+
+            fixtureDouble.componentInstance.viewer1.ngOnChanges();
+            fixtureDouble.componentInstance.viewer2.ngOnChanges();
+
+            await fixtureDouble.detectChanges();
+            await fixtureDouble.whenStable();
+
+            expect(fixtureDouble.componentInstance.viewer1.viewerType).toBe('pdf');
+            expect(fixtureDouble.componentInstance.viewer2.viewerType).toBe('unknown');
+
+            fixtureDouble.componentInstance.viewer1.urlFile = 'fake-test-file.pdf';
+            fixtureDouble.componentInstance.viewer2.urlFile = 'fake-test-file-two.png';
+
+            (fixtureDouble.componentInstance.viewer2 as any).viewUtilService.viewerTypeChange.next('png');
+
+            expect(fixtureDouble.componentInstance.viewer1.viewerType).toBe('pdf');
+            expect(fixtureDouble.componentInstance.viewer2.viewerType).toBe('png');
+        });
+    });
+
     describe('Extension Type Test', () => {
         it('should display pdf external viewer via wildcard notation', async () => {
             const extension: ViewerExtensionRef = {
@@ -1273,7 +1321,13 @@ describe('ViewerComponent', () => {
         describe('display name property override by nodeId', () => {
 
             const contentUrl = '/content/url/path';
-            const nodeDetails = new NodeEntry({ entry: { name: 'node-id-name', id: '12', content: { mimeType: 'txt' } } });
+            const nodeDetails = new NodeEntry({
+                entry: {
+                    name: 'node-id-name',
+                    id: '12',
+                    content: { mimeType: 'txt' }
+                }
+            });
 
             it('should use the node name if displayName is NOT set and nodeId is set', (done) => {
                 spyOn(component['nodesApi'], 'getNode').and.returnValue(Promise.resolve(nodeDetails));
diff --git a/lib/core/viewer/components/viewer.component.ts b/lib/core/viewer/components/viewer.component.ts
index 8db5b108780..0786c81116d 100644
--- a/lib/core/viewer/components/viewer.component.ts
+++ b/lib/core/viewer/components/viewer.component.ts
@@ -50,7 +50,8 @@ import { FileModel } from '../../models';
     templateUrl: './viewer.component.html',
     styleUrls: ['./viewer.component.scss'],
     host: { class: 'adf-viewer' },
-    encapsulation: ViewEncapsulation.None
+    encapsulation: ViewEncapsulation.None,
+    providers: [ViewUtilService]
 })
 export class ViewerComponent implements OnChanges, OnInit, OnDestroy {