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

Remove onAccountSet from ReceiveExternalFilesActivity #4700

Merged

Conversation

ezaquarii
Copy link
Collaborator

  1. onAccountSet logic inlined in onStart and changeAccount
  2. call to onAccountSet eliminated

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

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

@Override
protected void onAccountSet(boolean stateWasRecovered) {
super.onAccountSet(mAccountWasRestored);
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This flag from BaseActivity is not modified in the subclass so it's safe to remove it.

@nextcloud-android-bot
Copy link
Collaborator

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

299

Lint

TypemasterPR
Warnings5959
Errors00

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings69
Internationalization Warnings12
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings9
Performance Warnings120
Security Warnings44
Dodgy code Warnings138
Total422

SpotBugs (master)

Warning TypeNumber
Bad practice Warnings26
Correctness Warnings69
Internationalization Warnings12
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings9
Performance Warnings120
Security Warnings44
Dodgy code Warnings138
Total422

@codecov
Copy link

codecov bot commented Oct 18, 2019

Codecov Report

Merging #4700 into master will increase coverage by 0.13%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #4700      +/-   ##
============================================
+ Coverage     17.51%   17.64%   +0.13%     
  Complexity        3        3              
============================================
  Files           374      374              
  Lines         32101    32102       +1     
  Branches       4541     4541              
============================================
+ Hits           5621     5663      +42     
+ Misses        25569    25530      -39     
+ Partials        911      909       -2
Impacted Files Coverage Δ Complexity Δ
...roid/ui/activity/ReceiveExternalFilesActivity.java 0% <0%> (ø) 0 <0> (ø) ⬇️
...oud/android/ui/activity/SyncedFoldersActivity.java 22.79% <0%> (ø) 0% <0%> (ø) ⬇️
...cloud/android/ui/activity/FileDisplayActivity.java 20.25% <0%> (+0.24%) 0% <0%> (ø) ⬇️
...ncloud/android/ui/fragment/OCFileListFragment.java 25.57% <0%> (+0.27%) 0% <0%> (ø) ⬇️
...xtcloud/client/preferences/AppPreferencesImpl.java 54.03% <0%> (+0.62%) 0% <0%> (ø) ⬇️
...loud/android/datamodel/ThumbnailsCacheManager.java 36.31% <0%> (+0.72%) 0% <0%> (ø) ⬇️
.../java/com/owncloud/android/utils/DisplayUtils.java 25.73% <0%> (+0.84%) 0% <0%> (ø) ⬇️
.../third_parties/daveKoeller/AlphanumComparator.java 83.33% <0%> (+1.19%) 0% <0%> (ø) ⬇️
...owncloud/android/ui/adapter/OCFileListAdapter.java 30.72% <0%> (+1.71%) 0% <0%> (ø) ⬇️
...om/owncloud/android/utils/FileSortOrderByName.java 17.14% <0%> (+2.85%) 0% <0%> (ø) ⬇️
... and 5 more

@tobiasKaminsky tobiasKaminsky merged commit 2a8164a into master Oct 18, 2019
@delete-merged-branch delete-merged-branch bot deleted the account-cleanup-receive-external-file-activity branch October 18, 2019 08:17
@AndyScherzinger AndyScherzinger added this to the Nextcloud App 3.9.0 milestone Oct 18, 2019
tobiasKaminsky added a commit that referenced this pull request Oct 19, 2019
954c76b Merge pull request #4692 from nextcloud/markdownPreview
2a8164a Merge pull request #4700 from nextcloud/account-cleanup-receive-external-file-activity
76622c8 [tx-robot] updated from transifex
8fa348f Bump findsecbugs-plugin from 1.9.0 to 1.10.0 (#4703)
e31ff99 daily dev 20191018
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.

None yet

4 participants