Skip to content

Conversation

@MorrisJobke
Copy link
Member

* found while adding the strict_typing for PHP 7+ migration
* first argument is a boolean - second one is the path
* see http://php.net/manual/en/function.clearstatcache.php

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke MorrisJobke added the 3. to review Waiting for reviews label Jan 12, 2018
Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙈

Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

woaw

@codecov
Copy link

codecov bot commented Jan 12, 2018

Codecov Report

Merging #7811 into master will decrease coverage by 1.64%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #7811      +/-   ##
============================================
- Coverage     52.87%   51.23%   -1.65%     
- Complexity    23658    24970    +1312     
============================================
  Files          1449     1607     +158     
  Lines         80454    95008   +14554     
  Branches          0     1376    +1376     
============================================
+ Hits          42541    48674    +6133     
- Misses        37913    46334    +8421
Impacted Files Coverage Δ Complexity Δ
lib/private/Files/Storage/Local.php 78.19% <100%> (-3.49%) 86 <0> (ø)
.../tests/Unit/Collaboration/CommentersSorterTest.php 25.55% <0%> (-66.45%) 6% <0%> (ø)
apps/sharebymail/tests/SettingsTest.php 52.17% <0%> (-47.83%) 3% <0%> (ø)
lib/private/Security/RateLimiting/Limiter.php 55.55% <0%> (-44.45%) 5% <0%> (ø)
settings/Controller/EncryptionController.php 54.71% <0%> (-38.84%) 8% <0%> (ø)
settings/Controller/GroupsController.php 64.61% <0%> (-35.39%) 9% <0%> (ø)
...ps/comments/tests/Unit/AppInfo/ApplicationTest.php 69.56% <0%> (-30.44%) 4% <0%> (ø)
lib/private/AvatarManager.php 71.42% <0%> (-28.58%) 4% <0%> (ø)
apps/user_ldap/lib/Configuration.php 42.02% <0%> (-27.86%) 87% <0%> (ø)
apps/encryption/lib/Command/EnableMasterKey.php 75% <0%> (-25%) 5% <0%> (ø)
... and 349 more

@rullzer rullzer merged commit daa3393 into master Jan 12, 2018
@rullzer rullzer deleted the fix-clearstatcache branch January 12, 2018 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants