diff --git a/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuReaction.swift b/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionMenuItemViewData.swift similarity index 80% rename from Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuReaction.swift rename to Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionMenuItemViewData.swift index 93de3403d..df9f4ed27 100644 --- a/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuReaction.swift +++ b/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionMenuItemViewData.swift @@ -1,12 +1,12 @@ /* Copyright 2019 New Vector Ltd - + Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at - + http://www.apache.org/licenses/LICENSE-2.0 - + Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. @@ -16,9 +16,7 @@ import Foundation -enum ReactionsMenuReaction: String { - case agree = "👍" - case disagree = "👎" - case like = "🙂" - case dislike = "😔" +struct ReactionMenuItemViewData { + let emoji: String + let isSelected: Bool } diff --git a/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuViewAction.swift b/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuViewAction.swift index b98ac829a..3608a9187 100644 --- a/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuViewAction.swift +++ b/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuViewAction.swift @@ -18,5 +18,6 @@ import UIKit /// Action chosen by the user enum ReactionsMenuViewAction { - case toggleReaction(ReactionsMenuReaction) + case loadData + case tap(reaction: String) } diff --git a/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuViewModel.swift b/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuViewModel.swift index 846ce2049..c2ec99830 100644 --- a/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuViewModel.swift +++ b/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuViewModel.swift @@ -14,158 +14,71 @@ limitations under the License. */ -import UIKit +import Foundation @objc final class ReactionsMenuViewModel: NSObject, ReactionsMenuViewModelType { - + // MARK: - Properties - + + private let reactions = ["👍", "👎", "😄", "🎉", "😕", "❤️", "🚀", "👀"] + private var currentViewDatas: [ReactionMenuItemViewData] = [] + // MARK: Private - private let aggregations: MXAggregations - private let roomId: String + + private let aggregatedReactions: MXAggregatedReactions? + private let reactionsViewData: [ReactionMenuItemViewData] = [] private let eventId: String - + // MARK: Public - - private(set) var isAgreeButtonSelected: Bool = false - private(set) var isDisagreeButtonSelected: Bool = false - private(set) var isLikeButtonSelected: Bool = false - private(set) var isDislikeButtonSelected: Bool = false - - weak var viewDelegate: ReactionsMenuViewModelDelegate? - @objc weak var coordinatorDelegate: ReactionsMenuViewModelCoordinatorDelegate? - + + @objc weak var viewModelDelegate: ReactionsMenuViewModelCoordinatorDelegate? + weak var viewDelegate: ReactionsMenuViewModelViewDelegate? + // MARK: - Setup - - @objc init(aggregations: MXAggregations, roomId: String, eventId: String) { - self.aggregations = aggregations - self.roomId = roomId + + @objc init(aggregatedReactions: MXAggregatedReactions?, + eventId: String) { + self.aggregatedReactions = aggregatedReactions self.eventId = eventId - - super.init() - - self.loadData() - self.listenToDataUpdate() } - + // MARK: - Public - + func process(viewAction: ReactionsMenuViewAction) { - var reaction: ReactionsMenuReaction? - var newState: Bool? - switch viewAction { - case .toggleReaction(let menuReaction): - reaction = menuReaction - - switch menuReaction { - case .agree: - newState = !self.isAgreeButtonSelected - case .disagree: - newState = !self.isDisagreeButtonSelected - case .like: - newState = !self.isLikeButtonSelected - case .dislike: - newState = !self.isDislikeButtonSelected - } - } - - guard let theReaction = reaction, let theNewState = newState else { - return - } - - self.react(withReaction: theReaction, selected: theNewState) - } - - // MARK: - Private - - private func resetData() { - self.isAgreeButtonSelected = false - self.isDisagreeButtonSelected = false - self.isLikeButtonSelected = false - self.isDislikeButtonSelected = false - } - - private func loadData() { - guard let reactionCounts = self.aggregations.aggregatedReactions(onEvent: self.eventId, inRoom: self.roomId)?.withNonZeroCount()?.reactions else { - return - } - - self.resetData() - reactionCounts.forEach { (reactionCount) in - if reactionCount.myUserHasReacted { - if let reaction = ReactionsMenuReaction(rawValue: reactionCount.reaction) { - switch reaction { - case .agree: - self.isAgreeButtonSelected = true - case .disagree: - self.isDisagreeButtonSelected = true - case .like: - self.isLikeButtonSelected = true - case .dislike: - self.isDislikeButtonSelected = true - } + case .loadData: + self.loadData() + case .tap(let reaction): + if let viewData = self.currentViewDatas.first(where: { $0.emoji == reaction }) { + if viewData.isSelected { + self.coordinatorDelegate?.reactionsMenuViewModel(self, didRemoveReaction: reaction, forEventId: self.eventId) + } else { + self.coordinatorDelegate?.reactionsMenuViewModel(self, didAddReaction: reaction, forEventId: self.eventId) } } } - - self.viewDelegate?.reactionsMenuViewModelDidUpdate(self) - } - - private func listenToDataUpdate() { - self.aggregations.listenToReactionCountUpdate(inRoom: self.roomId) { [weak self] (changes) in - - guard let sself = self else { - return - } - - if changes[sself.eventId] != nil { - sself.loadData() - } - } } - private func react(withReaction reaction: ReactionsMenuReaction, selected: Bool) { + // MARK: - Private + + private func loadData() { + let reactionCounts = self.aggregatedReactions?.withNonZeroCount()?.reactions ?? [] - // If required, unreact first - if selected { - self.ensure3StateButtons(withReaction: reaction) + var quickReactionsWithUserReactedFlag: [String: Bool] = Dictionary(uniqueKeysWithValues: self.reactions.map { ($0, false) }) + + reactionCounts.forEach { (reactionCount) in + if let hasUserReacted = quickReactionsWithUserReactedFlag[reactionCount.reaction], hasUserReacted == false { + quickReactionsWithUserReactedFlag[reactionCount.reaction] = reactionCount.myUserHasReacted + } } - let reactionString = reaction.rawValue + let reactionMenuItemViewDatas: [ReactionMenuItemViewData] = self.reactions.map { reaction -> ReactionMenuItemViewData in + let isSelected = quickReactionsWithUserReactedFlag[reaction] ?? false + return ReactionMenuItemViewData(emoji: reaction, isSelected: isSelected) + } - if selected { - self.coordinatorDelegate?.reactionsMenuViewModel(self, didAddReaction: reactionString, forEventId: self.eventId) - } else { - self.coordinatorDelegate?.reactionsMenuViewModel(self, didRemoveReaction: reactionString, forEventId: self.eventId) - } - } - - // We can like, dislike, be indifferent but we cannot like & dislike at the same time - private func ensure3StateButtons(withReaction reaction: ReactionsMenuReaction) { - var unreaction: ReactionsMenuReaction? - - switch reaction { - case .agree: - if isDisagreeButtonSelected { - unreaction = .disagree - } - case .disagree: - if isAgreeButtonSelected { - unreaction = .agree - } - case .like: - if isDislikeButtonSelected { - unreaction = .dislike - } - case .dislike: - if isLikeButtonSelected { - unreaction = .like - } - } - - if let unreaction = unreaction { - self.react(withReaction: unreaction, selected: false) - } + self.currentViewDatas = reactionMenuItemViewDatas + + self.viewDelegate?.reactionsMenuViewModel(self, didUpdateViewState: ReactionsMenuViewState.loaded(reactionsViewData: reactionMenuItemViewDatas)) } } diff --git a/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuViewModelType.swift b/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuViewModelType.swift index 111c01d03..0e162f6c1 100644 --- a/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuViewModelType.swift +++ b/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuViewModelType.swift @@ -16,8 +16,8 @@ import Foundation -protocol ReactionsMenuViewModelDelegate: class { - func reactionsMenuViewModelDidUpdate(_ viewModel: ReactionsMenuViewModelType) +protocol ReactionsMenuViewModelViewDelegate: class { + func reactionsMenuViewModel(_ viewModel: ReactionsMenuViewModel, didUpdateViewState viewState: ReactionsMenuViewState) } @objc protocol ReactionsMenuViewModelCoordinatorDelegate: class { @@ -25,16 +25,10 @@ protocol ReactionsMenuViewModelDelegate: class { func reactionsMenuViewModel(_ viewModel: ReactionsMenuViewModel, didRemoveReaction reaction: String, forEventId eventId: String) } - protocol ReactionsMenuViewModelType { - - var isAgreeButtonSelected: Bool { get } - var isDisagreeButtonSelected: Bool { get } - var isLikeButtonSelected: Bool { get } - var isDislikeButtonSelected: Bool { get } - - var viewDelegate: ReactionsMenuViewModelDelegate? { get set } + var coordinatorDelegate: ReactionsMenuViewModelCoordinatorDelegate? { get set } - + var viewDelegate: ReactionsMenuViewModelViewDelegate? { get set } + func process(viewAction: ReactionsMenuViewAction) } diff --git a/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuViewState.swift b/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuViewState.swift new file mode 100644 index 000000000..e74a77ef7 --- /dev/null +++ b/Riot/Modules/Room/ContextualMenu/ReactionsMenu/ReactionsMenuViewState.swift @@ -0,0 +1,22 @@ +/* + Copyright 2019 New Vector Ltd + + Licensed under the Apache License, Version 2.0 (the "License"); + you may not use this file except in compliance with the License. + You may obtain a copy of the License at + + http://www.apache.org/licenses/LICENSE-2.0 + + Unless required by applicable law or agreed to in writing, software + distributed under the License is distributed on an "AS IS" BASIS, + WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + See the License for the specific language governing permissions and + limitations under the License. + */ + +import Foundation + +/// ReactionsMenuView view state +enum ReactionsMenuViewState { + case loaded(reactionsViewData: [ReactionMenuItemViewData]) +}