OCA\Maps\Controller\ApiKeyController

Complexity

Total Complexity

8

Size/Duplication

Total Lines	83
Duplicated Lines	19.28 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	updateKey()	16	16	2
A	addKey()	0	11	1
A	getKey()	0	9	2
A	removeApiKey()	0	7	2

<?php
/**
 * ownCloud - maps
 *
 * This file is licensed under the Affero General Public License version 3 or
 * later. See the COPYING file.
 *
 * @author Vinzenz Rosenkranz <vinzenz.rosenkranz@gmail.com>
 * @copyright Vinzenz Rosenkranz 2015
 */

namespace OCA\Maps\Controller;

use OCA\Maps\Db\ApiKey;
use OCA\Maps\Db\ApiKeyMapper;
use \OCP\IRequest;
use \OCP\AppFramework\Http\JSONResponse;
use \OCP\AppFramework\ApiController;


class ApiKeyController extends ApiController {

	private $userId;
	private $apiKeyMapper;

	public function __construct($appName, IRequest $request, ApiKeyMapper $apiKeyMapper, $userId) {
		parent::__construct($appName, $request);
		$this->apiKeyMapper = $apiKeyMapper;
		$this->userId = $userId;
	}

	/**
	 * @NoAdminRequired
	 *
	 * @param $key string
	 * @param $id int
	 * @return JSONResponse
	 */
	public function updateKey($key, $id) {

This method seems to be duplicated in your project.

		$apikey = new ApiKey();
		$apikey->setId($id);
		$apikey->setApiKey($key);

		/* Only save apiKey if it exists in db */
		try {
			$this->apiKeyMapper->find($id);
			return new JSONResponse($this->apiKeyMapper->update($apikey));
		} catch(\OCP\AppFramework\Db\DoesNotExistException $e) {

The class OCP\AppFramework\Db\DoesNotExistException does not exist. Did you forget a USE statement, or did you not list all dependencies?

			return new JSONResponse([
				'error' => $e->getMessage()
			]);
		}
	}

	/**
	 * @NoAdminRequired
	 *
	 * @param $key string
	 * @return JSONResponse
	 */
	public function addKey($key){
		$apikey = new ApiKey();
		$apikey->setApiKey($key);
		$apikey->setUserId($this->userId);

		/* @var $apikey ApiKey */
		$apikey = $this->apiKeyMapper->insert($apikey);

		$response = array('id'=> $apikey->getId());
		return new JSONResponse($response);
	}

	/**
	 * @NoAdminRequired
	 *
	 * @return JSONResponse
	 */
	public function getKey(){
		$apikey = new ApiKey();
		try {
			$apikey = $this->apiKeyMapper->findByUser($this->userId);
		} catch(\OCP\AppFramework\Db\DoesNotExistException $e) {

The class OCP\AppFramework\Db\DoesNotExistException does not exist. Did you forget a USE statement, or did you not list all dependencies?

			$apikey->setUserId($this->userId);
		}
		return new JSONResponse($apikey);
	}

	/**
	 * @NoAdminRequired
	 *
	 * @param $id int
	 * @return JSONResponse
	 */
	public function removeApiKey($id){
		$apikey = $this->apiKeyMapper->find($id);
		if($apikey->userId == $this->userId) {
			$this->apiKeyMapper->delete($apikey);
		}
		return new JSONResponse();
	}

}