[Phpmyadmin-devel] refactoring and superglobals

Thilina Buddika Abeyrathna thilinaabeyrathna at gmail.com
Mon May 7 15:44:19 CEST 2012


On Mon, May 7, 2012 at 5:34 PM, Marc Delisle <marc at infomarc.info> wrote:

> Hi,
>
> We previously had a script called grab_globals.lib.php, which created
> global variables from a few superglobals arrays.
>
> When this script was removed a few weeks ago, we added at the beginning
> of other scripts some code, for example in tbl_change.php:
>
> if (isset($_REQUEST['where_clause'])) {
>    $where_clause = $_REQUEST['where_clause'];
> }
>
> In the upcoming refactoring efforts, we should avoid this, because,
> looking at the code further in the script, it's difficult to see the
> origin of $where_clause. So, we should only refer to
> $_REQUEST['where_clause'] (except in the situations where the global is
> modified afterwards).
>
> Another benefit is that this variable would not need to be passed as a
> parameter to functions.


Hi Marc,
Thank you very much for pointing out.

-- 
Regards.

Thilina Buddika Abeyrathna,
Department of Computer Engineering,
Faculty Of Engineering,
University of Peradeniya,
Sri Lanka.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.phpmyadmin.net/pipermail/developers/attachments/20120507/7adf900e/attachment.html>


More information about the Developers mailing list