Merge branch 'master' into v2.4

mobile/build.gradle

@@ -42,6 +42,7 @@ android {
     }
     dataBinding.enabled = true
     flavorDimensions("freedom")
+    packagingOptions.exclude 'META-INF/*.kotlin_module'
     productFlavors {
         fdroid {
             dimension "freedom"
@@ -82,14 +83,14 @@ dependencies {
     implementation "androidx.room:room-ktx:$roomVersion"
     implementation 'com.android.billingclient:billing:1.2.2'
     implementation 'com.github.luongvo:BadgeView:1.1.5'
-    implementation 'com.github.topjohnwu.libsu:core:2.3.3'
+    implementation 'com.github.topjohnwu.libsu:core:2.4.0'
     implementation 'com.google.android.material:material:1.0.0'
     implementation 'com.jakewharton.timber:timber:4.7.1'
     implementation 'com.linkedin.dexmaker:dexmaker:2.25.0'
     implementation 'com.takisoft.preferencex:preferencex-simplemenu:1.0.0'
     implementation 'net.glxn.qrgen:android:2.0'
     implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlinVersion"
-    implementation 'org.jetbrains.kotlinx:kotlinx-coroutines-android:1.1.1'
+    implementation 'org.jetbrains.kotlinx:kotlinx-coroutines-android:1.2.0'
     for (dep in aux) {
         freedomImplementation dep
         googleImplementation dep

mobile/lint.xml

@@ -3,5 +3,5 @@
     <issue id="InvalidPackage">
         <ignore regexp="org.mockito.*" />
     </issue>
-    <issue id="MissingTranslation" severity="informational" />
+    <issue id="MissingTranslation" severity="ignore" />
 </lint>

mobile/src/main/java/be/mygod/vpnhotspot/client/ClientsFragment.kt

@@ -36,18 +36,15 @@ import be.mygod.vpnhotspot.room.AppDatabase
 import be.mygod.vpnhotspot.room.ClientStats
 import be.mygod.vpnhotspot.room.TrafficRecord
 import be.mygod.vpnhotspot.room.macToString
-import be.mygod.vpnhotspot.util.MainScope
 import be.mygod.vpnhotspot.util.SpanFormatter
 import be.mygod.vpnhotspot.util.computeIfAbsentCompat
 import be.mygod.vpnhotspot.util.toPluralInt
 import be.mygod.vpnhotspot.widget.SmartSnackbar
 import kotlinx.android.parcel.Parcelize
-import kotlinx.coroutines.Dispatchers
-import kotlinx.coroutines.GlobalScope
-import kotlinx.coroutines.launch
+import kotlinx.coroutines.*
 import java.text.NumberFormat
 
-class ClientsFragment : Fragment(), MainScope by MainScope.Supervisor() {
+class ClientsFragment : Fragment() {
     @Parcelize
     data class NicknameArg(val mac: Long, val nickname: CharSequence) : Parcelable
     class NicknameDialogFragment : AlertDialogFragment<NicknameArg, Empty>() {
@@ -140,7 +137,9 @@ class ClientsFragment : Fragment(), MainScope by MainScope.Supervisor() {
                     val wasWorking = TrafficRecorder.isWorking(client.mac)
                     client.obtainRecord().apply {
                         blocked = !blocked
-                        launch(Dispatchers.Unconfined) { AppDatabase.instance.clientRecordDao.update(this@apply) }
+                        GlobalScope.launch(Dispatchers.Unconfined) {
+                            AppDatabase.instance.clientRecordDao.update(this@apply)
+                        }
                     }
                     IpNeighbourMonitor.instance?.flush()
                     if (!wasWorking && item.itemId == R.id.block) {
@@ -150,7 +149,7 @@ class ClientsFragment : Fragment(), MainScope by MainScope.Supervisor() {
                 }
                 R.id.stats -> {
                     binding.client?.let { client ->
-                        launch(Dispatchers.Unconfined) {
+                        scope.launch {
                             StatsDialogFragment().withArg(StatsArg(
                                     client.title.value ?: return@launch,
                                     AppDatabase.instance.trafficRecordDao.queryStats(client.mac)
@@ -205,6 +204,7 @@ class ClientsFragment : Fragment(), MainScope by MainScope.Supervisor() {
         }
     }
 
+    private val scope = MainScope() + Dispatchers.Unconfined
     private lateinit var binding: FragmentClientsBinding
     private val adapter = ClientAdapter()
     private var rates = HashMap<Pair<String, Long>, TrafficRate>()
@@ -237,7 +237,7 @@ class ClientsFragment : Fragment(), MainScope by MainScope.Supervisor() {
     }
 
     override fun onDestroy() {
-        job.cancel()
+        scope.cancel()
         super.onDestroy()
     }
 }

mobile/src/main/java/be/mygod/vpnhotspot/manage/TetherManager.kt

@@ -163,7 +163,7 @@ sealed class TetherManager(protected val parent: TetheringFragment) : Manager(),
         override val type get() = VIEW_TYPE_BLUETOOTH
         override val isStarted get() = tethering.active == true
 
-        override fun onException() = ManageBar.start(parent.requireContext())
+        override fun onException() = ManageBar.start(parent.context ?: app)
 
         override fun start() = BluetoothTethering.start(this)
         override fun stop() {

mobile/src/main/java/be/mygod/vpnhotspot/manage/TetheringFragment.kt

@@ -103,7 +103,7 @@ class TetheringFragment : Fragment(), ServiceConnection, Toolbar.OnMenuItemClick
     }
 
     private fun updateMonitorList(canMonitor: List<String> = emptyList()) {
-        val item = requireActivity().toolbar.menu.findItem(R.id.monitor) ?: return  // assuming no longer foreground
+        val item = activity?.toolbar?.menu?.findItem(R.id.monitor) ?: return    // assuming no longer foreground
         item.isNotGone = canMonitor.isNotEmpty()
         item.subMenu.apply {
             clear()

mobile/src/main/java/be/mygod/vpnhotspot/util/MainScope.kt

@@ -1,14 +0,0 @@
-package be.mygod.vpnhotspot.util
-
-import kotlinx.coroutines.CoroutineScope
-import kotlinx.coroutines.Dispatchers
-import kotlinx.coroutines.Job
-import kotlinx.coroutines.SupervisorJob
-
-interface MainScope : CoroutineScope {
-    class Supervisor : MainScope {
-        override val job = SupervisorJob()
-    }
-    val job: Job
-    override val coroutineContext get() = Dispatchers.Main + job
-}

mobile/src/main/res/values-ru/strings.xml

@@ -6,11 +6,15 @@
     -->
 
     <string name="repeater_configure_failure">Действительный конфиг не найден. Пожалуйста, сначала запустите ретранслятор.</string>
+    <string name="repeater_reset_credentials">Сброс</string>
+    <string name="repeater_reset_credentials_success">Сброс учетных данных.</string>
+    <string name="repeater_reset_credentials_failure">Не удалось сбросить учетные данные (причина:%s)</string>
     <string name="repeater_clean_pog_failure">Не удалось удалить избыточную группу P2P (причина: %s)</string>
 
     <string name="repeater_p2p_unavailable">Wi-Fi директ недоступен, пожалуйста включите Wi-Fi</string>
     <string name="repeater_create_group_failure">Не удалось создать группу P2P (причина: %s)</string>
     <string name="repeater_remove_group_failure">Не удалось удалить группу P2P (причина: %s)</string>
+    <string name="repeater_remove_old_group_failure">Не удалось удалить старую группу P2P (причина:%s)</string>
 
     <string name="repeater_failure_reason_error">внутренняя ошибка</string>
     <string name="repeater_failure_reason_p2p_unsupported">Wi-Fi напрямую не поддерживается</string>
@@ -31,12 +35,15 @@
     <string name="clients_popup_block_service_inactive">Включите службу для этого интерфейса, чтобы заблокировать клиента.</string>
     <string name="clients_popup_unblock">Разблокировать</string>
     <string name="clients_popup_stats">Статистика…</string>
+    <string name="clients_mac_lookup_unexpected_error">Сервер вернул ошибку для %1$s: %2$s</string>
     <string name="clients_nickname_title">Никнэйм  для %s</string>
+    <string name="clients_nickname_set_to_vendor">Имя вендора</string>
     <string name="clients_stats_title">Статистика для %s</string>
 
 
     <string name="notification_tethering_title">VPN-модем активен</string>
     <string name="notification_channel_tethering">VPN модемная служба</string>
+    <string name="notification_interfaces_inactive">Неактивных: %s</string>
 
     <string name="failure_reason_unknown">Неизвестно #%d</string>
     <string name="exception_interface_not_found">Ошибка: Нисходящий интерфейс не найден</string>
@@ -55,7 +62,7 @@
 
     <!-- Based on: https://github.com/PrivacyApps/donations/blob/747d36a18433c7e9329691054122a8ad337a62d2/Donations/src/main/res/values-ru/donations__strings.xml -->
     <string name="donations__button_close">Закрыть</string>
-    <string name="donations__description">Считаете это приложение полезным?\\nПоддержите его разработку, отправив пожертвование разработчику!</string>
+    <string name="donations__description">Считаете это приложение полезным?\nПоддержите его разработку, отправив пожертвование разработчику!</string>
     <string name="donations__google_android_market">Google Play Store</string>
     <string name="donations__google_android_market_not_supported_title">In-App пожертвования не поддерживаются.</string>
     <string name="donations__google_android_market_not_supported">Пожертвования через приложение не поддерживаются. Google Play Store установлен правильно?</string>