From 34b86edc349540439277b3f9cd69fd271980c4de Mon Sep 17 00:00:00 2001
From: Roeland Jago Douma <roeland@famdouma.nl>
Date: Wed, 7 Mar 2018 12:55:05 +0100
Subject: [PATCH 1/2] Check if the cached js file exists

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>
---
 lib/private/Template/JSCombiner.php | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/lib/private/Template/JSCombiner.php b/lib/private/Template/JSCombiner.php
index c5adcee685..bc548c22fd 100644
--- a/lib/private/Template/JSCombiner.php
+++ b/lib/private/Template/JSCombiner.php
@@ -104,13 +104,20 @@ class JSCombiner {
 	 * @return bool
 	 */
 	protected function isCached($fileName, ISimpleFolder $folder) {
-		$fileName = str_replace('.json', '.js', $fileName) . '.deps';
+		$fileName = str_replace('.json', '.js', $fileName);
+
+		if (!$folder->fileExists($fileName)) {
+			return false;
+		}
+
+		$fileName = $fileName . '.deps';
 		try {
 			$deps = $this->depsCache->get($folder->getName() . '-' . $fileName);
 			if ($deps === null || $deps === '') {
 				$depFile = $folder->getFile($fileName);
 				$deps = $depFile->getContent();
 			}
+
 			// check again
 			if ($deps === null || $deps === '') {
 				$this->logger->info('JSCombiner: deps file empty: ' . $fileName);

From 448a5e574e91daf9ddb195bfc96695a4ab8e770f Mon Sep 17 00:00:00 2001
From: Roeland Jago Douma <roeland@famdouma.nl>
Date: Wed, 7 Mar 2018 14:04:21 +0100
Subject: [PATCH 2/2] Fix tests

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
---
 tests/lib/Template/JSCombinerTest.php | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/tests/lib/Template/JSCombinerTest.php b/tests/lib/Template/JSCombinerTest.php
index d5f7000e0a..bec88801d6 100644
--- a/tests/lib/Template/JSCombinerTest.php
+++ b/tests/lib/Template/JSCombinerTest.php
@@ -187,6 +187,10 @@ class JSCombinerTest extends \Test\TestCase {
 
 		$fileDeps->expects($this->once())->method('getContent')->willReturn('{}');
 
+		$folder->method('fileExists')
+			->with('combine.js')
+			->willReturn(true);
+
 		$folder->method('getFile')
 			->will($this->returnCallback(function($path) use ($file, $fileDeps) {
 				if ($path === 'combine.js') {
@@ -196,6 +200,7 @@ class JSCombinerTest extends \Test\TestCase {
 				if ($path === 'combine.js.deps') {
 					return $fileDeps;
 				}
+
 				$this->fail();
 			}));
 
@@ -221,6 +226,9 @@ class JSCombinerTest extends \Test\TestCase {
 			->willReturn($folder);
 		$folder->method('getName')
 			->willReturn('awesomeapp');
+		$folder->method('fileExists')
+			->with('combine.js')
+			->willReturn(true);
 
 		$file = $this->createMock(ISimpleFile::class);
 
@@ -263,6 +271,9 @@ class JSCombinerTest extends \Test\TestCase {
 	public function testIsCachedWithNotExistingFile() {
 		$fileName = 'combine.json';
 		$folder = $this->createMock(ISimpleFolder::class);
+		$folder->method('fileExists')
+			->with('combine.js')
+			->willReturn(true);
 		$file = $this->createMock(ISimpleFile::class);
 		$folder->method('getFile')
 			->with('combine.js.deps')
@@ -278,6 +289,9 @@ class JSCombinerTest extends \Test\TestCase {
 	public function testIsCachedWithOlderMtime() {
 		$fileName = 'combine.json';
 		$folder = $this->createMock(ISimpleFolder::class);
+		$folder->method('fileExists')
+			->with('combine.js')
+			->willReturn(true);
 		$file = $this->createMock(ISimpleFile::class);
 		$folder->method('getFile')
 			->with('combine.js.deps')
@@ -293,6 +307,9 @@ class JSCombinerTest extends \Test\TestCase {
 	public function testIsCachedWithoutContent() {
 		$fileName = 'combine.json';
 		$folder = $this->createMock(ISimpleFolder::class);
+		$folder->method('fileExists')
+			->with('combine.js')
+			->willReturn(true);
 		$file = $this->createMock(ISimpleFile::class);
 		$folder->method('getFile')
 			->with('combine.js.deps')