From 2c89a978946bf8104bcd0df705203ffe0fbc6ed2 Mon Sep 17 00:00:00 2001
From: abacab <wagnerch@glycerine.alpha>
Date: Tue, 30 Dec 2008 10:31:18 -0500
Subject: [PATCH] voip_prepaid::lowBalance fix, take 3.

---
 modules/voip_prepaid/voip_prepaid.inc.php | 35 ++++++++++++-----------
 1 file changed, 18 insertions(+), 17 deletions(-)

diff --git a/modules/voip_prepaid/voip_prepaid.inc.php b/modules/voip_prepaid/voip_prepaid.inc.php
index e260957a..32537894 100644
--- a/modules/voip_prepaid/voip_prepaid.inc.php
+++ b/modules/voip_prepaid/voip_prepaid.inc.php
@@ -83,26 +83,27 @@ class voip_prepaid
 	{
 		include_once(PATH_MODULES.'email_template/email_template.inc.php');
 
-		if (defined $this->lowBalance)
+		// do not run task if lowBalance is not set
+		if ($this->lowBalance == false)
+			return;
+
+		// delete expired pins?
+		// $delrs = & $db->Execute(sqlDelete($db,"voip_prepaid"," date_expire <> 0 and date_expire is not null and date_expire > ".time()));
+
+		// get low balances and notify
+		$db=&DB();
+		$rs = & $db->Execute($sql = sqlSelect($db,"voip_prepaid","*", "balance <= $this->lowBalance and (bulk is null or bulk=0) and (date_email is null or date_email = 0) "));
+		if($rs && $rs->RecordCount() > 0)
 		{
-			// delete expired pins?
-			// $delrs = & $db->Execute(sqlDelete($db,"voip_prepaid"," date_expire <> 0 and date_expire is not null and date_expire > ".time()));
-
-			// get low balances and notify
-			$db=&DB();
-			$rs = & $db->Execute($sql = sqlSelect($db,"voip_prepaid","*", "balance <= $this->lowBalance and (bulk is null or bulk=0) and (date_email is null or date_email = 0) "));
-			if($rs && $rs->RecordCount() > 0)
+			while(!$rs->EOF)
 			{
-				while(!$rs->EOF)
-				{
-					# send the user the details
-					$email = new email_template;
-					$email->send('voip_balance_prepaid', $rs->fields['account_id'], $rs->fields['id'], '', number_format($rs->fields['balance'],4));
+				# send the user the details
+				$email = new email_template;
+				$email->send('voip_balance_prepaid', $rs->fields['account_id'], $rs->fields['id'], '', number_format($rs->fields['balance'],4));
 
-					# update the record
-					$db->Execute( sqlUpdate($db, "voip_prepaid", array('date_email'=>time()),"id={$rs->fields['id']}"));
-					$rs->MoveNext();
-				}
+				# update the record
+				$db->Execute( sqlUpdate($db, "voip_prepaid", array('date_email'=>time()),"id={$rs->fields['id']}"));
+				$rs->MoveNext();
 			}
 		}
 	}