Cdf\BiCoreBundle\Utils\Tabella\TabellaOpzioniFromModelloColonneTrait

Complexity

Total Complexity

7

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	setOpzioniTabellaFromModellocolonne()	0	20	7

<?php

namespace Cdf\BiCoreBundle\Utils\Tabella;

trait TabellaOpzioniFromModelloColonneTrait
{
    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;
            }
        }
    }
}