dolibarr-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


From: Pierre Morin
Subject: Re: [Dolibarr-dev] Pas de test avant l'instanciation d'une classe de module incluse
Date: Tue, 20 Apr 2010 17:37:12 +0200

OK, désolé, j'avais pas compris que vous testiez à priori si le fichier à inclure existais aussi ou non.
Ca permet de régler le problème normalement (même si c'est pas blindé), mais pas le cas où le nom du fichier est différent de celui de la classe.
Je suis plutôt partisan du "on affiche le moins d'erreur en mode brutal et on affiche des beaux messages d'erreur tous clairs". ^^'

-- 
Pierre Morin
address@hidden


Le mardi 20 avril 2010 à 17:20 +0200, Régis Houssin a écrit :
Oui j’ai bien compris et c’est corrigé
La fonction Activate() est appelée une première fois par le formulaire, elle peut être appelée une deuxième fois ou plus si le module a des dépendances, c’est justement à ce niveau là que j’ai ajouté la vérification. La première fois la classe existe forcément car on utilise le formulaire pour l’activer.


Le 20/04/10 17:12, « Pierre Morin » <address@hidden> a écrit :

Ca ne réglera pas le problème que je soulève car celui-ci est à la ligne 89, directement à l'instanciation de la classe.
Au cas où, l'instanciation de la classe, c'est quand on fait une instance de classe, donc quand on fait "new $modName($db)" dans ce cas.


--
Régis Houssin
------------------------------------------------------
Cap-Networks
30, Quai de Verdun
71700 Tournus
Tél. +33 6 33 02 07 97
Web: http://www.cap-networks.com
Email: address@hidden

Développeur Dolibarr : address@hidden
Portail francophone : www.dolibarr.fr
Offres SaaS de Dolibarr : www.dolibox.fr
Development platform : www.dolibarr.pro
------------------------------------------------------
_______________________________________________
Dolibarr-dev mailing list
address@hidden
http://lists.nongnu.org/mailman/listinfo/dolibarr-dev

reply via email to

[Prev in Thread] Current Thread [Next in Thread]