signify-nz / silverstripe-security-headers

src/Models/CSPViolation.php

<?php

namespace Signify\Models;

use SilverStripe\ORM\DataObject;
use SilverStripe\ORM\FieldType\DBField;

class CSPViolation extends DataObject
{
    private static $plural_name = 'CSP Violations';

The private property $plural_name is not used, and could be removed.

    private static $table_name = 'Signify_CSPViolation';

The private property $table_name is not used, and could be removed.

    private static $db = [

The private property $db is not used, and could be removed.

        'ReportedTime' => 'Datetime',
        'Disposition' => 'Varchar(7)',
        'BlockedURI' => 'Varchar(255)',
        'EffectiveDirective' => 'Varchar(255)',
        'Violations' => 'Int',
    ];

    private static $many_many = [

The private property $many_many is not used, and could be removed.

        'Documents' => CSPDocument::class,
    ];

    private static $summary_fields = [

The private property $summary_fields is not used, and could be removed.

        'ReportedTime' => 'Latest Report',
        'Disposition',
        'BlockedURI',
        'DocumentURIs',
        'EffectiveDirective',
        'Violations',
    ];

    private static $default_sort = 'ReportedTime DESC';

The private property $default_sort is not used, and could be removed.

    public function getDocumentURIs()
    {
        return DBField::create_field('Text', implode(', ', $this->Documents()->Column('URI')));

The method Documents() does not exist on Signify\Models\CSPViolation. Since you implemented __call, consider adding a @method annotation.

    }
}