Revert "use reportID in getReportNotificationPreference"

This reverts commit d3daf61.
Expensify · Oct 20, 2023 · 8484427 · 8484427
1 parent d3daf61
commit 8484427
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/libs/ReportUtils.js b/src/libs/ReportUtils.js
@@ -778,13 +778,13 @@ function getReport(reportID) {
 }
 
 /**
- * Get the notification preference given a reportID
+ * Get the notification preference given a report
  *
- * @param {String} reportID
+ * @param {Object} report
  * @returns {String}
  */
-function getReportNotificationPreference(reportID) {
-    return lodashGet(getReport(reportID), 'notificationPreference', '');
+function getReportNotificationPreference(report) {
+    return lodashGet(report, 'notificationPreference', '');
 }
 
 /**

diff --git a/src/libs/actions/Report.js b/src/libs/actions/Report.js
@@ -329,7 +329,7 @@ function addActions(reportID, text = '', file) {
         isLastMessageDeletedParentAction: null,
     };
 
-    if (ReportUtils.getReportNotificationPreference(reportID) === CONST.REPORT.NOTIFICATION_PREFERENCE.HIDDEN) {
+    if (ReportUtils.getReportNotificationPreference(ReportUtils.getReport(reportID)) === CONST.REPORT.NOTIFICATION_PREFERENCE.HIDDEN) {
         optimisticReport.notificationPreference = CONST.REPORT.NOTIFICATION_PREFERENCE.ALWAYS;
     }