Ownnotev2ApiController - Code Metrics - Inspection of "Backend refactor" - brantje/nextnote - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Pull Request — master (#6)

by Sander

created 2017-07-03 17:22 UTC

Ownnotev2ApiController A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	92
Duplicated Lines	17.39 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	16
loc	92
rs	10
wmc	9
lcom	1
cbo	3

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	index()	0	5	1
A	create()	0	10	1
A	update()	0	17	2
A	get()	7	7	2
A	delete()	9	9	2

How to fix Duplicated Code

<?php
/**
 * Nextcloud - ownnote
 *
 * @copyright Copyright (c) 2015, Ben Curtis <[email protected]>
 * @copyright Copyright (c) 2017, Sander Brand ([email protected])
 * @license GNU AGPL version 3 or any later version
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License as
 * published by the Free Software Foundation, either version 3 of the
 * License, or (at your option) any later version.
 *
 * 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
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *
 */

namespace OCA\OwnNote\Controller;

use OCA\OwnNote\Service\OwnNoteService;
use OCA\OwnNote\Utility\NotFoundJSONResponse;
use \OCP\AppFramework\ApiController;
use OCP\AppFramework\Http\JSONResponse;
use OCP\IConfig;
use OCP\ILogger;
use \OCP\IRequest;
use \OCA\OwnNote\Lib\Backend;


class Ownnotev2ApiController extends ApiController {

	private $backend;
	private $config;
	private $noteService;

	public function __construct($appName, IRequest $request, ILogger $logger, IConfig $config, OwnNoteService $noteService) {
		parent::__construct($appName, $request);
		$this->backend = new Backend($config);
		$this->config = $config;
		$this->noteService = $noteService;
	}

	/**
	 * MOBILE FUNCTIONS
	 */

	/**
	 * @NoAdminRequired
	 * @NoCSRFRequired
	 */
	public function index() {
		$uid = \OC::$server->getUserSession()->getUser()->getUID();
		$results = $this->noteService->findNotesFromUser($uid, false);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
		return new JSONResponse($results);
	}

	/**
	 * @NoAdminRequired
	 * @NoCSRFRequired
	 */
	public function get($id) {
		$results = $this->noteService->find($id);
		if (!$results) {
			return new NotFoundJSONResponse();
		}
		return new JSONResponse($results);
	}


	/**
	 * @NoAdminRequired
	 * @NoCSRFRequired
	 */
	public function create($title, $group, $note) {
		$note = [
			'title' => $title,
			'group' => $group,
			'note' => $note
		];
		$uid = \OC::$server->getUserSession()->getUser()->getUID();
		$result = $this->noteService->create($note, $uid);
		return new JSONResponse($result);
	}

	/**
	 * @NoAdminRequired
	 * @NoCSRFRequired
	 */
	public function update($id, $title, $group, $content) {

		$note = [
			'id' => $id,
			'title' => $title,
			'group' => $group,
			'note' => $content
		];

		$entity = $this->noteService->find($id);
		if (!$entity) {
			return new NotFoundJSONResponse();
		}

		$results = $this->noteService->update($note);
		return new JSONResponse($results);
	}

	/**
	 * @NoAdminRequired
	 * @NoCSRFRequired
	 */
	public function delete($id) {
		$entity = $this->noteService->find($id);
		if (!$entity) {
			return new NotFoundJSONResponse();
		}

		$results = $this->noteService->delete($id);
		return new JSONResponse($results);
	}

}


1		<?php
2		/**
3		* Nextcloud - ownnote
4		*
5		* @copyright Copyright (c) 2015, Ben Curtis <[email protected]>
6		* @copyright Copyright (c) 2017, Sander Brand ([email protected])
7		* @license GNU AGPL version 3 or any later version
8		*
9		* This program is free software: you can redistribute it and/or modify
10		* it under the terms of the GNU Affero General Public License as
11		* published by the Free Software Foundation, either version 3 of the
12		* License, or (at your option) any later version.
13		*
14		* This program is distributed in the hope that it will be useful,
15		* but WITHOUT ANY WARRANTY; without even the implied warranty of
16		* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
17		* GNU Affero General Public License for more details.
18		*
19		* You should have received a copy of the GNU Affero General Public License
20		* along with this program. If not, see <http://www.gnu.org/licenses/>.
21		*
22		*/
23
24		namespace OCA\OwnNote\Controller;
25
26		use OCA\OwnNote\Service\OwnNoteService;
27		use OCA\OwnNote\Utility\NotFoundJSONResponse;
28		use \OCP\AppFramework\ApiController;
29		use OCP\AppFramework\Http\JSONResponse;
30		use OCP\IConfig;
31		use OCP\ILogger;
32		use \OCP\IRequest;
33		use \OCA\OwnNote\Lib\Backend;
34
35
36		class Ownnotev2ApiController extends ApiController {
37
38		private $backend;
39		private $config;
40		private $noteService;
41
42		public function __construct($appName, IRequest $request, ILogger $logger, IConfig $config, OwnNoteService $noteService) {
43		parent::__construct($appName, $request);
44		$this->backend = new Backend($config);
45		$this->config = $config;
46		$this->noteService = $noteService;
47		}
48
49		/**
50		* MOBILE FUNCTIONS
51		*/
52
53		/**
54		* @NoAdminRequired
55		* @NoCSRFRequired
56		*/
57		public function index() {
58		$uid = \OC::$server->getUserSession()->getUser()->getUID();
59		$results = $this->noteService->findNotesFromUser($uid, false);
		0 ignored issues – show Documentation introduced 2017-07-03 17:24 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `integer`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
60		return new JSONResponse($results);
61		}
62
63		/**
64		* @NoAdminRequired
65		* @NoCSRFRequired
66		*/
67	View Code Duplication	public function get($id) {
68		$results = $this->noteService->find($id);
69		if (!$results) {
70		return new NotFoundJSONResponse();
71		}
72		return new JSONResponse($results);
73		}
74
75
76		/**
77		* @NoAdminRequired
78		* @NoCSRFRequired
79		*/
80		public function create($title, $group, $note) {
81		$note = [
82		'title' => $title,
83		'group' => $group,
84		'note' => $note
85		];
86		$uid = \OC::$server->getUserSession()->getUser()->getUID();
87		$result = $this->noteService->create($note, $uid);
88		return new JSONResponse($result);
89		}
90
91		/**
92		* @NoAdminRequired
93		* @NoCSRFRequired
94		*/
95		public function update($id, $title, $group, $content) {
96
97		$note = [
98		'id' => $id,
99		'title' => $title,
100		'group' => $group,
101		'note' => $content
102		];
103
104		$entity = $this->noteService->find($id);
105		if (!$entity) {
106		return new NotFoundJSONResponse();
107		}
108
109		$results = $this->noteService->update($note);
110		return new JSONResponse($results);
111		}
112
113		/**
114		* @NoAdminRequired
115		* @NoCSRFRequired
116		*/
117	View Code Duplication	public function delete($id) {
118		$entity = $this->noteService->find($id);
119		if (!$entity) {
120		return new NotFoundJSONResponse();
121		}
122
123		$results = $this->noteService->delete($id);
124		return new JSONResponse($results);
125		}
126
127		}
128

brantje / nextnote

Pull Request — master (#6)

Ownnotev2ApiController A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

6 Methods

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like