Le 6 décembre 2011 09:47, David DURIEUX <d.duri...@siprossii.com> a écrit :
> Le Mon, 05 Dec 2011 20:43:55 +0100 > jmd <j...@indepnet.net> 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
_______________________________________________ Glpi-dev mailing list Glpi-dev@gna.org https://mail.gna.org/listinfo/glpi-dev
