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

Sharees will be in NC17 #4284

Merged
merged 1 commit into from
Aug 21, 2019
Merged

Sharees will be in NC17 #4284

merged 1 commit into from
Aug 21, 2019

Conversation

tobiasKaminsky
Copy link
Member

@tobiasKaminsky tobiasKaminsky commented Jul 31, 2019

Needs:

Needs:

TODO:

  • test transition of dev client before merging, as prevously userId as string was saved, but now gson seralized as string -> conflict
  • test with old server

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

@nextcloud-android-bot
Copy link
Collaborator

@nextcloud nextcloud deleted a comment Aug 6, 2019
@nextcloud nextcloud deleted a comment Aug 6, 2019
@nextcloud nextcloud deleted a comment from AndyScherzinger Aug 6, 2019
@nextcloud nextcloud deleted a comment from tobiasKaminsky Aug 6, 2019
@codecov
Copy link

codecov bot commented Aug 7, 2019

Codecov Report

Merging #4284 into master will decrease coverage by 0.7%.
The diff coverage is 46.15%.

@@             Coverage Diff              @@
##             master    #4284      +/-   ##
============================================
- Coverage     16.81%   16.11%   -0.71%     
  Complexity        1        1              
============================================
  Files           358      340      -18     
  Lines         31788    31198     -590     
  Branches       4490     4430      -60     
============================================
- Hits           5345     5026     -319     
+ Misses        25544    25307     -237     
+ Partials        899      865      -34
Impacted Files Coverage Δ Complexity Δ
...a/com/owncloud/android/utils/FileStorageUtils.java 20.29% <100%> (+0.19%) 0 <0> (ø) ⬇️
.../java/com/owncloud/android/utils/DisplayUtils.java 23.62% <100%> (+1.5%) 0 <0> (ø) ⬇️
...in/java/com/owncloud/android/datamodel/OCFile.java 64.67% <100%> (ø) 0 <0> (ø) ⬇️
...owncloud/android/ui/adapter/OCFileListAdapter.java 31% <7.14%> (+1%) 0 <0> (ø) ⬇️
...loud/android/datamodel/FileDataStorageManager.java 36.41% <83.33%> (+0.13%) 0 <0> (ø) ⬇️
...java/com/nextcloud/client/appinfo/AppInfoImpl.java 50% <0%> (-16.67%) 0% <0%> (ø)
...tcloud/client/network/ConnectivityServiceImpl.java 55.55% <0%> (-3.77%) 0% <0%> (ø)
...ncloud/android/ui/adapter/ActivityListAdapter.java 0% <0%> (-3.67%) 0% <0%> (ø)
...xtcloud/client/account/UserAccountManagerImpl.java 47.27% <0%> (-2.73%) 0% <0%> (ø)
...n/java/com/nextcloud/client/device/DeviceModule.kt 83.33% <0%> (-2.39%) 0% <0%> (ø)
... and 50 more

later initialization of storageManager
add support for groups
catch potential wrong saved sharee date

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

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

@tobiasKaminsky
Copy link
Member Author

/backport to stable-3.8

@nextcloud-android-bot
Copy link
Collaborator

@nextcloud-android-bot
Copy link
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 Warnings138
Total412

@tobiasKaminsky tobiasKaminsky merged commit 7a29f48 into master Aug 21, 2019
@tobiasKaminsky tobiasKaminsky deleted the adaptSharees branch August 21, 2019 11:21
@backportbot-nextcloud
Copy link

backport to stable-3.8 in #4366

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
@tobiasKaminsky tobiasKaminsky added this to the Nextcloud App 3.9.0 milestone Aug 23, 2019
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

3 participants