App\Modules\V1\Reporting\Repositories\ReportRepository

Complexity

Total Complexity

6

Size/Duplication

Total Lines	65
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	3
Bugs	1	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getModel()	0	4	1
B	getReport()	0	42	5

<?php namespace App\Modules\V1\Reporting\Repositories;

use App\Modules\V1\Core\AbstractRepositories\AbstractRepository;

class ReportRepository extends AbstractRepository
{
    /**
     * Return the model full namespace.
     * 
     * @return string
     */
    protected function getModel()
    {
        return 'App\Modules\V1\Reporting\Report';
    }

    /**
     * Render the given report db view based on the given
     * condition.
     *
     * @param  string  $reportName
     * @param  array   $conditions array of conditions
     * @param  integer $perPage
     * @param  array   $relations
     * @return object
     */
    public function getReport($reportName, $conditions = false, $perPage = 0, $relations = [])
    {
        /**
         * Fetch the report from db.
         */
        $reportConditions = $this->constructConditions(['report_name' => $reportName]);
        $report           = call_user_func_array("{$this->getModel()}::with", array($relations))->whereRaw($reportConditions['conditionString'], $reportConditions['conditionValues'])->first();
        
        /**
         * Check report existance and permission.
         */
        if ( ! $report) 
        {
            \ErrorHandler::notFound('report');
        }
        else if ( ! \Core::users()->can($report->view_name, 'reports'))
        {
            \ErrorHandler::noPermissions();
        }

        /**
         * Fetch data from the report based on the given conditions.
         */
        $report = \DB::table($report->view_name);
        unset($conditions['page']);
        if (count($conditions))
        {
            $conditions = $this->constructConditions($conditions);

It seems like $conditions can also be of type false; however, App\Modules\V1\Core\AbstractRepositories\AbstractRepository::constructConditions() does only seem to accept array, did you maybe forget to handle an error condition?

            $report->whereRaw($conditions['conditionString'], $conditions['conditionValues']);   
        }
        /**
         * Paginate or all data.
         */
        if ($perPage) 
        {
            return $report->paginate($perPage);
        }
        else
        {
            return $report->get();  
        }
    }
}