From 20409343fd02ae087f30bf82433a7fde37ca49e8 Mon Sep 17 00:00:00 2001
From: johan12345 <johan.forstner@gmail.com>
Date: Wed, 22 Jun 2022 22:35:37 +0200
Subject: [PATCH] Android Auto/Automotive: add "select all" option to tariffs
 selection screen

fixes #183
---
 .../vonforst/evmap/auto/SearchSelectScreen.kt | 38 +++++++++++++++++++
 .../vonforst/evmap/auto/SettingsScreens.kt    | 14 +++++++
 app/src/google/res/values-de/values.xml       |  2 +
 app/src/google/res/values/values.xml          |  2 +
 app/src/main/res/drawable/ic_select_all.xml   | 10 +++++
 app/src/main/res/drawable/ic_select_none.xml  | 10 +++++
 6 files changed, 76 insertions(+)
 create mode 100644 app/src/main/res/drawable/ic_select_all.xml
 create mode 100644 app/src/main/res/drawable/ic_select_none.xml

diff --git a/app/src/google/java/net/vonforst/evmap/auto/SearchSelectScreen.kt b/app/src/google/java/net/vonforst/evmap/auto/SearchSelectScreen.kt
index 0f898304..f36c571c 100644
--- a/app/src/google/java/net/vonforst/evmap/auto/SearchSelectScreen.kt
+++ b/app/src/google/java/net/vonforst/evmap/auto/SearchSelectScreen.kt
@@ -1,11 +1,14 @@
 package net.vonforst.evmap.auto
 
 import androidx.car.app.CarContext
+import androidx.car.app.CarToast
 import androidx.car.app.Screen
 import androidx.car.app.constraints.ConstraintManager
 import androidx.car.app.model.*
+import androidx.core.graphics.drawable.IconCompat
 import androidx.lifecycle.lifecycleScope
 import kotlinx.coroutines.launch
+import net.vonforst.evmap.R
 
 abstract class MultiSelectSearchScreen<T>(ctx: CarContext) : Screen(ctx),
     SearchTemplate.SearchCallback {
@@ -16,6 +19,7 @@ abstract class MultiSelectSearchScreen<T>(ctx: CarContext) : Screen(ctx),
         ctx.constraintManager.getContentLimit(ConstraintManager.CONTENT_LIMIT_TYPE_LIST)
     } else 6
     protected abstract val isMultiSelect: Boolean
+    protected abstract val shouldShowSelectAll: Boolean
 
     override fun onGetTemplate(): Template {
         if (fullList == null) {
@@ -33,6 +37,30 @@ abstract class MultiSelectSearchScreen<T>(ctx: CarContext) : Screen(ctx),
             } ?: run {
                 setLoading(true)
             }
+            if (isMultiSelect) {
+                setActionStrip(ActionStrip.Builder().apply {
+                    addAction(
+                        Action.Builder().setIcon(
+                            CarIcon.Builder(
+                                IconCompat.createWithResource(
+                                    carContext,
+                                    R.drawable.ic_select_all
+                                )
+                            ).build()
+                        ).setOnClickListener(::selectAll).build()
+                    )
+                    addAction(
+                        Action.Builder().setIcon(
+                            CarIcon.Builder(
+                                IconCompat.createWithResource(
+                                    carContext,
+                                    R.drawable.ic_select_none
+                                )
+                            ).build()
+                        ).setOnClickListener(::selectNone).build()
+                    )
+                }.build())
+            }
         }.build()
     }
 
@@ -86,6 +114,16 @@ abstract class MultiSelectSearchScreen<T>(ctx: CarContext) : Screen(ctx),
 
     abstract fun toggleSelected(item: T)
 
+    open fun selectAll() {
+        CarToast.makeText(carContext, R.string.selecting_all, CarToast.LENGTH_SHORT).show()
+        invalidate()
+    }
+
+    open fun selectNone() {
+        CarToast.makeText(carContext, R.string.selecting_none, CarToast.LENGTH_SHORT).show()
+        invalidate()
+    }
+
     abstract fun isSelected(it: T): Boolean
 
     abstract fun getLabel(it: T): String
diff --git a/app/src/google/java/net/vonforst/evmap/auto/SettingsScreens.kt b/app/src/google/java/net/vonforst/evmap/auto/SettingsScreens.kt
index 74cc236b..b668c8bf 100644
--- a/app/src/google/java/net/vonforst/evmap/auto/SettingsScreens.kt
+++ b/app/src/google/java/net/vonforst/evmap/auto/SettingsScreens.kt
@@ -183,6 +183,7 @@ class SelectVehiclesScreen(ctx: CarContext) : MultiSelectSearchScreen<Chargepric
     private val prefs = PreferenceDataSource(carContext)
     private var api = ChargepriceApi.create(carContext.getString(R.string.chargeprice_key))
     override val isMultiSelect = true
+    override val shouldShowSelectAll = false
 
     override fun isSelected(it: ChargepriceCar): Boolean {
         return prefs.chargepriceMyVehicles.contains(it.id)
@@ -207,6 +208,7 @@ class SelectTariffsScreen(ctx: CarContext) : MultiSelectSearchScreen<Chargeprice
     private val prefs = PreferenceDataSource(carContext)
     private var api = ChargepriceApi.create(carContext.getString(R.string.chargeprice_key))
     override val isMultiSelect = true
+    override val shouldShowSelectAll = true
 
     override fun isSelected(it: ChargepriceTariff): Boolean {
         return prefs.chargepriceMyTariffsAll or (prefs.chargepriceMyTariffs?.contains(it.id)
@@ -230,6 +232,17 @@ class SelectTariffsScreen(ctx: CarContext) : MultiSelectSearchScreen<Chargeprice
         }
     }
 
+    override fun selectAll() {
+        prefs.chargepriceMyTariffsAll = true
+        super.selectAll()
+    }
+
+    override fun selectNone() {
+        prefs.chargepriceMyTariffsAll = false
+        prefs.chargepriceMyTariffs = emptySet()
+        super.selectNone()
+    }
+
     override fun getLabel(it: ChargepriceTariff): String {
         return if (!it.name.lowercase().startsWith(it.provider.lowercase())) {
             "${it.provider} ${it.name}"
@@ -246,6 +259,7 @@ class SelectTariffsScreen(ctx: CarContext) : MultiSelectSearchScreen<Chargeprice
 class SelectCurrencyScreen(ctx: CarContext) : MultiSelectSearchScreen<Pair<String, String>>(ctx) {
     private val prefs = PreferenceDataSource(carContext)
     override val isMultiSelect = false
+    override val shouldShowSelectAll = false
 
     override fun isSelected(it: Pair<String, String>): Boolean =
         prefs.chargepriceCurrency == it.second
diff --git a/app/src/google/res/values-de/values.xml b/app/src/google/res/values-de/values.xml
index 42ccd5d9..bf0d7e5f 100644
--- a/app/src/google/res/values-de/values.xml
+++ b/app/src/google/res/values-de/values.xml
@@ -37,4 +37,6 @@
     <string name="auto_chargeprice_vehicle_ambiguous">Mehrere der in der App ausgewählten Fahrzeuge passen zu diesem Fahrzeug (%1$s %2$s).</string>
     <string name="settings_android_auto_chargeprice_range">Ladebereich für Preisvergleich</string>
     <string name="data_sources_hint">In den Einstellungen kannst du auch zwischen Google Maps und OpenStreetMap (Mapbox) für die Kartendaten wechseln.</string>
+    <string name="selecting_all">alle Einträge ausgewählt</string>
+    <string name="selecting_none">alle Einträge abgewählt</string>
 </resources>
\ No newline at end of file
diff --git a/app/src/google/res/values/values.xml b/app/src/google/res/values/values.xml
index 2a5f399f..2d24a883 100644
--- a/app/src/google/res/values/values.xml
+++ b/app/src/google/res/values/values.xml
@@ -47,4 +47,6 @@
     <string name="auto_chargeprice_vehicle_ambiguous">Mehrere der in der App ausgewählten Fahrzeuge passen zu diesem Fahrzeug (%1$s %2$s).</string>
     <string name="settings_android_auto_chargeprice_range">Charging range for price comparison</string>
     <string name="data_sources_hint">In the settings you can also switch between Google Maps and OpenStreetMap (Mapbox) for the map data.</string>
+    <string name="selecting_all">selected all items</string>
+    <string name="selecting_none">deselected all items</string>
 </resources>
\ No newline at end of file
diff --git a/app/src/main/res/drawable/ic_select_all.xml b/app/src/main/res/drawable/ic_select_all.xml
new file mode 100644
index 00000000..15b2df09
--- /dev/null
+++ b/app/src/main/res/drawable/ic_select_all.xml
@@ -0,0 +1,10 @@
+<vector xmlns:android="http://schemas.android.com/apk/res/android"
+    android:width="24dp"
+    android:height="24dp"
+    android:viewportWidth="24"
+    android:viewportHeight="24"
+    android:tint="?attr/colorControlNormal">
+  <path
+      android:fillColor="@android:color/white"
+      android:pathData="M18,7l-1.41,-1.41 -6.34,6.34 1.41,1.41L18,7zM22.24,5.59L11.66,16.17 7.48,12l-1.41,1.41L11.66,19l12,-12 -1.42,-1.41zM0.41,13.41L6,19l1.41,-1.41L1.83,12 0.41,13.41z" />
+</vector>
diff --git a/app/src/main/res/drawable/ic_select_none.xml b/app/src/main/res/drawable/ic_select_none.xml
new file mode 100644
index 00000000..f8b9c9d7
--- /dev/null
+++ b/app/src/main/res/drawable/ic_select_none.xml
@@ -0,0 +1,10 @@
+<vector xmlns:android="http://schemas.android.com/apk/res/android"
+    android:width="24dp"
+    android:height="24dp"
+    android:viewportWidth="24"
+    android:viewportHeight="24"
+    android:tint="?attr/colorControlNormal">
+    <path
+        android:fillColor="@android:color/white"
+        android:pathData="M1.79,12l5.58,5.59L5.96,19 0.37,13.41 1.79,12zM2.24,4.22L12.9,14.89l-1.28,1.28L7.44,12l-1.41,1.41L11.62,19l2.69,-2.69 4.89,4.89 1.41,-1.41L3.65,2.81 2.24,4.22zM17.14,13.49L23.62,7 22.2,5.59l-6.48,6.48 1.42,1.42zM17.96,7l-1.41,-1.41 -3.65,3.66 1.41,1.41L17.96,7z" />
+</vector>