TaxRuleRepository::delete() - Code Metrics - Inspection of "TransactionListener の追加" - EC-CUBE/ec-cube - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#1632)

by Kentaro

created 2016-07-26 02:39 UTC

TaxRuleRepository::delete() A

↳ Parent: TaxRuleRepository

Complexity

Conditions	3
Paths	4

Size

Total Lines	13
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	3

Importance

Changes

Metric	Value
dl	0
loc	13
rs	9.4285
c	0
b	0
f	0
ccs	4
cts	4
cp	1
cc	3
eloc	9
nc	4
nop	1
crap	3

<?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;
use Doctrine\ORM\NoResultException;
use Eccube\Common\Constant;

/**
 * TaxRuleRepository
 *
 * This class was generated by the Doctrine ORM. Add your own custom
 * repository methods below.
 */
class TaxRuleRepository extends EntityRepository
{
    private $rules = array();

    protected $app;

    public function setApplication($app)

    {
        $this->app = $app;
    }

    public function newTaxRule()

    {
        $TaxRule = new \Eccube\Entity\TaxRule();
        $CalcRule = $this->getEntityManager()
            ->getRepository('Eccube\Entity\Master\Taxrule')
            ->find(1);
        $TaxRule->setCalcRule($CalcRule);
        $TaxRule->setTaxAdjust(0);
        $TaxRule->setDelFlg(0);

        return $TaxRule;
    }

    /**
     * 現在有効な税率設定情報を返す
     *
     * @param  int|null|\Eccube\Entity\Product        $Product      商品
     * @param  int|null|\Eccube\Entity\ProductClass   $ProductClass 商品規格
     * @param  int|null|\Eccube\Entity\Master\Pref    $Pref         都道府県
     * @param  int|null|\Eccube\Entity\Master\Country $Country      国
     * @return \Eccube\Entity\TaxRule                 税設定情報
     *
     * @throws NoResultException
     */
    public function getByRule($Product = null, $ProductClass = null, $Pref = null, $Country = null)
    {
        if (!$this->app) {
            throw new \LogicException();
        }

        // Pref Country 設定
        if (!$Pref && !$Country && $this->app['security']->getToken() && $this->app['security']->isGranted('ROLE_USER')) {
            /* @var $Customer \Eccube\Entity\Customer */
            $Customer = $this->app['security']->getToken()->getUser();
            $Pref = $Customer->getPref();
            $Country = $Customer->getCountry();
        }

        // 商品単位税率設定がOFFの場合
        /** @var $BaseInfo \Eccube\Entity\BaseInfo */
        $BaseInfo = $this->app['eccube.repository.base_info']->get();
        if ($BaseInfo->getOptionProductTaxRule() !== Constant::ENABLED) {
            $Product = null;
            $ProductClass = null;
        }

        // Cache Key 設定
        if ($Product instanceof \Eccube\Entity\Product) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
            $productId = $Product->getId();
        } elseif ($Product) {
0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
            $productId = $Product;
        } else {
            $productId = '0';
        }
        if ($ProductClass instanceof \Eccube\Entity\ProductClass) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
            $productClassId = $ProductClass->getId();
        } elseif ($ProductClass) {
0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
            $productClassId = $ProductClass;
        } else {
            $productClassId = '0';
        }
        if ($Pref instanceof \Eccube\Entity\Master\Pref) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
            $prefId = $Pref->getId();
        } elseif ($Pref) {
            $prefId = $Pref;
        } else {
            $prefId = '0';
        }
        if ($Country instanceof \Eccube\Entity\Master\Country) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
            $countryId = $Country->getId();
        } elseif ($Country) {
            $countryId = $Country;
        } else {
            $countryId = '0';
        }
        $cacheKey = $productId.':'.$productClassId.':'.$prefId.':'.$countryId;

        // すでに取得している場合はキャッシュから
        if (isset($this->rules[$cacheKey])) {
            return $this->rules[$cacheKey];
        }

        $parameters = array();
        $qb = $this->createQueryBuilder('t')
            ->where('t.apply_date < CURRENT_TIMESTAMP()');

        // Pref
        if ($Pref) {
            $qb->andWhere('t.Pref IS NULL OR t.Pref = :Pref');
            $parameters['Pref'] = $Pref;
        } else {
            $qb->andWhere('t.Pref IS NULL');
        }

        // Country
        if ($Country) {
            $qb->andWhere('t.Country IS NULL OR t.Country = :Country');
            $parameters['Country'] = $Country;
        } else {
            $qb->andWhere('t.Country IS NULL');
        }

        /*
         * Product, ProductClass が persist される前に TaxRuleEventSubscriber によってアクセスされる
         * 場合があるため、ID の存在もチェックする.
         * https://github.com/EC-CUBE/ec-cube/issues/677
         */

        // Product
        if ($Product && $productId > 0) {
            $qb->andWhere('t.Product IS NULL OR t.Product = :Product');
            $parameters['Product'] = $Product;
        } else {
            $qb->andWhere('t.Product IS NULL');
        }

        // ProductClass
        if ($ProductClass && $productClassId > 0) {
            $qb->andWhere('t.ProductClass IS NULL OR t.ProductClass = :ProductClass');
            $parameters['ProductClass'] = $ProductClass;
        } else {
            $qb->andWhere('t.ProductClass IS NULL');
        }

        $TaxRules = $qb
            ->setParameters($parameters)
            ->orderBy('t.apply_date', 'DESC') // 実際は usort() でソートする
            ->getQuery()
            ->getResult();

        // 地域設定を優先するが、システムパラメーターなどに設定を持っていくか
        // 後に書いてあるほど優先される
        $priorityKeys = explode(',', $this->app['config']['tax_rule_priority']);
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        $priorityKeys = array();
        foreach (explode(',', $this->app['config']['tax_rule_priority']) as $key) {
            $priorityKeys[] = str_replace('_', '', preg_replace('/_id\z/', '', $key));
        }

        foreach ($TaxRules as $TaxRule) {
            $rank = 0;
            foreach ($priorityKeys as $index => $key) {
                $arrayProperties = array_change_key_case($TaxRule->toArray());
                if ($arrayProperties[$key]) {


                    // 配列の数値添字を重みとして利用する
                    $rank += 1 << ($index + 1);
                }
            }
            $TaxRule->setRank($rank);
        }

        // 適用日降順, rank 降順にソートする
        usort($TaxRules, function($a, $b) {

            return $a->compareTo($b);
        });

        if (!empty($TaxRules)) {
            $this->rules[$cacheKey] = $TaxRules[0];

            return $TaxRules[0];
        } else {
            throw new NoResultException();
        }
    }

    /**
     * getList
     *
     * @return array|null
     */
    public function getList()
    {
        $qb = $this->createQueryBuilder('t')
            ->orderBy('t.apply_date', 'DESC')
            ->where('t.Product IS NULL AND t.ProductClass IS NULL');
        $TaxRules = $qb
            ->getQuery()
            ->getResult();

        return $TaxRules;
    }

    /**
     * getById
     *
     * @param  int   $id

     * @return array
     */
    public function getById($id)
    {
        $criteria = array(
            'id' => $id,
        );

        return $this->findOneBy($criteria);
    }

    /**
     * getByTime
     *
     * @param  string $applyDate
     * @return mixed
     */
    public function getByTime($applyDate)
    {
        $criteria = array(
            'apply_date' => $applyDate,
        );

        return $this->findOneBy($criteria);
    }

    /**
     * 税規約の削除.
     *
     * @param  int|\Eccube\Entity\TaxRule $TaxRule 税規約
     * @return void
     * @throws NoResultException
     */
    public function delete($TaxRule)
    {
        if (!$TaxRule instanceof \Eccube\Entity\TaxRule) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
            $TaxRule = $this->find($TaxRule);
        }
        if (!$TaxRule) {
            throw new NoResultException;

        }
        $TaxRule->setDelFlg(1);
        $em = $this->getEntityManager();
        $em->persist($TaxRule);
        $em->flush();
    }

    /**
     * TaxRule のキャッシュをクリアする.
     *
     * getByRule() をコールすると、結果をキャッシュし、2回目以降はデータベースへアクセスしない.
     * このメソッドをコールすると、キャッシュをクリアし、再度データベースを参照して結果を取得する.
     */
    public function clearCache()
    {
        $this->rules = array();
    }
}


Pull Request — master (#1632)

TaxRuleRepository::delete() A

Complexity

Size

Duplication

Code Coverage

Importance

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

1		<?php
2		/*
3		* This file is part of EC-CUBE
4		*
5		* Copyright(c) 2000-2015 LOCKON CO.,LTD. All Rights Reserved.
6		*
7		* http://www.lockon.co.jp/
8		*
9		* This program is free software; you can redistribute it and/or
10		* modify it under the terms of the GNU General Public License
11		* as published by the Free Software Foundation; either version 2
12		* of the License, or (at your option) any later version.
13		*
14		* This program is distributed in the hope that it will be useful,
15		* but WITHOUT ANY WARRANTY; without even the implied warranty of
16		* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
17		* GNU General Public License for more details.
18		*
19		* You should have received a copy of the GNU General Public License
20		* along with this program; if not, write to the Free Software
21		* Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
22		*/
23
24
25		namespace Eccube\Repository;
26
27		use Doctrine\ORM\EntityRepository;
28		use Doctrine\ORM\NoResultException;
29		use Eccube\Common\Constant;
30
31		/**
32		* TaxRuleRepository
33		*
34		* This class was generated by the Doctrine ORM. Add your own custom
35		* repository methods below.
36		*/
37		class TaxRuleRepository extends EntityRepository
38		{
39		private $rules = array();
40
41		protected $app;
42
43	586	public function setApplication($app)
		0 ignored issues – show introduced 2015-11-21 02:41 UTC by Report Bug Copy Issue Report Missing function doc comment Loading history...
44		{
45	586	$this->app = $app;
46	586	}
47
48	1	public function newTaxRule()
		0 ignored issues – show introduced 2015-11-21 02:41 UTC by Report Bug Copy Issue Report Missing function doc comment Loading history...
49		{
50		$TaxRule = new \Eccube\Entity\TaxRule();
51	1	$CalcRule = $this->getEntityManager()
52	1	->getRepository('Eccube\Entity\Master\Taxrule')
53		->find(1);
54		$TaxRule->setCalcRule($CalcRule);
55		$TaxRule->setTaxAdjust(0);
56		$TaxRule->setDelFlg(0);
57
58	1	return $TaxRule;
59		}
60
61		/**
62		* 現在有効な税率設定情報を返す
63		*
64		* @param int\|null\|\Eccube\Entity\Product $Product 商品
65		* @param int\|null\|\Eccube\Entity\ProductClass $ProductClass 商品規格
66		* @param int\|null\|\Eccube\Entity\Master\Pref $Pref 都道府県
67		* @param int\|null\|\Eccube\Entity\Master\Country $Country 国
68		* @return \Eccube\Entity\TaxRule 税設定情報
69		*
70		* @throws NoResultException
71		*/
72	215	public function getByRule($Product = null, $ProductClass = null, $Pref = null, $Country = null)
73		{
74	2	if (!$this->app) {
75		throw new \LogicException();
76		}
77
78		// Pref Country 設定
79		if (!$Pref && !$Country && $this->app['security']->getToken() && $this->app['security']->isGranted('ROLE_USER')) {
80		/* @var $Customer \Eccube\Entity\Customer */
81		$Customer = $this->app['security']->getToken()->getUser();
82		$Pref = $Customer->getPref();
83		$Country = $Customer->getCountry();
84		}
85
86		// 商品単位税率設定がOFFの場合
87		/** @var $BaseInfo \Eccube\Entity\BaseInfo */
88		$BaseInfo = $this->app['eccube.repository.base_info']->get();
89		if ($BaseInfo->getOptionProductTaxRule() !== Constant::ENABLED) {
90	2	$Product = null;
91	2	$ProductClass = null;
92		}
93
94		// Cache Key 設定
95	2	if ($Product instanceof \Eccube\Entity\Product) {
		0 ignored issues – show Bug introduced 2015-11-21 02:41 UTC by Report Bug Copy Issue Report The class `Eccube\Entity\Product` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
96		$productId = $Product->getId();
97	2	} elseif ($Product) {
		0 ignored issues – show Bug Best Practice introduced 2015-11-21 02:41 UTC by Report Bug Copy Issue Report The expression `$Product` of type `integer\|null` is loosely compared to `true`; this is ambiguous if the integer can be zero. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `integer` values, zero is a special case, in particular the following results might be unexpected: 0 == false // true 0 == null // true 123 == false // false 123 == null // false // It is often better to use strict comparison 0 === false // false 0 === null // false Loading history...
98		$productId = $Product;
99		} else {
100	2	$productId = '0';
101	2	}
102	2	if ($ProductClass instanceof \Eccube\Entity\ProductClass) {
		0 ignored issues – show Bug introduced 2015-11-21 02:41 UTC by Report Bug Copy Issue Report The class `Eccube\Entity\ProductClass` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
103		$productClassId = $ProductClass->getId();
104	2	} elseif ($ProductClass) {
		0 ignored issues – show Bug Best Practice introduced 2015-11-21 02:41 UTC by Report Bug Copy Issue Report The expression `$ProductClass` of type `integer\|null` is loosely compared to `true`; this is ambiguous if the integer can be zero. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `integer` values, zero is a special case, in particular the following results might be unexpected: 0 == false // true 0 == null // true 123 == false // false 123 == null // false // It is often better to use strict comparison 0 === false // false 0 === null // false Loading history...
105		$productClassId = $ProductClass;
106		} else {
107	2	$productClassId = '0';
108	1	}
109	2	if ($Pref instanceof \Eccube\Entity\Master\Pref) {
		0 ignored issues – show Bug introduced 2015-11-21 02:41 UTC by Report Bug Copy Issue Report The class `Eccube\Entity\Master\Pref` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
110		$prefId = $Pref->getId();
111	2	} elseif ($Pref) {
112		$prefId = $Pref;
113		} else {
114	2	$prefId = '0';
115	1	}
116	2	if ($Country instanceof \Eccube\Entity\Master\Country) {
		0 ignored issues – show Bug introduced 2015-11-21 02:41 UTC by Report Bug Copy Issue Report The class `Eccube\Entity\Master\Country` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
117		$countryId = $Country->getId();
118	2	} elseif ($Country) {
119		$countryId = $Country;
120		} else {
121	2	$countryId = '0';
122	2	}
123	2	$cacheKey = $productId.':'.$productClassId.':'.$prefId.':'.$countryId;
124
125		// すでに取得している場合はキャッシュから
126	2	if (isset($this->rules[$cacheKey])) {
127	9	return $this->rules[$cacheKey];
128		}
129
130	215	$parameters = array();
131	215	$qb = $this->createQueryBuilder('t')
132		->where('t.apply_date < CURRENT_TIMESTAMP()');
133
134		// Pref
135	215	if ($Pref) {
136		$qb->andWhere('t.Pref IS NULL OR t.Pref = :Pref');
137	37	$parameters['Pref'] = $Pref;
138		} else {
139		$qb->andWhere('t.Pref IS NULL');
140	37	}
141
142		// Country
143	215	if ($Country) {
144		$qb->andWhere('t.Country IS NULL OR t.Country = :Country');
145	2	$parameters['Country'] = $Country;
146		} else {
147		$qb->andWhere('t.Country IS NULL');
148	2	}
149
150		/*
151		* Product, ProductClass が persist される前に TaxRuleEventSubscriber によってアクセスされる
152		* 場合があるため、ID の存在もチェックする.
153		* https://github.com/EC-CUBE/ec-cube/issues/677
154		*/
155
156		// Product
157	215	if ($Product && $productId > 0) {
158		$qb->andWhere('t.Product IS NULL OR t.Product = :Product');
159	2	$parameters['Product'] = $Product;
160		} else {
161		$qb->andWhere('t.Product IS NULL');
162	2	}
163
164		// ProductClass
165	215	if ($ProductClass && $productClassId > 0) {
166		$qb->andWhere('t.ProductClass IS NULL OR t.ProductClass = :ProductClass');
167	1	$parameters['ProductClass'] = $ProductClass;
168		} else {
169		$qb->andWhere('t.ProductClass IS NULL');
170	1	}
171
172		$TaxRules = $qb
173	215	->setParameters($parameters)
174	215	->orderBy('t.apply_date', 'DESC') // 実際は usort() でソートする
175	215	->getQuery()
176		->getResult();
177
178		// 地域設定を優先するが、システムパラメーターなどに設定を持っていくか
179		// 後に書いてあるほど優先される
180		$priorityKeys = explode(',', $this->app['config']['tax_rule_priority']);
		0 ignored issues – show Unused Code introduced 2015-11-21 02:41 UTC by Report Bug Copy Issue Report `$priorityKeys` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
181	215	$priorityKeys = array();
182		foreach (explode(',', $this->app['config']['tax_rule_priority']) as $key) {
183		$priorityKeys[] = str_replace('_', '', preg_replace('/_id\z/', '', $key));
184		}
185
186		foreach ($TaxRules as $TaxRule) {
187	215	$rank = 0;
188		foreach ($priorityKeys as $index => $key) {
189		$arrayProperties = array_change_key_case($TaxRule->toArray());
190		if ($arrayProperties[$key]) {
		0 ignored issues – show Coding Style introduced 2016-03-01 10:06 UTC by Report Bug Copy Issue Report Blank line found at start of control structure Loading history...
191
192		// 配列の数値添字を重みとして利用する
193	4	$rank += 1 << ($index + 1);
194		}
195		}
196		$TaxRule->setRank($rank);
197		}
198
199		// 適用日降順, rank 降順にソートする
200		usort($TaxRules, function($a, $b) {
		0 ignored issues – show Coding Style introduced 2016-03-01 10:06 UTC by Report Bug Copy Issue Report Expected 1 space after FUNCTION keyword; 0 found Loading history...
201		return $a->compareTo($b);
202		});
203
204	215	if (!empty($TaxRules)) {
205	215	$this->rules[$cacheKey] = $TaxRules[0];
206
207	215	return $TaxRules[0];
208		} else {
209		throw new NoResultException();
210		}
211		}
212
213		/**
214		* getList
215		*
216		* @return array\|null
217		*/
218	3	public function getList()
219		{
220	3	$qb = $this->createQueryBuilder('t')
221	3	->orderBy('t.apply_date', 'DESC')
222		->where('t.Product IS NULL AND t.ProductClass IS NULL');
223		$TaxRules = $qb
224	3	->getQuery()
225		->getResult();
226
227	3	return $TaxRules;
228		}
229
230		/**
231		* getById
232		*
233		* @param int $id
		0 ignored issues – show introduced 2015-11-21 02:41 UTC by Report Bug Copy Issue Report Expected 1 spaces after parameter type; 3 found Loading history...
234		* @return array
235		*/
236	1	public function getById($id)
237		{
238		$criteria = array(
239		'id' => $id,
240	1	);
241
242		return $this->findOneBy($criteria);
243		}
244
245		/**
246		* getByTime
247		*
248		* @param string $applyDate
249		* @return mixed
250		*/
251		public function getByTime($applyDate)
252		{
253		$criteria = array(
254		'apply_date' => $applyDate,
255		);
256
257		return $this->findOneBy($criteria);
258		}
259
260		/**
261		* 税規約の削除.
262		*
263		* @param int\|\Eccube\Entity\TaxRule $TaxRule 税規約
264		* @return void
265		* @throws NoResultException
266		*/
267	2	public function delete($TaxRule)
268		{
269	2	if (!$TaxRule instanceof \Eccube\Entity\TaxRule) {
		0 ignored issues – show Bug introduced 2015-11-21 02:41 UTC by Report Bug Copy Issue Report The class `Eccube\Entity\TaxRule` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
270		$TaxRule = $this->find($TaxRule);
271		}
272	2	if (!$TaxRule) {
273		throw new NoResultException;
		0 ignored issues – show introduced 2015-11-21 02:41 UTC by Report Bug Copy Issue Report Use parentheses when instantiating classes Loading history...
274		}
275		$TaxRule->setDelFlg(1);
276		$em = $this->getEntityManager();
277		$em->persist($TaxRule);
278		$em->flush();
279	2	}
280
281		/**
282		* TaxRule のキャッシュをクリアする.
283		*
284		* getByRule() をコールすると、結果をキャッシュし、2回目以降はデータベースへアクセスしない.
285		* このメソッドをコールすると、キャッシュをクリアし、再度データベースを参照して結果を取得する.
286		*/
287	7	public function clearCache()
288		{
289	7	$this->rules = array();
290	7	}
291		}
292

EC-CUBE / ec-cube

Pull Request — master (#1632)

TaxRuleRepository::delete() A

Complexity

Size

Duplication

Code Coverage

Importance

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

Duplication Side-by-Side

Filter issues like