function problem

[Martoni]

Can anyone see what's wrong with this function? Running the query directly in MySQL produces correct results (ie a value for free and total_entitlement).

function Count_licenses($sID)
{
global $adb,$bgcl,$bgcd, $free;
$query = "SELECT * FROM software_licenses WHERE (sID = '$sID')";
$sth = $adb->prepare($query);
if($sth)
{
$res = $sth->execute();
$numRows = $sth->rows();
$record = $sth->fetchrow_hash();
$free = $record[free];
if($free = "No" || $free = ""){
$total_entitlement=0;
for ($i=0;$i<$numRows;$i++) {
$record = $sth->fetchrow_hash();
$total_entitlement+= $record[entitlement];
}
}else {
$total_entitlement = "Not applicable";
}
$sth->finish();
} else
{
PRINT "Could not prepare query: ".$sth->errstr."<BR>\n";
}
return $total_entitlement;
}
/M.
--
Martin Skjöldebrand
Family site: http://www.skjoldebrand.org
"Art" site: http://martoni.deviantart.com
Public key available at: http://wwwkeys.pgp.net

[Jeffrey Silverman]

On Mon, 22 Mar 2004 20:43:38 +0000, Martoni wrote:

Can anyone see what's wrong with this function? Running the query
directly in MySQL produces correct results (ie a value for free and
total_entitlement).

Um, what is wrong with it? (I'm not gonna wade through it without some
more info)

Error messages?

--
Jeffrey D. Silverman | jeffrey AT jhu DOT edu
Website | http://www.wse.jhu.edu/newtnotes/

[Martoni]

It never returns a value of total_entitlement other than 0.

/M.
--
Martin Skjöldebrand
Family site: http://www.skjoldebrand.org
"Art" site: http://martoni.deviantart.com
Public key available at: http://wwwkeys.pgp.net

[Pedro Graca]

Martoni wrote:

function Count_licenses($sID)
{
global $adb,$bgcl,$bgcd, $free;
$query = "SELECT * FROM software_licenses WHERE (sID = '$sID')";
$sth = $adb->prepare($query);
if($sth)
{
$res = $sth->execute();
$numRows = $sth->rows();
$record = $sth->fetchrow_hash();
$free = $record[free];
if($free = "No" || $free = ""){ Uh Oh!
assignment, not comparison ... what do you expect the contents of $free
to be after this statement?
$total_entitlement=0;
for ($i=0;$i<$numRows;$i++) {
$record = $sth->fetchrow_hash();

Uh Oh!
read a second row and disregard the first?
It doesn't seem a good idea, especially when the second row is empty and
you don't test it.
--
USENET would be a better place if everybody read: : mail address :
http://www.catb.org/~esr/faqs/smart-questions.html : is valid for :
http://www.netmeister.org/news/learn2quote2.html : "text/plain" :
http://www.expita.com/nomime.html : to 10K bytes :

[Jeffrey Silverman]

On Tue, 23 Mar 2004 19:43:42 +0000, Martoni wrote:

It never returns a value of total_entitlement other than 0.

/M.

it looks like Pedro Garcia has provided clues for you.

== is NOT the same as =
later...
--
Jeffrey D. Silverman | jeffrey AT jhu DOT edu
Website | http://www.wse.jhu.edu/newtnotes/

[Martoni]

Ok, thanks.
I've been looking at for too long. Needed another pair of eyes....

/M.
--
Martin Skjöldebrand
Family site: http://www.skjoldebrand.org
"Art" site: http://martoni.deviantart.com
Public key available at: http://wwwkeys.pgp.net