Skip to content

Conversation

@tobiasKaminsky
Copy link
Member

Found via google play console:

  • have many files downloaded (so account removal takes a while)
  • remove account
  • see app sometimes crashing as storageManager is null

--> we do not need it at all, therefore removing it

Signed-off-by: tobiasKaminsky tobias@kaminsky.me

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@nextcloud-android-bot
Copy link
Collaborator

Lint

96 warnings

FindBugs (new)

Warning TypeNumber
Bad practice Warnings35
Correctness Warnings166
Experimental Warnings4
Internationalization Warnings12
Malicious code vulnerability Warnings9
Multithreaded correctness Warnings9
Performance Warnings158
Security Warnings163
Dodgy code Warnings209
Total765

FindBugs (master)

Warning TypeNumber
Bad practice Warnings35
Correctness Warnings166
Experimental Warnings4
Internationalization Warnings12
Malicious code vulnerability Warnings9
Multithreaded correctness Warnings9
Performance Warnings158
Security Warnings163
Dodgy code Warnings209
Total765

@codecov
Copy link

codecov bot commented Jun 29, 2018

Codecov Report

Merging #2775 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #2775      +/-   ##
=========================================
+ Coverage    6.37%   6.38%   +<.01%     
=========================================
  Files         294     294              
  Lines       29563   29555       -8     
  Branches     4268    4267       -1     
=========================================
+ Hits         1885    1886       +1     
+ Misses      27389   27380       -9     
  Partials      289     289
Impacted Files Coverage Δ
...m/owncloud/android/ui/activity/DrawerActivity.java 0% <ø> (ø) ⬆️
...owncloud/android/ui/activity/UserInfoActivity.java 0% <ø> (ø) ⬆️
...wncloud/android/ui/adapter/AccountListAdapter.java 0% <0%> (ø) ⬆️
...loud/android/datamodel/ThumbnailsCacheManager.java 13.56% <0%> (+0.05%) ⬆️
.../java/com/owncloud/android/utils/DisplayUtils.java 5.72% <0%> (ø) ⬆️
...m/owncloud/android/ui/adapter/UserListAdapter.java 0% <0%> (ø) ⬆️
.../third_parties/daveKoeller/AlphanumComparator.java 82.75% <0%> (+1.14%) ⬆️

@AndyScherzinger
Copy link
Member

AndyScherzinger commented Jun 29, 2018

👍

Approved with PullApprove

AndyScherzinger added a commit that referenced this pull request Jul 4, 2018
Backport of #2775: avatar generation does not need storageManager
@mario
Copy link
Contributor

mario commented Jul 12, 2018

👍

Approved with PullApprove

@AndyScherzinger AndyScherzinger merged commit 9b66189 into master Jul 12, 2018
@AndyScherzinger AndyScherzinger deleted the AvatarGenerationNoStorageManagerNeeded branch July 12, 2018 16:09
@AndyScherzinger AndyScherzinger added this to the Nextcloud App 3.3.0 milestone Jul 12, 2018
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.

5 participants