Crocodicstudio\Crudbooster\Controllers\CBController\ExportData::postExportData()

Complexity

Conditions	3
Paths	4

Size

Total Lines

16

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function postExportData()
    {
        $this->limit = request('limit');

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

        $this->indexReturn = true;

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

        $filename = request('filename');
        $papersize = request('page_size');
        $paperorientation = request('page_orientation');
        $indexContent = $this->getIndex();

It seems like getIndex() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if (request('default_paper_size')) {
            $this->table('cms_settings')->where('name', 'default_paper_size')->update(['content' => $papersize]);

It seems like table() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
        $format = request('fileformat');
        if(in_array($format, ['pdf', 'xls', 'csv']))
        {
            return app(IndexExport::class)->{$format}($filename, $indexContent, $paperorientation, $papersize);
        }
}