CreateCommentFormAction::getCommentForm() - Code Metrics - Inspection of "Merge remote-tracking branch 'upstream/3.x' into m..." - sonata-project/SonataNewsBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 37876c...34238f )

by Marko

created 2018-07-13 17:14 UTC

CreateCommentFormAction::getCommentForm() A

↳ Parent: CreateCommentFormAction

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	13
rs	9.8333
c	0
b	0
f	0
cc	1
nc	1
nop	1

<?php

declare(strict_types=1);

/*
 * This file is part of the Sonata Project package.
 *
 * (c) Thomas Rabaix <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Sonata\NewsBundle\Action;

use Sonata\NewsBundle\Form\Type\CommentType;
use Sonata\NewsBundle\Model\CommentManagerInterface;
use Sonata\NewsBundle\Model\PostInterface;
use Sonata\NewsBundle\Model\PostManagerInterface;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\Form\FormFactoryInterface;
use Symfony\Component\Form\FormInterface;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\Routing\RouterInterface;

final class CreateCommentFormAction extends Controller
{
    /**
     * @var RouterInterface
     */
    private $router;

    /**
     * @var PostManagerInterface
     */
    private $postManager;

    /**
     * @var CommentManagerInterface
     */
    private $commentManager;

    /**
     * @var FormFactoryInterface
     */
    private $formFactory;

    public function __construct(
        RouterInterface $router,
        PostManagerInterface $postManager,
        CommentManagerInterface $commentManager,
        FormFactoryInterface $formFactory
    ) {
        $this->router = $router;
        $this->postManager = $postManager;
        $this->commentManager = $commentManager;
        $this->formFactory = $formFactory;
    }

    /**
     * @param string $postId
     * @param bool   $form
     *
     * @return Response
     */
    public function __invoke($postId, $form = false)
    {
        if (!$form) {
            $post = $this->postManager->findOneBy([
                'id' => $postId,
            ]);

            $form = $this->getCommentForm($post);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        return $this->render('@SonataNews/Post/comment_form.html.twig', [
            'form' => $form->createView(),
function someFunction(A $objectMaybe = null)
{
    if ($objectMaybe instanceof A) {
        $objectMaybe->doSomething();
    }
}
            'post_id' => $postId,
        ]);
    }

    /**
     * @return FormInterface
     */
    private function getCommentForm(PostInterface $post)
    {
        $comment = $this->commentManager->create();
        $comment->setPost($post);
        $comment->setStatus($post->getCommentsDefaultStatus());

        return $this->formFactory->createNamed('comment', CommentType::class, $comment, [
            'action' => $this->router->generate('sonata_news_add_comment', [
                'id' => $post->getId(),
            ]),
            'method' => 'POST',
        ]);
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/*
6			* This file is part of the Sonata Project package.
7			*
8			* (c) Thomas Rabaix <[email protected]>
9			*
10			* For the full copyright and license information, please view the LICENSE
11			* file that was distributed with this source code.
12			*/
13
14			namespace Sonata\NewsBundle\Action;
15
16			use Sonata\NewsBundle\Form\Type\CommentType;
17			use Sonata\NewsBundle\Model\CommentManagerInterface;
18			use Sonata\NewsBundle\Model\PostInterface;
19			use Sonata\NewsBundle\Model\PostManagerInterface;
20			use Symfony\Bundle\FrameworkBundle\Controller\Controller;
21			use Symfony\Component\Form\FormFactoryInterface;
22			use Symfony\Component\Form\FormInterface;
23			use Symfony\Component\HttpFoundation\Response;
24			use Symfony\Component\Routing\RouterInterface;
25
26			final class CreateCommentFormAction extends Controller
27			{
28			/**
29			* @var RouterInterface
30			*/
31			private $router;
32
33			/**
34			* @var PostManagerInterface
35			*/
36			private $postManager;
37
38			/**
39			* @var CommentManagerInterface
40			*/
41			private $commentManager;
42
43			/**
44			* @var FormFactoryInterface
45			*/
46			private $formFactory;
47
48			public function __construct(
49			RouterInterface $router,
50			PostManagerInterface $postManager,
51			CommentManagerInterface $commentManager,
52			FormFactoryInterface $formFactory
53			) {
54			$this->router = $router;
55			$this->postManager = $postManager;
56			$this->commentManager = $commentManager;
57			$this->formFactory = $formFactory;
58			}
59
60			/**
61			* @param string $postId
62			* @param bool $form
63			*
64			* @return Response
65			*/
66			public function __invoke($postId, $form = false)
67			{
68			if (!$form) {
69			$post = $this->postManager->findOneBy([
70			'id' => $postId,
71			]);
72
73			$form = $this->getCommentForm($post);
			0 ignored issues – show Documentation introduced 2015-11-23 07:52 UTC by Report Bug Copy Issue Report `$post` is of type `object\|null`, but the function expects a `object<Sonata\NewsBundle\Model\PostInterface>`. 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...
74			}
75
76			return $this->render('@SonataNews/Post/comment_form.html.twig', [
77			'form' => $form->createView(),
			0 ignored issues – show Bug introduced 2015-11-23 07:52 UTC by Report Bug Copy Issue Report It seems like `$form` is not always an object, but can also be of type `boolean`. Maybe add an additional type check? If a variable is not always an object, we recommend to add an additional type check to ensure your method call is safe: function someFunction(A $objectMaybe = null) { if ($objectMaybe instanceof A) { $objectMaybe->doSomething(); } } Loading history...
78			'post_id' => $postId,
79			]);
80			}
81
82			/**
83			* @return FormInterface
84			*/
85			private function getCommentForm(PostInterface $post)
86			{
87			$comment = $this->commentManager->create();
88			$comment->setPost($post);
89			$comment->setStatus($post->getCommentsDefaultStatus());
90
91			return $this->formFactory->createNamed('comment', CommentType::class, $comment, [
92			'action' => $this->router->generate('sonata_news_add_comment', [
93			'id' => $post->getId(),
94			]),
95			'method' => 'POST',
96			]);
97			}
98			}
99

sonata-project / SonataNewsBundle

Push — master ( 37876c...34238f )

CreateCommentFormAction::getCommentForm() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like