[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