Hey,
I'm having a bit of trouble with this script.
Can anybody help me out?

PHP Code:

<?php
/////////////////
// Connection strings
/////////////////
$host = "localhost";
$user = "fake";
$password = "fake";
$database = "fake";


 mysql_connect($host, $user, $password);
  mysql_select_db($database);


function checkstock($card, $string2)
{
 $query = "SELECT cardstock FROM cards WHERE cardid = $card";
 $result = mysql_query($query);
  if ($result > 0) {
  echo "$result";
  $string2 = true;
  } elseif ($result == 0) {
  echo "<font color=\"red\"><b>Out of Stock</b></font>";
  $string2 = false;
  } 
}
$get = $_GET['x'];
if ($get = "test") {
$fake = "";
checkstock("testcard", $fake);
echo "$fake";
} else {}

?>

Thanks in advance

Chris

What kind of errors are you getting?

Well. it's just showing out of stock (witch is what it does when theres a value of 0 in the database)
It's ether in the database or the script.
Heres a demo: www.dragongames.ca/dbconnect.php?x=test
it should show 1 but it doesn't

Chris

Your using $result as if it is an actual number. mysql_query is just returing a result set, you have to use other functions to get that data. To get just the number, you can try:

PHP Code:

$query = "SELECT cardstock FROM cards WHERE cardid = $card";
$result = mysql_query($query); 
$result = mysql_fetch_row($result); // fetch the row
$result = $result[0]; // which will retrun a 1 element array, so just grab it 


I think that should work. I've got a DB class that does most of my work for me so I'm a little rusty

// edit, you might have to use more the one variable for query and result Try this if the first doesn't work:

PHP Code:

$query = "SELECT cardstock FROM cards WHERE cardid = $card";
$result = mysql_query($query); 
$cardstock = mysql_fetch_row($result); // fetch the row
$cardstock = $cardstock[0]; // which will retrun a 1 element array, so just grab it 


Originally Posted by Chroder

Your using $result as if it is an actual number. mysql_query is just returing a result set, you have to use other functions to get that data. To get just the number, you can try:

PHP Code:

$query = "SELECT cardstock FROM cards WHERE cardid = $card";
$result = mysql_query($query); 
$result = mysql_fetch_row($result); // fetch the row
$result = $result[0]; // which will retrun a 1 element array, so just grab it 


I think that should work. I've got a DB class that does most of my work for me so I'm a little rusty

// edit, you might have to use more the one variable for query and result Try this if the first doesn't work:

PHP Code:

$query = "SELECT cardstock FROM cards WHERE cardid = $card";
$result = mysql_query($query); 
$cardstock = mysql_fetch_row($result); // fetch the row
$cardstock = $cardstock[0]; // which will retrun a 1 element array, so just grab it 


It still doesn't work.
the error I'm getting is this: Warning: mysql_fetch_row(): supplied argument is not a valid MySQL result resource in /home/draconum/public_html/dbconnect.php on line 19


Chris

It may make no difference, but try this:

PHP Code:

$query = "SELECT cardstock FROM cards WHERE cardid = '".$card"'"; 


See if there's an error:

PHP Code:

$result = mysql_query($query) or die('Error: ' . mysql_error()); 


Originally Posted by david

It may make no difference, but try this:

PHP Code:

$query = "SELECT cardstock FROM cards WHERE cardid = '".$card"'"; 


I think that did it.
Thanks alot both of you.

Chris

So cardid was a string all along?

yep and the script works like a charm.

Chris

If you get used to doing this
$query = "SELECT cardstock FROM cards WHERE cardid = '{$card}'";
it is good practice because then you can use arrays in your query!