jecasc - 9:59 pm on Dec 27, 2011 (gmt 0)

Should it not be:

else {
echo "login fail – please click here to <a href=\"login.htm\">login</a>";
}

or

else echo "login fail – please click here to <a href=\"login.htm\">login</a>";