Skip to content

Connectivity check: no need to use owncloudClient - #4344

Merged
tobiasKaminsky merged 3 commits into
masterfrom
plainConnection
Aug 21, 2019
Merged

Connectivity check: no need to use owncloudClient#4344
tobiasKaminsky merged 3 commits into
masterfrom
plainConnection

Conversation

@tobiasKaminsky

Copy link
Copy Markdown
Member

As both endpoints are accessible without credentials, we do not have to use ownCloudClient, which makes invocation a bit easier.

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

@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
… [skip ci]
Signed-off-by: nextcloud-android-bot <android@nextcloud.com>
@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Collaborator

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

@nextcloudnextcloud deleted a comment Aug 21, 2019
@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

Codacy

286

Lint

TypemasterPR
Warnings5858
Errors00

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings24
Correctness Warnings69
Internationalization Warnings12
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings9
Performance Warnings110
Security Warnings46
Dodgy code Warnings138
Total412

SpotBugs (master)

Warning TypeNumber
Bad practice Warnings24
Correctness Warnings69
Internationalization Warnings12
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings9
Performance Warnings110
Security Warnings46
Dodgy code Warnings139
Total413

@codecov

codecovBot commented Aug 21, 2019

Copy link
Copy Markdown

Codecov Report

Merging #4344 into master will increase coverage by 0.01%.
The diff coverage is 60%.

@@ Coverage Diff @@## master #4344 +/- ##
============================================
+ Coverage 16.81% 16.82% +0.01% 
Complexity 1 1 ============================================
Files 358 358 Lines 31791 31788 -3 Branches 4490 4490 ============================================
+ Hits 5345 5349 +4 + Misses 25547 25541 -6 + Partials 899 898 -1
Impacted FilesCoverage ΔComplexity Δ
...om/nextcloud/client/network/ClientFactoryImpl.java37.5% <0%> (-5.36%)0 <0> (ø)
...xtcloud/client/account/UserAccountManagerImpl.java50% <100%> (+2.72%)0 <0> (ø)⬇️
...tcloud/client/network/ConnectivityServiceImpl.java59.32% <66.66%> (+3.76%)0 <0> (ø)⬇️
.../third_parties/daveKoeller/AlphanumComparator.java82.14% <0%> (+1.19%)0% <0%> (ø)⬇️

1 similar comment
@codecov

codecovBot commented Aug 21, 2019

Copy link
Copy Markdown

Codecov Report

Merging #4344 into master will increase coverage by 0.01%.
The diff coverage is 60%.

@@ Coverage Diff @@## master #4344 +/- ##
============================================
+ Coverage 16.81% 16.82% +0.01% 
Complexity 1 1 ============================================
Files 358 358 Lines 31791 31788 -3 Branches 4490 4490 ============================================
+ Hits 5345 5349 +4 + Misses 25547 25541 -6 + Partials 899 898 -1
Impacted FilesCoverage ΔComplexity Δ
...om/nextcloud/client/network/ClientFactoryImpl.java37.5% <0%> (-5.36%)0 <0> (ø)
...xtcloud/client/account/UserAccountManagerImpl.java50% <100%> (+2.72%)0 <0> (ø)⬇️
...tcloud/client/network/ConnectivityServiceImpl.java59.32% <66.66%> (+3.76%)0 <0> (ø)⬇️
.../third_parties/daveKoeller/AlphanumComparator.java82.14% <0%> (+1.19%)0% <0%> (ø)⬇️

@tobiasKaminsky
tobiasKaminsky merged commit 66a8898 into masterAug 21, 2019
@tobiasKaminsky
tobiasKaminsky deleted the plainConnection branch August 21, 2019 05:57
@AndyScherzinger

Copy link
Copy Markdown
Member

/backport to stable-3.8

@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable-3.8 in #4362

tobiasKaminsky added a commit that referenced this pull request Aug 22, 2019
903b38c [tx-robot] updated from transifex
7a29f48 Sharees will be in NC17 (#4284)
39cb5d9 sharees will be in NC17 later initialization of storageManager add support for groups catch potential wrong saved sharee date
66a8898 Connectivity check: no need to use owncloudClient (#4344)
6473e7e fix unit tests
0d65c29 [tx-robot] updated from transifex
9da4251 daily dev 20190821
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@tobiasKaminsky@nextcloud-android-bot@AndyScherzinger