From be39a06a110df826ced9fa6bd03e79ccd17f695c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Michael=20T=C3=A4nzer?= Date: Tue, 3 Aug 2010 21:19:05 +0200 Subject: [PATCH] Fix Bug #830: Negative assurance points MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit If the user had more than 100 points adding more points through an automated assurance resulted in negative points Signed-off-by: Michael Tänzer --- manager/application/controllers/ManageAccountController.php | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/manager/application/controllers/ManageAccountController.php b/manager/application/controllers/ManageAccountController.php index 33eed81..e900d72 100644 --- a/manager/application/controllers/ManageAccountController.php +++ b/manager/application/controllers/ManageAccountController.php @@ -91,6 +91,11 @@ class ManageAccountController extends Zend_Controller_Action $assurance['points'] = $assurance['awarded']; } + // Only assign positive amounts + if ($assurance['points'] < 0){ + $assurance['points'] = 0; + } + $this->db->insert('notary', $assurance); $user['points'] += $assurance['points'];