LogRepository::getAccessibleSubQuery() - Code Metrics - Inspection of "bionic !!!" - Ecodev/my-ichtus - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — master ( a427b2...509ea4 )

by Adrien

created 2022-02-21 08:40 UTC

LogRepository::getAccessibleSubQuery() A

↳ Parent: LogRepository

Complexity

Conditions	3
Paths	3

Size

Total Lines	17
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	4.5435

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	8
dl	0
loc	17
ccs	4
cts	9
cp	0.4444
rs	10
c	1
b	0
f	0
cc	3
nc	3
nop	1
crap	4.5435

<?php

declare(strict_types=1);

namespace Application\Repository;

use Application\Model\User;
use Ecodev\Felix\Repository\LimitedAccessSubQuery;

class LogRepository extends AbstractRepository implements \Ecodev\Felix\Repository\LogRepository, LimitedAccessSubQuery
{
    /**
     * Log message to be used when a door is opened.
     */
    final public const DOOR_OPENED = 'door opened: ';
    /**
     * Log message to be used when the datatrans webhook starts.
     */
    final public const DATATRANS_WEBHOOK_BEGIN = 'datatrans webhook begin';
    /**
     * Log message to be used when the datatrans webhook finishes.
     */
    final public const DATATRANS_WEBHOOK_END = 'datatrans webhook end';

    use \Ecodev\Felix\Repository\Traits\LogRepository;

    /**
     * Returns pure SQL to get ID of all objects that are accessible to given user.
     */
    public function getAccessibleSubQuery(?\Ecodev\Felix\Model\User $user): string
    {
        if (!$user) {
            return '-1';
        }

        // Sysops and responsible can read all logs
        if (in_array($user->getRole(), [User::ROLE_RESPONSIBLE, User::ROLE_ADMINISTRATOR], true)) {
            return $this->getAllIdsQuery();
        }

        $subquery = '
            SELECT log.id FROM `log` WHERE
            message LIKE ' . $this->getEntityManager()->getConnection()->quote(self::DOOR_OPENED . '%') . '
            AND log.creator_id = ' . $user->getId();

        return $subquery;
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Application\Repository;
6
7		use Application\Model\User;
8		use Ecodev\Felix\Repository\LimitedAccessSubQuery;
9
10		class LogRepository extends AbstractRepository implements \Ecodev\Felix\Repository\LogRepository, LimitedAccessSubQuery
11		{
12		/**
13		* Log message to be used when a door is opened.
14		*/
15		final public const DOOR_OPENED = 'door opened: ';
16		/**
17		* Log message to be used when the datatrans webhook starts.
18		*/
19		final public const DATATRANS_WEBHOOK_BEGIN = 'datatrans webhook begin';
20		/**
21		* Log message to be used when the datatrans webhook finishes.
22		*/
23		final public const DATATRANS_WEBHOOK_END = 'datatrans webhook end';
24
25		use \Ecodev\Felix\Repository\Traits\LogRepository;
26
27		/**
28		* Returns pure SQL to get ID of all objects that are accessible to given user.
29		*/
30	1	public function getAccessibleSubQuery(?\Ecodev\Felix\Model\User $user): string
31		{
32	1	if (!$user) {
33		return '-1';
34		}
35
36		// Sysops and responsible can read all logs
37	1	if (in_array($user->getRole(), [User::ROLE_RESPONSIBLE, User::ROLE_ADMINISTRATOR], true)) {
38	1	return $this->getAllIdsQuery();
39		}
40
41		$subquery = '
42		SELECT log.id FROM `log` WHERE
43		message LIKE ' . $this->getEntityManager()->getConnection()->quote(self::DOOR_OPENED . '%') . '
44		AND log.creator_id = ' . $user->getId();
45
46		return $subquery;
47		}
48		}
49

Ecodev / my-ichtus

Push — master ( a427b2...509ea4 )

LogRepository::getAccessibleSubQuery() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like