Merge pull request Expensify#36914 from tienifr/fix/36865

fix: On full screen mode, when the video ends, the play button does nothing

src/components/VideoPlayer/BaseVideoPlayer.js

@@ -13,6 +13,7 @@ import addEncryptedAuthTokenToURL from '@libs/addEncryptedAuthTokenToURL';
 import * as Browser from '@libs/Browser';
 import * as DeviceCapabilities from '@libs/DeviceCapabilities';
 import {videoPlayerDefaultProps, videoPlayerPropTypes} from './propTypes';
+import shouldReplayVideo from './shouldReplayVideo';
 import VideoPlayerControls from './VideoPlayerControls';
 
 const isMobileSafari = Browser.isMobileSafari();
@@ -95,6 +96,9 @@ function BaseVideoPlayer({
 
     const handlePlaybackStatusUpdate = useCallback(
         (e) => {
+            if (shouldReplayVideo(e, isPlaying, duration, position)) {
+                videoPlayerRef.current.setStatusAsync({positionMillis: 0, shouldPlay: true});
+            }
             const isVideoPlaying = e.isPlaying || false;
             preventPausingWhenExitingFullscreen(isVideoPlaying);
             setIsPlaying(isVideoPlaying);
@@ -105,7 +109,7 @@ function BaseVideoPlayer({
 
             onPlaybackStatusUpdate(e);
         },
-        [onPlaybackStatusUpdate, preventPausingWhenExitingFullscreen, videoDuration],
+        [onPlaybackStatusUpdate, preventPausingWhenExitingFullscreen, videoDuration, isPlaying, duration, position],
     );
 
     const handleFullscreenUpdate = useCallback(

src/components/VideoPlayer/shouldReplayVideo.android.ts

@@ -0,0 +1,11 @@
+import type {AVPlaybackStatusSuccess} from 'expo-av';
+
+/**
+ * Whether to replay the video when users press play button
+ */
+export default function shouldReplayVideo(e: AVPlaybackStatusSuccess, isPlaying: boolean, duration: number, position: number): boolean {
+    if (!isPlaying && !e.didJustFinish && duration === position) {
+        return true;
+    }
+    return false;
+}

src/components/VideoPlayer/shouldReplayVideo.ios.ts

@@ -0,0 +1,11 @@
+import type {AVPlaybackStatusSuccess} from 'expo-av';
+
+/**
+ * Whether to replay the video when users press play button
+ */
+export default function shouldReplayVideo(e: AVPlaybackStatusSuccess, isPlaying: boolean, duration: number, position: number): boolean {
+    if (!isPlaying && e.isPlaying && duration === position) {
+        return true;
+    }
+    return false;
+}

src/components/VideoPlayer/shouldReplayVideo.ts

@@ -0,0 +1,9 @@
+import type {AVPlaybackStatusSuccess} from 'expo-av';
+
+/**
+ * Whether to replay the video when users press play button
+ */
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
+export default function shouldReplayVideo(e: AVPlaybackStatusSuccess, isPlaying: boolean, duration: number, position: number): boolean {
+    return false;
+}