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
