Fix Shortcut handling for API 25 and lower

VladimirWrites · May 30, 2020 · a943905 · a943905
1 parent 96821c8
commit a943905
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 10 deletions.
diff --git a/app/src/main/java/com/vlad1m1r/bltaxi/ApplicationTaxi.kt b/app/src/main/java/com/vlad1m1r/bltaxi/ApplicationTaxi.kt
@@ -2,6 +2,7 @@ package com.vlad1m1r.bltaxi
 
 import android.app.Application
 import android.content.SharedPreferences
+import android.os.Build
 import androidx.appcompat.app.AppCompatDelegate.*
 import com.vlad1m1r.actions.di.actionsModule
 import com.vlad1m1r.basedata.di.baseDataModule
@@ -31,7 +32,7 @@ class ApplicationTaxi : Application() {
             logger(Logger())
             androidContext(this@ApplicationTaxi)
             modules(
-                listOf(
+                mutableListOf(
                     appModule,
                     baseDataModule,
                     localModule,
@@ -42,9 +43,12 @@ class ApplicationTaxi : Application() {
                     aboutModule,
                     analyticsModule,
                     settingsModule,
-                    syncModule,
-                    shortcutsModule
-                )
+                    syncModule
+                ).apply {
+                    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N_MR1) {
+                        add(shortcutsModule)
+                    }
+                }
             )
         }
 

diff --git a/buildSrc/src/main/java/Dependencies.kt b/buildSrc/src/main/java/Dependencies.kt
@@ -49,8 +49,8 @@ object Versions {
     const val compile_sdk = 29
     const val build_tools = "29.0.3"
 
-    const val version_code = 20111
-    const val version_name = "2.1.1"
+    const val version_code = 20121
+    const val version_name = "2.1.2"
 }
 
 object Deps {

diff --git a/shortcuts/src/main/java/com/vlad1m1r/bltaxi/shortcuts/di/ShortcutModul.kt b/shortcuts/src/main/java/com/vlad1m1r/bltaxi/shortcuts/di/ShortcutModul.kt
@@ -1,14 +1,19 @@
 package com.vlad1m1r.bltaxi.shortcuts.di
 
 import android.content.pm.ShortcutManager
+import android.os.Build
+import androidx.annotation.RequiresApi
 import com.vlad1m1r.bltaxi.shortcuts.ShortcutHandler
 import com.vlad1m1r.bltaxi.shortcuts.ShortcutHandlerImpl
 import com.vlad1m1r.bltaxi.shortcuts.ShortcutInfoProvider
 import org.koin.android.ext.koin.androidContext
 import org.koin.dsl.module
 
+@RequiresApi(Build.VERSION_CODES.N_MR1)
 val shortcutsModule = module {
-    single { androidContext().getSystemService(ShortcutManager::class.java)!! }
+    single {
+        androidContext().getSystemService(ShortcutManager::class.java)!!
+    }
     single { ShortcutInfoProvider(androidContext()) }
     single<ShortcutHandler> { ShortcutHandlerImpl(get(), get()) }
 }
diff --git a/taxi/src/main/java/com/vlad1m1r/bltaxi/taxi/TaxiViewModel.kt b/taxi/src/main/java/com/vlad1m1r/bltaxi/taxi/TaxiViewModel.kt
@@ -22,7 +22,7 @@ import kotlinx.coroutines.withContext
 
 class TaxiViewModel(
     private val saveTaxiOrder: SaveTaxiOrder,
-    private val shortcutHandler: ShortcutHandler,
+    private val shortcutHandler: ShortcutHandler?,
     private val getOrderedTaxiList: GetOrderedTaxiList,
     private val executeAction: ExecuteAction,
     private val tracker: Tracker,
@@ -60,7 +60,7 @@ class TaxiViewModel(
     fun setTaxiOrder(viewModelList: List<ItemTaxiViewModel>) {
         val taxis = viewModelList.map { it.itemTaxi }
         if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N_MR1) {
-            shortcutHandler.addShortcutsForTaxis(taxis)
+            shortcutHandler!!.addShortcutsForTaxis(taxis)
         }
         viewModelScope.launch(dispatchers.io) {
             saveTaxiOrder(taxis)

diff --git a/taxi/src/main/java/com/vlad1m1r/bltaxi/taxi/di/TaxiModule.kt b/taxi/src/main/java/com/vlad1m1r/bltaxi/taxi/di/TaxiModule.kt
@@ -13,5 +13,5 @@ val taxiModule = module {
     factory { GetTaxiPosition(get()) }
     factory { GetOrderedTaxiList(get(), get()) }
     factory { ExecuteAction(get()) }
-    viewModel { TaxiViewModel(get(), get(), get(), get(), get(), get()) }
+    viewModel { TaxiViewModel(get(), getOrNull(), get(), get(), get(), get()) }
 }