Activer l'affichage en thread (#878)

tchapgouv · Sep 22, 2023 · f7966c4 · f7966c4
1 parent da37f78
commit f7966c4
Show file tree

Hide file tree

Showing 16 changed files with 470 additions and 487 deletions.
diff --git a/Riot/Managers/PushNotification/PushNotificationService.m b/Riot/Managers/PushNotification/PushNotificationService.m
@@ -487,28 +487,28 @@ - (void)handleNotificationInlineReplyForRoomId:(NSString*)roomId
             //  initialize data source for a thread or a room
             __block MXKRoomDataSource *dataSource;
             dispatch_group_t dispatchGroupDataSource = dispatch_group_create();
-            // Tchap: Disable Threads
-//            if (RiotSettings.shared.enableThreads && threadId)
-//            {
-//                dispatch_group_enter(dispatchGroupDataSource);
-//                [ThreadDataSource loadRoomDataSourceWithRoomId:roomId
-//                                                initialEventId:nil
-//                                                      threadId:threadId
-//                                              andMatrixSession:mxSession
-//                                                    onComplete:^(MXKRoomDataSource *threadDataSource) {
-//                    dataSource = threadDataSource;
-//                    dispatch_group_leave(dispatchGroupDataSource);
-//                }];
-//            }
-//            else
-//            {
+
+            if (RiotSettings.shared.enableThreads && threadId)
+            {
+                dispatch_group_enter(dispatchGroupDataSource);
+                [ThreadDataSource loadRoomDataSourceWithRoomId:roomId
+                                                initialEventId:nil
+                                                      threadId:threadId
+                                              andMatrixSession:mxSession
+                                                    onComplete:^(MXKRoomDataSource *threadDataSource) {
+                    dataSource = threadDataSource;
+                    dispatch_group_leave(dispatchGroupDataSource);
+                }];
+            }
+            else
+            {
                 dispatch_group_enter(dispatchGroupDataSource);
                 MXKRoomDataSourceManager *manager = [MXKRoomDataSourceManager sharedManagerForMatrixSession:mxSession];
                 [manager roomDataSourceForRoom:roomId create:YES onComplete:^(MXKRoomDataSource *roomDataSource) {
                     dataSource = roomDataSource;
                     dispatch_group_leave(dispatchGroupDataSource);
                 }];
-//            }
+            }
 
             dispatch_group_notify(dispatchGroupDataSource, dispatch_get_main_queue(), ^{
                 if (responseText != nil && responseText.length != 0)

diff --git a/Riot/Modules/Room/CellData/RoomBubbleCellData.m b/Riot/Modules/Room/CellData/RoomBubbleCellData.m
@@ -838,9 +838,9 @@ - (CGFloat)threadSummaryViewHeightForEventId:(NSString*)eventId
         //  component is not a thread root
         return 0;
     }
-    return 0; // Threads are disabled in Tchap
-//    return PlainRoomCellLayoutConstants.threadSummaryViewTopMargin +
-//        [ThreadSummaryView contentViewHeightForThread:component.thread fitting:self.maxTextViewWidth];
+//    return 0; // Threads are disabled in Tchap
+    return PlainRoomCellLayoutConstants.threadSummaryViewTopMargin +
+        [ThreadSummaryView contentViewHeightForThread:component.thread fitting:self.maxTextViewWidth];
 }
 
 - (CGFloat)fromAThreadViewHeightForEventId:(NSString*)eventId
@@ -866,9 +866,9 @@ - (CGFloat)fromAThreadViewHeightForEventId:(NSString*)eventId
         //  event is not in a thread
         return 0;
     }
-    return 0; // Threads are disabled in Tchap
-//    return PlainRoomCellLayoutConstants.fromAThreadViewTopMargin +
-//        [FromAThreadView contentViewHeightForEvent:component.event fitting:self.maxTextViewWidth];
+//    return 0; // Threads are disabled in Tchap
+    return PlainRoomCellLayoutConstants.fromAThreadViewTopMargin +
+        [FromAThreadView contentViewHeightForEvent:component.event fitting:self.maxTextViewWidth];
 }
 
 - (CGFloat)urlPreviewHeightForEventId:(NSString*)eventId

diff --git a/Riot/Modules/Room/DataSources/RoomDataSource.m b/Riot/Modules/Room/DataSources/RoomDataSource.m
@@ -266,47 +266,46 @@ - (void)fetchEncryptionTrustedLevel
 
 - (BOOL)shouldQueueEventForProcessing:(MXEvent *)event roomState:(MXRoomState *)roomState direction:(MXTimelineDirection)direction
 {
-    // Tchap: Disable Threads
-//    if (self.threadId)
-//    {
-//        //  if in a thread, ignore non-root event or events from other threads
-//        if (![event.eventId isEqualToString:self.threadId] && ![event.threadId isEqualToString:self.threadId])
-//        {
-//            //  Ignore the event
-//            return NO;
-//        }
-//        //  also ignore events related to un-threaded or events from other threads
-//        if (!event.isInThread && event.relatesTo.eventId)
-//        {
-//            MXEvent *relatedEvent = [self.mxSession.store eventWithEventId:event.relatesTo.eventId
-//                                                                    inRoom:event.roomId];
-//            if (![relatedEvent.threadId isEqualToString:self.threadId])
-//            {
-//                //  ignore the event
-//                return NO;
-//            }
-//        }
-//    }
-//    else if (RiotSettings.shared.enableThreads)
-//    {
-//        //  if not in a thread, ignore all threaded events
-//        if (event.isInThread)
-//        {
-//            //  ignore the event
-//            return NO;
-//        }
-//        //  also ignore events related to threaded events
-//        if (event.relatesTo.eventId)
-//        {
-//            MXEvent *relatedEvent = [self.mxSession.store eventWithEventId:event.relatesTo.eventId
-//                                                                    inRoom:event.roomId];
-//            if (relatedEvent.isInThread)
-//            {
-//                //  ignore the event
-//                return NO;
-//            }
-//        }
-//    }
+    if (self.threadId)
+    {
+        //  if in a thread, ignore non-root event or events from other threads
+        if (![event.eventId isEqualToString:self.threadId] && ![event.threadId isEqualToString:self.threadId])
+        {
+            //  Ignore the event
+            return NO;
+        }
+        //  also ignore events related to un-threaded or events from other threads
+        if (!event.isInThread && event.relatesTo.eventId)
+        {
+            MXEvent *relatedEvent = [self.mxSession.store eventWithEventId:event.relatesTo.eventId
+                                                                    inRoom:event.roomId];
+            if (![relatedEvent.threadId isEqualToString:self.threadId])
+            {
+                //  ignore the event
+                return NO;
+            }
+        }
+    }
+    else if (RiotSettings.shared.enableThreads)
+    {
+        //  if not in a thread, ignore all threaded events
+        if (event.isInThread)
+        {
+            //  ignore the event
+            return NO;
+        }
+        //  also ignore events related to threaded events
+        if (event.relatesTo.eventId)
+        {
+            MXEvent *relatedEvent = [self.mxSession.store eventWithEventId:event.relatesTo.eventId
+                                                                    inRoom:event.roomId];
+            if (relatedEvent.isInThread)
+            {
+                //  ignore the event
+                return NO;
+            }
+        }
+    }
 
     return [super shouldQueueEventForProcessing:event roomState:roomState direction:direction];
 }
@@ -478,26 +477,25 @@ - (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(N
                                               cellData:cellData contentViewPositionY:bottomPositionY upperDecorationView:urlPreviewView];
                     }
 
-                    // Tchap: Disable Threads
-//                    ThreadSummaryView *threadSummaryView;
-//                    
-//                    //  display thread summary view if the component has a thread in the room timeline
-//                    if (RiotSettings.shared.enableThreads && component.thread && !self.threadId)
-//                    {
-//                        threadSummaryView = [[ThreadSummaryView alloc] initWithThread:component.thread
-//                                                                              session:self.mxSession];
-//                        threadSummaryView.delegate = self;
-//                        threadSummaryView.tag = index;
-//                        
-//                        [temporaryViews addObject:threadSummaryView];
-//                        UIView *upperDecorationView = reactionsView ?: urlPreviewView;
-//
-//                        [cellDecorator addThreadSummaryView:threadSummaryView
-//                                                     toCell:bubbleCell
-//                                                   cellData:cellData
-//                                       contentViewPositionY:bottomPositionY
-//                                        upperDecorationView:upperDecorationView];
-//                    }
+                    ThreadSummaryView *threadSummaryView;
+
+                    //  display thread summary view if the component has a thread in the room timeline
+                    if (RiotSettings.shared.enableThreads && component.thread && !self.threadId)
+                    {
+                        threadSummaryView = [[ThreadSummaryView alloc] initWithThread:component.thread
+                                                                              session:self.mxSession];
+                        threadSummaryView.delegate = self;
+                        threadSummaryView.tag = index;
+
+                        [temporaryViews addObject:threadSummaryView];
+                        UIView *upperDecorationView = reactionsView ?: urlPreviewView;
+
+                        [cellDecorator addThreadSummaryView:threadSummaryView
+                                                     toCell:bubbleCell
+                                                   cellData:cellData
+                                       contentViewPositionY:bottomPositionY
+                                        upperDecorationView:upperDecorationView];
+                    }
 
                     MXKReceiptSendersContainer* avatarsContainer;
 
@@ -1213,12 +1211,11 @@ - (void)didCloseURLPreviewView:(URLPreviewView *)previewView for:(NSString *)eve
 
 #pragma mark - ThreadSummaryViewDelegate
 
-// Tchap: Disable Threads
-//- (void)threadSummaryViewTapped:(ThreadSummaryView *)summaryView
-//{
-//    [self.roomDataSourceDelegate roomDataSource:self
-//                                   didTapThread:summaryView.thread];
-//}
+- (void)threadSummaryViewTapped:(ThreadSummaryView *)summaryView
+{
+    [self.roomDataSourceDelegate roomDataSource:self
+                                   didTapThread:summaryView.thread];
+}
 
 #pragma mark - Location sharing
 

diff --git a/Riot/Modules/Room/RoomCoordinator.swift b/Riot/Modules/Room/RoomCoordinator.swift
@@ -80,13 +80,12 @@ final class RoomCoordinator: NSObject, RoomCoordinatorProtocol {
         self.selectedEventId = parameters.eventId
         self.userIndicatorStore = UserIndicatorStore(presenter: parameters.userIndicatorPresenter)
 
-        // Tchap: Disable Threads
-//        if let threadId = parameters.threadId {
-//            self.roomViewController = ThreadViewController.instantiate(withThreadId: threadId,
-//                                                                       configuration: parameters.displayConfiguration)
-//        } else {
+        if let threadId = parameters.threadId {
+            self.roomViewController = ThreadViewController.instantiate(withThreadId: threadId,
+                                                                       configuration: parameters.displayConfiguration)
+        } else {
             self.roomViewController = RoomViewController.instantiate(with: parameters.displayConfiguration)
-//        }
+        }
         self.roomViewController.userIndicatorStore = userIndicatorStore
         self.roomViewController.showSettingsInitially = parameters.showSettingsInitially
 

diff --git a/Riot/Modules/Room/RoomViewController.h b/Riot/Modules/Room/RoomViewController.h
@@ -31,9 +31,8 @@
 @class UniversalLinkParameters;
 @protocol RoomViewControllerDelegate;
 @class RoomDisplayConfiguration;
-// Tchap: Disable Threads
+@class ThreadsCoordinatorBridgePresenter;
 // Tchap: Disable Live location sharing
-//@class ThreadsCoordinatorBridgePresenter;
 //@class LiveLocationSharingBannerView;
 @class VoiceBroadcastService;
 @class ComposerLinkActionBridgePresenter;
@@ -355,8 +354,7 @@ didRequestEditForPollWithStartEvent:(MXEvent *)startEvent;
 - (void)roomViewControllerDidTapLiveLocationSharingBanner:(RoomViewController *)roomViewController;
 
 /// Request a threads coordinator for a given threadId, used to open a thread from within a room.
-// Tchap: Disable Threads
-//- (nullable ThreadsCoordinatorBridgePresenter *)threadsCoordinatorForRoomViewController:(RoomViewController *)roomViewController threadId:(nullable NSString *)threadId;
+- (nullable ThreadsCoordinatorBridgePresenter *)threadsCoordinatorForRoomViewController:(RoomViewController *)roomViewController threadId:(nullable NSString *)threadId;
 
 @end