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

[stable10] Fix quota federated sharing (new) #29325

Merged
merged 6 commits into from
Oct 26, 2017

Conversation

PVince81
Copy link
Contributor

New backport of #29220 to stable10

@PVince81
Copy link
Contributor Author

PVince81 commented Oct 23, 2017

@codecov
Copy link

codecov bot commented Oct 23, 2017

Codecov Report

Merging #29325 into stable10 will increase coverage by 0.58%.
The diff coverage is 100%.

Impacted file tree graph

@@              Coverage Diff               @@
##             stable10   #29325      +/-   ##
==============================================
+ Coverage       58.43%   59.01%   +0.58%     
- Complexity      17707    17720      +13     
==============================================
  Files            1059     1060       +1     
  Lines           59386    59421      +35     
==============================================
+ Hits            34702    35068     +366     
+ Misses          24684    24353     -331
Impacted Files Coverage Δ Complexity Δ
lib/private/Files/Storage/DAV.php 88.81% <100%> (+77.75%) 163 <5> (+2) ⬆️
lib/private/Server.php 82.22% <100%> (+0.34%) 118 <2> (+3) ⬆️
apps/dav/lib/Connector/Sabre/QuotaPlugin.php 88.52% <100%> (ø) 26 <0> (+1) ⬆️
lib/private/Http/Client/WebDavClientService.php 100% <100%> (ø) 7 <7> (?)
lib/private/Security/CertificateManager.php 94.5% <0%> (+1.09%) 36% <0%> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cbd3bcc...7e9b5ad. Read the comment docs.

Vincent Petry added 6 commits October 26, 2017 11:08
Properly detect SPACE_UNLIMITED in quota check.
Improve error handling when receiving errors like 507 when uploading
files to a remote server.
Added tests for all FS methods of the Storage\DAV class and also fixed
some related issues.
@PVince81
Copy link
Contributor Author

applied tweak commit, rebased

@DeepDiver1975 please review

@PVince81 PVince81 merged commit a605468 into stable10 Oct 26, 2017
@PVince81 PVince81 deleted the stable10-fix-quota-fedsharing-new branch October 26, 2017 16:41
@lock
Copy link

lock bot commented Aug 2, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants