<div dir="ltr">On Mon, Jul 15, 2013 at 8:28 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">Le 2013-07-15 10:44, Supun Nakandala a écrit :<br>
<div><div class="h5">> Hi devs,<br>
> I am facing a difficulty in setting up the goals for the week 5. I think<br>
> I have considerably refactored the sql.php using extract method. I would<br>
> like to hear from you whether I should consider further refactoring<br>
> using extract method or any other suggestions to work on during the next<br>
> week.<br>
><br>
> Regards Supun<br>
<br>
</div></div>Hi Supun,<br>
<br>
Regarding code extraction, I think we are fine. However, here is what I<br>
told you on June 24 on this list, regarding sql.php:<br>
<br>
-------<br>
as discussed previously, your proposal says "The sql.php is a<br>
very large script and is also one of the central components in<br>
phpMyAdmin. As it is large it is hard to read and maintain. It needs<br>
refactoring and better integration with other scripts calling it."<br>
<br>
So, all scripts that are including sql.php (for example db_qbe.php)<br>
should probably be including a library file then call a high-level<br>
function. Hint: sql.php's main job is to deal with $sql_query.<br>
-------<br>
<br>
In summary, you should transform all places where we include sql.php, to<br>
a function call that deals with $sql_query.<br></blockquote><div><br></div><div>Hi Marc,</div><div>I can see certain high level functions that sql.php does such as executing the query, displaying results, and sending values for set, enum and relational fields. But apart from these functions sql,php does some other work such variable setting (db, table), setting error url etc. If I am going to change the paces where sql.php is including with function calls to sql,lib.php how can I do the above mentioned tasks. Have I got this correct or am I missing something.</div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
--<br>
Marc Delisle<br>
<a href="http://infomarc.info" target="_blank">http://infomarc.info</a><br>
<br>
------------------------------------------------------------------------------<br>
See everything from the browser to the database with AppDynamics<br>
Get end-to-end visibility with application monitoring from AppDynamics<br>
Isolate bottlenecks and diagnose root cause in seconds.<br>
Start your free trial of AppDynamics Pro today!<br>
<a href="http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk" target="_blank">http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk</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>
</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>