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

[stable-3.19] Fix crashes and double loading in GalleryFragment when changing dark theme #9767

Merged
merged 2 commits into from
Jan 27, 2022

Conversation

backportbot-nextcloud[bot]
Copy link

backport of #9743

getStorageManager() is not available in onCreate

Signed-off-by: Álvaro Brey Vilas <alvaro.brey@nextcloud.com>
…imations

This fixes bugs with double-loading and weird animations

Signed-off-by: Álvaro Brey Vilas <alvaro.brey@nextcloud.com>
@github-actions
Copy link

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/9767.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@nextcloud-android-bot
Copy link
Collaborator

Lint

Typestable-3.19PR
Warnings104104
Errors00

SpotBugs (new)

Warning Type Number
Bad practice Warnings 28
Correctness Warnings 82
Experimental Warnings 1
Internationalization Warnings 9
Malicious code vulnerability Warnings 57
Multithreaded correctness Warnings 9
Performance Warnings 68
Security Warnings 29
Dodgy code Warnings 358
Total 641

SpotBugs (stable-3.19)

Warning Type Number
Bad practice Warnings 28
Correctness Warnings 82
Experimental Warnings 1
Internationalization Warnings 9
Malicious code vulnerability Warnings 57
Multithreaded correctness Warnings 9
Performance Warnings 68
Security Warnings 29
Dodgy code Warnings 358
Total 641

@github-actions
Copy link

@codecov
Copy link

codecov bot commented Jan 27, 2022

Codecov Report

Merging #9767 (52e3fae) into stable-3.19 (d59bdc3) will increase coverage by 28.89%.
The diff coverage is 0.00%.

@@                Coverage Diff                 @@
##             stable-3.19    #9767       +/-   ##
==================================================
+ Coverage           2.71%   31.60%   +28.89%     
==================================================
  Files                505      505               
  Lines              38594    38597        +3     
  Branches            5379     5380        +1     
==================================================
+ Hits                1048    12200    +11152     
+ Misses             37496    24563    -12933     
- Partials              50     1834     +1784     
Impacted Files Coverage Δ
...cloud/android/ui/activity/FileDisplayActivity.java 24.14% <0.00%> (+24.14%) ⬆️
.../owncloud/android/ui/fragment/GalleryFragment.java 0.00% <0.00%> (ø)
...ui/fragment/contactsbackup/BackupListFragment.java 0.94% <0.00%> (+0.94%) ⬆️
src/main/java/com/nextcloud/client/core/Task.kt 92.30% <0.00%> (+3.84%) ⬆️
.../owncloud/android/syncadapter/FileSyncAdapter.java 4.25% <0.00%> (+4.25%) ⬆️
...tcloud/client/network/ConnectivityServiceImpl.java 75.55% <0.00%> (+4.44%) ⬆️
...loud/android/datamodel/FilesystemDataProvider.java 4.49% <0.00%> (+4.49%) ⬆️
...roid/ui/adapter/ActivityAndVersionListAdapter.java 5.08% <0.00%> (+5.08%) ⬆️
...ncloud/android/ui/activity/UploadListActivity.java 5.21% <0.00%> (+5.21%) ⬆️
... and 245 more

@github-actions
Copy link

@AlvaroBrey AlvaroBrey merged commit 7dfaf3e into stable-3.19 Jan 27, 2022
@delete-merged-branch delete-merged-branch bot deleted the backport/9743/stable-3.19 branch January 27, 2022 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants