Checking a value

SFDonovan

A pretty simple issue. I need to check and varify the value of multiple records in a query. I may have 2 or 3 records or as many as 18 in a query. I need to check the value of a field called status. Either 0 or 1.

0 = not complete.
1 = complete.

My first attempt:

I pass the value of Course_ID to the page, then run the query.

$Course_ID = $_GET['Course_ID'];
$getcourseinfo = $db->sql_query("SELECT status FROM ".$prefix."_tl_session WHERE Course_ID = '$Course_ID'");
while($row = $db->sql_fetchrow($getcourseinfo)) {
$status = $row[status];
if ($status == 0) {
echo"<tr><td>Status Check confirms all TL sessions are not complete.</td></tr>";
}else {
echo"<tr><td>Status Check confirms all TL sessions have been completed.</td></tr>";
}
}

This is just some simple error checking to make sure all sessions are complete and status is "checked" before I continue.

In my example I had three records in the query with the 2nd one having a value of 0, so I returned:

Status Check confirms all TL sessions have been completed.Status Check confirms all TL sessions are not complete.Status Check confirms all TL sessions have been completed.

Not really what I wanted.

Maybe I should not use a while loop. I need to verify the status of each record.

Weedpacket

You need to check all the records, so you'll be using a loop for that. But you don't want the output to be part of the loop.

$Course_ID = $_GET['Course_ID'];
$getcourseinfo = $db->sql_query("SELECT status FROM ".$prefix."_tl_session WHERE Course_ID = '$Course_ID'");
$all_complete = true;
while($row = $db->sql_fetchrow($getcourseinfo)) {
$status = $row['status']; // you missed the quotes here
	if ($status == 0) {
		$all_complete = false;
	} 
}

if($all_complete) {
	echo"<tr><td>Status Check confirms not all TL sessions are complete.</td></tr>";
} else {
	echo"<tr><td>Status Check confirms all TL sessions have been completed.</td></tr>";
}

For bonus points you can add a "break;" statement after the line that sets $all_complete to false. If any of the records have $status==0 then not all sessions are complete - so once you find one you've got your answer.

For even more points you do all the checking in the database.

$Course_ID = $_GET['Course_ID'];
$query = $db->sql_query("SELECT count(*) AS incomplete from {$prefix}_tl_session WHERE Course_ID='$Course_ID' AND status=0");
$result = $db->sql_fetchrow($query);
$count = $result['incomplete'];
if($count>0) {
	echo"<tr><td>Status Check confirms not all TL sessions are complete.</td></tr>";
} else {
	echo"<tr><td>Status Check confirms all TL sessions have been completed.</td></tr>";
}

And for a pass mark you should sanitise the value of your $Course_ID variable.