Make your own PHP adoptables system

[Turquoise]

Did you remember to comment out the "header('Content-Type ..." line as I instructed? If you leave that line in, your browser will try to interpret the error message as an image instead of showing it as text.

The error I got printed (after saving the image and opening it in notepad):

Table 'a3459659_122195.adoptables' doesn't exist

What tables have you created in your database?

Oh, I missed that.

Now I have changed it.

I did create a table in my database, and its number was a3459659_122195. It is called adoptables as well, although it starts with a capital "A". Would that make a difference, and where would I change that?

Thank You.

[Nick]

Yep, the names are case-sensitive. Either rename your table in your database, or change all the places in the code where you are SELECTing FROM adoptables or INSERTing INTO adoptables to use the capital letter.

[Turquoise]

Okay, so I made a whole new table, with everything the same, except for making "adptables" lowercase.

It must be getting somewhere, because the pets are finally showing up, but they are showing as white boxes with a red "x" in them:



I noticed something, though. When I get the code after I adopt, this is the code for the Forum:

Code: Select all

[url=http://laughingwolflane.comze.com/][img]http://laughingwolflane.comze.com//simple.php?idp=0&.jpg[/img][/url]
                Owner's name: Sarah
                Pet's name: Test
                Adopted on: 2010-08-25

It says that idp=0&.jpg

Usually it's a long number after that, and all of my file names for pets on that site are .png, not .jpg . Does this have anything to do with it?

[Nick]

Again, comment out the "header('Content-type:" line, and go to the image's address in your browser directly, so you can actually read the error message. Unless you can read the error message, you aren't going to get anywhere.

I would also suggest adding error checking in adopt.php:

Code: Select all

$result1 = mysql_query($sql);

Becomes:

Code: Select all

$result1 = mysql_query($sql) or die(mysql_error());

Add that sort of error checking to every place you call mysql_query, it'll then be able to tell you exactly when an error occurs.

[Turquoise]

Alright, first of all I wanted to say sorry for making this a lot more difficult than it needs to be.

So I changed the two things, and when I adopt, the message that comes up is:

"Adoptables
Duplicate entry '0' for key 1".

[Nick]

You need to change your primary key column on your adoptables table to add the "auto increment" attribute - that makes it so that the ID numbers of your pets are generated automatically in increasing order. Are you using phpMyAdmin to make your database changes? Go to the structure of your adoptables table, and click the pencil icon next to your primary column. In the page that appears, tick the "AUTO_INCREMENT" box and click save.

[Turquoise]

You need to change your primary key column on your adoptables table to add the "auto increment" attribute - that makes it so that the ID numbers of your pets are generated automatically in increasing order. Are you using phpMyAdmin to make your database changes? Go to the structure of your adoptables table, and click the pencil icon next to your primary column. In the page that appears, tick the "AUTO_INCREMENT" box and click save.

Alright, I added the "auto increment", and now I get this message when I view the pet:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

[Nick]

Your adopt.php reads a value from a variable which does not exist, "$adopt". Change this line:

Code: Select all

$sql = "INSERT INTO Adoptables (date,petID,varID,owner) VALUE (NOW(),'$adopt','$v_id','$adopter')";

To:

Code: Select all

$sql = "INSERT INTO Adoptables (date,petID,varID,owner) VALUE (NOW(),".$adoptable['id'].",'$v_id','$adopter')";

[Turquoise]

Okay, I changed that, but this error message still comes up:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

[Nick]

You must be viewing a pet that was already adopted. Adopt a new one. All the species work for me now, you just need to un-comment the header('Content-Type... line and the image should appear too.