TabellaOpzioniFromModelloColonneTrait::setOpzioniTabellaFromModellocolonne()

Complexity

Conditions	7
Paths	7

Size

Total Lines	20
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    protected function setOpzioniTabellaFromModellocolonne(&$opzionibuilder)
    {
        foreach ($this->modellocolonne as $modellocolonna) {
            $campo = $this->bonificaNomeCampo($modellocolonna['nomecampo']);

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

            $this->getOpzionitabellaCampiExtra($campo, $modellocolonna, $opzionibuilder);

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

            foreach ($modellocolonna as $key => $value) {
                if (!array_key_exists($campo, $opzionibuilder)) {
                    if ((isset($modellocolonna['campoextra']) && true == $modellocolonna['campoextra'])) {

This if statement is empty and can be removed.

                        // tuttapposto
                    } else {
                        $ex = 'Fifree: '.$campo." field table option not found, did you mean one of these:\n".
                                implode("\n", array_keys($opzionibuilder)).
                                ' ?';
                        throw new \Exception($ex);
                    }
                }
                if ('ordine' == $key) {
                    $this->setMaxOrdine($value);

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

                }
                $opzionibuilder[$campo][$key] = $value;
            }