BaseEntityAdminForm::buildForm() - Code Metrics - Inspection of "Fixed strict types exception in attachments list." - Part-DB/Part-DB-symfony - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 0c21d6...074f02 )

by Jan

created 2020-01-07 18:38 UTC

BaseEntityAdminForm::buildForm() C

↳ Parent: BaseEntityAdminForm

Complexity

Conditions	11
Paths	1

Size

Total Lines	77
Code Lines	52

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	11
eloc	52
c	1
b	0
f	0
nc	1
nop	2
dl	0
loc	77
rs	6.9006

How to fix Long Method Complexity

<?php

declare(strict_types=1);

/**
 * This file is part of Part-DB (https://github.com/Part-DB/Part-DB-symfony).
 *
 * Copyright (C) 2019 Jan Böhmer (https://github.com/jbtronics)
 *
 * 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., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
 */

namespace App\Form\AdminPages;

use App\Entity\Attachments\Attachment;
use App\Entity\Base\NamedDBElement;
use App\Entity\Base\StructuralDBElement;
use App\Form\AttachmentFormType;
use App\Form\Type\MasterPictureAttachmentType;
use App\Form\Type\StructuralEntityType;
use FOS\CKEditorBundle\Form\Type\CKEditorType;
use function get_class;
use Symfony\Component\DependencyInjection\ParameterBag\ParameterBagInterface;
use Symfony\Component\Form\AbstractType;
use Symfony\Component\Form\Extension\Core\Type\CheckboxType;
use Symfony\Component\Form\Extension\Core\Type\CollectionType;
use Symfony\Component\Form\Extension\Core\Type\ResetType;
use Symfony\Component\Form\Extension\Core\Type\SubmitType;
use Symfony\Component\Form\Extension\Core\Type\TextType;
use Symfony\Component\Form\FormBuilderInterface;
use Symfony\Component\OptionsResolver\OptionsResolver;
use Symfony\Component\Security\Core\Security;

class BaseEntityAdminForm extends AbstractType
{
    protected $security;
    protected $params;

    public function __construct(Security $security, ParameterBagInterface $params)
    {
        $this->security = $security;
        $this->params = $params;
    }

    public function configureOptions(OptionsResolver $resolver): void
    {
        parent::configureOptions($resolver); // TODO: Change the autogenerated stub
        $resolver->setRequired('attachment_class');
    }

    public function buildForm(FormBuilderInterface $builder, array $options): void
    {
        /** @var StructuralDBElement $entity */
        $entity = $options['data'];
        $is_new = null === $entity->getID();

        $builder
            ->add('name', TextType::class, [
                'empty_data' => '',
                'label' => 'name.label',
                'attr' => [
                    'placeholder' => 'part.name.placeholder',
                ],
                'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),

            ])

            ->add('parent', StructuralEntityType::class, [
                'class' => get_class($entity),
                'required' => false,
                'label' => 'parent.label',
                'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'move', $entity),

            ])

            ->add('not_selectable', CheckboxType::class, [
                'required' => false,
                'label' => 'entity.edit.not_selectable',
                'help' => 'entity.edit.not_selectable.help',
                'label_attr' => [
                    'class' => 'checkbox-custom',
                ],
                'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),

            ])

            ->add('comment', CKEditorType::class, [
                'required' => false,
                'empty_data' => '',
                'label' => 'comment.label',
                'attr' => [
                    'rows' => 4,
                ],
                'help' => 'bbcode.hint',
                'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),

            ]);

        $this->additionalFormElements($builder, $options, $entity);

        //Attachment section
        $builder->add('attachments', CollectionType::class, [
            'entry_type' => AttachmentFormType::class,
            'allow_add' => true,
            'allow_delete' => true,
            'label' => false,
            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),

            'entry_options' => [
                'data_class' => $options['attachment_class'],
            ],
            'by_reference' => false,
        ]);

        $builder->add('master_picture_attachment', MasterPictureAttachmentType::class, [
            'required' => false,
            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),

            'label' => 'part.edit.master_attachment',
            'entity' => $entity,
        ]);

        //Buttons
        $builder->add('save', SubmitType::class, [
            'label' => $is_new ? 'entity.create' : 'entity.edit.save',

            'attr' => [
                'class' => $is_new ? 'btn-success' : '',

            ],
            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),

        ])
            ->add('reset', ResetType::class, [
                'label' => 'entity.edit.reset',
                'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),

            ]);
    }

    protected function additionalFormElements(FormBuilderInterface $builder, array $options, NamedDBElement $entity): void
    protected function additionalFormElements(FormBuilderInterface $builder, array $options, /** @scrutinizer ignore-unused */ NamedDBElement $entity): void
    {
        //Empty for Base
    }
}


Part-DB / Part-DB-symfony

Push — master ( 0c21d6...074f02 )

BaseEntityAdminForm::buildForm() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like