UpdateTask::__invoke() - Code Metrics - Inspection of "Cambiando rutas y nombre de metodos en controller..." - maurobonfietti/api-rest-slimphp - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( cacde2...cef74a )

by Mauro

created 2017-07-22 14:27 UTC

UpdateTask::__invoke() A

↳ Parent: UpdateTask

Complexity

Conditions	2
Paths	5

Size

Total Lines	12
Code Lines	8

Duplication

Lines	12
Ratio	100 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	12
loc	12
rs	9.4285
cc	2
eloc	8
nc	5
nop	3

<?php

namespace App\Controller\Task;

use Slim\Http\Request;
use Slim\Http\Response;

/**
 * Update Task Controller.
 */
class UpdateTask extends BaseTask
{
    /**
     * Update a task.
     *
     * @param Request $request
     * @param Response $response
     * @param array $args
     * @return array
     */
    public function __invoke($request, $response, $args)
    {
        try {
            $this->setParams($request, $response, $args);
            $input = $this->request->getParsedBody();
            $result = $this->getTaskService()->updateTask($input, $this->args['id']);
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}

            return $this->jsonResponse('success', $result, 200);
        } catch (\Exception $ex) {
            return $this->jsonResponse('error', $ex->getMessage(), $ex->getCode());
        }
    }
}


1		<?php
2
3		namespace App\Controller\Task;
4
5		use Slim\Http\Request;
6		use Slim\Http\Response;
7
8		/**
9		* Update Task Controller.
10		*/
11	View Code Duplication	class UpdateTask extends BaseTask
12		{
13		/**
14		* Update a task.
15		*
16		* @param Request $request
17		* @param Response $response
18		* @param array $args
19		* @return array
20		*/
21		public function __invoke($request, $response, $args)
22		{
23		try {
24		$this->setParams($request, $response, $args);
25		$input = $this->request->getParsedBody();
26		$result = $this->getTaskService()->updateTask($input, $this->args['id']);
		0 ignored issues – show Bug introduced 2017-06-26 22:33 UTC by Report Bug Copy Issue Report It seems like `$input` defined by `$this->request->getParsedBody()` on line `25` can also be of type `null` or `object`; however, `App\Service\TaskService::updateTask()` does only seem to accept `array`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
27
28		return $this->jsonResponse('success', $result, 200);
29		} catch (\Exception $ex) {
30		return $this->jsonResponse('error', $ex->getMessage(), $ex->getCode());
31		}
32		}
33		}
34

maurobonfietti / api-rest-slimphp

Push — master ( cacde2...cef74a )

UpdateTask::__invoke() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like