ControllerHelper::isOrderInProgress() - Code Metrics - Inspection of "Merge pull request #71 from Dev-Int/Order" - Dev-Int/glsr - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( ae829f...1d5438 )

by Laurent

created 2017-03-09 19:22 UTC

ControllerHelper::isOrderInProgress() A

↳ Parent: ControllerHelper

Complexity

Conditions	3
Paths	3

Size

Total Lines	12
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php
/**
 * ControllerHelper Helpers des Controller de l'application GLSR.
 *
 * PHP Version 5
 *
 * @author    Quétier Laurent <[email protected]>
 * @copyright 2014 Dev-Int GLSR
 * @license   http://opensource.org/licenses/gpl-license.php GNU Public License
 *
 * @version since 1.0.0
 *
 * @link      https://github.com/Dev-Int/glsr
 */
namespace AppBundle\Helper;

use AppBundle\Entity\Article;
use Doctrine\Common\Persistence\ObjectManager;

/**
 * Controller helper.
 *
 * @category Helper
 */
class ControllerHelper
{
    /**
     * Tests of creation conditions.
     *
     * @param array Articles à tester
     * @return boolean
     */
    public function hasSupplierArticles(array $articles)
    {
        $return = true;

        // This supplier has no articles!
        if (count($articles) < 1) {
            $message = $this->get('translator')->trans('settings.no_articles', array(), 'gs_suppliers');

            $this->addFlash('danger', $message);

            $return = false;
        }

        return $return;
    }

    /**
     * Tests Order in progress for a supplier.
     *
     * @param \AppBundle\Entity\Article $articles Articles to test
     * @param \Doctrine\Common\Persistence\ObjectManager $etm Named object manager
     * @return boolean
     */
    public function isOrderInProgress(Article $articles, ObjectManager $etm)
    {
        $return = false;
        $orders = $etm->getRepository('AppBundle:Orders')->findAll();
        // This provider already has an order in progress!
        foreach ($orders as $order) {
            if ($order->getSupplier() === $articles->getSupplier()) {
                $return = true;
            }
        }
        return $return;
    }
}


1			<?php
2			/**
3			* ControllerHelper Helpers des Controller de l'application GLSR.
4			*
5			* PHP Version 5
6			*
7			* @author Quétier Laurent <[email protected]>
8			* @copyright 2014 Dev-Int GLSR
9			* @license http://opensource.org/licenses/gpl-license.php GNU Public License
10			*
11			* @version since 1.0.0
12			*
13			* @link https://github.com/Dev-Int/glsr
14			*/
15			namespace AppBundle\Helper;
16
17			use AppBundle\Entity\Article;
18			use Doctrine\Common\Persistence\ObjectManager;
19
20			/**
21			* Controller helper.
22			*
23			* @category Helper
24			*/
25			class ControllerHelper
26			{
27			/**
28			* Tests of creation conditions.
29			*
30			* @param array Articles à tester
31			* @return boolean
32			*/
33			public function hasSupplierArticles(array $articles)
34			{
35			$return = true;
36
37			// This supplier has no articles!
38			if (count($articles) < 1) {
39			$message = $this->get('translator')->trans('settings.no_articles', array(), 'gs_suppliers');
			0 ignored issues – show Bug introduced 2016-12-20 12:17 UTC by Report Bug Copy Issue Report The method `get()` does not seem to exist on `object<AppBundle\Helper\ControllerHelper>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
40			$this->addFlash('danger', $message);
			0 ignored issues – show Bug introduced 2016-12-20 12:17 UTC by Report Bug Copy Issue Report The method `addFlash()` does not seem to exist on `object<AppBundle\Helper\ControllerHelper>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
41			$return = false;
42			}
43
44			return $return;
45			}
46
47			/**
48			* Tests Order in progress for a supplier.
49			*
50			* @param \AppBundle\Entity\Article $articles Articles to test
51			* @param \Doctrine\Common\Persistence\ObjectManager $etm Named object manager
52			* @return boolean
53			*/
54			public function isOrderInProgress(Article $articles, ObjectManager $etm)
55			{
56			$return = false;
57			$orders = $etm->getRepository('AppBundle:Orders')->findAll();
58			// This provider already has an order in progress!
59			foreach ($orders as $order) {
60			if ($order->getSupplier() === $articles->getSupplier()) {
61			$return = true;
62			}
63			}
64			return $return;
65			}
66			}
67

Dev-Int / glsr

Push — master ( ae829f...1d5438 )

ControllerHelper::isOrderInProgress() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like