<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2014-04-19 13:13 GMT+02:00 Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Le 2014-04-19 05:17, Hugues Peccatte a écrit :<br>
<div class="">> 2014-04-18 14:18 GMT+02:00 Marc Delisle <<a href="mailto:marc@infomarc.info">marc@infomarc.info</a><br>
</div>> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>>>:<br>
<div class="">><br>
> Le 2014-04-18 08:00, Hugues Peccatte a écrit :<br>
> > 2014-04-18 12:41 GMT+02:00 Marc Delisle <<a href="mailto:marc@infomarc.info">marc@infomarc.info</a><br>
> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>><br>
</div><div class="">> > <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>>>>:<br>
> ><br>
> > Le 2014-04-18 05:34, Hugues Peccatte a écrit :<br>
> > > 2014-04-17 23:38 GMT+02:00 Marc Delisle <<a href="mailto:marc@infomarc.info">marc@infomarc.info</a><br>
> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>><br>
> > <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>>><br>
</div><div><div class="h5">> > > <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>><br>
> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>>>>>:<br>
> > ><br>
> > > Le 2014-04-17 13:43, Hugues Peccatte a écrit :<br>
> > > > 2014-04-17 14:07 GMT+02:00 Marc Delisle<br>
> <<a href="mailto:marc@infomarc.info">marc@infomarc.info</a> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>><br>
> > <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>>><br>
> > > <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>><br>
> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>>>><br>
> > > > <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>><br>
> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>>><br>
> > <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>><br>
> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>>>>>>:<br>
> > > ><br>
> > > > Hugues Peccatte a écrit :<br>
> > > > > 2014-04-17 11:59 GMT+02:00 Hugues Peccatte<br>
> > > > <<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>><br>
> > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>>><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>><br>
> > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>>>><br>
> > > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>><br>
> > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>>><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>><br>
> > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>>>>>>:<br>
> > > > ><br>
> > > > >> 2014-04-16 22:16 GMT+02:00 Hugues Peccatte<br>
> > > > <<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>><br>
> > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>>><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>><br>
> > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>>>><br>
> > > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>><br>
> > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>>><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>><br>
> > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>>>>>>:<br>
> > > > >><br>
> > > > >> 2014-04-14 13:20 GMT+02:00 Michal Čihař<br>
> > <<a href="mailto:michal@cihar.com">michal@cihar.com</a> <mailto:<a href="mailto:michal@cihar.com">michal@cihar.com</a>><br>
> <mailto:<a href="mailto:michal@cihar.com">michal@cihar.com</a> <mailto:<a href="mailto:michal@cihar.com">michal@cihar.com</a>>><br>
> > > <mailto:<a href="mailto:michal@cihar.com">michal@cihar.com</a> <mailto:<a href="mailto:michal@cihar.com">michal@cihar.com</a>><br>
> <mailto:<a href="mailto:michal@cihar.com">michal@cihar.com</a> <mailto:<a href="mailto:michal@cihar.com">michal@cihar.com</a>>>><br>
> > > > <mailto:<a href="mailto:michal@cihar.com">michal@cihar.com</a> <mailto:<a href="mailto:michal@cihar.com">michal@cihar.com</a>><br>
> <mailto:<a href="mailto:michal@cihar.com">michal@cihar.com</a> <mailto:<a href="mailto:michal@cihar.com">michal@cihar.com</a>>><br>
> > <mailto:<a href="mailto:michal@cihar.com">michal@cihar.com</a> <mailto:<a href="mailto:michal@cihar.com">michal@cihar.com</a>><br>
> <mailto:<a href="mailto:michal@cihar.com">michal@cihar.com</a> <mailto:<a href="mailto:michal@cihar.com">michal@cihar.com</a>>>>>>:<br>
> > > > >>> Hi<br>
> > > > >>>> Dne Mon, 14 Apr 2014 09:53:52 +0200<br>
> > > > >>>> Hugues Peccatte <<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>><br>
> > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>>><br>
> > > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>><br>
> > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>>>><br>
> > > > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>><br>
> > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>>><br>
> > > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>><br>
> > <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a><br>
> <mailto:<a href="mailto:hugues.peccatte@gmail.com">hugues.peccatte@gmail.com</a>>>>>> napsal(a):<br>
> > > > >>>><br>
> > > > >>>>> I was looking for something more implicit, so we<br>
> > > wouldn't have to<br>
> > > > >>>> change<br>
> > > > >>>>> the calls to $.get. But that seems to be<br>
> > difficult, so your<br>
> > > > solution<br>
> > > > >>>> might<br>
> > > > >>>>> be the best.<br>
> > > > >>>> Well we can also override jQuery.get to do<br>
> what we want<br>
> > > (basically<br>
> > > > >>>> just wrapper around original code to do<br>
> compose the URL<br>
> > > > correctly. That<br>
> > > > >>>> might be cleaner approach.<br>
> > > > >>>><br>
> > > > >>> Thanks for this very interesting feedback !<br>
> > > > >>> I found this :<br>
> > > > >>><br>
> > > ><br>
> > ><br>
> ><br>
> <a href="http://www.bennadel.com/blog/1624-ask-ben-overriding-core-jquery-methods.htm" target="_blank">http://www.bennadel.com/blog/1624-ask-ben-overriding-core-jquery-methods.htm</a><br>
> > > > >>><br>
> > > > >>> I'll try to override the $.get method by this way.<br>
> > > > >>><br>
> > > > >> It's done… I overwrote $.get… But there is many<br>
> > Jquery methods<br>
> > > > which use<br>
> > > > >> the "&" as an URL parameter separator… I would have<br>
> > love that<br>
> > > > they have a<br>
> > > > >> $.getArgSeparator, like pMA, so we could overwrite<br>
> > this method<br>
> > > > only! :)<br>
> > > > >> We can overwrite all methods which use "&". But<br>
> when<br>
> > we'll want<br>
> > > > to change<br>
> > > > >> the Jquery version, it will be a mess.<br>
> > > > >><br>
> > > > >> Shall we continue to search a way to use another<br>
> > separator<br>
> > > than "&"?<br>
> > > > >> Do you want we to push my work on my repository, so<br>
> > you can see<br>
> > > > what I've<br>
> > > > >> done?<br>
> > > > >><br>
> > > > >><br>
> > > > > I didn't wait for an answer. If you want to check:<br>
> > > > > <a href="https://github.com/Tithugues/phpmyadmin/tree/b3733" target="_blank">https://github.com/Tithugues/phpmyadmin/tree/b3733</a><br>
> > > > ><br>
> > > > > Hugues.<br>
> > > ><br>
> > > > Hi Hugues,<br>
> > > > I have started to review the code; so far it looks<br>
> fine.<br>
> > > ><br>
> > > > In js/messages.php, maybe it would be more<br>
> consistent to<br>
> > call<br>
> > > > PMA_URL_getArgSeparator() ?<br>
> > > ><br>
> > > ><br>
> > > > You're right. It's done.<br>
> > > ><br>
> > > > Hugues.<br>
> > ><br>
> > > Ok. Feel free to merge to master; I'll release -beta1<br>
> afterwards.<br>
> > ><br>
> > ><br>
> > > I really can't… There is still issues with Jquery methods which<br>
> > use the<br>
> > > "&".<br>
> > > That why I asked:<br>
> > > Shall we continue to search a way to use another separator<br>
> than "&"?<br>
> > > Override all methods? I don't think this is a good thing…<br>
> > ><br>
> > > Thanks for your feedback and code review.<br>
> > ><br>
> > > Hugues.<br>
> ><br>
> > Well, it depends: have you tested the behavior of the other jQuery<br>
> > methods that use the "&", when PHP is configured to use ";" as the<br>
> > argument separator?<br>
> ><br>
> ><br>
> > Yes, I still have issues with "&" which is used to concat the URL GET<br>
> > parameters.<br>
> > The "&" is used in 4 Jquery methods. All methods could be overwriten…<br>
> > But what about changing Jquery version ?<br>
> > We'll have to update the code. That's not infeasable, but we have to<br>
> > think remember it. And also to check the other methods of Jquery, to<br>
> > check than another one is not using "&".<br>
> ><br>
> > Hugues.<br>
><br>
> It's not a good idea to change jQuery itself. I'll release -beta1<br>
> without this patch, let's continue the discussion afterwards.<br>
><br>
><br>
> Maybe, the best thing is to… send a pull request to Jquery to manage a<br>
> different arg separator. :)<br>
> If nobody has any idea, I'll may try this…<br>
<br>
</div></div>Yes, but first ask on stackoverflow: there are already 470 000 questions<br>
tagged "jQuery". Or <a href="http://forum.jquery.com" target="_blank">http://forum.jquery.com</a>.</blockquote><div><br></div><div>You were right, there is a ticket for this feature (that I didn't find before…) : <a href="http://bugs.jquery.com/ticket/6924">http://bugs.jquery.com/ticket/6924</a> </div>
<div>It won't be fix…</div><div><br></div><div>I opened a topic on the forum: <a href="https://forum.jquery.com/topic/manage-url-argument-separator">https://forum.jquery.com/topic/manage-url-argument-separator</a></div>
<div>Their solution is to override the methods… We can choose this solution, but so, we'll have to remember to update the methods when we'll want to change the Jquery version.</div><div><br></div><div>Hugues.</div>
</div></div></div>