Tests: Updated comment test to account for new editor usage

[bookstack] / app / Uploads / FileStorage.php

diff --git a/app/Uploads/FileStorage.php b/app/Uploads/FileStorage.php
index 278484e519de0629b8a29c0c5d4ee98a158b6b71..70040725a3df118027225d494d445815b175bd88 100644 (file)
--- a/app/Uploads/FileStorage.php
+++ b/app/Uploads/FileStorage.php
@@ -3,12 +3,12 @@
 namespace BookStack\Uploads;
 
 use BookStack\Exceptions\FileUploadException;
 namespace BookStack\Uploads;
 
 use BookStack\Exceptions\FileUploadException;

+use BookStack\Util\FilePathNormalizer;

 use Exception;
 use Illuminate\Contracts\Filesystem\Filesystem as Storage;
 use Illuminate\Filesystem\FilesystemManager;
 use Illuminate\Support\Facades\Log;
 use Illuminate\Support\Str;
 use Exception;
 use Illuminate\Contracts\Filesystem\Filesystem as Storage;
 use Illuminate\Filesystem\FilesystemManager;
 use Illuminate\Support\Facades\Log;
 use Illuminate\Support\Str;

-use League\Flysystem\WhitespacePathNormalizer;

 use Symfony\Component\HttpFoundation\File\UploadedFile;
 
 class FileStorage
 use Symfony\Component\HttpFoundation\File\UploadedFile;
 
 class FileStorage


@@ -70,6 +70,26 @@ class FileStorage
         return $filePath;
     }
 
         return $filePath;
     }
 

+    /**
+     * Check whether the configured storage is remote from the host of this app.
+     */
+    public function isRemote(): bool
+    {
+        return $this->getStorageDiskName() === 's3';
+    }
+
+    /**
+     * Get the actual path on system for the given relative file path.
+     */
+    public function getSystemPath(string $filePath): string
+    {
+        if ($this->isRemote()) {
+            return '';
+        }
+
+        return storage_path('uploads/files/' . ltrim($this->adjustPathForStorageDisk($filePath), '/'));
+    }
+

     /**
      * Get the storage that will be used for storing files.
      */
     /**
      * Get the storage that will be used for storing files.
      */


@@ -83,7 +103,7 @@ class FileStorage
      */
     protected function getStorageDiskName(): string
     {
      */
     protected function getStorageDiskName(): string
     {

-        $storageType = config('filesystems.attachments');
+        $storageType = trim(strtolower(config('filesystems.attachments')));

 
         // Change to our secure-attachment disk if any of the local options
         // are used to prevent escaping that location.
 
         // Change to our secure-attachment disk if any of the local options
         // are used to prevent escaping that location.


@@ -100,12 +120,13 @@ class FileStorage
      */
     protected function adjustPathForStorageDisk(string $path): string
     {
      */
     protected function adjustPathForStorageDisk(string $path): string
     {

-        $path = (new WhitespacePathNormalizer())->normalizePath(str_replace('uploads/files/', '', $path));
+        $trimmed = str_replace('uploads/files/', '', $path);
+        $normalized = FilePathNormalizer::normalize($trimmed);

 
         if ($this->getStorageDiskName() === 'local_secure_attachments') {
 
         if ($this->getStorageDiskName() === 'local_secure_attachments') {

-            return $path;
+            return $normalized;

         }
 
         }
 

-        return 'uploads/files/' . $path;
+        return 'uploads/files/' . $normalized;

     }
 }
     }
 }