Eccube\Repository\CustomerAddressRepository

Complexity

Total Complexity

4

Size/Duplication

Total Lines	61
Duplicated Lines	37.7 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

90.48%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	deleteByCustomerAndId()	23	23	2
A	findOrCreateByCustomerAndId()	0	22	2

<?php
/*
 * This file is part of EC-CUBE
 *
 * Copyright(c) 2000-2015 LOCKON CO.,LTD. All Rights Reserved.
 *
 * http://www.lockon.co.jp/
 *
 * This program is free software; you can redistribute it and/or
 * modify it under the terms of the GNU General Public License
 * as published by the Free Software Foundation; either version 2
 * of the License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
 * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
 */


namespace Eccube\Repository;

use Doctrine\ORM\EntityRepository;

/**
 * CustomerAddressRepository
 *
 * This class was generated by the Doctrine ORM. Add your own custom
 * repository methods below.
 */
class CustomerAddressRepository extends EntityRepository
{
    /**
     * @param \Eccube\Entity\Customer $Customer
     * @param null $id

Expected 20 spaces after parameter type; 1 found

     * @return \Eccube\Entity\CustomerAddress|mixed
     * @throws \Doctrine\ORM\NoResultException
     * @throws \Doctrine\ORM\NonUniqueResultException
     */
    public function findOrCreateByCustomerAndId(\Eccube\Entity\Customer $Customer, $id = null)
    {
        if (!$id) {
            $CustomerAddress = new \Eccube\Entity\CustomerAddress();
            $CustomerAddress
                ->setCustomer($Customer)
                ->setDelFlg(0);
        } else {
            $qb = $this->createQueryBuilder('od')
                ->andWhere('od.Customer = :Customer AND od.id = :id')
                ->setParameters(array(
                    'Customer' => $Customer,
                    'id' => $id,
                ));

            $CustomerAddress = $qb
                ->getQuery()
                ->getSingleResult();
        }

        return $CustomerAddress;
    }

    /**
     * @param  \Eccube\Entity\Customer $Customer
     * @param  integer                 $id
     * @return bool
     */
    public function deleteByCustomerAndId(\Eccube\Entity\Customer $Customer, $id)

This method seems to be duplicated in your project.

    {
        $qb = $this->createQueryBuilder('od')
            ->andWhere('od.Customer = :Customer AND od.id = :id')
            ->setParameters(array(
                'Customer' => $Customer,
                'id' => $id,
            ));

        try {
            $CustomerAddress = $qb
                ->getQuery()
                ->getSingleResult();
        } catch (\Exception $e) {
            return false;
        }

        $em = $this->getEntityManager();
        $em->remove($CustomerAddress);
        $em->flush();

        return true;
    }
}