Re: [Dotclear Dev] [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
8:17 a.m.
2014-12-15 15:42 GMT+01:00 <hg(a)dotclear.org>:
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
10:05 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
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 <dsls(a)morefnu.org> a écrit :
2014-12-15 15:42 GMT+01:00 <hg(a)dotclear.org>:
>
> 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 - Dev(a)list.dotclear.org -
http://ml.dotclear.org/listinfo/dev
10:08 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Le 16 décembre 2014 10:05, Nicolas <nikrou77(a)gmail.com> a écrit :
Je n'y vois pas d'objection. Enfin sur la branche 2.8, hein, pas sur la 2.7
L'impact est quand même minime, il faut vraiment attendre la 2.8 ?
10:09 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
C'est si pressé que ça ?
Le 16 décembre 2014 10:08, Bruno <dsls(a)morefnu.org> a écrit :
Le 16 décembre 2014 10:05, Nicolas <nikrou77(a)gmail.com> a écrit :
>
> Je n'y vois pas d'objection. Enfin sur la branche 2.8, hein, pas sur la
2.7
>
>
L'impact est quand même minime, il faut vraiment attendre la 2.8 ?
--
Dev mailing list - Dev(a)list.dotclear.org -
http://ml.dotclear.org/listinfo/dev
--
Franck
10:15 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Le 16 décembre 2014 10:09, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
C'est si pressé que ça ?
Non, c'est juste pour simplifier la gestion des branches, et d'éviter que
des gens aient à réadapter leur plugin s'ils ont déjà fait l'effort pour la
2.7...
--
Bruno
10:17 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
En même temps decode ne fait que faire un urldecode() sut le retour du get.
Le 16 décembre 2014 10:15, Bruno <dsls(a)morefnu.org> a écrit :
Le 16 décembre 2014 10:09, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
>
> C'est si pressé que ça ?
Non, c'est juste pour simplifier la gestion des branches, et d'éviter que
des gens aient à réadapter leur plugin s'ils ont déjà fait l'effort pour la
2.7...
--
Bruno
--
Dev mailing list - Dev(a)list.dotclear.org -
http://ml.dotclear.org/listinfo/dev
--
Franck
10:26 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Le 16 décembre 2014 10:17, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
En même temps decode ne fait que faire un urldecode() sut le retour du get.
Certes, mais je trouve ça dommage d'avoir 2 méthodes distinctes pour
remplir un rôle très similaire.
--
Bruno
10:36 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Peu importe, fais pour le mieux (je peux m'en occuper si tu n'as pas le
temps) et j'adapterai sysInfo en conséquence.
Le 16 décembre 2014 10:26, Bruno <dsls(a)morefnu.org> a écrit :
Le 16 décembre 2014 10:17, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
>
> En même temps decode ne fait que faire un urldecode() sut le retour du
get.
>
Certes, mais je trouve ça dommage d'avoir 2 méthodes distinctes pour
remplir un rôle très similaire.
--
Bruno
--
Dev mailing list - Dev(a)list.dotclear.org -
http://ml.dotclear.org/listinfo/dev
--
Franck
10:41 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Le 16 décembre 2014 10:36, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
Peu importe, fais pour le mieux (je peux m'en occuper si tu n'as pas le
temps) et j'adapterai sysInfo en conséquence.
On peut toujours laisser decode et la passer en deprecated, de manière
transitoire.
10:44 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Bof, spa certain que d'autres que moi aient commencé à utiliser adminurl,
d'ailleurs le job est même pas encore fini côté Dotclear, je pense qu'on
peut la faire sauter si tu estimes qu'il vaut mieux.
Le 16 décembre 2014 10:41, Bruno <dsls(a)morefnu.org> a écrit :
Le 16 décembre 2014 10:36, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
>
> Peu importe, fais pour le mieux (je peux m'en occuper si tu n'as pas le
> temps) et j'adapterai sysInfo en conséquence.
>
On peut toujours laisser decode et la passer en deprecated, de manière
transitoire.
--
Dev mailing list - Dev(a)list.dotclear.org -
http://ml.dotclear.org/listinfo/dev
--
Franck
11:16 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Le 16 décembre 2014 10:44, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
Bof, spa certain que d'autres que moi aient commencé à utiliser adminurl,
d'ailleurs le job est même pas encore fini côté Dotclear, je pense qu'on
peut la faire sauter si tu estimes qu'il vaut mieux.
Je regarde ça (et je m'occupe du ?pf par la même occaz).
--
Bruno
11:47 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Le 16 décembre 2014 11:16, Bruno <dsls(a)morefnu.org> a écrit :
Le 16 décembre 2014 10:44, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
>
> Bof, spa certain que d'autres que moi aient commencé à utiliser adminurl,
> d'ailleurs le job est même pas encore fini côté Dotclear, je pense qu'on
> peut la faire sauter si tu estimes qu'il vaut mieux.
>
Je regarde ça (et je m'occupe du ?pf par la même occaz).
et voilou :)
A noter, pour inclure un fichier de plugin, c'est désormais
dcPage::getPF($fichier) .
--
Bruno
12:03 p.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Bonjour Bruno,
Le mardi 16 décembre 2014 à 11:47:56, vous écriviez :
A noter, pour inclure un fichier de plugin, c'est désormais
dcPage::getPF($fichier) .
Et ça sert comment, pourquoi, etc. ? Des exemples concrets pour les
néophytes dans mon genre ?
Une page de récap peut-être pour aider les qui n'y travent que dalle
ou ne suivent pas assidûment vos modifs ?
Merci
--
brol
12:29 p.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
brol, attends un peu, on a pas fini de mettre en place la nouvelle gestion
des URLs d'admin, on fera une récap' le moment venu.
Le 16 décembre 2014 12:03, brol <brol31(a)gmail.com> a écrit :
Bonjour Bruno,
Le mardi 16 décembre 2014 à 11:47:56, vous écriviez :
> A noter, pour inclure un fichier de plugin, c'est désormais
> dcPage::getPF($fichier) .
Et ça sert comment, pourquoi, etc. ? Des exemples concrets pour les
néophytes dans mon genre ?
Une page de récap peut-être pour aider les qui n'y travent que dalle
ou ne suivent pas assidûment vos modifs ?
Merci
--
brol
--
Dev mailing list - Dev(a)list.dotclear.org -
http://ml.dotclear.org/listinfo/dev
--
Franck
12:31 p.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Ok, mais pourquoi sur la branche 2.8-dev (default) ?
Le 16 décembre 2014 11:47, Bruno <dsls(a)morefnu.org> a écrit :
Le 16 décembre 2014 11:16, Bruno <dsls(a)morefnu.org> a écrit :
>
>
>
> Le 16 décembre 2014 10:44, Franck Paul <carnet.franck.paul(a)gmail.com> a
> écrit :
>>
>> Bof, spa certain que d'autres que moi aient commencé à utiliser
adminurl,
>> d'ailleurs le job est même pas encore fini côté Dotclear, je pense
qu'on
>> peut la faire sauter si tu estimes qu'il vaut mieux.
>>
>
> Je regarde ça (et je m'occupe du ?pf par la même occaz).
>
et voilou :)
A noter, pour inclure un fichier de plugin, c'est désormais
dcPage::getPF($fichier) .
--
Bruno
--
Dev mailing list - Dev(a)list.dotclear.org -
http://ml.dotclear.org/listinfo/dev
--
Franck
12:38 p.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Le 16 décembre 2014 12:31, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
Ok, mais pourquoi sur la branche 2.8-dev (default) ?
Je ne voulais pas soulever de débat, vu qu'initialement il était évoqué que
l'évolution serait à faire en 2.8, je me suis exécuté :)
On peut toujours faire le merge sur la 2.7, ce n'est pas trop tard :)
--
Bruno
1:40 p.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Yep, fusionnons tant qu'il n'y a pas trop de divergences avec les plugins
tiers déjà adaptés (a priori juste le mien).
Tu t'en occupes ?
Le 16 décembre 2014 12:38, Bruno <dsls(a)morefnu.org> a écrit :
Le 16 décembre 2014 12:31, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
>
> Ok, mais pourquoi sur la branche 2.8-dev (default) ?
Je ne voulais pas soulever de débat, vu qu'initialement il était évoqué que
l'évolution serait à faire en 2.8, je me suis exécuté :)
On peut toujours faire le merge sur la 2.7, ce n'est pas trop tard :)
--
Bruno
--
Dev mailing list - Dev(a)list.dotclear.org -
http://ml.dotclear.org/listinfo/dev
--
Franck
1:57 p.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Le 16 décembre 2014 13:40, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
Yep, fusionnons tant qu'il n'y a pas trop de divergences avec les plugins
tiers déjà adaptés (a priori juste le mien).
Tu t'en occupes ?
ok chef :)
3:20 p.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
C'est le souk cet encodage forcé par le http_build_query() :
http://dev.dotclear.org/2.0/ticket/2022 (voir mon commentaire également)
Le 16 décembre 2014 13:57, Bruno <dsls(a)morefnu.org> a écrit :
Le 16 décembre 2014 13:40, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
>
> Yep, fusionnons tant qu'il n'y a pas trop de divergences avec les plugins
> tiers déjà adaptés (a priori juste le mien).
>
> Tu t'en occupes ?
ok chef :)
--
Dev mailing list - Dev(a)list.dotclear.org -
http://ml.dotclear.org/listinfo/dev
--
Franck
3:53 p.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Le 16 décembre 2014 15:20, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
C'est le souk cet encodage forcé par le http_build_query() :
http://dev.dotclear.org/2.0/ticket/2022 (voir mon commentaire également)
Je reste quasiment convaincu que dans 99% des cas il faut garder les liens
urlencodés, et que le urldecode peut entraîner plus de soucis qu'il n'est
censé en résoudre.
Pour moi la seule variable d'ajustement à régler c'est le séparateur des
paramètres, qui est toujours à fixer à &, sauf quand on fait un
redirect (le redirect étant du http, le reste étant dans les pages html).
Je vais refaire une passe ce soir pour confirmer mes propos.
--
Bruno
3:59 p.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Ok, je touche à rien d'ici là et je te laisse regarder tout ça.
Si tu fais des commits, fais-le sur la branche 2.7, on fera un merge sur la
default plus tard…
Le 16 décembre 2014 15:53, Bruno <dsls(a)morefnu.org> a écrit :
Le 16 décembre 2014 15:20, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
>
> C'est le souk cet encodage forcé par le http_build_query() :
> http://dev.dotclear.org/2.0/ticket/2022 (voir mon commentaire également)
>
>
Je reste quasiment convaincu que dans 99% des cas il faut garder les liens
urlencodés, et que le urldecode peut entraîner plus de soucis qu'il n'est
censé en résoudre.
Pour moi la seule variable d'ajustement à régler c'est le séparateur des
paramètres, qui est toujours à fixer à &, sauf quand on fait un
redirect (le redirect étant du http, le reste étant dans les pages html).
Je vais refaire une passe ce soir pour confirmer mes propos.
--
Bruno
--
Dev mailing list - Dev(a)list.dotclear.org -
http://ml.dotclear.org/listinfo/dev
--
Franck
7:36 p.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Le 16 décembre 2014 15:59, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
Ok, je touche à rien d'ici là et je te laisse regarder tout ça.
Si tu fais des commits, fais-le sur la branche 2.7, on fera un merge sur la
default plus tard…
Ca roule. Je suis en train de retravailler media.php, ça vous dérange si je
remplace le behavior adminMediaURL (qui ne supporte pas les plugins
multiples) en adminMediaURLParams (et du coup les plugins modifient des
paramètres d'URL, et pas une URL) ?
--
Bruno
10:14 p.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Le 16 décembre 2014 19:36, Bruno <dsls(a)morefnu.org> a écrit :
Le 16 décembre 2014 15:59, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
>
> Ok, je touche à rien d'ici là et je te laisse regarder tout ça.
>
> Si tu fais des commits, fais-le sur la branche 2.7, on fera un merge sur
> la
> default plus tard…
>
>
Ca roule. Je suis en train de retravailler media.php, ça vous dérange si
je remplace le behavior adminMediaURL (qui ne supporte pas les plugins
multiples) en adminMediaURLParams (et du coup les plugins modifient des
paramètres d'URL, et pas une URL) ?
Bon, j'ai commité ma bafouille. Pour éviter d'avoir à se poser la question
je mets un "&" ou un "&" dans adminurl, j'ai ajouté
une méthode
redirect à adminurl, qui simplifie du coup les redirections, et qui fait ce
choix toute seule.
Et dcPage::getPF est bien plus court à taper :) .J'ai adapté en vitesse le
_admin de sysInfo, que je viens de découvrir, et ce plugin me plaît/sert
bien, je vais le garder sous le coude \o/
J'ai essayé de faire le plus de tests possibles, apparemment rien n'est
cassé, le #2022 devrait être réparé. La seule entorse dont je ne suis pas
fier, c'est pour les url paramétriques où je dois décoder les %25 en % à la
main. Je vous laisse m'insulter si quelque chose ne marche plus.
--
Bruno
10:55 p.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Super,
Je regarde ça en détail demain, et oui sysInfo me sert de temps en temps,
c'est assez pratique, entre autre pour la gestion du cache des templates
Le 16 décembre 2014 22:14, Bruno <dsls(a)morefnu.org> a écrit :
Le 16 décembre 2014 19:36, Bruno <dsls(a)morefnu.org> a écrit :
>
>
>
> Le 16 décembre 2014 15:59, Franck Paul <carnet.franck.paul(a)gmail.com> a
> écrit :
>>
>> Ok, je touche à rien d'ici là et je te laisse regarder tout ça.
>>
>> Si tu fais des commits, fais-le sur la branche 2.7, on fera un merge sur
>> la
>> default plus tard…
>>
>>
> Ca roule. Je suis en train de retravailler media.php, ça vous dérange si
> je remplace le behavior adminMediaURL (qui ne supporte pas les plugins
> multiples) en adminMediaURLParams (et du coup les plugins modifient des
> paramètres d'URL, et pas une URL) ?
>
>
Bon, j'ai commité ma bafouille. Pour éviter d'avoir à se poser la question
je mets un "&" ou un "&" dans adminurl, j'ai ajouté
une méthode
redirect à adminurl, qui simplifie du coup les redirections, et qui fait ce
choix toute seule.
Et dcPage::getPF est bien plus court à taper :) .J'ai adapté en vitesse le
_admin de sysInfo, que je viens de découvrir, et ce plugin me plaît/sert
bien, je vais le garder sous le coude \o/
J'ai essayé de faire le plus de tests possibles, apparemment rien n'est
cassé, le #2022 devrait être réparé. La seule entorse dont je ne suis pas
fier, c'est pour les url paramétriques où je dois décoder les %25 en % à la
main. Je vous laisse m'insulter si quelque chose ne marche plus.
--
Bruno
--
Dev mailing list - Dev(a)list.dotclear.org -
http://ml.dotclear.org/listinfo/dev
--
Franck
7:54 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Bonjour,
cela n'engage que moi mais je trouve que ça commence à faire beaucoup pour
une version de correction. Après je dois avouer que j'ai lu un peu en
diagonal et je ne me mesure pas forcément tous les impacts.
Nicolas
Le 16 décembre 2014 22:55, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
Super,
Je regarde ça en détail demain, et oui sysInfo me sert de temps en temps,
c'est assez pratique, entre autre pour la gestion du cache des templates
Le 16 décembre 2014 22:14, Bruno <dsls(a)morefnu.org> a écrit :
>
> Le 16 décembre 2014 19:36, Bruno <dsls(a)morefnu.org> a écrit :
> >
> >
> >
> > Le 16 décembre 2014 15:59, Franck Paul <carnet.franck.paul(a)gmail.com>
a
> > écrit :
> >>
> >> Ok, je touche à rien d'ici là et je te laisse regarder tout ça.
> >>
> >> Si tu fais des commits, fais-le sur la branche 2.7, on fera un merge
sur
> >> la
> >> default plus tard…
> >>
> >>
> > Ca roule. Je suis en train de retravailler media.php, ça vous dérange
si
> > je remplace le behavior adminMediaURL (qui ne supporte pas les plugins
> > multiples) en adminMediaURLParams (et du coup les plugins modifient des
> > paramètres d'URL, et pas une URL) ?
> >
> >
> Bon, j'ai commité ma bafouille. Pour éviter d'avoir à se poser la
question
> je mets un "&" ou un "&" dans adminurl, j'ai
ajouté une méthode
> redirect à adminurl, qui simplifie du coup les redirections, et qui fait
ce
> choix toute seule.
>
> Et dcPage::getPF est bien plus court à taper :) .J'ai adapté en vitesse
le
> _admin de sysInfo, que je viens de découvrir, et ce plugin me plaît/sert
> bien, je vais le garder sous le coude \o/
>
> J'ai essayé de faire le plus de tests possibles, apparemment rien n'est
> cassé, le #2022 devrait être réparé. La seule entorse dont je ne suis pas
> fier, c'est pour les url paramétriques où je dois décoder les %25 en % à
la
> main. Je vous laisse m'insulter si quelque chose ne marche plus.
>
> --
> Bruno
> --
> Dev mailing list - Dev(a)list.dotclear.org -
> http://ml.dotclear.org/listinfo/dev
>
--
Franck
--
Dev mailing list - Dev(a)list.dotclear.org -
http://ml.dotclear.org/listinfo/dev
8:04 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Le 17 décembre 2014 07:54, Nicolas <nikrou77(a)gmail.com> a écrit :
Bonjour,
cela n'engage que moi mais je trouve que ça commence à faire beaucoup pour
une version de correction. Après je dois avouer que j'ai lu un peu en
diagonal et je ne me mesure pas forcément tous les impacts.
Au moins ça nous permettra d'avoir des testeurs...
8:06 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Le 17 décembre 2014 08:04, Bruno <dsls(a)morefnu.org> a écrit :
Le 17 décembre 2014 07:54, Nicolas <nikrou77(a)gmail.com> a écrit :
> Bonjour,
> cela n'engage que moi mais je trouve que ça commence
à faire beaucoup
pour
> une version de correction. Après je dois avouer que j'ai lu un peu en
> diagonal et je ne me mesure pas forcément tous les impacts.
Au moins ça nous permettra
d'avoir des testeurs...
Tout à fait. Après peut-être que ce n'est rien mais ne mesurant pas trop
l'impact je ne saurai pas trop me prononcer sur les potentiels problèmes.
8:08 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Tout à fait. Après peut-être que ce n'est rien mais ne mesurant pas trop
l'impact je ne saurai pas trop me prononcer sur les potentiels problèmes.
Je pense toutefois qu'on tient le bon bout, les dernier commits évitent de
se poser trop de questions sur les encodages d'url, les redirections, ...
tout en rendant le code plus concis.
--
Bruno
8:10 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Le 17 décembre 2014 08:08, Bruno <dsls(a)morefnu.org> a écrit :
>
>
> Tout à fait. Après peut-être que ce n'est rien mais ne mesurant pas trop
> l'impact je ne saurai pas trop me prononcer sur les potentiels problèmes.
>
>
Je pense toutefois qu'on tient le bon bout, les dernier commits évitent de
se poser trop de questions sur les encodages d'url, les redirections, ...
tout en rendant le code plus concis.
Ok je te fais confiance. Et pour plus de sécurité, je passerai ton numéro
de téléphone pour les vacances à mon utilisateur fou qui dès qu'il voit le
bouton "Mise à jour" clique dessus ! :-)
--
Bruno
--
Dev mailing list - Dev(a)list.dotclear.org -
http://ml.dotclear.org/listinfo/dev
8:34 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Concernant adminurl, on ira pas plus loin pour la 2.7.1, on est en effet à
la limite de la correction. Le "hack" pour le %25s est moche, t'as raison
Bruno. Y'a pas moyen de centraliser ça dans adminurl (via une méthode
utilitaire), plutôt que de propager partout le strreplace ?
Maintenant, une fois tout ça rodé, il faudra écrire quelque part un petit
tuto pour l'adaptation d'un plugin (par exemple).
Le 17 décembre 2014 08:10, Nicolas <nikrou77(a)gmail.com> a écrit :
Le 17 décembre 2014 08:08, Bruno <dsls(a)morefnu.org> a écrit :
>
> >
> >
> > Tout à fait. Après peut-être que ce n'est rien mais ne mesurant pas
trop
> > l'impact je ne saurai pas trop me prononcer sur les potentiels
problèmes.
> >
> >
> Je pense toutefois qu'on tient le bon bout, les dernier commits évitent
de
> se poser trop de questions sur les encodages d'url, les redirections, ...
> tout en rendant le code plus concis.
>
Ok je te fais confiance. Et pour plus de sécurité, je passerai ton numéro
de téléphone pour les vacances à mon utilisateur fou qui dès qu'il voit le
bouton "Mise à jour" clique dessus ! :-)
>
> --
> Bruno
> --
> Dev mailing list - Dev(a)list.dotclear.org -
> http://ml.dotclear.org/listinfo/dev
>
--
Dev mailing list - Dev(a)list.dotclear.org -
http://ml.dotclear.org/listinfo/dev
--
Franck
8:07 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
Le 17 décembre 2014 08:04, Bruno <dsls(a)morefnu.org> a écrit :
Le 17 décembre 2014 07:54, Nicolas <nikrou77(a)gmail.com> a écrit :
>
> Bonjour,
>
> cela n'engage que moi mais je trouve que ça commence à faire beaucoup pour
> une version de correction. Après je dois avouer que j'ai lu un peu en
> diagonal et je ne me mesure pas forcément tous les impacts.
>
>
Au moins ça nous permettra d'avoir des testeurs...
Et d'un autre coté, on avait le choix entre faire les adaptions d'hier, ou
faire un rollback sur toutes le adminurl, ce qui peu ou prou touche la même
quantité de code...
8:48 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
2014-12-17 8:07 GMT+01:00 Bruno <dsls(a)morefnu.org>:
Le 17 décembre 2014 08:04, Bruno <dsls(a)morefnu.org> a écrit :
>
>
>
> Le 17 décembre 2014 07:54, Nicolas <nikrou77(a)gmail.com> a écrit :
>>
>> Bonjour,
>>
>> cela n'engage que moi mais je trouve que ça commence à faire beaucoup
pour
>> une version de correction. Après je dois avouer que j'ai lu un peu en
>> diagonal et je ne me mesure pas forcément tous les impacts.
>>
>>
> Au moins ça nous permettra d'avoir des testeurs...
>
Et d'un autre coté, on avait le choix entre faire les adaptions d'hier, ou
faire un rollback sur toutes le adminurl, ce qui peu ou prou touche la même
quantité de code...
Mes 2 cents: avec un rollback t'es à peu près sûr de corriger le souci
(avec évidemment le risque posé par les conflits) alors qu'en faisant un
fix tu risques de devoir en faire d'autres après. En tant que dev, on a un
peu trop tendance à penser qu'avec "le" fix ça ira :)
Après, vous connaissez mieux que moi le risque pour les 2 possibilités :)
9:05 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
2014-12-17 15:48 GMT+08:00 Julien Wajsberg <felash(a)gmail.com>:
2014-12-17 8:07 GMT+01:00 Bruno <dsls(a)morefnu.org>:
>
> Le 17 décembre 2014 08:04, Bruno <dsls(a)morefnu.org> a écrit :
> >
> >
> >
> > Le 17 décembre 2014 07:54, Nicolas <nikrou77(a)gmail.com> a écrit :
> >>
> >> Bonjour,
> >>
> >> cela n'engage que moi mais je trouve que ça commence à faire beaucoup
> pour
> >> une version de correction. Après je dois avouer que j'ai lu un peu en
> >> diagonal et je ne me mesure pas forcément tous les impacts.
> >>
> >>
> > Au moins ça nous permettra d'avoir des testeurs...
> >
>
> Et d'un autre coté, on avait le choix entre faire les adaptions d'hier,
ou
> faire un rollback sur toutes le adminurl, ce qui peu ou prou touche la
même
> quantité de code...
>
>
Mes 2 cents: avec un rollback t'es à peu près sûr de corriger le souci
(avec évidemment le risque posé par les conflits) alors qu'en faisant un
fix tu risques de devoir en faire d'autres après. En tant que dev, on a un
peu trop tendance à penser qu'avec "le" fix ça ira :)
Sages paroles, Julien :)
Après, vous connaissez mieux que moi le risque pour les 2 possibilités :)
--
Dev mailing list - Dev(a)list.dotclear.org -
http://ml.dotclear.org/listinfo/dev
--
Pierre Equoy
10:15 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
En fait ça n'a pas beaucoup d'importance, par contre si tu vires la méthode
decode, tu casses mon plugin sysInfo !
Le 16 décembre 2014 10:09, Franck Paul <carnet.franck.paul(a)gmail.com> a
écrit :
C'est si pressé que ça ?
Le 16 décembre 2014 10:08, Bruno <dsls(a)morefnu.org> a écrit :
> Le 16 décembre 2014 10:05, Nicolas <nikrou77(a)gmail.com> a écrit :
> >
> > Je n'y vois pas d'objection. Enfin sur la branche 2.8, hein, pas sur la
> 2.7
> >
> >
> L'impact est quand même minime, il faut vraiment attendre la 2.8 ?
> --
> Dev mailing list - Dev(a)list.dotclear.org -
> http://ml.dotclear.org/listinfo/dev
>
--
Franck
--
Franck
10:09 a.m.
New subject: [Dotclear Tracker] [2847]: 2.7 - Use adminurl->decode instead of ->get, URL encodin...
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 <nikrou77(a)gmail.com> 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 <dsls(a)morefnu.org> a écrit :
>
> 2014-12-15 15:42 GMT+01:00 <hg(a)dotclear.org>:
> >
> > 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 - Dev(a)list.dotclear.org -
> http://ml.dotclear.org/listinfo/dev
>
--
Dev mailing list - Dev(a)list.dotclear.org -
http://ml.dotclear.org/listinfo/dev
--
Franck
3437
days inactive
3438
days old
34 comments
6 participants
participants (6)
-
brol -
Bruno -
Franck Paul -
Julien Wajsberg -
Nicolas -
Pierre Equoy