GeminiLabs\SiteReviews\Integrations\Bricks\BricksElement

Complexity

Total Complexity

24

Size/Duplication

Total Lines	158
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

12 Methods

Rating	Name	Duplication	Size	Complexity
A	load()	0	14	4
A	registerElement()	0	7	1
A	set_control_groups()	0	12	2
A	__construct()	0	7	1
A	setNumberControl()	0	8	1
A	get_label()	0	3	1
A	setCheckboxControl()	0	13	3
A	elementConfig()	0	3	1
A	render()	0	5	1
A	get_keywords()	0	3	1
A	set_controls()	0	22	5
A	setSelectControl()	0	15	3

<?php

namespace GeminiLabs\SiteReviews\Integrations\Bricks;

use GeminiLabs\SiteReviews\Contracts\ShortcodeContract;
use GeminiLabs\SiteReviews\Helper;

abstract class BricksElement extends \Bricks\Element

The type Bricks\Element was not found. Maybe you did not declare it correctly or list all dependencies?

{
    public $nestable = false;
    public $scripts = ['GLSR_init'];

    /**
     * @var ShortcodeContract
     */
    protected $elShortcode;

    public function __construct($element = null)
    {
        $this->category = glsr()->id;

The property category does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->elShortcode = static::shortcode();
        $this->icon = "ti-{$this->elShortcode->tag}";

The property icon does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

Accessing tag on the interface GeminiLabs\SiteReviews\Contracts\ShortcodeContract suggest that you code against a concrete implementation. How about adding an instanceof check?

        $this->name = $this->elShortcode->tag;

The property name does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        parent::__construct($element);
    }

    public function elementConfig(): array
    {
        return $this->elShortcode->settings();
    }

    public function get_keywords()
    {
        return ['review', 'reviews', 'site reviews'];
    }

    public function get_label()
    {
        return $this->elShortcode->name;
    }

    public function load()
    {
        parent::load();
        $groups = array_map(fn ($args) => $args['group'] ?? '', $this->controls);
        $groups = array_filter($groups, fn ($value) => !str_starts_with($value, '_'));
        $groups = array_values(array_filter(array_unique($groups)));
        foreach ($this->control_groups as $key => $value) {
            if (str_starts_with($key, '_')) {
                continue;
            }
            if (in_array($key, $groups)) {
                continue;
            }
            unset($this->control_groups[$key]);
        }
    }

    public static function registerElement(): void
    {
        $reflection = new \ReflectionClass(static::class);
        $file = $reflection->getFileName();
        $name = static::shortcode()->tag;

Accessing tag on the interface GeminiLabs\SiteReviews\Contracts\ShortcodeContract suggest that you code against a concrete implementation. How about adding an instanceof check?

        $className = $reflection->getName();
        \Bricks\Elements::register_element($file, $name, $className);

The type Bricks\Elements was not found. Maybe you did not declare it correctly or list all dependencies?

    }

    public function render()
    {
        echo "<{$this->get_tag()} {$this->render_attributes('_root')}>";
        echo $this->elShortcode->build($this->settings, 'bricks');
        echo "</{$this->get_tag()}>";
    }

    /**
     * @see https://academy.bricksbuilder.io/article/filter-bricks-elements-element_name-control_groups/
     */
    public function set_control_groups()
    {
        $groups = [ // order is intentional
            'display' => esc_html_x('Display', 'admin-text', 'site-reviews'),
            'hide' => esc_html_x('Hide', 'admin-text', 'site-reviews'),
            'schema' => esc_html_x('Schema', 'admin-text', 'site-reviews'),
            'advanced' => esc_html_x('Advanced', 'admin-text', 'site-reviews'),
        ];
        foreach ($groups as $key => $title) {
            $this->control_groups[$key] = [

The property control_groups does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

                'tab' => 'content',
                'title' => $title,
            ];
        }
    }

    /**
     * @see https://academy.bricksbuilder.io/article/filter-bricks-elements-element_name-controls
     */
    public function set_controls()
    {
        foreach ($this->elementConfig() as $key => $args) {
            $args = wp_parse_args($args, [
                'group' => '',
                // 'hasDynamicData' => false,
                // 'hasVariables' => true,
                'tab' => 'content',
                'type' => 'text',
            ]);
            if (!array_key_exists($args['group'], $this->control_groups)) {
                $args['group'] = '';
            }
            $method = Helper::buildMethodName('set', $args['type'], 'control');
            if (!method_exists($this, $method)) {
                $this->controls[$key] = $args;

The property controls does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

                continue;
            }
            call_user_func([$this, $method], $key, $args);
        }
        if (count(glsr()->retrieveAs('array', 'review_types', [])) < 2) {
            unset($this->controls['type']);
        }
    }

    abstract public static function shortcode(): ShortcodeContract;

    protected function setCheckboxControl(string $key, array $args): void
    {
        if (empty($args['options'])) {
            $this->controls[$key] = $args;

The property controls does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            return;
        }
        foreach ($args['options'] as $value => $label) {
            $this->controls["{$key}_{$value}"] = [
                'default' => false,
                'group' => $args['group'],
                'label' => $label,
                'tab' => 'content',
                'type' => 'checkbox',
            ];
        }
    }

    protected function setNumberControl(string $key, array $args): void
    {
        $control = [
            // 'small' => true,
            'type' => 'slider',
            'units' => false,
        ];
        $this->controls[$key] = wp_parse_args($control, $args);

The property controls does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

    protected function setSelectControl(string $key, array $args): void
    {
        if (!in_array($key, ['assigned_posts', 'assigned_terms', 'assigned_users', 'post_id'])) {
            $this->controls[$key] = $args;

The property controls does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            return;
        }
        $control = [
            'type' => 'select',
            'searchable' => true,
            'optionsAjax' => ['action' => "bricks_glsr_{$key}"],
        ];
        if ('post_id' !== $key) {
            $control['multiple'] = true;
        }
        $this->controls[$key] = wp_parse_args($control, $args);
    }
}