OCA\Ocr\Controller\JobController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	85
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	process()	0	5	1
A	getAllJobs()	0	5	1
A	deleteJob()	0	5	1
A	finish()	0	5	1

<?php

/**
 * Nextcloud - OCR
 *
 * This file is licensed under the Affero General Public License version 3 or
 * later. See the COPYING file.
 *
 * @author Janis Koehr <janiskoehr@icloud.com>
 * @copyright Janis Koehr 2017
 */
namespace OCA\Ocr\Controller;

use OCA\Ocr\Service\JobService;
use OCP\AppFramework\Http\DataResponse;
use OCP\IRequest;
use OCP\AppFramework\Controller;

/**
 * Class JobController
 *
 * @package OCA\Ocr\Controller
 */
class JobController extends Controller {
	
	/**
	 *
	 * @var string
	 */
	private $userId;
	
	/**
	 *
	 * @var OcrService
	 */
	private $service;
	
	use Errors;
	
	/**
	 * JobController constructor.
	 *
	 * @param string $AppName        	
	 * @param IRequest $request        	
	 * @param JobService $service        	
	 * @param
	 *        	$UserId
	 */
	public function __construct($AppName, IRequest $request, JobService $service, $UserId) {
		parent::__construct ( $AppName, $request );
		$this->userId = $UserId;
		$this->service = $service;

It seems like $service of type object<OCA\Ocr\Service\JobService> is incompatible with the declared type object<OCA\Ocr\Controller\OcrService> of property $service.

	}
	
	/**
	 * Processing the srcFile(s)
	 * @NoAdminRequired
	 * 
	 * @param string[] $languages
	 *        	- deu, eng...
	 * @param array $files        	
	 * @return DataResponse
	 */
	public function process($languages, $files) {
		return $this->handleNotFound ( function () use ($languages, $files) {
			return $this->service->process ( $languages, $files );
		} );
	}
	
	/**
	 * @NoAdminRequired
	 *
	 * @return \OCP\AppFramework\Http\DataResponse
	 */
	public function getAllJobs() {
		return $this->handleNotFound ( function () {
			return $this->service->getAllJobsForUser ( $this->userId );
		} );
	}
	
	/**
	 * @NoAdminRequired
	 *
	 * @param integer $id
	 * @return \OCP\AppFramework\Http\DataResponse
	 */
	public function deleteJob($id) {
		return $this->handleNotFound ( function () use ($id) {
			return $this->service->deleteJob ( $id, $this->userId );
		} );
	}
	
	/**
	 * TODO: maybe get the response over redis.. this would make sense because no additional http call is necessary then.
	 * @NoAdminRequired
	 * @NoCSRFRequired
	 *
	 * @param integer $id
	 * @param boolean $failed
	 * @param string $error
	 * @return \OCP\AppFramework\Http\DataResponse
	 */
	public function finish($id, $failed, $error) {
		return $this->handleNotFound ( function () use ($id, $failed, $error) {
			return $this->service->jobFinished ( $id, $failed, $error );
		} );
	}
}