On Sat, Mar 19, 2011 at 11:19 PM, Andreas Lundquist <span dir="ltr"><<a href="mailto:andreas.lunkan@gmail.com">andreas.lunkan@gmail.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Hi Dieter,<div><br></div><div>Thanks. Don't know if this is the right place but I think I found a bug in inline editing of rows. The save button don't work like it should and when I looked through</div><div>the code I found a strange looking line. sql_query += ' `' + field_name + "`='" + this_field_params[field_name].replace(/'/g, "''") + "' , ";</div>

<div>This is line 842 in js/sql.js.</div><div><br></div><div>I don't know how the end on that line should look but it's definitely not right with the quotations. Should I add a bug to the tracker about this and try to fix it?<br clear="all">

<br>Kind regards,<br>Andreas<div><div></div><div class="h5"><br></div></div></div></blockquote><div><br>Hi,<br><br>What's wrong with the quotes, it loops over and collects all the fields, then trims the sql query for the additional "," that is appended for the last column.<br>
<br>--<br>Rohit Sharma<br>CSE<br><a href="http://iiit.net/">IIIT-Hyderabad</a><br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div>
<div><div class="h5">
<br><br><div class="gmail_quote">2011/3/19 Dieter Adriaenssens <span dir="ltr"><<a href="mailto:dieter.adriaenssens@gmail.com" target="_blank">dieter.adriaenssens@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

2011/3/19 Lunkan <<a href="mailto:andreas.lunkan@gmail.com" target="_blank">andreas.lunkan@gmail.com</a>>:<br>
<div><div></div><div>> Hi pma developing community!<br>
> First of all, a small presentation, my name is Andreas Lundquist and is a<br>
> pre-graduate student from Sweden. I have been using phpmyadmin a lot when<br>
> doing things like small websites and so.<br>
> It's always been my first choice when doing web development.<br>
> I'm interested in the idea of improving the browse-mode like said on the<br>
> idea page<br>
> at <a href="http://wiki.phpmyadmin.net/pma/GSoC_2011_Ideas_List#Browse-mode_improvements" target="_blank">http://wiki.phpmyadmin.net/pma/GSoC_2011_Ideas_List#Browse-mode_improvements</a>.<br>
> There were quite many users who have sent in requests about this topic and<br>
> many of them I think are possible to implement.<br>
> Sadly I have never contributed to any open-source projects before so this<br>
> mailinglist and stuff like git is quite new for me. However I have been<br>
> using svn on some projects so I'm not to blown away of the idea.<br>
> I have managed to get the source code to a local git repository and will<br>
> create a repository on the site recommended by the git guide in the wiki.<br>
> I will look through the source code and hopefully understand the structure<br>
> and try to fix some bugs or requests from the bug tracker so I get the idea<br>
> of sending in code to be reviewed.<br>
> Just one question though just to be clear, am I supposed to write a line in<br>
> the bug tracker before I start try to implement it even if I'm not sure I<br>
> could do it?<br>
> That's all for now, just thought I should present myself to the community so<br>
> you know who I am.<br>
><br>
> Best regards,<br>
> Andreas Lundquist<br>
><br>
<br>
</div></div>Hi Andreas,<br>
<br>
Thanks for the elaborate introduction. Welcome on the mailing list.<br>
<br>
To answer your question about the bug/feature tracker : it is always a<br>
good idea to let others know you are working on a patch, to avoid that<br>
two or more people are working on the same patch.<br>
<br>
But I guess before you start working on a patch, you reviewed the bug,<br>
tried to reproduce it and looked for the problem in order to get an<br>
idea on how to solve it, so by that time you should know if you will<br>
be able to fix it. ;)<br>
<br>
In the case you stop working on it, or don't plan to finish it, add<br>
another comment that you're not working on it anymore, so that others<br>
can work on it.<br>
<br>
Good luck with GSoC (and submitting your proposal).<br>
<br>
Kind regards,<br>
<br>
Dieter<br>
<br>
------------------------------------------------------------------------------<br>
Colocation vs. Managed Hosting<br>
A question and answer guide to determining the best fit<br>
for your organization - today and in the future.<br>
<a href="http://p.sf.net/sfu/internap-sfd2d" target="_blank">http://p.sf.net/sfu/internap-sfd2d</a><br>
_______________________________________________<br>
Phpmyadmin-devel mailing list<br>
<a href="mailto:Phpmyadmin-devel@lists.sourceforge.net" target="_blank">Phpmyadmin-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel" target="_blank">https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel</a><br>
</blockquote></div><br></div></div></div>
<br>------------------------------------------------------------------------------<br>
Colocation vs. Managed Hosting<br>
A question and answer guide to determining the best fit<br>
for your organization - today and in the future.<br>
<a href="http://p.sf.net/sfu/internap-sfd2d" target="_blank">http://p.sf.net/sfu/internap-sfd2d</a><br>_______________________________________________<br>
Phpmyadmin-devel mailing list<br>
<a href="mailto:Phpmyadmin-devel@lists.sourceforge.net">Phpmyadmin-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel" target="_blank">https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel</a><br>
<br></blockquote></div><br><br clear="all"><br>-- <br><font style="font-family: verdana,sans-serif;" size="2"><i><b><span style="color: rgb(51, 51, 51);">Baby, Why not smile?  </span></b></i></font><br><br>