UserService::deleteUserData() - Code Metrics - Inspection of "Merge remote-tracking branch 'origin/master'" - Rello/analytics - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( fc42a6...b0746a )

by Marcel

created 2022-05-29 20:04 UTC

UserService::deleteUserData() A

↳ Parent: UserService

Complexity

Conditions	1
Paths	1

Size

Total Lines	6
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	4
nc	1
nop	1
dl	0
loc	6
rs	10
c	1
b	0
f	0

<?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]>
 * @copyright 2021 Marcel Scherello
 */

namespace OCA\Analytics\Service;

use Psr\Log\LoggerInterface;

class UserService
{
    private $logger;

    public function __construct(
        LoggerInterface $logger,
        ReportService $reportService,
        DatasetService $datasetService
    )
    {
        $this->logger = $logger;
        $this->ReportService = $reportService;

        $this->DatasetService = $datasetService;

    }

    /**
     * delete all user data
     *
     * @param $userId
     * @return bool
     * @throws \OCP\DB\Exception
     */
    public function deleteUserData($userId)
    {
        $this->logger->info('Deleting all Analytics data for: ' . $userId);
        $this->ReportService->deleteByUser($userId);
        $this->DatasetService->deleteByUser($userId);
        return true;
    }
}

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			* @copyright 2021 Marcel Scherello
10			*/
11
12			namespace OCA\Analytics\Service;
13
14			use Psr\Log\LoggerInterface;
15
16			class UserService
17			{
18			private $logger;
19
20			public function __construct(
21			LoggerInterface $logger,
22			ReportService $reportService,
23			DatasetService $datasetService
24			)
25			{
26			$this->logger = $logger;
27			$this->ReportService = $reportService;
			0 ignored issues – show Bug Best Practice introduced 2022-05-29 20:07 UTC by Report Bug Copy Issue Report The property `ReportService` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
28			$this->DatasetService = $datasetService;
			0 ignored issues – show Bug Best Practice introduced 2022-05-29 20:07 UTC by Report Bug Copy Issue Report The property `DatasetService` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
29			}
30
31			/**
32			* delete all user data
33			*
34			* @param $userId
35			* @return bool
36			* @throws \OCP\DB\Exception
37			*/
38			public function deleteUserData($userId)
39			{
40			$this->logger->info('Deleting all Analytics data for: ' . $userId);
41			$this->ReportService->deleteByUser($userId);
42			$this->DatasetService->deleteByUser($userId);
43			return true;
44			}
45			}

Rello / analytics

Push — master ( fc42a6...b0746a )

UserService::deleteUserData() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like