[Pkg-owncloud-commits] [owncloud] 29/223: movie previews: use file directly when it's stored locally and encryption is not enabled, fixes #7756

David Prévot taffit at moszumanska.debian.org
Sun Jun 22 01:54:02 UTC 2014


This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository owncloud.

commit 7396e5f455b0e723e9c7ef41bfb34a17de0569cf
Author: Georg Ehrke <developer at georgehrke.com>
Date:   Tue Jun 10 22:08:12 2014 +0200

    movie previews: use file directly when it's stored locally and encryption is not enabled, fixes #7756
---
 lib/private/preview/movies.php | 25 +++++++++++++++++--------
 1 file changed, 17 insertions(+), 8 deletions(-)

diff --git a/lib/private/preview/movies.php b/lib/private/preview/movies.php
index 72ccfad..2a23c21 100644
--- a/lib/private/preview/movies.php
+++ b/lib/private/preview/movies.php
@@ -41,14 +41,22 @@ if (!\OC_Util::runningOnWindows()) {
 
 			public function getThumbnail($path, $maxX, $maxY, $scalingup, $fileview) {
 				// TODO: use proc_open() and stream the source file ?
-				$absPath = \OC_Helper::tmpFile();
 
-				$handle = $fileview->fopen($path, 'rb');
+				$fileInfo = $fileview->getFileInfo($path);
+				$useFileDirectly = (!$fileInfo->isEncrypted() && !$fileInfo->isMounted());
 
-				// we better use 5MB (1024 * 1024 * 5 = 5242880) instead of 1MB.
-				// in some cases 1MB was no enough to generate thumbnail
-				$firstmb = stream_get_contents($handle, 5242880);
-				file_put_contents($absPath, $firstmb);
+				if ($useFileDirectly) {
+					$absPath = $fileview->getLocalFile($path);
+				} else {
+					$absPath = \OC_Helper::tmpFile();
+
+					$handle = $fileview->fopen($path, 'rb');
+
+					// we better use 5MB (1024 * 1024 * 5 = 5242880) instead of 1MB.
+					// in some cases 1MB was no enough to generate thumbnail
+					$firstmb = stream_get_contents($handle, 5242880);
+					file_put_contents($absPath, $firstmb);
+				}
 
 				$result = $this->generateThumbNail($maxX, $maxY, $absPath, 5);
 				if ($result === false) {
@@ -58,8 +66,9 @@ if (!\OC_Util::runningOnWindows()) {
 					}
 				}
 
-				unlink($absPath);
-
+				if (!$useFileDirectly) {
+					unlink($absPath);
+				}
 
 				return $result;
 			}

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-owncloud/owncloud.git



More information about the Pkg-owncloud-commits mailing list