change didSubscribeToReportLeavingEvents check to early return

Expensify · Sep 20, 2023 · b2303b2 · b2303b2
1 parent 8926377
commit b2303b2
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/pages/home/ReportScreen.js b/src/pages/home/ReportScreen.js
@@ -290,9 +290,11 @@ function ReportScreen({
         fetchReportIfNeeded();
         ComposerActions.setShouldShowComposeInput(true);
         return () => {
-            if (didSubscribeToReportLeavingEvents) {
-                Report.unsubscribeFromLeavingRoomReportChannel(report.reportID);
+            if (!didSubscribeToReportLeavingEvents) {
+                return;
             }
+
+            Report.unsubscribeFromLeavingRoomReportChannel(report.reportID);
         };
 
         // I'm disabling the warning, as it expects to use exhaustive deps, even though we want this useEffect to run only on the first render.