LeKoala\Tabulator\BulkActions\BulkDeleteAction

Complexity

Total Complexity

3

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	process()	0	10	2
A	getI18nLabel()	0	3	1

<?php

namespace LeKoala\Tabulator\BulkActions;

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

/**
 * Bulk action handler for deleting records.
 */
class BulkDeleteAction extends AbstractBulkAction
{
    protected string $name = 'delete';
    protected string $label = 'Delete';
    protected bool $xhr = true;
    protected bool $destructive = true;

    public function getI18nLabel(): string
    {
        return _t(__CLASS__ . '.DELETE_SELECT_LABEL', $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) {
            $record->delete();
            $i++;
        }
        $result = _t(__CLASS__ . ".RECORDSDELETED", "{count} records deleted", ["count" => $i]);
        return $result;
    }
}