Merge pull request #9 from SimformSolutionsPvtLtd/fix/UNT-T22087

fix(UNT-T22087): fix audio playback flickering when seek in android

Author: virajpsimformsolutions

android/src/main/java/com/audiowaveform/AudioPlayer.kt

@@ -151,7 +151,7 @@ class AudioPlayer(
 
     private fun startListening(promise: Promise) {
         try {
-            audioPlaybackListener = object : CountDownTimer(player.duration, 10) {
+            audioPlaybackListener = object : CountDownTimer(player.duration, UpdateFrequency.Low.value) {
                 override fun onTick(millisUntilFinished: Long) {
                     val currentPosition = player.currentPosition.toString()
                     val args: WritableMap = Arguments.createMap()

android/src/main/java/com/audiowaveform/AudioWaveformModule.kt

@@ -309,13 +309,13 @@ class AudioWaveformModule(context: ReactApplicationContext): ReactContextBaseJav
                     args.putDouble(Constants.currentDecibel, currentDecibel/1000)
                 }
             }
-            handler.postDelayed(this, 500)
+            handler.postDelayed(this, UpdateFrequency.Low.value)
             reactApplicationContext.getJSModule(DeviceEventManagerModule.RCTDeviceEventEmitter::class.java)?.emit(Constants.onCurrentRecordingWaveformData, args)
         }
     }
 
     private fun startEmittingRecorderValue() {
-        handler.postDelayed(emitLiveRecordValue, 500)
+        handler.postDelayed(emitLiveRecordValue, UpdateFrequency.Low.value)
     }
 
     private fun stopEmittingRecorderValue() {

example/android/app/src/main/AndroidManifest.xml

@@ -2,7 +2,6 @@
 
     <uses-permission android:name="android.permission.INTERNET" />
     <uses-permission android:name="android.permission.RECORD_AUDIO" />
-    <uses-permission android:name="android.permission.READ_MEDIA_AUDIO" />
 
     <application
       android:name=".MainApplication"

src/components/Waveform/Waveform.tsx

@@ -193,7 +193,7 @@ export const Waveform: <T extends StaticOrLive>(
   const startPlayerAction = async (args?: IStartPlayerRef) => {
     try {
       const play = await playPlayer({
-        finishMode: FinishMode.loop,
+        finishMode: FinishMode.stop,
         playerKey: `PlayerFor${path}`,
         path: path,
         ...args,
@@ -391,15 +391,12 @@ export const Waveform: <T extends StaticOrLive>(
     });
     const subscribeData2 = onCurrentDuration(data => {
       if (data.playerKey === `PlayerFor${path}`) {
-        if (!panMoving) {
-          setCurrentProgress(data.currentDuration);
-        }
+        setCurrentProgress(data.currentDuration);
       }
     });
     const subscribeData3 = onCurrentExtractedWaveformData(() => {
       // write logic for subscription
     });
-
     const subscribeData4 = onCurrentRecordingWaveformData(result => {
       if (mode === 'live') {
         if (!isNil(result.currentDecibel)) {
@@ -416,7 +413,7 @@ export const Waveform: <T extends StaticOrLive>(
       subscribeData3.remove();
       subscribeData4.remove();
     };
-  }, [panMoving]);
+  }, []);
 
   useEffect(() => {
     if (!isNil(onPlayerStateChange)) {
@@ -430,15 +427,24 @@ export const Waveform: <T extends StaticOrLive>(
     }
   }, [recorderState]);
 
+  useEffect(() => {
+    if (panMoving) {
+      if (playerState === PlayerState.playing) {
+        pausePlayerAction();
+      }
+    } else {
+      if (playerState === PlayerState.paused) {
+        startPlayerAction();
+      }
+    }
+  }, [panMoving]);
+
   const panResponder = useRef(
     PanResponder.create({
       onMoveShouldSetPanResponder: () => true,
       onPanResponderGrant: () => {
         setPanMoving(true);
         (onPanStateChange as Function)(true);
-        if (playerState === PlayerState.playing) {
-          pausePlayerAction();
-        }
       },
       onPanResponderStart: () => {},
       onPanResponderMove: event => {