mailRe: [Glpi-dev] GLPI 0.84dev : autoload avec des dossiers dans inc/


Others Months | Index by Date | Thread Index
>>   [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Header


Content

Posted by Gonéri Le Bouder on December 06, 2011 - 15:50:
Le 6 décembre 2011 09:47, David DURIEUX <d.durieux@xxxxxxxxxxxxx> a écrit :
Le Mon, 05 Dec 2011 20:43:55 +0100
jmd <jmd@xxxxxxxxxxxx> a écrit:

>Bonsoir,
>
>Le 05/12/2011 18:42, David DURIEUX a écrit :
>(...)
>> en commençant a travailler sur le plugin FusionInventory pour la 0.84
>> (on merge les différents plugin en 1 seul), on s'est rendu compte
>> qu'on risque d'avoir un très grand nombre de classes dans le dossier
>> inc.
>(...)
>> Donc on pourrait avoir la structure suivante :
>>
>> inc/computer.class.php (class Computer)
>> inc/computer/disk.class.php (class ComputerDisk)
>(....)
>> Qu'en pensez-vous?
>
>Je ne vois pas bien l'intérêt. A part se balader de dossier en dossier
>pour trouver la bonne classe, qu'est ce que ça apporte en terme de
>développement ?
>
>Pour mémoire, nous avions cette structure dans les premières versions
>de GLPI et c'était pénible au possible...
Bonjour,

Nous en avons discuté à plusieurs sur le canal de #FusionInventory, l'idée est
reproduire ce qui existe déjà dans d'autres frameworks PHP ainsi qu'en Java ou en Perl.
Remi a fait remarquer qu'une standardisation dans ce sens existe aussi dans le monde
PHP avec la PSR-0 et sa RFC ( https://wiki.php.net/rfc/splclassloader ).

Cette organisation, dans le cas du plugin FusionInventory nous permet de
gagner en lisibilité car nous allons avoir plus de 100 classes et qu'on souhaite
augmenter le niveau d'héritage entre elles.

Le patch de David ne permet pas de conserver le fonctionnement actuel de l'autoloader
en parallèle, ce qui ne me semble pas une bonne idée. Que pense vous, d'une
modification similaire mais qui cherche le fichier dans les sous dossiers que si celui-ci
n'existe pas dans inc/.
Une autre option serait de pouvoir surcharger l'autoloader directement dans le plugin
FusionInventory. Je ne sais pas si c'est réalisable.

Cordialement,
--
     Gonéri Le Bouder

Related Messages


Powered by MHonArc, Updated Tue Dec 06 21:20:03 2011