LeKoala\Tabulator\BulkActions\GenericBulkAction

Complexity

Total Complexity

8

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Importance

Changes	1
Bugs	1	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getCallable()	0	3	1
A	getI18nLabel()	0	3	1
A	process()	0	12	3
A	setCallable()	0	4	1
A	__construct()	0	9	2

<?php

namespace LeKoala\Tabulator\BulkActions;

use SilverStripe\Control\HTTPRequest;
use LeKoala\Tabulator\AbstractBulkAction;

/**
 *  Generic bulk action that can run a callable
 */
class GenericBulkAction extends AbstractBulkAction
{
    protected string $name = '';
    protected string $label = '';
    protected $callable = null;
    protected bool $xhr = true;

    public function __construct($name, $label, $callable = null)
    {
        parent::__construct();

        $this->name = $name;
        $this->label = $label;

        if ($callable) {
            $this->callable = $callable;
        }
    }

    public function getI18nLabel(): string
    {
        return $this->getLabel();
    }

    public function process(HTTPRequest $request): string
    {
        $records = $this->getRecords() ?? [];

Are you sure the usage of $this->getRecords() targeting LeKoala\Tabulator\AbstractBulkAction::getRecords() seems to always return null.

        $i = 0;
        foreach ($records as $record) {
            if ($this->callable) {
                $this->callable($record, $this->tabulatorGrid);

The method callable() does not exist on LeKoala\Tabulator\BulkActions\GenericBulkAction. Since you implemented __call, consider adding a @method annotation.

            }
            $i++;
        }
        $result = _t(__CLASS__ . ".RECORDSPROCSSED", "{count} records processed", ["count" => $i]);
        return $result;
    }

    /**
     * Get the value of callable
     */
    public function getCallable()
    {
        return $this->callable;
    }

    /**
     * Set the value of callable
     *
     * @param callable $callable
     */
    public function setCallable($callable): self
    {
        $this->callable = $callable;
        return $this;
    }
}