<div dir="ltr">On Tue, Jun 18, 2013 at 5:43 PM, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<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">Supun Nakandala a écrit :<br>
<div><div>> Hi devs,<br>
>  I am refactoring sql.php and I realized in line 389 of file the check for<br>
> a call to a stored procedure is done by using regexp. I would like to know<br>
> is it ok to keep it like that or is there any better approach to handle<br>
> that.<br>
><br>
> Regards Supun.<br>
<br>
</div></div>Hi Supun,<br>
the better approach is to move this test to the SQL analyzer, like you<br>
did a few days ago for other cases.<br>
<br></blockquote><div>Hi Marc,</div><div>I checked PMA_SQP_analyze() method with a test query calling a stored procedure. But it didn't returned any information. Can you guide me how to implement the required behavior in PMA_SQP_analyze() method such that it can identify a call to a stored procedure.</div>
<div> </div><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">
Using preg_match() can lead to mistake (for example, a SQL comment<br>
located in the query and containing the word "call").<br>
<br>
<br>
--<br>
Marc Delisle<br>
<a href="http://infomarc.info" target="_blank">http://infomarc.info</a><br>
<br>
------------------------------------------------------------------------------<br>
This SF.net email is sponsored by Windows:<br>
<br>
Build for Windows Store.<br>
<br>
<a href="http://p.sf.net/sfu/windows-dev2dev" target="_blank">http://p.sf.net/sfu/windows-dev2dev</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><br clear="all"><div><br></div>-- <br>Thank you<br>Supun Nakandala<br>Dept. Computer Science and Engineering<br>University of Moratuwa<br>
</div></div>