<br><br><div class="gmail_quote">On Tue, May 29, 2012 at 10:04 PM, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">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 2012-05-29 10:46, Atul Pratap Singh a écrit :<br>
<div class="im">> Hi Marc,<br>
> While looking at "tbl_zoom_select.php":-<br>
><br>
> Line 191-201<br>
> if (isset($inputs) && ($inputs[0] != 'pma_null' || $inputs[1] !=<br>
> 'pma_null')) {<br>
>     $flag = 2;<br>
>     for ($i = 0 ; $i < 4 ; $i++) {<br>
>         if ($inputs[$i] != 'pma_null') {<br>
>             $key = array_search($inputs[$i], $fields_list);<br>
>             $tbl_fields_type[$i] = $fields_type[$key];<br>
>             $tbl_fields_collation[$i] = $fields_collation[$key];<br>
>             $tbl_fields_null[$i] = $fields_null[$key];<br>
>         }<br>
>     }<br>
> }<br>
><br>
> As the condition would be true in cases when post param $inputs is set and<br>
> has index 0 or 1 set to some column name. It would mostly happen if a half<br>
> or full filled search form is submitted, but there seems to be no further<br>
> use of $tbl_fields_collation & $tbl_fields_null. Though $tbl_fields_type<br>
> gets used afterwards with indexes >= 4.<br>
<br>
</div>Atul,<br>
Indeed, $tbl_fields_collation and $tbl_fields_null could be removed.<br>
<div class="im"><br>
<br>
> I think as a result after the zoom plot is displayed, clicking at "show<br>
> search criteria" doesn't show any previously filled criteria.<br>
<br>
</div>Well, I see the chosen two column names, but indeed there seems to be a<br>
bug because at least the operator and value should be displayed.<br></blockquote><div>Ok Marc, I'll remove the bug and also any unused variables. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div class="im"><br>
> I also didn't understand the use of $flag variable.<br>
<br>
</div>This must have been some debugging info, you can safely remove it.<br>
<br>
><br>
> Thanks<br>
><br>
><br>
><br>
> ------------------------------------------------------------------------------<br>
> Live Security Virtual Conference<br>
> Exclusive live event will cover all the ways today's security and<br>
> threat landscape has changed and how IT managers can respond. Discussions<br>
> will include endpoint security, mobile security and the latest in malware<br>
> threats. <a href="http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/" target="_blank">http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/</a><br>
><br>
><br>
><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>
<span class="HOEnZb"><font color="#888888"><br>
<br>
--<br>
Marc Delisle<br>
<a href="http://infomarc.info" target="_blank">http://infomarc.info</a><br>
<br>
</font></span><br>------------------------------------------------------------------------------<br>
Live Security Virtual Conference<br>
Exclusive live event will cover all the ways today's security and<br>
threat landscape has changed and how IT managers can respond. Discussions<br>
will include endpoint security, mobile security and the latest in malware<br>
threats. <a href="http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/" target="_blank">http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/</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"><div><br></div>-- <br>Atul Pratap Singh<div>Senior Undergraduate Student</div><div>Institute Of Technology,<br>Banaras Hindu University, Varanasi,<br>India.</div><br>