<br><br><div class="gmail_quote">On Sat, Mar 26, 2011 at 7:14 PM, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Le 2011-03-26 09:21, rohit sharma a écrit :<br>
<div><div></div><div class="h5">> Hi,<br>
><br>
> On Sat, Mar 26, 2011 at 5:32 PM, Marc Delisle <<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>> wrote:<br>
><br>
>> Le 2011-03-20 10:12, rohit sharma a écrit :<br>
>><br>
>>>> In case of an error, it would be better to display the query<br>
>> permanently,<br>
>>> so to  bring uniformity (either success or failure), the inline_edit<br>
>> query<br>
>>> should be displayed permanently like in case of normal query with<br>
>>> $cfg['ShowSQL'] set to true.<br>
>><br>
>><br>
>> In case of success, query is now displayed; please try it.<br>
>><br>
>><br>
> Consider this scenario, I have a table with no index defined, and two<br>
> columns, when I click "inline_edit" for any row, the row gets updated, and<br>
> the message is shown properly. Now when I click "inline_edit" again for the<br>
> same row, but changing the other column, the message shown is "0 Rows<br>
> affected", although the row being displayed gets updated without any error.<br>
> However a look at the query reveals that the query was in-fact incorrect, it<br>
> was using the old value for the column that  had been previously edited.<br>
> Thus the row displayed is incorrect, because it never got updated.<br>
> I am sending some screenshots to depict the same.<br>
><br>
> [0]<br>
> <a href="http://web.iiit.ac.in/%7Erohit.sharmaug08/phpmyadmin/inline_edit/inline-edit_1.jpeg" target="_blank">http://web.iiit.ac.in/~rohit.sharmaug08/phpmyadmin/inline_edit/inline-edit_1.jpeg</a><br>
> [ initial_setup ]<br>
> [1]<br>
> <a href="http://web.iiit.ac.in/%7Erohit.sharmaug08/phpmyadmin/inline_edit/inline-edit_2.jpeg" target="_blank">http://web.iiit.ac.in/~rohit.sharmaug08/phpmyadmin/inline_edit/inline-edit_2.jpeg</a><br>
> [ updating row 2, col 2 ]<br>
> [2]<br>
> <a href="http://web.iiit.ac.in/%7Erohit.sharmaug08/phpmyadmin/inline_edit/inline-edit_3.jpeg" target="_blank">http://web.iiit.ac.in/~rohit.sharmaug08/phpmyadmin/inline_edit/inline-edit_3.jpeg</a><br>
> [ updating row 2, col 1 ]<br>
><br>
> For [2] it outputs, "0 Rows updated", although it should update that row,<br>
> because nothing is incorrect with the query. A look at sql.js reveals, that<br>
> the column value in the new query does not get updated.<br>
> Although while iterating over the elements "this_field_params[field_name] "<br>
> contains the correct value, however the problem is the "where_clause" as it<br>
> does not get updated. So the issue boils down to simply the " <input class="<br>
> where_clause"..>  being incorrect. Which means as soon as we find a<br>
> successful sql query, we must update the "where_clause".<br></div></div></blockquote><div> </div><div>Hi,<br> <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 class="h5">
> Should I file this as a bug ?<br>
<br>
</div></div>Please do.<br>
<div><div></div><div class="h5"><br></div></div></blockquote><div> </div><div>Bug reported, updated diff uploaded, please test it.<br><br><a href="https://sourceforge.net/tracker/?func=detail&aid=3249406&group_id=23067&atid=377408">https://sourceforge.net/tracker/?func=detail&aid=3249406&group_id=23067&atid=377408</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 class="h5">
<br>
--<br>
Marc Delisle<br>
<a href="http://infomarc.info" target="_blank">http://infomarc.info</a><br>
<br>
------------------------------------------------------------------------------<br>
Enable your software for Intel(R) Active Management Technology to meet the<br>
growing manageability and security demands of your customers. Businesses<br>
are taking advantage of Intel(R) vPro (TM) technology - will your software<br>
be a part of the solution? Download the Intel(R) Manageability Checker<br>
today! <a href="http://p.sf.net/sfu/intel-dev2devmar" target="_blank">http://p.sf.net/sfu/intel-dev2devmar</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>
</div></div></blockquote></div><br><br>--<br>Regards<br><br>Rohit Sharma<br clear="all"><br>