Backpack\CRUD\app\View\Components\Dataform

Complexity

Total Complexity

6

Size/Duplication

Total Lines	87
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	31	4
A	applySetupClosure()	0	22	1
A	render()	0	11	1

<?php

namespace Backpack\CRUD\app\View\Components;

use Backpack\CRUD\app\Library\CrudPanel\CrudPanel;
use Backpack\CRUD\CrudManager;
use Closure;
use Illuminate\View\Component;

class Dataform extends Component
{
    public $crud;

    /**
     * Create a new component instance.
     *
     * @param  string  $controller  The CRUD controller class name
     * @param  string  $operation  The operation to use (create, update, etc.)
     * @param  string|null  $action  Custom form action URL
     * @param  string  $method  Form method (post, put, etc.)
     */
    public function __construct(
        public string $controller,
        public string $id = 'backpack-form',
        public string $operation = 'create',
        public ?string $action = null,
        public string $method = 'post',
        public bool $hasUploadFields = false,
        public $entry = null,
        public ?Closure $setup = null,

    ) {        
        // Get CRUD panel instance from the controller
        CrudManager::setActiveController($controller);

The method setActiveController() does not exist on Backpack\CRUD\CrudManager. Since you implemented __callStatic, consider adding a @method annotation.

        $this->crud = CrudManager::setupCrudPanel($controller, $operation);

The method setupCrudPanel() does not exist on Backpack\CRUD\CrudManager. Since you implemented __callStatic, consider adding a @method annotation.

        //dd($this->crud);
        if ($this->entry && $this->operation === 'update') {
            $this->action = $action ?? url($this->crud->route.'/'.$this->entry->getKey());
            $this->method = 'put';
            $this->crud->entry = $this->entry;
            $this->crud->setOperationSetting('fields', $this->crud->getUpdateFields());
        } else {
            $this->action = $action ?? url($this->crud->route);

It seems like $action ?? url($this->crud->route) can also be of type Illuminate\Contracts\Routing\UrlGenerator. However, the property $action is declared as type null|string. Maybe add an additional type check?

        }
        $this->hasUploadFields = $this->crud->hasUploadFields($operation, $this->entry?->getKey());
        $this->id = $id.md5($this->action.$this->operation.$this->method.$this->controller);
         if ($this->setup) {
            $this->applySetupClosure();
        }

        CrudManager::unsetActiveController();

The method unsetActiveController() does not exist on Backpack\CRUD\CrudManager. Since you implemented __callStatic, consider adding a @method annotation.

        
    }

    public function applySetupClosure(): bool
    {
        $originalSetup = $this->setup;
        $controllerClass = $this->controller;
        $crud = $this->crud;
        $entry = $this->entry;

        $modifiedSetup = function ($crud, $entry) use ($originalSetup, $controllerClass) {
            CrudManager::setActiveController($controllerClass);

            // Run the original closure
            return ($originalSetup)($crud, $entry);
        };

        try {
            // Execute the modified closure
            ($modifiedSetup)($crud, $entry);

            return true;
        } finally {
            // Clean up
            CrudManager::unsetActiveController();
        }
    }

    /**
     * Get the view / contents that represent the component.
     *
     * @return \Illuminate\Contracts\View\View|\Closure|string
     */
    public function render()
    {
        return view('crud::components.dataform.form', [
            'crud' => $this->crud,
            'saveAction' => $this->crud->getSaveAction(),
            'id' => $this->id,
            'operation' => $this->operation,
            'action' => $this->action,
            'method' => $this->method,
            'hasUploadFields' => $this->hasUploadFields,
            'entry' => $this->entry,
        ]);
    }
}