SilverStripe\Comments\Admin\CommentsGridFieldBulkAction\SpamHandler

Complexity

Total Complexity

2

Size/Duplication

Total Lines	37
Duplicated Lines	100 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	index()	18	18	2

<?php

namespace SilverStripe\Comments\Admin\CommentsGridFieldBulkAction;

use Colymba\BulkManager\BulkAction\Handler;
use SilverStripe\Core\Convert;
use SilverStripe\Control\HTTPRequest;
use SilverStripe\Control\HTTPResponse;

/**
 * A {@link GridFieldBulkActionHandler} for bulk marking comments as spam
 */
class SpamHandler extends Handler

This class seems to be duplicated in your project.

{
    private static $allowed_actions = ['index'];

Consider using a different property name as you override a private property of the parent class.

The property $allowed_actions is not used and could be removed.

    private static $url_segment = 'spam';

Consider using a different property name as you override a private property of the parent class.

The property $url_segment is not used and could be removed.

    protected $xhr = true;

    protected $buttonClasses = 'font-icon-cross-mark';

    protected $destructive = false;

    protected $label = 'Spam';

    /**
     * @param  HTTPRequest $request
     * @return HTTPResponse
     */
    public function index(HTTPRequest $request)

The parameter $request is not used and could be removed.

    {
        $ids = array();

        foreach ($this->getRecords() as $record) {
            array_push($ids, $record->ID);
            $record->markSpam();
        }

        $response = new HTTPResponse(Convert::raw2json(array(
            'done' => true,
            'records' => $ids
        )));

        $response->addHeader('Content-Type', 'text/json');

        return $response;
    }
}