Bring back correct const and make 'unreportedExpenses' prop dependency more specific

OlGierd03 · OlGierd03 · commit 408e21545071 · 2025-12-05T17:18:24.000+01:00
diff --git a/src/pages/AddUnreportedExpense.tsx b/src/pages/AddUnreportedExpense.tsx
@@ -70,7 +70,7 @@ function AddUnreportedExpense({route}: AddUnreportedExpensePageType) {
                 return [];
             }
             return Object.values(transactions || {}).filter((item) => {
-                const isUnreported = true;
+                const isUnreported = item?.reportID === CONST.REPORT.UNREPORTED_REPORT_ID || item?.reportID === '';
                 if (!isUnreported) {
                     return false;
                 }
@@ -221,7 +221,7 @@ function AddUnreportedExpense({route}: AddUnreportedExpensePageType) {
             return translate('common.noResultsFound');
         }
         return '';
-    }, [debouncedSearchValue, unreportedExpenses, translate]);
+    }, [debouncedSearchValue, unreportedExpenses?.length, translate]);
 
     const textInputOptions = useMemo(
         () => ({

Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ function AddUnreportedExpense({route}: AddUnreportedExpensePageType) {`
`70`	`70`	`return [];`
`71`	`71`	`}`
`72`	`72`	`return Object.values(transactions \|\| {}).filter((item) => {`
`73`		`- const isUnreported = true;`
	`73`	`+ const isUnreported = item?.reportID === CONST.REPORT.UNREPORTED_REPORT_ID \|\| item?.reportID === '';`
`74`	`74`	`if (!isUnreported) {`
`75`	`75`	`return false;`
`76`	`76`	`}`
`@@ -221,7 +221,7 @@ function AddUnreportedExpense({route}: AddUnreportedExpensePageType) {`
`221`	`221`	`return translate('common.noResultsFound');`
`222`	`222`	`}`
`223`	`223`	`return '';`
`224`		`- }, [debouncedSearchValue, unreportedExpenses, translate]);`
	`224`	`+ }, [debouncedSearchValue, unreportedExpenses?.length, translate]);`
`225`	`225`
`226`	`226`	`const textInputOptions = useMemo(`
`227`	`227`	`() => ({`