Le 2 août 2013 11:11, pascal chevrel <[email protected]> a écrit :

> Le 02/08/2013 08:00, Franck Paul a écrit :
>
>>
>> Pour les PR sur Clearbricks, ça serait plus facile de les faire sur le
>> dépôt Clearbricks de Bitbucket je pense.
>>
>
> Là ma PR ne touche pas à clearbricks, elle remplace les appels à la
> méthode clearbricks dans le code Dotclear par des appels à la fonction
> native PHP.
>
> Oups, j'ai lu de travers le PR :-p


>
>> 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.
>>
>
> Je vais tester sur OVH/Fre/Online où j'ai des comptes pour voir, mais même
> dans ce cas, je pense qu'il faudrait patcher Clearbricks cette fois ci pour
> qu'il utilise la fonction native si elle est activée et sa réimplémentation
> autrement.


Ok


> Note que ma première PR est pour remplacer l'utilisation d'une fonction
> clearBricks pour son équivalent en natif qui existait déjà en PHP 4.0 et
> que certaines fonctions sont l'équivalent de fonctions qui ont été ajoutées
> en 5.0 ou 5.1 qui est déjà le minimum actuel des pages perso de free, donc
> pour celles là je pense qu'on peut envisager d'utiliser des fonctions
> natives, ça fait moins de dépendances envers clearbricks, je pense qu'il ne
> faut utiliser les méthodes clearbricks que lorsque ça apporte des solutions
> qui ne sont pas déjà dans le langage.


Suis d'ac avec ça.
-- 
Dev mailing list - [email protected] - http://ml.dotclear.org/listinfo/dev

Répondre à