App\Controller\NotificationsController::data_tables()

Complexity

Conditions	3
Paths	3

Size

Total Lines	80
Code Lines	57

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	47
CRAP Score	3

Importance

Changes

0

<?php
    public function data_tables(): Response
    {
        $current_developer = TableRegistry::getTableLocator()->get('Developers')->
                    findById($this->request->getSession()->read('Developer.id'))->all()->first();

        $aColumns = [
            'report_id' => 'Reports.id',
            'error_message' => 'Reports.error_message',
            'error_name' => 'Reports.error_name',
            'pma_version' => 'Reports.pma_version',
            'exception_type' => 'Reports.exception_type',
            'created_time' => 'Notifications.created',
        ];

        $orderConditions = $this->OrderSearch->getOrder($aColumns);

The property OrderSearch does not exist on App\Controller\NotificationsController. Since you implemented __get, consider adding a @property annotation.

        $searchConditions = $this->OrderSearch->getSearchConditions($aColumns);

        $aColumns['id'] = 'Notifications.id';
        $params = [
            'contain' => 'Reports',
            'fields' => $aColumns,
            'conditions' => [
                'AND' => [
                    ['Notifications.developer_id ' => $current_developer['id']],
                    $searchConditions,
                ],
            ],
            'order' => $orderConditions,
        ];

        $pagedParams = $params;
        $pagedParams['limit'] = intval($this->request->query('iDisplayLength'));

The function Cake\Http\ServerRequest::query() has been deprecated: 3.4.0 Use getQuery() or the PSR-7 getQueryParams() and withQueryParams() methods instead.

        $pagedParams['offset'] = intval($this->request->query('iDisplayStart'));

The function Cake\Http\ServerRequest::query() has been deprecated: 3.4.0 Use getQuery() or the PSR-7 getQueryParams() and withQueryParams() methods instead.

        $rows = $this->Notifications->find('all', $pagedParams);

The property Notifications does not exist on App\Controller\NotificationsController. Since you implemented __get, consider adding a @property annotation.

        $totalFiltered = $this->Notifications->find(
            'all',
            [
                'conditions' => [
                    'developer_id' => $current_developer['id'],
                ],
            ]
        )->count();

        // Make the display rows array
        $dispRows = [];
        $tmp_row = [];
        foreach ($rows as $row) {
            $tmp_row[0] = '<input type="checkbox" name="notifs[]" value="'
                . $row['id']
                . '"/>';
            $tmp_row[1] = '<a href="'
                . Router::url(
                    [
                        'controller' => 'reports',
                        'action' => 'view',
                        $row['report_id'],
                    ]
                )
                . '">'
                . $row['report_id']
                . '</a>';
            $tmp_row[2] = $row['error_name'];
            $tmp_row[3] = $row['error_message'];
            $tmp_row[4] = $row['pma_version'];
            $tmp_row[5] = $row['exception_type'] ? 'php' : 'js';
            $tmp_row[6] = $row['created_time'];
            array_push($dispRows, $tmp_row);
        }

        $response = [
            'iTotalDisplayRecords' => $totalFiltered,
            'iTotalRecords' => $this->Notifications->find('all', $params)->count(),
            'sEcho' => intval($this->request->query('sEcho')),

The function Cake\Http\ServerRequest::query() has been deprecated: 3.4.0 Use getQuery() or the PSR-7 getQueryParams() and withQueryParams() methods instead.

            'aaData' => $dispRows,
        ];
        $this->autoRender = false;
        $this->response->body(json_encode($response));

The function Cake\Http\Response::body() has been deprecated: 3.4.0 Mutable response methods are deprecated. Use `withBody()`/`withStringBody()` and `getBody()` instead.

        return $this->response;
    }