Skip to content

Commit

Permalink
Do not search on page load (#291)
Browse files Browse the repository at this point in the history
Only trigger a search if the user submits the search or the URL has a
term (deep link).
  • Loading branch information
kofrezo committed Feb 17, 2023
1 parent 3f26285 commit 3d6ce09
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 2 deletions.
7 changes: 7 additions & 0 deletions serveradmin/servershell/static/js/servershell/search.js
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,13 @@ servershell.submit_search = function() {
return servershell.alert('Pending request, cancel it or wait for it to finish!', 'danger');
}

// Do not submit search on load.
let params = new URLSearchParams(window.location.search);
if (servershell.term === null && !params.has('term')) {
servershell.term = '';
return;
}

let url = $('#search_form').get(0).action;
console.debug(`Submitting query to URL "${url}" with data:`);
spinner.enable('search');
Expand Down
2 changes: 1 addition & 1 deletion serveradmin/servershell/templates/servershell/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -200,7 +200,7 @@

// Initialize servershell object properties with initial values
// from backend. This allows us setting defaults from backend.
servershell.term = "{{ term|escapejs }}";
servershell.term = {% if term %}"{{ term|escapejs }}"{% else %}null{% endif %};
servershell._term = "{{ term|escapejs }}";
servershell.command = "";
servershell.understood = "";
Expand Down
2 changes: 1 addition & 1 deletion serveradmin/servershell/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ def index(request):
shown_attributes = get_default_shown_attributes()

return TemplateResponse(request, 'servershell/index.html', {
'term': request.GET.get('term', request.session.get('term', '')),
'term': request.GET.get('term', request.session.get('term')),
'shown_attributes': shown_attributes,
'deep_link': bool(strtobool(request.GET.get('deep_link', 'false'))),
'attributes': attributes_json,
Expand Down

0 comments on commit 3d6ce09

Please sign in to comment.