Fix: Fix mediaContentComplete calculation (#66) (#68)

Mansi-mParticle · web-flow · commit a7942be350ce · 2025-02-18T13:45:19.000-05:00
diff --git a/media/src/main/java/com/mparticle/media/MediaSession.kt b/media/src/main/java/com/mparticle/media/MediaSession.kt
@@ -602,7 +602,7 @@ class MediaSession protected constructor(builder: Builder) {
         }
 
         mediaSessionEndTimestamp = System.currentTimeMillis()
-        if (mediaContentCompleteLimit < 100 && (duration != null && currentPlayheadPosition != null) && ((currentPlayheadPosition!! / duration!!) >= (mediaContentCompleteLimit / 100))) {
+        if (mediaContentCompleteLimit < 100 && (duration != null && currentPlayheadPosition != null) && ((currentPlayheadPosition!! / duration!!.toDouble()) >= (mediaContentCompleteLimit / 100.0))) {
             mediaContentComplete = true
         }
 

Original file line number	Diff line number	Diff line change
`@@ -602,7 +602,7 @@ class MediaSession protected constructor(builder: Builder) {`
`602`	`602`	`}`
`603`	`603`
`604`	`604`	`mediaSessionEndTimestamp = System.currentTimeMillis()`
`605`		`- if (mediaContentCompleteLimit < 100 && (duration != null && currentPlayheadPosition != null) && ((currentPlayheadPosition!! / duration!!) >= (mediaContentCompleteLimit / 100))) {`
	`605`	`+ if (mediaContentCompleteLimit < 100 && (duration != null && currentPlayheadPosition != null) && ((currentPlayheadPosition!! / duration!!.toDouble()) >= (mediaContentCompleteLimit / 100.0))) {`
`606`	`606`	`mediaContentComplete = true`
`607`	`607`	`}`
`608`	`608`