ProtoneMedia\LaravelFormComponents\Components\HandlesDefaultAndOldValue::setValue()

Complexity

Conditions	8
Paths	7

Size

Total Lines	29
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    private function setValue(
        string $name,
        $bind = null,
        $default = null,
        $language = null
    ) {
        if ($this->isWired()) {

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

            return;
        }

        $inputName = static::convertBracketsToDots($name);

        if (!$language) {
            $boundValue = $this->getBoundValue($bind, $name);

            $default = is_null($boundValue) ? $default : $boundValue;

            return $this->value = old($inputName, $default);

The property value does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        }

        if ($bind !== false) {
            $bind = $bind ?: $this->getBoundTarget();
        }

        if ($bind) {
            $default = $bind->getTranslation($name, $language, false) ?: $default;
        }

        $this->value = old("{$inputName}.{$language}", $default);
    }