Skip to content

Commit 3f38b2e

Browse files
authored
Merge pull request #88133 from Expensify/jsenyitko-fix-reset-sort
[CP Staging] Rename saved search when re-saving
2 parents 2f5201a + 55d72c8 commit 3f38b2e

1 file changed

Lines changed: 3 additions & 9 deletions

File tree

src/pages/Search/SearchSavePage.tsx

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -92,25 +92,19 @@ function FilterValue({filterKey, value}: FilterValueWithKeyProps) {
9292
function SearchSavePage() {
9393
const styles = useThemeStyles();
9494
const {translate, localeCompare} = useLocalize();
95-
const [savedSearches] = useOnyx(ONYXKEYS.SAVED_SEARCHES);
9695
const [searchAdvancedFiltersForm = getEmptyObject<Partial<SearchAdvancedFiltersForm>>()] = useOnyx(ONYXKEYS.FORMS.SEARCH_ADVANCED_FILTERS_FORM);
9796
const [name, setName] = useState('');
9897

9998
const {currentSearchQueryJSON} = useSearchStateContext();
10099

101100
const onSaveSearch = () => {
102-
const savedSearchKeys = Object.keys(savedSearches ?? {});
103-
if (!currentSearchQueryJSON || (savedSearches && savedSearchKeys.includes(String(currentSearchQueryJSON.hash)))) {
104-
// If the search is already saved, we only display the results as we don't need to save it.
101+
if (!currentSearchQueryJSON) {
105102
Navigation.goBack();
106103
return;
107104
}
108105

109-
if (name) {
110-
saveSearch({queryJSON: currentSearchQueryJSON, newName: name});
111-
} else {
112-
saveSearch({queryJSON: currentSearchQueryJSON});
113-
}
106+
const newName = name.trim() || currentSearchQueryJSON?.inputQuery;
107+
saveSearch({queryJSON: currentSearchQueryJSON, newName});
114108
Navigation.goBack();
115109
};
116110

0 commit comments

Comments
 (0)