third-party-mirrors/ultrasonic
1
0
Fork 0
mirror of https://gitlab.com/ultrasonic/ultrasonic.git synced 2025-04-14 00:17:15 +03:00
Code Issues Packages Projects Releases Wiki Activity

Merge branch 'root' into 'develop'

Browse Source 
Avoid rare NPE

See merge request ultrasonic/ultrasonic!1073

(cherry picked from commit 7209779b64820572910059fdd24342bb73734b54)

ecee57e1 Avoid rare NPE
This commit is contained in:
birdbird 2023-07-06 20:49:29 +00:00
parent 698360b77a
commit 21e4293adb
3 changed files with 24 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ultrasonic/src/main/kotlin/org/moire/ultrasonic/activity/NavigationActivity.kt
View File

@ -306,7 +306,7 @@ class NavigationActivity : AppCompatActivity() {
Storage.reset()
lifecycleScope.launch(Dispatchers.IO) {
Storage.ensureRootIsAvailable()
Storage.checkForErrorsWithCustomRoot()
}
setMenuForServerCapabilities()

2
ultrasonic/src/main/kotlin/org/moire/ultrasonic/fragment/SettingsFragment.kt
View File

@ -344,7 +344,7 @@ class SettingsFragment :
// Clear download queue.
mediaPlayerManager.clear()
Storage.reset()
Storage.ensureRootIsAvailable()
Storage.checkForErrorsWithCustomRoot()
}
private fun setDebugLogToFile(writeLog: Boolean) {

33
ultrasonic/src/main/kotlin/org/moire/ultrasonic/util/Storage.kt
View File

@ -22,19 +22,23 @@ import timber.log.Timber
object Storage {
val mediaRoot: ResettableLazy<AbstractFile> = ResettableLazy {
getRoot()!!
val ret = getRoot()
rootNotFoundError = ret.second
ret.first
}
var rootNotFoundError: Boolean = false
fun reset() {
StorageFile.storageFilePathDictionary.clear()
StorageFile.notExistingPathDictionary.clear()
mediaRoot.reset()
rootNotFoundError = false
Timber.i("StorageFile caches were reset")
}
fun ensureRootIsAvailable() {
val root = getRoot()
if (root == null) {
fun checkForErrorsWithCustomRoot() {
if (rootNotFoundError) {
Settings.customCacheLocation = false
Settings.cacheLocationUri = ""
Util.toast(UApp.applicationContext(), R.string.settings_cache_location_error)
@ -98,18 +102,25 @@ object Storage {
return success
}
private fun getRoot(): AbstractFile? {
private fun getRoot(): Pair<AbstractFile, Boolean> {
return if (Settings.customCacheLocation) {
if (Settings.cacheLocationUri.isBlank()) return null
if (Settings.cacheLocationUri.isBlank()) return Pair(getDefaultRoot(), true)
val documentFile = DocumentFile.fromTreeUri(
UApp.applicationContext(),
Uri.parse(Settings.cacheLocationUri)
) ?: return null
if (!documentFile.exists()) return null
StorageFile(null, documentFile.uri, documentFile.name!!, documentFile.isDirectory)
) ?: return Pair(getDefaultRoot(), true)
if (!documentFile.exists()) return Pair(getDefaultRoot(), true)
Pair(
StorageFile(null, documentFile.uri, documentFile.name!!, documentFile.isDirectory),
false
)
} else {
val file = File(FileUtil.defaultMusicDirectory.path)
JavaFile(null, file)
Pair(getDefaultRoot(), false)
}
}
private fun getDefaultRoot(): JavaFile {
val file = File(FileUtil.defaultMusicDirectory.path)
return JavaFile(null, file)
}
}