don't call getFileInfo() to avoid to open the same file twice

diff --git a/apps/files_encryption/lib/stream.php b/apps/files_encryption/lib/stream.php
index 58ac03373a737bc2136a5cde4fad3b385114b1fc..1a163db887c4bbad61520a9e3c5e74dca195d482 100644 (file)
--- a/apps/files_encryption/lib/stream.php
+++ b/apps/files_encryption/lib/stream.php
@@ -568,21 +568,25 @@ class Stream {
                        // part file.
                        $path = Helper::stripPartialFileExtension($this->rawPath);
 
-                       // get file info
-                       $fileInfo = $this->rootView->getFileInfo($path);
-                       if ($fileInfo) {
-                               // set encryption data
-                               $fileInfo['encrypted'] = true;
-                               $fileInfo['size'] = $this->size;
-                               $fileInfo['unencrypted_size'] = $this->unencryptedSize;
-
-                               // set fileinfo
-                               $this->rootView->putFileInfo($path, $fileInfo);
-                       }
+                       $fileInfo = array(
+                               'encrypted' => true,
+                               'size' => $this->size,
+                               'unencrypted_size' => $this->unencryptedSize,
+                       );
+
+                       // set fileinfo
+                       $this->rootView->putFileInfo($path, $fileInfo);
 
                }
 
-               return fclose($this->handle);
+               $result = fclose($this->handle);
+
+               if ($result === false) {
+                       \OCP\Util::writeLog('Encryption library', 'Could not close stream, file could be corrupted', \OCP\Util::FATAL);
+               }
+
+               return $result;
+
        }
 
 }