Sunnysideup\ConfigManager\Control\CheckConfigsController

Complexity

Total Complexity

14

Size/Duplication

Total Lines	149
Duplicated Lines	0 %

Importance

Changes	7
Bugs	0	Features	2

7 Methods

Rating	Name	Duplication	Size	Complexity
A	Title()	0	3	1
A	init()	0	5	2
A	package()	0	3	1
A	location()	0	8	2
A	Data()	0	24	5
A	model()	0	4	1
A	index()	0	69	2

<?php

namespace Sunnysideup\ConfigManager\Control;

use SilverStripe\Control\Controller;
use SilverStripe\Core\Config\Config;
use SilverStripe\Security\Permission;
use SilverStripe\Security\Security;
use Sunnysideup\ConfigManager\Api\ConfigList;
use Sunnysideup\ConfigManager\View\YmlProvider;
use Sunnysideup\TableFilterSort\Api\TableFilterSortAPI;

class CheckConfigsController extends Controller
{
    protected $title = 'Check configs';

    protected $description = 'Runs through all classes and looks for private statics';

    private static $url_segment = 'dev/checkconfigs';

    /**
     * Defines methods that can be called directly.
     *
     * @var array
     */
    private static $allowed_actions = [
        'index' => 'ADMIN',
        'location' => 'ADMIN',
        'package' => 'ADMIN',
        'model' => 'ADMIN',
    ];

    public function Title()
    {
        return $this->title;
    }

    public function location()
    {
        $location = $this->request->param('ID');
        if (! $location) {
            die('please set a location in the url /dev/checkconfigs/location/framework/ - e.g. framework');

Using exit here is not recommended.

        }

        return (new YmlProvider())->getYmlForLocation($location);
    }

    public function model()
    {
        return (new YmlProvider())
            ->getModel()
        ;
    }

    public function package()
    {
        return (new YmlProvider())->getYmlForPackage($this->request->param('ID'), $this->request->param('OtherID'));
    }

    public function index($request)

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

    {
        if (class_exists(\Sunnysideup\WebpackRequirementsBackend\View\RequirementsBackendForWebpack::class, true)) {

The type Sunnysideup\WebpackRequirementsBackend\View\RequirementsBackendForWebpack was not found. Maybe you did not declare it correctly or list all dependencies?

            Config::modify()->set(
                \Sunnysideup\WebpackRequirementsBackend\View\RequirementsBackendForWebpack::class,
                'enabled',
                false
            );
        }
        TableFilterSortAPI::include_requirements(
            $tableSelector = '.tfs-holder',
            $blockArray = [],
            $jqueryLocation = 'https://code.jquery.com/jquery-3.5.1.min.js',
            $includeInPage = false,
            $jsSettings = [
                'rowRawData' => $this->Data(),
                'scrollToTopAtPageOpening' => true,
                'sizeOfFixedHeader' => 0,
                'maximumNumberOfFilterOptions' => 20,
                'filtersParentPageID' => '',
                'favouritesParentPageID' => '',
                'visibleRowCount' => 20,
                'startWithOpenFilter' => true,
                'dataDictionary' => [
                    'Type' => [
                        'Label' => 'Type',
                        'CanFilter' => true,
                    ],
                    'Vendor' => [
                        'Label' => 'Vendor',
                        'CanFilter' => true,
                    ],
                    'Package' => [
                        'Label' => 'Package',
                        'CanFilter' => true,
                    ],
                    'ShorterClassName' => [
                        'Label' => 'Class',
                        'CanFilter' => true,
                    ],
                    'Property' => [
                        'Label' => 'Property',
                        'CanFilter' => true,
                    ],
                    'IsDefault' => [
                        'Label' => 'Is Default',
                        'CanFilter' => true,
                    ],
                    'HasDefault' => [
                        'Label' => 'Has Default',
                        'CanFilter' => false,
                    ],
                    'HasValue' => [
                        'Label' => 'Has Value',
                        'CanFilter' => true,
                    ],
                    'Value' => [
                        'Label' => 'Value',
                        'CanFilter' => false,
                    ],
                    'Default' => [
                        'Label' => 'Default Value',
                        'CanFilter' => false,
                    ],
                ],
            ]
        );

        return $this->renderWith('Sunnysideup/ConfigManager/Control/CheckConfigsTable');
    }

    public function Data()
    {
        $list = new ConfigList();
        $list = $list->getListOfConfigs();
        ksort($list);
        $finalArray = [];
        $count = 0;
        foreach ($list as $values) {
            ++$count;
            $id = 'row' . $count;
            $finalArray[$id] = [];
            $finalArray[$id]['Vendor'] = $values['Vendor'];
            $finalArray[$id]['Package'] = $values['Package'];
            $finalArray[$id]['ShorterClassName'] = $values['ShorterClassName'];
            $finalArray[$id]['Property'] = $values['Property'];
            $finalArray[$id]['Type'] = $values['Type'];
            $finalArray[$id]['IsDefault'] = $values['IsDefault'] ? 'yes' : 'no';
            $finalArray[$id]['HasDefault'] = $values['HasDefault'] ? 'yes' : 'no';
            $finalArray[$id]['HasValue'] = $values['HasValue'] ? 'yes' : 'no';
            $finalArray[$id]['Value'] = '<pre>' . print_r($values['Value'], 1) . '</pre>';

Are you sure print_r($values['Value'], 1) of type string|true can be used in concatenation?

            $finalArray[$id]['Default'] = '<pre>' . print_r($values['Default'], 1) . '</pre>';

Are you sure print_r($values['Default'], 1) of type string|true can be used in concatenation?

        }

        return $finalArray;
    }

    protected function init()
    {
        parent::init();
        if (! Permission::check('Admin')) {
            return Security::permissionFailure($this);
        }
    }
}