FusionForge de l'ADULLACT: openMairie - Framework: [#8672] Notices PHP liés à la méthode get_database_extra

Date:
2017-01-12 15:29

Priority:
2

State:
Open

Submitted by:
Jean-Yves Madier de Champvermeil (jymadier)

Assigned to:
Jean-Yves Madier de Champvermeil (jymadier)

Sévérité:
none

Version cible:
none

Version concernée:
4.5.0

Summary:
Notices PHP liés à la méthode get_database_extra_parameters

Detailed description
Le ticket https://adullact.net/tracker/index.php?func=detail&aid=8544&group_id=265&atid=1999 a permis d'ajouter une nouvelle entrée dans le tableau de configuration database.inc.php. Dans le cas où la méthode get_database_extra_parameters() est appelée alors que le tableau associatif "extras" n'existe pas dans le tableau $conn du fichier database.inc.php, un PHP Notice apparaît à cause d'un "undefined offset". Il faut ajouter un isset() dans la méthode get_database_extra_parameters(), dans le fichier om_application.class.php:6280.
Le ticket https://adullact.net/tracker/index.php?func=detail&aid=8544&group_id=265&atid=1999 a permis d'ajouter une nouvelle entrée dans le tableau de configuration database.inc.php. Dans le cas où la méthode get_database_extra_parameters() est appelée alors que le tableau associatif "extras" n'existe pas dans le tableau $conn du fichier database.inc.php, un PHP Notice apparaît à cause d'un "undefined offset". Il faut ajouter un isset() dans la méthode get_database_extra_parameters(), dans le fichier om_application.class.php:6280.

Message
Date: 2017-01-25 13:29 Sender: Nicolas Meucci Si aucun paramètre n'est fourni à get_database_extra_parameters() cette méthode doit renvoyer le tableau extra, sinon la valeur du paramètre s'il existe, sinon NULL. Au préalable vérifier que extra existe et qu'il s'agit d'un tableau non vide.
Date: 2017-01-25 13:06 Sender: Nicolas Meucci Il faut également définir extras en tableau vide s'il n'existe pas, dans setParamsFromFiles().
Date: 2017-01-25 12:22 Sender: Nicolas Meucci Il faut également vérifier qu'il s'agisse d'un tableau.

No attached documents

No changes have been made to this item