TransactionLineRepository::getAccessibleSubQuery() - Code Metrics - Inspection of "Add test for expenseClaim marked as processed afte..." - Ecodev/my-ichtus - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — master ( 472bc6...94471a )

by Sylvain

created 2020-06-19 13:37 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\Action\ExportTransactionLinesAction;
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(ExportTransactionLinesAction::class))->generate($query);
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Application\Repository;
6
7		use Application\Action\ExportTransactionLinesAction;
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	13	public function getAccessibleSubQuery(?\Ecodev\Felix\Model\User $user): string
20		{
21	13	if (!$user) {
22	2	return '-1';
23		}
24
25	11	if (in_array($user->getRole(), [User::ROLE_RESPONSIBLE, User::ROLE_ADMINISTRATOR], true)) {
26	8	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(ExportTransactionLinesAction::class))->generate($query);
50		}
51		}
52

Ecodev / my-ichtus

Push — master ( 472bc6...94471a )

TransactionLineRepository::getAccessibleSubQuery() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like