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

[stable12] Check if owner of share exists #7407

Merged
merged 2 commits into from Dec 8, 2017

Conversation

Projects
None yet
3 participants
@rullzer
Member

rullzer commented Dec 6, 2017

Backport of #7348

rullzer added some commits Nov 30, 2017

A failed storage is a not available storage
We have to double check. Since getting the info of the root returns a
generic entry. But actually the stroage is not available. Else we get
very weird sync and web behavior.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Fix tests
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@codecov

This comment has been minimized.

codecov bot commented Dec 6, 2017

Codecov Report

Merging #7407 into stable12 will decrease coverage by <.01%.
The diff coverage is 50%.

@@              Coverage Diff              @@
##             stable12   #7407      +/-   ##
=============================================
- Coverage        53.9%   53.9%   -0.01%     
- Complexity      22692   22694       +2     
=============================================
  Files            1384    1384              
  Lines           86933   86935       +2     
  Branches         1330    1330              
=============================================
  Hits            46862   46862              
- Misses          40071   40073       +2
Impacted Files Coverage Δ Complexity Δ
apps/dav/lib/Connector/Sabre/ObjectTree.php 55.68% <50%> (-0.14%) 35 <0> (+2)
apps/files_trashbin/lib/Expiration.php 90.32% <0%> (-1.62%) 29% <0%> (ø)

@MorrisJobke MorrisJobke merged commit c9318dd into stable12 Dec 8, 2017

1 of 3 checks passed

codecov/patch 50% of diff hit (target 53.9%)
Details
codecov/project 53.9% (-0.01%) compared to f3b9b21
Details
continuous-integration/drone/pr the build was successful
Details

@MorrisJobke MorrisJobke deleted the 12_7348 branch Dec 8, 2017

@MorrisJobke MorrisJobke referenced this pull request Jan 8, 2018

Merged

12.0.5 RC1 #7740

13 of 13 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment