github/szkolny
1
0
Fork 1
mirror of https://github.com/szkolny-eu/szkolny-android.git synced 2025-04-02 06:44:27 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity

[UI/Login] Fix ignoring API signature errors in login chooser.

Browse Source
This commit is contained in:
Kuba Szczodrzyński 2021-03-29 09:41:50 +02:00
parent 17351066d1
commit 2808c66840
2 changed files with 28 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/src/main/java/pl/szczodrzynski/edziennik/MainActivity.kt
View File

@ -650,7 +650,7 @@ class MainActivity : AppCompatActivity(), CoroutineScope {
return@let return@let
} }
is Throwable -> { is Throwable -> {
errorSnackbar.addError(result.toApiError(SzkolnyApi.TAG)).show() errorSnackbar.addError(result.toApiError(TAG)).show()
return return
} }
is RegisterAvailabilityStatus -> { is RegisterAvailabilityStatus -> {

34
app/src/main/java/pl/szczodrzynski/edziennik/ui/modules/login/LoginChooserFragment.kt
View File

@ -16,15 +16,16 @@ import android.view.ViewGroup
import android.view.animation.AccelerateDecelerateInterpolator import android.view.animation.AccelerateDecelerateInterpolator
import android.view.animation.Animation import android.view.animation.Animation
import android.view.animation.RotateAnimation import android.view.animation.RotateAnimation
import android.widget.Toast
import androidx.core.view.isVisible import androidx.core.view.isVisible
import androidx.fragment.app.Fragment import androidx.fragment.app.Fragment
import androidx.recyclerview.widget.LinearLayoutManager import androidx.recyclerview.widget.LinearLayoutManager
import com.google.android.material.dialog.MaterialAlertDialogBuilder import com.google.android.material.dialog.MaterialAlertDialogBuilder
import com.google.firebase.analytics.FirebaseAnalytics
import kotlinx.coroutines.* import kotlinx.coroutines.*
import pl.szczodrzynski.edziennik.* import pl.szczodrzynski.edziennik.*
import pl.szczodrzynski.edziennik.data.api.* import pl.szczodrzynski.edziennik.data.api.*
import pl.szczodrzynski.edziennik.data.api.szkolny.SzkolnyApi import pl.szczodrzynski.edziennik.data.api.szkolny.SzkolnyApi
import pl.szczodrzynski.edziennik.data.api.szkolny.response.RegisterAvailabilityStatus
import pl.szczodrzynski.edziennik.databinding.LoginChooserFragmentBinding import pl.szczodrzynski.edziennik.databinding.LoginChooserFragmentBinding
import pl.szczodrzynski.edziennik.ui.dialogs.RegisterUnavailableDialog import pl.szczodrzynski.edziennik.ui.dialogs.RegisterUnavailableDialog
import pl.szczodrzynski.edziennik.ui.modules.feedback.FeedbackActivity import pl.szczodrzynski.edziennik.ui.modules.feedback.FeedbackActivity
@ -249,19 +250,38 @@ class LoginChooserFragment : Fragment(), CoroutineScope {
}?.let { registerName -> }?.let { registerName ->
var status = app.config.sync.registerAvailability[registerName] var status = app.config.sync.registerAvailability[registerName]
if (status == null || status.nextCheckAt < currentTimeUnix()) { if (status == null || status.nextCheckAt < currentTimeUnix()) {
withContext(Dispatchers.IO) { val api = SzkolnyApi(app)
val api = SzkolnyApi(app) val result = withContext(Dispatchers.IO) {
api.runCatching(activity) { return@withContext api.runCatching({
val availability = getRegisterAvailability() val availability = getRegisterAvailability()
app.config.sync.registerAvailability = availability app.config.sync.registerAvailability = availability
status = availability[registerName] availability[registerName]
}, onError = {
if (it.toErrorCode() == ERROR_API_INVALID_SIGNATURE) {
return@withContext false
}
return@withContext it
})
}
when (result) {
false -> {
Toast.makeText(activity, R.string.error_no_api_access, Toast.LENGTH_SHORT).show()
return@let
}
is Throwable -> {
activity.errorSnackbar.addError(result.toApiError(TAG)).show()
return false
}
is RegisterAvailabilityStatus -> {
status = result
} }
} }
} }
if (status?.available != true) { if (status?.available != true || status.minVersionCode > BuildConfig.VERSION_CODE) {
if (status != null) if (status != null)
RegisterUnavailableDialog(activity, status!!) RegisterUnavailableDialog(activity, status)
return false return false
} }
} }