android icon indicating copy to clipboard operation
android copied to clipboard

After removing last favorite item show correct placeholder

Open tobiasKaminsky opened this issue 3 years ago • 3 comments
trafficstars

Test

  • add a file as favorited
  • go to "favorites"
  • unset this file
  • see correct empty placeholder

Signed-off-by: tobiasKaminsky [email protected]

  • [ ] Tests written, or not not needed

tobiasKaminsky avatar Jul 13 '22 09:07 tobiasKaminsky

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/10519.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.

github-actions[bot] avatar Jul 13 '22 09:07 github-actions[bot]

Codacy

Lint

TypemasterPR
Warnings8585
Errors00

SpotBugs

CategoryBaseNew
Bad practice2929
Correctness4646
Dodgy code353353
Experimental11
Internationalization99
Multithreaded correctness99
Performance5959
Security2828
Total534534

github-actions[bot] avatar Jul 13 '22 09:07 github-actions[bot]

master-IT test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/5494-IT-master-10-13

nextcloud-android-bot avatar Jul 13 '22 10:07 nextcloud-android-bot

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/10519.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.

github-actions[bot] avatar Feb 09 '23 15:02 github-actions[bot]

Codacy

Lint

TypemasterPR
Warnings7676
Errors00

SpotBugs

CategoryBaseNew
Bad practice2727
Correctness4444
Dodgy code334334
Internationalization99
Multithreaded correctness99
Performance5858
Security1818
Total499499

github-actions[bot] avatar Feb 09 '23 16:02 github-actions[bot]

Codecov Report

Merging #10519 (cf28c7e) into master (6178a14) will increase coverage by 0.20%. The diff coverage is 0.00%.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #10519      +/-   ##
============================================
+ Coverage     31.36%   31.57%   +0.20%     
- Complexity     3408     3431      +23     
============================================
  Files           576      576              
  Lines         42121    42122       +1     
  Branches       5684     5684              
============================================
+ Hits          13212    13300      +88     
+ Misses        26938    26858      -80     
+ Partials       1971     1964       -7     
Impacted Files Coverage Δ
...ncloud/android/ui/fragment/OCFileListFragment.java 39.82% <0.00%> (-0.05%) :arrow_down:
.../owncloud/android/datamodel/MediaFoldersModel.java 40.00% <0.00%> (-20.00%) :arrow_down:
...ncloud/android/ui/activity/FolderPickerActivity.kt 29.93% <0.00%> (-7.83%) :arrow_down:
...nextcloud/client/jobs/MediaFoldersDetectionWork.kt 22.87% <0.00%> (-5.23%) :arrow_down:
.../third_parties/daveKoeller/AlphanumComparator.java 86.90% <0.00%> (-1.20%) :arrow_down:
...loud/android/ui/fragment/ExtendedListFragment.java 53.35% <0.00%> (-1.07%) :arrow_down:
...oud/android/operations/RefreshFolderOperation.java 77.38% <0.00%> (-1.07%) :arrow_down:
...xtcloud/client/account/UserAccountManagerImpl.java 69.33% <0.00%> (-0.67%) :arrow_down:
...cloud/android/ui/activity/SyncedFoldersActivity.kt 20.53% <0.00%> (-0.45%) :arrow_down:
...cloud/android/ui/activity/FileDisplayActivity.java 28.99% <0.00%> (-0.09%) :arrow_down:
... and 11 more

codecov[bot] avatar Feb 09 '23 19:02 codecov[bot]