ADOdb functions

janet287

I am working on converting a function from using mysql calls to using ADOdb calls. I am not sure that I've done it right, particularly the loop using FetchRow. Can someone take a look and see if it makes any sense?

function sync_credit_type()
{
   global $CFG;


$userid=$_SESSION['USER']->id; 
$temp=$_SESSION['USER']->idnumber;

	$enroldb = $this->enrol_connect();
		if (!$enroldb) {        
		notify("enrol/database cannot connect to server");
    return false;

	if($userid == ''|$userid == 0)
{
	$query_in=$enroldb>Execute("SELECT ".$CFG->enrol_remotecoursefield.",".$CFG->enrol_remoteuserfield.",".$CFG->credit_type." FROM ".$CFG->enrol_dbtable); 
}
else
{
	$query_in=$enroldb>Execute("SELECT ".$CFG->enrol_remotecoursefield.",".$CFG->enrol_remoteuserfield.",".$CFG->credit_type." FROM ".$CFG->enrol_dbtable." where ".$CFG->enrol_remoteuserfield." = '".$temp."'"); 
}


while($res=FetchRow($query_in))
{

$Credit_type = $res[$CFG->credit_type];
$coursefield = $res[$CFG->enrol_remotecoursefield];
$userfield = $res[$CFG->enrol_remoteuserfield];


$cou_id=$enroldb("select id from mc where idnumber='".$coursefield."'"); 
$coid=FetchRow($cou_id);
$courseid=$coid[0];

	if ($courseid > 0) { 

$result = $enroldb("select distinct id from mu where idnumber = '".$userfield."'");
$userId_row = FetchRow($result);
$user_Id = $userId_row[0];

	$check_data = $enroldb("SELECT * FROM `mgd` WHERE UserID='".$user_Id."' AND CourseID='".$courseid."'");
			if($result->RecordCount() >0)
			{
				if($check_data->RecordCount() >0)
				{
					$sql1=$enroldb("UPDATE `mgd` SET 
								Credit_Type ='".$Credit_type."' ,
								Timemodified = '".time()."'  WHERE UserID = '".$user_Id."' AND CourseID='".$courseid."'");
				}else
				{

					$sql2=$enroldb("INSERT INTO `mgd` (
								`UserID` ,
								`CourseID` ,
								`Credit_Type` ,
								`Timemodified` ,
								`modifierid`
								)
								VALUES (
								'".$user_Id."', '".$courseid."', '".$Credit_type."', '".time()."', '1')") or die(mysql_error());
				}
			}


	else {
	echo "course mismatch";
	}
}	
}	

						}
						}

dagon

I am not sure that I've done it right

does it work? if yes, then its right if not then its not.

janet287

No, it doesn't work. If it worked, I wouldn't be posting for someone to review it.

dagon

well you dident say it wasn't working, and we have a whole section for posting working code for review. Now, how about be specific on what is not working?

How To Ask Questions The Smart Way

janet287

Sure, I can try. I think the connection to the database is working, because it isn't giving me an errors, which is it setup to do if it doesn't work.

So, I think the problem is the FetchRow loop stating with

while($res=FetchRow($query_in))

All I did was replace what was

while($res=mssql_fetch_array($query_in))

with the FetchRow words. I suspect I need to do more to rewrite this loop to work as an ADOdb compliant call instead of a mssql_fetch_array.

Can you help here? I am in way over my head in terms of understanding how to code a loop. Thanks!