Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open a remote folder that does not exist in realm #829

Merged
merged 2 commits into from
Jun 30, 2022

Conversation

sirambd
Copy link
Member

@sirambd sirambd commented Jun 29, 2022

Fix Sentry - Null exception when open a remote file that doesn't exists in realm

java.lang.NullPointerException: null
    at com.infomaniak.drive.data.cache.FileController.downloadAndSaveFiles(FileController.kt:674)
    at com.infomaniak.drive.data.cache.FileController.access$downloadAndSaveFiles(FileController.kt:44)
    at com.infomaniak.drive.data.cache.FileController$getFilesFromCacheOrDownload$operation$1.invoke(FileController.kt:628)
    at com.infomaniak.drive.data.cache.FileController$getFilesFromCacheOrDownload$operation$1.invoke(FileController.kt:611)
    at com.infomaniak.drive.data.cache.FileController.getFilesFromCacheOrDownload(FileController.kt:642)
    at com.infomaniak.drive.data.cache.FileController.getFilesFromCacheOrDownload$default(FileController.kt:596)
    at com.infomaniak.drive.ui.fileList.FileListViewModel$getFiles$1.invokeSuspend$recursiveDownload(FileListViewModel.kt:67)
    at com.infomaniak.drive.ui.fileList.FileListViewModel$getFiles$1.invokeSuspend(FileListViewModel.kt:90)
    at com.infomaniak.drive.ui.fileList.FileListViewModel$getFiles$1.invoke
    at com.infomaniak.drive.ui.fileList.FileListViewModel$getFiles$1.invoke
    at androidx.lifecycle.BlockRunner$maybeRun$1.invokeSuspend(CoroutineLiveData.kt:176)
    at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
    at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
    at kotlinx.coroutines.internal.LimitedDispatcher.run(LimitedDispatcher.kt:42)
    at kotlinx.coroutines.scheduling.TaskImpl.run(Tasks.kt:95)
    at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:570)
    at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:749)
    at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:677)
    at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:664)

@sirambd sirambd added the bug Something isn't working label Jun 29, 2022
@sirambd sirambd added this to the 4.2 milestone Jun 29, 2022
@sirambd sirambd self-assigned this Jun 29, 2022
@sirambd sirambd requested a review from a team as a code owner June 29, 2022 16:15
@sirambd sirambd added the api-v2 Related to API v2 migration label Jun 29, 2022
@sonarcloud
Copy link

sonarcloud bot commented Jun 30, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@sirambd sirambd merged commit 995318a into master Jun 30, 2022
@sirambd sirambd deleted the fix-open-remote-folder branch June 30, 2022 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api-v2 Related to API v2 migration bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants