openMairie.org | Démonstration | Documentation | Forum

Prop0076 - Correctif de bogue mineur

Porteur principal : Laurent Groleau

Porteur secondaire : à déterminer

Résumé

Il y a un test pas assez strict dans om_formulaire::get_action_crud($maj = null)
Au lieu de ($maj === null) , on a ($maj == null) qui est vrai quand $maj est égal à 0

Motivation

Fiabiliser l’emploi de « crud » de préférence à celui de « maj ».

Pré-requis

Proposition et implémentation

Au lieu de

function get_action_crud($maj = null) {  
        // Si la clé de l'action n'est pas définie on la récupère
        if ($maj === null) {
            $maj = $this->getParameter("maj");
        }

il faudrait

    function get_action_crud($maj = null) {  
    // Si la clé de l'action n'est pas définie on la récupère
    if ($maj === null) {
        $maj = $this->getParameter("maj");
    }

car dans le cas d’un appel avec $maj = 0, la condition ( 0 == null ) est évaluée à true et $maj est re-positionné. Ce qui peut poser problème dans des instanciations particulières, telle le snippet d’openARIA snippet__filterselect() , ou la méthode renvoie null, et donc on ne peut plus s’appuyer sur le « crud » dans etablissement::set_select

Risques