Le 2 août 2013 02:18, pascal chevrel <[email protected]> a écrit :
> Salut, > Bonjour, > > J'ai peut être trouvé quelque chose de mon modeste niveau pour aider au > code de dotclear. J'ai ouvert un premier ticket et fait une Pull Request ( > https://bitbucket.org/**dotclear/dotclear/pull-**request/38/ticket-1461-** > replace-all-calls-to/diff<https://bitbucket.org/dotclear/dotclear/pull-request/38/ticket-1461-replace-all-calls-to/diff>) > en espérant avoir tout bien fait comme il faut :) > > Pour les PR sur Clearbricks, ça serait plus facile de les faire sur le dépôt Clearbricks de Bitbucket je pense. > Je regardais un peu les fonctions de clearbricks et j'ai trouvé cette > méthode qui réinvente la roue puisqu'il existe une même fonction native (et > pas récente, je vous parle de PHP 4 là ;) ) qui fait exactement la même > chose. > Je crois que c'était, à l'époque, pour pallier quelques restrictions d'hébergeurs qui ne voulaient pas qu'on fouille trop dans leur filesystems. > > J'ai ensuite regardé une autre méthode ( path:real() ), et ça fait > exactement la même chose que la fonction realpath() native de php depuis > 5.0. Au passage j'ai fait un benchmark et la méthode clearbricks est 20 > fois plus lente que la méthode native. > Gloups :-) > > Juste dans lib.files.php, je vois plusieurs autres méthodes qui me > semblent bien exister en PHP natif, même si on se cantonne à 5.2 > Il y a probablement pas mal de choses à revoir dans CB si, en effet, on limite la version de PHP à la 5.2 (tant pis pour Free et ses pages perso), voire tant qu'on y est à la 5.3 puisque la 5.2 n'est plus supportée (d'ailleurs la 5.3 ne va pas l'être encore longtemps). > Avant que je fasse d'autres pull requests pour remplacer ces méthodes qui > réinventent la roue, sont probablement moins fiables que les fonctions > natives et sont plus lentes, est-ce qu'il y a une bonne raison qui m'a > échappée pour qu'elles aient été créées (en dehors de la dette technique > bien sûr) ? > > Il faut quoi qu'il en soit qu'on décide d'une version minimale de PHP à adopter — faire une revue des gros hébergeurs et leurs versions de PHP ? — et revoir CB en conséquence. My two cents Franck
-- Dev mailing list - [email protected] - http://ml.dotclear.org/listinfo/dev
