Model

Complexity

Total Complexity

192

Size/Duplication

Total Lines	1177
Duplicated Lines	4.76 %

Coupling/Cohesion

Components	1
Dependencies	9

Importance

Changes

0

68 Methods

Rating	Name	Duplication	Size	Complexity
A	getIsLogue()	0	4	1
A	setIsLogue()	0	5	1
A	__construct()	0	10	3
A	getAttributes()	0	4	1
A	initialisationVariablesAvantContructeur()	0	4	1
A	getControllerName()	0	8	2
A	objectToTable()	0	8	2
A	objectToTableSansId()	0	10	3
A	create()	0	8	2
A	__toString()	0	6	1
A	arrayFromCollection()	0	16	4
A	arrayFromObject()	0	8	2
B	objectsFromMatrix()	0	16	7
A	objectsFromArray()	0	4	1
B	objectsFromMatrixByAttribute()	0	11	5
B	findAll()	0	22	5
A	findOne()	0	15	2
A	findFirst()	0	15	2
A	count()	0	5	1
A	delete()	0	6	1
A	afficherParDefautNonSupprimes()	0	4	1
A	afficherParDefautDataVisibles()	0	4	1
C	findAllFromTable()	0	23	10
B	findAllAttributeFromTable()	21	21	6
B	findAllAttributeFromLeftJoinTable()	25	26	5
A	countFromTable()	0	12	2
A	sumFromTable()	0	15	4
C	deleteFromTable()	0	44	8
A	getDb()	0	7	2
A	setDb()	0	4	1
B	save()	0	26	6
	insert()	0	1	?
	update()	0	1	?
A	getMessageUpdateOK()	0	4	1
A	getMessageSuppressionOK()	0	4	1
A	getCreateur()	0	8	2
A	getModificateur()	0	8	2
A	setDateCreation()	0	11	2
A	setIdCreateur()	0	12	3
A	setDateModification()	0	5	2
A	setIdModificateur()	0	15	4
A	initialiseErrors()	0	4	1
A	getErrors()	0	13	3
A	compareVersion()	0	4	1
A	setErrors()	0	5	1
A	getAttribute()	0	4	1
A	setAttribute()	0	5	1
A	modify()	0	6	2
C	__call()	0	22	7
A	getCodeLangue()	0	4	1
A	listeChampsModifiable()	0	4	1
A	setIdNew()	0	4	1
A	isChampModifiable()	0	9	2
C	setAttributesSecure()	0	24	8
A	setValeursDefautPostSave()	0	3	1
A	listeChampsObjet()	0	4	1
A	fieldsToCompare()	0	5	1
A	fieldsInException()	0	6	1
C	logerChangement()	10	52	15
D	comparerDifferences()	0	82	19
A	getSqlLog()	0	15	2
A	returnArrayByJson()	0	4	1
A	getHistoriqueSurObjetDiffArray()	0	20	2
	canBeDeleted()	0	1	?
A	isUniqueAttribute()	0	15	3
A	setSession()	0	4	1
A	getSession()	0	7	2
A	hasSession()	0	4	3

<?php

/**
 * Classe mère de tous les modèles
 * @version 1.0
 * @todo deprecate this. Model is not MVC compliant
 */
abstract class Model

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    public static $is_logue = false;
    protected static $dbInstance = null;
    protected $errors = array();
    protected $nombre_ligne; // pour l'affichage du nombre de lignes des listes (utilisés dans les Xobjets)
    protected $log_id;
    private static $session;
    public $requete;

    public function getIsLogue()
    {
        return (bool)self::$is_logue;
    }

    public function setIsLogue($bool = false)
    {
        self::$is_logue = (bool)$bool;
        return $this;
    }

    /* **********
    * Créateur	*
    ********** */
    /**
     * Crée une instance de classe avec les paramètres transmis
     * Typiquement $_REQUEST d'un formulaire
     **/
    public function __construct($params = array())
    {
        $this->initialisationVariablesAvantContructeur();
        foreach (array_keys(get_object_vars($this)) as $attribute) {
            if (isset($params[$attribute])) {
                $this->setAttribute($attribute, $params[$attribute]);
            }
        }
        return $this;

Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct?

    }

    public function getAttributes()
    {
        return array_keys(get_object_vars($this));
    }

    /**
     * cette fonction est appelée au tout début du constructeur.
     * elle est principalement utilisée pour la gestion des cases à cocher qui doivent être décochée
     * si elles ne sont pas envoyée dans le $_POST du formulaire.
     */
    public function initialisationVariablesAvantContructeur()
    {
        /* fonction à redéfinir dans le modèle au besoin */
    }

    /**
     * Retourne le nom du controller associé à la classe appelante
     * @return string : Le nom du controleur
     */
    public static function getControllerName()
    {
        $classe_appelante = get_called_class();
        if (substr($classe_appelante, 0, 4) == 'Base') {
            $classe_appelante = substr($classe_appelante, 4);
        }
        return \FMUP\StringHandling::toCamelCase($classe_appelante);
    }

    /**
     * transforme un objet en tableau
     */
    public function objectToTable($objet)
    {
        $tableau = array();
        foreach ($objet as $attribute => $value) {
            $tableau[$attribute] = $value;
        }
        return $tableau;
    }

    /**
     * transforme un objet en tableau sans recupérer l'id
     */
    public function objectToTableSansId($objet)
    {
        $tableau = array();
        foreach ($objet as $attribute => $value) {
            if ($attribute != "id") {
                $tableau[$attribute] = $value;
            }
        }
        return $tableau;
    }

    /**
     * Crée une instance d'un modèle avec les paramètres transmis
     * Typiquement le résultat d'une requète
     * @param array $params : Données de l'objet
     * @param string $class_name : Type d'objet
     * @return Object
     */
    protected static function create($params, $class_name)
    {
        $class = new $class_name();
        foreach ($params as $attribut => $value) {
            $class->$attribut = $value;
        }
        return $class;
    }

    /* *************************
    * Affichage et convertion *
    ************************* */
    /**
     * Retourne une classe sous forme de chaîne
     **/
    public function __toString()
    {
        ob_start();
        var_dump($this);

var_dump($this); looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data.

        return ob_get_clean();
    }

    /**
     * Convertit une collection en tableau
     * @param {Array} La collection
     * @param {Integer} L'attribut à mettre dans les index du tableau
     * @param {Integer} L'attribut ou les attributs (tableau) à mettre dans les valeurs du tableau
     **/
    public static function arrayFromCollection($collection, $element_value, $element_text)
    {
        $array = array();
        foreach ($collection as $element) {
            if (is_array($element_text)) {
                $liste_attributs = array();
                foreach ($element_text as $attribut) {
                    $liste_attributs[] = $element->getAttribute($attribut);
                }
                $array[$element->getAttribute($element_value)] = implode(' - ', $liste_attributs);
            } else {
                $array[$element->getAttribute($element_value)] = $element->getAttribute($element_text);
            }
        }
        return $array;
    }

    /**
     * Convertit un objet en tableau
     * @param {Array} La collection
     **/
    public function arrayFromObject()
    {
        $array = array();
        foreach (array_keys(get_object_vars($this)) as $attribute) {
            $array[$attribute] = $this->getAttribute($attribute);
        }
        return $array;
    }

    /**
     * Crée des objets à partir d'une matrice (typiquement le résultat d'une requète)
     */
    protected static function objectsFromMatrix($matrix, $class_name, $modeIterator = true)
    {
        if ($modeIterator) {
            if (!$matrix instanceof \Iterator && is_array($matrix)) {
                $matrix = new \ArrayIterator($matrix);
            }
            return new \ArrayToObjectIterator($matrix, $class_name);
        }
        $liste = array();
        if ($matrix instanceof Traversable || is_array($matrix)) {
            foreach ($matrix as $array) {
                array_push($liste, $class_name::create($array, $class_name));
            }
        }
        return $liste;
    }

    public static function objectsFromArray($array, $class_name)
    {
        return $class_name::create($array, $class_name);
    }

    protected static function objectsFromMatrixByAttribute($matrix, $class_name, $attribute = 'id')
    {
        $liste = array();
        if (!empty($matrix) && (is_array($matrix) || $matrix instanceof Traversable)) {
            foreach ($matrix as $array) {
                $objet = $class_name::create($array, $class_name);
                $liste[$objet->getAttribute($attribute)] = $objet;
            }
        }
        return $liste;
    }

    /**
     * Retourne tous les éléments éventuellement filtrés de la table
     * @param array $where : [OPT] Un tableau de tous les filtres éventuels
     * @param array $options : [OPT] Options disponibles :
     *        - order : Tri sur la requête
     *        - findOne : Utilisé pour déterminer qu'on ne recherche qu'un résultat
     *          (TODO pour afficher les objets supprimés, ça ressemble à un doublon, fonctionnement ésotérique...)
     *        - afficher_supprimes : TODO vérifier si c'est vraiment utilisé, apparement non
     *    Seulement MySQL
     *        - limit : Pagination
     *    Seulement MSSQL
     *        - top : Semi-pagination qui ne passe pas par la sous-vue
     *        - paging : C'est un tableau composé de numero_page et nb_element
     *                     Utilisé pour simuler une pagination grace à une sous-vue
     * @return array[object]

The doc-type array[object] could not be parsed: Expected "]" at position 2, but found "object". (view supported doc-types)

     */
    public static function findAll($where = array(), $options = array())
    {
        $classe_appelante = get_called_class();

        //si on appelle depuis un object complexe, on recupere la requete correspondante
        if (call_user_func(array($classe_appelante, 'afficherParDefautNonSupprimes'))) {
            if (!isset($where['supprime']) &&
                (!isset($options['fonction']) || $options['fonction'] != 'findOne')
            ) {
                $where['supprime'] = 'supprime = 0';
            }
        }
        //sinon appelle de l'objet de Base généré par le génératOR
        $result = Model::findAllFromTable(
            call_user_func(array($classe_appelante, 'getTableName')),
            $where,
            $options
        );

        // Création d'un tableau d'objets
        return Model::objectsFromMatrix($result, $classe_appelante);
    }

    /**
     * Retourne un élément grâce à son ID
     * @param int $id : Un identifiant
     * @return null|object
     */
    public static function findOne($id)
    {
        $classe_appelante = get_called_class();

        $return = call_user_func(
            array($classe_appelante, 'findAll'),
            array('id = ' . Sql::secureId($id)),
            array('fonction' => 'findOne')
        );
        if (count($return) > 0) {
            return $return[0];
        } else {
            return null;
        }
    }

    /**
     * Retourne le premier élément d'un findAll
     * @param array $where : [OPT] Un tableau de tous les filtres éventuels
     * @param string $order : [OPT] Le champ sur lequel ordonner
     * @return false|object
     */
    public static function findFirst($where = array(), $order = '')
    {
        $classe_appelante = get_called_class();

        $return = call_user_func(
            array($classe_appelante, 'findAll'),
            $where,
            array('order' => $order, 'limit' => '0, 1', 'top' => '1')
        );
        if (count($return)) {
            return $return[0];
        } else {
            return false;
        }
    }

    /**
     * Retourne le nombre d'éléments d'une requête
     * @param array $where : Un tableau de clauses pour la requête
     * @return int : Le nombre d'éléments
     */
    public static function count($where = array())
    {
        $classe_appelante = get_called_class();
        return Model::countFromTable(call_user_func(array($classe_appelante, 'getTableName')), $where);
    }


    /**
     * Supprime l'objet dans la base de données
     * @return bool : Le résultat du traitement, VRAI si suppression
     * @todo : Supprimer les objets liés
     */
    public function delete()
    {
        $classe_appelante = get_called_class();
        $retour = $this->deleteFromTable(call_user_func(array($classe_appelante, 'getTableName')));
        return $retour;
    }

    /**
     * si la table de l'objet contient un champ date_suppression,
     * et qu'il ne faut afficher que les données non supprimées par défaut
     * alors réécrire cette fonction dans l'objet avec return true
     * @return bool
     */
    public static function afficherParDefautNonSupprimes()
    {
        return false;
    }

    /**
     * si la table de l'objet contient un champ visible, et qu'il ne faut afficher que les données visibles par défaut
     * alors réécrire cette fonction dans l'objet avec return true
     * @return bool
     */
    public static function afficherParDefautDataVisibles()
    {
        return false;
    }


    /* **************************
    * Requète et SQL générique *
    ************************** */
    /**
     * Trouve tous les enregistrements d'une table donnée
     * @param string $table : Le nom de la table
     * @param array $where :Un tableau de conditions
     * @param array $options : L'ordre, le limit, ...
     * @return array : Tableau contenant les objets
     */
    protected static function findAllFromTable($table, $where = array(), $options = array())
    {
        $SQL = "SELECT * FROM $table";
        $SQL .= Sql::parseWhere($where);
        $isIterator = (!isset($options["iterator"]) || $options["iterator"]);
        if (isset($options["group_by"]) && $options["group_by"]) {
            $SQL .= " group by " . $options["group_by"];
        }
        if (isset($options["order"]) && $options["order"]) {
            $SQL .= " ORDER BY " . $options["order"];
        }

        if (isset($options["top"]) && $options["top"]) {
            $SQL .= " LIMIT " . $options["top"];
        } elseif (!empty($options['limit'])) {
            $SQL .= " LIMIT " . $options["limit"];
        }

        // Exécution de la requète
        $db = \Model::getDb();
        $result = $isIterator ? $db->getIterator($SQL) : $db->fetchAll($SQL);

The method FMUP\Db::fetchAll() has been deprecated with message: use self::getIterator() instead

        return $result;
    }

    /**
     * Retourne un tableau avec l'ID de la DMD et un attribut
     *      --> utilsié pour alléger les menus déroulant de l'application
     * @param {Array} un tableau de tous les filtres éventuels
     * @param {string} le champ sur lequel ordonner
     **/
    public static function findAllAttributeFromTable($table, $attribute, $where = array(), $options = array())

This method seems to be duplicated in your project.

    {
        $SQL = "SELECT    id\n";
        if ($attribute != 'id') {
            $SQL .= "," . $attribute . "\n";
        }
        $SQL .= "FROM $table ";
        $SQL .= Sql::parseWhere($where);

        if (isset($options["order"]) && $options["order"]) {
            $SQL .= " ORDER BY " . $options["order"];
        }
        if (isset($options["limit"]) && $options["limit"]) {
            $SQL .= " LIMIT " . $options["limit"];
        }
        //echo($SQL);
        // Exécution de la requète
        $db = \Model::getDb();
        $result = $db->fetchAll($SQL);

The method FMUP\Db::fetchAll() has been deprecated with message: use self::getIterator() instead

        return $result;
    }

    public static function findAllAttributeFromLeftJoinTable(

This method seems to be duplicated in your project.

        $table,
        $left_table,
        $link_table,
        $attribute,
        $where = array(),
        $options = array()
    )
    {
        $SQL = "SELECT $attribute \n";
        $SQL .= " FROM $table  \n LEFT JOIN $left_table  \n";
        $SQL .= " ON $link_table  \n";
        $SQL .= Sql::parseWhere($where);

        if (isset($options["order"]) && $options["order"]) {
            $SQL .= " ORDER BY " . $options["order"] . "  \n";
        }
        if (isset($options["limit"]) && $options["limit"]) {
            $SQL .= " LIMIT " . $options["limit"];
        }
        //debug::output($SQL);

72% of this comment could be valid code. Did you maybe forget this after debugging?

        // Exécution de la requète
        $db = \Model::getDb();
        $result = $db->getIterator($SQL);
        return $result;
    }

    /**
     * Retourne le nombre d'éléments d'une requéte pour une table donnée
     * @param {string} Le nom de la table
     * @param {Array} un tableau de condititions
     **/
    protected static function countFromTable($table, $where = array(), $options = array())
    {
        $SQL = "SELECT COUNT(*) AS nb FROM $table";
        $SQL .= sql::ParseWhere($where);
        if (!empty($options["group_by"])) {
            $SQL .= " group by " . $options["group_by"];
        }
        // Exécution de la requète
        $db = \Model::getDb();
        $result = $db->fetchRow($SQL);
        return $result["nb"];
    }

    /**
     * Retourne la somme des valeurs d'une colonne pour une table et une colonne donnée
     * @param {string} Le nom de la table
     * @param {string} le nom de la colonne à sommer
     * @param {Array} un tableau de condition
     * @param {Array} un tableau d'options
     * @return mixed La somme si pas de group by en option, un tableau de couple somme / valeur de la colonne sinon
     **/
    protected static function sumFromTable($table, $colonne, $where = array(), $options = array())
    {
        $select = "SUM($colonne) as somme";
        $group_by = "";
        if (isset($options["group_by"]) && $options["group_by"]) {
            $select .= ", $table.$colonne";
            $group_by = " GROUP BY " . $options["group_by"];
        }
        $sql = "SELECT $select FROM $table";
        $sql .= sql::ParseWhere($where);
        $sql .= $group_by;
        // Exécution de la requête
        $db = \Model::getDb();
        return ($group_by != "") ? $db->fetchAll($sql) : $db->fetchRow($sql)["somme"];

The method FMUP\Db::fetchAll() has been deprecated with message: use self::getIterator() instead

    }

    /**
     * Supprime l'objet dans la base de données
     * Place le champ supprime à 1 à la place si le champ est présent dans la table
     * @param string $table : Le nom de la table
     * @return bool : VRAI si la suppression est effectuée
     */
    protected function deleteFromTable($table)
    {
        if (($this->id > 0) && ($this->canBeDeleted())) {
            // Cas ou le champ de suppression existe
            if (property_exists($this, 'supprime')) {
                $this->supprime = true;

The property supprime does not exist. Did you maybe forget to declare it?

                $infos_suppression = '';
                if (property_exists($this, 'date_suppression')) {
                    $infos_suppression .= ', date_suppression = CURRENT_TIMESTAMP()';
                    $this->date_suppression = date('Y-m-d H:i:s');

The property date_suppression does not exist. Did you maybe forget to declare it?

                }
                if (property_exists($this, 'id_suppresseur')) {
                    if (self::getSession()->has('id_utilisateur')) {
                        $id_utilisateur = self::getSession()->get('id_utilisateur');
                    }
                    $infos_suppression .= ', id_suppresseur = ' . Sql::secureId($id_utilisateur);

The variable $id_utilisateur does not seem to be defined for all execution paths leading up to this point.

                    $this->id_suppresseur = $id_utilisateur;

The property id_suppresseur does not exist. Did you maybe forget to declare it?

                }
                // Loger le changement
                $this->logerChangement("delete");
                $SQL = "UPDATE $table
                        SET supprime = 1
                        $infos_suppression
                        WHERE id = " . $this->id;

The property id does not exist. Did you maybe forget to declare it?

                $db = Model::getDb();
                return (bool)$db->execute($SQL);

The method execute() does not seem to exist on object<FMUP\Db>.

                // Cas de la suppression physique
            } else {
                // Loger le changement
                $this->logerChangement("delete");
                $SQL = "DELETE FROM $table WHERE id = " . $this->id;
                $db = Model::getDb();
                $return = (bool)$db->query($SQL);
                if ($return) {
                    $this->id = "";
                    return true;
                } else {
                    return false;
                }
            }
        } else {
            return false;
        }
    }

    /**
     * Retourne l'instance de base de données du controlleur actif
     * @return \FMUP\Db
     */
    public static function getDb()
    {
        if (!self::$dbInstance) {
            self::$dbInstance = \FMUP\Db\Manager::getInstance()->get();
        }
        return self::$dbInstance;
    }

    public static function setDb($dbInstance)
    {
        self::$dbInstance = $dbInstance;
    }

    /* ************************
    * Sauvegarde des données *
    ************************ */
    /**
     * Sauvegarde ou met à jour l'objet dans la base de donnée
     * @param bool $force_enregistrement
     *      si TRUE, alors le système contrepasse le VALIDATE et enregistre quand même l'objet
     *            (ATTENTION à l'utilisation de ce paramètre)
     * @return bool
     *      VRAI si une action a eu lieu en base (si la requête ne change rien ou le validate bloque, retourne FAUX)
     * @throws \FMUP\Exception
     */
    public function save($force_enregistrement = false)
    {
        // debug::output($this, true);

64% of this comment could be valid code. Did you maybe forget this after debugging?

        if ($force_enregistrement || $this->validate(true)) {

The call to Model::validate() has too many arguments starting with true.

            if (Is::id($this->id)) {
                /* Loger le changement */
                $this->logerChangement("update");
                if ($this->update() !== false) {
                    $this->comparerDifferences();
                } else {
                    throw new \FMUP\Exception("Erreur pendant l'enregistrement");
                }
            } else {
                /* Loger le changement */
                $this->id = $this->insert();
                $this->logerChangement("insert");
            }
            //appel de la fonction post SAVE, si l'enregistrement a fonctionné
            if ($this->id) {
                $this->setValeursDefautPostSave();
            }
            return $this->id;
        } else {
            return false;
        }
    }

    /**
     * Insertion d'un objet en base
     * @return bool
     */
    abstract protected function insert();

    /**
     * Mise à jour d'un objet en base
     * @return bool
     */
    abstract protected function update();

    /**
     * Le message affiché à l'update
     */
    public static function getMessageUpdateOK()
    {
        return "Mise à jour réalisée avec succès.";
    }

    /**
     * Le message affiché à la suppression
     */
    public static function getMessageSuppressionOK()
    {
        return "Suppression réalisée avec succès.";
    }

    /* ***************************************
    * gestion des créateur et modificateurs *
    *            d'objet en base            *
    *************************************** */

    /**
     * Retourne le createur de l'objet
     * @return Utilisateur
     */
    public function getCreateur()
    {
        $createur = Utilisateur::findOne($this->id_createur);

The property id_createur does not exist. Did you maybe forget to declare it?

        if (!$createur) {
            $createur = new Utilisateur();
        }
        return $createur;
    }

    /**
     * Retourne le dernier modificateur de l'objet
     * @return Utilisateur
     */
    public function getModificateur()
    {
        $modificateur = Utilisateur::findOne($this->id_modificateur);

The property id_modificateur does not exist. Did you maybe forget to declare it?

        if (!$modificateur) {
            $modificateur = new Utilisateur();
        }
        return $modificateur;
    }

    /**
     * Modifie le champ date_creation par la date actuelle
     **/
    public function setDateCreation($value = null)
    {
        $creationDate = null;
        try {
            $creationDate = new \DateTime($value);
        } catch (\Exception $e) {
            $creationDate = \DateTime::createFromFormat('d/m/Y H:i:s', $value);
        }
        $this->date_creation = $creationDate->format('Y-m-d H:i:s');

The property date_creation does not exist. Did you maybe forget to declare it?

        return true;
    }

    /**
     * Modifie le champ createur par la personne connectée
     **/
    public function setIdCreateur($value = 0)
    {
        $sessionIdUser = self::getSession()->get('id_utilisateur');
        if ($value) {
            $this->id_createur = $value;
        } elseif (!empty($sessionIdUser)) {
            $this->id_createur = $sessionIdUser;
        } else {
            $this->id_createur = -1;
        }
        return true;
    }

    /**
     * Modifie le champ date de dernière modification par la date actuelle
     **/
    public function setDateModification($value = null)
    {
        $this->date_modification = date('Y-m-d H:i:s', $value ? strtotime($value) : null);

The property date_modification does not exist. Did you maybe forget to declare it?

        return true;
    }

    /**
     * Modifie le champ dernier modificateur par la personne connectée
     **/
    public function setIdModificateur($value = 0)
    {
        if ($value === 'null') {

The strict comparison === seems to always evaluate to false as the types of $value (integer) and 'null' (string) can never be identical. Maybe you want to use a loose comparison == instead?

            $this->id_modificateur = '';
        } elseif ($value) {
            $this->id_modificateur = $value;
        } elseif (self::getSession()->has('id_utilisateur')) {
            $this->id_modificateur = self::getSession()->get('id_utilisateur');
        } else {
            // ce cas ne peut arriver que si l'on modifie l'objet dans une page où personne n'est connecté
            // (ex: page de première connexion)
            $this->id_modificateur = 1;
        }
        return true;
    }

    /* ******************************
    * Affichage d'infos et erreurs *
    ****************************** */

    /**
     * Réinitialise le tableau d'erreurs
     */
    public function initialiseErrors()
    {
        $this->errors = array();
    }

    /**
     * Retourne les erreurs de validation d'un objet
     * @param string $attribute : [OPT] Pour vérifier les erreurs sur un atribut en particulier
     * @return array
     */
    public function getErrors($attribute = false)
    {
        if ($attribute) {

The expression $attribute of type false|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== false instead.

            $tab_error = $this->errors;
            if (isset($tab_error[$attribute])) {
                return $tab_error[$attribute];
            } else {
                return false;

The return type of return false; (false) is incompatible with the return type documented by Model::getErrors of type array.

            }
        } else {
            return $this->errors;
        }
    }

    /*
     * Fonction utilisée par les demandes et commandes afin de trouver les différences
     * par rapport a une version modifiée
     */
    public function compareVersion()
    {
        return array();
    }


    /**
     * Met à jour le tableau des erreurs d'un objet.
     * @param {Array} $errors Le tableau d'erreurs à ajouter.

The doc-type {Array} could not be parsed: Unknown type name "{Array}" at position 0. (view supported doc-types)

     * @return true
     */
    public function setErrors($errors)
    {
        $this->errors = array_merge($errors, $this->errors);
        return true;
    }

    /* *********************
    * Accesseurs généraux *
    ********************* */
    /**
     * Retourne la valeur d'un attribut si il existe
     * @param {String} l'attribut auquel accéder
     **/
    public function getAttribute($attribute)
    {
        return call_user_func(array($this, 'get' . \FMUP\StringHandling::toCamelCase($attribute)));
    }

    /**
     * Modifie la valeur d'un attribut si il existe
     * @param {String} l'attribut auquel accéder
     * @param {string} la valeur à enregistrer
     **/
    public function setAttribute($attribute, $value)
    {
        call_user_func(array($this, 'set' . \FMUP\StringHandling::toCamelCase($attribute)), $value);
        return true;
    }

    /**
     * Met à jour un objet (*sans* l'enregistrer dans la base de données) avec de
     * nouveaux paramètres
     * @param {Array} un tableau de nouvelles valeurs
     **/
    public function modify($params)
    {
        foreach ($params as $attribute => $value) {
            $this->setAttribute($attribute, $value);
        }
    }

    /**
     * Retourne ou modifie la valeur d'un attribut
     * @param string $function : L'attribut auquel accéder
     * @param string $argument : [OPT] La valeur à affecter dans le cas d'une affectation
     * @return mixed|bool|null : L'argument demandée pour une lecture, VRAI si affectation réussie, null sinon
     */
    public function __call($function, $argument = array())
    {
        $attribut = \FMUP\StringHandling::toCamelCase(substr($function, 3));
        if (property_exists($this, $attribut)) {
            if (preg_match('#^get#i', $function)) {
                return $this->$attribut;
            }

            if (preg_match('#^set#i', $function) && count($argument)) {
                $this->$attribut = $argument[0];
                return true;
            }
        } else {
            if (preg_match('#^get#i', $function) || preg_match('#^set#i', $function)) {
                $message = "Attribut inexistant $attribut dans l'objet " . get_called_class();
            } else {
                $message = "Fonction inexistante $function dans l'objet " . get_called_class();
            }
            throw new \FMUP\Exception($message);
        }
        return null;
    }

    /**
     * Retourne l'id de l'objet
     **/
    /*public function getId()

56% of this comment could be valid code. Did you maybe forget this after debugging?

    {
        return $this->id;
    }*/

    /**
     * Retourne le code langue de l'utilisateur
     **/
    public function getCodeLangue()
    {
        return $this->code_langue;

The property code_langue does not exist. Did you maybe forget to declare it?

    }

    /* *********************
    *  Sécurisation des   *
    *      éditions       *
    ***********************/

    /**
     * Renvoi le tableau des champs autorisés
     * pour l'utilisateur en cours
     *
     * @return tableau des champs autorisés
     */
    public function listeChampsModifiable()
    {
        return array();
    }

    /**
     * Donne le droit à modifier l'objet mais par une méthode différente (non directe par l'attribut)
     **/
    public function setIdNew($valeur = '')
    {
        $this->id = $valeur;
    }

    /**
     * Renvoi l'autorisation d'un champ pour l'utilisateur en cours
     *
     * @param champ à vérifier
     * @return booléen d'autorisation
     */
    public function isChampModifiable($champ)
    {
        $liste = $this->listeChampsModifiable();
        if (isset($liste[$champ])) {
            return true;
        } else {
            return false;
        }
    }

    /**
     * @param Tableau des valeurs de l'objet typiquement $_POST
     * @return l'objet

The doc-type l'objet could not be parsed: Unknown type name "l'objet" at position 0. (view supported doc-types)

     */
    public function setAttributesSecure($attributes)
    {
        //Sécurisation de l'id
        $identifiant = (isset($attributes['id']) && Is::id($attributes['id'])) ? $attributes['id'] : 0;
        //Récupération de l'objet en base (inutile de charger s'il n'y a pas d'ID)
        if ($identifiant) {
            $object = call_user_func(array(get_class($this), 'FindOne'), $identifiant);
        }
        if (!$identifiant || !$object) {

The variable $object does not seem to be defined for all execution paths leading up to this point.

            $object = $this;
        }
        // récuperation des champs modifiable pour l'utilisateur courant
        $editable_fields = $object->listeChampsModifiable();
        //Si on fournit une donnée et si l'on peut la modifier
        // dans le cas de POST, il faut gérer les checkbox non cochées
        $object->initialisationVariablesAvantContructeur();
        foreach ($editable_fields as $field) {
            if (isset($attributes[$field])) {
                $object->setAttribute($field, $attributes[$field]);
            }
        }

        return $object;
    }

    public function setValeursDefautPostSave()
    {
    }

    /* *********************
    *       Logs          *
    ********************* */

    /**
     * fonction utilisée pour récupérer la liste des champs (dans l'ordre) de la table pour les requete de log
     * /!\  la clef primaire ID doit s'appeler 'id_objet_log' !!!
     * @return string
     */
    public static function listeChampsObjet()
    {
        return 'id_objet_log';
    }

    /**
     * Spécifie les champs de la table à comparer
     * Retourne un taleau vide si tous les champs de la table à comparer
     * @return Array $array
     */
    public function fieldsToCompare()
    {
        $array = array();
        return $array;
    }

    /**
     * Spécifie les champs à ne pas prendre en compte pour la comparaison
     * Tableau non vide contenant tout le temps, au moins le champ id
     * @return Array $array
     */
    public function fieldsInException()
    {
        $array = array();
        $array['id'] = 'Id';
        return $array;
    }

    /**
     * fonction de log
     * @param $type_action
     */
    public function logerChangement($type_action)
    {
        if ($this->getIsLogue() && $this->tableToLog() && $this->id) {

The method tableToLog does not exist on object<Model>? Since you implemented __call, maybe consider adding a @method annotation.

            $default_id = "";
            $tab = call_user_func(array(get_class($this), 'listeChampsObjet'));
            $tab = explode(', ', $tab);

            if (isset($tab[0]) && trim($tab[0]) == '') {
                unset($tab[0]);
            }

            if (isset($tab[0]) && (trim($tab[0]) == 'id')) {

This code seems to be duplicated across your project.

                $tab[0] = 'id_objet_log';
            } elseif (isset($tab[1]) && (trim($tab[1]) == 'id')) {
                $tab[1] = 'id_objet_log';
            }
            $liste_champ = implode(', ', $tab);

            if (isset($tab[0]) && (trim($tab[0]) == 'id_objet_log')) {

This code seems to be duplicated across your project.

                $tab[0] = 'id';
            } elseif (isset($tab[1]) && (trim($tab[1]) == 'id_objet_log')) {
                $tab[1] = 'id';
            }
            $liste_champ_valeur = implode(', ', $tab);

            $id_utilisateur = self::getSession()->has('id_utilisateur')
                ? self::getSession()->get('id_utilisateur')
                : -1;

            $SQL = 'INSERT INTO log__' . $this->getTableName() . '

The method getTableName does not exist on object<Model>? Since you implemented __call, maybe consider adding a @method annotation.

                            (' . $liste_champ . '
                                , libelle_historisation
                                , contenu_log
                                , id_utilisateur_log
                                , date_action_log
                                , action_log
                            )
                        SELECT ' . $default_id . '
                                ' . $liste_champ_valeur . ',
                                \'\',
                                \'\',
                                ' . Sql::secureId($id_utilisateur) . ',
                                ' . Sql::secureDate(date('Y-m-d H:i:s')) . ',
                                ' . Sql::secure($type_action) . '
                        FROM ' . $this->getTableName() . ' T

The method getTableName does not exist on object<Model>? Since you implemented __call, maybe consider adding a @method annotation.

                        WHERE id = ' . Sql::secureId($this->id) . '
                    ';
            $db = Model::getDb();
            $db->query($SQL);
            $this->log_id = $db->lastInsertId();
        }
    }

    /**
     * fonction permettant de comparer 2 tableaux de données d'un objet et sa sauvegarde en log
     * et d'enregistrer les modifications effectuées
     */
    public function comparerDifferences()
    {
        $tab_champs_comparaison = $this->fieldsToCompare();
        $tab_champs_exception = $this->fieldsInException();
        $tab_champs_base = array();
        $tab_champs_log = array();
        $tab_contenu = array();

        $champs_specifiques = false;
        if (!empty($tab_champs_comparaison)) {
            $champs_specifiques = true;
        }

        if (empty($tab_champs_exception)) {
            $tab_champs_exception["#exception#"] = "Exception";
        }


        if ($this->getIsLogue() && $this->tableToLog()) {

The method tableToLog does not exist on object<Model>? Since you implemented __call, maybe consider adding a @method annotation.

            // données de la table courante
            $sql = $this->getSqlLog();
            $db = Model::getDb();
            $res = $db->fetchRow($sql);

            if ($res) {

The expression $res of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

                foreach ($res as $index => $value) {
                    if ($champs_specifiques) {
                        if (array_key_exists($index, $tab_champs_comparaison)) {
                            $tab_champs_base[$index] = $value;
                        }
                    } else {
                        if (!array_key_exists($index, $tab_champs_exception)) {
                            $tab_champs_base[$index] = $value;
                        }
                    }
                }
            }
            // données de la table de log
            $sql = $this->getSqlLog('log');
            $res = $db->fetchRow($sql);

            if ($res) {

The expression $res of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

                foreach ($res as $index => $value) {
                    if (!array_key_exists($index, $tab_champs_exception)) {
                        if ($index == "id_objet_log") {
                            $index = "id";
                        }

                        if (array_key_exists($index, $tab_champs_base)) {
                            $tab_champs_log[$index] = $value;
                        }
                    }
                }
            }
            $tab_diff = array_diff_assoc($tab_champs_base, $tab_champs_log);
            // insertion de la différence dans la table de log
            if (count($tab_diff) > 0) {
                $libelle = "";

                foreach ($tab_diff as $index => $value) {
                    $field = ($champs_specifiques) ? $tab_champs_comparaison[$index] : $index;
                    $libelle .= "Le champ '" . $field . "' a été modifié : '"
                        . $field . "' a été remplacé par '" . $value . "'\n";

                    $tab_contenu[$index] = array(
                        "old_value" => isset($tab_champs_log[$index]) ? $tab_champs_log[$index] : null,
                        "new_value" => ($value)
                    );
                }

                $contenu = serialize($tab_contenu);
                //$contenu = json_encode($tab_contenu);

56% of this comment could be valid code. Did you maybe forget this after debugging?

                $sql = "UPDATE log__" . $this->getTableName() . "

The method getTableName does not exist on object<Model>? Since you implemented __call, maybe consider adding a @method annotation.

                        SET libelle_historisation = " . Sql::secure(($libelle)) . "
                        , contenu_log = " . Sql::secure($contenu) . "
                        WHERE id = " . Sql::secureId($this->log_id);
                $db = Model::getDb();
                $db->query($sql);
            }
        }
    }

    /**
     * formate les requetes de log
     */
    public function getSqlLog($from = "")
    {
        if ($from == 'log') {
            $table = 'log__' . $this->getTableName();

The method getTableName does not exist on object<Model>? Since you implemented __call, maybe consider adding a @method annotation.

            $condition = " T.id = " . Sql::secureId($this->log_id);
        } else {
            $table = $this->getTableName();

The method getTableName does not exist on object<Model>? Since you implemented __call, maybe consider adding a @method annotation.

            $condition = " T.id = " . Sql::secureId($this->id);
        }

        $sql = "SELECT T.*
                FROM " . $table . " T
                WHERE " . $condition;
        return $sql;
    }

    /**
     * fonction permettant de récupérer sous forme de tableau les différences
     */
    public static function returnArrayByJson($string = "")
    {
        return json_decode($string, true);
    }

    /**
     * fonction permettant de récupérer les historiques tableau d'un objet
     */
    public function getHistoriqueSurObjetDiffArray()
    {
        $array = array();
        $sql = "SELECT id, contenu_log, id_utilisateur_log, date_action_log, action_log
                FROM log__" . $this->getTableName() . "

The method getTableName does not exist on object<Model>? Since you implemented __call, maybe consider adding a @method annotation.

                WHERE id_objet_log = " . Sql::secureId($this->id) . "
                ORDER BY id";
        $db = \Model::getDb();
        $res = $db->getIterator($sql);

        foreach ($res as $rs) {
            $array[$rs["id"]] = array(
                "id_utilisateur_log" => $rs["id_utilisateur_log"]
            , "date_action_log" => $rs["date_action_log"]
            , "action_log" => $rs["action_log"]
            , "contenu_log" => unserialize($rs["contenu_log"])
            );
        }
        return $array;
    }

    /* ************
    * Validation *
    ************ */
    /**
     * L'objet est-il enregistrable en base de données
     * @return bool
     */
    abstract public function validate();

    /**
     * L'objet est-il effaçable
     * @return bool
     */
    abstract public function canBeDeleted();

    /**
     * Détermine si notre objet est unique par rapport aux attributs donnés
     * @param mixed $attribut : Attribut (ou liste d'attribut, dans ce cas tableau) à comparer
     * @param array $where : [OPT] Clauses supplémentaires à prendre en compte pour notre recherche
     * @return bool : VRAI si aucun doublon est trouvé
     */
    public function isUniqueAttribute($attribut, $where = array())
    {
        if (is_array($attribut)) {
            foreach ($attribut as $current_attribut) {
                $where[$current_attribut] = 'IFnull(' . $current_attribut . ', 0) = '
                    . 'IFnull(' . sql::Secure($this->$current_attribut) . ', 0)';
            }
        } else {
            $where[$attribut] = 'IFnull(' . $attribut . ', 0) = IFnull(' . sql::Secure($this->$attribut) . ', 0)';
        }
        $where['id'] = "IFnull(id, 0) <> IFnull(" . sql::secureId($this->id) . ", 0)";
        $doublon = $this->findFirst($where);

        return !$doublon;
    }

    /**
     * @param \FMUP\Session $session
     */
    public static function setSession(\FMUP\Session $session)
    {
        self::$session = $session;
    }

    /**
     * @return \FMUP\Session
     */
    public static function getSession()
    {
        if (!self::$session) {
            self::setSession(\FMUP\Session::getInstance());
        }
        return self::$session;
    }

    public static function hasSession()
    {
        return \FMUP\Sapi::getInstance() && \FMUP\Sapi::getInstance()->is(\FMUP\Sapi::CGI) && (bool)self::$session;
    }
}