Skip to content
This repository was archived by the owner on Jan 16, 2023. It is now read-only.

Commit a30589b

Browse files
authored
Merge pull request #435 from clappr/fix/seek_start_at
Adjust observe AVItem status to call delegate only once
2 parents d009698 + aef8617 commit a30589b

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

Sources/Clappr/Classes/Plugin/Playback/AVPlayerItemInfo.swift

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ class AVPlayerItemInfo {
1414
}
1515
private unowned var delegate: AVPlayerItemInfoDelegate
1616
private var assetInfo: AVAssetInfo
17+
private var becameReady = false
1718

1819
private var durationLoaded: Bool {
1920
item.status == .readyToPlay
@@ -64,9 +65,11 @@ class AVPlayerItemInfo {
6465
*/
6566
func waitForDuration() {
6667
observers = [
67-
item.observe(\.status) { [weak delegate] item, _ in
68-
if item.status == .readyToPlay {
69-
delegate?.didLoadDuration()
68+
item.observe(\.status) { [weak self] item, _ in
69+
guard let self = self else { return }
70+
if item.status == .readyToPlay && !self.becameReady {
71+
self.becameReady.toggle()
72+
self.delegate.didLoadDuration()
7073
}
7174
}
7275
]
@@ -77,6 +80,7 @@ class AVPlayerItemInfo {
7780
}
7881

7982
private func clearObservers() {
83+
becameReady = false
8084
observers.forEach { $0.invalidate() }
8185
observers.removeAll()
8286
}

0 commit comments

Comments
 (0)