BulkPublishAction::process() - Code Metrics - lekoala/silverstripe-tabulator - Measure and Improve Code Quality continuously with Scrutinizer

BulkPublishAction::process() A
last analyzed 2025-02-07 09:14 UTC

↳ Parent: BulkPublishAction

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	7
c	1
b	0
f	0
dl	0
loc	10
rs	10
cc	2
nc	2
nop	1

<?php

namespace LeKoala\Tabulator\BulkActions;

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

/**
 * Bulk action handler for publishing records.
 */
class BulkPublishAction extends AbstractBulkAction
{
    protected string $name = 'publish';
    protected string $label = 'Publish';
    protected bool $xhr = true;

    public function getI18nLabel(): string
    {
        return _t(__CLASS__ . '.PUBLISH_SELECT_LABEL', $this->getLabel());
    }

    public function process(HTTPRequest $request): string
    {
        $records = $this->getRecords() ?? [];
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
        $i = 0;
        foreach ($records as $record) {
            $record->publishRecursive();
            $i++;
        }
        $result = _t(__CLASS__ . ".RECORDSPUBLISHED", "{count} records published", ["count" => $i]);
        return $result;
    }
}


1			<?php
2
3			namespace LeKoala\Tabulator\BulkActions;
4
5			use SilverStripe\Control\HTTPRequest;
6			use LeKoala\Tabulator\AbstractBulkAction;
7
8			/**
9			* Bulk action handler for publishing records.
10			*/
11			class BulkPublishAction extends AbstractBulkAction
12			{
13			protected string $name = 'publish';
14			protected string $label = 'Publish';
15			protected bool $xhr = true;
16
17			public function getI18nLabel(): string
18			{
19			return _t(__CLASS__ . '.PUBLISH_SELECT_LABEL', $this->getLabel());
20			}
21
22			public function process(HTTPRequest $request): string
23			{
24			$records = $this->getRecords() ?? [];
			0 ignored issues – show Bug introduced 2022-06-17 16:05 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->getRecords()` targeting `LeKoala\Tabulator\AbstractBulkAction::getRecords()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
25			$i = 0;
26			foreach ($records as $record) {
27			$record->publishRecursive();
28			$i++;
29			}
30			$result = _t(__CLASS__ . ".RECORDSPUBLISHED", "{count} records published", ["count" => $i]);
31			return $result;
32			}
33			}
34

lekoala / silverstripe-tabulator

BulkPublishAction::process() A last analyzed 2025-02-07 09:14 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

BulkPublishAction::process() A
last analyzed 2025-02-07 09:14 UTC