I'm not sure what's causing it, I'm using a bit of the code from when you guys helped me last which seemed work work, but now it's not.

http://phpfi.com/271062

For the elseif section to see if any variables are empty, would that be best to put into one if condition and use the || operator?

Also, I have another question. Is all that in the __construct() function necessary? It doesn't seem like it, but I'm not sure how else to pass the variables off into member variables.