PiedWeb\ConversationBundle\Form\MessageForm

Complexity

Total Complexity

6

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getUser()	0	12	3
A	getStepOne()	0	19	3

<?php

namespace PiedWeb\ConversationBundle\Form;

use Symfony\Component\Form\Extension\Core\Type\EmailType;
use Symfony\Component\Form\Extension\Core\Type\TextareaType;
use Symfony\Component\Form\FormBuilderInterface;

class MessageForm
{
    use FormTrait;

The trait PiedWeb\ConversationBundle\Form\FormTrait requires the property $query which is not provided by PiedWeb\ConversationBundle\Form\MessageForm.

    protected function getStepOne(): FormBuilderInterface
    {
        /*
        if ($this->getUser()) {
            $this->message->setAuthorEmail($this->getUser()->getEmail());
            $this->message->setAuthorName($this->getUser()->getUsername());
            $user = true;
        }/**/

        $form = $this->initForm();

        if (!isset($user) || !$this->getUser()) {

The variable $user seems to never exist and therefore isset should always be false.

            $form->add('authorEmail', EmailType::class, ['constraints' => $this->getAuthorEmailConstraints()]);
            $form->add('authorName', null, ['constraints' => $this->getAuthorNameConstraints()]);
        }

        $form->add('content', TextareaType::class);

        return $form;
    }

    protected function getUser()
    {
        if (null === $token = $this->security->getToken()) {
            return null;
        }

        if (!\is_object($user = $token->getUser())) {
            // e.g. anonymous authentication
            return null;
        }

        return $user;
    }
}