TransactionLineRepository::getAccessibleSubQuery() - Code Metrics - Inspection of "Prevent removing the last transactionLine of a tra..." - Ecodev/my-ichtus - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 4e7aa8...dae217 )

by Adrien

created 2020-12-29 09:56 UTC

TransactionLineRepository::getAccessibleSubQuery() A

↳ Parent: TransactionLineRepository

Complexity

Conditions	4
Paths	4

Size

Total Lines	19
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	4

Importance

Changes

Metric	Value
cc	4
eloc	10
nc	4
nop	1
dl	0
loc	19
rs	9.9332
c	0
b	0
f	0
ccs	9
cts	9
cp	1
crap	4

<?php

declare(strict_types=1);

namespace Application\Repository;

use Application\Handler\ExportTransactionLinesHandler;
use Application\Model\User;
use Doctrine\ORM\Query;
use Ecodev\Felix\Repository\LimitedAccessSubQuery;

class TransactionLineRepository extends AbstractRepository implements ExportExcelInterface, LimitedAccessSubQuery
{
    /**
     * Returns pure SQL to get ID of all objects that are accessible to given user.
     *
     * @param null|User $user
     */
    public function getAccessibleSubQuery(?\Ecodev\Felix\Model\User $user): string
    {
        if (!$user) {
            return '-1';
        }

        if (in_array($user->getRole(), [User::ROLE_RESPONSIBLE, User::ROLE_ADMINISTRATOR], true)) {
            return $this->getAllIdsQuery();
        }

        if ($user->getOwner()) {
            $id = $user->getOwner()->getId();
        } else {
            $id = $user->getId();
        }

        return 'SELECT transaction_line.id FROM transaction_line
              JOIN account ON transaction_line.debit_id = account.id OR transaction_line.credit_id = account.id 
              WHERE account.owner_id = ' . $id;
    }

    /**
     * Generates an Excel spreadsheet with the query result
     *
     * @return string name of the temporary file
     */
    public function exportExcel(Query $query): string
    {
        global $container;

        return ($container->get(ExportTransactionLinesHandler::class))->generate($query);
    }

    public function importedIdExists(string $importedId): bool
    {
        $connection = $this->getEntityManager()->getConnection();
        $count = $connection->fetchOne('SELECT COUNT(*) > 0 FROM transaction_line WHERE imported_id = :importedId', ['importedId' => $importedId]);

        return (bool) $count;
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Application\Repository;
6
7		use Application\Handler\ExportTransactionLinesHandler;
8		use Application\Model\User;
9		use Doctrine\ORM\Query;
10		use Ecodev\Felix\Repository\LimitedAccessSubQuery;
11
12		class TransactionLineRepository extends AbstractRepository implements ExportExcelInterface, LimitedAccessSubQuery
13		{
14		/**
15		* Returns pure SQL to get ID of all objects that are accessible to given user.
16		*
17		* @param null\|User $user
18		*/
19	14	public function getAccessibleSubQuery(?\Ecodev\Felix\Model\User $user): string
20		{
21	14	if (!$user) {
22	2	return '-1';
23		}
24
25	12	if (in_array($user->getRole(), [User::ROLE_RESPONSIBLE, User::ROLE_ADMINISTRATOR], true)) {
26	9	return $this->getAllIdsQuery();
27		}
28
29	3	if ($user->getOwner()) {
30	1	$id = $user->getOwner()->getId();
31		} else {
32	2	$id = $user->getId();
33		}
34
35		return 'SELECT transaction_line.id FROM transaction_line
36		JOIN account ON transaction_line.debit_id = account.id OR transaction_line.credit_id = account.id
37	3	WHERE account.owner_id = ' . $id;
38		}
39
40		/**
41		* Generates an Excel spreadsheet with the query result
42		*
43		* @return string name of the temporary file
44		*/
45		public function exportExcel(Query $query): string
46		{
47		global $container;
48
49		return ($container->get(ExportTransactionLinesHandler::class))->generate($query);
50		}
51
52	4	public function importedIdExists(string $importedId): bool
53		{
54	4	$connection = $this->getEntityManager()->getConnection();
55	4	$count = $connection->fetchOne('SELECT COUNT(*) > 0 FROM transaction_line WHERE imported_id = :importedId', ['importedId' => $importedId]);
56
57	4	return (bool) $count;
58		}
59		}
60

Ecodev / my-ichtus

Push — master ( 4e7aa8...dae217 )

TransactionLineRepository::getAccessibleSubQuery() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like