[skymonkier]

Alright, so I'm just working on a simple little update script using the new PDO (of which, I am a first-time user) and when testing my script, I get an error saying Fatal error: Call to a member function prepare() on a non-object, and the script is pasted below.

Le Code:

<?PHP
	$name = $_POST['name'];
	$passcode = $_POST['passcode']; //Don't comment on this, I'll use it after this error is fixed...
	$news = $_POST['news'];
					
	try {
		//Connect to the database
		include 'dbconnect.php';
		
		$updatenews = $pdo->prepare('UPDATE news SET name = :name, news = :news WHERE id = :id');
		$updatenews->execute(array(':id' => 1, ':name' => $name, ':news' => $news));
	} catch(PDOException $e) {
		echo "<p>Sorry, there was an error;   " . $e->getMessage() . ".</p>"; //Print out the exception
	}
?>

[Atli]

It's telling you that your $pdo variable is not an object, and thus calling prepare() on it will not work. Presumably the PDO connection code in your 'dbconnect.php' file is failing somehow. We'd need to see that to know more.

[skymonkier]

Well, here's 'dbconnect.php', and I'm thinking of removing the try-catch block from it.

Le Code:

<?PHP
$username = user;
$password = password;

try {
		$conn = new PDO('mysql: host=localhost; dbname=db', $username, $password);
		$conn->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
	} catch(PDOException $e) {
		echo "<p>Sorry, there was an error;   " . $e->getMessage() . ".</p>";
	}
?>

This post has been edited by skymonkier: 10 September 2012 - 12:22 PM

[JackOfAllTrades]

So, is the object named $pdo, or is it named $conn?

[skymonkier]

...



Thanks!

[Dormilich]

skymonkier, on 10 September 2012 - 09:20 PM, said:

<?PHP
$username = user;
$password = password;

try {
		$conn = new PDO('mysql: host=localhost; dbname=db', $username, $password);
		$conn->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
	} catch(PDOException $e) {
		echo "<p>Sorry, there was an error;   " . $e->getMessage() . ".</p>";
	}
?>

two notes:
- unless user & password are actual constants (in which case they should be written upper-case), strings are delimited by quotes. if you indeed use constants, put them into the PDO constructor directly.
- remove that try...catch block. exceptions are meant to be caught where you can handle the problem (which is the try...catch block of the other script (where you can skip all the other DB code))

[skymonkier]

I thought about removing the try-catch block from it, but I wasn't sure. I'll also fix user/pass. Thanks!