Fixed Checking permission despite requiring no permission
This commit is contained in:
@@ -18,7 +18,6 @@
|
||||
|
||||
package xyz.quaver.pupil.ui
|
||||
|
||||
import android.Manifest
|
||||
import android.content.Intent
|
||||
import android.graphics.drawable.Animatable
|
||||
import android.graphics.drawable.Drawable
|
||||
@@ -50,7 +49,6 @@ import xyz.quaver.pupil.adapters.ReaderAdapter
|
||||
import xyz.quaver.pupil.util.GalleryDownloader
|
||||
import xyz.quaver.pupil.util.Histories
|
||||
import xyz.quaver.pupil.util.ItemClickSupport
|
||||
import xyz.quaver.pupil.util.hasPermission
|
||||
|
||||
class ReaderActivity : AppCompatActivity() {
|
||||
|
||||
@@ -373,17 +371,6 @@ class ReaderActivity : AppCompatActivity() {
|
||||
with(reader_fab_download) {
|
||||
setImageResource(R.drawable.ic_download)
|
||||
setOnClickListener {
|
||||
|
||||
if (!this@ReaderActivity.hasPermission(Manifest.permission.WRITE_EXTERNAL_STORAGE)) {
|
||||
AlertDialog.Builder(this@ReaderActivity).apply {
|
||||
setTitle(R.string.warning)
|
||||
setMessage(R.string.update_no_permission)
|
||||
setPositiveButton(android.R.string.ok) { _, _ -> }
|
||||
}.show()
|
||||
|
||||
return@setOnClickListener
|
||||
}
|
||||
|
||||
downloader.download = !downloader.download
|
||||
|
||||
if (!downloader.download)
|
||||
|
||||
@@ -18,24 +18,16 @@
|
||||
|
||||
package xyz.quaver.pupil.util
|
||||
|
||||
import android.content.Context
|
||||
import android.content.pm.PackageManager
|
||||
import android.os.Build
|
||||
import androidx.core.content.ContextCompat
|
||||
import java.util.*
|
||||
import kotlin.collections.ArrayList
|
||||
import kotlin.math.round
|
||||
|
||||
//Android Q+ uses scoped storage thus not requiring permission
|
||||
fun Context.hasPermission(permission: String) =
|
||||
Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q || ContextCompat.checkSelfPermission(this, permission) == PackageManager.PERMISSION_GRANTED
|
||||
|
||||
@UseExperimental(ExperimentalStdlibApi::class)
|
||||
fun String.wordCapitalize() : String {
|
||||
val result = ArrayList<String>()
|
||||
|
||||
for (word in this.split(" "))
|
||||
result.add(word.capitalize(Locale.getDefault()))
|
||||
result.add(word.capitalize(Locale.US))
|
||||
|
||||
return result.joinToString(" ")
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user