fix: allow config or cache files to not be saved in any storage to en…

…sure a least that the html page will load
theus77 · Aug 23, 2024 · 5cd496c · 5cd496c
1 parent d2d1e1e
commit 5cd496c
Showing 1 changed file with 24 additions and 18 deletions.
diff --git a/EMS/common-bundle/src/Storage/StorageManager.php b/EMS/common-bundle/src/Storage/StorageManager.php
@@ -139,34 +139,40 @@ public function saveContents(string $contents, string $filename, string $mimetyp
     {
         $hash = $this->computeStringHash($contents);
         $count = 0;
-
         foreach ($this->adapters as $adapter) {
-            if (!$this->isUsageSupported($adapter, $usageType)) {
-                continue;
-            }
+            try {
+                if (!$this->isUsageSupported($adapter, $usageType)) {
+                    continue;
+                }
 
-            if ($adapter->head($hash)) {
-                ++$count;
-                continue;
-            }
+                if ($adapter->head($hash)) {
+                    ++$count;
+                    continue;
+                }
 
-            if (!$adapter->initUpload($hash, \strlen($contents), $filename, $mimetype)) {
-                continue;
-            }
+                if (!$adapter->initUpload($hash, \strlen($contents), $filename, $mimetype)) {
+                    continue;
+                }
 
-            if (!$adapter->addChunk($hash, $contents)) {
-                continue;
-            }
+                if (!$adapter->addChunk($hash, $contents)) {
+                    continue;
+                }
 
-            $adapter->initFinalize($hash);
+                $adapter->initFinalize($hash);
 
-            if ($adapter->finalizeUpload($hash)) {
-                ++$count;
+                if ($adapter->finalizeUpload($hash)) {
+                    ++$count;
+                }
+            } catch (\Throwable $e) {
+                $this->logger->error(\sprintf('Not able to save %s in %s with message: %s', $hash, $adapter->__toString(), $e->getMessage()));
             }
         }
 
         if (0 === $count) {
-            throw new NotSavedException($hash);
+            $this->logger->error(\sprintf('The config %s was not able to be saved', $hash));
+            if ($usageType >= StorageInterface::STORAGE_USAGE_ASSET) {
+                throw new NotSavedException($hash);
+            }
         }
 
         return $hash;