<div dir="ltr">On Wed, Jun 19, 2013 at 4:32 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Supun Nakandala a écrit :<br>
<div>> On Tue, Jun 18, 2013 at 5:43 PM, Marc Delisle <<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>> wrote:<br>
><br>
>> Supun Nakandala a écrit :<br>
>>> Hi devs,<br>
>>>  I am refactoring sql.php and I realized in line 389 of file the check<br>
>> for<br>
>>> a call to a stored procedure is done by using regexp. I would like to<br>
>> 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>
>> 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>
>> Hi Marc,<br>
> I checked PMA_SQP_analyze() method with a test query calling a stored<br>
> procedure. But it didn't returned any information. Can you guide me how to<br>
> implement the required behavior in PMA_SQP_analyze() method such that it<br>
> can identify a call to a stored procedure.<br>
<br>
</div>Hi Supun,<br>
first I must ask whether you passed your query to PMA_SQP_parse() and<br>
used the output to feed PMA_SQP_analyze().<br></blockquote><div><br></div><div style>Hi Marc,</div><div style>Yes I did that way. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div><div><br>
><br>
><br>
>> 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>
</div></div></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>