Skip to content

Conversation

@ezaquarii
Copy link
Collaborator

Signed-off-by: Chris Narkiewicz hello@ezaquarii.com

@codecov
Copy link

codecov bot commented Feb 9, 2020

Codecov Report

Merging #5432 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #5432      +/-   ##
============================================
- Coverage     19.88%   19.84%   -0.04%     
  Complexity        3        3              
============================================
  Files           394      394              
  Lines         33210    33211       +1     
  Branches       4666     4667       +1     
============================================
- Hits           6603     6591      -12     
- Misses        25524    25533       +9     
- Partials       1083     1087       +4     
Impacted Files Coverage Δ Complexity Δ
...m/nextcloud/client/account/UserAccountManager.java 0.00% <0.00%> (-33.34%) 0.00% <0.00%> (ø%)
...d/android/operations/GetCapabilitiesOperation.java 75.00% <0.00%> (-12.50%) 0.00% <0.00%> (ø%)
...ain/java/com/owncloud/android/ui/TextDrawable.java 74.07% <0.00%> (-7.41%) 0.00% <0.00%> (ø%)
...loud/android/datamodel/ThumbnailsCacheManager.java 32.12% <0.00%> (-1.09%) 0.00% <0.00%> (ø%)
...oud/android/operations/RefreshFolderOperation.java 58.94% <0.00%> (-0.41%) 0.00% <0.00%> (ø%)
...in/java/com/owncloud/android/utils/ThemeUtils.java 52.69% <0.00%> (-0.39%) 0.00% <0.00%> (ø%)
...owncloud/android/ui/adapter/OCFileListAdapter.java 28.08% <0.00%> (-0.18%) 0.00% <0.00%> (ø%)
.../third_parties/daveKoeller/AlphanumComparator.java 82.14% <0.00%> (ø) 0.00% <0.00%> (ø%) ⬆️

Signed-off-by: Chris Narkiewicz <hello@ezaquarii.com>
@ezaquarii ezaquarii force-pushed the ezaquarii/migrate-drawer-menu-utils-to-new-user-model branch from 7d3d613 to b5fb9c1 Compare February 9, 2020 10:11
@nextcloud-android-bot
Copy link
Collaborator

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

Codacy

343

Lint

TypemasterPR
Warnings7777
Errors00

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings71
Internationalization Warnings13
Malicious code vulnerability Warnings5
Multithreaded correctness Warnings9
Performance Warnings79
Security Warnings45
Dodgy code Warnings138
Total386

SpotBugs (master)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings71
Internationalization Warnings13
Malicious code vulnerability Warnings5
Multithreaded correctness Warnings9
Performance Warnings79
Security Warnings45
Dodgy code Warnings138
Total386

@tobiasKaminsky tobiasKaminsky merged commit 5905b1c into master Feb 10, 2020
@delete-merged-branch delete-merged-branch bot deleted the ezaquarii/migrate-drawer-menu-utils-to-new-user-model branch February 10, 2020 11:41
@AndyScherzinger AndyScherzinger added this to the Nextcloud App 3.11.0 milestone Feb 10, 2020
tobiasKaminsky added a commit that referenced this pull request Feb 11, 2020
92ddf10 Merge remote-tracking branch 'origin/master' into dev
5905b1c Merge pull request #5432 from nextcloud/ezaquarii/migrate-drawer-menu-utils-to-new-user-model
4be18ad Merge pull request #5064 from nextcloud/ezaquarii/extract-account-logic-from-base-activity-into-mixin
619b5d0 Merge pull request #5419 from nextcloud/dependabot/gradle/org.projectlombok-lombok-1.18.12
2d6e73d Merge pull request #5414 from nextcloud/fixContactCrash
1bb04aa also increase annotationProcessor
6048243 Fix crash: Caused by: java.lang.ClassCastException: java.lang.Object[] cannot be cast to java.lang.Void[]
0027df9 [tx-robot] updated from transifex
6dd5d92 Merge pull request #5443 from nextcloud/fixTransifex
b4db9d0 do not break line on transifex
b5fb9c1 Migrate drawer menu utilities to new user model
a906a14 daily dev 20200208
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants