From 6c1bd9c241437473875a1c34fb12ff3cc4264991 Mon Sep 17 00:00:00 2001
From: Page Asgardius <asgardius@asgardius.company>
Date: Sun, 28 Aug 2022 07:42:08 -0700
Subject: [PATCH] add error handling to video player

---
 .../asgardius/page/s3manager/VideoPlayer.java | 44 ++++++++++++-------
 app/src/main/res/values/strings.xml           |  1 +
 2 files changed, 30 insertions(+), 15 deletions(-)

diff --git a/app/src/main/java/asgardius/page/s3manager/VideoPlayer.java b/app/src/main/java/asgardius/page/s3manager/VideoPlayer.java
index 25e6047..a835f8c 100644
--- a/app/src/main/java/asgardius/page/s3manager/VideoPlayer.java
+++ b/app/src/main/java/asgardius/page/s3manager/VideoPlayer.java
@@ -4,10 +4,14 @@ import androidx.appcompat.app.AppCompatActivity;
 
 import android.os.Bundle;
 import android.view.View;
+import android.widget.Toast;
 
 import com.google.android.exoplayer2.ExoPlayer;
 import com.google.android.exoplayer2.MediaItem;
+import com.google.android.exoplayer2.PlaybackException;
+import com.google.android.exoplayer2.Player;
 import com.google.android.exoplayer2.ui.StyledPlayerView;
+import com.google.android.exoplayer2.upstream.HttpDataSource;
 
 /**
  * An example full-screen activity that shows and hides the system UI (i.e.
@@ -31,24 +35,34 @@ public class VideoPlayer extends AppCompatActivity {
         playerView = findViewById(R.id.player_view);
         // creating a variable for exoplayer
         player = new ExoPlayer.Builder(this).build();
-        try {
+        // Attach player to the view.
+        playerView.setPlayer(player);
+        MediaItem mediaItem = MediaItem.fromUri(videoURL);
 
-            // Attach player to the view.
-            playerView.setPlayer(player);
-            MediaItem mediaItem = MediaItem.fromUri(videoURL);
+        // Set the media item to be played.
+        player.setMediaItem(mediaItem);
+        // Prepare the player.
+        player.prepare();
+        // Start the playback.
+        player.play();
 
-            // Set the media item to be played.
-            player.setMediaItem(mediaItem);
-            // Prepare the player.
-            player.prepare();
-            // Start the playback.
-            player.play();
 
-        } catch (Exception e) {
-            // below line is used for
-            // handling our errors.
-            System.out.println("Error : " + e.toString());
-        }
+        player.addListener(new Player.Listener() {
+            @Override
+
+            public void onPlayerError(PlaybackException error) {
+                Throwable cause = error.getCause();
+                if (cause instanceof HttpDataSource.HttpDataSourceException) {
+                    // An HTTP error occurred.
+                    //System.out.println("Playback error F");
+                    Toast.makeText(getApplicationContext(),getResources().getString(R.string.media_load_fail), Toast.LENGTH_SHORT).show();
+                    player.release();
+                    finish();
+                }
+            }
+
+
+        });
     }
 
     @Override
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index b8e4e95..f089f4f 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -6,6 +6,7 @@
     <string name="access_key">username</string>
     <string name="secret_key">password</string>
     <string name="video_test_button">Video Test</string>
+    <string name="media_load_fail">Cannot load media file</string>
     <string name="dummy_button">Dummy Button</string>
     <string name="dummy_content">DUMMY\nCONTENT</string>
 </resources>
\ No newline at end of file