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

Reduced room summary provider logging

Browse Source
This commit is contained in:
Stefan Ceriu
2023-08-22 15:59:47 +03:00
committed by Stefan Ceriu
parent 4b1083bf5e
commit 2b53104de1
3 changed files with 17 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files
ElementX/Sources/Screens/HomeScreen/HomeScreenViewModel.swift
+2 -2
View File
@@ -240,7 +240,7 @@ class HomeScreenViewModel: HomeScreenViewModelType, HomeScreenViewModelProtocol
return return
} }
MXLog.info("Updating rooms") MXLog.verbose("Updating rooms")
var rooms = [HomeScreenRoom]() var rooms = [HomeScreenRoom]()
@@ -259,7 +259,7 @@ class HomeScreenViewModel: HomeScreenViewModelType, HomeScreenViewModelProtocol
state.rooms = rooms state.rooms = rooms
MXLog.info("Finished updating rooms") MXLog.verbose("Finished updating rooms")
} }
private func buildRoom(with details: RoomSummaryDetails, invalidated: Bool) -> HomeScreenRoom { private func buildRoom(with details: RoomSummaryDetails, invalidated: Bool) -> HomeScreenRoom {
ElementX/Sources/Screens/MessageForwardingScreen/MessageForwardingScreenViewModel.swift
+2 -2
View File
@@ -69,7 +69,7 @@ class MessageForwardingScreenViewModel: MessageForwardingScreenViewModelType, Me
return return
} }
MXLog.info("Updating rooms") MXLog.verbose("Updating rooms")
var rooms = [MessageForwardingRoom]() var rooms = [MessageForwardingRoom]()
@@ -89,6 +89,6 @@ class MessageForwardingScreenViewModel: MessageForwardingScreenViewModelType, Me
state.rooms = rooms state.rooms = rooms
MXLog.info("Finished updating rooms") MXLog.verbose("Finished updating rooms")
} }
} }
ElementX/Sources/Services/Room/RoomSummary/RoomSummaryProvider.swift
+13 -13
View File
@@ -82,7 +82,7 @@ class RoomSummaryProvider: RoomSummaryProviderProtocol {
do { do {
listUpdatesSubscriptionResult = roomList.entriesWithDynamicFilter(listener: RoomListEntriesListenerProxy { [weak self] updates in listUpdatesSubscriptionResult = roomList.entriesWithDynamicFilter(listener: RoomListEntriesListenerProxy { [weak self] updates in
guard let self else { return } guard let self else { return }
MXLog.info("\(name): Received list update") MXLog.verbose("\(name): Received list update")
diffsPublisher.send(updates) diffsPublisher.send(updates)
}) })
@@ -131,7 +131,7 @@ class RoomSummaryProvider: RoomSummaryProviderProtocol {
span.exit() span.exit()
} }
MXLog.info("\(name): Received \(diffs.count) diffs, current room list \(rooms.compactMap { $0.id ?? "Empty" })") MXLog.verbose("\(name): Received \(diffs.count) diffs, current room list \(rooms.compactMap { $0.id ?? "Empty" })")
for diff in diffs { for diff in diffs {
// Special case resets in order to prevent large updates from blocking the UI // Special case resets in order to prevent large updates from blocking the UI
@@ -147,7 +147,7 @@ class RoomSummaryProvider: RoomSummaryProviderProtocol {
detectDuplicatesInRoomList(rooms) detectDuplicatesInRoomList(rooms)
MXLog.info("\(name): Finished applying \(diffs.count) diffs, new room list \(rooms.compactMap { $0.id ?? "Empty" })") MXLog.verbose("\(name): Finished applying \(diffs.count) diffs, new room list \(rooms.compactMap { $0.id ?? "Empty" })")
} }
private func processDiff(_ diff: RoomListEntriesUpdate) { private func processDiff(_ diff: RoomListEntriesUpdate) {
@@ -248,36 +248,36 @@ class RoomSummaryProvider: RoomSummaryProviderProtocol {
switch diff { switch diff {
case .pushFront(let value): case .pushFront(let value):
MXLog.info("\(name): Push Front \(value.debugIdentifier)") MXLog.verbose("\(name): Push Front \(value.debugIdentifier)")
let summary = buildSummaryForRoomListEntry(value) let summary = buildSummaryForRoomListEntry(value)
changes.append(.insert(offset: 0, element: summary, associatedWith: nil)) changes.append(.insert(offset: 0, element: summary, associatedWith: nil))
case .pushBack(let value): case .pushBack(let value):
MXLog.info("\(name): Push Back \(value.debugIdentifier)") MXLog.verbose("\(name): Push Back \(value.debugIdentifier)")
let summary = buildSummaryForRoomListEntry(value) let summary = buildSummaryForRoomListEntry(value)
changes.append(.insert(offset: rooms.count, element: summary, associatedWith: nil)) changes.append(.insert(offset: rooms.count, element: summary, associatedWith: nil))
case .append(values: let values): case .append(values: let values):
let debugIdentifiers = values.map(\.debugIdentifier) let debugIdentifiers = values.map(\.debugIdentifier)
MXLog.info("\(name): Append \(debugIdentifiers)") MXLog.verbose("\(name): Append \(debugIdentifiers)")
for (index, value) in values.enumerated() { for (index, value) in values.enumerated() {
let summary = buildSummaryForRoomListEntry(value) let summary = buildSummaryForRoomListEntry(value)
changes.append(.insert(offset: rooms.count + index, element: summary, associatedWith: nil)) changes.append(.insert(offset: rooms.count + index, element: summary, associatedWith: nil))
} }
case .set(let index, let value): case .set(let index, let value):
MXLog.info("\(name): Update \(value.debugIdentifier) at \(index)") MXLog.verbose("\(name): Update \(value.debugIdentifier) at \(index)")
let summary = buildSummaryForRoomListEntry(value) let summary = buildSummaryForRoomListEntry(value)
changes.append(.remove(offset: Int(index), element: summary, associatedWith: nil)) changes.append(.remove(offset: Int(index), element: summary, associatedWith: nil))
changes.append(.insert(offset: Int(index), element: summary, associatedWith: nil)) changes.append(.insert(offset: Int(index), element: summary, associatedWith: nil))
case .insert(let index, let value): case .insert(let index, let value):
MXLog.info("\(name): Insert at \(value.debugIdentifier) at \(index)") MXLog.verbose("\(name): Insert at \(value.debugIdentifier) at \(index)")
let summary = buildSummaryForRoomListEntry(value) let summary = buildSummaryForRoomListEntry(value)
changes.append(.insert(offset: Int(index), element: summary, associatedWith: nil)) changes.append(.insert(offset: Int(index), element: summary, associatedWith: nil))
case .remove(let index): case .remove(let index):
let summary = rooms[Int(index)] let summary = rooms[Int(index)]
MXLog.info("\(name): Remove \(summary.id ?? "") from \(index)") MXLog.verbose("\(name): Remove \(summary.id ?? "") from \(index)")
changes.append(.remove(offset: Int(index), element: summary, associatedWith: nil)) changes.append(.remove(offset: Int(index), element: summary, associatedWith: nil))
case .reset(let values): case .reset(let values):
let debugIdentifiers = values.map(\.debugIdentifier) let debugIdentifiers = values.map(\.debugIdentifier)
MXLog.info("\(name): Replace all items with \(debugIdentifiers)") MXLog.verbose("\(name): Replace all items with \(debugIdentifiers)")
for (index, summary) in rooms.enumerated() { for (index, summary) in rooms.enumerated() {
changes.append(.remove(offset: index, element: summary, associatedWith: nil)) changes.append(.remove(offset: index, element: summary, associatedWith: nil))
} }
@@ -286,16 +286,16 @@ class RoomSummaryProvider: RoomSummaryProviderProtocol {
changes.append(.insert(offset: index, element: buildSummaryForRoomListEntry(value), associatedWith: nil)) changes.append(.insert(offset: index, element: buildSummaryForRoomListEntry(value), associatedWith: nil))
} }
case .clear: case .clear:
MXLog.info("\(name): Clear all items") MXLog.verbose("\(name): Clear all items")
for (index, value) in rooms.enumerated() { for (index, value) in rooms.enumerated() {
changes.append(.remove(offset: index, element: value, associatedWith: nil)) changes.append(.remove(offset: index, element: value, associatedWith: nil))
} }
case .popFront: case .popFront:
MXLog.info("\(name): Pop Front") MXLog.verbose("\(name): Pop Front")
let summary = rooms[0] let summary = rooms[0]
changes.append(.remove(offset: 0, element: summary, associatedWith: nil)) changes.append(.remove(offset: 0, element: summary, associatedWith: nil))
case .popBack: case .popBack:
MXLog.info("\(name): Pop Back") MXLog.verbose("\(name): Pop Back")
guard let value = rooms.last else { guard let value = rooms.last else {
fatalError() fatalError()
} }