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.

Sign up for GitHub

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

Jump to bottom

some cleanups #3918

Merged
merged 8 commits into from
Apr 18, 2019
Merged

some cleanups #3918

merged 8 commits into from
Apr 18, 2019

Conversation

tobiasKaminsky
Copy link
Member

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

@tobiasKaminsky
direct usage of library project
52020a9 
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@tobiasKaminsky
some cleanups
334155e 
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@AndyScherzinger
Merge pull request #3912 from nextcloud/libraryUsage
caa2a91 
Setup.md: direct usage of library project
jmue and others added 5 commits April 17, 2019 21:16
@jmue
use constant
e6604c8 
Signed-off-by: Jens Mueller <tschenser@gmx.de>
@jmue
use constant
c7c6a98 
Signed-off-by: Jens Mueller <tschenser@gmx.de>
@tobiasKaminsky
Merge pull request #3915 from jmue/cleanup/ocfile
b22f517 
use path separator constant
Merge commit '334155e707975e740a0a13b33d9b7bbedf40e9bc'
da3c497
@nextcloud-android-bot
Drone: update FindBugs results to reflect reduced error/warning count…
091c003 
… [skip ci]

Signed-off-by: nextcloud-android-bot <android@nextcloud.com>
@codecov
Copy link

codecov bot commented Apr 18, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@28f80cd). Click here to learn what that means.
The diff coverage is 100%.

@@           Coverage Diff            @@
##             master   #3918   +/-   ##
========================================
  Coverage          ?   6.58%           
  Complexity        ?       1           
========================================
  Files             ?     320           
  Lines             ?   30807           
  Branches          ?    4393           
========================================
  Hits              ?    2030           
  Misses            ?   28481           
  Partials          ?     296
Impacted Files Coverage Δ Complexity Δ
...wncloud/android/utils/glide/HttpStreamFetcher.java 0% <ø> (ø) 0 <0> (?)
...java/com/owncloud/android/jobs/OfflineSyncJob.java 0% <ø> (ø) 0 <0> (?)
...cloud/android/datamodel/UploadsStorageManager.java 20.16% <ø> (ø) 0 <0> (?)
...com/owncloud/android/ui/activity/FileActivity.java 0% <ø> (ø) 0 <0> (?)
.../android/authentication/AuthenticatorActivity.java 1.89% <ø> (ø) 0 <0> (?)
...cloud/android/ui/preview/PreviewImageActivity.java 0% <ø> (ø) 0 <0> (?)
...droid/providers/DiskLruImageCacheFileProvider.java 9.3% <ø> (ø) 0 <0> (?)
...ndroid/providers/UsersAndGroupsSearchProvider.java 18.48% <ø> (ø) 0 <0> (?)
.../owncloud/android/authentication/AccountUtils.java 45.16% <ø> (ø) 0 <0> (?)
...ain/java/com/owncloud/android/ui/TextDrawable.java 0% <ø> (ø) 0 <0> (?)
... and 7 more

@codecov
Copy link

codecov bot commented Apr 18, 2019 •
edited
Loading

Codecov Report

❗ No coverage uploaded for pull request base (master@28f80cd). Click here to learn what that means.
The diff coverage is 10.81%.

@@           Coverage Diff            @@
##             master   #3918   +/-   ##
========================================
  Coverage          ?   6.54%           
  Complexity        ?       1           
========================================
  Files             ?     320           
  Lines             ?   30807           
  Branches          ?    4393           
========================================
  Hits              ?    2015           
  Misses            ?   28497           
  Partials          ?     295
Impacted Files Coverage Δ Complexity Δ
...cloud/android/datamodel/UploadsStorageManager.java 20.16% <ø> (ø) 0 <0> (?)
.../owncloud/android/authentication/AccountUtils.java 45.16% <ø> (ø) 0 <0> (?)
...owncloud/android/ui/activity/UserInfoActivity.java 0% <ø> (ø) 0 <0> (?)
...ndroid/providers/UsersAndGroupsSearchProvider.java 18.48% <ø> (ø) 0 <0> (?)
...ain/java/com/owncloud/android/ui/TextDrawable.java 0% <ø> (ø) 0 <0> (?)
src/main/java/com/owncloud/android/MainApp.java 45.7% <ø> (ø) 0 <0> (?)
...wncloud/android/utils/glide/HttpStreamFetcher.java 0% <ø> (ø) 0 <0> (?)
...owncloud/android/ui/activity/SettingsActivity.java 0% <ø> (ø) 0 <0> (?)
...com/owncloud/android/ui/activity/FileActivity.java 0% <ø> (ø) 0 <0> (?)
.../android/authentication/AuthenticatorActivity.java 1.89% <ø> (ø) 0 <0> (?)
... and 20 more

@nextcloud-android-bot
Copy link
Collaborator

Codacy

302

Lint

TypemasterPR
Warnings5757
Errors00

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings25
Correctness Warnings79
Internationalization Warnings15
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings9
Performance Warnings123
Security Warnings47
Dodgy code Warnings132
Total434

SpotBugs (master)

Warning TypeNumber
Bad practice Warnings25
Correctness Warnings79
Internationalization Warnings15
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings9
Performance Warnings123
Security Warnings47
Dodgy code Warnings132
Total434

@AndyScherzinger AndyScherzinger merged commit 92df32e into master Apr 18, 2019
@AndyScherzinger AndyScherzinger deleted the codacyCleanup branch April 18, 2019 06:12
@AndyScherzinger AndyScherzinger added this to the Nextcloud App 3.7.0 milestone Apr 18, 2019
nextcloud-android-bot pushed a commit that referenced this pull request May 2, 2019
@tobiasKaminsky
daily dev 20190502
3c51355 
71f3370 Drone: update FindBugs results to reflect reduced error/warning count [skip ci]
0f50088 Merge pull request #3916 from jmue/cleanup/string_compare
7bb07a0 Update wording for store text (#3903)
4c72f3f Merge pull request #3923 from nextcloud/editOnShareLink
81ed32f allow edit on link share on a file
7d01ff7 - fix IT tests - re-organized build.gradle a bit
a9afff7 Merge pull request #3929 from nextcloud/dependabot/gradle/org.powermock-powermock-core-2.0.2
71c6d99 bump all other powermock parts to 2.0.2
0421d26 Bump powermock-core from 2.0.0 to 2.0.2
1d28d1b Update with new wording from frontpage
de3d3de Merge pull request #3924 from nextcloud/warnOnStable
69dd243 warn for wrong library branch on stable
2379b49 Merge pull request #3921 from nextcloud/dependabot/gradle/com.android.tools.build-gradle-3.4.0
c2730e8 Bump gradle from 3.3.2 to 3.4.0
b69169f unify empty string compare
8bb654b fix test & review comments
d8b9d10 unify empty string compare
2f23a30 remove unused variable (#3920)
92df32e Merge pull request #3918 from nextcloud/codacyCleanup
091c003 Drone: update FindBugs results to reflect reduced error/warning count [skip ci]
15ec487 Drone: update FindBugs results to reflect reduced error/warning count [skip ci]
da3c497 Merge commit '334155e707975e740a0a13b33d9b7bbedf40e9bc'
b22f517 Merge pull request #3915 from jmue/cleanup/ocfile
c7c6a98 use constant
e6604c8 use constant
caa2a91 Merge pull request #3912 from nextcloud/libraryUsage
334155e some cleanups
28f80cd Merge pull request #3907 from nextcloud/ezaquarii/initialize-main-app-context-before-content-providers-start
2afb934 Drone: update FindBugs results to reflect reduced error/warning count [skip ci]
8be1157 Merge pull request #3899 from jmue/cleanup/uriutils
52020a9 direct usage of library project
4062d40 Merge pull request #3908 from nextcloud/missingInjectable
2a08cb4 Initialize global context before MainApp.onCreate()
f02f482 Missing Injectable lead to half-stored account
02cbafe Merge pull request #3765 from nextcloud/showSharees
dc7a2c0 show sharees
03e04e5 Merge pull request #3889 from nextcloud/checkUserId
d65e94b Update wording for store text
585a3d0 Drone: update FindBugs results to reflect reduced error/warning count [skip ci]
533debf - removed useNextcloudUserAgent -> we now use this everytime - removed any oAuth and saml stuff, as we rely on weblogin flow
d544fa5 Merge pull request #3897 from nextcloud/ezaquarii/move-current-account-getter-to-user-account-manager
5b4ffba Merge pull request #3900 from nextcloud/codacyOnMaster
2757244 remove unused code
428a8bb show codacy count only on master
6b44074 Drone: update FindBugs results to reflect reduced error/warning count [skip ci]
c62e89b Merge commit '442394f949ff2de86bcccae172d2e9d997ab2ef8'
442394f Migrate current account getter from AccountUtils to UserAccountManager
182b65e daily dev 20190414
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AndyScherzinger AndyScherzinger AndyScherzinger approved these changes

Assignees
No one assigned
Labels
3. to review
Projects
None yet
Milestone
Nextcloud App 3.7.0
Development

Successfully merging this pull request may close these issues.
4 participants
@tobiasKaminsky @nextcloud-android-bot @AndyScherzinger @jmue