summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLukas Reschke <lukas@owncloud.com>2014-11-24 01:12:43 +0100
committerLukas Reschke <lukas@owncloud.com>2014-11-24 01:12:43 +0100
commitcebfdd877d49208f2a89e6661a019eb592617e92 (patch)
treed4a5e5b13607ef7aaa11f4216845a2dfa1a3de5d
parenta3496cf7fab6f4e9d4ce804e3b3ce0fc83a863dc (diff)
parent45d985f2d8f3df26960b36e7cb6d75513e5e7dfc (diff)
downloadnextcloud-server-cebfdd877d49208f2a89e6661a019eb592617e92.tar.gz
nextcloud-server-cebfdd877d49208f2a89e6661a019eb592617e92.zip
Merge pull request #12372 from owncloud/preview_tiff_fix
remove ugly hack and don't use OC\Preview\Image for tiffs and svgs
-rw-r--r--lib/private/preview.php10
-rw-r--r--lib/private/preview/image.php2
2 files changed, 1 insertions, 11 deletions
diff --git a/lib/private/preview.php b/lib/private/preview.php
index f50bdcb4c9e..778007b21fd 100644
--- a/lib/private/preview.php
+++ b/lib/private/preview.php
@@ -892,16 +892,6 @@ class Preview {
self::initProviders();
}
- // FIXME: Ugly hack to prevent SVG of being returned if the SVG
- // provider is not enabled.
- // This is required because the preview system is designed in a
- // bad way and relies on opt-in with asterisks (i.e. image/*)
- // which will lead to the fact that a SVG will also match the image
- // provider.
- if($mimeType === 'image/svg+xml' && !array_key_exists('/image\/svg\+xml/', self::$providers)) {
- return false;
- }
-
foreach(self::$providers as $supportedMimetype => $provider) {
if(preg_match($supportedMimetype, $mimeType)) {
return true;
diff --git a/lib/private/preview/image.php b/lib/private/preview/image.php
index ec5b87befea..7bcbddc649e 100644
--- a/lib/private/preview/image.php
+++ b/lib/private/preview/image.php
@@ -11,7 +11,7 @@ namespace OC\Preview;
class Image extends Provider {
public function getMimeType() {
- return '/image\/.*/';
+ return '/image\/(?!tiff$)(?!svg.*).*/';
}
public function getThumbnail($path, $maxX, $maxY, $scalingup, $fileview) {