Skip to content

Check if the cached js file exists - #8714

Merged
MorrisJobke merged 2 commits into
masterfrom
8705
Mar 7, 2018
Merged

Check if the cached js file exists#8714
MorrisJobke merged 2 commits into
masterfrom
8705

Conversation

@rullzer

Copy link
Copy Markdown
Member

Fixes#8705

If the file does not exist (for whatever reason). It is never cached. No
matter what the depscache etc tell you.

@MorrisJobke please double check but I think this should do the trick.

Fixes#8705
If the file does not exist (for whatever reason). It is never cached. No
matter what the depscache etc tell you.
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

@skjnldsvskjnldsv left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes, I had issues with this before!

@MorrisJobkeMorrisJobke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tested and works 👍

@MorrisJobke

Copy link
Copy Markdown
Member

Tests fail.

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

codecovBot commented Mar 7, 2018

Copy link
Copy Markdown

Codecov Report

Merging #8714 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@ Coverage Diff @@## master #8714 +/- ##
============================================
- Coverage 51.87% 51.87% -0.01% - Complexity 25367 25368 +1 
============================================
Files 1607 1607 Lines 95154 95157 +3 Branches 1379 1379 ============================================
Hits 49362 49362 - Misses 45792 45795 +3
Impacted FilesCoverage ΔComplexity Δ
lib/private/Template/JSCombiner.php87.5% <100%> (-1.61%)31 <0> (+1)
lib/private/Files/Cache/Propagator.php94.93% <0%> (-1.27%)16% <0%> (ø)

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

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@rullzer@MorrisJobke@skjnldsv