Skip to content

Comments

Empty search no longer works#5836

Merged
LukasReschke merged 1 commit intomasterfrom
preview_update
Jul 25, 2017
Merged

Empty search no longer works#5836
LukasReschke merged 1 commit intomasterfrom
preview_update

Conversation

@rullzer
Copy link
Member

@rullzer rullzer commented Jul 22, 2017

We killed off the full wildcard search. So the preview cleanup should properly traverse.

Signed-off-by: Roeland Jago Douma <[email protected]>
@codecov
Copy link

codecov bot commented Jul 24, 2017

Codecov Report

Merging #5836 into master will increase coverage by <.01%.
The diff coverage is 80%.

@@             Coverage Diff              @@
##             master    #5836      +/-   ##
============================================
+ Coverage     53.81%   53.82%   +<.01%     
- Complexity    22732    22734       +2     
============================================
  Files          1403     1403              
  Lines         86644    86648       +4     
  Branches       1327     1327              
============================================
+ Hits          46629    46636       +7     
+ Misses        40015    40012       -3
Impacted Files Coverage Δ Complexity Δ
lib/private/Preview/Watcher.php 75.86% <80%> (-0.14%) 14 <4> (+2)
lib/private/Files/Cache/Cache.php 82.14% <0%> (-0.26%) 103% <0%> (ø)
lib/private/Server.php 93.72% <0%> (-0.14%) 121% <0%> (ø)
lib/private/Mail/EMailTemplate.php 60.48% <0%> (ø) 41% <0%> (ø) ⬇️
core/js/js.js 61.83% <0%> (+0.55%) 0% <0%> (ø) ⬇️
lib/private/Security/CertificateManager.php 92.07% <0%> (+0.99%) 39% <0%> (ø) ⬇️

@LukasReschke LukasReschke merged commit 63676d3 into master Jul 25, 2017
@LukasReschke LukasReschke deleted the preview_update branch July 25, 2017 15:08
@MorrisJobke
Copy link
Member

@rullzer Could you open the backport PR?

@rullzer
Copy link
Member Author

rullzer commented Aug 1, 2017

Backport in #5945

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.

4 participants