Datatable::applyCachedSetupClosure() - Code Metrics - Inspection of "wip" - Laravel-Backpack/CRUD - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — datatable-single-component ( cc93c3...b8f315 )

by Pedro

created 2025-06-04 12:42 UTC

Datatable::applyCachedSetupClosure() B

↳ Parent: Datatable

Complexity

Conditions	10
Paths	13

Size

Total Lines	41
Code Lines	24

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	10
eloc	24
nc	13
nop	1
dl	0
loc	41
rs	7.6666
c	0
b	0
f	0

How to fix Complexity

<?php

namespace Backpack\CRUD\app\Library\Datatable;

use Backpack\CRUD\app\Library\CrudPanel\CrudPanel;
use Backpack\CRUD\app\Library\Widget;
use Backpack\CRUD\CrudManager;
use Illuminate\Support\Facades\Cache;
use Illuminate\View\Component;

class Datatable extends Component
{
    protected string $tableId;

    public function __construct(
        private string $controller,
        private ?CrudPanel $crud = null,
        private bool $modifiesUrl = false,
        private ?\Closure $setup = null,
        private ?string $name = null,
    ) {
        // Set active controller for proper context
        CrudManager::setActiveController($controller);
                     setActiveController($controller);

        $this->crud ??= CrudManager::setupCrudPanel($controller, 'list');
        $this->crud ??= CrudManager::/** @scrutinizer ignore-call */ setupCrudPanel($controller, 'list');

        $this->tableId = $this->generateTableId();

        if ($this->setup) {
            // Apply the configuration
            ($this->setup)($this->crud, $this->getParentCrudEntry());

            // Store the configuration in cache for Ajax requests
            $this->cacheSetupClosure();
        }

        if (! $this->crud->has('list.datatablesUrl')) {
        if (! $this->crud->/** @scrutinizer ignore-call */ has('list.datatablesUrl')) {
            $this->crud->set('list.datatablesUrl', $this->crud->getRoute());
        }

        // Reset the active controller
        CrudManager::unsetActiveController($controller);
                     unsetActiveController($controller);
    }

    private function getParentCrudEntry()
    {
        $cruds = CrudManager::getCrudPanels();
        $cruds = CrudManager::getCrudPanels();
        $parentCrud = reset($cruds);

        if ($parentCrud && $parentCrud->getCurrentEntry()) {
            CrudManager::storeInitializedOperation(
                         storeInitializedOperation(
                $parentCrud->controller,
                $parentCrud->getCurrentOperation()
            );
            return $parentCrud->getCurrentEntry();
        }

        return null;
    }

    private function generateTableId(): string
    {
        $controllerPart = str_replace('\\', '_', $this->controller);
        $namePart = $this->name ?? 'default';
        $uniqueId = md5($controllerPart.'_'.$namePart);

        return 'crudTable_'.$uniqueId;
    }

    /**
     * Store the datatable configuration in the cache for later use in Ajax requests.
     */
    private function cacheSetupClosure()
    {
        if (! $this->setup) {
            return;
        }

        $controllerClass = $this->controller;
        $cruds = CrudManager::getCrudPanels();
        $parentCrud = reset($cruds);

        if ($parentCrud && $parentCrud->getCurrentEntry()) {
            $parentEntry = $parentCrud->getCurrentEntry();
            $parentController = $parentCrud->controller;
            $cacheKey = 'datatable_config_'.$this->tableId;

            Cache::forget($cacheKey);

            // Store the controller class, parent entry, element type and name
            Cache::put($cacheKey, [
                'controller' => $controllerClass,
                'parentController' => $parentController,
                'parent_entry' => $parentEntry,
                'element_name' => $this->name,
                'operations' => CrudManager::getInitializedOperations($parentController),
                'operations' => CrudManager::/** @scrutinizer ignore-call */ getInitializedOperations($parentController),
            ], now()->addHours(1));

            $this->crud->set('list.datatable_id', $this->tableId);
        }
    }

    public static function applyCachedSetupClosure($crud)
    {
        $tableId = request('datatable_id');

        if (! $tableId) {
            \Log::debug('Missing datatable_id in request parameters');

            return false;
        }

        $cacheKey = 'datatable_config_'.$tableId;
        $cachedData = Cache::get($cacheKey);

        if (! $cachedData) {
            return false;
        }

        try {
            // Get the parent crud instance
            self::initializeOperations($cachedData['parentController'], $cachedData['operations']);

            $entry = $cachedData['parent_entry'];
            $elementName = $cachedData['element_name'];
            $widgets = Widget::collection();
            foreach ($widgets as $widget) {
                if ($widget['type'] === 'datatable' &&
                    (isset($widget['name']) && $widget['name'] === $elementName) &&
                    (isset($widget['setup']) && $widget['setup'] instanceof \Closure)) {
                    $widget['setup']($crud, $entry);
                    return true;
                }
            }

            return false;
            
        } catch (\Exception $e) {
            \Log::error('Error applying cached datatable config: '.$e->getMessage(), [
                'exception' => $e,
            ]);
        }
        return false;
    }

    private static function initializeOperations(string $parentController, $operations)
    {
        $parentCrud = CrudManager::setupCrudPanel($parentController);

        foreach($operations as $operation) {
            $parentCrud->initialized = false;
            CrudManager::setupCrudPanel($parentController, $operation);
        }
    }

    public function render()
    {
        return view('crud::datatable.datatable', [
            'crud' => $this->crud,
            'modifiesUrl' => $this->modifiesUrl,
            'tableId' => $this->tableId,
        ]);
    }
}


Scrutinizer GitHub App not installed

Push — datatable-single-component ( cc93c3...b8f315 )

Datatable::applyCachedSetupClosure() B

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Laravel-Backpack / CRUD

Scrutinizer GitHub App not installed

Push — datatable-single-component ( cc93c3...b8f315 )

Datatable::applyCachedSetupClosure() B

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like