chore: add error logging

oat-sa · Oct 23, 2024 · 6e33eee · 6e33eee
1 parent 45e5f47
commit 6e33eee
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 3 deletions.
diff --git a/common/oatbox/filesystem/Directory.php b/common/oatbox/filesystem/Directory.php
@@ -138,6 +138,7 @@ public function deleteSelf()
             $this->getFileSystem()->deleteDirectory($this->getPrefix());
             return true;
         } catch (FilesystemException $e) {
+            $this->logWarning($e->getMessage());
         }
 
         return false;

diff --git a/common/oatbox/filesystem/File.php b/common/oatbox/filesystem/File.php
@@ -21,6 +21,7 @@
 
 namespace oat\oatbox\filesystem;
 
+use common_Logger;
 use GuzzleHttp\Psr7\Stream;
 use GuzzleHttp\Psr7\StreamWrapper;
 use Psr\Http\Message\StreamInterface;
@@ -53,6 +54,7 @@ public function getMimeType()
 
             return $mimeType;
         } catch (FilesystemException $e) {
+            $this->logWarning($e->getMessage());
         }
         return false;
     }
@@ -109,6 +111,7 @@ public function write($mixed, $mimeType = null)
                 ));
             }
         } catch (FilesystemException $e) {
+            $this->logWarning($e->getMessage());
             return false;
         }
 
@@ -131,7 +134,7 @@ public function update($mixed, $mimeType = null)
             throw new \RuntimeException('File "' . $this->getPrefix() . '" not found."');
         }
 
-        \common_Logger::i('Writing in ' . $this->getPrefix());
+        common_Logger::i('Writing in ' . $this->getPrefix());
 
         return $this->write($mixed, $mimeType);
     }
@@ -148,7 +151,7 @@ public function update($mixed, $mimeType = null)
      */
     public function put($mixed, $mimeType = null)
     {
-        \common_Logger::i('Writting in ' . $this->getPrefix());
+        common_Logger::i('Writting in ' . $this->getPrefix());
 
         return $this->write($mixed, $mimeType);
     }
@@ -163,6 +166,7 @@ public function read()
         try {
             return $this->getFileSystem()->read($this->getPrefix());
         } catch (FilesystemException $e) {
+            $this->logWarning($e->getMessage());
         }
 
         return false;
@@ -178,6 +182,7 @@ public function readStream()
         try {
             return $this->getFileSystem()->readStream($this->getPrefix());
         } catch (FilesystemException $e) {
+            $this->logWarning($e->getMessage());
         }
 
         return false;
@@ -194,6 +199,7 @@ public function readPsrStream()
         try {
             $resource = $this->getFileSystem()->readStream($this->getPrefix());
         } catch (FilesystemException $e) {
+            $this->logWarning($e->getMessage());
         }
 
         return new Stream($resource);
@@ -204,6 +210,7 @@ public function exists(): bool
         try {
             return $this->getFileSystem()->fileExists($this->getPrefix());
         } catch (FilesystemException $e) {
+            $this->logWarning($e->getMessage());
         }
         return false;
     }
@@ -214,6 +221,7 @@ public function delete(): bool
             $this->getFileSystem()->delete($this->getPrefix());
             return true;
         } catch (FilesystemException $e) {
+            $this->logWarning($e->getMessage());
         }
 
         return false;

diff --git a/common/oatbox/filesystem/FileSystemHandler.php b/common/oatbox/filesystem/FileSystemHandler.php
@@ -22,6 +22,7 @@
 
 namespace oat\oatbox\filesystem;
 
+use oat\oatbox\log\LoggerAwareTrait;
 use oat\oatbox\service\ServiceManager;
 use ReflectionClass;
 use ReflectionProperty;
@@ -31,8 +32,9 @@
 abstract class FileSystemHandler implements ServiceLocatorAwareInterface
 {
     use ServiceLocatorAwareTrait;
+    use LoggerAwareTrait;
 
-    private const NOT_SERIALIZABLE_PROPERTIES = ['fileSystem', 'serviceLocator'];
+    private const NOT_SERIALIZABLE_PROPERTIES = ['fileSystem', 'serviceLocator', 'logger'];
 
     /**
      * @var mixed