WizzardController::dismiss() - Code Metrics - Inspection of "First Start Wizzard #103" - Rello/analytics - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( a242e3...dfddba )

by Marcel

created 2021-02-07 10:23 UTC

WizzardController::dismiss() A

↳ Parent: WizzardController

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	2
eloc	5
c	1
b	0
f	0
nc	2
nop	0
dl	0
loc	8
rs	10

<?php
/**
 * Analytics
 *
 * This file is licensed under the Affero General Public License version 3 or
 * later. See the LICENSE.md file.
 *
 * @author Marcel Scherello <[email protected]>
 * @author Arthur Schiwon <[email protected]>
 * @author Christoph Wurst <[email protected]>
 * @copyright 2020 Marcel Scherello
 */

namespace OCA\Analytics\Controller;

use OCP\AppFramework\Controller;
use OCP\AppFramework\Http\DataResponse;
use OCP\IConfig;
use OCP\IRequest;
use OCP\IUserSession;

class WizzardController extends Controller
{

    /** @var IConfig */
    protected $config;
    /** @var IUserSession */
    private $userSession;
    private $AppName;

    public function __construct(
        string $AppName,
        IRequest $request,
        IUserSession $userSession,
        IConfig $config
    )
    {
        parent::__construct($AppName, $request);
        $this->AppName = $AppName;
        $this->config = $config;
        $this->userSession = $userSession;
    }


    /**
     * @NoAdminRequired
     *
     * @return DataResponse
     * @throws \OCP\PreConditionNotMetException
     */
    public function dismiss(): DataResponse
    {
        $user = $this->userSession->getUser();
        if ($user === null) {
            throw new \RuntimeException("Acting user cannot be resolved");
        }
        $this->config->setUserValue($user->getUID(), $this->AppName, 'wizzard', 1);
        return new DataResponse();
    }
}

1			<?php
2			/**
3			* Analytics
4			*
5			* This file is licensed under the Affero General Public License version 3 or
6			* later. See the LICENSE.md file.
7			*
8			* @author Marcel Scherello <[email protected]>
9			* @author Arthur Schiwon <[email protected]>
10			* @author Christoph Wurst <[email protected]>
11			* @copyright 2020 Marcel Scherello
12			*/
13
14			namespace OCA\Analytics\Controller;
15
16			use OCP\AppFramework\Controller;
17			use OCP\AppFramework\Http\DataResponse;
18			use OCP\IConfig;
19			use OCP\IRequest;
20			use OCP\IUserSession;
21
22			class WizzardController extends Controller
23			{
24
25			/** @var IConfig */
26			protected $config;
27			/** @var IUserSession */
28			private $userSession;
29			private $AppName;
30
31			public function __construct(
32			string $AppName,
33			IRequest $request,
34			IUserSession $userSession,
35			IConfig $config
36			)
37			{
38			parent::__construct($AppName, $request);
39			$this->AppName = $AppName;
40			$this->config = $config;
41			$this->userSession = $userSession;
42			}
43
44
45			/**
46			* @NoAdminRequired
47			*
48			* @return DataResponse
49			* @throws \OCP\PreConditionNotMetException
50			*/
51			public function dismiss(): DataResponse
52			{
53			$user = $this->userSession->getUser();
54			if ($user === null) {
55			throw new \RuntimeException("Acting user cannot be resolved");
56			}
57			$this->config->setUserValue($user->getUID(), $this->AppName, 'wizzard', 1);
58			return new DataResponse();
59			}
60			}

Rello / analytics

Push — master ( a242e3...dfddba )

WizzardController::dismiss() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like