Suis d'ac aussi, sur la 2.8

Et j'envisage de coder quelque part une fonction loadPluginFile(filename),
histoire de simplifier un peu ça (l'écriture est passablement lourde avec
le array('pf' => '…')

Le 16 décembre 2014 10:05, Nicolas <[email protected]> a écrit :
>
> Je n'y vois pas d'objection. Enfin sur la branche 2.8, hein, pas sur la 2.7
>
> Le 16 décembre 2014 08:17, Bruno <[email protected]> a écrit :
> >
> > 2014-12-15 15:42 GMT+01:00 <[email protected]>:
> > >
> > >   Revision: 2847 (3fb59a8bc157} Branch: 2.7 Author: franck  Date:
> > 2014-12-15
> > > 15:42 +0100 Details: Details:
> > > http://dev.dotclear.org/2.0/changeset/3fb59a8bc157
> > > <http://dev.dotclear.org/2.0/changeset/3fb59a8bc157> Log Message
> > >
> > > Use adminurl->decode instead of ->get, URL encoding is not necessary
> > here (implicit when using http_build_query()), fixes #2017
> > >
> > >
> >
> > Hello,
> >
> > A bien y réfléchir, je pense qu'on devrait fusionner adminurl->get et
> > adminurl->decode, et ajouter un paramètre urldecode à adminurl->get. Je
> > préfère qu'on ait un unique point d'entrée sur adminurl. A discuter par
> la
> > suite, les cas où on doit garder les url encodées, et les cas où on doit
> le
> > décoder (le coup du $next_link/$prev_link est un cas spécifique, vu que
> > c'est le %s du sprintf qui est encodé à tort et qui pose souci).
> >
> > Des objections ? Si aucune, je peux commiter l'évolution dans la journée
> > (enfin ... si on ne me tombe pas sur le dos).
> >
> > --
> > Bruno
> > --
> > Dev mailing list - [email protected] -
> > http://ml.dotclear.org/listinfo/dev
> >
> --
> Dev mailing list - [email protected] -
> http://ml.dotclear.org/listinfo/dev
>


-- 
Franck
-- 
Dev mailing list - [email protected] - http://ml.dotclear.org/listinfo/dev

Répondre à