<br><br><div class="gmail_quote">On Mon, May 7, 2012 at 5:34 PM, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi,<br>
<br>
We previously had a script called grab_globals.lib.php, which created<br>
global variables from a few superglobals arrays.<br>
<br>
When this script was removed a few weeks ago, we added at the beginning<br>
of other scripts some code, for example in tbl_change.php:<br>
<br>
if (isset($_REQUEST['where_clause'])) {<br>
    $where_clause = $_REQUEST['where_clause'];<br>
}<br>
<br>
In the upcoming refactoring efforts, we should avoid this, because,<br>
looking at the code further in the script, it's difficult to see the<br>
origin of $where_clause. So, we should only refer to<br>
$_REQUEST['where_clause'] (except in the situations where the global is<br>
modified afterwards).<br>
<br>
Another benefit is that this variable would not need to be passed as a<br>
parameter to functions.</blockquote><div><br></div><div>Hi Marc, </div><div>Thank you very much for pointing out.</div></div><div><br></div>-- <br><font face="georgia, serif">Regards.</font><br><br><font face="'trebuchet ms', sans-serif" size="4">Thilina Buddika Abeyrathna,</font><br>

<font face="georgia, serif">Department of Computer Engineering,</font><br><font face="georgia, serif">Faculty Of Engineering,</font><br><font face="georgia, serif">University of Peradeniya,</font><div><font face="georgia, serif">Sri Lanka.<br>

<br></font></div><br>