Anomaly\ConfigurationModule\Configuration\Form\ConfigurationFormBuilder

Complexity

Total Complexity

5

Size/Duplication

Total Lines	77
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	onReady()	0	6	3
A	getScope()	0	4	1
A	setScope()	0	6	1

<?php namespace Anomaly\ConfigurationModule\Configuration\Form;

use Anomaly\Streams\Platform\Ui\Form\FormBuilder;

/**
 * Class ConfigurationFormBuilder
 *
 * @link          http://pyrocms.com/
 * @author        PyroCMS, Inc. <support@pyrocms.com>
 * @author        Ryan Thompson <ryan@pyrocms.com>
 */
class ConfigurationFormBuilder extends FormBuilder
{

    /**
     * The configuration scope.
     *
     * @var null|string
     */
    protected $scope = null;

    /**
     * No model needed.
     *
     * @var bool
     */
    protected $model = false;

    /**
     * The form fields.
     *
     * @var ConfigurationFormFields
     */
    protected $fields = ConfigurationFormFields::class;

    /**
     * The form actions handler.
     *
     * @var string
     */
    protected $actions = [
        'save',
    ];

    /**
     * The form buttons handler.
     *
     * @var string
     */
    protected $buttons = [
        'cancel',
    ];

    /**
     * Fired when the builder is ready to build.
     *
     * @throws \Exception
     */
    public function onReady()
    {
        if (!$this->getScope() && !$this->getEntry()) {

The expression $this->getScope() of type null|string is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

            throw new \Exception('The $scope parameter is required when creating configuration.');
        }
    }

    /**
     * Get the scope.
     *
     * @return null|string
     */
    public function getScope()
    {
        return $this->scope;
    }

    /**
     * Set the scope.
     *
     * @param $scope
     * @return $this
     */
    public function setScope($scope)
    {
        $this->scope = $scope;

        return $this;
    }
}