Merge pull request #128 from tidal-music/alberto/offlineengine-suppor…

…t-offlineplay [Player] Support OfflinePlay event in OfflineEngine items
tidal-music · Oct 23, 2024 · 4a87b1b · 4a87b1b
2 parents 22f4bca + fc83488
commit 4a87b1b
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/Sources/Player/PlaybackEngine/Internal/PlaybackSource.swift b/Sources/Player/PlaybackEngine/Internal/PlaybackSource.swift
@@ -4,4 +4,13 @@ enum PlaybackSource {
 	case INTERNET
 	case LOCAL_STORAGE
 	case LOCAL_STORAGE_LEGACY
+
+	func isOfflineSource() -> Bool {
+		switch self {
+		case .LOCAL_STORAGE, .LOCAL_STORAGE_LEGACY:
+			true
+		case .INTERNET:
+			false
+		}
+	}
 }
diff --git a/Sources/Player/PlaybackEngine/Internal/PlayerItem.swift b/Sources/Player/PlaybackEngine/Internal/PlayerItem.swift
@@ -439,11 +439,12 @@ private extension PlayerItem {
 
 	func emitOfflinePlay() {
 		guard let metrics,
+		      metrics.actualStartTime != nil,
+		      let metadata,
+		      metadata.playbackSource.isOfflineSource(),
 		      let playbackContext,
-		      let asset,
 		      let productId = Int(playbackContext.productId),
-		      metrics.actualStartTime != nil,
-		      mediaProduct is StoredMediaProduct
+		      let asset
 		else {
 			return
 		}