From 5959fe8be4159cefd20bbc686ed4c66c64ca7739 Mon Sep 17 00:00:00 2001
From: johan12345 <johan.forstner@gmail.com>
Date: Sat, 2 Oct 2021 13:40:22 +0200
Subject: [PATCH] limit number of autocomplete results

---
 .../net/vonforst/evmap/adapter/PlaceAutocompleteAdapter.kt | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/app/src/main/java/net/vonforst/evmap/adapter/PlaceAutocompleteAdapter.kt b/app/src/main/java/net/vonforst/evmap/adapter/PlaceAutocompleteAdapter.kt
index b208d03d..b8ac2610 100644
--- a/app/src/main/java/net/vonforst/evmap/adapter/PlaceAutocompleteAdapter.kt
+++ b/app/src/main/java/net/vonforst/evmap/adapter/PlaceAutocompleteAdapter.kt
@@ -131,12 +131,15 @@ class PlaceAutocompleteAdapter(val context: Context, val location: LiveData<LatL
                                 publishResults(constraint, resultList.asFilterResults())
                             }
 
-                            if (query.length < 3) break
+                            // if we already have enough results or the query is short, stop here
+                            if (query.length < 3 || recentResults.size >= maxItems) break
 
                             // then search online
+                            val recentIds = recentPlaces.map { it.id }
                             resultList =
-                                recentPlaces.map { it.asAutocompletePlace(location.value) } +
+                                (recentPlaces.map { it.asAutocompletePlace(location.value) } +
                                         provider.autocomplete(query, location.value)
+                                            .filter { !recentIds.contains(it.id) }).take(maxItems)
                             break
                         } catch (e: ApiUnavailableException) {
                             e.printStackTrace()