ProgramForm - Code Metrics - Inspection of "Release v2.10.0" - literat/srazvs - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 118c60...254398 )

by Litera

created 2017-12-30 07:59 UTC

ProgramForm A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	211
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	10

Test Coverage

Coverage

33.78%

Importance

Changes

Metric	Value
dl	0
loc	211
ccs	25
cts	74
cp	0.3378
rs	10
c	0
b	0
f	0
wmc	14
lcom	1
cbo	10

12 Methods

Rating	Name	Size	Complexity
A	__construct()	7	1
A	render()	7	1
A	setDefaults()	6	1
B	createComponentProgramForm()	45	1
A	processForm()	6	1
A	processReset()	6	1
A	buildBlockSelect()	18	2
A	buildCategorySelect()	12	2
A	getBlockRepository()	4	1
A	setBlockRepository()	6	1
A	getCategoryRepository()	4	1
A	setCategoryRepository()	6	1

<?php

namespace App\Components\Forms;

use App\Models\MeetingModel;
use App\Repositories\BlockRepository;
use App\Repositories\CategoryRepository;
use Nette\Application\UI\Form;
use Nette\Database\Table\ActiveRow;
use Nette\Forms\Controls\SubmitButton;

class ProgramForm extends BaseForm
{

	const TEMPLATE_NAME = 'ProgramForm';

	const MESSAGE_REQUIRED = 'Hodnota musí být vyplněna!';
	const MESSAGE_MAX_LENGTH = '%label nesmí mít více jak %d znaků!';

	/**
	 * @var Closure
	 */
	public $onProgramSave;

	/**
	 * @var Closure
	 */
	public $onProgramReset;

	/**
	 * @var BlockRepository
	 */
	protected $blockRepository;

	/**
	 * @var CategoryRepository
	 */
	protected $categoryRepository;

	/**
	 * @param BlockRepository    $blockRepository
	 * @param CategoryRepository $categoryRepository
	 */
	public function __construct(
		BlockRepository $blockRepository,
		CategoryRepository $categoryRepository
	) {
		$this->setBlockRepository($blockRepository);
		$this->setCategoryRepository($categoryRepository);
	}

	/**
	 * @return void
	 */
	public function render()
	{
		$template = $this->getTemplate();
		$template->setFile($this->buildTemplatePath());
		$template->backlink = 'Program:listing';
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
		$template->render();
	}

	/**
	 * @param  array $defaults
	 * @return AnnotationForm
	 */
	public function setDefaults(ActiveRow $defaults): ProgramForm
	{
		$this['programForm']->setDefaults($defaults);

		return $this;
	}

	/**
	 * @return Form
	 */
	public function createComponentProgramForm(): Form
	{
		$form = new Form;

		$form->addText('name', 'Název:')
			->setRequired(static::MESSAGE_REQUIRED)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			->addRule(Form::MAX_LENGTH, static::MESSAGE_MAX_LENGTH, 50)
			->setAttribute('size', 50)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			->getLabelPrototype()->setAttribute('class', 'required');
		$form->addTextArea('description', 'Popis:')
			->setAttribute('rows', 10)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			->setAttribute('cols', 80);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
		$form->addTextArea('material', 'Materiál:')
			->setAttribute('rows', 2)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			->setAttribute('cols', 80);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
		$form->addText('tutor', 'Lektor:')
			->setAttribute('size', 30);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
		$form->addText('email', 'E-mail:')
			->setAttribute('size', 30);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
		$form->addText('capacity', 'Kapacita:')
			->setDefaultValue(0)
			->setAttribute('size', 10)
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			->setAttribute('placeholder', 0);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
		$form->addRadioList('display_in_reg', 'Zobrazit v registraci:', [1 => 'Ano', 0 => 'Ne'])
			->getSeparatorPrototype()->setName(null);
		$form->addSelect('block', 'Blok:', $this->buildBlockSelect());
		$form->addSelect('category', 'Kategorie:', $this->buildCategorySelect());

		$form->addHidden('id');
		$form->addHidden('guid');
		$form->addHidden('backlink');

		$form->addSubmit('save', 'Uložit')
			->setAttribute('class', 'btn-primary')
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			->onClick[] = [$this, 'processForm'];
		$form->addSubmit('reset', 'Storno')
			->setAttribute('class', 'btn-reset')
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
			->onClick[] = [$this, 'processReset'];

		$form->setDefaults(['display_in_reg' => 1]);

		$form = $this->setupRendering($form);

		return $form;
	}

	/**
	 * @param  SubmitButton $button
	 * @return void
	 */
	public function processForm(SubmitButton $button)
	{
		$program = $button->getForm()->getValues();
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}

		$this->onProgramSave($this, $program);
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
	}

	/**
	 * @param  SubmitButton $button
	 * @return void
	 */
	public function processReset(SubmitButton $button)
	{
		$program = $button->getForm()->getValues();
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}

		$this->onProgramReset($this, $program);
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
	}

	/**
	 * @return array
	 */
	protected function buildBlockSelect(): array
	{
		$blocks = $this->getBlockRepository()->findByMeeting($this->getMeetingId());

		$selectContent = [];

		foreach($blocks as $block) {
			$selectContent[$block->id] = sprintf(
				'%s, %s - %s : %s',
				$block->day,
				$block->from->format('%H:%I:%S'),
				$block->to->format('%H:%I:%S'),
				$block->name
			);
		}

		return $selectContent;
	}

	/**
	 * @return array
	 */
	protected function buildCategorySelect(): array
	{
		$categories = $this->getCategoryRepository()->findAll();

		$selectContent = [];

		foreach($categories as $category) {
			$selectContent[$category->id] = $category->name;
		}

		return $selectContent;
	}

	/**
	 * @return BlockRepository
	 */
	public function getBlockRepository(): BlockRepository
	{
		return $this->blockRepository;
	}

	/**
	 * @param BlockRepository $blockRepository
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
	 *
	 * @return self
	 */
	public function setBlockRepository(BlockRepository $repository): self
	{
		$this->blockRepository = $repository;

		return $this;
	}

	/**
	 * @return CategoryRepository
	 */
	public function getCategoryRepository(): CategoryRepository
	{
		return $this->categoryRepository;
	}

	/**
	 * @param CategoryRepository $categoryRepository
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
	 *
	 * @return self
	 */
	public function setCategoryRepository(CategoryRepository $repository): self
	{
		$this->categoryRepository = $repository;

		return $this;
	}
}


Push — master ( 118c60...254398 )

ProgramForm A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

12 Methods

Available Fixes

Available Fixes

Available Fixes

1		<?php
2
3		namespace App\Components\Forms;
4
5		use App\Models\MeetingModel;
6		use App\Repositories\BlockRepository;
7		use App\Repositories\CategoryRepository;
8		use Nette\Application\UI\Form;
9		use Nette\Database\Table\ActiveRow;
10		use Nette\Forms\Controls\SubmitButton;
11
12	1	class ProgramForm extends BaseForm
13		{
14
15		const TEMPLATE_NAME = 'ProgramForm';
16
17		const MESSAGE_REQUIRED = 'Hodnota musí být vyplněna!';
18		const MESSAGE_MAX_LENGTH = '%label nesmí mít více jak %d znaků!';
19
20		/**
21		* @var Closure
22		*/
23		public $onProgramSave;
24
25		/**
26		* @var Closure
27		*/
28		public $onProgramReset;
29
30		/**
31		* @var BlockRepository
32		*/
33		protected $blockRepository;
34
35		/**
36		* @var CategoryRepository
37		*/
38		protected $categoryRepository;
39
40		/**

literat / srazvs

Push — master ( 118c60...254398 )

ProgramForm A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

12 Methods

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like