fix(JSResourceLocator): Handle missing translations silently

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

1 files changed, 16 insertions, 9 deletions

diff --git a/lib/private/Template/JSResourceLocator.php b/lib/private/Template/JSResourceLocator.php
index 120234146e1..61611c69b0f 100644
--- a/lib/private/Template/JSResourceLocator.php
+++ b/lib/private/Template/JSResourceLocator.php
@@ -99,21 +99,28 @@ class JSResourceLocator extends ResourceLocator {
 			// gets turned into cwd.
 			$app_path = realpath($app_path);
 
-			// missing translations files will be ignored
-			if (strpos($script, 'l10n/') === 0) {
-				$this->appendScriptIfExist($app_path, $script, $app_url);
+			// check combined files
+			if ($this->cacheAndAppendCombineJsonIfExist($app_path, $script.'.json', $app)) {
 				return;
 			}
 
-			if (!$this->cacheAndAppendCombineJsonIfExist($app_path, $script.'.json', $app)) {
-				$this->appendScriptIfExist($app_path, $script, $app_url);
+			// fallback to plain file location
+			if ($this->appendScriptIfExist($app_path, $script, $app_url)) {
+				return;
 			}
 		} catch (AppPathNotFoundException) {
-			$this->logger->error('Could not find resource {resource} to load', [
-				'resource' => $app . '/' . $script . '.js',
-				'app' => 'jsresourceloader',
-			]);
+			// pass (general error handling happens below)
 		}
+
+		// missing translations files will be ignored
+		if (strpos($script, 'l10n/') === 0) {
+			return;
+		}
+
+		$this->logger->error('Could not find resource {resource} to load', [
+			'resource' => $app . '/' . $script . '.js',
+			'app' => 'jsresourceloader',
+		]);
 	}
 
 	/**