crocodicstudio\crudbooster\controllers\traits\ColumnIntervention::columnIntervention()

Complexity

Conditions	4
Paths	2

Size

Total Lines	11
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function columnIntervention()
    {
        if(request("sub_module") && Cache::has("subModule".request("sub_module")) && cb()->getCurrentMethod() != "getDetail") {

Are you sure request('sub_module') of type Illuminate\Http\Request|array|string can be used in concatenation?

            /*
             * If there is sub module, the column that has same name with foreign key should be remove
             * And change to hidden. So we can save the foreign Key id from the parent module.
             */
            $subModule = Cache::get("subModule".request("sub_module"));
            $this->removeColumn($subModule["foreignKey"])

It seems like removeColumn() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                ->addHidden($subModule["foreignKey"],$subModule['foreignKey'])
                ->defaultValue($subModule['foreignValue']);
        }
}