[SUCS Devel] site r705 - trunk/lib
Tim Clark
eclipse at sucs.org
Tue May 5 10:06:11 BST 2015
There is no advantage to this change, it makes it less clear and relying
on type juggling to always do as intended is risky and not recommended.
Tim
On 01/05/2015 23:08, andy at sucs.org wrote:
> Author: andy
> Date: 2015-05-01 23:08:04 +0100 (Fri, 01 May 2015)
> New Revision: 705
>
> Modified:
> trunk/lib/banana-admin.php
> Log:
> Removed unnecessary comparison operator
>
> Modified: trunk/lib/banana-admin.php
> ===================================================================
> --- trunk/lib/banana-admin.php 2015-05-01 21:51:44 UTC (rev 704)
> +++ trunk/lib/banana-admin.php 2015-05-01 22:08:04 UTC (rev 705)
> @@ -28,7 +28,7 @@
>
> $why = $_POST['why'];
>
> - if ( (abs($number) <= $maxbanana ) && $number <> 0 ){
> + if ( (abs($number) <= $maxbanana ) && $number ){
>
> $DB->Query("INSERT INTO awards (username, score, whn, who, why) VALUES (?,?,NOW(),?,?)", array($who, $number, $session->username, $why));
> header("Location: " . $_SERVER['PHP_SELF']);
>
>
> _______________________________________________
> Devel mailing list
> Devel at lists.sucs.org
> http://lists.sucs.org/mailman/listinfo/devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sucs.org/pipermail/devel/attachments/20150505/233da10a/attachment.html>
More information about the Devel
mailing list