ehealthsolutions/element-x-ios
3
0
Fork 0
mirror of https://github.com/Novaloop-AG/element-x-ios.git synced 2026-11-03 16:10:20 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix a crash in the media events screen by storing the active timeline context. (#3746)

Browse Source
This commit is contained in:
Doug
2025-02-06 12:27:39 +00:00
committed by GitHub
parent 922ebf47e6
commit b900eec5a2
3 changed files with 22 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files
ElementX/Sources/Screens/MediaEventsTimelineScreen/MediaEventsTimelineScreenModels.swift
+1 -1
View File
@@ -28,7 +28,7 @@ struct MediaEventsTimelineScreenViewState: BindableState {
var groups = [MediaEventsTimelineGroup]() var groups = [MediaEventsTimelineGroup]()
var activeTimelineContextProvider: (() -> TimelineViewModel.Context)! var activeTimelineContext: TimelineViewModel.Context
var bindings: MediaEventsTimelineScreenViewStateBindings var bindings: MediaEventsTimelineScreenViewStateBindings
} }
ElementX/Sources/Screens/MediaEventsTimelineScreen/MediaEventsTimelineScreenViewModel.swift
+17 -7
View File
@@ -37,7 +37,7 @@ class MediaEventsTimelineScreenViewModel: MediaEventsTimelineScreenViewModelType
init(mediaTimelineViewModel: TimelineViewModelProtocol, init(mediaTimelineViewModel: TimelineViewModelProtocol,
filesTimelineViewModel: TimelineViewModelProtocol, filesTimelineViewModel: TimelineViewModelProtocol,
initialViewState: MediaEventsTimelineScreenViewState = .init(bindings: .init(screenMode: .media)), initialScreenMode: MediaEventsTimelineScreenMode = .media,
mediaProvider: MediaProviderProtocol, mediaProvider: MediaProviderProtocol,
userIndicatorController: UserIndicatorControllerProtocol, userIndicatorController: UserIndicatorControllerProtocol,
appMediator: AppMediatorProtocol) { appMediator: AppMediatorProtocol) {
@@ -47,14 +47,24 @@ class MediaEventsTimelineScreenViewModel: MediaEventsTimelineScreenViewModelType
self.userIndicatorController = userIndicatorController self.userIndicatorController = userIndicatorController
self.appMediator = appMediator self.appMediator = appMediator
super.init(initialViewState: initialViewState, mediaProvider: mediaProvider) let activeTimelineContext = switch initialScreenMode {
case .media: mediaTimelineViewModel.context
state.activeTimelineContextProvider = { [weak self] in case .files: filesTimelineViewModel.context
guard let self else { fatalError() }
return activeTimelineViewModel.context
} }
super.init(initialViewState: .init(activeTimelineContext: activeTimelineContext, bindings: .init(screenMode: initialScreenMode)), mediaProvider: mediaProvider)
context.$viewState.map(\.bindings.screenMode)
.removeDuplicates()
.map {
switch $0 {
case .media: mediaTimelineViewModel.context
case .files: filesTimelineViewModel.context
}
}
.weakAssign(to: \.state.activeTimelineContext, on: self)
.store(in: &cancellables)
mediaTimelineViewModel.context.$viewState.sink { [weak self] timelineViewState in mediaTimelineViewModel.context.$viewState.sink { [weak self] timelineViewState in
guard let self, state.bindings.screenMode == .media else { guard let self, state.bindings.screenMode == .media else {
return return
ElementX/Sources/Screens/MediaEventsTimelineScreen/View/MediaEventsTimelineScreen.swift
+4 -4
View File
@@ -18,8 +18,8 @@ struct MediaEventsTimelineScreen: View {
// Doesn't play well with the transformed scrollView // Doesn't play well with the transformed scrollView
.toolbarBackground(.visible, for: .navigationBar) .toolbarBackground(.visible, for: .navigationBar)
.toolbar { toolbar } .toolbar { toolbar }
.environmentObject(context.viewState.activeTimelineContextProvider()) .environmentObject(context.viewState.activeTimelineContext)
.environment(\.timelineContext, context.viewState.activeTimelineContextProvider()) .environment(\.timelineContext, context.viewState.activeTimelineContext)
.onChange(of: context.screenMode) { _, _ in .onChange(of: context.screenMode) { _, _ in
context.send(viewAction: .changedScreenMode) context.send(viewAction: .changedScreenMode)
} }
@@ -137,7 +137,7 @@ struct MediaEventsTimelineScreen: View {
AudioMediaEventsTimelineView(timelineItem: timelineItem) AudioMediaEventsTimelineView(timelineItem: timelineItem)
case .voice(let timelineItem): case .voice(let timelineItem):
let defaultPlayerState = AudioPlayerState(id: .timelineItemIdentifier(timelineItem.id), title: L10n.commonVoiceMessage, duration: 0) let defaultPlayerState = AudioPlayerState(id: .timelineItemIdentifier(timelineItem.id), title: L10n.commonVoiceMessage, duration: 0)
let playerState = context.viewState.activeTimelineContextProvider().viewState.audioPlayerStateProvider?(timelineItem.id) ?? defaultPlayerState let playerState = context.viewState.activeTimelineContext.viewState.audioPlayerStateProvider?(timelineItem.id) ?? defaultPlayerState
VoiceMessageMediaEventsTimelineView(timelineItem: timelineItem, playerState: playerState) VoiceMessageMediaEventsTimelineView(timelineItem: timelineItem, playerState: playerState)
default: default:
EmptyView() EmptyView()
@@ -251,7 +251,7 @@ struct MediaEventsTimelineScreen_Previews: PreviewProvider, TestablePreview {
screenMode: MediaEventsTimelineScreenMode) -> MediaEventsTimelineScreenViewModel { screenMode: MediaEventsTimelineScreenMode) -> MediaEventsTimelineScreenViewModel {
MediaEventsTimelineScreenViewModel(mediaTimelineViewModel: makeTimelineViewModel(empty: empty), MediaEventsTimelineScreenViewModel(mediaTimelineViewModel: makeTimelineViewModel(empty: empty),
filesTimelineViewModel: makeTimelineViewModel(empty: empty), filesTimelineViewModel: makeTimelineViewModel(empty: empty),
initialViewState: .init(bindings: .init(screenMode: screenMode)), initialScreenMode: screenMode,
mediaProvider: MediaProviderMock(configuration: .init()), mediaProvider: MediaProviderMock(configuration: .init()),
userIndicatorController: UserIndicatorControllerMock(), userIndicatorController: UserIndicatorControllerMock(),
appMediator: AppMediatorMock()) appMediator: AppMediatorMock())