ExchangeRateController::editAction() - Code Metrics - Inspection of "Documented, stabilized..." - RunOpenCode/exchange-rate-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( b26542...bb7d10 )

by Nikola

created 2016-01-08 11:42 UTC

ExchangeRateController::editAction() B

↳ Parent: ExchangeRateController

Complexity

Conditions	3
Paths	2

Size

Total Lines	24
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	24
ccs	0
cts	9
cp	0
rs	8.9714
cc	3
eloc	13
nc	2
nop	1
crap	12

<?php
/*
 * This file is part of the Exchange Rate Bundle, an RunOpenCode project.
 *
 * (c) 2016 RunOpenCode
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace RunOpenCode\Bundle\ExchangeRate\Controller;

use RunOpenCode\Bundle\ExchangeRate\Form\Type\NewType;
use RunOpenCode\ExchangeRate\Contract\RateInterface;
use RunOpenCode\ExchangeRate\Contract\RepositoryInterface;
use RunOpenCode\Bundle\ExchangeRate\Model\Rate;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\Form\FormError;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
use Symfony\Component\Security\Core\Exception\InvalidCsrfTokenException;

/**
 * Class ExchangeRateController
 *
 * Default exchange rate controller.
 *
 * @package RunOpenCode\Bundle\ExchangeRate\Controller
 */
class ExchangeRateController extends Controller
{
    /**
     * @var RepositoryInterface
     */
    protected $repository;

    /**
     * @var string
     */
    protected $baseCurrency;

    /**
     * @var array
     */
    protected $settings;

    public function __construct(
        RepositoryInterface $repository,
        $baseCurrency,
        $settings
    ) {
        $this->repository = $repository;
        $this->baseCurrency = $baseCurrency;
        $this->settings = $settings;
    }

    /**
     * List rates.
     *
     * @return \Symfony\Component\HttpFoundation\Response
     */
    public function indexAction()
    {
        $this->denyAccessUnlessGranted(array('ROLE_EXCHANGE_RATE_MANAGER', 'ROLE_EXCHANGE_RATE_LIST'));

        return $this->render($this->settings['list'], array(
            'base_template' => $this->settings['base_template'],
            'rates' => $this->repository->all(),
            'date_format' => $this->settings['date_format'],
            'time_format' => $this->settings['time_format'],
            'secure' => $this->settings['secure']
        ));
    }

    /**
     * Add new rate.
     *
     * @param Request $request
     * @return \Symfony\Component\HttpFoundation\RedirectResponse|\Symfony\Component\HttpFoundation\Response
     */
    public function newAction(Request $request)
    {
        $this->denyAccessUnlessGranted(array('ROLE_EXCHANGE_RATE_MANAGER', 'ROLE_EXCHANGE_RATE_CREATE'));

        $form = $this->createForm(NewType::class, $this->getNewRate());

        $form->handleRequest($request);

        if ($form->isSubmitted() && $form->isValid()) {

            /**
             * @var RateInterface $rate
             */
            $rate = $form->getData();

            if ($this->repository->has($rate->getCurrencyCode(), $rate->getDate(), $rate->getRateType())) {
                $form->addError(new FormError($this->get('translator')->trans('exchange_rate.form.error.new_exists', array(), 'roc_exchange_rate')));
            } else {
                $this->repository->save(array(
                    $form->getData()
                ));

                $this->get('session')->getFlashBag()->add('success', 'exchange_rate.flash.new.success');
                return $this->redirectToRoute('roc_exchange_rate_list');
            }
        }

        return $this->render($this->settings['new'], array(
            'base_template' => $this->settings['base_template'],
            'form' => $form->createView(),
            'secure' => $this->settings['secure']
        ));
    }

    /**
     * Edit rate.
     *
     * @param Request $request
     * @return \Symfony\Component\HttpFoundation\RedirectResponse|\Symfony\Component\HttpFoundation\Response
     */
    public function editAction(Request $request)
    {
        $this->denyAccessUnlessGranted(array('ROLE_EXCHANGE_RATE_MANAGER', 'ROLE_EXCHANGE_RATE_EDIT'));

        $form = $this->createForm(NewType::class, $this->getRateFromRequest($request));

        $form->handleRequest($request);

        if ($form->isSubmitted() && $form->isValid()) {

            $this->repository->save(array(
                $form->getData()
            ));

            $this->get('session')->getFlashBag()->add('success', 'exchange_rate.flash.edit.success');
            return $this->redirectToRoute('roc_exchange_rate_list');
        }

        return $this->render($this->settings['edit'], array(
            'base_template' => $this->settings['base_template'],
            'form' => $form->createView(),
            'secure' => $this->settings['secure']
        ));
    }

    /**
     * Delete rate.
     *
     * @param Request $request
     * @return \Symfony\Component\HttpFoundation\RedirectResponse
     */
    public function deleteAction(Request $request)
    {
        $this->denyAccessUnlessGranted(array('ROLE_EXCHANGE_RATE_MANAGER', 'ROLE_EXCHANGE_RATE_DELETE'));

        if (!$this->isCsrfTokenValid($request->getRequestUri(), $request->get('_csrf_token'))) {
            throw new InvalidCsrfTokenException;
        }

        $rate = $this->getRateFromRequest($request);

        $this->repository->delete(array($rate));

        $this->get('session')->getFlashBag()->add('success', 'exchange_rate.flash.delete.success');
        return $this->redirectToRoute('roc_exchange_rate_list');
    }

    /**
     * Get new rate object for new form.
     *
     * @return Rate
     */
    protected function getNewRate()
    {
        $rate = new Rate(null, null, $this->baseCurrency, null, null, $this->baseCurrency, null, null);
        $rate->setBaseCurrencyCode($this->baseCurrency);

        return $rate;
    }

    /**
     * Find rate based on values of request parameters.
     *
     * @param Request $request
     * @return static
     */
    protected function getRateFromRequest(Request $request)
    {
        if (!$this->repository->has($request->get('source'), $request->get('currency_code'), \DateTime::createFromFormat('Y-m-d', $request->get('date')), $request->get('rate_type'))) {
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            throw new NotFoundHttpException();
        }

        return Rate::fromRateInterface($this->repository->get($request->get('source'), $request->get('currency_code'), \DateTime::createFromFormat('Y-m-d', $request->get('date')), $request->get('rate_type')));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * {@inheritdoc}
     */
    protected function denyAccessUnlessGranted($attributes, $object = null, $message = 'Access Denied.')
    {
        if ($this->settings['secure']) {
            parent::denyAccessUnlessGranted($attributes, $object, $message);
        }
    }
}


1			<?php
2			/*
3			* This file is part of the Exchange Rate Bundle, an RunOpenCode project.
4			*
5			* (c) 2016 RunOpenCode
6			*
7			* For the full copyright and license information, please view the LICENSE
8			* file that was distributed with this source code.
9			*/
10			namespace RunOpenCode\Bundle\ExchangeRate\Controller;
11
12			use RunOpenCode\Bundle\ExchangeRate\Form\Type\NewType;
13			use RunOpenCode\ExchangeRate\Contract\RateInterface;
14			use RunOpenCode\ExchangeRate\Contract\RepositoryInterface;
15			use RunOpenCode\Bundle\ExchangeRate\Model\Rate;
16			use Symfony\Bundle\FrameworkBundle\Controller\Controller;
17			use Symfony\Component\Form\FormError;
18			use Symfony\Component\HttpFoundation\Request;
19			use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
20			use Symfony\Component\Security\Core\Exception\InvalidCsrfTokenException;
21
22			/**
23			* Class ExchangeRateController
24			*
25			* Default exchange rate controller.
26			*
27			* @package RunOpenCode\Bundle\ExchangeRate\Controller
28			*/
29			class ExchangeRateController extends Controller
30			{
31			/**
32			* @var RepositoryInterface
33			*/
34			protected $repository;
35
36			/**
37			* @var string
38			*/
39			protected $baseCurrency;
40
41			/**
42			* @var array
43			*/
44			protected $settings;
45
46			public function __construct(
47			RepositoryInterface $repository,
48			$baseCurrency,
49			$settings
50			) {
51			$this->repository = $repository;
52			$this->baseCurrency = $baseCurrency;
53			$this->settings = $settings;
54			}
55
56			/**
57			* List rates.
58			*
59			* @return \Symfony\Component\HttpFoundation\Response
60			*/
61			public function indexAction()
62			{
63			$this->denyAccessUnlessGranted(array('ROLE_EXCHANGE_RATE_MANAGER', 'ROLE_EXCHANGE_RATE_LIST'));
64
65			return $this->render($this->settings['list'], array(
66			'base_template' => $this->settings['base_template'],
67			'rates' => $this->repository->all(),
68			'date_format' => $this->settings['date_format'],
69			'time_format' => $this->settings['time_format'],
70			'secure' => $this->settings['secure']
71			));
72			}
73
74			/**
75			* Add new rate.
76			*
77			* @param Request $request
78			* @return \Symfony\Component\HttpFoundation\RedirectResponse\|\Symfony\Component\HttpFoundation\Response
79			*/
80			public function newAction(Request $request)
81			{
82			$this->denyAccessUnlessGranted(array('ROLE_EXCHANGE_RATE_MANAGER', 'ROLE_EXCHANGE_RATE_CREATE'));
83
84			$form = $this->createForm(NewType::class, $this->getNewRate());
85
86			$form->handleRequest($request);
87
88			if ($form->isSubmitted() && $form->isValid()) {
89
90			/**
91			* @var RateInterface $rate
92			*/
93			$rate = $form->getData();
94
95			if ($this->repository->has($rate->getCurrencyCode(), $rate->getDate(), $rate->getRateType())) {
96			$form->addError(new FormError($this->get('translator')->trans('exchange_rate.form.error.new_exists', array(), 'roc_exchange_rate')));
97			} else {
98			$this->repository->save(array(
99			$form->getData()
100			));
101
102			$this->get('session')->getFlashBag()->add('success', 'exchange_rate.flash.new.success');
103			return $this->redirectToRoute('roc_exchange_rate_list');
104			}
105			}
106
107			return $this->render($this->settings['new'], array(
108			'base_template' => $this->settings['base_template'],
109			'form' => $form->createView(),
110			'secure' => $this->settings['secure']
111			));
112			}
113
114			/**
115			* Edit rate.
116			*
117			* @param Request $request
118			* @return \Symfony\Component\HttpFoundation\RedirectResponse\|\Symfony\Component\HttpFoundation\Response
119			*/
120			public function editAction(Request $request)
121			{
122			$this->denyAccessUnlessGranted(array('ROLE_EXCHANGE_RATE_MANAGER', 'ROLE_EXCHANGE_RATE_EDIT'));
123
124			$form = $this->createForm(NewType::class, $this->getRateFromRequest($request));
125
126			$form->handleRequest($request);
127
128			if ($form->isSubmitted() && $form->isValid()) {
129
130			$this->repository->save(array(
131			$form->getData()
132			));
133
134			$this->get('session')->getFlashBag()->add('success', 'exchange_rate.flash.edit.success');
135			return $this->redirectToRoute('roc_exchange_rate_list');
136			}
137
138			return $this->render($this->settings['edit'], array(
139			'base_template' => $this->settings['base_template'],
140			'form' => $form->createView(),
141			'secure' => $this->settings['secure']
142			));
143			}
144
145			/**
146			* Delete rate.
147			*
148			* @param Request $request
149			* @return \Symfony\Component\HttpFoundation\RedirectResponse
150			*/
151			public function deleteAction(Request $request)
152			{
153			$this->denyAccessUnlessGranted(array('ROLE_EXCHANGE_RATE_MANAGER', 'ROLE_EXCHANGE_RATE_DELETE'));
154
155			if (!$this->isCsrfTokenValid($request->getRequestUri(), $request->get('_csrf_token'))) {
156			throw new InvalidCsrfTokenException;
157			}
158
159			$rate = $this->getRateFromRequest($request);
160
161			$this->repository->delete(array($rate));
162
163			$this->get('session')->getFlashBag()->add('success', 'exchange_rate.flash.delete.success');
164			return $this->redirectToRoute('roc_exchange_rate_list');
165			}
166
167			/**
168			* Get new rate object for new form.
169			*
170			* @return Rate
171			*/
172			protected function getNewRate()
173			{
174			$rate = new Rate(null, null, $this->baseCurrency, null, null, $this->baseCurrency, null, null);
175			$rate->setBaseCurrencyCode($this->baseCurrency);
176
177			return $rate;
178			}
179
180			/**
181			* Find rate based on values of request parameters.
182			*
183			* @param Request $request
184			* @return static
185			*/
186			protected function getRateFromRequest(Request $request)
187			{
188			if (!$this->repository->has($request->get('source'), $request->get('currency_code'), \DateTime::createFromFormat('Y-m-d', $request->get('date')), $request->get('rate_type'))) {
			0 ignored issues – show Documentation introduced 2016-01-08 11:49 UTC by Report Bug Copy Issue Report `\DateTime::createFromFor... $request->get('date'))` is of type `object<DateTime>\|false`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history... Unused Code introduced 2016-01-08 11:49 UTC by Report Bug Copy Issue Report The call to `RepositoryInterface::has()` has too many arguments starting with `$request->get('rate_type')`. This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue. If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. In this case you can add the `@ignore` PhpDoc annotation to the duplicate definition and it will be ignored. Loading history...
189			throw new NotFoundHttpException();
190			}
191
192			return Rate::fromRateInterface($this->repository->get($request->get('source'), $request->get('currency_code'), \DateTime::createFromFormat('Y-m-d', $request->get('date')), $request->get('rate_type')));
			0 ignored issues – show Documentation introduced 2016-01-08 11:49 UTC by Report Bug Copy Issue Report `\DateTime::createFromFor... $request->get('date'))` is of type `object<DateTime>\|false`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history... Unused Code introduced 2016-01-08 11:49 UTC by Report Bug Copy Issue Report The call to `RepositoryInterface::get()` has too many arguments starting with `$request->get('rate_type')`. This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue. If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. In this case you can add the `@ignore` PhpDoc annotation to the duplicate definition and it will be ignored. Loading history...
193			}
194
195			/**
196			* {@inheritdoc}
197			*/
198			protected function denyAccessUnlessGranted($attributes, $object = null, $message = 'Access Denied.')
199			{
200			if ($this->settings['secure']) {
201			parent::denyAccessUnlessGranted($attributes, $object, $message);
202			}
203			}
204			}
205

RunOpenCode / exchange-rate-bundle

Push — master ( b26542...bb7d10 )

ExchangeRateController::editAction() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like