cleanup

ojasjadhav2 · Jun 21, 2023 · 325935c · 325935c
1 parent f2f010c
commit 325935c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/libs/ReportUtils.js b/src/libs/ReportUtils.js
@@ -1047,6 +1047,7 @@ function getReportName(report) {
 
 /**
  * Recursively navigates through thread parents to get the root report and workspace name.
+ * The recursion stops when we find a non thread or money request report, whichever comes first.
  * @param {Object} report
  * @returns {Object}
  */
@@ -1100,7 +1101,7 @@ function getChatRoomSubtitle(report) {
 }
 
 /**
- * Get the subtitleLink for the report
+ * Get the subtitle link for the report
  * @param {Object} report
  * @returns {String}
  */

diff --git a/src/pages/home/HeaderView.js b/src/pages/home/HeaderView.js
@@ -83,11 +83,11 @@ function HeaderView(props) {
     const participantPersonalDetails = OptionsListUtils.getPersonalDetailsForAccountIDs(participants, props.personalDetails);
     const isMultipleParticipant = participants.length > 1;
     const displayNamesWithTooltips = ReportUtils.getDisplayNamesWithTooltips(participantPersonalDetails, isMultipleParticipant);
-    const isThread = ReportUtils.isChatThread(props.report);
+    const isChatThread = ReportUtils.isChatThread(props.report);
     const isChatRoom = ReportUtils.isChatRoom(props.report);
     const isPolicyExpenseChat = ReportUtils.isPolicyExpenseChat(props.report);
     const isTaskReport = ReportUtils.isTaskReport(props.report);
-    const reportHeaderData = !isTaskReport && !isThread && props.report.parentReportID ? props.parentReport : props.report;
+    const reportHeaderData = !isTaskReport && !isChatThread && props.report.parentReportID ? props.parentReport : props.report;
     const title = ReportUtils.getReportName(reportHeaderData);
     const subtitle = ReportUtils.getChatRoomSubtitle(reportHeaderData);
     const subtitleLink = ReportUtils.getChatRoomSubtitleLink(reportHeaderData);
@@ -175,7 +175,7 @@ function HeaderView(props) {
                             ) : (
                                 <MultipleAvatars
                                     icons={icons}
-                                    shouldShowTooltip={!isChatRoom || isThread}
+                                    shouldShowTooltip={!isChatRoom || isChatThread}
                                 />
                             )}
                             <View style={[styles.flex1, styles.flexColumn]}>
@@ -185,7 +185,7 @@ function HeaderView(props) {
                                     tooltipEnabled
                                     numberOfLines={1}
                                     textStyles={[styles.headerText, styles.pre]}
-                                    shouldUseFullTitle={isChatRoom || isPolicyExpenseChat || isThread || isTaskReport}
+                                    shouldUseFullTitle={isChatRoom || isPolicyExpenseChat || isChatThread || isTaskReport}
                                 />
                                 {!_.isEmpty(subtitleLink) && (
                                     <PressableWithoutFeedback