OCA\Chat\Controller\ConfigController

Complexity

Total Complexity

8

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	set()	0	10	3
A	enableBackend()	0	8	2
A	disableBackend()	0	8	2

<?php

namespace OCA\Chat\Controller;

use \OCA\Chat\Db\ConfigMapper;
use \OCA\Chat\IBackendManager;
use \OCP\AppFramework\Controller;
use \OCP\AppFramework\Http\JSONResponse;
use \OCP\AppFramework\Http;
use \OCP\IRequest;
use \OCA\Chat\BackendNotFoundException;


class ConfigController extends Controller {

	/**
	 * @var \OCA\Chat\Db\ConfigMapper
	 */
	private $configMapper;

	/**
	 * @var \OCA\Chat\BackendManager
	 */
	private $backendManager;

	public function __construct($appName, IRequest $request, ConfigMapper $configMapper, IBackendManager $backendManager){
		parent::__construct($appName, $request);
		$this->configMapper = $configMapper;
		$this->backendManager = $backendManager;

$backendManager is of type object<OCA\Chat\IBackendManager>, but the property $backendManager was declared to be of type object<OCA\Chat\BackendManager>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

	}

	/**
	 * @NoAdminRequired
	 * @param $backends
	 * @return JSONResponse
	 */
	public function set($backends){
		foreach($backends as $backend){
			foreach($backend['config'] as $key=>$value){
				$this->configMapper->set($backend['id'], $key, $value);
			}
		}
		$res = new JSONResponse();
		$res->setStatus(Http::STATUS_OK);
		return $res;
	}

	/**
	 * @param $id
	 * @return JSONResponse
	 */
	public function enableBackend($id){
		try {
			$this->backendManager->enableBackend($id);
			return new JSONResponse(array("status" => "success"));
		} Catch (BackendNotFoundException $e) {
			return new JSONResponse(array("status" => "error", "msg" => 404));
		}
	}

	/**
	 * @param $id
	 * @return JSONResponse
	 */
	public function disableBackend($id){
		try {
			$this->backendManager->disableBackend($id);
			return new JSONResponse(array("status" => "success"));
		} Catch (BackendNotFoundException $e) {
			return new JSONResponse(array("status" => "error", "msg" => 404));
		}
	}
}