Further tweaks and improvements to SUSignup Admin component

components/susignup-admin.php

@@ -206,13 +206,23 @@ if (isset($session->groups[$permission])) {
 		$apiResult = curl_exec($apiReq);
 		$sumembers = su_response_decode($apiResult);
 		$matches = array();
+		$others=0;
+		$paidup=0;
 		foreach ($sumembers as $sumem) {
 			$sucsmem = get_sucs_record($sumem['uni_card_number']);
 			if ($sucsmem && $sucsmem['paid'] != paidUntil(time()) && $sucsmem['type']==1) {
 				$matches[]=array($sumem['firstName']." ".$sumem['lastName'], $sucsmem['realname'], $sumem['uni_card_number'], $sucsmem['username'], $sucsmem['paid']);
+			} else if ($sucsmem && $sucsmem['paid'] == paidUntil(time())) {
+				$others++;
+				$paidup++;
+			} else {
+				$others++;
 			}
 		}
 		$smarty->assign("matches", $matches);
+		$smarty->assign("others", $others);
+		$smarty->assign("paidup", $paidup);
+		$smarty->assign("pending", $others - $paidup);
 	} else if ($mode == 'renewals2') {
 		$failures = array();
 		$successes = array();
@@ -255,7 +265,7 @@ function get_sucs_record($sid) {
 
 	$query = "SELECT * FROM members WHERE sid=?;";
 	$res  = $sucsDB->Execute($query, array($sid));
-	if (!$res || $res->RecordCount()>1) {
+	if (!$res || $res->RecordCount()<>1) {
 		return FALSE;
 	}
         return $res->FetchRow();	

htdocs/css/susignup-admin.css

@@ -26,18 +26,18 @@ div.susignup-admin-menu input#rnsubmit {
 
 #susignup-renewals {
 	width: 90%;
-	border: 2px ridge;
+	border: 1px solid black;
 	border-collapse: collapse;
 }
 
 #susignup-renewals td {
-	border: 2px ridge;
+	border: 1px solid black;
 	padding-left: 4px;
 	padding-right: 2px;
 }
 
 #susignup-renewals th {
-	border: 2px ridge;
+	border: 1px solid black;
 	text-align: center;
 	padding-left: 2px;
 	padding-right: 2px;

templates/susignup-admin.tpl

@@ -23,6 +23,10 @@
 </table>
 <input type='hidden' name='mode' id='mode' value='renewals2' />
 </form>
+{if $others > 0}
+<p>There were also <strong>{$others}</strong> other records found.<br />
+Of these, {$paidup} are current members and {$pending} are pending signups.</p>
+{/if}
  {elseif $mode == 'renewals2'}
 <div class='centre'>
 <div class='cbb' style='text-align: left'>