Sunnysideup\ExportAllFromModelAdmin\ExportAllFromModelAdminTrait::getExportFields()

Complexity

Conditions	4
Paths	4

Size

Total Lines	25
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes	15
Bugs	0	Features	0

<?php
    public function getExportFields(): array
    {
        if (Permission::check('ADMIN')) {
            //set to ten minutes
            Environment::setTimeLimitMax(600);
            $singleton = Injector::inst()->get($this->modelClass);
            if ($singleton) {
                $obj = AllFields::create($this->modelClass);
                $this->exportFields = $obj->getExportFields();

The property exportFields does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

                if ($singleton->hasMethod('getFieldsToIncludeInExport')) {
                    $this->exportFields += $singleton->getFieldsToIncludeInExport();
                }
                // if(Director::isDev()) {
                //     foreach($this->exportFields as $fieldName => $title) {
                //         echo "\n'$fieldName',";
                //     }
                // }
            } else {
                $this->exportFields = parent::getExportFields();
            }
        } else {
            $this->exportFields = parent::getExportFields();
        }

        return $this->exportFields;
    }