OCA\Passman\Controller\FileController

Complexity

Total Complexity

11

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

45.95%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getFile()	0	3	1
A	deleteFile()	0	3	1
A	__construct()	0	13	1
A	uploadFile()	0	10	1
B	updateFile()	0	18	7

<?php
/**
 * Nextcloud - passman
 *
 * This file is licensed under the Affero General Public License version 3 or
 * later. See the COPYING file.
 *
 * @author Sander Brand <brantje@gmail.com>
 * @copyright Sander Brand 2016
 */

namespace OCA\Passman\Controller;

use OCP\AppFramework\Db\DoesNotExistException;
use OCP\IRequest;
use OCP\AppFramework\Http\JSONResponse;
use OCP\AppFramework\ApiController;
use OCA\Passman\Service\FileService;

class FileController extends ApiController {
	private $userId;
	private $fileService;
	public function __construct($AppName,
								IRequest $request,
								$UserId,
								FileService $fileService){
		parent::__construct(
			$AppName,
			$request,
			'GET, POST, DELETE, PUT, PATCH, OPTIONS',
			'Authorization, Content-Type, Accept',
			86400);
		$this->userId = $UserId;
		$this->fileService = $fileService;
	}


	/**
	 * @NoAdminRequired
	 * @NoCSRFRequired
	 */
	public function uploadFile($data, $filename, $mimetype, $size) {
		$file = array(
			'filename' => $filename,
			'size' => $size,
			'mimetype' => $mimetype,
			'file_data' => $data,
			'user_id' => $this->userId
		);
		return new JSONResponse($this->fileService->createFile($file, $this->userId));
	}

	/**
	 * @NoAdminRequired
	 * @NoCSRFRequired
	 */
	public function getFile($file_id) {
		return new JSONResponse($this->fileService->getFile($file_id, $this->userId));
	}
	/**
	 * @NoAdminRequired
	 * @NoCSRFRequired
	 */
	public function deleteFile($file_id) {
		return new JSONResponse($this->fileService->deleteFile($file_id, $this->userId));
	}

	public function updateFile($file_id, $file_data, $filename){
		try{
			$file = $this->fileService->getFile($file_id, $this->userId);
		} catch (\Exception $doesNotExistException){

Consider adding a comment why this CATCH block is empty.

		}
		if($file){
			if($file_data) {
				$file->setFileData($file_data);
			}
			if($filename) {
				$file->setFilename($filename);
			}
			if($filename || $file_data){
				new JSONResponse($this->fileService->updateFile($file));
			}
		}
	}
}