From e62d5b7e55ea5eab098751146f6c49641574c7c9 Mon Sep 17 00:00:00 2001
From: tobiasKaminsky <tobias@kaminsky.me>
Date: Sat, 9 Aug 2014 10:39:12 +0200
Subject: [PATCH] Route for thumbnail generation

Thumbnail generation

Removed Log

Added requested changes

Added requested changes.

- Fix code style
- Add exception if file does not exist
- Switch route styling

Replaces https://github.com/owncloud/core/pull/10805

Fix codestyle

Fix codestyle

Migrate to appframework

Fix typo
---
 apps/files/appinfo/application.php      | 31 +++++++++++++++
 apps/files/appinfo/routes.php           | 12 +++++-
 apps/files/controller/apicontroller.php | 52 +++++++++++++++++++++++++
 lib/private/preview.php                 |  6 +++
 4 files changed, 99 insertions(+), 2 deletions(-)
 create mode 100644 apps/files/appinfo/application.php
 create mode 100644 apps/files/controller/apicontroller.php

diff --git a/apps/files/appinfo/application.php b/apps/files/appinfo/application.php
new file mode 100644
index 0000000000..7ca48bab47
--- /dev/null
+++ b/apps/files/appinfo/application.php
@@ -0,0 +1,31 @@
+<?php
+/**
+ * Copyright (c) 2014 Lukas Reschke <lukas@owncloud.com>
+ * This file is licensed under the Affero General Public License version 3 or
+ * later.
+ * See the COPYING-README file.
+ */
+
+namespace OCA\Files\Appinfo;
+
+use OC\AppFramework\Utility\SimpleContainer;
+use OCA\Files\Controller\ApiController;
+use OCP\AppFramework\App;
+
+class Application extends App {
+	public function __construct(array $urlParams=array()) {
+		parent::__construct('files', $urlParams);
+		$container = $this->getContainer();
+
+
+		/**
+		 * Controllers
+		 */
+		$container->registerService('APIController', function (SimpleContainer $c) {
+			return new ApiController(
+				$c->query('AppName'),
+				$c->query('Request')
+			);
+		});
+	}
+}
diff --git a/apps/files/appinfo/routes.php b/apps/files/appinfo/routes.php
index 4d77065a34..a99b5de3af 100644
--- a/apps/files/appinfo/routes.php
+++ b/apps/files/appinfo/routes.php
@@ -6,7 +6,15 @@
  * See the COPYING-README file.
  */
 
-/** @var $this OC\Route\Router */
+namespace OCA\Files\Appinfo;
+
+$application = new Application();
+$application->registerRoutes($this, array('routes' => array(
+	array('name' => 'API#getThumbnail', 'url' => '/api/v1/thumbnail/{x}/{y}/{file}', 'verb' => 'GET'),
+)));
+
+
+/** @var $this \OC\Route\Router */
 
 $this->create('files_index', '/')
 	->actionInclude('files/index.php');
@@ -38,4 +46,4 @@ $this->create('download', 'download{file}')
 	->actionInclude('files/download.php');
 	
 // Register with the capabilities API
-OC_API::register('get', '/cloud/capabilities', array('OCA\Files\Capabilities', 'getCapabilities'), 'files', OC_API::USER_AUTH);
+\OC_API::register('get', '/cloud/capabilities', array('OCA\Files\Capabilities', 'getCapabilities'), 'files', \OC_API::USER_AUTH);
diff --git a/apps/files/controller/apicontroller.php b/apps/files/controller/apicontroller.php
new file mode 100644
index 0000000000..01f9086c27
--- /dev/null
+++ b/apps/files/controller/apicontroller.php
@@ -0,0 +1,52 @@
+<?php
+/**
+ * Copyright (c) 2014 Lukas Reschke <lukas@owncloud.com>
+ * This file is licensed under the Affero General Public License version 3 or
+ * later.
+ * See the COPYING-README file.
+ */
+
+namespace OCA\Files\Controller;
+
+use OCP\AppFramework\Http;
+use OCP\AppFramework\Controller;
+use OCP\IRequest;
+use OCP\AppFramework\Http\JSONResponse;
+use OCP\AppFramework\Http\DownloadResponse;
+use OC\Preview;
+
+class ApiController extends Controller {
+
+	public function __construct($appName, IRequest $request){
+		parent::__construct($appName, $request);
+	}
+
+
+	/**
+	 * Gets a thumbnail of the specified file
+	 *
+	 * @since API version 1.0
+	 *
+	 * @NoAdminRequired
+	 * @NoCSRFRequired
+	 *
+	 * @param int $x
+	 * @param int $y
+	 * @param string $file
+	 * @return JSONResponse|DownloadResponse
+	 */
+	public function getThumbnail($x, $y, $file) {
+		if($x < 1 || $y < 1) {
+			return new JSONResponse('Requested size must be numeric and a positive value.', Http::STATUS_BAD_REQUEST);
+		}
+
+		try {
+			$preview = new Preview('', 'files', $file, $x, $y, true);
+			echo($preview->showPreview('image/png'));
+			return new DownloadResponse($file.'.png', 'image/png');
+		} catch (\Exception $e) {
+			return new JSONResponse('File not found.', Http::STATUS_NOT_FOUND);
+		}
+	}
+
+}
diff --git a/lib/private/preview.php b/lib/private/preview.php
index d6bff961a7..bc3eccbafb 100755
--- a/lib/private/preview.php
+++ b/lib/private/preview.php
@@ -15,6 +15,7 @@ namespace OC;
 
 use OC\Files\Filesystem;
 use OC\Preview\Provider;
+use OCP\Files\NotFoundException;
 
 require_once 'preview/image.php';
 require_once 'preview/movies.php';
@@ -110,6 +111,11 @@ class Preview {
 			\OC_Log::write('core', 'No preview providers exist', \OC_Log::ERROR);
 			throw new \Exception('No preview providers');
 		}
+
+		// Check if file is valid
+		if($this->isFileValid() === false) {
+			throw new NotFoundException('File not found.');
+		}
 	}
 
 	/**