crocodicstudio\crudbooster\controllers\ApiController\HandleListAction

Complexity

Total Complexity

5

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	handleRows()	0	10	2
A	handle()	0	9	2
A	sortRows()	0	9	1

<?php

namespace crocodicstudio\crudbooster\controllers\ApiController;

class HandleListAction
{
    /**
     * @param $table
     * @param $data
     * @param $responsesFields
     * @param $ctrl
     * @return array
     */
    public static function handle($table, $data, $responsesFields, $ctrl)
    {
        $rows = self::sortRows($table, $data);
        if ($rows) {
            return self::handleRows($responsesFields, $rows);
        }
        $result = ApiResponder::makeResult(0, 'No data found !');
        $result['data'] = [];
        ApiResponder::send($result, request()->all(), $ctrl);
    }

    /**
     * @param $responsesFields
     * @param $rows
     * @return array
     */
    private static function handleRows($responsesFields, $rows)
    {
        foreach ($rows as &$row) {
            HandleDetailsAction::handleFile($row, $responsesFields);
        }

        $result = ApiResponder::makeResult(1, 'success');
        $result['data'] = $rows;

        return $result;
    }


    /**
     * @param $table
     * @param $data
     * @return mixed
     */
    private static function sortRows($table, $data)
    {
        $orderBy = request('orderby', $table.'.id,desc');

        list($orderByCol, $orderByVal) = explode(',', $orderBy);

It seems like $orderBy can also be of type array; however, parameter $string of explode() does only seem to accept string, maybe add an additional type check?

        $rows = $data->orderby($orderByCol, $orderByVal)->get();

        return $rows;
    }

}