From d17b8741562056ee28a949d6d23c904f732ef829 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?D=C3=A1niel=20P=C3=A9ter=20Sipos?= <dsipos@monguz.hu>
Date: Fri, 6 Nov 2020 13:40:28 +0100
Subject: [PATCH] #885 remove auto play and filter unsupported media types

---
 .../media-viewer-image.component.ts           | 20 ++++++++++---------
 .../media-viewer-video.component.html         | 14 ++++++-------
 .../media-viewer-video.component.ts           |  5 +++++
 3 files changed, 23 insertions(+), 16 deletions(-)

diff --git a/src/app/+item-page/media-viewer/media-viewer-image/media-viewer-image.component.ts b/src/app/+item-page/media-viewer/media-viewer-image/media-viewer-image.component.ts
index f0a5962e1f..9d7b3da45a 100644
--- a/src/app/+item-page/media-viewer/media-viewer-image/media-viewer-image.component.ts
+++ b/src/app/+item-page/media-viewer/media-viewer-image/media-viewer-image.component.ts
@@ -73,15 +73,17 @@ export class MediaViewerImageComponent implements OnInit {
   convertToGalleryImage(medias: MediaViewerItem[]): NgxGalleryImage[] {
     const mappadImages = [];
     for (const image of medias) {
-      mappadImages.push({
-        small: image.thumbnail
-          ? image.thumbnail
-          : './assets/images/replacement_image.svg',
-        medium: image.thumbnail
-          ? image.thumbnail
-          : './assets/images/replacement_image.svg',
-        big: image.bitstream._links.content.href,
-      });
+      if (image.format === 'image') {
+        mappadImages.push({
+          small: image.thumbnail
+            ? image.thumbnail
+            : './assets/images/replacement_image.svg',
+          medium: image.thumbnail
+            ? image.thumbnail
+            : './assets/images/replacement_image.svg',
+          big: image.bitstream._links.content.href,
+        });
+      }
     }
     return mappadImages;
   }
diff --git a/src/app/+item-page/media-viewer/media-viewer-video/media-viewer-video.component.html b/src/app/+item-page/media-viewer/media-viewer-video/media-viewer-video.component.html
index 1288af99b1..27d36c8a1b 100644
--- a/src/app/+item-page/media-viewer/media-viewer-video/media-viewer-video.component.html
+++ b/src/app/+item-page/media-viewer/media-viewer-video/media-viewer-video.component.html
@@ -1,15 +1,15 @@
 <video
   #media
-  [src]="medias[currentIndex].bitstream._links.content.href"
+  [src]="filteredMedias[currentIndex].bitstream._links.content.href"
   id="singleVideo"
   [poster]="
-    medias[currentIndex].thumbnail ||
-    this.replacements[this.medias[this.currentIndex].format]
+    filteredMedias[currentIndex].thumbnail ||
+    replacements[filteredMedias[currentIndex].format]
   "
-  preload="auto"
+  preload="none"
   controls
 ></video>
-<div class="buttons" *ngIf="medias?.length > 1">
+<div class="buttons" *ngIf="filteredMedias?.length > 1">
   <button
     class="btn btn-primary previous"
     [disabled]="currentIndex === 0"
@@ -20,7 +20,7 @@
 
   <button
     class="btn btn-primary next"
-    [disabled]="currentIndex === medias.length - 1"
+    [disabled]="currentIndex === filteredMedias.length - 1"
     (click)="nextMedia()"
   >
     {{ "media-viewer.next" | translate }}
@@ -36,7 +36,7 @@
     <div ngbDropdownMenu aria-labelledby="dropdownBasic1">
       <button
         ngbDropdownItem
-        *ngFor="let item of medias; let i = index"
+        *ngFor="let item of filteredMedias; let i = index"
         class="list-element"
         (click)="selectedMedia(i)"
       >
diff --git a/src/app/+item-page/media-viewer/media-viewer-video/media-viewer-video.component.ts b/src/app/+item-page/media-viewer/media-viewer-video/media-viewer-video.component.ts
index 1b74a69e42..4c578a51bb 100644
--- a/src/app/+item-page/media-viewer/media-viewer-video/media-viewer-video.component.ts
+++ b/src/app/+item-page/media-viewer/media-viewer-video/media-viewer-video.component.ts
@@ -12,6 +12,8 @@ import { MediaViewerItem } from '../../../core/shared/media-viewer-item.model';
 export class MediaViewerVideoComponent implements OnInit {
   @Input() medias: MediaViewerItem[];
 
+  filteredMedias: MediaViewerItem[];
+
   isCollapsed: boolean;
   currentIndex = 0;
 
@@ -24,6 +26,9 @@ export class MediaViewerVideoComponent implements OnInit {
 
   ngOnInit() {
     this.isCollapsed = false;
+    this.filteredMedias = this.medias.filter(
+      (media) => media.format === 'audio' || media.format === 'video'
+    );
   }
 
   /**