Fabrica\Http\Api\ReportController::getRegressions()

Complexity

Conditions	43
Paths	> 20000

Size

Total Lines

162

Duplication

Lines	43
Ratio	26.54 %

Importance

Changes

0

<?php
    public function getRegressions(Request $request, $project_key)
    {
        $where = $this->getIssueQueryWhere($project_key, $request->all());

        $his_resolvers = $request->input('his_resolvers') ? explode(',', $request->input('his_resolvers')) : [];
        $or = [];
        foreach ($his_resolvers as $resolver)
        {
            $or[] = [ 'his_resolvers' => $resolver ];
        }
        if ($or) {

The expression $or of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $where['$and'][] = [ '$or' => $or ];
        }

        $where['$and'][] = [ 'regression_times' => [ '$exists' => 1 ] ];

        $sprint_start_time = $sprint_complete_time = 0;
        $sprint_no = $request->input('sprint') ?: '';
        if ($sprint_no) {
            $sprint = Sprint::where('project_key', $project_key)->where('no', intval($sprint_no))->first();

            $sprint_start_time = strtotime(date('Ymd', $sprint->start_time));
            if (isset($sprint->real_complete_time) && $sprint->real_complete_time > 0) {
                $sprint_complete_time = $sprint->real_complete_time;
            }
        }

        $dimension = $request->input('stat_dimension') ?: '';
        if ($dimension) {
            $results = [];

$results is not used, you could remove the assignment.

        }
        else
        {
            $results = [ 'ones' => [], 'gt_ones' => [] ];

$results is not used, you could remove the assignment.

        }

        $results = [];
        $query = DB::collection('issue_' . $project_key)->whereRaw($where);
        $issues = $query->orderBy('no', 'desc')->take(1000)->get();
        foreach($issues as $issue)
        {
            $no = $issue['no'];
            $regression_times = $issue['regression_times'];
            $resolved_logs = isset($issue['resolved_logs']) ? $issue['resolved_logs'] : [];

            if ($sprint_start_time > 0) {
                $tmp_log = [];
                foreach($resolved_logs as $rl)
                {
                    if ($rl['at'] > $sprint_start_time && ($sprint_complete_time <= 0 || $rl['at'] < $sprint_complete_time)) {
                        if ($his_resolvers && !in_array($rl['user']['id'], $his_resolvers)) {

The expression $his_resolvers of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

                            continue;
                        }
                        else
                        {
                            $tmp_log[] = $rl;
                        }
                    }
                }
                if (!$tmp_log) {

The expression $tmp_log of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

                    continue;
                }

                $resolved_logs = $tmp_log;
                $regression_times = count($tmp_log);
            }

            if ($dimension == 'resolver') {
                $log_cnt = count($resolved_logs);
                $tmp_uids = [];
                foreach ($resolved_logs as $key => $log) 
                {
                    $uid = $log['user']['id'];
                    if (in_array($uid, $tmp_uids)) {
                        continue;
                    }
                    else
                    {
                        $tmp_uids[] = $uid;
                    }
                    
                    if (!isset($results[$uid])) {

This code seems to be duplicated across your project.

                        $results[$uid] = [ 'ones' => [], 'gt_ones' => [], 'name' => isset($log['user']['name']) ? $log['user']['name'] : '' ];
                    }
                    if ($regression_times > 1) {
                        if ($key == $log_cnt - 1) {
                            $results[$uid]['ones'][] = $no;
                        }
                        else
                        {
                            $results[$uid]['gt_ones'][] = $no;
                        }
                    }
                    else
                    {
                        $results[$uid]['ones'][] = $no;
                    }
                }
            }
            else if ($dimension) {
                $dimension_values = isset($issue[$dimension]) ? $issue[$dimension] : '';
                if ($dimension_values && is_array($dimension_values)) {
                    if (isset($dimension_values['id'])) {
                        if (!isset($results[$dimension_values['id']])) {

This code seems to be duplicated across your project.

                            $results[$dimension_values['id']] = [ 'ones' => [], 'gt_ones' => [], 'name' => isset($dimension_values['name']) ? $dimension_values['name'] : '' ];
                        }
                        if ($regression_times > 1) {
                            $results[$dimension_values['id']]['gt_ones'][] = $no;
                        }
                        else
                        {
                            $results[$dimension_values['id']]['ones'][] = $no;
                        }
                    }
                    else

This code seems to be duplicated across your project.

                    {
                        foreach($dimension_values as $val)
                        {
                            if (!isset($results[$val])) {
                                $results[$val] = [ 'ones' => [], 'gt_ones' => [] ];
                            }

                            if ($regression_times > 1) {
                                $results[$val]['gt_ones'][] = $no;
                            }
                            else
                            {
                                $results[$val]['ones'][] = $no;
                            }
                        }                        
                    }
                }
                else if ($dimension_values) {

This code seems to be duplicated across your project.

                    if (!isset($results[$dimension_values])) {
                        $results[$dimension_values] = [ 'ones' => [], 'gt_ones' => [] ];
                    }

                    if ($regression_times > 1) {
                        $results[$dimension_values]['gt_ones'][] = $no;
                    }
                    else
                    {
                        $results[$dimension_values]['ones'][] = $no;
                    }
                }
            }
            else
            {
                if (!isset($results[0])) {
                    $results[0] = [ 'ones' => [], 'gt_ones' => [] ];
                }
                if ($regression_times > 1) {
                    $results[0]['gt_ones'][] = $no;
                }
                else
                {
                    $results[0]['ones'][] = $no;
                }
            }
        }
        return response()->json([ 'ecode' => 0, 'data' => $this->arrangeRegressionData($results, $project_key, $dimension) ]);

The method json does only exist in Illuminate\Contracts\Routing\ResponseFactory, but not in Illuminate\Http\Response.

    }