Autofill: number-only results are now subject to user input filtering

master
Skylar Ittner 4 years ago
parent 261b797a85
commit e1ded04c9b

@ -76,7 +76,7 @@ function addAutofillEntry(address) {
function searchAutofill(q, number) { function searchAutofill(q, number) {
var byNumber = []; var byNumber = [];
if (typeof number != 'undefined') { if (typeof number != 'undefined') {
byNumber = searchAutofillByNumber(number); byNumber = searchAutofillByNumber(number, q);
} }
var byStreet = []; var byStreet = [];
@ -87,7 +87,7 @@ function searchAutofill(q, number) {
return byNumber.concat(byStreet.filter((item) => byNumber.indexOf(item) < 0)); return byNumber.concat(byStreet.filter((item) => byNumber.indexOf(item) < 0));
} }
function searchAutofillByNumber(number) { function searchAutofillByNumber(number, q) {
if (typeof autofillDB[number] == 'undefined') { if (typeof autofillDB[number] == 'undefined') {
return []; return [];
} }
@ -96,10 +96,18 @@ function searchAutofillByNumber(number) {
return b[1] - a[1]; return b[1] - a[1];
}); });
var query = false;
if (typeof q != 'undefined' && q != "") {
query = true;
}
var streets = []; var streets = [];
for (var i = 0; i < sorted.length; i++) { for (var i = 0; i < sorted.length; i++) {
streets.push(sorted[i][0]); // if there's no search query OR if the query matches the current item
if (!query || (query && sorted[i][0].toLowerCase().includes(q))) {
streets.push(sorted[i][0]);
}
} }
return streets; return streets;

Loading…
Cancel
Save