InstitutionWithPersonalRaDetailsRepository::addIfNotExists() - Code Metrics - Inspection of "Add controller for institutions with personal RA d..." - OpenConext/Stepup-Middleware - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — feature/institutions-with-pers... ( 5416f5 )

by A.

created 2016-06-20 09:42 UTC

addIfNotExists() A

↳ Parent: InstitutionWithPersonalRaDetailsRepository

Complexity

Conditions	2
Paths	2

Size

Total Lines	17
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
c	1
b	0
f	1
dl	0
loc	17
rs	9.4285
cc	2
eloc	12
nc	2
nop	1

<?php

/**
 * Copyright 2016 SURFnet B.V.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

namespace Surfnet\StepupMiddleware\ApiBundle\Configuration\Repository;

use Doctrine\ORM\EntityRepository;
use Surfnet\StepupMiddleware\ApiBundle\Configuration\Entity\InstitutionWithPersonalRaDetails;

final class InstitutionWithPersonalRaDetailsRepository extends EntityRepository
{
    /**
     * @param InstitutionWithPersonalRaDetails $institutionWithPersonalRaDetails
     */
    public function addIfNotExists(
        InstitutionWithPersonalRaDetails $institutionWithPersonalRaDetails

    ) {
        $existsQuery = $this->createQueryBuilder('ipr')
            ->where('ipr.institution = :institution')
            ->setParameter('institution', $institutionWithPersonalRaDetails->institution)
            ->getQuery()
            ->getOneOrNullResult();

        if ($existsQuery) {
            return;
        }

        $em = $this->getEntityManager();
        $em->persist($institutionWithPersonalRaDetails);
        $em->flush($institutionWithPersonalRaDetails);
    }
}


1			<?php
2
3			/**
4			* Copyright 2016 SURFnet B.V.
5			*
6			* Licensed under the Apache License, Version 2.0 (the "License");
7			* you may not use this file except in compliance with the License.
8			* You may obtain a copy of the License at
9			*
10			* http://www.apache.org/licenses/LICENSE-2.0
11			*
12			* Unless required by applicable law or agreed to in writing, software
13			* distributed under the License is distributed on an "AS IS" BASIS,
14			* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
15			* See the License for the specific language governing permissions and
16			* limitations under the License.
17			*/
18
19			namespace Surfnet\StepupMiddleware\ApiBundle\Configuration\Repository;
20
21			use Doctrine\ORM\EntityRepository;
22			use Surfnet\StepupMiddleware\ApiBundle\Configuration\Entity\InstitutionWithPersonalRaDetails;
23
24			final class InstitutionWithPersonalRaDetailsRepository extends EntityRepository
25			{
26			/**
27			* @param InstitutionWithPersonalRaDetails $institutionWithPersonalRaDetails
28			*/
29			public function addIfNotExists(
30			InstitutionWithPersonalRaDetails $institutionWithPersonalRaDetails
			0 ignored issues – show Comprehensibility Naming introduced 2016-06-20 09:47 UTC by Report Bug Copy Issue Report The variable name `$institutionWithPersonalRaDetails` exceeds the maximum configured length of `30`. Very long variable names usually make code harder to read. It is therefore recommended not to make variable names too verbose. Loading history...
31			) {
32			$existsQuery = $this->createQueryBuilder('ipr')
33			->where('ipr.institution = :institution')
34			->setParameter('institution', $institutionWithPersonalRaDetails->institution)
35			->getQuery()
36			->getOneOrNullResult();
37
38			if ($existsQuery) {
39			return;
40			}
41
42			$em = $this->getEntityManager();
43			$em->persist($institutionWithPersonalRaDetails);
44			$em->flush($institutionWithPersonalRaDetails);
45			}
46			}
47

OpenConext / Stepup-Middleware

Push — feature/institutions-with-pers... ( 5416f5 )

addIfNotExists() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like