mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Set the JSCombiner cache if needed 

Found while debugging a customer setup. They had to flush their Redis.
Hence the info was no longer there. Since they also used S3 this meant
requesting the files over and over on template render. Which on S3 is
not cheap.

Now we just write it back if we can't get it from the cache in the first
place. So that the next run has it cached properly again.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
This commit is contained in:
Roeland Jago Douma 2021-01-06 16:15:17 +01:00 committed by backportbot[bot]
parent 2f1c118d3b
commit 82fab4d621
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/private/Template/JSCombiner.php
View File

@ -121,7 +121,9 @@ class JSCombiner {
$fileName = $fileName . '.deps';
try {
$deps = $this->depsCache->get($folder->getName() . '-' . $fileName);
$fromCache = true;
if ($deps === null || $deps === '') {
$fromCache = false;
$depFile = $folder->getFile($fileName);
$deps = $depFile->getContent();
}
@ -144,6 +146,10 @@ class JSCombiner {
}
}
if ($fromCache === false) {
$this->depsCache->set($folder->getName() . '-' . $fileName, json_encode($deps));
}
return true;
} catch (NotFoundException $e) {
return false;