@picsa/capacitor-video-player 8.0.2 → 8.0.4

package/android/src/main/java/com/jeep/plugin/capacitor/capacitorvideoplayer/FullscreenExoPlayerFragment.java

@@ -301,7 +301,9 @@ public class FullscreenExoPlayerFragment extends Fragment {
       new PlayerView.ControllerVisibilityListener() {
         @Override
         public void onVisibilityChanged(int visibility) {
-          linearLayout.setVisibility(visibility);
+          if (linearLayout != null) {
+            linearLayout.setVisibility(visibility);
+          }
         }
       }
     );
@@ -310,6 +312,9 @@ public class FullscreenExoPlayerFragment extends Fragment {
       new Player.Listener() {
         @Override
         public void onPlayerStateChanged(boolean playWhenReady, int state) {
+          if (isFragmentDestroyed || player == null || styledPlayerView == null) {
+            return;
+          }
           String stateString;
           Map<String, Object> info = new HashMap<String, Object>() {
             {
@@ -430,14 +435,17 @@ public class FullscreenExoPlayerFragment extends Fragment {
     }
     if (uri != null || Boolean.TRUE.equals(isInternal)) {
       // go fullscreen
-      getActivity().getWindow().addFlags(WindowManager.LayoutParams.FLAG_FULLSCREEN);
-      getActivity().getWindow().addFlags(WindowManager.LayoutParams.FLAG_KEEP_SCREEN_ON);
+      Activity hostActivity = getActivity();
+      if (hostActivity != null && hostActivity.getWindow() != null) {
+        hostActivity.getWindow().addFlags(WindowManager.LayoutParams.FLAG_FULLSCREEN);
+        hostActivity.getWindow().addFlags(WindowManager.LayoutParams.FLAG_KEEP_SCREEN_ON);
+      }
       if (savedInstanceState != null) {
         mCurrentPosition = savedInstanceState.getInt(PLAYBACK_TIME);
       }
 
-      getActivity()
-        .runOnUiThread(
+      if (hostActivity != null) {
+        hostActivity.runOnUiThread(
           new Runnable() {
             @Override
             public void run() {
@@ -511,10 +519,11 @@ public class FullscreenExoPlayerFragment extends Fragment {
             }
           }
         );
-    } else {
-      Log.d(TAG, "Video path wrong or type not supported");
-      Toast.makeText(context, "Video path wrong or type not supported", Toast.LENGTH_SHORT).show();
-    }
+        }
+      } else {
+        Log.d(TAG, "Video path wrong or type not supported");
+        Toast.makeText(context, "Video path wrong or type not supported", Toast.LENGTH_SHORT).show();
+      }
     adjustAspectRatio();
     return view;
   }
@@ -526,7 +535,7 @@ public class FullscreenExoPlayerFragment extends Fragment {
 
     protected Bitmap doInBackground(Void... params) {
       final String image = artwork;
-      if (image != "") {
+      if (image != null && !image.trim().isEmpty()) {
         try {
           URL url = new URL(image);
           HttpURLConnection connection = (HttpURLConnection) url.openConnection();
@@ -535,7 +544,7 @@ public class FullscreenExoPlayerFragment extends Fragment {
           InputStream input = connection.getInputStream();
           Bitmap myBitmap = BitmapFactory.decodeStream(input);
           return myBitmap;
-        } catch (IOException e) {
+        } catch (IOException | IllegalArgumentException e) {
           e.printStackTrace();
           return null;
         }
@@ -546,7 +555,9 @@ public class FullscreenExoPlayerFragment extends Fragment {
 
     @Override
     protected void onPostExecute(Bitmap result) {
-      cast_image.setImageBitmap(result);
+      if (cast_image != null) {
+        cast_image.setImageBitmap(result);
+      }
     }
   }
 
@@ -554,14 +565,16 @@ public class FullscreenExoPlayerFragment extends Fragment {
    * Show controller
    */
   public void showController() {
-    styledPlayerView.showController();
+    if (styledPlayerView != null) {
+      styledPlayerView.showController();
+    }
   }
 
   /**
    * isControllerIsFullyVisible
    */
   public boolean isControllerIsFullyVisible() {
-    return styledPlayerView.isControllerFullyVisible();
+    return styledPlayerView != null && styledPlayerView.isControllerFullyVisible();
   }
 
   /**
@@ -587,6 +600,7 @@ public class FullscreenExoPlayerFragment extends Fragment {
     if (
       !isInPictureInPictureMode &&
         Build.VERSION.SDK_INT >= Build.VERSION_CODES.N &&
+        packageManager != null &&
         packageManager.hasSystemFeature(PackageManager.FEATURE_PICTURE_IN_PICTURE) &&
         Boolean.TRUE.equals(isPIPModeeEnabled) &&
         Boolean.TRUE.equals(pipEnabled) &&
@@ -599,18 +613,21 @@ public class FullscreenExoPlayerFragment extends Fragment {
   }
 
   private void resizePressed() {
+    if (styledPlayerView == null) {
+      return;
+    }
     if (resizeStatus == AspectRatioFrameLayout.RESIZE_MODE_FIT) {
       styledPlayerView.setResizeMode(AspectRatioFrameLayout.RESIZE_MODE_FILL);
       resizeStatus = AspectRatioFrameLayout.RESIZE_MODE_FILL;
-      resizeBtn.setImageResource(R.drawable.ic_zoom);
+      if (resizeBtn != null) resizeBtn.setImageResource(R.drawable.ic_zoom);
     } else if (resizeStatus == AspectRatioFrameLayout.RESIZE_MODE_FILL) {
       styledPlayerView.setResizeMode(AspectRatioFrameLayout.RESIZE_MODE_ZOOM);
       resizeStatus = AspectRatioFrameLayout.RESIZE_MODE_ZOOM;
-      resizeBtn.setImageResource(R.drawable.ic_fit);
+      if (resizeBtn != null) resizeBtn.setImageResource(R.drawable.ic_fit);
     } else {
       styledPlayerView.setResizeMode(AspectRatioFrameLayout.RESIZE_MODE_FIT);
       resizeStatus = AspectRatioFrameLayout.RESIZE_MODE_FIT;
-      resizeBtn.setImageResource(R.drawable.ic_expand);
+      if (resizeBtn != null) resizeBtn.setImageResource(R.drawable.ic_expand);
     }
   }
 
@@ -645,7 +662,7 @@ public class FullscreenExoPlayerFragment extends Fragment {
    * Perform pictureInPictureMode Action
    */
   private void pictureInPictureMode() {
-    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N && packageManager.hasSystemFeature(PackageManager.FEATURE_PICTURE_IN_PICTURE)) {
+    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N && packageManager != null && packageManager.hasSystemFeature(PackageManager.FEATURE_PICTURE_IN_PICTURE)) {
       // Store activity reference to avoid multiple getActivity() calls and check for null
       // Fragment may be detached from activity during transitions or configuration changes
       Activity activity = getActivity();
@@ -653,9 +670,13 @@ public class FullscreenExoPlayerFragment extends Fragment {
         Log.v(TAG, "pictureInPictureMode: activity is null");
         return;
       }
-      styledPlayerView.setUseController(false);
-      styledPlayerView.setControllerAutoShow(false);
-      linearLayout.setVisibility(View.INVISIBLE);
+      if (styledPlayerView != null) {
+        styledPlayerView.setUseController(false);
+        styledPlayerView.setControllerAutoShow(false);
+      }
+      if (linearLayout != null) {
+        linearLayout.setVisibility(View.INVISIBLE);
+      }
       Log.v(TAG, "PIP break 1");
       // require android O or higher
       if (
@@ -663,9 +684,13 @@ public class FullscreenExoPlayerFragment extends Fragment {
       ) {
         pictureInPictureParams = new PictureInPictureParams.Builder();
         // setup aspect ratio of the PIP window; fall back to 16:9 for audio-only or not-yet-decoded video
-        VideoSize videoSize = player.getVideoSize();
-        if (videoSize != VideoSize.UNKNOWN) {
-          pictureInPictureParams.setAspectRatio(new Rational(videoSize.width, videoSize.height));
+        if (player != null) {
+          VideoSize videoSize = player.getVideoSize();
+          if (videoSize != VideoSize.UNKNOWN) {
+            pictureInPictureParams.setAspectRatio(new Rational(videoSize.width, videoSize.height));
+          } else {
+            pictureInPictureParams.setAspectRatio(new Rational(16, 9));
+          }
         } else {
           pictureInPictureParams.setAspectRatio(new Rational(16, 9));
         }
@@ -726,12 +751,10 @@ public class FullscreenExoPlayerFragment extends Fragment {
         if (castContext != null && Boolean.TRUE.equals(chromecast) && castPlayer != null && castPlayer.isCastSessionAvailable()) return;
 
         initializePlayer();
-        if (player.getCurrentPosition() != 0) {
+        if (player != null && player.getCurrentPosition() != 0) {
           firstReadyToPlay = false;
           play();
         }
-      } else {
-        getActivity().finishAndRemoveTask();
       }
     }
   }
@@ -743,11 +766,16 @@ public class FullscreenExoPlayerFragment extends Fragment {
   public void onStop() {
     super.onStop();
     boolean isAppBackground = false;
-    if (Boolean.TRUE.equals(bkModeEnabled)) isAppBackground = isApplicationSentToBackground(context);
+    if (context != null && Boolean.TRUE.equals(bkModeEnabled)) isAppBackground = isApplicationSentToBackground(context);
     if (isInPictureInPictureMode) {
-      linearLayout.setVisibility(View.VISIBLE);
+      if (linearLayout != null) {
+        linearLayout.setVisibility(View.VISIBLE);
+      }
       playerExit();
-      getActivity().finishAndRemoveTask();
+      Activity activity = getActivity();
+      if (activity != null) {
+        activity.finishAndRemoveTask();
+      }
     }
   }
 
@@ -756,12 +784,12 @@ public class FullscreenExoPlayerFragment extends Fragment {
    */
   @Override
   public void onDestroy() {
-    isFragmentDestroyed = true;
-    
-    // Release player before calling super to ensure proper cleanup order
+    // Release player before setting isFragmentDestroyed and calling super to ensure proper cleanup order
     releasePlayer();
+    isFragmentDestroyed = true;
     
-    // Clean up handler
+    // Clean up handlers
+    handler.removeCallbacks(mRunnable);
     if (mainHandler != null) {
         mainHandler.removeCallbacksAndMessages(null);
         mainHandler = null;
@@ -781,6 +809,18 @@ public class FullscreenExoPlayerFragment extends Fragment {
     super.onDetach();
   }
 
+  /**
+   * Perform onDestroyView Action
+   */
+  @Override
+  public void onDestroyView() {
+    if (styledPlayerView != null) {
+      styledPlayerView.setPlayer(null);
+      styledPlayerView = null;
+    }
+    super.onDestroyView();
+  }
+
   /**
    * Perform onPause Action
    */
@@ -791,7 +831,7 @@ public class FullscreenExoPlayerFragment extends Fragment {
       castContext.removeCastStateListener(castStateListener);
     }
     boolean isAppBackground = false;
-    if (Boolean.TRUE.equals(bkModeEnabled)) isAppBackground = isApplicationSentToBackground(context);
+    if (context != null && Boolean.TRUE.equals(bkModeEnabled)) isAppBackground = isApplicationSentToBackground(context);
 
     if (!isInPictureInPictureMode) {
       if (Util.SDK_INT < 24) {
@@ -807,7 +847,7 @@ public class FullscreenExoPlayerFragment extends Fragment {
         }
       }
     } else {
-      if (linearLayout.getVisibility() == View.VISIBLE) {
+      if (linearLayout != null && linearLayout.getVisibility() == View.VISIBLE) {
         linearLayout.setVisibility(View.INVISIBLE);
       }
       if ((isInPictureInPictureMode || isAppBackground) && player != null) play();
@@ -843,6 +883,9 @@ public class FullscreenExoPlayerFragment extends Fragment {
   }
 
   private void releasePlayerInternal() {
+    if (styledPlayerView != null) {
+      styledPlayerView.setPlayer(null);
+    }
     if (player != null && !isFragmentDestroyed) {
         try {
             playWhenReady = player.getPlayWhenReady();
@@ -906,7 +949,9 @@ public class FullscreenExoPlayerFragment extends Fragment {
     } else {
       isInPictureInPictureMode = false;
       if (
-        Build.VERSION.SDK_INT >= Build.VERSION_CODES.O && packageManager.hasSystemFeature(PackageManager.FEATURE_PICTURE_IN_PICTURE)
+        Build.VERSION.SDK_INT >= Build.VERSION_CODES.O &&
+        packageManager != null &&
+        packageManager.hasSystemFeature(PackageManager.FEATURE_PICTURE_IN_PICTURE)
       ) {
         boolean isShowControls = showControls != null ? showControls : true;
         if (!isShowControls) {
@@ -940,9 +985,15 @@ public class FullscreenExoPlayerFragment extends Fragment {
    * Leave the fullsreen mode and reset the status bar to visible
    */
   private void showSystemUI() {
-    getActivity().getWindow().clearFlags(WindowManager.LayoutParams.FLAG_FULLSCREEN);
-    getActivity().getWindow().clearFlags(WindowManager.LayoutParams.FLAG_KEEP_SCREEN_ON);
-    getActivity().getWindow().getDecorView().setSystemUiVisibility(View.VISIBLE);
+    Activity activity = getActivity();
+    if (activity != null && activity.getWindow() != null) {
+      activity.getWindow().clearFlags(WindowManager.LayoutParams.FLAG_FULLSCREEN);
+      activity.getWindow().clearFlags(WindowManager.LayoutParams.FLAG_KEEP_SCREEN_ON);
+      View decorView = activity.getWindow().getDecorView();
+      if (decorView != null) {
+        decorView.setSystemUiVisibility(View.VISIBLE);
+      }
+    }
   }
 
   /**
@@ -954,14 +1005,19 @@ public class FullscreenExoPlayerFragment extends Fragment {
     if (styledPlayerView == null) {
       return;
     }
+    Context playerContext = context != null ? context : getContext();
+    if (playerContext == null) {
+      Log.w(TAG, "initializePlayer: context is null, skipping");
+      return;
+    }
     if (player == null) {
       ExoTrackSelection.Factory videoTrackSelectionFactory = new AdaptiveTrackSelection.Factory();
-      trackSelector = new DefaultTrackSelector(context, videoTrackSelectionFactory);
+      trackSelector = new DefaultTrackSelector(playerContext, videoTrackSelectionFactory);
       LoadControl loadControl = new DefaultLoadControl();
 
       // Media3 Migration Note: DefaultBandwidthMeter now configured automatically by default
       player =
-        new ExoPlayer.Builder(context)
+        new ExoPlayer.Builder(playerContext)
           .setSeekBackIncrementMs(10000)
           .setSeekForwardIncrementMs(10000)
           .setTrackSelector(trackSelector)
@@ -973,6 +1029,24 @@ public class FullscreenExoPlayerFragment extends Fragment {
 
     MediaSource mediaSource = null;
     if (!Boolean.TRUE.equals(isInternal)) {
+      // Re-derive uri/vType from videoPath on every init. They are cleared by
+      // resetVariables() after release, and onStart/onResume can re-enter
+      // initializePlayer() without going through onCreateView, which was the
+      // sole place they were previously computed (NPE on vType.equals()).
+      if (uri == null && videoPath != null && !videoPath.trim().isEmpty()) {
+        try {
+          uri = Uri.parse(videoPath);
+        } catch (Exception e) {
+          Log.e(TAG, "Invalid video path: " + videoPath, e);
+          uri = null;
+        }
+      }
+      if (vType == null && uri != null) {
+        vType = getVideoType(uri);
+      }
+      if (vType == null) {
+        vType = "";
+      }
       if (videoPath != null && (videoPath.startsWith("file:///android_asset") || videoPath.startsWith("content://media"))) {
         mediaSource = buildAssetMediaSource(uri);
       } else if (videoPath != null && !videoPath.trim().isEmpty()) {
@@ -1006,24 +1080,31 @@ public class FullscreenExoPlayerFragment extends Fragment {
     }
     // Media3 Migration Note: MediaSessionConnector replaced with MediaSession.Builder; null check prevents duplicate session ID
     if (mediaSession == null) {
-      mediaSession = new MediaSession.Builder(context, player)
-        .setId("capacitorvideoplayer")
-        .build();
+      try {
+        mediaSession = new MediaSession.Builder(playerContext, player)
+          .setId("capacitorvideoplayer")
+          .build();
+      } catch (Exception e) {
+        Log.e(TAG, "Failed to build MediaSession", e);
+      }
     }
 
     NotificationCenter.defaultCenter().postNotification("initializePlayer", info);
   }
 
   private void setSubtitle(boolean transparent) {
+    if (styledPlayerView == null || styledPlayerView.getSubtitleView() == null) {
+      return;
+    }
     int foreground;
     int background;
     if (!transparent) {
       foreground = Color.WHITE;
       background = Color.BLACK;
-      if (stForeColor.length() > 4 && stForeColor.substring(0, 4).equals("rgba")) {
+      if (stForeColor != null && stForeColor.length() > 4 && stForeColor.substring(0, 4).equals("rgba")) {
         foreground = getColorFromRGBA(stForeColor);
       }
-      if (stBackColor.length() > 4 && stBackColor.substring(0, 4).equals("rgba")) {
+      if (stBackColor != null && stBackColor.length() > 4 && stBackColor.substring(0, 4).equals("rgba")) {
         background = getColorFromRGBA(stBackColor);
       }
     } else {
@@ -1035,7 +1116,7 @@ public class FullscreenExoPlayerFragment extends Fragment {
       .setStyle(
         new CaptionStyleCompat(foreground, background, Color.TRANSPARENT, CaptionStyleCompat.EDGE_TYPE_NONE, Color.WHITE, null)
       );
-    styledPlayerView.getSubtitleView().setFixedTextSize(TypedValue.COMPLEX_UNIT_DIP, stFontSize);
+    styledPlayerView.getSubtitleView().setFixedTextSize(TypedValue.COMPLEX_UNIT_DIP, stFontSize != null ? stFontSize : 16);
     styledPlayerView.setShowSubtitleButton(true);
   }
 
@@ -1043,9 +1124,16 @@ public class FullscreenExoPlayerFragment extends Fragment {
    * Build the Asset MediaSource
    */
   private MediaSource buildAssetMediaSource(Uri uri) {
+    if (uri == null) {
+      return null;
+    }
+    Context dataContext = context != null ? context : getContext();
+    if (dataContext == null) {
+      return null;
+    }
     MediaSource mediaSource = null;
     // Media3 Migration Note: DefaultDataSourceFactory replaced with DefaultDataSource.Factory
-    DataSource.Factory dataSourceFactory = new DefaultDataSource.Factory(context);
+    DataSource.Factory dataSourceFactory = new DefaultDataSource.Factory(dataContext);
     mediaSource = new ProgressiveMediaSource.Factory(dataSourceFactory).createMediaSource(MediaItem.fromUri(uri));
     // Get the subtitles if any
     if (sturi != null) {
@@ -1058,7 +1146,14 @@ public class FullscreenExoPlayerFragment extends Fragment {
    * Build the Internal MediaSource
    */
   private MediaSource buildInternalMediaSource(Uri uri) {
-    DataSource.Factory dataSourceFactory = new DefaultDataSource.Factory(context);
+    if (uri == null) {
+      return null;
+    }
+    Context dataContext = context != null ? context : getContext();
+    if (dataContext == null) {
+      return null;
+    }
+    DataSource.Factory dataSourceFactory = new DefaultDataSource.Factory(dataContext);
     return new ProgressiveMediaSource.Factory(dataSourceFactory).createMediaSource(MediaItem.fromUri(uri));
   }
 
@@ -1067,6 +1162,13 @@ public class FullscreenExoPlayerFragment extends Fragment {
    * @return MediaSource
    */
   private MediaSource buildHttpMediaSource() {
+    if (uri == null) {
+      Log.e(TAG, "buildHttpMediaSource: uri is null, cannot build media source");
+      return null;
+    }
+    // vType may still be null if getVideoType was never called (e.g. re-init
+    // after release). Default to progressive rather than NPE on equals().
+    String type = vType != null ? vType : "";
     MediaSource mediaSource = null;
 
     DefaultHttpDataSource.Factory httpDataSourceFactory = new DefaultHttpDataSource.Factory();
@@ -1076,38 +1178,57 @@ public class FullscreenExoPlayerFragment extends Fragment {
     httpDataSourceFactory.setAllowCrossProtocolRedirects(true);
 
     // If headers is not null and has data we pass them to the HttpDataSourceFactory
-    if (headers != null && headers.length() > 0) {
-      // We map the headers(JSObject) to a Map<String, String>
-      Map<String, String> headersMap = new HashMap<String, String>();
-      for (int i = 0; i < headers.names().length(); i++) {
-        try {
-          headersMap.put(headers.names().getString(i), headers.get(headers.names().getString(i)).toString());
-        } catch (JSONException e) {
-          e.printStackTrace();
+    if (headers != null) {
+      try {
+        if (headers.length() > 0 && headers.names() != null) {
+          // We map the headers(JSObject) to a Map<String, String>
+          Map<String, String> headersMap = new HashMap<String, String>();
+          for (int i = 0; i < headers.names().length(); i++) {
+            try {
+              String key = headers.names().getString(i);
+              Object value = headers.get(key);
+              if (key != null && value != null) {
+                headersMap.put(key, value.toString());
+              }
+            } catch (JSONException e) {
+              e.printStackTrace();
+            }
+          }
+          httpDataSourceFactory.setDefaultRequestProperties(headersMap);
         }
+      } catch (Exception e) {
+        Log.e(TAG, "Error applying headers", e);
       }
-      httpDataSourceFactory.setDefaultRequestProperties(headersMap);
     }
 
-    DataSource.Factory dataSourceFactory = new DefaultDataSource.Factory(context, httpDataSourceFactory);
+    DataSource.Factory dataSourceFactory;
+    Context dataContext = context != null ? context : getContext();
+    if (dataContext == null) {
+      Log.e(TAG, "buildHttpMediaSource: context is null, cannot build media source");
+      return null;
+    }
+    dataSourceFactory = new DefaultDataSource.Factory(dataContext, httpDataSourceFactory);
 
     if (
-      vType.equals("mp4") ||
-        vType.equals("webm") ||
-        vType.equals("ogv") ||
-        vType.equals("3gp") ||
-        vType.equals("flv") ||
-        vType.equals("")
+      "mp4".equals(type) ||
+        "webm".equals(type) ||
+        "ogv".equals(type) ||
+        "3gp".equals(type) ||
+        "flv".equals(type) ||
+        "".equals(type)
     ) {
       mediaSource = new ProgressiveMediaSource.Factory(dataSourceFactory).createMediaSource(MediaItem.fromUri(uri));
-    } else if (vType.equals("dash") || vType.equals("mpd")) {
+    } else if ("dash".equals(type) || "mpd".equals(type)) {
       /* adaptive streaming Dash stream */
       DashMediaSource.Factory mediaSourceFactory = new DashMediaSource.Factory(dataSourceFactory);
       mediaSource = mediaSourceFactory.createMediaSource(MediaItem.fromUri(uri));
-    } else if (vType.equals("m3u8")) {
+    } else if ("m3u8".equals(type)) {
       mediaSource = new HlsMediaSource.Factory(dataSourceFactory).createMediaSource(MediaItem.fromUri(uri));
-    } else if (vType.equals("ism")) {
+    } else if ("ism".equals(type)) {
       mediaSource = new SsMediaSource.Factory(dataSourceFactory).createMediaSource(MediaItem.fromUri(uri));
+    } else {
+      Log.w(TAG, "buildHttpMediaSource: unsupported video type '" + type + "', falling back to progressive");
+      mediaSource = new ProgressiveMediaSource.Factory(dataSourceFactory).createMediaSource(MediaItem.fromUri(uri));
     }
     // Get the subtitles if any
     if (sturi != null) {
@@ -1152,13 +1273,27 @@ public class FullscreenExoPlayerFragment extends Fragment {
   }
 
   private MediaSource getSubTitle(MediaSource mediaSource, Uri sturi, DataSource.Factory dataSourceFactory) {
+    if (mediaSource == null || sturi == null) {
+      return mediaSource;
+    }
     // Create mediaSource with subtitle
     MediaSource[] mediaSources = new MediaSource[2];
     mediaSources[0] = mediaSource;
     String mimeType = getMimeType(sturi);
+    if (mimeType == null || mimeType.isEmpty()) {
+      Log.w(TAG, "getSubTitle: unknown subtitle mime type, skipping subtitles");
+      return mediaSource;
+    }
 
     // We get the language label from the language code
-    String languageLabel = Locale.forLanguageTag(language).getDisplayLanguage();
+    String safeLanguage = language != null ? language : "";
+    String languageLabel;
+    try {
+      languageLabel = Locale.forLanguageTag(safeLanguage).getDisplayLanguage();
+    } catch (Exception e) {
+      Log.e(TAG, "Invalid language tag: " + safeLanguage, e);
+      languageLabel = safeLanguage;
+    }
     MediaItem.SubtitleConfiguration subConfig = new MediaItem.SubtitleConfiguration.Builder(sturi)
       .setMimeType(mimeType)
       .setUri(sturi)
@@ -1166,7 +1301,7 @@ public class FullscreenExoPlayerFragment extends Fragment {
       .setLabel(languageLabel)
       .setRoleFlags(C.ROLE_FLAG_CAPTION)
       .setSelectionFlags(C.SELECTION_FLAG_DEFAULT)
-      .setLanguage(language)
+      .setLanguage(safeLanguage)
       .build();
 
     SingleSampleMediaSource subtitleSource = new SingleSampleMediaSource.Factory(dataSourceFactory)
@@ -1179,16 +1314,29 @@ public class FullscreenExoPlayerFragment extends Fragment {
   }
 
   private String getMimeType(Uri sturi) {
-    String lastSegment = sturi.getLastPathSegment();
-    String extension = lastSegment.substring(lastSegment.lastIndexOf(".") + 1);
+    if (sturi == null) {
+      return "";
+    }
+    String lastSegment;
+    try {
+      lastSegment = sturi.getLastPathSegment();
+    } catch (Exception e) {
+      Log.e(TAG, "getMimeType: failed to get last path segment", e);
+      return "";
+    }
+    if (lastSegment == null || lastSegment.isEmpty()) {
+      return "";
+    }
+    int dotIndex = lastSegment.lastIndexOf(".");
+    String extension = (dotIndex != -1 ? lastSegment.substring(dotIndex + 1) : lastSegment).toLowerCase(Locale.ROOT);
     String mimeType = "";
-    if (extension.equals("vtt")) {
+    if ("vtt".equals(extension)) {
       mimeType = MimeTypes.TEXT_VTT;
-    } else if (extension.equals("srt")) {
+    } else if ("srt".equals(extension)) {
       mimeType = MimeTypes.APPLICATION_SUBRIP;
-    } else if (extension.equals("ssa") || extension.equals("ass")) {
+    } else if ("ssa".equals(extension) || "ass".equals(extension)) {
       mimeType = MimeTypes.TEXT_SSA;
-    } else if (extension.equals("ttml") || extension.equals("dfxp") || extension.equals("xml")) {
+    } else if ("ttml".equals(extension) || "dfxp".equals(extension) || "xml".equals(extension)) {
       mimeType = MimeTypes.APPLICATION_TTML;
     }
     return mimeType;
@@ -1198,7 +1346,7 @@ public class FullscreenExoPlayerFragment extends Fragment {
    * Fast Forward TV
    */
   private void fastForward(long position, int times) {
-    if (position < mDuration - videoStep) {
+    if (player != null && position < mDuration - videoStep) {
       if (player.isPlaying()) {
         player.setPlayWhenReady(false);
       }
@@ -1211,7 +1359,7 @@ public class FullscreenExoPlayerFragment extends Fragment {
    * Rewind TV
    */
   private void rewind(long position, int times) {
-    if (position > videoStep) {
+    if (player != null && position > videoStep) {
       if (player.isPlaying()) {
         player.setPlayWhenReady(false);
       }
@@ -1224,7 +1372,9 @@ public class FullscreenExoPlayerFragment extends Fragment {
    * Play Pause TV
    */
   private void play_pause() {
-    player.setPlayWhenReady(!player.isPlaying());
+    if (player != null) {
+      player.setPlayWhenReady(!player.isPlaying());
+    }
   }
 
   /**
@@ -1232,7 +1382,7 @@ public class FullscreenExoPlayerFragment extends Fragment {
    * @return boolean
    */
   public boolean isPlaying() {
-    return player.isPlaying();
+    return player != null && player.isPlaying();
   }
 
   /**
@@ -1312,6 +1462,9 @@ public class FullscreenExoPlayerFragment extends Fragment {
    * @return int in seconds
    */
   public int getDuration() {
+    if (player == null) {
+      return 0;
+    }
     return player.getDuration() == UNKNOWN_TIME ? 0 : (int) (player.getDuration() / 1000);
   }
 
@@ -1338,8 +1491,12 @@ public class FullscreenExoPlayerFragment extends Fragment {
     
     try {
         if (isInPictureInPictureMode) {
-            styledPlayerView.setUseController(false);
-            linearLayout.setVisibility(View.INVISIBLE);
+            if (styledPlayerView != null) {
+                styledPlayerView.setUseController(false);
+            }
+            if (linearLayout != null) {
+                linearLayout.setVisibility(View.INVISIBLE);
+            }
         }
         long seekPosition = player.getCurrentPosition() == UNKNOWN_TIME
             ? 0
@@ -1355,7 +1512,7 @@ public class FullscreenExoPlayerFragment extends Fragment {
    * @return float
    */
   public float getVolume() {
-    return player.getVolume();
+    return player != null ? player.getVolume() : curVolume;
   }
 
   /**
@@ -1364,7 +1521,10 @@ public class FullscreenExoPlayerFragment extends Fragment {
    */
   public void setVolume(float _volume) {
     float volume = Math.min(Math.max(0, _volume), 1L);
-    player.setVolume(volume);
+    curVolume = volume;
+    if (player != null) {
+      player.setVolume(volume);
+    }
   }
 
   /**
@@ -1381,8 +1541,10 @@ public class FullscreenExoPlayerFragment extends Fragment {
    */
   public void setRate(float _rate) {
     videoRate = _rate;
-    PlaybackParameters param = new PlaybackParameters(videoRate);
-    player.setPlaybackParameters(param);
+    if (player != null) {
+      PlaybackParameters param = new PlaybackParameters(videoRate != null ? videoRate : 1.0f);
+      player.setPlaybackParameters(param);
+    }
   }
 
   /**
@@ -1391,11 +1553,13 @@ public class FullscreenExoPlayerFragment extends Fragment {
    */
   public void setMuted(boolean _isMuted) {
     isMuted = _isMuted;
-    if (isMuted) {
-      curVolume = player.getVolume();
-      player.setVolume(0L);
-    } else {
-      player.setVolume(curVolume);
+    if (player != null) {
+      if (isMuted) {
+        curVolume = player.getVolume();
+        player.setVolume(0L);
+      } else {
+        player.setVolume(curVolume);
+      }
     }
   }
 
@@ -1412,7 +1576,7 @@ public class FullscreenExoPlayerFragment extends Fragment {
    * @param button
    */
   private void mediaRouteButtonColorWhite(MediaRouteButton button) {
-    if (button == null) return;
+    if (button == null || getContext() == null) return;
     Context castContext = new ContextThemeWrapper(getContext(), androidx.mediarouter.R.style.Theme_MediaRouter);
 
     TypedArray a = castContext.obtainStyledAttributes(
@@ -1423,6 +1587,9 @@ public class FullscreenExoPlayerFragment extends Fragment {
     );
     Drawable drawable = a.getDrawable(androidx.mediarouter.R.styleable.MediaRouteButton_externalRouteEnabledDrawable);
     a.recycle();
+    if (drawable == null) {
+      return;
+    }
     DrawableCompat.setTint(drawable, getContext().getResources().getColor(R.color.white));
     drawable.setState(button.getDrawableState());
     button.setRemoteIndicatorDrawable(drawable);
@@ -1434,6 +1601,9 @@ public class FullscreenExoPlayerFragment extends Fragment {
    * @return video type
    */
   private String getVideoType(Uri uri) {
+    if (uri == null) {
+      return "";
+    }
     String ret = null;
     Object obj = uri.getLastPathSegment();
     String lastSegment = (obj == null) ? "" : uri.getLastPathSegment();
@@ -1442,17 +1612,19 @@ public class FullscreenExoPlayerFragment extends Fragment {
       if (lastSegment.length() > 0 && lastSegment.contains(type)) ret = type;
       if (ret == null) {
         List<String> segments = uri.getPathSegments();
-        if (segments.size() > 0) {
+        if (segments != null && segments.size() > 0) {
           String segment;
-          if (segments.get(segments.size() - 1).equals("manifest")) {
+          if (segments.get(segments.size() - 1).equals("manifest") && segments.size() >= 2) {
             segment = segments.get(segments.size() - 2);
           } else {
             segment = segments.get(segments.size() - 1);
           }
-          for (String sType : supportedFormat) {
-            if (segment.contains(sType)) {
-              ret = sType;
-              break;
+          if (segment != null) {
+            for (String sType : supportedFormat) {
+              if (segment.contains(sType)) {
+                ret = sType;
+                break;
+              }
             }
           }
         }
@@ -1467,7 +1639,6 @@ public class FullscreenExoPlayerFragment extends Fragment {
    */
   private void resetVariables() {
     vType = null;
-    styledPlayerView = null;
     playWhenReady = true;
     firstReadyToPlay = true;
     isEnded = false;
@@ -1485,8 +1656,14 @@ public class FullscreenExoPlayerFragment extends Fragment {
    * @return boolean
    */
   public boolean isApplicationSentToBackground(final Context context) {
+    if (context == null) {
+      return false;
+    }
     int pid = android.os.Process.myPid();
     ActivityManager am = (ActivityManager) context.getSystemService(Context.ACTIVITY_SERVICE);
+    if (am == null) {
+      return false;
+    }
     List<ActivityManager.RunningAppProcessInfo> procInfos = am.getRunningAppProcesses();
     if (procInfos != null) {
       for (ActivityManager.RunningAppProcessInfo appProcess : procInfos) {
@@ -1509,6 +1686,9 @@ public class FullscreenExoPlayerFragment extends Fragment {
     task.addOnCompleteListener(new OnCompleteListener<CastContext>() {
       @Override
       public void onComplete(Task<CastContext> task) {
+        if (isFragmentDestroyed || !isAdded() || context == null || mediaRouteButton == null) {
+          return;
+        }
         if (task.isSuccessful()) {
           castContext = task.getResult();
           castPlayer = new CastPlayer(castContext);
@@ -1525,6 +1705,9 @@ public class FullscreenExoPlayerFragment extends Fragment {
 
           castStateListener =
                   state -> {
+                    if (mediaRouteButton == null) {
+                      return;
+                    }
                     if (state == CastState.NO_DEVICES_AVAILABLE) {
                       mediaRouteButton.setVisibility(View.GONE);
                     } else {
@@ -1536,13 +1719,17 @@ public class FullscreenExoPlayerFragment extends Fragment {
           CastButtonFactory.setUpMediaRouteButton(context, mediaRouteButton);
 
           MediaMetadata movieMetadata;
-          if (artwork != "") {
-            movieMetadata = new MediaMetadata.Builder()
-                    .setTitle(title)
-                    .setSubtitle(smallTitle)
-                    .setMediaType(MediaMetadata.MEDIA_TYPE_MOVIE)
-                    .setArtworkUri(Uri.parse(artwork))
-                    .build();
+          if (artwork != null && !artwork.trim().isEmpty()) {
+            MediaMetadata.Builder metadataBuilder = new MediaMetadata.Builder()
+              .setTitle(title)
+              .setSubtitle(smallTitle)
+              .setMediaType(MediaMetadata.MEDIA_TYPE_MOVIE);
+            try {
+              metadataBuilder.setArtworkUri(Uri.parse(artwork));
+            } catch (Exception e) {
+              Log.e(TAG, "Invalid artwork uri: " + artwork, e);
+            }
+            movieMetadata = metadataBuilder.build();
             new setCastImage().execute();
           } else {
             movieMetadata = new MediaMetadata.Builder()
@@ -1562,35 +1749,53 @@ public class FullscreenExoPlayerFragment extends Fragment {
                     @Override
                     public void onCastSessionAvailable() {
                       isCastSession = true;
-                      final Long videoPosition = player.getCurrentPosition();
-                      if (Boolean.TRUE.equals(pipEnabled)) {
+                      final Long videoPosition = player != null ? player.getCurrentPosition() : 0L;
+                      if (pipBtn != null && Boolean.TRUE.equals(pipEnabled)) {
                         pipBtn.setVisibility(View.GONE);
                       }
-                      resizeBtn.setVisibility(View.GONE);
-                      player.setPlayWhenReady(false);
-                      cast_image.setVisibility(View.VISIBLE);
+                      if (resizeBtn != null) {
+                        resizeBtn.setVisibility(View.GONE);
+                      }
+                      if (player != null) {
+                        player.setPlayWhenReady(false);
+                      }
+                      if (cast_image != null) {
+                        cast_image.setVisibility(View.VISIBLE);
+                      }
                       castPlayer.setMediaItem(mediaItem, videoPosition);
-                      styledPlayerView.setPlayer(castPlayer);
-                      styledPlayerView.setControllerShowTimeoutMs(0);
-                      styledPlayerView.setControllerHideOnTouch(false);
-                      //We perform a click because for some weird reason, the layout is black until the user clicks on it
-                      styledPlayerView.performClick();
+                      if (styledPlayerView != null) {
+                        styledPlayerView.setPlayer(castPlayer);
+                        styledPlayerView.setControllerShowTimeoutMs(0);
+                        styledPlayerView.setControllerHideOnTouch(false);
+                        //We perform a click because for some weird reason, the layout is black until the user clicks on it
+                        styledPlayerView.performClick();
+                      }
                     }
 
                     @Override
                     public void onCastSessionUnavailable() {
                       isCastSession = false;
-                      final Long videoPosition = castPlayer.getCurrentPosition();
-                      if (Boolean.TRUE.equals(pipEnabled)) {
+                      final Long videoPosition = castPlayer != null ? castPlayer.getCurrentPosition() : 0L;
+                      if (pipBtn != null && Boolean.TRUE.equals(pipEnabled)) {
                         pipBtn.setVisibility(View.VISIBLE);
                       }
-                      resizeBtn.setVisibility(View.VISIBLE);
-                      cast_image.setVisibility(View.GONE);
-                      styledPlayerView.setPlayer(player);
-                      player.setPlayWhenReady(true);
-                      player.seekTo(videoPosition);
-                      styledPlayerView.setControllerShowTimeoutMs(3000);
-                      styledPlayerView.setControllerHideOnTouch(true);
+                      if (resizeBtn != null) {
+                        resizeBtn.setVisibility(View.VISIBLE);
+                      }
+                      if (cast_image != null) {
+                        cast_image.setVisibility(View.GONE);
+                      }
+                      if (styledPlayerView != null) {
+                        styledPlayerView.setPlayer(player);
+                      }
+                      if (player != null) {
+                        player.setPlayWhenReady(true);
+                        player.seekTo(videoPosition);
+                      }
+                      if (styledPlayerView != null) {
+                        styledPlayerView.setControllerShowTimeoutMs(3000);
+                        styledPlayerView.setControllerHideOnTouch(true);
+                      }
                     }
                   }
           );
@@ -1602,12 +1807,12 @@ public class FullscreenExoPlayerFragment extends Fragment {
                       Map<String, Object> info = new HashMap<String, Object>() {
                         {
                           put("fromPlayerId", playerId);
-                          put("currentTime", String.valueOf(player.getCurrentPosition() / 1000));
+                          put("currentTime", String.valueOf(player != null ? player.getCurrentPosition() / 1000 : 0));
                         }
                       };
                       switch (state) {
                         case CastPlayer.STATE_READY:
-                          if (castPlayer.isPlaying()) {
+                          if (castPlayer != null && castPlayer.isPlaying()) {
                             NotificationCenter.defaultCenter().postNotification("playerItemPlay", info);
                           } else {
                             NotificationCenter.defaultCenter().postNotification("playerItemPause", info);
@@ -1621,11 +1826,17 @@ public class FullscreenExoPlayerFragment extends Fragment {
           );
 
           castContext.addCastStateListener(castStateListener);
-          mRouter.addCallback(mSelector, mCallback, MediaRouter.CALLBACK_FLAG_REQUEST_DISCOVERY);
+          if (mRouter != null && mSelector != null && mCallback != null) {
+            mRouter.addCallback(mSelector, mCallback, MediaRouter.CALLBACK_FLAG_REQUEST_DISCOVERY);
+          }
 
         } else {
           Exception e = task.getException();
-          e.printStackTrace();
+          if (e != null) {
+            e.printStackTrace();
+          } else {
+            Log.e(TAG, "initializeCastService: CastContext task failed with no exception");
+          }
         }
       }
     });
@@ -1640,6 +1851,8 @@ public class FullscreenExoPlayerFragment extends Fragment {
   }
 
   private void adjustAspectRatio() {
-    styledPlayerView.setResizeMode(resizeStatus);
+    if (styledPlayerView != null) {
+      styledPlayerView.setResizeMode(resizeStatus);
+    }
   }
 }

package/android/src/main/java/com/jeep/plugin/capacitor/capacitorvideoplayer/PickerVideo/PickerVideoFragment.java

@@ -78,6 +78,9 @@ public class PickerVideoFragment extends Fragment {
     }
 
     private void loadVideos() {
+        if (context == null) {
+            return;
+        }
         String[] projection = {
             MediaStore.Video.Media._ID,
             MediaStore.Video.Media.DISPLAY_NAME,
@@ -92,37 +95,46 @@ public class PickerVideoFragment extends Fragment {
             collection = MediaStore.Video.Media.EXTERNAL_CONTENT_URI;
         }
 
-        Cursor cursor = getActivity().getApplication().getContentResolver().query(collection, projection, null, null, sortOrder);
-        if (cursor != null) {
-            int idColumn = cursor.getColumnIndexOrThrow(MediaStore.Video.Media._ID);
-            int titleColumn = cursor.getColumnIndexOrThrow(MediaStore.Video.Media.DISPLAY_NAME);
-            int durationColumn = cursor.getColumnIndexOrThrow(MediaStore.Video.Media.DURATION);
+        try (Cursor cursor = context.getContentResolver().query(collection, projection, null, null, sortOrder)) {
+            if (cursor != null) {
+                int idColumn = cursor.getColumnIndexOrThrow(MediaStore.Video.Media._ID);
+                int titleColumn = cursor.getColumnIndexOrThrow(MediaStore.Video.Media.DISPLAY_NAME);
+                int durationColumn = cursor.getColumnIndexOrThrow(MediaStore.Video.Media.DURATION);
 
-            while (cursor.moveToNext()) {
-                long id = cursor.getLong(idColumn);
-                String title = cursor.getString(titleColumn);
-                int duration = cursor.getInt(durationColumn);
+                while (cursor.moveToNext()) {
+                    long id = cursor.getLong(idColumn);
+                    String title = cursor.getString(titleColumn);
+                    int duration = cursor.getInt(durationColumn);
 
-                Uri data = ContentUris.withAppendedId(collection, id);
+                    Uri data = ContentUris.withAppendedId(collection, id);
 
-                String duration_formatted;
-                int sec = (duration / 1000) % 60;
-                int min = (duration / (1000 * 60)) % 60;
-                int hrs = duration / (1000 * 60 * 60);
+                    String duration_formatted;
+                    int sec = (duration / 1000) % 60;
+                    int min = (duration / (1000 * 60)) % 60;
+                    int hrs = duration / (1000 * 60 * 60);
 
-                if (hrs == 0) {
-                    duration_formatted = String.valueOf(min).concat(":".concat(String.format(Locale.UK, "%02d", sec)));
-                } else {
-                    duration_formatted =
-                        String
-                            .valueOf(hrs)
-                            .concat(
-                                ":".concat(String.format(Locale.UK, "%02d", min).concat(":".concat(String.format(Locale.UK, "%02d", sec))))
-                            );
+                    if (hrs == 0) {
+                        duration_formatted = String.valueOf(min).concat(":".concat(String.format(Locale.UK, "%02d", sec)));
+                    } else {
+                        duration_formatted = String.valueOf(hrs).concat(
+                            ":".concat(String.format(Locale.UK, "%02d", min).concat(":".concat(String.format(Locale.UK, "%02d", sec))))
+                        );
+                    }
+                    Log.v(TAG, "**** video " + id + " " + title + " " + duration_formatted + " ****");
+                    videosList.add(new ModelVideo(id, data, title, duration_formatted));
                 }
-                Log.v(TAG, "**** video " + id + " " + title + " " + duration_formatted + " ****");
-                videosList.add(new ModelVideo(id, data, title, duration_formatted));
             }
+        } catch (Exception e) {
+            Log.e(TAG, "Error loading videos from media store", e);
         }
     }
+
+    @Override
+    public void onDestroyView() {
+        recyclerView = null;
+        recyclerViewLayoutManager = null;
+        adapterVideoList = null;
+        view = null;
+        super.onDestroyView();
+    }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@picsa/capacitor-video-player",
-  "version": "8.0.2",
+  "version": "8.0.4",
   "description": "custom Native Capacitor plugin to play a video",
   "main": "dist/plugin.cjs.js",
   "module": "dist/esm/index.js",