[Phpmyadmin-devel] further refactoring of sql.php

Supun Nakandala supun.nakandala at gmail.com
Mon Jun 24 17:43:51 CEST 2013


On Mon, Jun 24, 2013 at 8:41 PM, Marc Delisle <marc at infomarc.info> wrote:

> Hi Supun,
>
> If you are looking for further ideas about refactoring, I have suggestions.
>
> First, have a look at tbl_select.php, both in the QA_3_5 and the QA_4_0
> branches, and see what kind of high-level calls have been left in this
> refactored script. Everything else went to functions. In this case, Atul
> also OOPed the code, which is only expected from you if you have time,
> as it's not part of your proposal.
>
> All chunks of code having a common goal (see for example the "Bookmark
> add" section) should be extracted to sql.lib.php.
>
> I was pretty sure you were aware of this, as your proposal plans to
> spend 6 weeks x 40 hours on refactoring sql.php.
>
> Second, as discussed previously, your proposal says "The sql.php is a
> very large script and is also one of the central components in
> phpMyAdmin. As it is large it is hard to read and maintain. It needs
> refactoring and better integration with other scripts calling it."
>
> So, all scripts that are including sql.php (for example db_qbe.php)
> should probably be including a library file then call a high-level
> function. Hint: sql.php's main job is to deal with $sql_query.
>
> --
> Marc Delisle
> http://infomarc.info
>
>
> ------------------------------------------------------------------------------
> This SF.net email is sponsored by Windows:
>
> Build for Windows Store.
>
> http://p.sf.net/sfu/windows-dev2dev
> _______________________________________________
> Phpmyadmin-devel mailing list
> Phpmyadmin-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel
>

Thanks Marc. I'll consider those ideas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.phpmyadmin.net/pipermail/developers/attachments/20130624/eaa12aa4/attachment.html>


More information about the Developers mailing list