ApiController::getReverseQuickaccess() - Code Metrics - Inspection of "Merge pull request #10320 from nextcloud/feature/1..." - nextcloud/server - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a3fdec...f8e9f8 )

by John

created 2018-08-08 10:27 UTC

ApiController::getReverseQuickaccess() A

↳ Parent: Project

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
nc	2
nop	0
dl	0
loc	6
rs	10
c	0
b	0
f	0

<?php
/**
 * @copyright Copyright (c) 2016, ownCloud, Inc.
 *
 * @author Bjoern Schiessle <[email protected]>
 * @author Christoph Wurst <[email protected]>
 * @author Joas Schilling <[email protected]>
 * @author Lukas Reschke <[email protected]>
 * @author Morris Jobke <[email protected]>
 * @author Robin Appelman <[email protected]>
 * @author Roeland Jago Douma <[email protected]>
 * @author Tobias Kaminsky <[email protected]>
 * @author Vincent Petry <[email protected]>
 * @author Felix Nüsse <[email protected]>
 * @license AGPL-3.0
 *
 * This code is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License, version 3,
 * as published by the Free Software Foundation.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 * GNU Affero General Public License for more details.
 *
 * You should have received a copy of the GNU Affero General Public License, version 3,
 * along with this program.  If not, see <http://www.gnu.org/licenses/>
 *
 */

namespace OCA\Files\Controller;

use OCP\AppFramework\Http;
use OCP\AppFramework\Controller;
use OCP\Files\File;
use OCP\Files\Folder;
use OCP\Files\NotFoundException;
use OCP\IConfig;
use OCP\IRequest;
use OCP\AppFramework\Http\DataResponse;
use OCP\AppFramework\Http\FileDisplayResponse;
use OCP\AppFramework\Http\Response;
use OCA\Files\Service\TagService;
use OCP\IPreview;
use OCP\Share\IManager;
use OC\Files\Node\Node;
use OCP\IUserSession;
use Sabre\VObject\Property\Boolean;

/**
 * Class ApiController
 *
 * @package OCA\Files\Controller
 */
class ApiController extends Controller {
	/** @var TagService */
	private $tagService;
	/** @var IManager * */
	private $shareManager;
	/** @var IPreview */
	private $previewManager;
	/** IUserSession */
	private $userSession;
	/** IConfig */
	private $config;
	/** @var Folder */
	private $userFolder;

	/**
	 * @param string $appName
	 * @param IRequest $request
	 * @param IUserSession $userSession
	 * @param TagService $tagService
	 * @param IPreview $previewManager
	 * @param IManager $shareManager
	 * @param IConfig $config
	 * @param Folder $userFolder
	 */
	public function __construct($appName,
								IRequest $request,
								IUserSession $userSession,
								TagService $tagService,
								IPreview $previewManager,
								IManager $shareManager,
								IConfig $config,
								Folder $userFolder) {
		parent::__construct($appName, $request);
		$this->userSession = $userSession;
		$this->tagService = $tagService;
		$this->previewManager = $previewManager;
		$this->shareManager = $shareManager;
		$this->config = $config;
		$this->userFolder = $userFolder;
	}

	/**
	 * Gets a thumbnail of the specified file
	 *
	 * @since API version 1.0
	 *
	 * @NoAdminRequired
	 * @NoCSRFRequired
	 * @StrictCookieRequired
	 *
	 * @param int $x
	 * @param int $y
	 * @param string $file URL-encoded filename
	 * @return DataResponse|FileDisplayResponse
	 */
	public function getThumbnail($x, $y, $file) {
		if ($x < 1 || $y < 1) {
			return new DataResponse(['message' => 'Requested size must be numeric and a positive value.'], Http::STATUS_BAD_REQUEST);
		}

		try {
			$file = $this->userFolder->get($file);
			if ($file instanceof Folder) {
				throw new NotFoundException();
			}

			/** @var File $file */
			$preview = $this->previewManager->getPreview($file, $x, $y, true);

			return new FileDisplayResponse($preview, Http::STATUS_OK, ['Content-Type' => $preview->getMimeType()]);
		} catch (NotFoundException $e) {
			return new DataResponse(['message' => 'File not found.'], Http::STATUS_NOT_FOUND);
		} catch (\Exception $e) {
			return new DataResponse([], Http::STATUS_BAD_REQUEST);
		}
	}

	/**
	 * Updates the info of the specified file path
	 * The passed tags are absolute, which means they will
	 * replace the actual tag selection.
	 *
	 * @NoAdminRequired
	 *
	 * @param string $path path
	 * @param array|string $tags array of tags
	 * @return DataResponse
	 */
	public function updateFileTags($path, $tags = null) {
		$result = [];
		// if tags specified or empty array, update tags
		if (!is_null($tags)) {
			try {
				$this->tagService->updateFileTags($path, $tags);

			} catch (\OCP\Files\NotFoundException $e) {
				return new DataResponse([
					'message' => $e->getMessage()
				], Http::STATUS_NOT_FOUND);
			} catch (\OCP\Files\StorageNotAvailableException $e) {
				return new DataResponse([
					'message' => $e->getMessage()
				], Http::STATUS_SERVICE_UNAVAILABLE);
			} catch (\Exception $e) {
				return new DataResponse([
					'message' => $e->getMessage()
				], Http::STATUS_NOT_FOUND);
			}
			$result['tags'] = $tags;
		}
		return new DataResponse($result);
	}

	/**
	 * @param \OCP\Files\Node[] $nodes
	 * @return array
	 */
	private function formatNodes(array $nodes) {
		return array_values(array_map(function (Node $node) {
			/** @var \OC\Files\Node\Node $shareTypes */
			$shareTypes = $this->getShareTypes($node);
			$file = \OCA\Files\Helper::formatFileInfo($node->getFileInfo());
			$parts = explode('/', dirname($node->getPath()), 4);
			if (isset($parts[3])) {
				$file['path'] = '/' . $parts[3];
			} else {
				$file['path'] = '/';
			}
			if (!empty($shareTypes)) {
				$file['shareTypes'] = $shareTypes;
			}
			return $file;
		}, $nodes));
	}

	/**
	 * Returns a list of recently modifed files.
	 *
	 * @NoAdminRequired
	 *
	 * @return DataResponse
	 */
	public function getRecentFiles() {
		$nodes = $this->userFolder->getRecent(100);
		$files = $this->formatNodes($nodes);
		return new DataResponse(['files' => $files]);
	}

	/**
	 * Return a list of share types for outgoing shares
	 *
	 * @param Node $node file node
	 *
	 * @return int[] array of share types
	 */
	private function getShareTypes(Node $node) {
		$userId = $this->userSession->getUser()->getUID();
		$shareTypes = [];
		$requestedShareTypes = [
			\OCP\Share::SHARE_TYPE_USER,
			\OCP\Share::SHARE_TYPE_GROUP,
			\OCP\Share::SHARE_TYPE_LINK,
			\OCP\Share::SHARE_TYPE_REMOTE,
			\OCP\Share::SHARE_TYPE_EMAIL
		];
		foreach ($requestedShareTypes as $requestedShareType) {
			// one of each type is enough to find out about the types
			$shares = $this->shareManager->getSharesBy(
				$userId,
				$requestedShareType,
				$node,
				false,
				1
			);
			if (!empty($shares)) {
				$shareTypes[] = $requestedShareType;
			}
		}
		return $shareTypes;
	}

	/**
	 * Change the default sort mode
	 *
	 * @NoAdminRequired
	 *
	 * @param string $mode
	 * @param string $direction
	 * @return Response
	 */
	public function updateFileSorting($mode, $direction) {
		$allowedMode = ['name', 'size', 'mtime'];
		$allowedDirection = ['asc', 'desc'];
		if (!in_array($mode, $allowedMode) || !in_array($direction, $allowedDirection)) {
			$response = new Response();
			$response->setStatus(Http::STATUS_UNPROCESSABLE_ENTITY);
			return $response;
		}
		$this->config->setUserValue($this->userSession->getUser()->getUID(), 'files', 'file_sorting', $mode);
		$this->config->setUserValue($this->userSession->getUser()->getUID(), 'files', 'file_sorting_direction', $direction);
		return new Response();
	}

	/**
	 * Toggle default for showing/hiding hidden files
	 *
	 * @NoAdminRequired
	 *
	 * @param bool $show
	 */
	public function showHiddenFiles($show) {
		$this->config->setUserValue($this->userSession->getUser()->getUID(), 'files', 'show_hidden', (int)$show);
		return new Response();
	}

	/**
	 * Toggle default for showing/hiding xxx folder
	 *
	 * @NoAdminRequired
	 *
	 * @param bool $show 
	 * @param bool $key the key of the folder
	 *
	 * @return Response
	 */
	public function toggleShowFolder(int $show, string $key) {
		// ensure the edited key exists
		$navItems = \OCA\Files\App::getNavigationManager()->getAll();
		foreach ($navItems as $item) {
			// check if data is valid
			if (($show === 0 || $show === 1) && isset($item['expandedState']) && $key === $item['expandedState']) {
				$this->config->setUserValue($this->userSession->getUser()->getUID(), 'files', $key, (int)$show);
				return new Response();
			}
		}
		$response = new Response();
		$response->setStatus(Http::STATUS_FORBIDDEN);
		return $response;
	}


}


1		<?php
2		/**
3		* @copyright Copyright (c) 2016, ownCloud, Inc.
4		*
5		* @author Bjoern Schiessle <[email protected]>
6		* @author Christoph Wurst <[email protected]>
7		* @author Joas Schilling <[email protected]>
8		* @author Lukas Reschke <[email protected]>
9		* @author Morris Jobke <[email protected]>
10		* @author Robin Appelman <[email protected]>
11		* @author Roeland Jago Douma <[email protected]>
12		* @author Tobias Kaminsky <[email protected]>
13		* @author Vincent Petry <[email protected]>
14		* @author Felix Nüsse <[email protected]>
15		* @license AGPL-3.0
16		*
17		* This code is free software: you can redistribute it and/or modify
18		* it under the terms of the GNU Affero General Public License, version 3,
19		* as published by the Free Software Foundation.
20		*
21		* This program is distributed in the hope that it will be useful,
22		* but WITHOUT ANY WARRANTY; without even the implied warranty of
23		* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
24		* GNU Affero General Public License for more details.
25		*
26		* You should have received a copy of the GNU Affero General Public License, version 3,
27		* along with this program. If not, see <http://www.gnu.org/licenses/>
28		*
29		*/
30
31		namespace OCA\Files\Controller;
32
33		use OCP\AppFramework\Http;
34		use OCP\AppFramework\Controller;
35		use OCP\Files\File;
36		use OCP\Files\Folder;
37		use OCP\Files\NotFoundException;
38		use OCP\IConfig;
39		use OCP\IRequest;
40		use OCP\AppFramework\Http\DataResponse;
41		use OCP\AppFramework\Http\FileDisplayResponse;
42		use OCP\AppFramework\Http\Response;
43		use OCA\Files\Service\TagService;
44		use OCP\IPreview;
45		use OCP\Share\IManager;
46		use OC\Files\Node\Node;
47		use OCP\IUserSession;
48		use Sabre\VObject\Property\Boolean;
49
50		/**
51		* Class ApiController
52		*
53		* @package OCA\Files\Controller
54		*/
55		class ApiController extends Controller {
56		/** @var TagService */
57		private $tagService;
58		/** @var IManager * */
59		private $shareManager;
60		/** @var IPreview */
61		private $previewManager;
62		/** IUserSession */
63		private $userSession;
64		/** IConfig */
65		private $config;
66		/** @var Folder */
67		private $userFolder;
68
69		/**
70		* @param string $appName
71		* @param IRequest $request
72		* @param IUserSession $userSession
73		* @param TagService $tagService
74		* @param IPreview $previewManager
75		* @param IManager $shareManager
76		* @param IConfig $config
77		* @param Folder $userFolder
78		*/
79		public function __construct($appName,
80		IRequest $request,
81		IUserSession $userSession,
82		TagService $tagService,
83		IPreview $previewManager,
84		IManager $shareManager,
85		IConfig $config,
86		Folder $userFolder) {
87		parent::__construct($appName, $request);
88		$this->userSession = $userSession;
89		$this->tagService = $tagService;
90		$this->previewManager = $previewManager;
91		$this->shareManager = $shareManager;
92		$this->config = $config;
93		$this->userFolder = $userFolder;
94		}
95
96		/**
97		* Gets a thumbnail of the specified file
98		*
99		* @since API version 1.0
100		*
101		* @NoAdminRequired
102		* @NoCSRFRequired
103		* @StrictCookieRequired
104		*
105		* @param int $x
106		* @param int $y
107		* @param string $file URL-encoded filename
108		* @return DataResponse\|FileDisplayResponse
109		*/
110		public function getThumbnail($x, $y, $file) {
111		if ($x < 1 \|\| $y < 1) {
112		return new DataResponse(['message' => 'Requested size must be numeric and a positive value.'], Http::STATUS_BAD_REQUEST);
113		}
114
115		try {
116		$file = $this->userFolder->get($file);
117		if ($file instanceof Folder) {
118		throw new NotFoundException();
119		}
120
121		/** @var File $file */
122		$preview = $this->previewManager->getPreview($file, $x, $y, true);
123
124		return new FileDisplayResponse($preview, Http::STATUS_OK, ['Content-Type' => $preview->getMimeType()]);
125		} catch (NotFoundException $e) {
126		return new DataResponse(['message' => 'File not found.'], Http::STATUS_NOT_FOUND);
127		} catch (\Exception $e) {
128		return new DataResponse([], Http::STATUS_BAD_REQUEST);
129		}
130		}
131
132		/**
133		* Updates the info of the specified file path
134		* The passed tags are absolute, which means they will
135		* replace the actual tag selection.
136		*
137		* @NoAdminRequired
138		*
139		* @param string $path path
140		* @param array\|string $tags array of tags
141		* @return DataResponse
142		*/
143		public function updateFileTags($path, $tags = null) {
144		$result = [];
145		// if tags specified or empty array, update tags
146		if (!is_null($tags)) {
147		try {
148		$this->tagService->updateFileTags($path, $tags);
		0 ignored issues – show Bug introduced 2016-06-13 14:40 UTC by Report Bug Copy Issue Report It seems like `$tags` defined by parameter `$tags` on line 143 can also be of type `string`; however, `OCA\Files\Service\TagService::updateFileTags()` does only seem to accept `array`, maybe add an additional type check? This check looks at variables that have been passed in as parameters and are passed out again to other methods. If the outgoing method call has stricter type requirements than the method itself, an issue is raised. An additional type check may prevent trouble. Loading history...
149		} catch (\OCP\Files\NotFoundException $e) {
150		return new DataResponse([
151		'message' => $e->getMessage()
152		], Http::STATUS_NOT_FOUND);
153		} catch (\OCP\Files\StorageNotAvailableException $e) {
154		return new DataResponse([
155		'message' => $e->getMessage()
156		], Http::STATUS_SERVICE_UNAVAILABLE);
157		} catch (\Exception $e) {
158		return new DataResponse([
159		'message' => $e->getMessage()
160		], Http::STATUS_NOT_FOUND);
161		}
162		$result['tags'] = $tags;
163		}
164		return new DataResponse($result);
165		}
166
167		/**
168		* @param \OCP\Files\Node[] $nodes
169		* @return array
170		*/
171		private function formatNodes(array $nodes) {
172		return array_values(array_map(function (Node $node) {
173		/** @var \OC\Files\Node\Node $shareTypes */
174		$shareTypes = $this->getShareTypes($node);
175		$file = \OCA\Files\Helper::formatFileInfo($node->getFileInfo());
176		$parts = explode('/', dirname($node->getPath()), 4);
177		if (isset($parts[3])) {
178		$file['path'] = '/' . $parts[3];
179		} else {
180		$file['path'] = '/';
181		}
182		if (!empty($shareTypes)) {
183		$file['shareTypes'] = $shareTypes;
184		}
185		return $file;
186		}, $nodes));
187		}
188
189		/**
190		* Returns a list of recently modifed files.
191		*
192		* @NoAdminRequired
193		*
194		* @return DataResponse
195		*/
196		public function getRecentFiles() {
197		$nodes = $this->userFolder->getRecent(100);
198		$files = $this->formatNodes($nodes);
199		return new DataResponse(['files' => $files]);
200		}
201
202		/**
203		* Return a list of share types for outgoing shares
204		*
205		* @param Node $node file node
206		*
207		* @return int[] array of share types
208		*/
209	View Code Duplication	private function getShareTypes(Node $node) {
210		$userId = $this->userSession->getUser()->getUID();
211		$shareTypes = [];
212		$requestedShareTypes = [
213		\OCP\Share::SHARE_TYPE_USER,
214		\OCP\Share::SHARE_TYPE_GROUP,
215		\OCP\Share::SHARE_TYPE_LINK,
216		\OCP\Share::SHARE_TYPE_REMOTE,
217		\OCP\Share::SHARE_TYPE_EMAIL
218		];
219		foreach ($requestedShareTypes as $requestedShareType) {
220		// one of each type is enough to find out about the types
221		$shares = $this->shareManager->getSharesBy(
222		$userId,
223		$requestedShareType,
224		$node,
225		false,
226		1
227		);
228		if (!empty($shares)) {
229		$shareTypes[] = $requestedShareType;
230		}
231		}
232		return $shareTypes;
233		}
234
235		/**
236		* Change the default sort mode
237		*
238		* @NoAdminRequired
239		*
240		* @param string $mode
241		* @param string $direction
242		* @return Response
243		*/
244		public function updateFileSorting($mode, $direction) {
245		$allowedMode = ['name', 'size', 'mtime'];
246		$allowedDirection = ['asc', 'desc'];
247		if (!in_array($mode, $allowedMode) \|\| !in_array($direction, $allowedDirection)) {
248		$response = new Response();
249		$response->setStatus(Http::STATUS_UNPROCESSABLE_ENTITY);
250		return $response;
251		}
252		$this->config->setUserValue($this->userSession->getUser()->getUID(), 'files', 'file_sorting', $mode);
253		$this->config->setUserValue($this->userSession->getUser()->getUID(), 'files', 'file_sorting_direction', $direction);
254		return new Response();
255		}
256
257		/**
258		* Toggle default for showing/hiding hidden files
259		*
260		* @NoAdminRequired
261		*
262		* @param bool $show
263		*/
264		public function showHiddenFiles($show) {
265		$this->config->setUserValue($this->userSession->getUser()->getUID(), 'files', 'show_hidden', (int)$show);
266		return new Response();
267		}
268
269		/**
270		* Toggle default for showing/hiding xxx folder
271		*
272		* @NoAdminRequired
273		*
274		* @param bool $show
275		* @param bool $key the key of the folder
276		*
277		* @return Response
278		*/
279		public function toggleShowFolder(int $show, string $key) {
280		// ensure the edited key exists
281		$navItems = \OCA\Files\App::getNavigationManager()->getAll();
282		foreach ($navItems as $item) {
283		// check if data is valid
284		if (($show === 0 \|\| $show === 1) && isset($item['expandedState']) && $key === $item['expandedState']) {
285		$this->config->setUserValue($this->userSession->getUser()->getUID(), 'files', $key, (int)$show);
286		return new Response();
287		}
288		}
289		$response = new Response();
290		$response->setStatus(Http::STATUS_FORBIDDEN);
291		return $response;
292		}
293
294
295		}
296

nextcloud / server

Push — master ( a3fdec...f8e9f8 )

ApiController::getReverseQuickaccess() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like