Crocodicstudio\Crudbooster\Controllers\CBController\ExportData

Complexity

Total Complexity

4

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	postExportData()	0	16	3
A	getExportData()	0	3	1

<?php

namespace Crocodicstudio\Crudbooster\Controllers\CBController;

use Crocodicstudio\Crudbooster\Controllers\Helpers\IndexExport;

trait ExportData
{
    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);
        }
    }

    public function getExportData()
    {
        return redirect(CRUDBooster::mainpath());

The type Crocodicstudio\Crudbooster\Controllers\CBController\CRUDBooster was not found. Maybe you did not declare it correctly or list all dependencies?

    }

}