Re: [Dolibarr-dev] Pas de test avant l'instanciation d'une classe de module incluse

[Pierre Morin]

Pour info, pour tester ce cas, il suffit de mettre un nom de module n'existant pas dans le "$this->depends = array("")" d'un module à activer.

Bien cordialement,

-- 
Pierre Morin
address@hidden

Le mardi 20 avril 2010 à 16:03 +0200, Pierre Morin a écrit :

Bonjour,

Je viens de constater qu'il y a un gros problème dans "htdocs/admin/modules.php". En effet, à la ligne 72, une classe est instanciée avant même avoir testé que l'inclusion de la ligne précédante avais bien fonctionnée (ou, tout du moins, que la classe à instancier était bien déclarée quelque part).

Voici le code incriminé :
+------------------------------------------------------------------------------------------------------------------+
if ($modName)
{
    $file = $modName . ".class.php";
    include_once(DOL_DOCUMENT_ROOT."/includes/modules/$file");
    $objMod = new $modName($db);
+------------------------------------------------------------------------------------------------------------------+
Voici une mini-ébauche de ce qui pourrait régler ce problème :
+------------------------------------------------------------------------------------------------------------------+
if ($modName)
{
    $file = $modName . ".class.php";
    include_once(DOL_DOCUMENT_ROOT."/includes/modules/$file");
    if(class_exists($modName)) $objMod = new $modName($db);
    else
+------------------------------------------------------------------------------------------------------------------+

Par contre, je ne sais pas trop quoi mettre après le "else"...

Et j'avoue que j'ai un peu peur de toucher à quelque chose d'aussi central que l'activation des modules Dolibarr donc je préfère vous faire part de ma découverte.

Bien cordialement,

-- 
Pierre Morin
address@hidden

_______________________________________________
Dolibarr-dev mailing list
address@hidden
http://lists.nongnu.org/mailman/listinfo/dolibarr-dev