android
android copied to clipboard
SharedListFragment: replace file in adapter when fetching the complete data
This fixes bugs with selection
- [x] Tests written, or not not needed
Codecov Report
Merging #10621 (71fb20b) into master (71fb20b) will not change coverage. The diff coverage is
n/a.
:exclamation: Current head 71fb20b differs from pull request most recent head 1a4aaf5. Consider uploading reports for the commit 1a4aaf5 to get more accurate results
@@ Coverage Diff @@
## master #10621 +/- ##
=========================================
Coverage 31.48% 31.48%
Complexity 3307 3307
=========================================
Files 540 540
Lines 40221 40221
Branches 5561 5561
=========================================
Hits 12663 12663
Misses 25682 25682
Partials 1876 1876
Codacy
Lint
| Type | master | PR |
| Warnings | 85 | 85 |
| Errors | 0 | 0 |
SpotBugs
| Category | Base | New |
|---|---|---|
| Bad practice | 29 | 29 |
| Correctness | 46 | 46 |
| Dodgy code | 353 | 353 |
| Experimental | 1 | 1 |
| Internationalization | 9 | 9 |
| Multithreaded correctness | 9 | 9 |
| Performance | 58 | 58 |
| Security | 28 | 28 |
| Total | 533 | 533 |
APK file: https://www.kaminsky.me/nc-dev/android-artifacts/10621.apk
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.