FuelPHP Forums

I created an Observer to automatically update calculated fields, and I would like to know if this is the best way :


class Observer_Calculated extends \Orm\Observer
{
    protected $fields = array();
    protected $fcts = array();
   
    public function __construct($class){
        foreach($class::properties() as $name=>$field){
            if (!isset($field['calculated']))
                continue;
           
            $opt = $field['calculated'];
            if (is_string($opt))
                $opt = array('fct'=>$opt);
            elseif (!is_array($opt))
                $opt = array();
            if (!(isset($opt['fct']) && is_string($opt['fct'])))
                $opt['fct'] = 'calc_'.$name;
            if (!method_exists($class, $opt['fct']))
                continue;
           
            $this->fields[$name] = $opt;
        }
    }
   
    public function before_save(Model $model){
        $this->calculate($model);
    }
   
    /**
     * Calculate the fields
     * @param Model $model
     */
    public function calculate(Model $model){
        $this->fcts = array();
        foreach($this->fields as $name=>$opt){
            if (!in_array($opt['fct'], $this->fcts))
                $this->fcts[] = $opt['fct'];
        }
       
        foreach($this->fcts as $fct){
            $model->$fct();
        }
    }
}


And for the usage :


class Model_Panier_Panier extends \Matnat\Model
{
    ...

    protected static $_properties = array(
        'id',
        'created_at',
        'updated_at',
        'session_id' => array(
            'data_type'=>'varchar',
            'label'=>'ID de session',
        ),
        'adresse_cp'=>array(
        ),
        'article_nb'=>array(
            'calculated'=>'calcul',
        ),
        'article_prix'=>array(
            'calculated'=>'calcul',
        ),
        'port'=>array(
            'calculated'=>'calcul',
        ),
    );
   
    protected static $_has_many = array(
        'contenu'=>array(
            'key_to'=>'panier_id',
            'model_to'=>'Model_Panier_Contenu',
            'cascade_delete'=>true,
        ),
    );
   
    protected static $_observers = array(
        ...
        'Matnat\Observer_Calculated' => array(
            'events' => array('before_save')
        ),
        ...
    );
   
    ...
   
    public function calcul()
    {
        $nb = 0;
        $montant = 0;
       
        if (count($this->contenu)){
            $nb = count($this->contenu);
            foreach($this->contenu as $contenu){
                $montant += $contenu->prix;
            }
        }
       
        $this->article_nb = $nb;
        $this->article_prix = $montant;
        $this->port = ...;
    }
}