template: remove some unsecure dom interpretation (#682)

Author: codeurimpulsif

exodus/reports/templates/reports_list.html

@@ -34,8 +34,8 @@ <h4>
         <hr>
         {% if not handle %}
           <select class="custom-select"
-                  onchange="this.options[this.selectedIndex].value && (window.location = '/{{ LANGUAGE_CODE }}/reports/list' + this.options[this.selectedIndex].value);">
-            <option value=" ">{% trans "Latest reports" %}</option>
+                  onchange="const new_value = this.options[this.selectedIndex].value; if (new_value == '?filter=most_trackers') {window.location = '?filter=most_trackers';} else if (new_value == '?filter=no_trackers') {window.location = '?filter=no_trackers';} else {window.location = '?';}">
+            <option value="?">{% trans "Latest reports" %}</option>
             <option value="?filter=most_trackers" {% if filter == 'most_trackers' %}selected{% endif %}>
               {% trans "Sorted by number of trackers" %}</option>
             <option value="?filter=no_trackers" {% if filter == 'no_trackers' %}selected{% endif %}>

exodus/trackers/templates/trackers_list.html

@@ -24,8 +24,8 @@ <h4>
       <div class="col-xl-4 col-lg-5 col-md-8 col-12 mb-2">
         {% trans "Sorted by" %}&nbsp;
         <select class="custom-select"
-                onchange="this.options[this.selectedIndex].value && (window.location = '/{{ LANGUAGE_CODE }}/trackers/' + this.options[this.selectedIndex].value);">
-          <option value=" ">
+                onchange="const new_value = this.options[this.selectedIndex].value; if (new_value == '?filter=apps') {window.location = '?filter=apps';} else {window.location = '?';}">
+          <option value="?">
             {% trans "Alphabetical order" %}
           </option>
           <option value="?filter=apps" {% if filter == 'apps' %}selected{% endif %}>