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] Fix accesslist when a user has an ID only containting 0-9 #7262

Merged
merged 1 commit into from Nov 23, 2017

Conversation

Projects
None yet
3 participants
@nickvergessen
Member

nickvergessen commented Nov 23, 2017

backport #7168

Fix accesslist when a user has an ID only containting 0-9
Signed-off-by: Joas Schilling <coding@schilljs.com>
@blizzz

blizzz approved these changes Nov 23, 2017

@codecov

This comment has been minimized.

Show comment
Hide comment
@codecov

codecov bot Nov 23, 2017

Codecov Report

Merging #7262 into stable12 will not change coverage.
The diff coverage is 100%.

@@             Coverage Diff             @@
##             stable12    #7262   +/-   ##
===========================================
  Coverage       53.79%   53.79%           
  Complexity      22590    22590           
===========================================
  Files            1384     1384           
  Lines           86666    86666           
  Branches         1329     1329           
===========================================
  Hits            46625    46625           
  Misses          40041    40041
Impacted Files Coverage Δ Complexity Δ
lib/private/Share20/Manager.php 87.04% <100%> (ø) 233 <0> (ø) ⬇️
lib/private/Files/Cache/Propagator.php 94.93% <0%> (-1.27%) 16% <0%> (ø)
lib/private/Server.php 84.68% <0%> (-0.13%) 121% <0%> (ø)
apps/files_trashbin/lib/Trashbin.php 72.53% <0%> (+0.24%) 136% <0%> (ø) ⬇️
lib/private/Security/CertificateManager.php 92.07% <0%> (+0.99%) 39% <0%> (ø) ⬇️

codecov bot commented Nov 23, 2017

Codecov Report

Merging #7262 into stable12 will not change coverage.
The diff coverage is 100%.

@@             Coverage Diff             @@
##             stable12    #7262   +/-   ##
===========================================
  Coverage       53.79%   53.79%           
  Complexity      22590    22590           
===========================================
  Files            1384     1384           
  Lines           86666    86666           
  Branches         1329     1329           
===========================================
  Hits            46625    46625           
  Misses          40041    40041
Impacted Files Coverage Δ Complexity Δ
lib/private/Share20/Manager.php 87.04% <100%> (ø) 233 <0> (ø) ⬇️
lib/private/Files/Cache/Propagator.php 94.93% <0%> (-1.27%) 16% <0%> (ø)
lib/private/Server.php 84.68% <0%> (-0.13%) 121% <0%> (ø)
apps/files_trashbin/lib/Trashbin.php 72.53% <0%> (+0.24%) 136% <0%> (ø) ⬇️
lib/private/Security/CertificateManager.php 92.07% <0%> (+0.99%) 39% <0%> (ø) ⬇️

@MorrisJobke MorrisJobke merged commit b901f8b into stable12 Nov 23, 2017

3 checks passed

codecov/patch 100% of diff hit (target 53.79%)
Details
codecov/project 53.79% (+0%) compared to 4a4cf26
Details
continuous-integration/drone/pr the build was successful
Details

@MorrisJobke MorrisJobke referenced this pull request Nov 23, 2017

Merged

Prepare 12.0.4 RC2 #7261

4 of 5 tasks complete

@MorrisJobke MorrisJobke deleted the backport/7168/files-access-list-with-integer-userid branch Nov 23, 2017

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