Anomaly\Streams\Platform\Ui\Form\Component\Field\Guesser\NullableGuesser::guess()

Complexity

Conditions	9
Paths	11

Size

Total Lines	32
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

<?php namespace Anomaly\Streams\Platform\Ui\Form\Component\Field\Guesser;
    public function guess(FormBuilder $builder)
    {
        $fields = $builder->getFields();

        foreach ($fields as &$field) {

The expression $fields of type array|string is not guaranteed to be traversable. How about adding an additional type check?

            // Skip if nullable
            if (isset($field['rules']) && in_array('nullable', $field['rules'])) {
                continue;
            }

            // If the field depends on other fields, we not add nullable here
            // because validation will not be performed at all on this field
            if (! empty($field['rules'])) {
                if (preg_grep("/required_.*/", $field['rules'])) {
                    continue;
                }
            }

            // If not required then nullable.
            if (isset($field['required']) && $field['required'] == false) {
                $field['rules'][] = 'nullable';
            }

            // If not specified then it's nullable
            if (!isset($field['required'])) {
                $field['rules'][] = 'nullable';
            }
        }

        $builder->setFields($fields);
    }
}