Sunnysideup\HealthCheckProvider\Checks\Database\SizeOfDatabase

Complexity

Total Complexity

4

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	getCalculatedAnswer()	0	13	4

<?php

namespace Sunnysideup\HealthCheckProvider\Checks\Database;

use SilverStripe\ORM\DB;
use Sunnysideup\HealthCheckProvider\Checks\HealthCheckItemRunner;

class SizeOfDatabase extends HealthCheckItemRunner
{
    private static $fields_required = [

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

        'Name',
        'Engine',
        'Data_length',
        'Index_length',
        'Collation',
    ];

    public function getCalculatedAnswer(): array
    {
        $rows = DB::query('SHOW TABLE STATUS;');
        $allowedKeys = $this->Config()->get('fields_required');
        $array = $rows;
        foreach ($array as $pos => $row) {
            foreach (array_keys($row) as $key) {
                if (! in_array($key, $allowedKeys, true)) {
                    unset($array[$pos][$key]);
                }
            }
        }
        return $array;

The expression return $array returns the type SilverStripe\ORM\Connect\Query which is incompatible with the type-hinted return array.

    }
}