Encore\Admin\Form\Field\CheckboxButton

Complexity

Total Complexity

2

Size/Duplication

Total Lines	46
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	addScript()	21	21	1
A	render()	13	13	1

<?php

namespace Encore\Admin\Form\Field;

use Encore\Admin\Admin;

class CheckboxButton extends Checkbox

This class seems to be duplicated in your project.

{
    /**
     * @var string
     */
    protected $cascadeEvent = 'change';

    protected function addScript()
    {
        $script = <<<SCRIPT
$('.checkbox-group-toggle label').click(function(e) {
    e.stopPropagation();
    e.preventDefault();

    if ($(this).hasClass('active')) {
        $(this).removeClass('active');
        $(this).find('input').prop('checked', false);
    } else {
        $(this).addClass('active');
        $(this).find('input').prop('checked', true);
    }

    $(this).find('input').trigger('change');
});
SCRIPT;

        Admin::script($script);
    }

    /**
     * {@inheritdoc}
     */
    public function render()
    {
        $this->addScript();

        $this->addCascadeScript();

        $this->addVariables([
            'options' => $this->options,
            'checked' => $this->checked,
        ]);

        return parent::fieldRender();

The expression parent::fieldRender(); of type string|Illuminate\View\View|Illuminate\Contracts\View\Factory adds the type Illuminate\Contracts\View\Factory to the return on line 50 which is incompatible with the return type declared by the interface Illuminate\Contracts\Support\Renderable::render of type string.

It seems like you call parent on a different method (fieldRender() instead of render()). Are you sure this is correct? If so, you might want to change this to $this->fieldRender().

    }
}