LeKoala\Tabulator\BulkActions\BulkUnpublishAction

Complexity

Total Complexity

3

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Importance

Changes	2
Bugs	1	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 unpublishing records.
 */
class BulkUnpublishAction extends AbstractBulkAction
{
    protected string $name = 'unpublish';
    protected string $label = 'Unpublish';
    protected bool $xhr = true;

    public function getI18nLabel(): string
    {
        return _t(__CLASS__ . '.UNPUBLISH_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->doUnpublish();
            $i++;
        }
        $result = _t(__CLASS__ . ".RECORDSUNPUBLISHED", "{count} records unpublished", ["count" => $i]);
        return $result;
    }
}