From 0971fb0aeef926f16f416073419c6b32c084c9e3 Mon Sep 17 00:00:00 2001 From: Bnyro Date: Wed, 11 Oct 2023 20:30:37 +0200 Subject: [PATCH] [fix] search.js: crash on homepage when setting form listeners --- searx/static/themes/simple/src/js/main/search.js | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/searx/static/themes/simple/src/js/main/search.js b/searx/static/themes/simple/src/js/main/search.js index 2ad8e48c5..ac1ece535 100644 --- a/searx/static/themes/simple/src/js/main/search.js +++ b/searx/static/themes/simple/src/js/main/search.js @@ -150,8 +150,17 @@ } } - // vanilla js version of search_on_category_select.js - if (qinput !== null && searxng.settings.search_on_category_select) { + // Additionally to searching when selecting a new category, we also + // automatically start a new search request when the user changes a search + // filter (safesearch, time range or language) (this requires JavaScript + // though) + if ( + qinput !== null + && searxng.settings.search_on_category_select + // If .search_filters is undefined (invisible) we are on the homepage and + // hence don't have to set any listeners + && d.querySelector(".search_filters") != null + ) { searxng.on(d.getElementById('safesearch'), 'change', submitIfQuery); searxng.on(d.getElementById('time_range'), 'change', submitIfQuery); searxng.on(d.getElementById('language'), 'change', submitIfQuery);