From 25b9e90ef3b6018f898047be025ad859fcbbd96a Mon Sep 17 00:00:00 2001
From: Frederic Guillot <fred@kanboard.net>
Date: Wed, 23 Sep 2015 20:59:21 -0400
Subject: Do not check anymore data folder permissions

People who are using a remote database (Mysql/Postgresql) and a remote file storage (Aws S3 or similar) don't necessary needs to have a persistent local data folder or to change the permissions.
---
 app/Api/File.php | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

(limited to 'app/Api/File.php')

diff --git a/app/Api/File.php b/app/Api/File.php
index 97aa9d82..ad736ad4 100644
--- a/app/Api/File.php
+++ b/app/Api/File.php
@@ -2,6 +2,8 @@
 
 namespace Api;
 
+use Core\ObjectStorage\ObjectStorageException;
+
 /**
  * File API controller
  *
@@ -22,16 +24,17 @@ class File extends \Core\Base
 
     public function downloadFile($file_id)
     {
-        $file = $this->file->getById($file_id);
-
-        if (! empty($file)) {
+        try {
 
-            $filename = FILES_DIR.$file['path'];
+            $file = $this->file->getById($file_id);
 
-            if (file_exists($filename)) {
-                return base64_encode(file_get_contents($filename));
+            if (! empty($file)) {
+                return base64_encode($this->objectStorage->get($file['path']));
             }
         }
+        catch (ObjectStorageException $e) {
+            $this->logger->error($e->getMessage());
+        }
 
         return '';
     }
-- 
cgit v1.2.3